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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, 17 Oct 2024 08:31:17 -0700
"Hellwig, Christoph" <hch@xxxxxxxxxxxxx> 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




[Index of Archives]     [Linux RAID Wiki]     [ATA RAID]     [Linux SCSI Target Infrastructure]     [Linux Block]     [Linux IDE]     [Linux SCSI]     [Linux Hams]     [Device Mapper]     [Device Mapper Cryptographics]     [Kernel]     [Linux Admin]     [Linux Net]     [GFS]     [RPM]     [git]     [Yosemite Forum]


  Powered by Linux