Re: [PATCH] btrfs: scrub: avoid use-after-free when chunk end is not 64K aligned

[Qu Wenruo <quwenruo.btrfs@gmx.com>]

On 2024/1/17 04:58, David Sterba wrote:
> On Tue, Jan 16, 2024 at 09:20:58AM +1030, Qu Wenruo wrote:
>>
>>
>> On 2024/1/15 22:39, Johannes Thumshirn wrote:
>> [...]
>>>
>>>> - Make sure scrub_submit_initial_read() only to read the chunk range
>>>>      This is done by calculating the real number of sectors we need to
>>>>      read, and add sector-by-sector to the bio.
>>>
>>> Why can't you do it the same way the RST version does it by checking the
>>> extent_sector_bitmap and then add sector-by-sector from it?
>>
>> Sure, we can, although the whole new scrub code is before RST, and at
>> that time, the whole 64K read behavior is considered as a better option,
>> as it reduces the IOPS for a fragmented stripe.
>
> I'd like to keep the scrub fix separte from the RST code, even if
> there's a chance for some code sharing or reuse. The scrub fix needs to
> be backported so it's better to keep it independent.

So do I need to split the fix, so that the first part would be purely
for the non-RST scrub part, and then a small fix to the RST part?

I can try to do that, but since we need to touch the read endio function
anyway, it may mean if we don't do it properly, it may break bisection.

Thus I'd prefer to do a manual backport for the older branches without
the RST code.

Thanks,
Qu

>
> The change itself looks as an enhancement of the existing code so it's
> "obvious" and does not need any preparatory patches.
>