This a bit off the topic; however, I pointed out the problem of this mount_XXX
behaviour quite long time ago.  It currently maintains a list of NON-supported
FS by nmount(2).  Instead, it's better to maintain a list of SUPPORTED FS by 
nmount.

In the current scheme, mount has to be aware of all the FSs that nmount(2) 
cannot
support.  In other words, if one attempts to add a new FS, he/she has to modify
mount(8) program.  That doesn't sound right, does it?

I attached the patch, again, if anyone is interested in commiting this fix.

Thanks,
Hiro

On Fri, 30 Nov 2007 11:18:08 +0800
"Yi Wang" <[EMAIL PROTECTED]> wrote:

> Yes, I think it does the trick.
>
> some words from mount(8):
> 
> The default behavior of mount is to pass the -t option directly to the
> nmount(2) system call in the fstype option.
> However, for the following file system types: cd9660, mfs, msdosfs,
> nfs, nfs4, ntfs, nwfs, nullfs, portalfs, smbfs, udf, and unionfs,
> mount will not call nmount(2) directly and will instead attempt to
> execute a program in /sbin/mount_XXX where XXX is replaced by the file
> system type name.
> 
> according to this, your patch should works fine.
> 
> I didn't test it yet because I delete /usr/obj before. So I'm build world now.
> 
> I'll post the result when it finish.
> 
> On 11/30/07, Suhaib . <[EMAIL PROTECTED]> wrote:
> >
> > > ----- Original Message -----
> > > From: "Alejandro Pulver" <[EMAIL PROTECTED]>
> > > To: "Aryeh M. Friedman" <[EMAIL PROTECTED]>
> > > Subject: Re: problem with mount_ntfs-3g
> > > Date: Thu, 29 Nov 2007 20:16:14 -0300
> > >
> > >
> > > On Thu, 29 Nov 2007 19:42:00 -0300
> > > Alejandro Pulver <[EMAIL PROTECTED]> wrote:
> > >
> > > > On Tue, 27 Nov 2007 09:35:48 -0500
> > > > "Aryeh M. Friedman" <[EMAIL PROTECTED]> wrote:
> > > >
> > > > > > Then, I execute this command:
> > > > > > mount -t ntfs-3g -o ro /dev/ad6s10 /media/g
> > > > > > but is also says:
> > > > > > mount: /dev/ad6s10 : Operation not supported by device
> > > > > >
> > > > > > Is this a bug of mount?
> > > > > > I get exactly the same thing on -current AMD64.
> > > > > >
> > > > > > uname -a:
> > > > > > FreeBSD wangyi.com 7.0-BETA3 FreeBSD 7.0-BETA3 #0: Fri Nov 23 
> > > > > > 12:18:30
> > > > > > CST 2007     [EMAIL PROTECTED]:/usr/obj/usr/src/sys/MYKERNEL  i386
> > > > > >
> > > > > Can't reproduce in my machine, please try with the latest version
> > > > (attached as shell archive).
> > > >
> > >
> > > Sorry, it was stripped. It's available here:
> > >
> > > ftp://ftp.alepulver.com.ar/fusefs-ntfs.shar
> > >
> > > Best Regards,
> > > Ale
> > > << signature.asc >>
> >
> >
> > I had the same problem on RELENG_7 and made a minor edit to mount
> > for ntfs-3g to work. However,I have not had the time to investigate
> > whether or not it is the correct solution, see the comment about
> > nmount() in mount.c
> >
> > To try out the change apply the attached patch file from /usr/src,
> > then cd to /usr/src/sbin/mount and run make. A quick test...
> >
> > # mount -t ntfs-3g -o ro /dev/da8s1 /mnt
> > mount: /dev/da8s1 : Operation not supported by device
> >
> > # /usr/obj/usr/src/sbin/mount/mount -t ntfs-3g -o ro /dev/da8s1 /mnt
> >
> >
> > --
> > Suhaib
> >
> > --
> > _______________________________________________
> > Surf the Web in a faster, safer and easier way:
> > Download Opera 9 at http://www.opera.com
> >
> > Powered by Outblaze
> >
> >
> 
> 
> -- 
> Regards,
> Wang Yi
> _______________________________________________
> freebsd-ports@freebsd.org mailing list
> http://lists.freebsd.org/mailman/listinfo/freebsd-ports
> To unsubscribe, send any mail to "[EMAIL PROTECTED]"
Index: mount.c
===================================================================
RCS file: /home/ncvs/src/sbin/mount/mount.c,v
retrieving revision 1.92
diff -u -r1.92 mount.c
--- mount.c	14 Nov 2006 01:07:42 -0000	1.92
+++ mount.c	1 Feb 2007 05:25:55 -0000
@@ -133,18 +133,19 @@
 	 */
 	unsigned int i;
 	const char *fs[] = {
-	"cd9660", "mfs", "msdosfs", "nfs", "nfs4", "ntfs",
-	"nwfs", "nullfs", "portalfs", "smbfs", "udf", "umapfs",
-	"unionfs",
+	"ufs",
+	"ext2fs",
+	"devfs", "fdescfs", "procfs", "linprocfs", "linsysfs",
+	"std",
 	NULL
 	};
 
 	for (i = 0; fs[i] != NULL; ++i) {
 		if (strcmp(vfstype, fs[i]) == 0)
-			return (1);
+			return (0);
 	}
 	
-	return (0);
+	return (1);
 }
 
 static int
_______________________________________________
freebsd-ports@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-ports
To unsubscribe, send any mail to "[EMAIL PROTECTED]"

Reply via email to