[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <8b2c4fac-81be-4937-9619-c32813ab2650@kernel.org>
Date: Sat, 13 Dec 2025 12:22:02 +0900
From: Damien Le Moal <dlemoal@...nel.org>
To: Swarna Prabhu <sw.prabhu6@...il.com>
Cc: James.Bottomley@...senpartnership.com, martin.petersen@...cle.com,
linux-scsi@...r.kernel.org, bvanassche@....org,
linux-kernel@...r.kernel.org, mcgrof@...nel.org, kernel@...kajraghav.com
Subject: Re: [v1 0/2] enable sector size > PAGE_SIZE for scsi
On 2025/12/12 9:18, Swarna Prabhu wrote:
> On Tue, Dec 09, 2025 at 05:56:05PM -0800, Damien Le Moal wrote:
>> On 2025/12/09 17:41, sw.prabhu6@...il.com wrote:
>>> From: Swarna Prabhu <sw.prabhu6@...il.com>
>>>
>>> Hi All,
>>>
>>> This is non RFC v1 series sent based on the feedback received on RFC
>>> v2 [1] and RFC v1 [2]. This patchset enables sector sizes > PAGE_SIZE for
>>> sd driver and scsi_debug driver since block layer can support block
>>> size > PAGE_SIZE. There was one issue with write_same16 and write_same10
>>> command, which is fixed as a part of the series.
>>>
>>> Motivation:
>>> - While enabling LBS on ZoneFS, zonefs-tools tests were being skipped
>>> for conventional zones when tested on nvme block device emulated using
>>> QEMU. Hence there was a need to enable scsi with higher sector sizes
>>> to run zonefs tests for conventional zones as well.
>>
>> This is super confusing: there are no conventional zones with NVMe. And why
>> would a problem with NVMe require scsi patches ?
>>
> Agree with you. NVME Zoned Namespace require sequential writes.
>
> Our initial goal was to enable LBS on Zonefs. Running zonefs tests
> on a scsi device covered both conventional and sequential zone based
> tests. So we had to enable higher sector sizes on scsi device and while
> doing so fix the issue seen with WRITE SAME commands with higher sector
> sizes.
OK. Understood. But since this is in the end a scsi fix that is not directly
related to zonefs (zonefs does not use write same), I think you can drop any
mention of zonefs to avoid any confusion.
--
Damien Le Moal
Western Digital Research
Powered by blists - more mailing lists