Joerg Wunsch wrote:
> Maxime Henrion <[EMAIL PROTECTED]> wrote:
> 
> > I looked at the code a bit more closely and you're entirely right.  I
> > think I figured out why my patch caused a core dump.  Here is a more
> > correct patch that should fix the problem without causing core dumps.
> 
> Seems to work.  mount(8) has still the problem though:

Great, I'll file a PR for it.  Thanks for the feedback !

> uriah # mount -t local
> uriah # kldload nfs
> uriah # mount -t local
> uriah # mount -t nonfs
> /dev/da0a on / (ufs, local, soft-updates)
> devfs on /dev (devfs, local)
> /dev/vinum/var on /var (ufs, local, soft-updates)
> /dev/vinum/usr on /usr (ufs, local, soft-updates)
> /dev/vinum/home on /home (ufs, local, soft-updates)
> /dev/vinum/home_cvs on /home/cvs (ufs, NFS exported, local, soft-updates)
> /dev/vinum/src on /usr/src (ufs, local, soft-updates)
> /dev/vinum/othersrc on /usr/othersrc (ufs, local, soft-updates)
> /dev/vinum/obj on /usr/obj (ufs, local, soft-updates)
> /dev/vinum/ports on /usr/ports (ufs, local, soft-updates)
> /dev/vinum/distfiles on /usr/ports/distfiles (ufs, NFS exported, local, soft-updates)
> /dev/vinum/news on /var/spool/news (ufs, local, soft-updates)
> /dev/vinum/tmp on /tmp (ufs, NFS exported, local, soft-updates)
> /dev/vinum/release on /usr/release (ufs, NFS exported, local, soft-updates)
> /dev/vinum/junk on /junk (ufs, local, soft-updates)
> procfs on /proc (procfs, local, read-only)

I fail to see why should ``mount -t local'' work.  I don't see anything
related in the mount(8) manpage.  To my knowledge, -t is only used when
specifying a particular fs type (nfs, msdosfs, ...) optionally prepended
with a ``no''.  If you point me to the relevant documentation, I'll be
happy to fix this bug too.

Thanks,
Maxime Henrion
-- 
Don't be fooled by cheap finnish imitations ; BSD is the One True Code

To Unsubscribe: send mail to [EMAIL PROTECTED]
with "unsubscribe freebsd-current" in the body of the message

Reply via email to