On 24.02.23 08:26, Naohiro Aota wrote: > On Thu, Feb 23, 2023 at 07:40:35AM -0800, Johannes Thumshirn wrote: >> _require_scratch_dev_pool() zeros the first 100 sectors of each device to >> clear eventual remains of older filesystems. >> >> On zoned devices this creates all sorts of problems, so just skip the >> clearing there. >> >> Signed-off-by: Johannes Thumshirn <johannes.thumshirn@xxxxxxx> >> --- >> common/rc | 4 +++- >> 1 file changed, 3 insertions(+), 1 deletion(-) >> >> diff --git a/common/rc b/common/rc >> index 654730b21ead..d763501be2b2 100644 >> --- a/common/rc >> +++ b/common/rc >> @@ -3461,7 +3461,9 @@ _require_scratch_dev_pool() >> fi >> # to help better debug when something fails, we remove >> # traces of previous btrfs FS on the dev. >> - dd if=/dev/zero of=$i bs=4096 count=100 > /dev/null 2>&1 >> + if [ "`_zone_type "$i"`" = "none" ]; then >> + dd if=/dev/zero of=$i bs=4096 count=100 > /dev/null 2>&1 >> + fi > > How about resetting the first two zones on the zoned device case? > yep that could work.
