[developer] Re: [openzfs/openzfs] 8491 uberblock on-disk padding to reserve space for smoothly merging zpool checkpoint & MMP in ZFS (#425)

2017-07-21 Thread Prakash Surya
Closed #425 via d84fa5f171a900608bf459213fa0fb818ca8a60d. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/openzfs/openzfs/pull/425#event-1174720698 --

[developer] Re: [openzfs/openzfs] 8491 uberblock on-disk padding to reserve space for smoothly merging zpool checkpoint & MMP in ZFS (#425)

2017-07-20 Thread Matthew Ahrens
ahrens commented on this pull request. > @@ -54,6 +55,12 @@ struct uberblock { /* highest SPA_VERSION supported by software that wrote this txg */ uint64_tub_software_version; + + /* These fields are reserved for features that are under development: */ +

[developer] Re: [openzfs/openzfs] 8491 uberblock on-disk padding to reserve space for smoothly merging zpool checkpoint & MMP in ZFS (#425)

2017-07-20 Thread Gordon Ross
gwr commented on this pull request. > @@ -54,6 +55,12 @@ struct uberblock { /* highest SPA_VERSION supported by software that wrote this txg */ uint64_tub_software_version; + + /* These fields are reserved for features that are under development: */ +

[developer] Re: [openzfs/openzfs] 8491 uberblock on-disk padding to reserve space for smoothly merging zpool checkpoint & MMP in ZFS (#425)

2017-07-13 Thread Olaf Faaland
ofaaland approved this pull request. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/openzfs/openzfs/pull/425#pullrequestreview-49853449 -- openzfs-developer

[developer] Re: [openzfs/openzfs] 8491 uberblock on-disk padding to reserve space for smoothly merging zpool checkpoint & MMP in ZFS (#425)

2017-07-13 Thread Serapheim Dimitropoulos
New field and copyright notice added. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/openzfs/openzfs/pull/425#issuecomment-315147670 -- openzfs-developer Archives:

[developer] Re: [openzfs/openzfs] 8491 uberblock on-disk padding to reserve space for smoothly merging zpool checkpoint & MMP in ZFS (#425)

2017-07-13 Thread Olaf Faaland
ofaaland commented on this pull request. > @@ -54,6 +54,11 @@ struct uberblock { /* highest SPA_VERSION supported by software that wrote this txg */ uint64_tub_software_version; + + /* These fields are reserved for features that are under development: */ +

[developer] Re: [openzfs/openzfs] 8491 uberblock on-disk padding to reserve space for smoothly merging zpool checkpoint & MMP in ZFS (#425)

2017-07-13 Thread Olaf Faaland
ofaaland commented on this pull request. > @@ -54,6 +54,11 @@ struct uberblock { /* highest SPA_VERSION supported by software that wrote this txg */ uint64_tub_software_version; + + /* These fields are reserved for features that are under development: */ +

[developer] Re: [openzfs/openzfs] 8491 uberblock on-disk padding to reserve space for smoothly merging zpool checkpoint & MMP in ZFS (#425)

2017-07-13 Thread Brian Behlendorf
behlendorf requested changes on this pull request. > @@ -54,6 +54,11 @@ struct uberblock { /* highest SPA_VERSION supported by software that wrote this txg */ uint64_tub_software_version; + + /* These fields are reserved for features that are under development:

[developer] Re: [openzfs/openzfs] 8491 uberblock on-disk padding to reserve space for smoothly merging zpool checkpoint & MMP in ZFS (#425)

2017-07-13 Thread Matthew Ahrens
ahrens commented on this pull request. > @@ -54,6 +54,9 @@ struct uberblock { /* highest SPA_VERSION supported by software that wrote this txg */ uint64_tub_software_version; + + /* reserved */ + uint64_tpadding[3]; This is OK, but I think it would

[developer] Re: [openzfs/openzfs] 8491 uberblock on-disk padding to reserve space for smoothly merging zpool checkpoint & MMP in ZFS (#425)

2017-07-13 Thread Serapheim Dimitropoulos
@ofaaland could you take a look too please. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/openzfs/openzfs/pull/425#issuecomment-315098175 -- openzfs-developer