Feeback on patch to add ZFS support to mdconfig rc.d scripts

[Hiroki Sato]

"Steven Hartland" <smh at freebsd.org> wrote
  in <B5D8F93BF50D4D57BF31A4B2FCBA7621 at multiplay.co.uk>:

sm> Attached is a patch which adds ZFS support to the
sm> mdconfig rc.d scritps along with some other little
sm> fixes while I was there.
sm>
sm> Our use case for this here is to create a ZFS pool
sm> on a swap backed node for temporary data which supports
sm> all the cool features of ZFS, for us thats compression.
sm>
sm> Possible enhancements, suggested by pjd in IRC include
sm> default flags such as:-o cachefile=none
sm>
sm> Current summary:-
sm> =======
sm> Add ZFS pool creation support to mdconfig rc.d script
sm>
sm> Fix failure error redirection in mount check
sm>
sm> Fix invalid configuration detection enabling -t vnode
sm> to be skipped if -f <file> is specified as supported
sm> by mdconfig.
sm>
sm> Fix fsck not working if mount point is not present in fstab.
sm>
sm> TODO: update rc.conf man page
sm> =======

 I think this should be separated into adding non-fs md support into
 rc.d/mdconfig and on-the-fly zpool creation/import into rc.d/zfs
 since the rc.d/mdconfig script does not (and should not) depend on
 zfs.ko module.

 rc.d/mdconfig is located before mountcritlocal, so probably
 rc.d/zfs_md or something is needed just after rc.d/mdconfig2.

-- Hiroki
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 196 bytes
Desc: not available
URL: <http://lists.freebsd.org/pipermail/freebsd-rc/attachments/20130609/c699e0ad/attachment.sig>