All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jeff Mahoney <jeffm@suse.com>
To: Hans Reiser <reiser@namesys.com>
Cc: Valdis.Kletnieks@vt.edu, 7eggert@gmx.de,
	Eric Dumazet <dada1@cosmosbay.com>,
	ReiserFS List <reiserfs-list@namesys.com>,
	Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
	Andrew Morton <akpm@osdl.org>
Subject: Re: [PATCH] reiserfs: fix handling of device names with /'s in them
Date: Mon, 17 Jul 2006 15:07:51 -0400	[thread overview]
Message-ID: <44BBE007.1030209@suse.com> (raw)
In-Reply-To: <44BBDCB5.8070906@namesys.com>

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Hans Reiser wrote:
> Jeff Mahoney wrote:
> 
>> Valdis.Kletnieks@vt.edu wrote:
>>
>>> On Sun, 16 Jul 2006 20:02:27 PDT, Hans Reiser said:
>>>> Create a mountpoint which knows how to resolve a/b without using a
>>>> "directory".
>>> And said mountpoint gets past the '/' interpretation in the VFS, how,
>> exactly?
>>
>>> fs/namei.c, do_path_lookup() does magic on a '/' on about the 3rd line.
>>> So you're going to get handed 'a'.
>>
>> That's where he started talking about how BSD gets namei() right by
>> allowing each file system to deal with it how it chooses.
>>
>> Personally, I think it's insane. On occasion, I've started to port
>> ReiserFS to BSD-like systems, 
> 
> Porting V3 to anything is insane.  Why would you even consider it?

Because I have an iBook I dual boot, and I wanted access to my reiserfs
file systems while using OSX. I'd call it more of a write-from-scratch
than a port, actually.

>> and I get so fed up with how you have to
>> reinvent the wheel for everything. There's something to be said for
>> replaceable-anything semantics, but personally I like the Linux model
>> and having an agreed-upon framework to work with.
> 
> Linux vs. BSD's namei is the difference between thinking you know how to
> do things and everyone should be forced into your mold, and thinking
> that someone will always be more clever, at the very least with regards
> to some special case you could never have anticipated.

That's great, except that by and large, the Linux VFS covers all the
common cases for you. This is such a ridiculous corner case that it
hardly justifies using the BSD namei() semantics.

>> I also think it's insane to come up with a reisermetafs to export procfs
>> information when a simple s#/#!# _on a single directory name_ will do
>> the job.
> 
> Or just create a parent directory and skip the metafs.  Look, I don't
> much care about the other details of coding it, but if you are changing
> !'s to /'s, as an architect my intuition says something is wrong and
> being papered over.  /'s are just fine, and what the block devices do is
> elegant.   You are doing a quick hack.

Yes, it is a quick hack. It's called being practical and consistent
until we can get around to removing slashes. Slashes *were* ok in block
device names until we started using them as path name components.

I've stated the reasons why adding a subdirectory is a bad idea multiple
times. The fix is already accepted. I'm done discussing this.

- -Jeff

- --
Jeff Mahoney
SUSE Labs
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.2 (GNU/Linux)
Comment: Using GnuPG with SUSE - http://enigmail.mozdev.org

iD8DBQFEu+AGLPWxlyuTD7IRAjzqAKCk/nSheinL6AL4m+9YbG5FIo7f/ACeKWOd
Urif7U9OMFwX9JzywowrMiM=
=86sM
-----END PGP SIGNATURE-----

  reply	other threads:[~2006-07-17 19:07 UTC|newest]

Thread overview: 38+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <6xQ4C-6NB-43@gated-at.bofh.it>
     [not found] ` <6xQea-6ZX-13@gated-at.bofh.it>
2006-07-14 16:10   ` [PATCH] reiserfs: fix handling of device names with /'s in them Bodo Eggert
2006-07-14 17:50     ` Jeff Mahoney
2006-07-16  7:12       ` Hans Reiser
2006-07-16 15:56         ` Jeffrey Mahoney
2006-07-16 18:14           ` Hans Reiser
2006-07-16 22:17             ` Jeff Mahoney
2006-07-17  1:21               ` Hans Reiser
2006-07-17  1:50                 ` Jeffrey Mahoney
2006-07-17  3:02                   ` Hans Reiser
2006-07-17  3:17                     ` Jeffrey Mahoney
2006-07-17  7:29                       ` Hans Reiser
2006-07-17 14:55                         ` Jeff Mahoney
2006-07-17 15:39                           ` Carlos Carvalho
2006-07-17 16:05                             ` Jeff Mahoney
2006-07-17 18:21                               ` Hans Reiser
2006-07-17 18:19                           ` Hans Reiser
2006-07-17 18:38                             ` Jeff Mahoney
2006-07-17 19:07                               ` Hans Reiser
2006-07-17 19:59                                 ` Jeff Mahoney
2006-07-17 21:03                                   ` Hans Reiser
2006-07-17 22:26                                     ` Rudy Zijlstra
2006-07-18  0:26                                     ` Horst von Brand
2006-07-18 16:38                                     ` Jan Engelhardt
2006-07-18 18:37                                 ` Pekka Enberg
2006-07-17 18:08                     ` Valdis.Kletnieks
2006-07-17 18:16                       ` Jeff Mahoney
2006-07-17 18:53                         ` Hans Reiser
2006-07-17 19:07                           ` Jeff Mahoney [this message]
2006-07-17 18:27                       ` Hans Reiser
2006-07-17 19:52                         ` Valdis.Kletnieks
2006-07-12 16:42 Jeff Mahoney
2006-07-12 16:57 ` Eric Dumazet
2006-07-12 17:02   ` Jeff Mahoney
2006-07-13  0:55 ` Andrew Morton
2006-07-13  3:51   ` Hans Reiser
2006-07-13  4:00     ` Andrew Morton
2006-07-13  7:34       ` Hans Reiser
2006-07-13  5:23   ` H. Peter Anvin

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=44BBE007.1030209@suse.com \
    --to=jeffm@suse.com \
    --cc=7eggert@gmx.de \
    --cc=Valdis.Kletnieks@vt.edu \
    --cc=akpm@osdl.org \
    --cc=dada1@cosmosbay.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=reiser@namesys.com \
    --cc=reiserfs-list@namesys.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.