All of lore.kernel.org
 help / color / mirror / Atom feed
From: Mariusz Tkaczyk <mariusz.tkaczyk@linux.intel.com>
To: Andreas Klauer <Andreas.Klauer@metamorpher.de>
Cc: linux-raid@vger.kernel.org
Subject: Re: mdadm 4.3 rejects /dev/md128 and larger numbers
Date: Mon, 4 Mar 2024 16:00:51 +0100	[thread overview]
Message-ID: <20240304160051.000037a7@linux.intel.com> (raw)
In-Reply-To: <ZeXKYbxagk7SD0UH@metamorpher.de>

On Mon, 4 Mar 2024 14:19:29 +0100
Andreas Klauer <Andreas.Klauer@metamorpher.de> wrote:

> Hello,
> 
> since mdadm 4.3, trying to use numbers larger than 127 results in:
> 
>   mdadm: Value "/dev/md3032" cannot be set as devname. 
>   Reason: Not POSIX compatible. Value ignored.
> 
> Because in util.c :: is_devname_numbered() (commit 25aa73291):
> 
>   if (val > 127)
>     return false;
> 
> The kernel seems to be fine with MINORMASK (2^20 - 1).
> If so, instead of 127, the limit here should be 1048575?
> 
> I don't need a million arrays. But I do have more arrays than 
> average because I use partitions instead of one big array 
> for everything. And some flexibility in using distinct number 
> ranges per group of arrays makes /proc/mdstat easier to read.
> 
> Regards,
> Andreas Klauer
> 

Hello Andreas,
Indeed, that it the case I missed. Sorry for bringing regression.

In the longer term, for better user experience I would like promote conception
called "named arrays", so could you please try to create "/dev/md/md_3032" and
test and see if that works for you?

The difference is that you will not be restricted to numbered devices, you can
also use asci letters like: "md_myarray", "md_do_not_touch".

Of course, it is kind of regression so I'm open to fix it for you but if I will
be able to convince you to use named arrays format (and help me fix issues!)
then whole community will gain.

You have to enable it first, but please note that it is not widely used now:

echo CREATE name=yes > /etc/mdadm.conf

Please test it and let me know!

Thanks,
Mariusz

      reply	other threads:[~2024-03-04 15:01 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-03-04 13:19 mdadm 4.3 rejects /dev/md128 and larger numbers Andreas Klauer
2024-03-04 15:00 ` Mariusz Tkaczyk [this message]

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=20240304160051.000037a7@linux.intel.com \
    --to=mariusz.tkaczyk@linux.intel.com \
    --cc=Andreas.Klauer@metamorpher.de \
    --cc=linux-raid@vger.kernel.org \
    /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.