Received-SPF: pass (google.com: domain of linux-kernel+bounces-3160-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.80.249 as permitted sender) client-ip=147.75.80.249;
Date: Mon, 18 Dec 2023 09:32:01 +0100
From: Mariusz Tkaczyk <mariusz.tkaczyk@linux.intel.com>
To: Song Liu <song@kernel.org>
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>
Subject: Re: [PATCH 0/3] md: Remove deprecated flavors
Message-ID: <20231218093201.000020dd@linux.intel.com>
In-Reply-To: <CAPhsuW6GZnufqFseLvgpMrrX6qRXodX1n89vEbbC-FqTjsWPDg@mail.gmail.com>
References: <20231214222107.2016042-1-song@kernel.org>
	<20231215125059.00006270@linux.intel.com>
	<CAPhsuW6GZnufqFseLvgpMrrX6qRXodX1n89vEbbC-FqTjsWPDg@mail.gmail.com>
Precedence: bulk
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable

On Fri, 15 Dec 2023 07:37:54 -0800
Song Liu <song@kernel.org> wrote:

> Hi Mariusz,
>=20
> On Fri, Dec 15, 2023 at 3:51=E2=80=AFAM Mariusz Tkaczyk
> <mariusz.tkaczyk@linux.intel.com> wrote:
> >
> > On Thu, 14 Dec 2023 14:21:04 -0800
> > Song Liu <song@kernel.org> wrote:
> > =20
> > > Linear, multipath, and faulty have been marked as deprecated for 2.5
> > > years. Let's remove them.
> > >
> > > Thanks,
> > > Song =20
> >
> > Hi Song,
> > Great idea!
> >
> > Please note that there are mdadm tests for those levels. I can approve =
it
> > only when mdadm clean-up is merged. Our tests must pass continuously. =
=20
>=20
> 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 pas=
sing,
I don't need results to see that. We should keep both mdadm and md compatib=
le.
We are continuously adding new MD regression tests to mdadm (at least Kuai =
is
doing that) so we should also care about removing things.

>=20
> >
> > It is a nice code complexity improvement so let me know if you would
> > like to get my help with mdadm patches. =20
>=20
> On my local tests with mdadm, I need to make changes to the following
> tests:
>=20
> 00linear...
> 00names...
> 00raid0...
> 00readonly...
> 02lineargrow...
> 03r0assem...
> 04r0update...
> 04update-metadata...
>=20
> The changes are all straightforward (just remove things related to
> linear/multipath/faulty).
>=20

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?

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

Thanks,
Mariusz