Date:   Tue, 7 Mar 2023 22:18:35 +0100
From:   Pavel Machek <pavel@ucw.cz>
To:     Eric Biggers <ebiggers@kernel.org>
Cc:     Sasha Levin <sashal@kernel.org>, linux-kernel@vger.kernel.org,
        stable@vger.kernel.org, viro@zeniv.linux.org.uk,
        linux-fsdevel@vger.kernel.org
Subject: Re: AUTOSEL process
Message-ID: <ZATC3djtr9/uPX+P@duo.ucw.cz>
References: <20230226034256.771769-1-sashal@kernel.org>
 <20230226034256.771769-12-sashal@kernel.org>
 <Y/rbGxq8oAEsW28j@sol.localdomain>
 <Y/rufenGRpoJVXZr@sol.localdomain>
 <Y/ux9JLHQKDOzWHJ@sol.localdomain>
 <Y/y70zJj4kjOVfXa@sashalap>
 <Y/zswi91axMN8OsA@sol.localdomain>
 <Y/zxKOBTLXFjSVyI@sol.localdomain>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha1;
        protocol="application/pgp-signature"; boundary="rghMMqG7Ff+4WZ/W"
Content-Disposition: inline
In-Reply-To: <Y/zxKOBTLXFjSVyI@sol.localdomain>
Precedence: bulk


--rghMMqG7Ff+4WZ/W
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

Hi!

> > So to summarize, that buggy commit was backported even though:
> >=20
> >   * There were no indications that it was a bug fix (and thus potential=
ly
> >     suitable for stable) in the first place.
> >   * On the AUTOSEL thread, someone told you the commit is broken.
> >   * There was already a thread that reported a regression caused by the=
 commit.
> >     Easily findable via lore search.
> >   * There was also already a pending patch that Fixes the commit.  Agai=
n easily
> >     findable via lore search.
> >=20
> > So it seems a *lot* of things went wrong, no?  Why?  If so many things =
can go
> > wrong, it's not just a "mistake" but rather the process is the problem.=
=2E.
>=20
> BTW, another cause of this is that the commit (66f99628eb24) was AUTOSEL'=
d after
> only being in mainline for 4 days, and *released* in all LTS kernels afte=
r only
> being in mainline for 12 days.  Surely that's a timeline befitting a crit=
ical
> security vulnerability, not some random neural-network-selected commit th=
at
> wasn't even fixing anything?

I see this problem, too, "-stable" is more experimental than Linus's
releases.

I believe that -stable would be more useful without AUTOSEL process.

Best regards,
									Pavel
--=20
People of Russia, stop Putin before his war on Ukraine escalates.

--rghMMqG7Ff+4WZ/W
Content-Type: application/pgp-signature; name="signature.asc"

-----BEGIN PGP SIGNATURE-----

iF0EABECAB0WIQRPfPO7r0eAhk010v0w5/Bqldv68gUCZAeqKwAKCRAw5/Bqldv6
8nfRAJwPw04Enp8eMKpkA3BuciBXfqt9kgCfTBR5ceYD64Tyfu6k4KqHZpIU4dk=
=ErRX
-----END PGP SIGNATURE-----

--rghMMqG7Ff+4WZ/W--