Received-SPF: pass (google.com: domain of linux-kernel+bounces-103532-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.80.249 as permitted sender) client-ip=147.75.80.249;
Date: Thu, 14 Mar 2024 18:55:08 +0300
From: Fedor Pchelkin <pchelkin@ispras.ru>
To: Jens Axboe <axboe@kernel.dk>
Cc: Pavel Begunkov <asml.silence@gmail.com>, 
	Greg Kroah-Hartman <gregkh@linuxfoundation.org>, stable@vger.kernel.org, io-uring@vger.kernel.org, 
	linux-kernel@vger.kernel.org, Alexey Khoroshilov <khoroshilov@ispras.ru>, 
	lvc-project@linuxtesting.org, Nikita Zhandarovich <n.zhandarovich@fintech.ru>, 
	Roman Belyaev <belyaevrd@yandex.ru>
Subject: Re: [PATCH 5.10/5.15] io_uring: fix registered files leak
Message-ID: <e7cbf950-c732-4eb3-a91f-8f09249e4f72-pchelkin@ispras.ru>
References: <20240312142313.3436-1-pchelkin@ispras.ru>
 <8a9993c7-fd4d-44ff-8971-af59c7f3052c@kernel.dk>
 <466e842f-66c6-4530-8c16-2b008fc3fbc6-pchelkin@ispras.ru>
 <fb57be64-4da6-418b-9369-eae0db42a570@kernel.dk>
 <085beb85-d1a4-4cb0-969b-e0f895a95738@kernel.dk>
 <a8c81d35-e6ac-420c-9ffa-24dd9e009e29-pchelkin@ispras.ru>
 <3f17f1d9-9029-4d03-9b0a-9c500cce54e9@kernel.dk>
Precedence: bulk
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <3f17f1d9-9029-4d03-9b0a-9c500cce54e9@kernel.dk>

On 24/03/13 06:40PM, Jens Axboe wrote:
> Hi,
> 
> OK, here they are. Two patches attached for every stable kernel, that
> gets rid of the remnants of the SCM related code:
> 
> 5.4
> 5.10 and 5.15 (same patches)
> 6.1
> 6.6
> 6.7
> 
> Would appreciate if Fedor and Pavel could give them a once over, but I
> think they are all fine. It's just deleting the code...

Thank you, Jens!

FWIW, I think it's all good and it eliminates the reported problem
obviously. Compiled and tested the repro with my kernel config.

Just a minor notice - stable rules declare two common ways for upstream
patch mentioning in backports [1]. And the first one starts from
lowercase. No big deal here definitely but maybe somebody has some
handling of these two variants - by regexps or similar, I actually don't
know. But I see in the git history that Greg also applies the variant
you've used.

[1]: https://www.kernel.org/doc/html/latest/process/stable-kernel-rules.html#option-3

--
Fedor