Date: Thu, 10 Nov 2022 09:32:06 +0900 From: Tomoaki AOKI <junchoon@dec.sakura.ne.jp> To: "Patrick M. Hausen" <pmh@hausen.com> Cc: Alexander Leidinger <Alexander@leidinger.net>, Warner Losh <imp@bsdimp.com>, Mark Millard <marklmi@yahoo.com>, tsoome@freebsd.org, Li-Wen Hsu <lwhsu@freebsd.org>, current@freebsd.org Subject: Re: changes to the zfs boot (was: Re: git: 72a1cb05cd23 - main - rc(8): Add a zpoolupgrade rc.d script) Message-ID: <20221110093206.661bbe915fd2e37338df51be@dec.sakura.ne.jp> In-Reply-To: <E6722C9D-BB70-4515-8944-5FE17B0542AF@hausen.com> References: <202211070339.2A73dJlO027991@gitrepo.freebsd.org> <20221108105053.Horde.eqgFiBJe2ngGAj6BkXcv5-Z@webmail.leidinger.net> <20221109134610.Horde.JB7ibQTWprHbmIUfhg7JY7f@webmail.leidinger.net> <CANCZdfptmipq%2BsS0AQ1%2B7EmLT-7YdKv8s%2BnCV7ON1Qy6-C6N9A@mail.gmail.com> <460205F9-5D59-4033-813B-C34E01BFD6C4@hausen.com> <CANCZdfoeDvxSqqEjpnfK=FCY0K1aYEqTw80SKLAZ-5d2qrW5EA@mail.gmail.com> <AFF06A1C-A58F-40E7-9A21-A585F95E447D@hausen.com> <CANCZdfq0Q67tqE-6%2BSjbdJVH_p0oMh5Xz_bW=SWWM%2BhWnDLZYw@mail.gmail.com> <20221109220540.Horde.6GRfN8bbUC4AsLcvjhWbUgm@webmail.leidinger.net> <65689194-1F56-4429-9A52-794A8354D995@hausen.com> <20221109222600.Horde.yo1Ry8tPu9udiPqUgtdDHNG@webmail.leidinger.net> <40D4F908-CE30-4AAF-89FA-E6849027FB17@hausen.com> <F450AC9B-15E8-4E60-83CC-1A90404D01BD@hausen.com> <20221109225904.Horde.a8xggy6etutDDjHEEeWpdpb@webmail.leidinger.net> <E6722C9D-BB70-4515-8944-5FE17B0542AF@hausen.com>
next in thread | previous in thread | raw e-mail | index | archive | help
On Wed, 9 Nov 2022 23:04:17 +0100 "Patrick M. Hausen" <pmh@hausen.com> wrote: > Hi, > > > Am 09.11.2022 um 22:59 schrieb Alexander Leidinger <Alexander@leidinger.net>: > > Quoting "Patrick M. Hausen" <pmh@hausen.com> (from Wed, 9 Nov 2022 22:47:28 +0100): > >> > >> I apologize, should have included that in the last mail. > >> This is a current FreeBSD 13.1-p2 hosting system we run. > >> [ List of features from an active root pool ] > >> Boots quite fine ;-) > > > > There are several in the list which are not in the list in zfsipl.c. So that list is not the full truth... > > Or whatever is missing is not critical to booting. The boot loader does not need > read/write access to the zpool. It only needs to be able to locate and read the kernel. > > Kind regards, > Patrick loader needs to read /boot/loader.conf, related scripts and kmods under /boot/ from boot pool, too. Not just kernel. Maybe, `zpool upgrade` would need some overhaul, with small change to stand/libsa/zfs/zfsimpl.c. *Move features_for_read[] alone to new, dedicated header that can be included from both stand/libsa/zfs/zfsimpl.c and any others (including zpool). *Let zpool to check whether the pool is bootable or not, and if bootable, only enable features included in the matrix, otherwise, enable all supported features on FreeBSD. I don't think ALL OS'es that have OpenZFS inported can always boot with all features enabled. So changes to zpool would be able to OS independent except the header defining features supported by its loader. There can be other matrixes for implemented (after boot) and default-to-be-enabled features. The OS-dependent matrixes would ease ZFS developers to determine which features are implemented / defaulted / bootable on specific OS. (Already there?) -- Tomoaki AOKI <junchoon@dec.sakura.ne.jp>
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20221110093206.661bbe915fd2e37338df51be>