Re: [PATCH v3 3/6] badblocks: improvement badblocks_set() for multiple ranges handling

[Coly Li <colyli@suse.de>]

On 12/10/21 2:37 AM, Wols Lists wrote:
> On 09/12/2021 07:28, Geliang Tang wrote:
>> On Thu, Dec 02, 2021 at 08:52:41PM +0800, Coly Li wrote:
>>> Recently I received a bug report that current badblocks code does not
>
>>
>>> + *        +--------+----+
>>> + *        |    S   | E  |
>>> + *        +--------+----+
>>> + * 2.2) The setting range size == already set range size
>>> + * 2.2.1) If S and E are both acked or unacked range, the setting 
>>> range S can
>>> + *    be merged into existing bad range E. The result is,
>>> + *        +-------------+
>>> + *        |      S      |
>>> + *        +-------------+
>>> + * 2.2.2) If S is uncked setting and E is acked, the setting will 
>>> be denied, and
>>
>> uncked -> unacked
>>
>>> + *    the result is,
>>> + *        +-------------+
>>> + *        |      E      |
>>> + *        +-------------+
>>> + * 2.2.3) If S is acked setting and E is unacked, range S can 
>>> overwirte all of
>
> overwirte -> overwrite

Thanks, I update my local codespell dictionary file with all the typo 
checks. They will be fixed in next version.

Coly Li