| To: | Eryu Guan <guaneryu@xxxxxxxxx> |
|---|---|
| Subject: | Re: [PATCH] xfstests 273 274 275: do test in dedicated dir and leave test files in it |
| From: | Eric Sandeen <sandeen@xxxxxxxxxxx> |
| Date: | Fri, 27 Jan 2012 15:16:19 -0600 |
| Cc: | xfs@xxxxxxxxxxx |
| In-reply-to: | <1326388815-21181-1-git-send-email-guaneryu@xxxxxxxxx> |
| References: | <1326388815-21181-1-git-send-email-guaneryu@xxxxxxxxx> |
| User-agent: | Mozilla/5.0 (Macintosh; Intel Mac OS X 10.6; rv:9.0) Gecko/20111222 Thunderbird/9.0.1 |
On 1/12/12 11:20 AM, Eryu Guan wrote: > Do all testings in dedicated dir($SCRATCH_MNT/$seq) instead of > $SCRATCH_MNT and don't remove test files in _cleanup() for debug > purpose. > > Signed-off-by: Eryu Guan <guaneryu@xxxxxxxxx> > --- ... > @@ -58,8 +60,9 @@ umount $SCRATCH_DEV 2>/dev/null > _scratch_mkfs_sized $((1 * 1024 * 1024 * 1024)) >>$seq.full 2>&1 > _scratch_mount > > -rm -rf $SCRATCH_MNT/* > -cd $SCRATCH_MNT > +rm -rf $WORK_DIR > +mkdir -p $WORK_DIR Seems like the simplest solution is to just not rm -rf _anything_ immediately after the mkfs. Why bother? All it does is remove lost+found/ ... wouldn't it be simpler to just remove that one line? -Eric > +cd $WORK_DIR > > dd if=/dev/zero of=tmp1 bs=4K count=1 >/dev/null 2>&1 > if [ $? -ne 0 ] |
| <Prev in Thread] | Current Thread | [Next in Thread> |
|---|---|---|
| ||
| Previous by Date: | [PATCH 3/3] filemap: don't call generic_write_sync for -EIOCBQUEUED, Jeff Moyer |
|---|---|
| Next by Date: | Re: XFS for-next branch, Stephen Rothwell |
| Previous by Thread: | Re: [PATCH] xfstests 273 274 275: do test in dedicated dir and leave test files in it, Eric Sandeen |
| Next by Thread: | [PATCH] xfs: remove unecessary AIL push wakeups, JiSheng Zhang |
| Indexes: | [Date] [Thread] [Top] [All Lists] |