Skip to content
Browse files

Remove zfs.h comments about GRUB

Nobody is going to be bumping SPA_VERSION again, as OpenZFS has moved on
to feature flags.  Also, there is no requirement to keep GRUB
up-to-date, nor has that been happening.

The ZPL_VERSION could be bumped, but that would likely be handled in a
similar way, by adding filesystem feature flags.  In any event, we do
not need this comment, and we certainly don't need a reference to the
GRUB 0.97 source code in a Solaris tree.

Signed-off-by: Richard Laager <>
  • Loading branch information...
rlaager committed Apr 15, 2019
1 parent 87a42be commit 833310c7d6e920f25870134bc6b10e38ab7300e7
Showing with 0 additions and 9 deletions.
  1. +0 −9 include/sys/fs/zfs.h
@@ -488,12 +488,6 @@ typedef enum zfs_key_location {
#define SPA_VERSION_28 28ULL
#define SPA_VERSION_5000 5000ULL

* When bumping up SPA_VERSION, make sure GRUB ZFS understands the on-disk
* format change. Go to usr/src/grub/grub-0.97/stage2/{zfs-include/, fsys_zfs*},
* and do the appropriate changes. Also bump the version number in
* usr/src/grub/capability.
#define SPA_VERSION_STRING "5000"

@@ -557,9 +551,6 @@ typedef enum zfs_key_location {
* ZPL version - rev'd whenever an incompatible on-disk format change
* occurs. This is independent of SPA/DMU/ZAP versioning. You must
* also update the version_table[] and help message in zfs_prop.c.
* When changing, be sure to teach GRUB how to read the new format!
* See usr/src/grub/grub-0.97/stage2/{zfs-include/,fsys_zfs*}
#define ZPL_VERSION_1 1ULL
#define ZPL_VERSION_2 2ULL

0 comments on commit 833310c

Please sign in to comment.
You can’t perform that action at this time.