From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from relay.sgi.com (relay2.corp.sgi.com [137.38.102.29]) by oss.sgi.com (Postfix) with ESMTP id 7A9C27F7E for ; Tue, 9 Apr 2013 00:29:23 -0500 (CDT) Received: from cuda.sgi.com (cuda3.sgi.com [192.48.176.15]) by relay2.corp.sgi.com (Postfix) with ESMTP id 5A069304059 for ; Mon, 8 Apr 2013 22:29:20 -0700 (PDT) Date: Tue, 9 Apr 2013 13:29:13 +0800 From: Eryu Guan Subject: Re: [PATCH 3/3] xfstests generic 310: fix common file path and other cleanups Message-ID: <20130409052913.GF2453@eguan-t400.redhat.com> References: <1365331147-15179-1-git-send-email-eguan@redhat.com> <1365331147-15179-3-git-send-email-eguan@redhat.com> <5162CE93.7010301@sgi.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <5162CE93.7010301@sgi.com> List-Id: XFS Filesystem from SGI List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: xfs-bounces@oss.sgi.com Sender: xfs-bounces@oss.sgi.com To: Rich Johnston Cc: zhaohongjiang@huawei.com, xfs@oss.sgi.com On Mon, Apr 08, 2013 at 09:05:07AM -0500, Rich Johnston wrote: > Hi Eryu, > > Thanks for this cleanup patch. I was going to revert patch > "bbaf78c0" which introduced test generic/310 but will wait and see > if Zhao will provide more information which could be added to this > patch. Make sense, thanks for the review. > > > On 04/07/2013 05:39 AM, Eryu Guan wrote: > >1. add one space between # and test description > > The rest of the changes look good, sorry I missed them when I reviewed ;). > > >2. remove creator/owner info > >3. fix common/rc and common/filter path so they can be sourced correctly > >4. no need to remove $seq.full cause it's not used(or if verbose output > > is needed, $seqres.full should be used) > > > >Signed-off-by: Eryu Guan > >--- > > tests/generic/310 | 12 +++++------- > > 1 file changed, 5 insertions(+), 7 deletions(-) > > > >diff --git a/tests/generic/310 b/tests/generic/310 > >index ef51422..35baa23 100644 > >--- a/tests/generic/310 > >+++ b/tests/generic/310 > >@@ -1,8 +1,8 @@ > > #! /bin/bash > > # FS QA Test No. 310 > > # > >-#Check if there are two threads,one keeps calling read() or lseek(), and > >-#the other calling readdir(), both on the same directory fd. > >+# Check if there are two threads,one keeps calling read() or lseek(), and > >+# the other calling readdir(), both on the same directory fd. > > # > > Hi Zhao, > > I did see both threads running at the same time, but the more I look > at this, the more I am a loss as to what this test is doing. > > Will you expand this a little please. I should have asked for more > justification the first time I reviewed this. Please provide what > bug > this is testing or what failure/weakness this test exposes. If > there is a commit this is related to, please reference it. I didn't look into the test deeply but I found the test seems to run for hours on my 3.9-rc4 test box, not sure what goes wrong here. Thanks, Eryu > > Thanks > --Rich _______________________________________________ xfs mailing list xfs@oss.sgi.com http://oss.sgi.com/mailman/listinfo/xfs