cvs commit: src/sys/kern vfs_mount.c

Dag-Erling Smørgrav des at des.no
Mon Feb 18 04:50:38 PST 2008


Yar Tikhiy <yar at FreeBSD.org> writes:
>   Log:
>   In the new order of things dictated by nmount(2), a read-only mount
>   is to be requested via a "ro" option.  At the same time, MNT_RDONLY
>   is gradually becoming an indicator of the current state of the FS
>   instead of a command flag.  Today passing MNT_RDONLY alone to the
>   kernel's mount machinery will lead to various glitches.  (See the
>   PRs for examples.)
>   
>   Therefore mount the root FS with a "ro" option instead of the
>   MNT_RDONLY flag.  (Note that MNT_RDONLY still is added to the mount
>   flags internally, by vfs_donmount(), if "ro" was specified.)

Can you guarantee that this will not f*** up the bootp / dhcp + nfsroot
case?  There are dragons in that code which were decidedly not funny to
track down and fix.

DES
-- 
Dag-Erling Smørgrav - des at des.no


More information about the cvs-all mailing list