Re: [PATCH 0/4] Few config related refactors

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

On Fri, 24 Mar 2023 10:13:04 +0800
Xiao Ni <xni@redhat.com> wrote:

> On Fri, Mar 24, 2023 at 12:50 AM Mariusz Tkaczyk
> <mariusz.tkaczyk@linux.intel.com> wrote:
> >
> > Hi Jes,
> > These patches remove multiple inlines across code and replace them
> > by defines or functions. No functional changes intended. The goal
> > is to make this some code reusable for both config and cmdline
> > (mdadm.c). I next patchset I will start optimizing names verification
> > (extended v2 of previous patchset).
> >
> > Mariusz Tkaczyk (4):
> >   mdadm: define DEV_MD_DIR
> >   mdadm: define DEV_NUM_PREF
> >   mdadm: define is_devname_ignore()
> >   mdadm: numbered names verification
> >
> >  Create.c      |  7 +++----
> >  Detail.c      |  9 ++++-----
> >  Incremental.c | 10 ++++------
> >  Monitor.c     | 34 +++++++++++++++++++---------------
> >  config.c      | 43 +++++++++++++++++++++----------------------
> >  lib.c         |  4 ++--
> >  mapfile.c     | 12 ++++++------
> >  mdadm.c       |  5 ++---
> >  mdadm.h       | 21 ++++++++++++++++++++-
> >  mdopen.c      | 16 ++++++++--------
> >  super-ddf.c   |  2 +-
> >  super-intel.c |  2 +-
> >  super1.c      |  3 +--
> >  sysfs.c       |  2 +-
> >  util.c        | 44 ++++++++++++++++++++++++++++++++++++++++++++
> >  15 files changed, 137 insertions(+), 77 deletions(-)
> >
> > --
> > 2.26.2
> >  
> 
> Acked-by: Xiao Ni <xni@redhat.com>
> 

Hi Jes,
Could you please take those patches?
We are working on changes in other areas and the error enum will be useful.

Thanks,
Mariusz