Messages in this thread | | | Date | Wed, 12 Jul 2006 21:00:27 -0700 | From | Andrew Morton <> | Subject | Re: [PATCH] reiserfs: fix handling of device names with /'s in them |
| |
On Wed, 12 Jul 2006 20:51:47 -0700 Hans Reiser <reiser@namesys.com> wrote:
> Andrew Morton wrote: > > >On Wed, 12 Jul 2006 12:42:28 -0400 > >Jeff Mahoney <jeffm@suse.com> wrote: > > > > > > > >> On systems with block devices containing slashes (virtual dasd, cciss, > >> etc), reiserfs will fail to initialize /proc/fs/reiserfs/<dev> due to > >> it being interpreted as a subdirectory. The generic block device code > >> changes the / to ! for use in the sysfs tree. This patch uses that > >> convention. > >> > >> > > > >Isn't it a bit dumb of us to be putting slashes in the device names anyway? > > It would be better, if poss, to alter dasd/cciss/etc and stop all these > >s@/@!@everywhere games. > > > > > > > > > Isn't better to ask why there is a problem with the /'s? It would be > bad for Linux as a design to prevent passing arbitrary tail ends of > filenames off to arbitrary plugins of some kind. In general, in > namespace design, you want to allow delegating the job of > resolving/interpreting the tail end of a file that the front end has > identified as something that can interpret it. > > Forgive me, I probably understand something wongly about procfs and this > issue.... >
It's a question of being *practical*. Your observations are in the realm of the theoretical.
Software sucks, and we get along better by not provoking it. So don't put spaces, let alone slashes into strings which we offer to userspace. - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |