Re: [PATCH v2 0/8] zbd: support continue_on_error for zonemode=zbd

On 5/7/25 11:19 AM, Vincent Fu wrote:
> On 5/7/25 7:29 AM, Jens Axboe wrote:
>>
>> On Fri, 25 Apr 2025 14:21:40 +0900, Shin'ichiro Kawasaki wrote:
>>> When the continue_on_error options is specified, it is expected that the
>>> workload continues to run when non-critical errors happen. However,
>>> write workloads with zonemode=zbd option can not continue after errors,
>>> if the failed writes cause partial data write on the target device. This
>>> partial write creates write pointer gap between the device and fio, then
>>> the next write requests by fio will fail due to unaligned write command
>>> errors. This restriction results in undesirable test stops during long
>>> runs for SMR drives which can recover defect sectors.
>>>
>>> [...]
>>
>> Applied, thanks!
>>
>> [1/8] oslib: blkzoned: add blkzoned_move_zone_wp() helper function
>>        commit: 4175f4dbec5d1d9e5e0490026e98b1806188e098
>> [2/8] ioengine: add move_zone_wp() callback
>>        commit: 6f635d6f72ad3d4ae76fff63671a208c27bdaf2d
>> [3/8] engines/libzbc: implement move_zone_wp callback
>>        commit: d4f6fa5e35d6bd20dd648b3aaaad84a0a9a4fa6e
>> [4/8] zbd: introduce zbd_move_zone_wp()
>>        commit: 143aaff963694ee3745c204f414e1e27a759a7df
>> [5/8] zbd: add the recover_zbd_write_error option
>>        commit: 650c4ad385cf7ff320cb34f76784ca63d2daa32e
>> [6/8] t/zbd: set badblocks related parameters in run-tests-against-nullb
>>        commit: 5cbd1644e0dcfeb55e6cb4717259778ed11cc70e
>> [7/8] t/zbd: add the test cases to confirm continue_on_error option
>>        commit: b6002e78926bce20bc168d02c83b7c8f5dd37470
>> [8/8] t/zbd: add run-tests-against-scsi_debug
>>        commit: 4dc6c8da6ed938f12a42f167839100ab551ae8d1
>>
>> Best regards,
> 
> Shin'ichiro, there are some build failures with macOS and Windows:
> 
> https://github.com/axboe/fio/actions/runs/14882253382

Indeed... Guess I should've staged it, in lieu of our CI being
able to pick up and build on-list patches.

Shin'ichiro, please fix those up, thanks.

-- 
Jens Axboe