From: Zorro Lang <zlang@redhat.com>
To: "Darrick J. Wong" <darrick.wong@oracle.com>
Cc: fstests@vger.kernel.org, linux-xfs@vger.kernel.org
Subject: Re: [PATCH v2] xfstests: xfs mount option sanity test
Date: Mon, 16 Dec 2019 17:36:08 +0800 [thread overview]
Message-ID: <20191216093607.GD14328@dhcp-12-102.nay.redhat.com> (raw)
In-Reply-To: <20191216091707.GC14328@dhcp-12-102.nay.redhat.com>
On Mon, Dec 16, 2019 at 05:17:07PM +0800, Zorro Lang wrote:
> On Wed, Dec 11, 2019 at 04:33:20PM +0800, Ian Kent wrote:
> > On Wed, 2019-12-11 at 14:42 +0800, Zorro Lang wrote:
> > > On Thu, Oct 31, 2019 at 07:24:53AM +0800, Zorro Lang wrote:
> > > > On Wed, Oct 30, 2019 at 09:39:22AM -0700, Darrick J. Wong wrote:
> > > > > On Wed, Oct 30, 2019 at 06:34:10PM +0800, Zorro Lang wrote:
> > > > > > XFS is changing to suit the new mount API, so add this case to
> > > > > > make
> > > > > > sure the changing won't bring in regression issue on xfs mount
> > > > > > option
> > > > > > parse phase, and won't change some default behaviors either.
> > > > > >
> > > > > > Signed-off-by: Zorro Lang <zlang@redhat.com>
> > > > > > ---
> > > > > >
> > > > > > Hi,
> > > > > >
> > > > > > V2 did below changes:
> > > > > > 1) Fix wrong output messages in _do_test function
> > > > >
> > > > > Hmm, I still see this on 5.4-rc4:
> > > > >
> > > > > +[FAILED]: mount /dev/loop0 /mnt/148.mnt -o logbsize=16384
> > > > > +ERROR: expect there's logbsize=16k in , but found
> > > > ^^^
> > > > not
> > > >
> > > > > +[FAILED]: mount /dev/loop0 /mnt/148.mnt -o logbsize=16k
> > > > > +ERROR: expect there's logbsize=16k in , but found
> > > > ^^^
> > > > not
> > > >
> > > > Sorry for this typo, I'll fix it.
> > > >
> > > > > Oh, right, you're stripping out MKFS_OPTIONS and formatting a
> > > > > loop
> > > > > device, which on my system means you get rmapbt=1 by default and
> > > > > whatnot.
> > > >
> > > > Hmm... why rmapbt=1 cause logbsize=16k can't be displayed in
> > > > /proc/mounts?
> > > >
> > > > Actually set MKFS_OPTIONS="" is not helpful for this case, due to I
> > > > run
> > > > "$MKFS_XFS_PROG -f $* $LOOP_DEV" directly. I strip out
> > > > MKFS_OPTIONS,
> > > > because I use SCRACH_DEV at first :)
> > > >
> > > > > I think the larger problem here might be that now we have to
> > > > > figure out
> > > > > the special-casing of some of these options.
> > > >
> > > > Maybe we should avoid the testing about those behaviors can't be
> > > > sure, if we
> > > > can't make it have a fixed output.
> > >
> > > It's been long time passed. I still don't have a proper way to
> > > reproduce and
> > > avoid this failure you hit. Does anyone has a better idea for this
> > > case?
> >
> > I think I understand the problem but correct me if I'm wrong.
> >
> > Couldn't you cover both cases, pass an additional parameter that
> > says what the default is if it isn't specified as an option and
> > don't fail if it is present in the options.
> >
> > You could check kernel versions to decide whether to pass the
> > third parameter and so decide if you need to allow for a default
> > option to account for the differing behaviour.
>
> Hi Ian,
>
> It's not a issue about displaying default mount options. It's:
>
> # mount $dev_xfs $mnt -o logbsize=16k
> # findmnt --source $dev_xfs -o OPTIONS -n |grep "logbsize=16k"
> <nothing output>
>
> The case expects there's "logbsize=16k", but it can't find that option.
>
> I don't know how to reproduce it.
Hi Darrick,
Can you still reproduce this failure? Would you mind telling me how to reproduce
it or how to correct my case :-P
The xfs_fs_show_options() function shows:
if (mp->m_logbsize > 0)
seq_printf(m, ",logbsize=%dk", mp->m_logbsize >> 10);
I think the logbsize should be printed if we set '-o logbsize=16k'.
Thanks,
Zorro
>
> Thanks,
> Zorro
>
> >
> > Ian
> > >
> > > Thanks,
> > > Zorro
> > >
> > > > Thanks,
> > > > Zorro
> > > >
> > > > > --D
> > > > >
> > > > > > 2) Remove logbufs=N and logbsize=N default display test.
> > > > > > Lastest upstream
> > > > > > kernel displays these options in /proc/mounts by default,
> > > > > > but old kernel
> > > > > > doesn't show them except user indicate these options when
> > > > > > mount xfs.
> > > > > > Refer to https://marc.info/?l=fstests&m=157199699615477&w=2
> > > > > >
> > > > > > Thanks,
> > > > > > Zorro
> > > > > >
> > > > > > tests/xfs/148 | 320
> > > > > > ++++++++++++++++++++++++++++++++++++++++++++++
> > > > > > tests/xfs/148.out | 6 +
> > > > > > tests/xfs/group | 1 +
> > > > > > 3 files changed, 327 insertions(+)
> > > > > > create mode 100755 tests/xfs/148
> > > > > > create mode 100644 tests/xfs/148.out
> > > > > >
> > > > > > diff --git a/tests/xfs/148 b/tests/xfs/148
> > > > > > new file mode 100755
> > > > > > index 00000000..a662f6f7
> > > > > > --- /dev/null
> > > > > > +++ b/tests/xfs/148
> > > > > > @@ -0,0 +1,320 @@
> > > > > > +#! /bin/bash
> > > > > > +# SPDX-License-Identifier: GPL-2.0
> > > > > > +# Copyright (c) 2019 Red Hat, Inc. All Rights Reserved.
> > > > > > +#
> > > > > > +# FS QA Test 148
> > > > > > +#
> > > > > > +# XFS mount options sanity check, refer to 'man 5 xfs'.
> > > > > > +#
> > > > > > +seq=`basename $0`
> > > > > > +seqres=$RESULT_DIR/$seq
> > > > > > +echo "QA output created by $seq"
> > > > > > +
> > > > > > +here=`pwd`
> > > > > > +tmp=/tmp/$$
> > > > > > +status=1 # failure is the default!
> > > > > > +trap "_cleanup; exit \$status" 0 1 2 3 15
> > > > > > +
> > > > > > +_cleanup()
> > > > > > +{
> > > > > > + cd /
> > > > > > + rm -f $tmp.*
> > > > > > + $UMOUNT_PROG $LOOP_MNT 2>/dev/null
> > > > > > + if [ -n "$LOOP_DEV" ];then
> > > > > > + _destroy_loop_device $LOOP_DEV 2>/dev/null
> > > > > > + fi
> > > > > > + if [ -n "$LOOP_SPARE_DEV" ];then
> > > > > > + _destroy_loop_device $LOOP_SPARE_DEV
> > > > > > 2>/dev/null
> > > > > > + fi
> > > > > > + rm -f $LOOP_IMG
> > > > > > + rm -f $LOOP_SPARE_IMG
> > > > > > + rmdir $LOOP_MNT
> > > > > > +}
> > > > > > +
> > > > > > +# get standard environment, filters and checks
> > > > > > +. ./common/rc
> > > > > > +. ./common/filter
> > > > > > +
> > > > > > +# remove previous $seqres.full before test
> > > > > > +rm -f $seqres.full
> > > > > > +
> > > > > > +# real QA test starts here
> > > > > > +_supported_fs xfs
> > > > > > +_supported_os Linux
> > > > > > +_require_test
> > > > > > +_require_loop
> > > > > > +_require_xfs_io_command "falloc"
> > > > > > +
> > > > > > +LOOP_IMG=$TEST_DIR/$seq.dev
> > > > > > +LOOP_SPARE_IMG=$TEST_DIR/$seq.logdev
> > > > > > +LOOP_MNT=$TEST_DIR/$seq.mnt
> > > > > > +
> > > > > > +echo "** create loop device"
> > > > > > +$XFS_IO_PROG -f -c "falloc 0 1g" $LOOP_IMG
> > > > > > +LOOP_DEV=`_create_loop_device $LOOP_IMG`
> > > > > > +
> > > > > > +echo "** create loop log device"
> > > > > > +$XFS_IO_PROG -f -c "falloc 0 512m" $LOOP_SPARE_IMG
> > > > > > +LOOP_SPARE_DEV=`_create_loop_device $LOOP_SPARE_IMG`
> > > > > > +
> > > > > > +echo "** create loop mount point"
> > > > > > +rmdir $LOOP_MNT 2>/dev/null
> > > > > > +mkdir -p $LOOP_MNT || _fail "cannot create loopback mount
> > > > > > point"
> > > > > > +
> > > > > > +# avoid the effection from MKFS_OPTIONS
> > > > > > +MKFS_OPTIONS=""
> > > > > > +do_mkfs()
> > > > > > +{
> > > > > > + $MKFS_XFS_PROG -f $* $LOOP_DEV | _filter_mkfs
> > > > > > >$seqres.full 2>$tmp.mkfs
> > > > > > + if [ "${PIPESTATUS[0]}" -ne 0 ]; then
> > > > > > + _fail "Fails on _mkfs_dev $* $LOOP_DEV"
> > > > > > + fi
> > > > > > + . $tmp.mkfs
> > > > > > +}
> > > > > > +
> > > > > > +is_dev_mounted()
> > > > > > +{
> > > > > > + findmnt --source $LOOP_DEV >/dev/null
> > > > > > + return $?
> > > > > > +}
> > > > > > +
> > > > > > +get_mount_info()
> > > > > > +{
> > > > > > + findmnt --source $LOOP_DEV -o OPTIONS -n
> > > > > > +}
> > > > > > +
> > > > > > +force_unmount()
> > > > > > +{
> > > > > > + $UMOUNT_PROG $LOOP_MNT >/dev/null 2>&1
> > > > > > +}
> > > > > > +
> > > > > > +# _do_test <mount options> <should be mounted?> [<key string>
> > > > > > <key should be found?>]
> > > > > > +_do_test()
> > > > > > +{
> > > > > > + local opts="$1"
> > > > > > + local mounted="$2" # pass or fail
> > > > > > + local key="$3"
> > > > > > + local found="$4" # true or false
> > > > > > + local rc
> > > > > > + local info
> > > > > > +
> > > > > > + # mount test
> > > > > > + _mount $LOOP_DEV $LOOP_MNT $opts 2>/dev/null
> > > > > > + rc=$?
> > > > > > + if [ $rc -eq 0 ];then
> > > > > > + if [ "${mounted}" = "fail" ];then
> > > > > > + echo "[FAILED]: mount $LOOP_DEV
> > > > > > $LOOP_MNT $opts"
> > > > > > + echo "ERROR: expect ${mounted}, but
> > > > > > pass"
> > > > > > + return 1
> > > > > > + fi
> > > > > > + is_dev_mounted
> > > > > > + if [ $? -ne 0 ];then
> > > > > > + echo "[FAILED]: mount $LOOP_DEV
> > > > > > $LOOP_MNT $opts"
> > > > > > + echo "ERROR: fs not mounted even mount
> > > > > > return 0"
> > > > > > + return 1
> > > > > > + fi
> > > > > > + else
> > > > > > + if [ "${mount_ret}" = "pass" ];then
> > > > > > + echo "[FAILED]: mount $LOOP_DEV
> > > > > > $LOOP_MNT $opts"
> > > > > > + echo "ERROR: expect ${mounted}, but
> > > > > > fail"
> > > > > > + return 1
> > > > > > + fi
> > > > > > + is_dev_mounted
> > > > > > + if [ $? -eq 0 ];then
> > > > > > + echo "[FAILED]: mount $LOOP_DEV
> > > > > > $LOOP_MNT $opts"
> > > > > > + echo "ERROR: fs is mounted even mount
> > > > > > return non-zero"
> > > > > > + return 1
> > > > > > + fi
> > > > > > + fi
> > > > > > +
> > > > > > + # Skip below checking if "$key" argument isn't
> > > > > > specified
> > > > > > + if [ -z "$key" ];then
> > > > > > + return 0
> > > > > > + fi
> > > > > > + # Check the mount options after fs mounted.
> > > > > > + info=`get_mount_info`
> > > > > > + echo $info | grep -q "${key}"
> > > > > > + rc=$?
> > > > > > + if [ $rc -eq 0 ];then
> > > > > > + if [ "$found" != "true" ];then
> > > > > > + echo "[FAILED]: mount $LOOP_DEV
> > > > > > $LOOP_MNT $opts"
> > > > > > + echo "ERROR: expect there's not $key in
> > > > > > $info, but not found"
> > > > > > + return 1
> > > > > > + fi
> > > > > > + else
> > > > > > + if [ "$found" != "false" ];then
> > > > > > + echo "[FAILED]: mount $LOOP_DEV
> > > > > > $LOOP_MNT $opts"
> > > > > > + echo "ERROR: expect there's $key in
> > > > > > $info, but found"
> > > > > > + return 1
> > > > > > + fi
> > > > > > + fi
> > > > > > +
> > > > > > + return 0
> > > > > > +}
> > > > > > +
> > > > > > +do_test()
> > > > > > +{
> > > > > > + # force unmount before testing
> > > > > > + force_unmount
> > > > > > + _do_test "$@"
> > > > > > + # force unmount after testing
> > > > > > + force_unmount
> > > > > > +}
> > > > > > +
> > > > > > +echo "** start xfs mount testing ..."
> > > > > > +# Test allocsize=size
> > > > > > +# Valid values for this option are page size (typically 4KiB)
> > > > > > through to 1GiB
> > > > > > +do_mkfs
> > > > > > +if [ $dbsize -ge 1024 ];then
> > > > > > + blsize="$((dbsize / 1024))k"
> > > > > > +fi
> > > > > > +do_test "" pass "allocsize" "false"
> > > > > > +do_test "-o allocsize=$blsize" pass "allocsize=$blsize" "true"
> > > > > > +do_test "-o allocsize=1048576k" pass "allocsize=1048576k"
> > > > > > "true"
> > > > > > +do_test "-o allocsize=$((dbsize / 2))" fail
> > > > > > +do_test "-o allocsize=2g" fail
> > > > > > +
> > > > > > +# Test attr2
> > > > > > +do_mkfs -m crc=1
> > > > > > +do_test "" pass "attr2" "true"
> > > > > > +do_test "-o attr2" pass "attr2" "true"
> > > > > > +do_test "-o noattr2" fail
> > > > > > +do_mkfs -m crc=0
> > > > > > +do_test "" pass "attr2" "true"
> > > > > > +do_test "-o attr2" pass "attr2" "true"
> > > > > > +do_test "-o noattr2" pass "attr2" "false"
> > > > > > +
> > > > > > +# Test discard
> > > > > > +do_mkfs
> > > > > > +do_test "" pass "discard" "false"
> > > > > > +do_test "-o discard" pass "discard" "true"
> > > > > > +do_test "-o nodiscard" pass "discard" "false"
> > > > > > +
> > > > > > +# Test grpid|bsdgroups|nogrpid|sysvgroups
> > > > > > +do_test "" pass "grpid" "false"
> > > > > > +do_test "-o grpid" pass "grpid" "true"
> > > > > > +do_test "-o bsdgroups" pass "grpid" "true"
> > > > > > +do_test "-o nogrpid" pass "grpid" "false"
> > > > > > +do_test "-o sysvgroups" pass "grpid" "false"
> > > > > > +
> > > > > > +# Test filestreams
> > > > > > +do_test "" pass "filestreams" "false"
> > > > > > +do_test "-o filestreams" pass "filestreams" "true"
> > > > > > +
> > > > > > +# Test ikeep
> > > > > > +do_test "" pass "ikeep" "false"
> > > > > > +do_test "-o ikeep" pass "ikeep" "true"
> > > > > > +do_test "-o noikeep" pass "ikeep" "false"
> > > > > > +
> > > > > > +# Test inode32|inode64
> > > > > > +do_test "" pass "inode64" "true"
> > > > > > +do_test "-o inode32" pass "inode32" "true"
> > > > > > +do_test "-o inode64" pass "inode64" "true"
> > > > > > +
> > > > > > +# Test largeio
> > > > > > +do_test "" pass "largeio" "false"
> > > > > > +do_test "-o largeio" pass "largeio" "true"
> > > > > > +do_test "-o nolargeio" pass "largeio" "false"
> > > > > > +
> > > > > > +# Test logbufs=value. Valid numbers range from 2–8 inclusive.
> > > > > > +# New kernel (refer to 4f62282a3696 xfs: cleanup
> > > > > > xlog_get_iclog_buffer_size)
> > > > > > +# prints "logbufs=N" in /proc/mounts, but old kernel not. So
> > > > > > the default
> > > > > > +# 'display' about logbufs can't be expected, disable this
> > > > > > test.
> > > > > > +#do_test "" pass "logbufs" "false"
> > > > > > +do_test "-o logbufs=8" pass "logbufs=8" "true"
> > > > > > +do_test "-o logbufs=2" pass "logbufs=2" "true"
> > > > > > +do_test "-o logbufs=1" fail
> > > > > > +do_test "-o logbufs=9" fail
> > > > > > +do_test "-o logbufs=99999999999999" fail
> > > > > > +
> > > > > > +# Test logbsize=value.
> > > > > > +do_mkfs -m crc=1 -l version=2
> > > > > > +# New kernel (refer to 4f62282a3696 xfs: cleanup
> > > > > > xlog_get_iclog_buffer_size)
> > > > > > +# prints "logbsize=N" in /proc/mounts, but old kernel not. So
> > > > > > the default
> > > > > > +# 'display' about logbsize can't be expected, disable this
> > > > > > test.
> > > > > > +#do_test "" pass "logbsize" "false"
> > > > > > +do_test "-o logbsize=16384" pass "logbsize=16k" "true"
> > > > > > +do_test "-o logbsize=16k" pass "logbsize=16k" "true"
> > > > > > +do_test "-o logbsize=32k" pass "logbsize=32k" "true"
> > > > > > +do_test "-o logbsize=64k" pass "logbsize=64k" "true"
> > > > > > +do_test "-o logbsize=128k" pass "logbsize=128k" "true"
> > > > > > +do_test "-o logbsize=256k" pass "logbsize=256k" "true"
> > > > > > +do_test "-o logbsize=8k" fail
> > > > > > +do_test "-o logbsize=512k" fail
> > > > > > +do_mkfs -m crc=0 -l version=1
> > > > > > +# New kernel (refer to 4f62282a3696 xfs: cleanup
> > > > > > xlog_get_iclog_buffer_size)
> > > > > > +# prints "logbsize=N" in /proc/mounts, but old kernel not. So
> > > > > > the default
> > > > > > +# 'display' about logbsize can't be expected, disable this
> > > > > > test.
> > > > > > +#do_test "" pass "logbsize" "false"
> > > > > > +do_test "-o logbsize=16384" pass "logbsize=16k" "true"
> > > > > > +do_test "-o logbsize=16k" pass "logbsize=16k" "true"
> > > > > > +do_test "-o logbsize=32k" pass "logbsize=32k" "true"
> > > > > > +do_test "-o logbsize=64k" fail
> > > > > > +
> > > > > > +# Test logdev
> > > > > > +do_mkfs
> > > > > > +do_test "" pass "logdev" "false"
> > > > > > +do_test "-o logdev=$LOOP_SPARE_DEV" fail
> > > > > > +do_mkfs -l logdev=$LOOP_SPARE_DEV
> > > > > > +do_test "-o logdev=$LOOP_SPARE_DEV" pass
> > > > > > "logdev=$LOOP_SPARE_DEV" "true"
> > > > > > +do_test "" fail
> > > > > > +
> > > > > > +# Test noalign
> > > > > > +do_mkfs
> > > > > > +do_test "" pass "noalign" "false"
> > > > > > +do_test "-o noalign" pass "noalign" "true"
> > > > > > +
> > > > > > +# Test norecovery
> > > > > > +do_test "" pass "norecovery" "false"
> > > > > > +do_test "-o norecovery,ro" pass "norecovery" "true"
> > > > > > +do_test "-o norecovery" fail
> > > > > > +
> > > > > > +# Test nouuid
> > > > > > +do_test "" pass "nouuid" "false"
> > > > > > +do_test "-o nouuid" pass "nouuid" "true"
> > > > > > +
> > > > > > +# Test noquota
> > > > > > +do_test "" pass "noquota" "true"
> > > > > > +do_test "-o noquota" pass "noquota" "true"
> > > > > > +
> > > > > > +# Test uquota/usrquota/quota/uqnoenforce/qnoenforce
> > > > > > +do_test "" pass "usrquota" "false"
> > > > > > +do_test "-o uquota" pass "usrquota" "true"
> > > > > > +do_test "-o usrquota" pass "usrquota" "true"
> > > > > > +do_test "-o quota" pass "usrquota" "true"
> > > > > > +do_test "-o uqnoenforce" pass "usrquota" "true"
> > > > > > +do_test "-o qnoenforce" pass "usrquota" "true"
> > > > > > +
> > > > > > +# Test gquota/grpquota/gqnoenforce
> > > > > > +do_test "" pass "grpquota" "false"
> > > > > > +do_test "-o gquota" pass "grpquota" "true"
> > > > > > +do_test "-o grpquota" pass "grpquota" "true"
> > > > > > +do_test "-o gqnoenforce" pass "gqnoenforce" "true"
> > > > > > +
> > > > > > +# Test pquota/prjquota/pqnoenforce
> > > > > > +do_test "" pass "prjquota" "false"
> > > > > > +do_test "-o pquota" pass "prjquota" "true"
> > > > > > +do_test "-o prjquota" pass "prjquota" "true"
> > > > > > +do_test "-o pqnoenforce" pass "pqnoenforce" "true"
> > > > > > +
> > > > > > +# Test sunit=value and swidth=value
> > > > > > +do_mkfs -d sunit=128,swidth=128
> > > > > > +do_test "-o sunit=8,swidth=8" pass "sunit=8,swidth=8" "true"
> > > > > > +do_test "-o sunit=8,swidth=64" pass "sunit=8,swidth=64" "true"
> > > > > > +do_test "-o sunit=128,swidth=128" pass "sunit=128,swidth=128"
> > > > > > "true"
> > > > > > +do_test "-o sunit=256,swidth=256" pass "sunit=256,swidth=256"
> > > > > > "true"
> > > > > > +do_test "-o sunit=2,swidth=2" fail
> > > > > > +
> > > > > > +# Test swalloc
> > > > > > +do_mkfs
> > > > > > +do_test "" pass "swalloc" "false"
> > > > > > +do_test "-o swalloc" pass "swalloc" "true"
> > > > > > +
> > > > > > +# Test wsync
> > > > > > +do_test "" pass "wsync" "false"
> > > > > > +do_test "-o wsync" pass "wsync" "true"
> > > > > > +
> > > > > > +echo "** end of testing"
> > > > > > +# success, all done
> > > > > > +status=0
> > > > > > +exit
> > > > > > diff --git a/tests/xfs/148.out b/tests/xfs/148.out
> > > > > > new file mode 100644
> > > > > > index 00000000..a71d9231
> > > > > > --- /dev/null
> > > > > > +++ b/tests/xfs/148.out
> > > > > > @@ -0,0 +1,6 @@
> > > > > > +QA output created by 148
> > > > > > +** create loop device
> > > > > > +** create loop log device
> > > > > > +** create loop mount point
> > > > > > +** start xfs mount testing ...
> > > > > > +** end of testing
> > > > > > diff --git a/tests/xfs/group b/tests/xfs/group
> > > > > > index f4ebcd8c..019aebad 100644
> > > > > > --- a/tests/xfs/group
> > > > > > +++ b/tests/xfs/group
> > > > > > @@ -145,6 +145,7 @@
> > > > > > 145 dmapi
> > > > > > 146 dmapi
> > > > > > 147 dmapi
> > > > > > +148 auto quick mount
> > > > > > 150 dmapi
> > > > > > 151 dmapi
> > > > > > 152 dmapi
> > > > > > --
> > > > > > 2.20.1
> > > > > >
> >
next prev parent reply other threads:[~2019-12-16 9:28 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-10-30 10:34 [PATCH v2] xfstests: xfs mount option sanity test Zorro Lang
2019-10-30 16:39 ` Darrick J. Wong
2019-10-30 23:24 ` Zorro Lang
2019-12-11 6:42 ` Zorro Lang
2019-12-11 8:33 ` Ian Kent
2019-12-16 9:17 ` Zorro Lang
2019-12-16 9:36 ` Zorro Lang [this message]
2020-01-13 15:40 ` Zorro Lang
2020-01-13 17:33 ` Darrick J. Wong
2019-12-16 10:42 ` Ian Kent
2020-01-13 18:00 ` Darrick J. Wong
2020-01-13 18:23 ` Darrick J. Wong
2020-01-14 6:44 ` Zorro Lang
2020-01-14 6:41 ` Darrick J. Wong
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20191216093607.GD14328@dhcp-12-102.nay.redhat.com \
--to=zlang@redhat.com \
--cc=darrick.wong@oracle.com \
--cc=fstests@vger.kernel.org \
--cc=linux-xfs@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).