On 6/1/23 03:27, Mariusz Tkaczyk wrote: > Hi Jes, > To avoid problem with udev and VROC UEFI driver I developed stronger > naming policy, basing on POSIX portable names standard. Verification is > added for names and config entries. In case of an issue, user can update > name to make it compatible (for IMSM and native). > > The changes here may cause /dev/md/ link will be different than before > mdadm update. To make any of that happen user need to use unusual naming > convention, like: > - special, non standard signs like, $,%,&,* or space. > - '-' used as first sign. > - locals. > > Note: I didn't analyze configurations with "names=1". If name cannot be > determined mdadm should fallback to default numbered dev creation. > > If you are planning release soon then feel free to merge it after the > release. It is a potential regression point. > > It is a new version of [1] but it is strongly rebuild. Here is a list > of changes: > 1. negative and positive test scenarios for both create and config > entries are added. > 2. Save parsed parameters in dedicated structs. It is a way to control > what is parsed, assuming that we should use dedicated set_* function. > 3. Verification for config entries is added. > 5. Improved error logging for names: > - during creation, these messages are errors, printed to stderr. > - for config entries, messages are just a warnings printed to stdout. > 6. Error messages reworked. > 7. Updates in manual. > > [1] https://lore.kernel.org/linux-raid/20221221115019.26276-1-mariusz.tkaczyk@xxxxxxxxxxxxxxx/ > > Mariusz Tkaczyk (6): > tests: create names_template > tests: create 00confnames > mdadm: set ident.devname if applicable > mdadm: refactor ident->name handling > mdadm: define ident_set_devname() > mdadm: Follow POSIX Portable Character Set Applied! Thanks, Jes
