From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from cuda.sgi.com (cuda2.sgi.com [192.48.176.25]) by oss.sgi.com (8.14.3/8.14.3/SuSE Linux 0.8) with ESMTP id o4JMQCGS159189 for ; Wed, 19 May 2010 17:26:12 -0500 Received: from mail.internode.on.net (localhost [127.0.0.1]) by cuda.sgi.com (Spam Firewall) with ESMTP id 9447835CC3D for ; Wed, 19 May 2010 15:28:29 -0700 (PDT) Received: from mail.internode.on.net (bld-mail14.adl6.internode.on.net [150.101.137.99]) by cuda.sgi.com with ESMTP id cOqW4ikIORQ9hgiO for ; Wed, 19 May 2010 15:28:29 -0700 (PDT) Date: Thu, 20 May 2010 08:28:13 +1000 From: Dave Chinner Subject: Re: [PATCH 4/7] Add test of quota accounting using fsx Message-ID: <20100519222813.GN8120@dastard> References: <1274276289-8839-1-git-send-email-jack@suse.cz> <1274276289-8839-5-git-send-email-jack@suse.cz> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <1274276289-8839-5-git-send-email-jack@suse.cz> 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 Sender: xfs-bounces@oss.sgi.com Errors-To: xfs-bounces@oss.sgi.com To: Jan Kara Cc: hch@infradead.org, xfs@oss.sgi.com On Wed, May 19, 2010 at 03:38:06PM +0200, Jan Kara wrote: > Run fsx (and also several fsx threads in parallel) and verify that > quota accounting is correct after they finish. > > Signed-off-by: Jan Kara > --- > 231 | 139 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ > 231.out | 10 +++++ > group | 1 + > 3 files changed, 150 insertions(+), 0 deletions(-) > create mode 100755 231 > create mode 100644 231.out > > diff --git a/231 b/231 > new file mode 100755 > index 0000000..0e34d33 > --- /dev/null > +++ b/231 > @@ -0,0 +1,139 @@ > +#! /bin/bash > +# FSQA Test No. 231 > +# > +# Run fsx with quotas enabled and verify accounted quotas in the end > +# > +# Derived from test 127 > +#----------------------------------------------------------------------- > +# Copyright (c) 2006 Silicon Graphics, 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=jack@suse.cz > + > +seq=`basename $0` > +echo "QA output created by $seq" > + > +here=`pwd` > +tmp=/tmp/$$ > +status=1 # failure is the default! > +trap "rm -f $tmp.*; exit \$status" 0 1 2 3 15 > + > +# get standard environment, filters and checks > +. ./common.rc > +. ./common.filter > +. ./common.quota > + > +FSX_FILE_SIZE=64000000 > +FSX_ARGS="-q -l $FSX_FILE_SIZE -o 65536 -S 191110531 -N 20000" My only question about this test is the use of a fixed seed for the test - shouldn't we randomise it and save the seed to $seq.full? This will results in different combinations of operations being tested (better long term coverage), but we still have the seed available to try to reproduce failures if they are based on a specific pattern of ops... As it stands, this would still be fine with a fixed seed... Cheers, Dave. -- Dave Chinner david@fromorbit.com _______________________________________________ xfs mailing list xfs@oss.sgi.com http://oss.sgi.com/mailman/listinfo/xfs