Re: [PATCH] common: fix _require_xfs_io_command for specific swapext versions

["Darrick J. Wong" <djwong@kernel.org>]

On Wed, Nov 08, 2023 at 01:33:34PM -0800, Darrick J. Wong wrote:
> On Wed, Nov 08, 2023 at 04:24:01PM +0100, Christoph Hellwig wrote:
> > The xfs_io command (only in Darricks trees and not upstream yet)
> > supports the -v switch to select the swapext version.  When the selected
> > swapext version is not supported it reports a 'version must be $foo'
> > message.  Parse that and don't _notfun the test cases.
> 
> Er... which _require_xfs_io_command call fails?  The only patch that
> adds 'version must' adds both '-v swapext' and '-v exchrange', and
> AFAICT all tests use one or the other if they specify a -v at all.

DOH.  I forgot that the command line switches all got renamed when I
withdrew FIEXCHANGE from consideration as a VFS concept and turned it
back into a private XFS ioctl.

I'm about to email the conversion series for fstests that absorbs that
change.   Would you mind taking a look to see if that solves your
problems?

--D

> --D
> 
> > This fixes various generic tests trying to use the vfs version that
> > doesn't exist in most branches implementing the swapext command.
> > 
> > Signed-off-by: Christoph Hellwig <hch@lst.de>
> > ---
> >  common/rc | 2 ++
> >  1 file changed, 2 insertions(+)
> > 
> > diff --git a/common/rc b/common/rc
> > index 26a26085..d8a0b19b 100644
> > --- a/common/rc
> > +++ b/common/rc
> > @@ -2706,6 +2706,8 @@ _require_xfs_io_command()
> >  			_notrun "xfs_io $command $param ioctl support is missing"
> >  		echo $testio | grep -q "Operation not supported" && \
> >  			_notrun "xfs_io $command $param kernel support is missing"
> > +		echo $testio | grep -q "version must" && \
> > +			_notrun "xfs_io $command $param kernel support is missing"
> >  		rm -f $testfile.1
> >  		param_checked="$param"
> >  		;;
> > -- 
> > 2.39.2
> > 
> > 
>