Re: [PATCH] Add the ":" to the allowed_symbols list to work with the latest POSIX changes

[Mariusz Tkaczyk <mariusz.tkaczyk@linux.intel.com>]

On Thu, 17 Oct 2024 08:31:17 -0700
"Hellwig, Christoph" <hch@infradead.org> wrote:

> On Thu, Oct 17, 2024 at 11:27:24AM -0400, Laurence Oberman wrote:
> > I re-ran all the tests in house and I can actually start the arrays
> > that have the ":" in the names, just not create new ones. So I think we
> > leave this as is and we keep the adherence to POSIX for newly created
> > arrays only.   
> 
> Can we please stop talking about Posix compliance here?
> 
> The POSIX Portable Character Set is the minimum set of characters that
> need to be supported in each character set for Posix compliance.
> 
> Including other characters is perfectly Posix complain, just not fully
> portable.
> 

Right, good one. I named it just POSIX for simplicity and now we are abusing
it.

Cool, the main reasons ":" is excluded is special meaning for native metadata
and I still think it is right change. They can use --homehost option if they
want to customize it, it should be perfectly valid:

# mdadm -CR vol0 --homehost myhost -l0 -n2 /dev/nvme[45]n1 -z5G --assume-clean

# mdadm -E /dev/nvme4n1 | grep Name
           Name : myhost:vol0

Isn't result the same?
I still need to look into assemble issue to understand why it is not working.
Maybe there is small functional difference I'm not familiar with.

Thanks,
Mariusz