behlendorf requested changes on this pull request.


> @@ -54,6 +54,11 @@ 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;

In order in avoid a potential future conflict let's reserve one additional MMP 
field for a sequence number.  This is something we've talked about using in the 
past since the existing timestamp granularity is measured in seconds.  I'll add 
this reserved field to ZoL MMP patch.

```c
uint64_t ub_mmp_seq;
```

-- 
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-49838376
------------------------------------------
openzfs-developer
Archives: 
https://openzfs.topicbox.com/groups/developer/discussions/Td128c23c12b31635-M3890767e5c3301cfd4db317a
Powered by Topicbox: https://topicbox.com

Reply via email to