Skip to content
Permalink
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 <rlaager@wiktel.com>
  • Loading branch information...
rlaager committed Apr 15, 2019
1 parent 83472fa commit 2189f837cad164204386ba10851277b2f60643fb
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 SPA_VERSION_5000
#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 2189f83

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