public inbox for linux-fsdevel@vger.kernel.org
 help / color / mirror / Atom feed
From: Oleg Drokin <green@linuxhacker.ru>
To: Bryan Henderson <hbryan@us.ibm.com>
Cc: linux-fsdevel@vger.kernel.org, neilb@cse.unsw.edu.au
Subject: Re: FS-specified FSID for non-device based filesystems?
Date: Sat, 15 Apr 2006 00:41:31 +0300	[thread overview]
Message-ID: <20060414214131.GK10907@linuxhacker.ru> (raw)
In-Reply-To: <OF17C55B57.244C73CC-ON88257150.007544C1-88257150.00766198@us.ibm.com>

Hello!

On Fri, Apr 14, 2006 at 02:32:44PM -0700, Bryan Henderson wrote:
> >From my reading of the code it seems that fsid is composed of device's
> >minor/major and export point's inode ino.
> >I propose to replace (if supported by fs) device's minor/major by
> >some unique number returned by fs itself.
> >Should be no worse than what we have now.
> That's for a device-based filesystem.  But isn't your proposal primarily 

Yes. Device-based fs has root->s_dev. For non-device based (and for
device-based too, if they would like) - method to get some sort of
unique id to be used in the same way root->s_dev is used.

> about non-device-based filesystems, and replacing a user-supplied value 
> with this filesystem identifier?  It isn't so clear that that can't make 
> things worse.

No, replacing whole fsid= with fs-supplied value would have implications like
that you mentioned - we might have two different points in filesystem
exported and we do not want those to have same id.

> But I don't actually remember the specific cases in which the fsid was 
> considered insufficient; only that someone (probably Neil) did find that 
> it was.  (Before Linux had export IDs, you simply couldn't export a 
> non-device-based filesystem at all, so we had to make a fake device on 
> which the filesystem could "reside."  The user managed the device number 
> of that device as the filesystem ID).

We still have those "fake devices". In fact non-device based fs can set
REQUIRES_FS_DEV in fs_flags and it can be exported after that without
manually specifying fsid. (yeah, I tried). But distributed filesystems would
like to have same device id on different nodes.

Bye,
    Oleg

  reply	other threads:[~2006-04-14 21:41 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-04-12 11:10 FS-specified FSID for non-device based filesystems? Oleg Drokin
2006-04-14 20:22 ` Bryan Henderson
2006-04-14 20:45   ` Oleg Drokin
2006-04-14 20:59     ` J. Bruce Fields
2006-04-14 21:32     ` Bryan Henderson
2006-04-14 21:41       ` Oleg Drokin [this message]
2006-04-17  0:54 ` Neil Brown
2006-04-17  3:50   ` Greg KH
2006-04-17  4:32     ` Neil Brown
2006-04-18 18:07       ` Greg KH
2006-04-19 12:15         ` Miklos Szeredi
2006-04-20 11:35           ` Al Viro
2006-04-20 11:58             ` Miklos Szeredi
2006-04-20 12:30               ` Al Viro
2006-04-20 12:43                 ` Miklos Szeredi
2006-04-17  8:00   ` Anton Altaparmakov

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=20060414214131.GK10907@linuxhacker.ru \
    --to=green@linuxhacker.ru \
    --cc=hbryan@us.ibm.com \
    --cc=linux-fsdevel@vger.kernel.org \
    --cc=neilb@cse.unsw.edu.au \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox