Re: [PATCH] fstests: generic/730: exclude btrfs for now

[Christoph Hellwig <hch@infradead.org>]

On Wed, Jun 04, 2025 at 03:55:09PM +0930, Qu Wenruo wrote:
> - Btrfs doesn't support shutdown callbacks
> 
> - The current shutdown callbacks are per-fs

The callsbacks are per-block device.

>   Meanwhile btrfs is a multi-device fs, it needs to know which block
>   device is triggerring shutdown, and needs to do extra evaluation
>   (e.g. can the remaining devices support RW operations) before
>   triggering a full fs shutdown.

Exactly for that reason.

> +++ b/tests/generic/730
> @@ -26,6 +26,10 @@ _require_test
>  _require_block_device $TEST_DEV
>  _require_scsi_debug
>  
> +if [ "$FSTYP" = "btrfs" ]; then
> +	_notrun "btrfs doesn't support per-fs shutdown yet"
> +fi

Please don't add these horrible fs excludes to tests.  Add a helper
in common to check if something is supported.