All of lore.kernel.org
 help / color / mirror / Atom feed
From: David Greaves <david@dgreaves.com>
To: Bill Davidsen <davidsen@tmr.com>
Cc: Jan Engelhardt <jengelh@computergmbh.de>,
	neilb@suse.de, linux-raid@vger.kernel.org
Subject: Re: [PATCH] Use new sb type
Date: Wed, 30 Jan 2008 12:53:18 +0000	[thread overview]
Message-ID: <47A0733E.60008@dgreaves.com> (raw)
In-Reply-To: <479FB1FB.6040500@tmr.com>

Bill Davidsen wrote:
> David Greaves wrote:
>> Jan Engelhardt wrote:
>>  
>>> This makes 1.0 the default sb type for new arrays.
>>>
>>>     
>>
>> IIRC there was a discussion a while back on renaming mdadm options
>> (google "Time
>> to  deprecate old RAID formats?") and the superblocks to emphasise the
>> location
>> and data structure. Would it be good to introduce the new names at the
>> same time
>> as changing the default format/on-disk-location?
>>   
> 
> Yes, I suggested some layout names, as did a few other people, and a few
> changes to separate metadata type and position were discussed. BUT,
> changing the default layout, no matter how "better" it seems, is trumped
> by "breaks existing setups and user practice." For all of the reasons
> something else is preferable, 1.0 *works*.

It wasn't my intention to change anything other than the naming.

If the default layout was being updated to 1.0 then I thought it would be a good
time to introduce 1-start, 1-4k and 1-end names and actually announce a default
of "1-end" and not "1.0".

Although I still prefer a full separation:
  mdadm --create /dev/md0 --metadata 1 --meta-location start

David


  reply	other threads:[~2008-01-30 12:53 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-01-28 17:22 [PATCH] Use new sb type Jan Engelhardt
2008-01-28 18:19 ` David Greaves
2008-01-28 18:47   ` Peter Rabbitson
2008-01-28 19:32     ` David Greaves
2008-01-29  4:09       ` Tim Southerwood
2008-01-29  9:37         ` Peter Rabbitson
2008-01-29 11:25           ` Yes, but please provide the clue (was Re: [PATCH] Use new sb type) Moshe Yudkowsky
2008-01-28 19:10   ` [PATCH] Use new sb type Jan Engelhardt
2008-01-29 23:08   ` Bill Davidsen
2008-01-30 12:53     ` David Greaves [this message]
2008-02-08  0:23     ` Jan Engelhardt
2008-02-10 10:34       ` David Greaves
2008-02-10 12:12         ` Jan Engelhardt
2008-02-10 12:27           ` David Greaves
2008-02-10 12:30             ` Jan Engelhardt
2008-02-11  3:32               ` Bill Davidsen
2008-02-11  8:21               ` David Greaves
2008-02-11 18:28                 ` 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=47A0733E.60008@dgreaves.com \
    --to=david@dgreaves.com \
    --cc=davidsen@tmr.com \
    --cc=jengelh@computergmbh.de \
    --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.