cvs commit: src/lib/libufs type.c

Pawel Jakub Dawidek pjd at FreeBSD.org
Fri Mar 16 03:26:23 UTC 2007


On Fri, Mar 16, 2007 at 03:13:29AM +0000, Pawel Jakub Dawidek wrote:
> pjd         2007-03-16 03:13:29 UTC
> 
>   FreeBSD src repository
> 
>   Modified files:
>     lib/libufs           type.c 
>   Log:
>   The ufs_disk_fillout(3) can take special device name (with or without /dev/
>   prefix) as an argument and mount point path. At the end it has to find
>   device name file system is stored on, which means when mount point path is
>   given, it tries to look into /etc/fstab and find special device
>   corresponding to the given mount point. This is not perfect, because it
>   doesn't handle the case when file system is mounted by hand and mount point
>   is given as an argument.
>   
>   I found this problem while trying to use snapinfo(8), which passes mount
>   points to the ufs_disk_fillout(3) function, but I had file system mounted
>   manually, so snapinfo(8) was exiting with the error below:
>   
>           ufs_disk_fillout: No such file or directory
>   
>   I modified libufs(3) to handle those arguments (the order is important):
>   
>   1. special device with /dev/ prefix
>   2. special device without /dev/ prefix
>   3. mount point listed in /etc/fstab, directory exists
>   4. mount point listed in /etc/fstab, directory doesn't exist
>   5. mount point of a file system mounted by hand

In my opinion, when mount point is given, it should always just use
statfs(2) and don't touch /etc/fstab, but I didn't want to change the
current behaviour. Juli, do you have an opinion about this?

-- 
Pawel Jakub Dawidek                       http://www.wheel.pl
pjd at FreeBSD.org                           http://www.FreeBSD.org
FreeBSD committer                         Am I Evil? Yes, I Am!
-------------- 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/cvs-src/attachments/20070316/75e5d0c9/attachment.pgp


More information about the cvs-src mailing list