Date: Tue, 26 Jan 2010 22:52:35 +0100 From: =?ISO-8859-1?Q?Fernando_Apestegu=EDa?= <fernando.apesteguia@gmail.com> To: John Baldwin <jhb@freebsd.org> Cc: freebsd-hackers@freebsd.org Subject: Re: [PATCH] linprocfs dofilesystems Message-ID: <1bd550a01001261352h1ecc22dbh8fd2dcf9c8ac51c5@mail.gmail.com> In-Reply-To: <201001151110.29417.jhb@freebsd.org> References: <1bd550a01001080919p20ec1b20t82b3761705cd1ff3@mail.gmail.com> <201001151110.29417.jhb@freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
2010/1/15 John Baldwin <jhb@freebsd.org>: > On Friday 08 January 2010 12:19:29 pm Fernando Apestegu=EDa wrote: >> Hi all, >> >> This patch implements the "filesystems" file in the linux proc fs. >> I have used it for some time without seeing any problems. Let me >> know in case this is useful. >> >> Tested against 8.0-RELEASE-p1 > > This patch is not correct. =A0It seems that /proc/filesystems is a list o= f > available filesystems, not a list of mounted filesystems. =A0E.g.: > >> cat /proc/filesystems > nodev =A0 sysfs > nodev =A0 rootfs > nodev =A0 bdev > nodev =A0 proc > nodev =A0 sockfs > nodev =A0 binfmt_misc > nodev =A0 usbfs > nodev =A0 usbdevfs > nodev =A0 futexfs > nodev =A0 tmpfs > nodev =A0 pipefs > nodev =A0 eventpollfs > nodev =A0 devpts > =A0 =A0 =A0 =A0ext2 > nodev =A0 ramfs > nodev =A0 hugetlbfs > =A0 =A0 =A0 =A0iso9660 > nodev =A0 relayfs > nodev =A0 mqueue > =A0 =A0 =A0 =A0ext3 > nodev =A0 rpc_pipefs > nodev =A0 nfs > nodev =A0 nfs4 > nodev =A0 autofs > > To do the same thing in FreeBSD you would need to walk the vfsconf list > instead. =A0However, I'm not sure it is worth it to add this unless there > are apps people commonly use that need it. You are right. I have another patch to do the right thing. However as you pointed out maybe it is not useful after all. Is it possible to delete the PR? Sorry for the noise PS: My current patch does not distinguish between common filesystems and pseudo filesystems, where could I find that info? Cheers > > -- > John Baldwin >
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?1bd550a01001261352h1ecc22dbh8fd2dcf9c8ac51c5>