TBH, I don't know if this is posix-specified, but I found out the hard way that when trying to re-create existing files on a readonly filesystem, some apps expect/handle EEXIST, but fail on EROFS. This will test mkdir, mknod, and symlinks for that behavior. Signed-off-by: Eric Sandeen <sandeen@xxxxxxxxxx> --- V2: Change test dir var name, do scratch mkfs, drop _fail diff --git a/292 b/292 new file mode 100755 index 0000000..4f0cc87 --- /dev/null +++ b/292 @@ -0,0 +1,74 @@ +#! /bin/bash +# FS QA Test No. 292 +# +# Tests for EEXIST (not EROFS) for inode creations, if +# we ask to create an already-existing entity on an RO filesystem +# +#----------------------------------------------------------------------- +# Copyright (c) 2012 Red Hat, Inc. All Rights Reserved. +# +# This program is free software; you can redistribute it and/or +# modify it under the terms of the GNU General Public License as +# published by the Free Software Foundation. +# +# This program is distributed in the hope that it would be useful, +# but WITHOUT ANY WARRANTY; without even the implied warranty of +# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the +# GNU General Public License for more details. +# +# You should have received a copy of the GNU General Public License +# along with this program; if not, write the Free Software Foundation, +# Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA +#----------------------------------------------------------------------- +# +# creator +owner=sandeen@xxxxxxxxxx + +seq=`basename $0` +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.* +} + +# get standard environment, filters and checks +. ./common.rc +. ./common.filter + +# real QA test starts here + +# Modify as appropriate. +_supported_fs generic +_supported_os Linux +_require_scratch + +MYDIR=$SCRATCH_MNT/$seq.test + +_create_files() +{ + mknod $MYDIR/testnode c 1 3 + mkdir $MYDIR/testdir + touch $MYDIR/testtarget + ln -s $MYDIR/testtarget $MYDIR/testlink +} + +_scratch_mkfs >/dev/null 2>&1 +_scratch_mount + +rm -rf $MYDIR +mkdir $MYDIR + +_create_files 2>&1 | _filter_scratch +_scratch_mount -o remount,ro +_create_files 2>&1 | _filter_scratch + +# success, all done +status=0 +exit diff --git a/292.out b/292.out new file mode 100644 index 0000000..485c567 --- /dev/null +++ b/292.out @@ -0,0 +1,5 @@ +QA output created by 292 +mknod: `SCRATCH_MNT/292.test/testnode': File exists +mkdir: cannot create directory `SCRATCH_MNT/292.test/testdir': File exists +touch: cannot touch `SCRATCH_MNT/292.test/testtarget': Read-only file system +ln: creating symbolic link `SCRATCH_MNT/292.test/testlink': File exists diff --git a/group b/group index dc8db65..030082b 100644 --- a/group +++ b/group @@ -410,3 +410,4 @@ deprecated 289 auto quick 290 auto rw prealloc quick ioctl 291 repair +292 auto quick _______________________________________________ xfs mailing list xfs@xxxxxxxxxxx http://oss.sgi.com/mailman/listinfo/xfs