From mboxrd@z Thu Jan 1 00:00:00 1970 From: Pavel Shilovsky Subject: Re: [PATCH v2 3/3] cifs: skip tests that need POSIX support for nounix mounts Date: Wed, 27 Aug 2014 23:49:56 +0400 Message-ID: <2DA51A6D-2D91-4582-831B-23C997C37543@gmail.com> References: <1409084918-17764-1-git-send-email-pshilovsky@samba.org> <1409084918-17764-4-git-send-email-pshilovsky@samba.org> <20140827155832.GA3103@infradead.org> Mime-Version: 1.0 (1.0) Content-Type: text/plain; charset=koi8-r Content-Transfer-Encoding: QUOTED-PRINTABLE Cc: Pavel Shilovsky , "fstests-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , "linux-cifs-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , "samba-technical-eUNUBHrolfbYtjvyW6yDsg@public.gmane.org" , David Disseldorp , Steve French To: Christoph Hellwig Return-path: In-Reply-To: <20140827155832.GA3103-wEGCiKHe2LqWVfeAwA7xHQ@public.gmane.org> Sender: linux-cifs-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-ID: > 27 =C1=D7=C7. 2014 =C7., =D7 19:58, Christoph Hellwig =CE=C1=D0=C9=D3=C1=CC(=C1): >=20 >> On Wed, Aug 27, 2014 at 12:28:38AM +0400, Pavel Shilovsky wrote: >> CIFS/SMB protocol without POSIX extensions doesn't support operation= s >> with symbolic links and advisory byte-range locks from the same proc= ess. >> Add a check for nounix mounts and use it in generic tests that >> require such operations. >=20 > +_require_test_posix_ext seems very cifs specific. Can you take > a look at the tests and see what posix feature they require and > add features based on that? Let's have a quick discussion here on th= e > requirements of the tests before even writing the code. >=20 Agree. >> diff --git a/tests/generic/005 b/tests/generic/005 >> index d78e43f..0c2b51f 100755 >> --- a/tests/generic/005 >> +++ b/tests/generic/005 >> @@ -67,6 +67,7 @@ _touch() >> # real QA test starts here >> _supported_fs generic >> _require_test >> +_require_test_posix_ext >>=20 >> # IRIX UDF does not support symlinks >> if [ $FSTYP =3D=3D 'udf' ]; then >=20 > this suggest 005 needs symlinks and plain cifs doesn't support them. > We should also fold this test for IRIX udf into the _requires_symlink > tests. >=20 >> diff --git a/tests/generic/023 b/tests/generic/023 >> index 114485c..91b8a37 100755 >> --- a/tests/generic/023 >> +++ b/tests/generic/023 >> @@ -45,6 +45,7 @@ _supported_os Linux >>=20 >> _require_test >> _requires_renameat2 >> +_require_test_posix_ext >=20 >=20 > 023-025 just require a working renameat2, and nothing in Posix. What= 's > the problem for cifs here? These tests try to create symlinks and then rename them. >=20 >> diff --git a/tests/generic/131 b/tests/generic/131 >> index b4e3ff0..9736963 100755 >> --- a/tests/generic/131 >> +++ b/tests/generic/131 >> @@ -45,6 +45,7 @@ _cleanup() >> _supported_fs generic >> _supported_os Linux >> _require_test >> +_require_test_posix_ext >>=20 >> TESTFILE=3D$TEST_DIR/lock_file >=20 > 131 tests fcntl style file locking, so we should test for that. >=20 > -- > To unsubscribe from this list: send the line "unsubscribe linux-cifs"= in > the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org > More majordomo info at http://vger.kernel.org/majordomo-info.html So for these tests we need two check functions: _require_symlink for 00= 5, 023, 024, 025 and _require_fcntl for 131. Right?