From: Jeff Liu Subject: Re: [PATCH] xfstests: set umask to avoid spurious generic/314 test failures Date: Fri, 13 Dec 2013 12:06:14 +0800 Message-ID: <52AA87B6.6010607@oracle.com> References: <20130921211735.GC7855@wallace> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Cc: linux-ext4@vger.kernel.org To: "xfs@oss.sgi.com" , Eric Whitney Return-path: Received: from userp1040.oracle.com ([156.151.31.81]:51437 "EHLO userp1040.oracle.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751715Ab3LMEHG (ORCPT ); Thu, 12 Dec 2013 23:07:06 -0500 In-Reply-To: <20130921211735.GC7855@wallace> Sender: linux-ext4-owner@vger.kernel.org List-ID: Hi Folks, We missed this patch from Eric for a few months, I observed the same false alarm while analyzing xfstests logs against our next tree... Eric's fix works fine to me. Thanks, -Jeff On 09/22 2013 05:17 AM, Eric Whitney wrote: > Generic/314 can fail when the group write file mode bit for "subdir" does not > match that found in the golden output, as has been seen in ext4 regression > testing. It appears that the golden output for generic/314 was taken on a > system where the $qa_user's umask cleared that mode bit - most likely, where > the umask was 022. Depending upon the distro, it's not uncommon for a user's > default umask to have a different value, such as 002. When that's the case, > we get a false negative failure when the group write mode bit for "subdir" is > not cleared. This failure is unrelated to the value of the SGID mode bit > that is the object of this test. > > We could either require that $qa_user's account be configured in advance with > a umask of 022, or explicitly set a umask value compatible with the golden > output when creating "subdir". The latter option is more robust. > > Signed-off-by: Eric Whitney > --- > tests/generic/314 | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > mode change 100644 => 100755 tests/generic/314 > > diff --git a/tests/generic/314 b/tests/generic/314 > old mode 100644 > new mode 100755 > index 0dd98a3..f430b82 > --- a/tests/generic/314 > +++ b/tests/generic/314 > @@ -57,7 +57,7 @@ chown $qa_user:12345 $TEST_DIR/$seq-dir > chmod 2775 $TEST_DIR/$seq-dir > > # Make subdirs before & after acl set > -su $qa_user -c "mkdir $TEST_DIR/$seq-dir/subdir" > +su $qa_user -c "umask 022; mkdir $TEST_DIR/$seq-dir/subdir" > su $qa_user -c "setfacl -m u:$qa_user:rwx,d:u:$qa_user:rwx $TEST_DIR/$seq-dir" > su $qa_user -c "mkdir $TEST_DIR/$seq-dir/subdir2" > >