All of lore.kernel.org
 help / color / mirror / Atom feed
From: Iustin Pop <iusty@k1024.org>
To: Neil Brown <neilb@suse.de>
Cc: Bill Davidsen <davidsen@tmr.com>, linux-raid@vger.kernel.org
Subject: Re: [PATCH 2.6.20-rc6] md: expose uuid and degraded attributes in sysfs
Date: Sat, 10 Feb 2007 22:50:41 +0100	[thread overview]
Message-ID: <20070210215040.GA12830@teal.hq.k1024.org> (raw)
In-Reply-To: <17870.13811.570367.617924@notabene.brown>

On Sun, Feb 11, 2007 at 08:15:31AM +1100, Neil Brown wrote:
> Resending after a suitable pause (1-2 weeks) is never a bad idea.
Ok, noted, thanks.

> Exposing the UUID isn't - and if it were it should be in
> "md_default_attrs" rather than "md_redundancy_attrs".
> 
> The UUID isn't an intrinsic aspect of the array.  It is simply part of
> the metadata that is used to match up different devices from the same
> array.
I see. Unfortunately, for now it's the only method of (more or less)
persistently identifying the array.

> I plan to add support for the 'DDF' metadata format (an 'industry
> standard') and that will be managed entirely in user-space.  The
> kernel won't know the uuid at all.
I've briefly looked over the spec, but this seems a non-trivial change,
away from current md superblocks to ddf... But the virtual disk GUIDs
seem nice. In the meantime, probably the solution you gave below is
best.

> So any solution for easy access to uuids should be done in user-space.
> Maybe mdadm could create a link
>    /dev/md/by-uuid/xxxxxxxx -> /dev/whatever.
> ??
That sounds like a good idea. mdadm (or udev or another userspace
solution) should work, given some safety measures against stale symlinks
and such. It seems to me that, since now it's possible to assemble
arrays without mdadm (by using sysfs), mdadm is not the best place to do
it. Probably relying on udev is a better option, however it right now it
seems that it gets only the block add events, and not the block remove
ones.

Thanks,
Iustin

  reply	other threads:[~2007-02-10 21:50 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-01-27  1:59 [PATCH 2.6.20-rc6] md: expose uuid and degraded attributes in sysfs Iustin Pop
2007-02-10 11:18 ` Iustin Pop
2007-02-10 18:09   ` Bill Davidsen
2007-02-10 21:15     ` Neil Brown
2007-02-10 21:50       ` Iustin Pop [this message]
2007-02-12  4:51       ` Bill Davidsen
2007-02-12  5:05         ` Neil Brown
2007-02-12 22:10           ` Bill Davidsen

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=20070210215040.GA12830@teal.hq.k1024.org \
    --to=iusty@k1024.org \
    --cc=davidsen@tmr.com \
    --cc=linux-raid@vger.kernel.org \
    --cc=neilb@suse.de \
    /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.