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