Review request.

[Brooks Davis]

On Tue, Jul 28, 2009 at 03:29:38PM +0200, Pawel Jakub Dawidek wrote:
> Hi.
> 
> I'm looking for a review of the following:
> 
> Currently there is a problem with fscking UFS file systems created on
> top of ZVOLs. The problem is that rc.d/fsck runs before rc.d/zfs. The
> latter makes ZVOLs to appear in /dev/. In such case rc.d/fsck cannot
> find devfs entry and aborts. We cannot simply move rc.d/zfs before
> rc.d/fsck, because we first want kern.hostid to be configured (by
> rc.d/hostid). If we won't wait (hostid will be 0) we can reuse disks
> which are in use by different systems (eg. in SAN/NAS environment).
> We also cannot move rc.d/hostid before rc.d/fsck, because rc.d/hostid on
> first system start stores generated kern.hostuuid in /etc/hostid file,
> so it needs root file system to be mounted read-write.
>   
> My fix is to split rc.d/hostid so that rc.d/hostid (which will now run
> before rc.d/fsck) only generates hostid and sets up sysctls, but doesn't
> touch root file system and rc.d/hostid_save (which is run after
> rc.d/root) and only creates /etc/hostid file.
>   
> With that in place, I can move ZVOL initialization to dedicated
> rc.d/zvol script which runs before rc.d/fsck.
> 
> The patch is here:
> 
> 	http://people.freebsd.org/~pjd/patches/zvol_fsck.patch

This looks fine to me and seems like a good solution to this problem.

-- Brooks
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 187 bytes
Desc: not available
Url : http://lists.freebsd.org/pipermail/freebsd-rc/attachments/20090728/1631bd74/attachment.pgp