Received-SPF: pass (google.com: domain of linux-kernel+bounces-4043-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45e3:2400::1 as permitted sender) client-ip=2604:1380:45e3:2400::1;
Precedence: bulk
MIME-Version: 1.0
References: <20231214222107.2016042-1-song@kernel.org> <20231215125059.00006270@linux.intel.com>
 <CAPhsuW6GZnufqFseLvgpMrrX6qRXodX1n89vEbbC-FqTjsWPDg@mail.gmail.com> <20231218093201.000020dd@linux.intel.com>
In-Reply-To: <20231218093201.000020dd@linux.intel.com>
From: Song Liu <song@kernel.org>
Date: Mon, 18 Dec 2023 08:11:53 -0800
Message-ID: <CAPhsuW7w7WbWePpQd4aqnvXHwbdEJzw9efEP_r6tJwpeg0_qLw@mail.gmail.com>
Subject: Re: [PATCH 0/3] md: Remove deprecated flavors
To: Mariusz Tkaczyk <mariusz.tkaczyk@linux.intel.com>
Cc: linux-raid@vger.kernel.org, linux-kernel@vger.kernel.org, 
	linux-block@vger.kernel.org, Paul E Luse <paul.e.luse@linux.intel.com>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

Hi Mariusz,

On Mon, Dec 18, 2023 at 12:32=E2=80=AFAM Mariusz Tkaczyk
<mariusz.tkaczyk@linux.intel.com> wrote:
>
[...]
> > >
> > > Please note that there are mdadm tests for those levels. I can approv=
e it
> > > only when mdadm clean-up is merged. Our tests must pass continuously.
> >
> > Is the continuous test result available publicly?
>
> We are working on public CI (Paul owns it). On my side I'm not executing =
all
> tests, IMSM only. In this case it is obvious that mdadm tests will stop p=
assing,
> I don't need results to see that. We should keep both mdadm and md compat=
ible.
> We are continuously adding new MD regression tests to mdadm (at least Kua=
i is
> doing that) so we should also care about removing things.
>
> >
> > >
> > > It is a nice code complexity improvement so let me know if you would
> > > like to get my help with mdadm patches.
> >
> > On my local tests with mdadm, I need to make changes to the following
> > tests:
> >
> > 00linear...
> > 00names...
> > 00raid0...
> > 00readonly...
> > 02lineargrow...
> > 03r0assem...
> > 04r0update...
> > 04update-metadata...
> >
> > The changes are all straightforward (just remove things related to
> > linear/multipath/faulty).
> >
>
> Please do not forgot remove dead code from mdadm. For example simple find
> "multipath" (case insensitive) reefers me to multiple places with special
> handling for this level. We need to remove it from code and documentation=
.
> Can you handle this too?

I think this is a bigger discussion: will mdadm stop supporting these
flavors on
older kernels? Say, mdadm-5.0+ (or a different number) will not support
multipath flavor on older kernels?

>
> Oh and last one, I can't find update for md man in your changes. Could yo=
u
> please remove those levels from md man?

man side is easier. Once we know which major will have this set (6.8 or lat=
er),
we can update the man pages with the information.

Thanks,
Song