Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 8 Aug 2017 11:19:57 +0000 (UTC)
From:      Andriy Gapon <avg@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-vendor@freebsd.org
Subject:   svn commit: r322240 - vendor-sys/illumos/dist/uts/common/fs/zfs/sys
Message-ID:  <201708081119.v78BJvjL043589@repo.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: avg
Date: Tue Aug  8 11:19:56 2017
New Revision: 322240
URL: https://svnweb.freebsd.org/changeset/base/322240

Log:
  8491 uberblock on-disk padding to reserve space for smoothly merging zpool checkpoint & MMP in ZFS
  
  illumos/illumos-gate@79c2b812ee2010ebf20fdd92dc5f06b59000a94c
  https://github.com/illumos/illumos-gate/commit/79c2b812ee2010ebf20fdd92dc5f06b59000a94c
  
  https://www.illumos.org/issues/8491
    The zpool checkpoint feature in DxOS added a new field in the uberblock.
    The Multi-Modifier Protection Pull Request from ZoL adds two new fields in the
    uberblock (Reference: https://github.com/zfsonlinux/zfs/pull/6279).
    As these two changes come from two different sources and once upstreamed and
    deployed will introduce an incompatibility with each other we want
    to upstream a change that will reserve the padding for both of them so
    integration goes smoothly and everyone gets both features.
  
  Reviewed by: Matthew Ahrens <mahrens@delphix.com>
  Reviewed by: Brian Behlendorf <behlendorf1@llnl.gov>
  Reviewed by: Olaf Faaland <faaland1@llnl.gov>
  Approved by: Gordon Ross <gwr@nexenta.com>
  Author: Serapheim Dimitropoulos <serapheim@delphix.com>

Modified:
  vendor-sys/illumos/dist/uts/common/fs/zfs/sys/uberblock_impl.h

Modified: vendor-sys/illumos/dist/uts/common/fs/zfs/sys/uberblock_impl.h
==============================================================================
--- vendor-sys/illumos/dist/uts/common/fs/zfs/sys/uberblock_impl.h	Tue Aug  8 11:19:14 2017	(r322239)
+++ vendor-sys/illumos/dist/uts/common/fs/zfs/sys/uberblock_impl.h	Tue Aug  8 11:19:56 2017	(r322240)
@@ -20,6 +20,7 @@
  */
 /*
  * Copyright (c) 2005, 2010, Oracle and/or its affiliates. All rights reserved.
+ * Copyright (c) 2017 by Delphix. All rights reserved.
  */
 
 #ifndef _SYS_UBERBLOCK_IMPL_H
@@ -54,6 +55,12 @@ struct uberblock {
 
 	/* highest SPA_VERSION supported by software that wrote this txg */
 	uint64_t	ub_software_version;
+
+	/* These fields are reserved for features that are under development: */
+	uint64_t	ub_mmp_magic;
+	uint64_t	ub_mmp_delay;
+	uint64_t	ub_mmp_seq;
+	uint64_t	ub_checkpoint_txg;
 };
 
 #ifdef	__cplusplus



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201708081119.v78BJvjL043589>