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

Steven Hartland killing at multiplay.co.uk
Sun Jun 9 18:13:37 UTC 2013


----- Original Message ----- 
From: "Hiroki Sato" <hrs at FreeBSD.org>
> 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.

Thanks for that Hiroki. If I understand correctly that your only
concern is the zfs module dependency? If so then its easier to make
that a dynamic dependency using a prestart.

Updated patch attached, which does just that.

Does this look better?

    Regards
    Steve

================================================
This e.mail is private and confidential between Multiplay (UK) Ltd. and the person or entity to whom it is addressed. In the event of misdirection, the recipient is prohibited from using, copying, printing or otherwise disseminating it or any information contained in it. 

In the event of misdirection, illegible or incomplete transmission please telephone +44 845 868 1337
or return the E.mail to postmaster at multiplay.co.uk.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: mdconfig-zfs.patch
Type: application/octet-stream
Size: 7406 bytes
Desc: not available
URL: <http://lists.freebsd.org/pipermail/freebsd-rc/attachments/20130609/12221891/attachment.obj>


More information about the freebsd-rc mailing list