Re: [PATCH 2/4] common: _notrun if _scratch_mkfs_xfs failed

[Christoph Hellwig <hch@lst.de>]

On Sat, Jul 27, 2024 at 01:14:34AM +0800, Zorro Lang wrote:
> So most of cases (on xfs) will be not changed, only those cases call
> _scratch_mkfs_xfs directly will be _notrun (if it fails to mkfs before).

Yes.

> I'm still thinking about if this behavior is needed. Last time we
> just let _scratch_mkfs_sized calls _fail if it fails to mkfs. And
> we hope to get a fail report generally, if a mkfs fails. But
> now we hope to _notrun? With this patchset, some mkfs failures will
> cause _fail, but some will cause _notrun. There'll be two kind of
> behaviors.

Only the ones that have explicit fail code.

But yes, that change is the point of this series - incompatible
options will cause mkfs to fail, and the reason for that is that
the test case can't work for that configuration.  We could try
to explicitly catch those cases, but it would be pretty messy.
(We actually do that for a bunch of tests, and it doesn't look
very nice)