Fix that the function libzbc_report_zones fails prematurely when encountering SOBR zone types if using libzbc engine

[rorychen]

Support random write on SWR zones on Hybrid SMR drives with libzbc engine

Hybrid SMR has SWR zones. This change can support libzbc engine to run random write workload on SWR zones of Hybrid SMR drives. Require libzbc revision >= 6.0.

Signed-off-by: Rory Chen rory.c.chen@seagate.com

[vincentkfu]

@kawasaki @damien-lemoal

[vincentkfu]

@rorychen please investigate the build errors.

[rorychen]

Hi Vincent,

I fixed it and verified the compile on bench. Sorry for wrong characters in last code.

[damien-lemoal]

While at it, please fix the typo: Sequential or Before Required
Also, this patch should be folded into the previous one.

[damien-lemoal]

The patch title does not make sense given the changes. All I see is that you are handling the SOBR zone type as a valid one, but there is nothing related to "running" nor to "SWR" in these changes.

Furthermore, "Hybrid SMR" is not a technical/standard term. Please refer to ZBC/ZAC "Zone Domains" or "Zone Realms" feature.

Finally, your patch also has not commit message nor a Signed-off-by tag. The commit message should really be added to explain what is missing and how your change improves support for Zone Domains/Zone Realms drives.

[rorychen]

The patch title does not make sense given the changes. All I see is that you are handling the SOBR zone type as a valid one, but there is nothing related to "running" nor to "SWR" in these changes.

Furthermore, "Hybrid SMR" is not a technical/standard term. Please refer to ZBC/ZAC "Zone Domains" or "Zone Realms" feature.

Finally, your patch also has not commit message nor a Signed-off-by tag. The commit message should really be added to explain what is missing and how your change improves support for Zone Domains/Zone Realms drives.

Thanks for the suggestion. Yes, it's not for the fix related to SWR and the patch title is not accurate. I reword the commit message and combine 2 commits to one new commit. The background is that I'd like to run fio with libzbc engine on SWR zones on a SATA HDD drive with Zone Domain feature enabled but fio aborted unexpectedly. The reason is that current function libzbc_report_zones lacks the handling for SOBR zone type so fio would quit early when processing zone information after reporting the zones from one SATA HDD with Zone Domain feature enabled.

[damien-lemoal]

Your patch format is not correct: the entire commit message is the commit title. Please see "git log" for examples of correctly formatted commits: the first line is the title, followed by a blank line and the commit message. After that, add a blank line and your Signed-off-by tag.