Whamcloud - gitweb
fs/lustre-release.git
12 years agoLU-1571 mdt: Do not update xid for open replay req
Wang Di [Fri, 11 Jan 2013 21:41:38 +0000 (16:41 -0500)]
LU-1571 mdt: Do not update xid for open replay req

Do not update last_xid for open replay req,
otherwise the following resend(after replay)
can not be matched with correct xid.

Remove unnecessary mti_transo zero check in
mdt_empty_transno.

Original Change-Id: I2a05f3ac05b301ae31641a1dc51f8c4eed96427d
Original Reviewed-on: http://review.whamcloud.com/3195
cherry-picked from 3402e6c6517cc15f159b19bf4462279285cd83f2

Change form Original:
Fixed mutex change in mdt_empty_transno()

Signed-off-by: Nathaniel Clark <nathaniel.l.clark@intel.com>
Signed-off-by: wang di <di.wang@whamcloud.com>
Reviewed-by: Andreas Dilger <adilger@whamcloud.com>
Reviewed-by: Alex Zhuravlev <bzzz@whamcloud.com>
Change-Id: I9a26b25ae88e0ae2299975ff4abce9b47289fb17
Reviewed-on: http://review.whamcloud.com/5003
Reviewed-by: Alex Zhuravlev <alexey.zhuravlev@intel.com>
Tested-by: Hudson
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
12 years agoLU-971 mdt: fix open resent issue with last_xid
Mikhail Pershin [Fri, 11 Jan 2013 21:38:26 +0000 (16:38 -0500)]
LU-971 mdt: fix open resent issue with last_xid

req_xid_is_last() compares xid with one in client data,
but in master it is not updated in case of open without create.
Patch updates export lcd with last xid, transno and rc for open
requests.

Original Change-Id: I9e769fecc1e93c7a556d4b9eeadc80a70369e901
Original Reviewed-on: http://review.whamcloud.com/1930
Cherry-picked ed5e47ac21b02abb16b10446dac7dae722259f0a

Signed-off-by: Mikhail Pershin <tappro@whamcloud.com>
Reviewed-by: Yu Jian <yujian@whamcloud.com>
Reviewed-by: Li Wei <liwei@whamcloud.com>
Reviewed-by: Oleg Drokin <green@whamcloud.com>
Change-Id: I015aa109bef65f6207f462358e40cae6e6092cc7
Reviewed-on: http://review.whamcloud.com/5002
Reviewed-by: Jian Yu <jian.yu@intel.com>
Tested-by: Hudson
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: Li Wei <wei.g.li@intel.com>
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
Reviewed-by: Mike Pershin <mike.pershin@intel.com>
12 years agoLU-744 obdclass: revise cl_page refcount
Gregoire Pichon [Tue, 12 Feb 2013 13:52:54 +0000 (14:52 +0100)]
LU-744 obdclass: revise cl_page refcount

By holding a refcount for radix tree, we can remove a couple of
spin lock and atomic operation for cl_page.

Signed-off-by: Jinshan Xiong <jinshan.xiong@intel.com>
Signed-off-by: Gregoire Pichon <gregoire.pichon@bull.net>
Change-Id: I4d8dc04b9b4ef066b54a90be3cfbd0c09d96c320
Reviewed-on: http://review.whamcloud.com/5429
Tested-by: Hudson
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: Bobi Jam <bobijam@gmail.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
12 years agoLU-2591 lov: race between ptlrpc_rcv and umount/umount thread
Hiroya Nozaki [Tue, 26 Feb 2013 01:54:59 +0000 (10:54 +0900)]
LU-2591 lov: race between ptlrpc_rcv and umount/umount thread

The race which is refered here happens in the following scenario

1) mount runs but fails to communicate with some OSTs. Then the
   import objects which represent the OSTs are registered to
   a pinger list.
2) pinger succeeds to communicate with, at least, ONE OST. Then
   ptlrpc_rcv calls lov_set_osc_active to activate the OST and
   holds lov_refcount.
3) For some reason ... possibly mount finally fails or umount runs,
   ll_put_super is called
4) ll_put_super tries to disconnect all the OSTs with
   lov_disconnect and this func calls lov_del_target to set all
   the OSC's target->ltd_reap flags in order for lov_putref to
   handle all of them.
5) ptlrpc_rcv thread puts lov_refcount and if lov_refcount becomes
   0 here, the thread has to disconnect all the OSCs whose ltd->reap
   has been set.
6) Some OSCs' imports have still been LUSTRE_IMP_CONNECTING state
   because of (2), so ptlrpc_rcv thread has to wait for these
   import states to be changed to non-recovery states, such as FULL,
   CLOSED or DISCON at ptlrpc_disconnect_import.

Now that ptlrpc_rcv thread is waiting for the import states to be
changed to non-recovery states but ptlrpc_rcv is the one who is
supposed to change a recovery state to a non-recovery state, So
ptlrpc_rcv must hung, And mount/umount thread which has called
ll_put_super has to also wait for changing state at
ptlrpc_disconnect_import, so that thread must hung too.

Signed-off-by: Hiroya Nozaki <nozaki.hiroya@jp.fujitsu.com>
Change-Id: I1d74967b883f079aafd11454dca44960a8f7a3f2
Reviewed-on: http://review.whamcloud.com/5527
Tested-by: Hudson
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
12 years agoLU-2183 quota: update tests for DNE
Niu Yawei [Fri, 1 Feb 2013 04:16:36 +0000 (23:16 -0500)]
LU-2183 quota: update tests for DNE

Update test_13 of sq since the osp is changed to lwp in DNE.

Signed-off-by: Niu Yawei <yawei.niu@intel.com>
Change-Id: I4ce0522aa1de46a02d2084f329471551fce3ee90
Reviewed-on: http://review.whamcloud.com/5240
Reviewed-by: Johann Lombardi <johann.lombardi@intel.com>
Tested-by: Hudson
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: wangdi <di.wang@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
12 years agoLU-1727 lprocfs: fix an old assertion
Bobi Jam [Fri, 10 Aug 2012 06:22:45 +0000 (14:22 +0800)]
LU-1727 lprocfs: fix an old assertion

lprocfs_counter_add() can be called in interrupt context, as a memory
allocation can trigger memory shrinker call ldlm_pool_shrink(), which
calls lprocfs_counter_add().

Signed-off-by: Bobi Jam <bobijam@whamcloud.com>
Change-Id: I0f5d3e07c234f7d615ca87761a7bd6367e72beac
Reviewed-on: http://review.whamcloud.com/3729
Tested-by: Hudson
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
12 years agoLU-2072 ldiskfs: Leak of s_mb_prealloc_table in ldiskfs
Emoly Liu [Mon, 18 Feb 2013 18:15:28 +0000 (02:15 +0800)]
LU-2072 ldiskfs: Leak of s_mb_prealloc_table in ldiskfs

ext4-prealloc-rhel[5,6] patches adds kmalloc'ed
sbi->s_mb_prealloc_table field which is never freed
in [ldiskfs,ext4]_mb_release()/ext4-vmalloc-rhel[5,6].patch

port of master patch 536464e609c46a3950d7742bc3860b87d6bfc726

Signed-off-by: Bruno Faccini <bruno.faccini@intel.com>
Signed-off-by: Liu Ying <emoly.liu@intel.com>
Change-Id: I0acd9f62d6ffc516540d707830b40c0bd211a43b
Reviewed-on: http://review.whamcloud.com/5474
Tested-by: Hudson
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
12 years agoLU-1484 lprocfs: enable HAVE_PROCFS_USERS for rhel kernels
Nathaniel Clark [Thu, 14 Feb 2013 21:27:05 +0000 (16:27 -0500)]
LU-1484 lprocfs: enable HAVE_PROCFS_USERS for rhel kernels

For rhel kernels, given only the kernel-devel rpm it is not possible
to tell if proc_dir_entry_aux is defined, so assume it is.  It has
been included since late in the 5.x release cycle.

Originally Change-Id: If17ecb0902ec90a1af6228d2b9b1b72bc68a6672

Test-Parameters: envdefinitions=SLOW=yes clientdistro=el5   serverdistro=el5 clientarch=x86_64 serverarch=x86_64   testlist=recovery-small
Signed-off-by: Nathaniel Clark <nathaniel.l.clark@intel.com>
Change-Id: I7cde85592759abe020ff31fbf4840717b3e95bef
Reviewed-on: http://review.whamcloud.com/5468
Reviewed-by: Bobi Jam <bobijam@gmail.com>
Tested-by: Hudson
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: Jian Yu <jian.yu@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
12 years agoLU-2601 kernel: kernel update [RHEL5.9 2.6.18-348.1.1.el5]
yangsheng [Sun, 17 Feb 2013 18:59:18 +0000 (02:59 +0800)]
LU-2601 kernel: kernel update [RHEL5.9 2.6.18-348.1.1.el5]

Update RHEL5.9 kernel to 2.6.18-348.1.1.el5.
Adding scsi/fc_compat.h to lustre-lnet.m4 and o2iblnd.h to build
o2ib modules.

Signed-off-by: Shuichi Ihara <sihara@ddn.com>
Signed-off-by: yang sheng <ys@whamcloud.com>
Change-Id: I1d66d12f64238cb410288715ff389ab96b15f0f3
Reviewed-on: http://review.whamcloud.com/5451
Tested-by: Hudson
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: Bobi Jam <bobijam@gmail.com>
Reviewed-by: Nathaniel Clark <nathaniel.l.clark@intel.com>
Reviewed-by: Lai Siyao <lai.siyao@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
12 years agoLU-2824 mds: don't deactivate OSC if osc_create returns -EBUSY
Niu Yawei [Tue, 19 Feb 2013 02:05:04 +0000 (21:05 -0500)]
LU-2824 mds: don't deactivate OSC if osc_create returns -EBUSY

During MDS<->OST orphan recovery, osc_create() will likely return
-EBUSY while OSCC_FLAG_SYNC_IN_PROGRESS flag is still set
due to slow ll_mdt service thread. The __mds_lov_synchronize() will
then deactivate the OSC, which will cause mds_create_objects() get
-EIO error.

This patch fixes the above issue by checking the return value of
mds_lov_clear_orphans(). If it's -EBUSY, then do not mark the
OSC as inactive.

Test-Parameters: envdefinitions=DURATION=14400 clientdistro=el5 serverdistro=el5 clientarch=x86_64 serverarch=x86_64 clientcount=4 osscount=2 mdscount=2 austeroptions=-R failover=true useiscsi=true testlist=recovery-mds-scale
Signed-off-by: Niu Yawei <yawei.niu@intel.com>
Change-Id: I7b3376e3b6087dd35c5f0e33e98b1e538ba711fc
Reviewed-on: http://review.whamcloud.com/5462
Tested-by: Hudson
Reviewed-by: Jian Yu <jian.yu@intel.com>
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
12 years agoLU-1666 obdclass: reduce lock contention on coh_page_guard
Gregoire Pichon [Tue, 12 Feb 2013 13:40:49 +0000 (14:40 +0100)]
LU-1666 obdclass: reduce lock contention on coh_page_guard

Define a per-page spinlock to get and put a cl_page instead of
grabbing per-object lock coh_page_guard.

Signed-off-by: Jinshan Xiong <jinshan.xiong@intel.com>
Signed-off-by: Gregoire Pichon <gregoire.pichon@bull.net>
Change-Id: I3c63f0e33f3d4d44fbb5de06bcd0670e76b872bd
Reviewed-on: http://review.whamcloud.com/5428
Tested-by: Hudson
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
12 years agoLU-2775 fid: allow FID-on-OST in fid_seq_is_mdt()
Andreas Dilger [Fri, 8 Feb 2013 00:04:12 +0000 (17:04 -0700)]
LU-2775 fid: allow FID-on-OST in fid_seq_is_mdt()

The old 2.1 LASSERT_SEQ_IS_MDT() macro used fid_seq_is_mdt() in
several places to verify that a FID was "sane" for where it was
being used.  However, fid_seq_is_mdt() only checked FID_SEQ_OST_MDT0
through FID_SEQ_OST_MDT9 values, which immediately broke 2.1 clients
when FID-on-OST was enabled on 2.4 MDTs.

The use of FID-on-OST will be limited to only DNE MDTs for 2.4, but
clients should not LASSERT() on a value that it got from the network.
The use of LASSERT_SEQ_IS_MDT() is removed from clients, since this
is "validating" data from the network.  Even with this removal, the
fid_seq_is_mdt() macro is updated to allow FID_SEQ_NORMAL sequences,
as in 2.4, in case it is needed elsewhere.

Signed-off-by: Andreas Dilger <andreas.dilger@intel.com>
Change-Id: I74bc9198799045b8bd91510cb45e8f8760300c1e
Reviewed-on: http://review.whamcloud.com/5304
Tested-by: Hudson
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: Mike Pershin <mike.pershin@intel.com>
Reviewed-by: wangdi <di.wang@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
12 years agoLU-2555 fsfilt: fix race in ldiskfs_ext_new_extent_cb
Niu Yawei [Wed, 9 Jan 2013 04:32:53 +0000 (23:32 -0500)]
LU-2555 fsfilt: fix race in ldiskfs_ext_new_extent_cb

In ldiskfs_ext_calc_credits_for_insert(), we should use the 'depth'
stored in the 'path' instead from inode, because the extent tree
could have been changed when ldiskfs_ext_calc_credits_for_insert()
is called (by ldiskfs_ext_new_extent_cb()).

Signed-off-by: Niu Yawei <yawei.niu@intel.com>
Change-Id: Ie28f7dfc393c8feb8336f6b391cebb0adc814283
Reviewed-on: http://review.whamcloud.com/4977
Tested-by: Hudson
Reviewed-by: Alex Zhuravlev <alexey.zhuravlev@intel.com>
Reviewed-by: Bobi Jam <bobijam@gmail.com>
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
12 years agoLU-2555 ldiskfs: fix ext4_mb_add_n_trim()
Niu Yawei [Wed, 9 Jan 2013 04:55:59 +0000 (23:55 -0500)]
LU-2555 ldiskfs: fix ext4_mb_add_n_trim()

In ext4_mb_add_n_trim(), lg_prealloc_lock should be taken
when changing the lg_prealloc_list.

Signed-off-by: Niu Yawei <yawei.niu@intel.com>
Change-Id: I4f3a4b8bc9697b7ccdeade43f5bfd10960533fca
Reviewed-on: http://review.whamcloud.com/4978
Reviewed-by: Alex Zhuravlev <alexey.zhuravlev@intel.com>
Reviewed-by: Bobi Jam <bobijam@gmail.com>
Tested-by: Hudson
Tested-by: Maloo <whamcloud.maloo@gmail.com>
12 years agoLU-2634 ldiskfs: clear extent flag for fast symlink
Emoly Liu [Mon, 18 Feb 2013 10:17:50 +0000 (18:17 +0800)]
LU-2634 ldiskfs: clear extent flag for fast symlink

Fast symlinks on MDT filesystems formatted with the "extents" feature
are created with EXT4_EXTENTS_FL. e2fsck considers this a corruption
and deletes the symlink.

The patch is to clear extent flag in osd_ldiskfs_writelink() just as
ext4_symlink() does.

Signed-off-by: Liu Ying <emoly.liu@intel.com>
Change-Id: Ic0c711cc82de140f86baa71bacd1721fd4a19e07
Reviewed-on: http://review.whamcloud.com/5458
Tested-by: Hudson
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: Bobi Jam <bobijam@gmail.com>
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
12 years agoLU-2657 recovery: don't delete objects
Niu Yawei [Wed, 30 Jan 2013 02:24:04 +0000 (21:24 -0500)]
LU-2657 recovery: don't delete objects

In mds_lov_update_objids(), when an data object id gap is
detected during recovery, it'll delete all the objects in
the gap, which isn't quite correct, because we can't guarantee
that the id is always increased by the transno order,
furthermore, that could also bring big trouble when the
lov_objid file was removed manually (to rebuild the corrupted
lov_objid file).

Fix type defect in filter_recov_log_unlink_cb(), where
oa->o_id should be increased by each loop cycle.

Signed-off-by: Niu Yawei <yawei.niu@intel.com>
Change-Id: I7553614f30fe60675b8dab405d9100cb63069b23
Reviewed-on: http://review.whamcloud.com/5206
Tested-by: Hudson
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: Mike Pershin <mike.pershin@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
12 years agoLU-1714 lnet: Properly initialize sg_magic value
Emoly Liu [Tue, 5 Feb 2013 18:10:25 +0000 (02:10 +0800)]
LU-1714 lnet: Properly initialize sg_magic value

When the CONFIG_DEBUG_SG flag is enabled in the kernel, we must ensure
the sg_magic field is properly initialized. Otherwise, internal kernel
assertions will fail when trying to verify this field. As a result,
certain calls to sg_* function had to be changed or inserted to ensure
the sg_init_table function would be called, initializing the magic
value. Also, we need to ensure this value isn't zeroed out in the
kiblnd_setup_rd_kiov function.

port of master patch d3972843ae3ba829f821bd135d5ba97d4b0356f3

Signed-off-by: Prakash Surya <surya1@llnl.gov>
Signed-off-by: Alexander Boyko <alexander_boyko@xyratex.com>
Signed-off-by: Liu Ying <emoly.liu@intel.com>
Change-Id: I82b19348bcf981b5d843ae4bf07cbd2a6d51b701
Reviewed-on: http://review.whamcloud.com/5452
Tested-by: Hudson
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
12 years agoLU-2638 ldiskfs: update dotdot entry for rename
Fan Yong [Thu, 10 Jan 2013 13:32:57 +0000 (21:32 +0800)]
LU-2638 ldiskfs: update dotdot entry for rename

On master, when rename a directory, its old dotdot entry will
be removed firstly, then insert the new dotdot entry, and try
to append FID-in-dirent. But the space for dotdot entry maybe
not enough to hold the new dotdot with FID-in-dirent, such as
the MDT device restored from file-level backup, or the device
is upgraded from 1.8. If we do not move data in the directory
block, the FID-in-dirent will overwrite the dx_root and cause
the directory crashed. Currently, we do not want to introduce
complex logic to handle directory data moving, instead, under
such case, ignore the FID-in-dirent for the new dotdot entry,
and just insert the new dotdot entry.

Signed-off-by: Fan Yong <fan.yong@intel.com>
Change-Id: Id38f5136fb2bc607344c37ca10a32e63ab427e3d
Reviewed-on: http://review.whamcloud.com/5179
Tested-by: Hudson
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
Reviewed-by: Alex Zhuravlev <alexey.zhuravlev@intel.com>
12 years agoLU-1486 tests: fix sanityn test_45g() to add error check
Yu Jian [Thu, 30 Aug 2012 06:50:10 +0000 (14:50 +0800)]
LU-1486 tests: fix sanityn test_45g() to add error check

This patch fixes sanityn test_45g() to add error check for
"stat must fail".

Signed-off-by: Yu Jian <yujian@whamcloud.com>
Change-Id: Iff65f2684244ab58e0d0cab68db0795f5604187e
Reviewed-on: http://review.whamcloud.com/3826
Tested-by: Hudson
Reviewed-by: Wei Liu <wei3.liu@intel.com>
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: Minh Diep <minh.diep@intel.com>
Reviewed-by: Oleg Drokin <green@whamcloud.com>
12 years agoLU-1448 llite: Prevent NULL pointer dereference on disabled OSC
Jeremy Filizetti [Thu, 31 May 2012 12:26:28 +0000 (08:26 -0400)]
LU-1448 llite: Prevent NULL pointer dereference on disabled OSC

When a file system is mounted with a disabled OSC reading the import
information from the proc file system can result in a NULL pointer
dereference. The Lustre import on a disabled OSC with remain
in the LUSTRE_IMP_NEW state and imp_connection will remain NULL.

Signed-off-by: Jeremy Filizetti <jeremy.filizetti@gmail.com>
Signed-off-by: Bobi Jam <bobijam@whamcloud.com>
Change-Id: Ia6df51a36efbcd5a7fc7668bb23455b253ae4855
Reviewed-on: http://review.whamcloud.com/2995
Reviewed-by: Liang Zhen <liang@whamcloud.com>
Tested-by: Hudson
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: Oleg Drokin <green@whamcloud.com>
12 years agoLU-582 test: fix bug in get_ost_param in sanity.sh
Hongchao Zhang [Fri, 17 Aug 2012 08:25:32 +0000 (16:25 +0800)]
LU-582 test: fix bug in get_ost_param in sanity.sh

in get_ost_param(), the ouput should be gl_sum, which is
the sum of 'gl' in all OSS.

Signed-off-by: Hongchao Zhang <hongchao.zhang@whamcloud.com>
Change-Id: I51d3aa591c3ca875f8a559974cf05d250b0c032a
Reviewed-on: http://review.whamcloud.com/3693
Reviewed-by: Jian Yu <jian.yu@intel.com>
Tested-by: Jian Yu <jian.yu@intel.com>
Tested-by: Hudson
Reviewed-by: Fan Yong <fan.yong@intel.com>
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
12 years agoLU-2550 osc: set resend count properly
Niu Yawei [Mon, 28 Jan 2013 04:25:00 +0000 (23:25 -0500)]
LU-2550 osc: set resend count properly

The resend count of new io request should be set properly
in osc_brw_redo_request().

Signed-off-by: Niu Yawei <yawei.niu@intel.com>
Change-Id: If02f0f9abeb316f0718d34d0d1858985209acbb0
Reviewed-on: http://review.whamcloud.com/5184
Tested-by: Hudson
Reviewed-by: Johann Lombardi <johann.lombardi@intel.com>
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: Bobi Jam <bobijam@gmail.com>
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
12 years agoLU-2526 quota: writer proper size in test_9
Niu Yawei [Mon, 28 Jan 2013 04:34:00 +0000 (23:34 -0500)]
LU-2526 quota: writer proper size in test_9

Write proper size (4.5G) in test_9 of s-q.

Signed-off-by: Niu Yawei <yawei.niu@intel.com>
Change-Id: I3dbb9f9b5e0b053529b97b90156dea1b75e366b1
Reviewed-on: http://review.whamcloud.com/5185
Tested-by: Hudson
Reviewed-by: Johann Lombardi <johann.lombardi@intel.com>
Reviewed-by: Fan Yong <fan.yong@intel.com>
Tested-by: Maloo <whamcloud.maloo@gmail.com>
12 years agoLU-2309 tests: ignore unknown configuration param
Yu Jian [Tue, 15 Jan 2013 14:39:35 +0000 (22:39 +0800)]
LU-2309 tests: ignore unknown configuration param

This patch improves conf-sanity test 42 to verify that
invalid sys config param should not prevent client/server
from mounting.

Signed-off-by: Jian Yu <jian.yu@intel.com>
Change-Id: I083f0c143753ff2b990c91319db057ef8221992b
Reviewed-on: http://review.whamcloud.com/5031
Tested-by: Hudson
Reviewed-by: Lai Siyao <lai.siyao@intel.com>
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
12 years agoLU-1887 ptlrpc: grant shrink rpc format is special
Lai Siyao [Mon, 15 Oct 2012 07:02:19 +0000 (00:02 -0700)]
LU-1887 ptlrpc: grant shrink rpc format is special

Grant shrink rpc is packed in OST_SET_INFO request, but its format is
different from others.

Change-Id: I24fef30504847d26699df427f787d7c1d594c7c5
Signed-off-by: Lai Siyao <laisiyao@whamcloud.com>
Reviewed-on: http://review.whamcloud.com/4269
Reviewed-by: Bobi Jam <bobijam@gmail.com>
Tested-by: Hudson
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: Jinshan Xiong <jinshan.xiong@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
12 years agoLU-1331 changelog: allow changelog to extend record
Lai Siyao [Fri, 1 Jun 2012 11:46:44 +0000 (19:46 +0800)]
LU-1331 changelog: allow changelog to extend record

Previously rename is split into two records, this isn't natural and
makes replication hard because rename needs two records, but they
may not be consecutive.

To solve this, allow Changelog to extend record, add fields sfid/spfid
in changelog record to store source/parent fid, and pack both source
and target name into the record.

Added changelog flag CLF_RENAME_LAST: rename unlink of last hardlink
of the target.

Added macro HAVE_CHANGELOG_EXTEND_REC: liblustreapi receives all
changelog records in changelog_ext_rec format.

Signed-off-by: Lai Siyao <laisiyao@whamcloud.com>
Change-Id: I97b990687726e45e661cfb11609f80132c84825d
Reviewed-on: http://review.whamcloud.com/3835
Reviewed-by: Lai Siyao <lai.siyao@intel.com>
Tested-by: Hudson
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: Niu Yawei <yawei.niu@intel.com>
Reviewed-by: Henri Doreau <henri.doreau@cea.fr>
Reviewed-by: Thomas LEIBOVICI - CEA <thomas.leibovici@cea.fr>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
12 years agoLU-676 tests: machinefile option for mpirun via a variable
Jay J. Lan [Tue, 18 Oct 2011 19:04:07 +0000 (12:04 -0700)]
LU-676 tests: machinefile option for mpirun via a variable

Not all MPI implementations pass the host file to mpirun via the same
option. Common options are -machinefile and -hostfile.

This problem can be resolved by using a variable MACHINEFILE_OPTION
instead. A default value is assigned if the variable not defined.

Signed-off-by: Jay J Lan <jay.j.lan@nasa.gov>
Signed-off-by: Bobi Jam <bobijam.xu@intel.com>
Change-Id: I3362a6e62a27318cff733aea2f99b1356b3ff02e
Reviewed-on: http://review.whamcloud.com/3874
Tested-by: Hudson
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: Jian Yu <jian.yu@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
12 years agoLU-1234 dcache: don't drop invalid dentry arbitrarily
Lai Siyao [Wed, 28 Mar 2012 17:07:53 +0000 (01:07 +0800)]
LU-1234 dcache: don't drop invalid dentry arbitrarily

This is a backport of part of LU-506 dcache scalability support:
* remove super hack d_rehash_cond(), and treats DCACHE_LUSTRE_INVALID
  similar to DCACHE_DISCONNECTED, therefore dentry doesn't need to
  be dropped and rehashed frequently.
* .lookup(LOOKUP_CREATE) calls d_add() dentry directly, and .create
  only needs to d_instantiate() this dentry.
* other cleanups.

Signed-off-by: Lai Siyao <laisiyao@whamcloud.com>
Change-Id: Ie169bc7e763e6891084999041aac9f62c8dee9f0
Reviewed-on: http://review.whamcloud.com/2400
Tested-by: Hudson
Reviewed-by: Fan Yong <fan.yong@intel.com>
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: Peng Tao <bergwolf@gmail.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
12 years agoLU-2237 tests: rename conf-sanity::test_61 as test_65
Fan Yong [Sun, 4 Nov 2012 07:41:51 +0000 (15:41 +0800)]
LU-2237 tests: rename conf-sanity::test_61 as test_65

To make it to be consistent with master branch.

Signed-off-by: Fan Yong <yong.fan@whamcloud.com>
Change-Id: I5cf281c74dbe66f742353843c813f6724340442a
Reviewed-on: http://review.whamcloud.com/4456
Tested-by: Hudson
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
Reviewed-by: Jian Yu <jian.yu@intel.com>
12 years agoLU-1018 tests: reduce runtime value for compilebench
Minh Diep [Thu, 17 Jan 2013 16:09:05 +0000 (08:09 -0800)]
LU-1018 tests: reduce runtime value for compilebench

We are reducing the cbench_IDIRS and cbench_RUNS to 2.
These variables can be adjusted to other runs beside
sanity check.

Test-Parameters: envdefinitions=SLOW=yes,ENABLE_QUOTA=yes testlist=parallel-scale
Signed-off-by: Minh Diep <minh.diep@intel.com>
Change-Id: I63694bd26cdf8cecdbabebf601c014718870af21
Reviewed-on: http://review.whamcloud.com/5053
Tested-by: Hudson
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: Keith Mannthey <keith.mannthey@intel.com>
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
12 years agoLU-2467 protocol: Add OBD_CONNECT_PINGLESS
Li Wei [Thu, 31 Jan 2013 09:48:23 +0000 (17:48 +0800)]
LU-2467 protocol: Add OBD_CONNECT_PINGLESS

Reserve a bit for OBD_CONNECT_PINGLESS, which indicates a client is
capable of suppressing keep-alive OBD_PINGs.  If granted by a server,
it means the server does not require (but still allows) pings.

Change-Id: Ie10f505badd4250582656295eac2fd6844bb76b3
Signed-off-by: Li Wei <wei.g.li@intel.com>
Reviewed-on: http://review.whamcloud.com/5232
Tested-by: Hudson
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
12 years agoLU-1770 idl: reserve new OBD_CONNECT feature flags
Andreas Dilger [Thu, 24 Jan 2013 23:03:09 +0000 (16:03 -0700)]
LU-1770 idl: reserve new OBD_CONNECT feature flags

Reserve protocol feature flags on b2_1 to match the definitions
on the master branch:
- OBD_CONNECT_FLOCK_OWNER
- OBD_CONNECT_LVB_TYPE
- OBD_CONNECT_NANOSEC_TIME
- OBD_CONNECT_LIGHTWEIGHT
- OBD_CONNECT_SHORTIO

The OBD_CONNECT_NANOSEC_TIME was originally added in commit
596816513b0603cb159353e1444c97d8bc3bcacf (on master it was
528feb1793e744c63c2cd137d11418c513eb92d), but this was changed
in master commit f100d508311d16a09040a057438ee06fd9e6d7fe due
to a conflict with OBD_CONNECT_FLOCK_OWNER in a customer patch.

As of the time of this patch, the nanosecond timestamp flag
is not yet in use on any branch, so changing the flag value
does not cause any protocol incompatibility.

Signed-off-by: Andreas Dilger <andreas.dilger@intel.com>
Change-Id: Iae34aeec54cdfeb39fe932aa132a3cac982d1ee5
Reviewed-on: http://review.whamcloud.com/5162
Tested-by: Hudson
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: Li Wei <wei.g.li@intel.com>
Reviewed-by: Johann Lombardi <johann.lombardi@intel.com>
12 years agoLU-20 ldiskfs: remove spurious warning message
Andreas Dilger [Fri, 21 Oct 2011 10:06:16 +0000 (04:06 -0600)]
LU-20 ldiskfs: remove spurious warning message

Remove the spurious warning message that we added for the missing
extents option since the MDS never gets the extents option enabled,
nor do we want it to be enabled for the MDT filesystem either.

ldiskfs_fill_super: extents feature not enabled on this
filesystem, use tune2fs

Original-Commit: 6541ea42978e73e0262273f6bd5ce0b71689fdc2

Signed-off-by: Andreas Dilger <andreas.dilger@intel.com>
Change-Id: I9d896258b89ec3db528db59094572daac8dccab0
Reviewed-on: http://review.whamcloud.com/4028
Tested-by: Hudson
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: Niu Yawei <yawei.niu@intel.com>
12 years ago2.1.4rc2 2.1.4 2.1.4-RC2 v2_1_4 v2_1_4_0 v2_1_4_0_RC2 v2_1_4_RC2
Oleg Drokin [Wed, 19 Dec 2012 19:19:59 +0000 (14:19 -0500)]
2.1.4rc2

Change-Id: Ie51d791502006d094219643e9b0286cba8ebe2dd

12 years agoLU-1526 tests: Support for MDS-initiated OST_DESTROYs
Li Wei [Fri, 14 Dec 2012 08:34:21 +0000 (16:34 +0800)]
LU-1526 tests: Support for MDS-initiated OST_DESTROYs

This patch is backported from commit af5f388 of LU-1303 to
support interoperating with 2.4 server.

The patch makes sure the tests work with MDSs that destroy OST
objects asynchronously on behalf of clients.

Signed-off-by: Li Wei <liwei@whamcloud.com>
Signed-off-by: Yu Jian <yujian@whamcloud.com>
Change-Id: I375d33b53e88a1649b68c1a3999d273d759d17b7
Reviewed-on: http://review.whamcloud.com/4832
Tested-by: Hudson
Reviewed-by: Li Wei <wei.g.li@intel.com>
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
Tested-by: Maloo <whamcloud.maloo@gmail.com>
12 years agoLU-1526 tests: new sanity-quota tests
Yu Jian [Thu, 13 Dec 2012 13:43:16 +0000 (21:43 +0800)]
LU-1526 tests: new sanity-quota tests

This patch is ported from commit d6f2a9f to add new sanity-quota.sh
for the interoprability testing with new quota architecture,
meanwhile, the old sanity-quota.sh is reserved for the interoprability
with old server.

The patch also contains the fixups for LU-2174, LU-2283, LU-2284
and LU-2329.

Test-Parameters: envdefinitions=SLOW=yes,ENABLE_QUOTA=yes testlist=conf-sanity,sanity-quota
Signed-off-by: Niu Yawei <niu@whamcloud.com>
Signed-off-by: Yu Jian <yujian@whamcloud.com>
Change-Id: I334890a4c7a233311e8f4075a8bf8dd904563525
Reviewed-on: http://review.whamcloud.com/4767
Tested-by: Hudson
Reviewed-by: Niu Yawei <yawei.niu@intel.com>
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: Fan Yong <fan.yong@intel.com>
Reviewed-by: Oleg Drokin <green@whamcloud.com>
12 years agoLU-691 tests: Fix OST index errors in test suite
James Simmons [Thu, 13 Dec 2012 09:28:15 +0000 (17:28 +0800)]
LU-691 tests: Fix OST index errors in test suite

Several tests run do_facet ost which assumes ost is ost0 which does
not exist according to the way the test suite works. This patch
address several areas where the wrong ost index is used. This patch
also introduces some short hand functions to get OST properties
from the index. Those functions ensure ost1, ost10 and ost100 are
seen as different which is not always the case.

This patch is backported from commit 8afd35c of LU-691.

Test-Parameters: envdefinitions=SLOW=yes,ENABLE_QUOTA=yes testlist=lfsck,sanity,replay-single
Signed-off-by: James Simmons <uja.ornl@gmail.com>
Signed-off-by: Yu Jian <yujian@whamcloud.com>
Change-Id: I97d9554a99be25db0e4f550bf8071ddc981dddc4
Reviewed-on: http://review.whamcloud.com/4821
Tested-by: Hudson
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: Lai Siyao <laisiyao@whamcloud.com>
Reviewed-by: Oleg Drokin <green@whamcloud.com>
12 years agoLU-1526 tests: Handle OFD procfs changes
Yu Jian [Thu, 13 Dec 2012 06:07:29 +0000 (14:07 +0800)]
LU-1526 tests: Handle OFD procfs changes

In order to interop with 2.4 server, the following procfs entry
changes need to be handled:

- obdfilter.*.mntdev -> osd-*.*.mntdev
- obdfilter.*.<cache_related> -> osd-*.*.<cache_related>

Test-Parameters: envdefinitions=SLOW=yes,ENABLE_QUOTA=yes testlist=lfsck,sanity
Signed-off-by: Li Wei <liwei@whamcloud.com>
Signed-off-by: Yu Jian <yujian@whamcloud.com>
Change-Id: I0bc3fc5a2f6d3a3ff6653175a8b00cd51d48534c
Reviewed-on: http://review.whamcloud.com/4783
Reviewed-by: Li Wei <wei.g.li@intel.com>
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
Tested-by: Hudson
Reviewed-by: Niu Yawei <yawei.niu@intel.com>
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: Oleg Drokin <green@whamcloud.com>
12 years agoLU-2483 build: Update Copyrights for b2_1
Keith Mannthey [Tue, 18 Dec 2012 22:29:08 +0000 (14:29 -0800)]
LU-2483 build: Update Copyrights for b2_1

In prepration of the release of 2.1.4 the Copyrights need to be
updated.

As I have bound the script to the changes so that it will not be lost
from the changes it helped make.

The accounting works for both Intel and Whamcloud email addresses.
See: lustre/ldlm/ldlm_request.c
It is able to add new Copyrights when aproprate
See: lustre/ptlrpc/layout.c
I didn't find any Intel, Inc signed files.
Two man pages were edited by hand and are included in this commit.

Signed-off-by: Keith Mannthey <keith.mannthey@intel.com>
Change-Id: I0c6daac2c6bfd2b6d5ed1f14bb5993519d099b91
Reviewed-on: http://review.whamcloud.com/4857
Tested-by: Hudson
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: Bob Glossman <bob.glossman@intel.com>
12 years agoLU-1484 kernel: pass RHEL5 build for 2.6.18-308
Bobi Jam [Tue, 11 Dec 2012 03:04:22 +0000 (11:04 +0800)]
LU-1484 kernel: pass RHEL5 build for 2.6.18-308

For vanilla kernel, proc_dir_entry::deleted and ::pde_users co-exists
from 2.6.23 to 2.6.23.17.

For some RHEL5 kernels, it defines co-existings
proc_dir_entry::deleted and proc_dir_entry_aux::pde_users.

Signed-off-by: Bobi Jam <bobijam@gmail.com>
Change-Id: I107e15e6e7d41680c99b324c35ddac0a7c3716e6
Reviewed-on: http://review.whamcloud.com/4794
Tested-by: Hudson
Reviewed-by: Peng Tao <bergwolf@gmail.com>
Reviewed-by: Stephen Champion <schamp@sgi.com>
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: Oleg Drokin <green@whamcloud.com>
12 years agoLU-797 tests: improve test_23b of ost-pools.sh
Emoly Liu [Mon, 17 Dec 2012 03:15:08 +0000 (11:15 +0800)]
LU-797 tests: improve test_23b of ost-pools.sh

- fail the test once the written space is greater than the
  available space.
- use it's own maxfree limit, and increase the limit a lot.
- terminate the test when the write speed is too slow.

Test-Parameters: envdefinitions=SLOW=yes testlist=ost-pools
Signed-off-by: Niu Yawei <niu@whamcloud.com>
Signed-off-by: Liu Ying <emoly.liu@intel.com>
Change-Id: Id9aef77ef07358a03133ed6d34c299609540ef42
Reviewed-on: http://review.whamcloud.com/4831
Reviewed-by: Niu Yawei <yawei.niu@intel.com>
Tested-by: Hudson
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: Jian Yu <jian.yu@intel.com>
Reviewed-by: Oleg Drokin <green@whamcloud.com>
12 years agoLU-2505 osc: use GFP_ATOMIC for ll_dirty_page_discard_warn()
Jinshan Xiong [Wed, 19 Dec 2012 06:44:42 +0000 (22:44 -0800)]
LU-2505 osc: use GFP_ATOMIC for ll_dirty_page_discard_warn()

Because it can be called inside client obd list lock as follows:

ll_dirty_page_discard_warn at ffffffffa0a3d252 [lustre]
vvp_page_completion_common at ffffffffa0a7adfc [lustre]
vvp_page_completion_write_common at ffffffffa0a7ae6b [lustre]
vvp_page_completion_write at ffffffffa0a7b83e [lustre]
cl_page_completion at ffffffffa05eed8f [obdclass]
osc_completion at ffffffffa0880812 [osc]
osc_ap_completion at ffffffffa086a544 [osc]
brw_interpret at ffffffffa0876d69 [osc]

Signed-off-by: Jinshan Xiong <jinshan.xiong@intel.com>
Change-Id: Ia1ee6c9885941068a358c5a6aeaa9bd47a9f9b7b
Reviewed-on: http://review.whamcloud.com/4866
Tested-by: Hudson
Reviewed-by: Niu Yawei <yawei.niu@intel.com>
Tested-by: Jian Yu <jian.yu@intel.com>
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
Reviewed-by: Oleg Drokin <green@whamcloud.com>
12 years agoLustre 2.1.4-rc1 2.1.4-RC1 v2_1_4_0_RC1 v2_1_4_RC1
Oleg Drokin [Sat, 15 Dec 2012 06:47:12 +0000 (01:47 -0500)]
Lustre 2.1.4-rc1

Change-Id: I53a3d90a9118d083b52bc2f08f0b7b99c70d48f4
Signed-off-by: Oleg Drokin <green@whamcloud.com>
12 years agoLU-797 tests: process lfs df output properly
Emoly Liu [Mon, 10 Dec 2012 13:11:16 +0000 (21:11 +0800)]
LU-797 tests: process lfs df output properly

There is a space in the "filesystem summary" of lfs df output,
which could cause trouble for the parsing script, test_23b of
ost-pool.sh is an example.

We introduced lfs_df() function to remove this space internally,
this will allow fixing the "lfs df" output line in the future.

port of master patch fb1a1ec6300a5ec3925b725d5d2b783314dff3f8

Signed-off-by: Niu Yawei <niu@whamcloud.com>
Signed-off-by: Liu Ying <emoly.liu@intel.com>
Change-Id: Iba57b4aac714e28ebdd985f15ff3e2444e64e007
Reviewed-on: http://review.whamcloud.com/4784
Tested-by: Hudson
Reviewed-by: Niu Yawei <yawei.niu@intel.com>
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
Tested-by: Maloo <whamcloud.maloo@gmail.com>
12 years agoLU-2463 security: osd_mknod shouldn't alter mode after creation
Lai Siyao [Thu, 13 Dec 2012 06:38:03 +0000 (14:38 +0800)]
LU-2463 security: osd_mknod shouldn't alter mode after creation

osd_mknod() calls init_special_inode(), but it should use
inode->i_mode instead of the one from client.

Test-Parameters: envdefinitions=SLOW=yes testlist=posix
Signed-off-by: Lai Siyao <laisiyao@whamcloud.com>
Change-Id: Iee0f49dc4beef4aae6b5b6f57679de2643887e25
Reviewed-on: http://review.whamcloud.com/4820
Tested-by: Hudson
Reviewed-by: Jian Yu <jian.yu@intel.com>
Tested-by: Jian Yu <jian.yu@intel.com>
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: Oleg Drokin <green@whamcloud.com>
12 years agoLU-2304 lov: check enq flags when matching top lock
Jinshan Xiong [Thu, 13 Dec 2012 01:17:47 +0000 (17:17 -0800)]
LU-2304 lov: check enq flags when matching top lock

cld_enq_flags must be checked when matching top lock, otherwise
if a sublock is missing then it will be reenqueued with wrong enq
flags.

For LU-2304, a previous truncate to zero lock(with CEF_DISCARD_DATA)
flags was matched by not-to-zero truncate and then wrongly reenqueued
with discard flag. This caused dirty pages to be discarded on the
other clients.

Signed-off-by: Jinshan Xiong <jinshan.xiong@intel.com>
Change-Id: I0575509b3ec30982fe7e5a7ff018ff4dd3dbe911
Reviewed-on: http://review.whamcloud.com/4818
Tested-by: Hudson
Reviewed-by: Bobi Jam <bobijam@gmail.com>
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: Oleg Drokin <green@whamcloud.com>
12 years agoLU-2309 config: ignore unknown configuration param
Yu Jian [Fri, 23 Nov 2012 12:57:27 +0000 (20:57 +0800)]
LU-2309 config: ignore unknown configuration param

Client or server should not fail to mount if it hits
a configuration parameter that it doesn't understand.
This patch fixes class_process_config() to meet
the above requirement.

Signed-off-by: Yu Jian <yujian@whamcloud.com>
Change-Id: Ida537193459f08d804fc4c0f1f72630b416440e3
Reviewed-on: http://review.whamcloud.com/4663
Reviewed-by: Lai Siyao <laisiyao@whamcloud.com>
Tested-by: Hudson
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
12 years agoLU-1075 tests: auster detect directory as script name
Minh Diep [Tue, 11 Dec 2012 11:22:48 +0000 (19:22 +0800)]
LU-1075 tests: auster detect directory as script name

Auster should use -f to detect the existing of
script name rather use -e which will treat directory
same as script.

Signed-off-by: Minh Diep <mdiep@whamcloud.com>
Signed-off-by: Yu Jian <yujian@whamcloud.com>
Change-Id: Ic05aeeb6cdda183dddfdc1bec2646546898922dd
Reviewed-on: http://review.whamcloud.com/4795
Tested-by: Hudson
Reviewed-by: Wei Liu <wei3.liu@intel.com>
Reviewed-by: Minh Diep <minh.diep@intel.com>
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: Oleg Drokin <green@whamcloud.com>
12 years agoLU-2420 tests: add add_group() and add_user()
Yu Jian [Tue, 11 Dec 2012 12:51:32 +0000 (20:51 +0800)]
LU-2420 tests: add add_group() and add_user()

This patch adds add_group() and add_user() functions into the
test-framework.sh. They are used by setup_posix_users() in
posix.sh.

This patch is backported from commit 34a6092 for LU-1661.
Signed-off-by: Yu Jian <yujian@whamcloud.com>
Change-Id: I33baec4abe60752ba236205e03d607a5da250216
Reviewed-on: http://review.whamcloud.com/4797
Tested-by: Hudson
Reviewed-by: Emoly Liu <emoly.liu@intel.com>
Reviewed-by: Minh Diep <minh.diep@intel.com>
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: Oleg Drokin <green@whamcloud.com>
12 years agoLU-2448 build: add {prepare-,}commit-msg git hooks
Andreas Dilger [Sun, 9 Dec 2012 23:58:03 +0000 (16:58 -0700)]
LU-2448 build: add {prepare-,}commit-msg git hooks

Add commit message and patch format hooks for b2_1 to match other
development branches.  This avoids problems when checking out b2_1
from master, where the .git/hooks/{prepare-,}commit-msg symlinks
are installed but not pointing at anything.

Signed-off-by: Andreas Dilger <andreas.dilger@intel.com>
Change-Id: I9bea14402d1f5668855b55230aaac919e5d0d90e
Reviewed-on: http://review.whamcloud.com/4782
Tested-by: Hudson
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: Oleg Drokin <green@whamcloud.com>
12 years agoLU-1299 clio: a combo patch to fix cl_lock
Jinshan Xiong [Fri, 7 Dec 2012 18:34:24 +0000 (10:34 -0800)]
LU-1299 clio: a combo patch to fix cl_lock

In this patch, the following issues are fixed:
* redefine cl_lock_error(). There are two kinds of error when
  enqueuing a cl_lock: internal errors return from server side and
  local error for example interrupted by signal when waiting for a
  lock. Obviously the latter one is not fatal so we shouldn't err the
  lock out;
* for fault handling, only signal SIGKILL and SIGTERM is not allowed;
* handle return code of ll_fault correctly;
* redefine ->unuse() to release resource it held if enqueuing lock
  failed;
* in cl_lock_hold_release(), only CLS_CACHED mode lock will be cached,
  for locks in other states, they will be cancelled voluntarily;
* fix the problem that cfs_block_sigs() is wrongly used as
  cfs_restore_sigs() and implementation of cfs_block_sigs() was wrong
* Unuse method releases the underneath resources even error occurs, so
  we should set the lock to be CLS_NEW anyway.
* other minor fixes.

Signed-off-by: Jinshan Xiong <jinshan.xiong@intel.com>
Change-Id: Iffce8be356723781b8f33ec9bdc2cf73e9e07138
Reviewed-on: http://review.whamcloud.com/2574
Tested-by: Hudson
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: Lai Siyao <laisiyao@whamcloud.com>
Reviewed-by: Oleg Drokin <green@whamcloud.com>
12 years agoLU-1833 util: don't update mtab if linked to /proc
Shuichi Ihara [Mon, 10 Dec 2012 16:58:50 +0000 (01:58 +0900)]
LU-1833 util: don't update mtab if linked to /proc

Backport patch of LU-1833 for b2_1.
This is needed for the diskless environment since their
mtab is linking to /proc/mount.

Signed-off-by: Shuichi Ihara <sihara@ddn.com>
Signed-off-by: Peng Tao <tao.peng@emc.com>
Change-Id: Id18cd6a820dff905f0d61ecc1f71f4ce7b48cf77
Reviewed-on: http://review.whamcloud.com/4785
Tested-by: Hudson
Reviewed-by: Peng Tao <bergwolf@gmail.com>
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
12 years agoLU-1526 tests: add version_code() and lustre_version_code()
Yu Jian [Thu, 6 Dec 2012 15:45:23 +0000 (23:45 +0800)]
LU-1526 tests: add version_code() and lustre_version_code()

This patch adds the version_code() and lustre_version_code()
functions into the test framework.

It also fixes the version number for the interoperability
of quota codes and reads the quota_type from MDS instead of
MGS.

Test-Parameters: envdefinitions=SLOW=yes,ENABLE_QUOTA=yes testlist=conf-sanity,ost-pools,sanity-quota
Signed-off-by: Yu Jian <yujian@whamcloud.com>
Change-Id: I5a4d26fb71a7e56fa8497688ee2492ec90c446f8
Reviewed-on: http://review.whamcloud.com/4754
Tested-by: Hudson
Reviewed-by: Yang Sheng <yang.sheng@intel.com>
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: Niu Yawei <niu@whamcloud.com>
Reviewed-by: Oleg Drokin <green@whamcloud.com>
12 years agoLU-847 quota: client retrieve quota usage directly
Niu Yawei [Thu, 15 Sep 2011 03:36:19 +0000 (20:36 -0700)]
LU-847 quota: client retrieve quota usage directly

Current 'lfs quota' sends getquota RPC to MDS, and MDS is responsible
for retrieving disk usage from all targets, this scheme will be
changed to client retrieving disk usage from all targets directly.

This patch addresses the compatibility issue as well: If the getquota
returned by MDS has QIF_SPACE, client just trust the disk usage
returned by MDS, otherwise, client has to issue RPCs to all OSTs to
collect disk usage by itself.

Signed-off-by: Niu Yawei <niu@whamcloud.com>
Signed-off-by: Bob Glossman <bogl@whamcloud.com>
Change-Id: Ia05e588e1a28abd3057274b38684057ecc887f5b
Reviewed-on: http://review.whamcloud.com/2794
Tested-by: Hudson
Reviewed-by: Fan Yong <yong.fan@whamcloud.com>
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: Oleg Drokin <green@whamcloud.com>
12 years agoLU-1693 obdfilter: Set bi_rw before calling bio_add_page()
Emoly Liu [Fri, 7 Dec 2012 06:54:57 +0000 (14:54 +0800)]
LU-1693 obdfilter: Set bi_rw before calling bio_add_page()

In filter_do_bio() set bi_rw before calling bio_add_page()
so that the correct bio direction is used in mgrge_bvec_fn().

port of master patch 79badd39c6584ae136b14d919eaf699ee812b666

Signed-off-by: John L. Hammond <jhammond@tacc.utexas.edu>
Signed-off-by: Liu Ying <emoly.liu@intel.com>
Change-Id: Ib43a5c198db691f8ed4001c41af7b984a264922b
Reviewed-on: http://review.whamcloud.com/4765
Tested-by: Hudson
Reviewed-by: Fan Yong <yong.fan@whamcloud.com>
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
12 years agoLU-2420 tests: Have POSIX testing on b2_1
Emoly Liu [Fri, 7 Dec 2012 07:29:55 +0000 (15:29 +0800)]
LU-2420 tests: Have POSIX testing on b2_1

Have POSIX testing automated on b2_1 as it is on master.
LU-2274 port is included to change the baseline filesystem to ext3
for POSIX testing on SLES distro.

Test-Parameters: envdefinitions=SLOW=yes clientdistro=sles11 testlist=posix
Signed-off-by: Liu Ying <emoly.liu@intel.com>
Change-Id: If20262a37d77053bda3a1e143d74c4ebd11b7ec5
Reviewed-on: http://review.whamcloud.com/4731
Reviewed-by: Jian Yu <jian.yu@intel.com>
Tested-by: Hudson
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: Oleg Drokin <green@whamcloud.com>
12 years agoLU-2405 tests: verify off-by-one symlink length fix
Nathaniel Clark [Thu, 29 Nov 2012 14:14:37 +0000 (09:14 -0500)]
LU-2405 tests: verify off-by-one symlink length fix

Add regression test for off-by-one bug in symlink code. Ensures boundary
condition of 60 character symlinks are readable after creation.

original commit 6c3606b79a4c25946255a680f79ff2d58df66bbc

Whamcloud-bug-id: LU-2241
Signed-off-by: Nathaniel Clark <nathaniel.l.clark@intel.com>
Change-Id: I4264558a5b638e73e7631b988b05b92405d13288
Reviewed-on: http://review.whamcloud.com/4711
Tested-by: Hudson
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
Tested-by: Maloo <whamcloud.maloo@gmail.com>
12 years agoLU-2371 ptlrpc: add support for -EINPROGRESS
Niu Yawei [Fri, 13 Jan 2012 08:33:22 +0000 (00:33 -0800)]
LU-2371 ptlrpc: add support for -EINPROGRESS

Backport patches from LU-904, LU-1329 and LU-1788 to introduce
support for -EINPROGRESS in lustre 2.1. This is needed for
quota interoperability with 2.4 servers.

Signed-off-by: Johann Lombardi <johann@whamcloud.com>
Change-Id: I17689380c8d42a8daef915342f63d96f777dc6fa
Signed-off-by: Niu Yawei <niu@whamcloud.com>
Reviewed-on: http://review.whamcloud.com/4645
Tested-by: Hudson
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
Reviewed-by: Oleg Drokin <green@whamcloud.com>
12 years agoLU-2170 osc: set osc_lock attribute only once
Jinshan Xiong [Tue, 6 Nov 2012 23:17:33 +0000 (15:17 -0800)]
LU-2170 osc: set osc_lock attribute only once

Set osc_lock's attribute by lock allocator, otherwise if this lock is
matched and enqueued by a glimpse thread, the osc_lock's ols_glimpse
will be set to true and the lock state will be messed in
osc_lock_upcall().

Signed-off-by: Jinshan Xiong <jinshan.xiong@intel.com>
Change-Id: I2aff3d372d059474fed2c5d0983bdb116e932701
Reviewed-on: http://review.whamcloud.com/4482
Tested-by: Hudson
Reviewed-by: Fan Yong <yong.fan@whamcloud.com>
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: Niu Yawei <niu@whamcloud.com>
Reviewed-by: Oleg Drokin <green@whamcloud.com>
12 years agoLU-1648 llog: prevent journal to restart for llog cancel
Emoly Liu [Wed, 5 Dec 2012 07:25:21 +0000 (15:25 +0800)]
LU-1648 llog: prevent journal to restart for llog cancel

Journal credit for llog cancel should include the blocks for
deleting the inode. Otherwise the journal handle may need to
be restarted during the transaction for more credit. Then it
may be blocked with holding llog_handle::lgh_lock, which may
cause deadlock with other llog operations those have started
journal handle but waiting for the llog_handle::lgh_lock.

port of master patch 8d334f67052ccf77713c780a8040593728977599

Signed-off-by: Fan Yong <yong.fan@whamcloud.com>
Signed-off-by: Liu Ying <emoly.liu@intel.com>
Change-Id: I4013eb764485c282ed2443d02556c3377e16c936
Reviewed-on: http://review.whamcloud.com/4743
Tested-by: Hudson
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: Niu Yawei <niu@whamcloud.com>
Reviewed-by: Oleg Drokin <green@whamcloud.com>
12 years agoLU-1749 llog: MDT should work if cancel llog failed (-ESTALE)
Emoly Liu [Wed, 5 Dec 2012 06:43:56 +0000 (14:43 +0800)]
LU-1749 llog: MDT should work if cancel llog failed (-ESTALE)

Cancel catlog may be stale, in this case MDT should continue to work.

port of b2_3 patch e5d5cd20169b476cdb4fe4857d64aef4bf03d497

Signed-off-by: Lai Siyao <laisiyao@whamcloud.com>
Signed-off-by: Liu Ying <emoly.liu@intel.com>
Change-Id: I85d571a953b2a34d51d586595e271933a016b04f
Reviewed-on: http://review.whamcloud.com/4742
Tested-by: Hudson
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: wangdi <di.wang@intel.com>
Reviewed-by: Oleg Drokin <green@whamcloud.com>
12 years agoLU-1650 obdclass: find the lock by index of subpage
Jinshan Xiong [Mon, 26 Nov 2012 20:04:31 +0000 (12:04 -0800)]
LU-1650 obdclass: find the lock by index of subpage

In check_and_discard_cb(), subpage's index should be used to find if
there exists another lock to cover the same page.

Signed-off-by: Jinshan Xiong <jinshan.xiong@intel.com>
Change-Id: Id8e1da89ef81fad4c7917360d591fc7bd1309a00
Reviewed-on: http://review.whamcloud.com/4698
Tested-by: Hudson
Reviewed-by: Bobi Jam <bobijam@whamcloud.com>
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: Johann Lombardi <johann.lombardi@intel.com>
12 years agoLU-2373 kernel: Kernel update [RHEL 6.3 2.6.32-279.14.1.el6]
yangsheng [Tue, 4 Dec 2012 05:34:35 +0000 (13:34 +0800)]
LU-2373 kernel: Kernel update [RHEL 6.3 2.6.32-279.14.1.el6]

Update RHEL6.3 kernel to 2.6.32-279.14.1.el6.

Signed-off-by: James Simmons <uja.ornl@gmail.com>
Signed-off-by: yangsheng <ys@whamcloud.com>
Change-Id: I43a6721663cfab794882f220f7c64a705be20c0f
Reviewed-on: http://review.whamcloud.com/3811
Tested-by: Hudson
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: Yang Sheng <yang.sheng@intel.com>
Reviewed-by: Oleg Drokin <green@whamcloud.com>
12 years agoLU-921 llite: warning in case of discarding dirty pages
Emoly Liu [Tue, 4 Dec 2012 05:00:10 +0000 (13:00 +0800)]
LU-921 llite: warning in case of discarding dirty pages

when a client is evicted, dirty pages may get silently discarded,
the caller of successful write(2) will not know that the data he
wrote have been discarded due to eviction before it can be flushed
to the OSS.
test 24 of recovery-small is now reenabled back.

port of master patch 624a3ac233938153e889ae81f639cf46c0a8312b
port of master patch 4cfcf91efa06decc155d8f12eafc8ede3e850fd5

Signed-off-by: Hongchao Zhang <hongchao.zhang@whamcloud.com>
Signed-off-by: Liu Ying <emoly.liu@intel.com>
Change-Id: If8aeed507890a656cbb8231879d0e31aa9f4ea02
Reviewed-on: http://review.whamcloud.com/4716
Tested-by: Hudson
Reviewed-by: Jinshan Xiong <jinshan.xiong@whamcloud.com>
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: Oleg Drokin <green@whamcloud.com>
12 years agoLU-2372 kernel: kernel update [RHEL5.8 2.6.18-308.20.1.el5]
yangsheng [Thu, 22 Nov 2012 19:15:24 +0000 (03:15 +0800)]
LU-2372 kernel: kernel update [RHEL5.8 2.6.18-308.20.1.el5]

Update RHEL5.8 kernel to 2.6.18-308.20.1.el5.

Signed-off-by: yang sheng <ys@whamcloud.com>
Change-Id: Iafb4088227fea098840b70e574f2afe5a1f4067c
Reviewed-on: http://review.whamcloud.com/4658
Tested-by: Hudson
Reviewed-by: Lai Siyao <laisiyao@whamcloud.com>
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: Johann Lombardi <johann.lombardi@intel.com>
12 years agoLU-1782 quota: ignore sb_has_quota_active() in OFED's header
Emoly Liu [Thu, 29 Nov 2012 07:42:32 +0000 (15:42 +0800)]
LU-1782 quota: ignore sb_has_quota_active() in OFED's header

sb_has_quota_active() and sb_any_quota_active() are defined
in ofed's backport headers, but if these are reffered on
RHEL5's kernel, quota is broken. So, it ignores them.

port of b1_8 patch df3a5407df2c21da2c43c8acc3624a93303c8856

Signed-off-by: Shuichi Ihara <sihara@ddn.com>
Signed-off-by: Liu Ying <emoly.liu@intel.com>
Change-Id: I608a45b86164fd42086a1e8e5fdc63071c3a17a8
Reviewed-on: http://review.whamcloud.com/4704
Reviewed-by: Niu Yawei <niu@whamcloud.com>
Tested-by: Hudson
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: Oleg Drokin <green@whamcloud.com>
12 years agoLU-1526 tests: add --index support to the test framework
Yu Jian [Thu, 29 Nov 2012 14:06:28 +0000 (22:06 +0800)]
LU-1526 tests: add --index support to the test framework

This patch improves mkfs_opts() in test-framework.sh to
add --index option for MDT and OST targets automatically.

Signed-off-by: Yu Jian <yujian@whamcloud.com>
Change-Id: I6a3efd17bbc2396962f860e1a9e0b3149e7302ec
Reviewed-on: http://review.whamcloud.com/4710
Reviewed-by: Andreas Dilger <adilger@whamcloud.com>
Tested-by: Hudson
Reviewed-by: Li Wei <wei.g.li@intel.com>
Tested-by: Maloo <whamcloud.maloo@gmail.com>
12 years agoLU-1544 llog: remove lfs catinfo command
Gregoire Pichon [Wed, 20 Jun 2012 11:14:53 +0000 (13:14 +0200)]
LU-1544 llog: remove lfs catinfo command

This patch removes the llog catinfo code, both client and server
side, since it has not been maintained and may crash the client
system in some cases.

Signed-off-by: Gregoire Pichon <gregoire.pichon@bull.net>
Signed-off-by: Bob Glossman <bogl@whamcloud.com>
Change-Id: Ia55e161ca118dad66a3136850de2f0e8f7aa5206
Reviewed-on: http://review.whamcloud.com/3450
Tested-by: Hudson
Reviewed-by: Mike Pershin <tappro@whamcloud.com>
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: Oleg Drokin <green@whamcloud.com>
12 years agoLU-974 security: ignore umask if acl enabled
Lai Siyao [Fri, 13 Jan 2012 10:34:33 +0000 (18:34 +0800)]
LU-974 security: ignore umask if acl enabled

Backport commit: Ibbb45dd79378d116eb428b76070a417fe08a8142.

* add OBD_CONNECT_UMASK to show whether MDS supports umask.
* client packs umask in create/open request, MDS will use it in
  object creation.
* client ignores umask if acl is enabled.
* client enforces umask if MDS doesn't support OBD_CONNECT_UMASK.
* don't update inode->i_mode after creation because la_mode may
  not be correct which doesn't consider umask.
* add an acl test for this.

Signed-off-by: Lai Siyao <laisiyao@whamcloud.com>
Change-Id: Ief42dd27fa500c2f13e7dc672c37744a8ecbf489
Reviewed-on: http://review.whamcloud.com/4660
Tested-by: Hudson
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: Fan Yong <yong.fan@whamcloud.com>
Reviewed-by: Bobi Jam <bobijam@whamcloud.com>
Reviewed-by: Oleg Drokin <green@whamcloud.com>
12 years agoLU-1842 quota: t-f changes for new quota
Niu Yawei [Tue, 18 Sep 2012 15:00:12 +0000 (11:00 -0400)]
LU-1842 quota: t-f changes for new quota

Added new quota functions in the t-f according to new quota
architecutre, at the same time, old functions are kept for
the interoprability with old server.

two minor defects fixed:
- in facet_up(), add '-x' option to grep for exact matching,
  otherwise this function will not work in single node test;
- missing 'shift' in wait_update_facet();

Test-Parameters: envdefinitions=ENABLE_QUOTA=yes

Signed-off-by: Niu Yawei <niu@whamcloud.com>
Change-Id: I86915976f53af8620679ab267e04beea18dd5cf3
Reviewed-on: http://review.whamcloud.com/4031
Reviewed-by: Yu Jian <yujian@whamcloud.com>
Tested-by: Hudson
Reviewed-by: Johann Lombardi <johann@whamcloud.com>
Tested-by: Maloo <whamcloud.maloo@gmail.com>
12 years agoLU-1363 llite: Not held lock when calling security_d_instantiate
Bobi Jam [Thu, 3 May 2012 15:28:26 +0000 (23:28 +0800)]
LU-1363 llite: Not held lock when calling security_d_instantiate

security_d_instantiate() could allocate momory and/or yield cpu, don't
held spin lock.

Signed-off-by: Bobi Jam <bobijam@whamcloud.com>
Signed-off-by: Bob Glossman <bob.glossman@intel.com>
Change-Id: Ifda26573a1c371de119fa922321dc0de83fb4b57
Reviewed-on: http://review.whamcloud.com/4517
Tested-by: Hudson
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: Fan Yong <yong.fan@whamcloud.com>
Reviewed-by: Oleg Drokin <green@whamcloud.com>
12 years agoLU-1592 ldlm: protect obd_export:exp_imp_reverse's change
Bobi Jam [Thu, 16 Aug 2012 07:52:09 +0000 (15:52 +0800)]
LU-1592 ldlm: protect obd_export:exp_imp_reverse's change

* Protect obd_export::exp_imp_reverse from reconnect and destroy race.
* Add an assertion in class_import_put() to catch race in the first
  place.

Signed-off-by: Bobi Jam <bobijam@whamcloud.com>
Change-Id: If0abf6717456931c567d8d41c1d20fe49452e959
Reviewed-on: http://review.whamcloud.com/3869
Tested-by: Hudson
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: Keith Mannthey <keith.mannthey@intel.com>
Reviewed-by: Fan Yong <yong.fan@whamcloud.com>
Reviewed-by: Oleg Drokin <green@whamcloud.com>
12 years agoLU-1597 obdfilter: fix some capa code for OST
Bobi Jam [Thu, 12 Jul 2012 03:51:20 +0000 (11:51 +0800)]
LU-1597 obdfilter: fix some capa code for OST

* A capability should be set for filter_sync(), and when the operation
  is come from OSS itself, the capability check can be passed.
* filter_capa_fixoa() need check whether filter enabled capability.

port of master commit 27fe2e8d24e70af2377a52d411d53e332c3a3406

Signed-off-by: Bobi Jam <bobijam@whamcloud.com>
Change-Id: Ie27497d2b61654a7f24c06f6975e8bfaf5e9ae13
Reviewed-on: http://review.whamcloud.com/3383
Tested-by: Hudson
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: Lai Siyao <laisiyao@whamcloud.com>
Reviewed-by: Oleg Drokin <green@whamcloud.com>
12 years agoLU-1716 test: Verify connection flags race
Andriy Skulysh [Wed, 15 Aug 2012 12:22:01 +0000 (15:22 +0300)]
LU-1716 test: Verify connection flags race

Add OBD_TIMEOUT after setting improt to FULL state

Signed-off-by: Andriy Skulysh <Andriy_Skulysh@xyratex.com>
Xyratex-bug-id: MRP-577
Change-Id: I301abdb1857e4105210a4af71bd57bbda89d1abb
Reviewed-on: http://review.whamcloud.com/3654
Tested-by: Hudson
Reviewed-by: Andreas Dilger <adilger@whamcloud.com>
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: Oleg Drokin <green@whamcloud.com>
12 years agoLU-1548 osd: move i_htree_lock to iam container
wangdi [Tue, 7 Aug 2012 21:40:45 +0000 (14:40 -0700)]
LU-1548 osd: move i_htree_lock to iam container

Move i_tree_lock from ldiskfs_inode to iam_container,
so to reduce ldiskfs_inode_info size.

Signed-off-by: wang di <di.wang@whamcloud.com>
Change-Id: I4277e73347380bcb3fd34e3d76f66b1d6ec881f5
Reviewed-on: http://review.whamcloud.com/3561
Tested-by: Hudson
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: Fan Yong <yong.fan@whamcloud.com>
Reviewed-by: Andreas Dilger <adilger@whamcloud.com>
Reviewed-on: http://review.whamcloud.com/4515
Reviewed-by: Bob Glossman <bob.glossman@intel.com>
12 years agoLU-32 osd: keep root node BH ref of IAM container
Liang Zhen [Mon, 27 Dec 2010 05:35:20 +0000 (13:35 +0800)]
LU-32 osd: keep root node BH ref of IAM container

IAM in ldiskfs-osd will always consume some slots in bh_lru (see:
fs/buffer.c), if we keep buffer_head reference on root node,
we can save one slot in bh_lru and could be somehow helpful for
overall performance, I did some tests, LRU hits rate increased
5%-10% while creating files if we always keep this reference.

Signed-off-by: Liang Zhen <liang@whamcloud.com>
Change-Id: I954f26932462169c9bfc6bbfe1a57b5348624179
Reviewed-on: http://review.whamcloud.com/1826
Tested-by: Hudson
Reviewed-by: Bobi Jam <bobijam@whamcloud.com>
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: Fan Yong <yong.fan@whamcloud.com>
Reviewed-by: Oleg Drokin <green@whamcloud.com>
Reviewed-on: http://review.whamcloud.com/4514
Reviewed-by: Bob Glossman <bob.glossman@intel.com>
Reviewed-by: Andreas Dilger <adilger@whamcloud.com>
12 years agoLU-1623 mdt: Atomically update MDT export connection flags
Ned Bass [Fri, 26 Oct 2012 22:32:26 +0000 (15:32 -0700)]
LU-1623 mdt: Atomically update MDT export connection flags

MDT processing of connect requests currently updates the export
connection flags in two steps: client/server feature matching is
performed first, then much later various security-related bits are
removed as needed.  Certain error paths may leave the export flags
partially initialized.

A problem arises if multiple connect requests from the same client are
handled out of order, as may occur due to network disruptions. If the
last such request to be handled has a lower connection count than one
that already completed, it will be aborted with -EALREADY after having
modified the connection flags in the export.  However, the
security-related flags are left with incorrect values, as the
top-level connect handler skips setting these in the error path.
Replies to subsequent client requests may then contain unexpected
security information, causing the client to crash.

Similar issues may exist with other target types having non-atomic
export flag updates, and these should be addressed in follow-up
patches.

This patch makes the following changes:

- To avoid the problem described above, update the export connection
  flags atomically, and only in the successful case.

- To make this important atomic operation more conspicuous, move it
  from mdt_init_sec_level() to the top-level handler mdt_connect().

- Add a comment to mdt_connect_internal(), and delete a disabled code
  block from it.

- Correct debug message in target_handle_connect() to match code.

port of master patch dc2e00df1892a287eaf9ad6fad557f2cd28c90c7

Signed-off-by: Ned Bass <bass6@llnl.gov>
LLNL-bug-id: bz1711
Change-Id: Ie75ba0839c18ff8cae3bbf7358fdd30129b5a3a9
Reviewed-on: http://review.whamcloud.com/4688
Tested-by: Hudson
Reviewed-by: Oleg Drokin <green@whamcloud.com>
Reviewed-by: Andreas Dilger <adilger@whamcloud.com>
Tested-by: Maloo <whamcloud.maloo@gmail.com>
12 years agoLU-1517 ptlrpc: throw net error to ptlrpc for bulk
Alexander.Boyko [Mon, 17 Sep 2012 13:48:59 +0000 (17:48 +0400)]
LU-1517 ptlrpc: throw net error to ptlrpc for bulk

Start reconnect and resend if network error occures
for the bulk transfer.

Signed-off-by: Alexander Boyko <alexander_boyko@xyratex.com>
Change-Id: I0cf2ee1230a039336f081fbb520c1ce768882088
Xyratex-bug-id: MRP-523
Reviewed-on: http://review.whamcloud.com/4296
Tested-by: Hudson
Reviewed-by: Johann Lombardi <johann.lombardi@intel.com>
Tested-by: Maloo <whamcloud.maloo@gmail.com>
12 years agoLU-1039 ptlrpc: handle bulk IO errors correctly.
Emoly Liu [Fri, 9 Nov 2012 06:51:29 +0000 (14:51 +0800)]
LU-1039 ptlrpc: handle bulk IO errors correctly.

Don't panic on incorrect bulk transfer,
correctly handle a bulk request reorder.

LustreError: 12927:0:(client.c:1696:ptlrpc_check_set())LBUG
Pid: 12927, comm: ptlrpcd-brw

Call Trace:
[<ffffffffa083c865>] libcfs_debug_dumpstack+0x55/0x80 [libcfs]
[<ffffffffa083cea5>] lbug_with_loc+0x75/0xe0 [libcfs]
[<ffffffffa0e2edd6>] ptlrpc_check_set+0x1a66/0x1b90 [ptlrpc]
[<ffffffffa0e61006>] ? ptlrpcd_check+0x46/0x290 [ptlrpc]
[<ffffffffa0e61006>] ? ptlrpcd_check+0x46/0x290 [ptlrpc]
[<ffffffffa0e611b8>] ptlrpcd_check+0x1f8/0x290 [ptlrpc]
[<ffffffffa0e6157b>] ptlrpcd+0x32b/0x3b0 [ptlrpc]
[<ffffffff81056388>] ? finish_task_switch+0x48/0x110
[<ffffffff8105f500>] ? default_wake_function+0x0/0x20
[<ffffffffa0e61250>] ? ptlrpcd+0x0/0x3b0 [ptlrpc]
[<ffffffff8100c2ca>] child_rip+0xa/0x20
[<ffffffff81500b70>] ? _spin_unlock_irq+0x30/0x40
[<ffffffff8100bc10>] ? restore_args+0x0/0x30
[<ffffffffa0e61250>] ? ptlrpcd+0x0/0x3b0 [ptlrpc]
[<ffffffff8100c2c0>] ? child_rip+0x0/0x20

port of master patch c9590221dc43dd5e7a7ede389f0a7d9cf566e5bf

Xyratex-bug-id: MRP-303
Signed-off-by: Alexey Lyashkov <alexey_lyashkov@xyratex.com>
Signed-off-by: Oleg Drokin <green@whamcloud.com>
Signed-off-by: Liu Ying <emoly.liu@intel.com>
Change-Id: Ie6d6c0f024605b2c6a7302cb5c424610b098f3e2
Reviewed-on: http://review.whamcloud.com/4499
Tested-by: Hudson
Tested-by: Maloo <whamcloud.maloo@gmail.com>
12 years agoLU-1115 kernel: software raid6 related BUG
Emoly Liu [Wed, 14 Nov 2012 01:32:11 +0000 (09:32 +0800)]
LU-1115 kernel: software raid6 related BUG

Software raid6 hit BUGON in fs/bio.c:222 when raid chunk > 64k.
We pull upstream patch: 5b99c2ffa980528a197f26c7d876cceeccce8dd5
to deal with this issue.

port of b1_8 patch 747d905b631d90ce82521aac6a11d5d0eb0f45d8

Signed-off-by: yang sheng <ys@whamcloud.com>
Signed-off-by: Liu Ying <emoly.liu@intel.com>
Change-Id: I2b971f49dd621240460c9b9bf10f73f3f0bce32d
Reviewed-on: http://review.whamcloud.com/4526
Tested-by: Hudson
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: Yang Sheng <yang.sheng@intel.com>
Reviewed-by: Johann Lombardi <johann.lombardi@intel.com>
12 years agoLU-812 replace get_sb with mount in struct file_system_type
James Simmons [Tue, 24 Jul 2012 14:39:22 +0000 (10:39 -0400)]
LU-812 replace get_sb with mount in struct file_system_type

get_sb() has been removed from struct file_system_type since
2.6.39 and switched to mount().

Signed-off-by: James Simmons <uja.ornl@gmail.com>
Signed-off-by: Wally Wang <wang@cray.com>
Signed-off-by: Bob Glossman <bogl@whamcloud.com>
Change-Id: I6b0762828b26f065732f7f0c528e0b20d41cb710
Reviewed-on: http://review.whamcloud.com/3661
Tested-by: Hudson
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: Andreas Dilger <adilger@whamcloud.com>
12 years agoLU-1283 utils: Use variable without initialize
Minh Diep [Wed, 4 Apr 2012 19:38:10 +0000 (12:38 -0700)]
LU-1283 utils: Use variable without initialize

We need to initialize char *savept before use it

Signed-off-by: Minh Diep <mdiep@whamcloud.com>
Change-Id: I0c8155d84e255fcf55e98f5b84a09faf8f8487ef
Reviewed-on: http://review.whamcloud.com/2448
Tested-by: Hudson
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: Cliff White <cliffw@whamcloud.com>
Reviewed-by: Andreas Dilger <adilger@whamcloud.com>
12 years agoLU-2007 ldlm: reconnect can be racy
Alex Zhuravlev [Sat, 4 Aug 2012 08:14:31 +0000 (12:14 +0400)]
LU-2007 ldlm: reconnect can be racy

exp_connecting should be checked under spinlock

Signed-off-by: Alex Zhuravlev <alexey.zhuravlev@intel.com>
Signed-off-by: Prakash Surya <surya1@llnl.gov>
Change-Id: I271bc349ae1ad63a4acd1042f30ac7aa7176f105
Reviewed-on: http://review.whamcloud.com/4075
Tested-by: Hudson
Reviewed-by: Oleg Drokin <green@whamcloud.com>
Reviewed-by: Mike Pershin <tappro@whamcloud.com>
Tested-by: Maloo <whamcloud.maloo@gmail.com>
12 years agoLU-1479 build: enable srp initiator module with lbuild
Shuichi Ihara [Wed, 6 Jun 2012 15:08:20 +0000 (00:08 +0900)]
LU-1479 build: enable srp initiator module with lbuild

kernel-ib doesn't include ib_srp module if RPMs built by lbuild.
It's required for Infiniband based storage.

Signed-off-by: Shuichi Ihara <sihara@ddn.com>
Change-Id: I16e940592f93abc9150f391316b8f55eb60c663a
Reviewed-on: http://review.whamcloud.com/3049
Tested-by: Hudson
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: Minh Diep <mdiep@whamcloud.com>
Reviewed-by: Oleg Drokin <green@whamcloud.com>
12 years agoLU-1923 lov: verify stripe is on given device
Andreas Dilger [Thu, 13 Sep 2012 02:35:55 +0000 (20:35 -0600)]
LU-1923 lov: verify stripe is on given device

When restarting FIEMAP from a file with many extents, the first
supplied fiemap extent contains the offset of the previous last
extent found.  If this data is invalid, it may cause the LOV code
to acccess out-of-bounds array indices.

Verify data passed from userspace is within bounds.

Signed-off-by: Andreas Dilger <adilger@whamcloud.com>
Change-Id: I8e70e891b9f23c8f72aa78a4807369584ac2b04f
Reviewed-on: http://review.whamcloud.com/3962
Tested-by: Hudson
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: Oleg Drokin <green@whamcloud.com>
Reviewed-by: wangdi <di.wang@whamcloud.com>
12 years agoLU-1688 test: fix the problem in test_58 of recovery_small
Hongchao Zhang [Sat, 4 Aug 2012 10:21:12 +0000 (18:21 +0800)]
LU-1688 test: fix the problem in test_58 of recovery_small

in test_58 of recovery_small, the 'df' command could trigger
the eviction caused by the above 'drop_bl_callback', then
the fail of 'df' could be not thought as the fail of th test,

Signed-off-by: Hongchao Zhang <hongchao.zhang@whamcloud.com>
Change-Id: I226a3d4772d033182ded28caacbecdc44c7af8c4
Reviewed-on: http://review.whamcloud.com/3506
Tested-by: Hudson
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: Jay J Lan <jay.j.lan@nasa.gov>
Reviewed-by: Andreas Dilger <adilger@whamcloud.com>
12 years agoLU-1438 quota: quota active checking is missed on slave
Niu Yawei [Wed, 13 Jun 2012 02:42:01 +0000 (19:42 -0700)]
LU-1438 quota: quota active checking is missed on slave

On quota slave, we missed checking if quota is enabled in the
quota_check_common() and several other places. Which could cause
slave retry acquire quota in quota_chk_acq_common() infinitely
when the quota is already turned off on master.

Signed-off-by: Niu Yawei <niu@whamcloud.com>
Change-Id: Iaa48c7cca05daf595b6d3b7e4025c7650e460918
Reviewed-on: http://review.whamcloud.com/3097
Tested-by: Hudson
Reviewed-by: Fan Yong <yong.fan@whamcloud.com>
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: Johann Lombardi <johann@whamcloud.com>
12 years agoLU-1438 quota: fix race in quota_chk_acq_common()
Niu Yawei [Fri, 1 Jun 2012 06:28:45 +0000 (23:28 -0700)]
LU-1438 quota: fix race in quota_chk_acq_common()

quota_check_common() & qctxt_adjust_qunit() uses different way
to check if quota is enforced on certain ID, which could result
in infinite loop in quota_chk_acq_common() when the QB/QI_SET
flag is cleared just after checking.

This patch used a non-instrusive way to fix this rare race.

Signed-off-by: Niu Yawei <niu@whamcloud.com>
Change-Id: Ia8c5cb593abf515cf4dc041c63c8a247ebe0cd03
Reviewed-on: http://review.whamcloud.com/2996
Reviewed-by: Johann Lombardi <johann@whamcloud.com>
Reviewed-by: Fan Yong <yong.fan@whamcloud.com>
Tested-by: Hudson
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: Oleg Drokin <green@whamcloud.com>
12 years agoLU-1518 mdd: Fixup mdd_{obf,dot_lustre}_obj_ops.
John L. Hammond [Wed, 13 Jun 2012 16:20:12 +0000 (11:20 -0500)]
LU-1518 mdd: Fixup mdd_{obf,dot_lustre}_obj_ops.

Define several missing md_object ops for .lustre/fid.  Unify
attribute handling for .lustre with that of normal md_objects.

Change-Id: I892904af4ef01b3687e60a41250b018d67c86734
Signed-off-by: John L. Hammond <jhammond@tacc.utexas.edu>
Reviewed-on: http://review.whamcloud.com/3103
Tested-by: Hudson
Reviewed-by: Andreas Dilger <adilger@whamcloud.com>
Reviewed-by: wangdi <di.wang@whamcloud.com>
Reviewed-by: Lai Siyao <laisiyao@whamcloud.com>
Reviewed-by: Alex Zhuravlev <bzzz@whamcloud.com>
Tested-by: Maloo <whamcloud.maloo@gmail.com>
12 years agoLU-1606 api: Include stddef.h in ll_fiemap.h
Christopher J. Morrone [Wed, 18 Jul 2012 00:40:10 +0000 (17:40 -0700)]
LU-1606 api: Include stddef.h in ll_fiemap.h

In user space only, include stddef.h in ll_fiemap.h to define
such things as size_t.

Signed-off-by: Christopher J. Morrone <morrone2@llnl.gov>
Change-Id: Ia5efa7758d27952009a50c479b3e02d4b6b7c9b6
Reviewed-on: http://review.whamcloud.com/3425
Tested-by: Hudson
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: Andreas Dilger <adilger@whamcloud.com>
Reviewed-by: Li Wei <liwei@whamcloud.com>
Reviewed-by: Oleg Drokin <green@whamcloud.com>
12 years agoLU-549 llite: Improve statfs performance if selinux is disabled
Keith [Thu, 28 Jun 2012 20:24:54 +0000 (13:24 -0700)]
LU-549 llite: Improve statfs performance if selinux is disabled

Don't disable all security xattrs when selinux is disabled just
disable security.selinux. Disabling all security xattrs was
overreaching and causing issues at a customer site.

This patch is built ontop of the original patch:
commit 7cc542fd4c26ccb117ceb13a47ac8ced3107b9b3.

Signed-off-by: Keith Mannthey <keith@whamcloud.com>
Change-Id: I73482d2ff1f166a48dceb3737df0e9e1b187dc85
Reviewed-on: http://review.whamcloud.com/2955
Reviewed-by: Andreas Dilger <adilger@whamcloud.com>
Tested-by: Hudson
Reviewed-by: Fan Yong <yong.fan@whamcloud.com>
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: Oleg Drokin <green@whamcloud.com>
12 years agoLU-1632 fid: remove fid_delete in delete_inode procedure
wangdi [Mon, 16 Jul 2012 17:27:56 +0000 (10:27 -0700)]
LU-1632 fid: remove fid_delete in delete_inode procedure

Remove fid_delete in delete_inode procedure, which is
completely obsolete in 2.x infrastructure , especially
when lmv layer is not being created during upgrade,
because it will cause the client fid space to be zero
out whenever an inode is being deleted.

Signed-off-by: Wang Di <di.wang@whamcloud.com>
Change-Id: I07a59da0db2ac284eb99f78a24b0e1af3080e2ca
Reviewed-on: http://review.whamcloud.com/3422
Tested-by: Hudson
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: Fan Yong <yong.fan@whamcloud.com>
Reviewed-by: Oleg Drokin <green@whamcloud.com>
12 years agoLU-2237 tests: new test for re-recreating last_rcvd
Fan Yong [Mon, 29 Oct 2012 15:58:03 +0000 (08:58 -0700)]
LU-2237 tests: new test for re-recreating last_rcvd

To verify that the lost "last_rcvd" file can be re-created
when server mount.

Signed-off-by: Fan Yong <yong.fan@whamcloud.com>
Change-Id: Ie6d23cafc511d2a99abfc7967cafb2ae0106c18c
Reviewed-on: http://review.whamcloud.com/4397
Tested-by: Hudson
Reviewed-by: Andreas Dilger <adilger@whamcloud.com>
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: Yu Jian <yujian@whamcloud.com>
Reviewed-by: Alex Zhuravlev <bzzz@whamcloud.com>
12 years agoLU-1203 mdt: map old params to new ones by using an array
Yu Jian [Fri, 7 Sep 2012 03:17:07 +0000 (11:17 +0800)]
LU-1203 mdt: map old params to new ones by using an array

This patch improves mdt_process_config() to use an array for
mapping old params to new ones.

The patch also adds a common function class_find_old_param()
to check whether a proc param is an old one or not, and return
new one if it's an old one.

Signed-off-by: Yu Jian <yujian@whamcloud.com>
Change-Id: If7a79cfa20c7cb57246f115b4af71ff996b77d8d
Reviewed-on: http://review.whamcloud.com/3900
Tested-by: Hudson
Reviewed-by: Liang Zhen <liang@whamcloud.com>
Reviewed-by: Niu Yawei <niu@whamcloud.com>
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: Oleg Drokin <green@whamcloud.com>
12 years agoLU-1057 quota: speed up lookup in osc_quota_chkdq
Gregoire Pichon [Thu, 4 Oct 2012 14:55:26 +0000 (16:55 +0200)]
LU-1057 quota: speed up lookup in osc_quota_chkdq

This patch replace the global hash table used to store uid/gid
about to run out of quota space with a per-OSC cfs_hash.

Signed-off-by: Gregoire Pichon <gregoire.pichon@bull.net>
Change-Id: If291ce125d79312ba7abc182a5561657fa63b6bd
Reviewed-on: http://review.whamcloud.com/4184
Tested-by: Hudson
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: Johann Lombardi <johann.lombardi@intel.com>
Reviewed-by: Niu Yawei <niu@whamcloud.com>
12 years agoLU-1342 test: sanity-quota test_29
Bobi Jam [Wed, 25 Apr 2012 13:44:09 +0000 (21:44 +0800)]
LU-1342 test: sanity-quota test_29

sanity-quota test_29() needs sleep enough seconds to take into account
both service time and network latency.

Signed-off-by: Bobi Jam <bobijam@whamcloud.com>
Change-Id: Ide2b321fbd104eede17eee109998d40d98fb2d30
Reviewed-on: http://review.whamcloud.com/3870
Tested-by: Hudson
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: Jay J Lan <jay.j.lan@nasa.gov>
Reviewed-by: Yu Jian <yujian@whamcloud.com>
Reviewed-by: Oleg Drokin <green@whamcloud.com>
12 years agoLU-1720 kernel: Quota doesn't work over 4TB on single OST
yangsheng [Mon, 8 Oct 2012 04:25:10 +0000 (12:25 +0800)]
LU-1720 kernel: Quota doesn't work over 4TB on single OST

Fix previous kernel update patch wrong update chunk.

Signed-off-by: yang sheng <ys@whamcloud.com>
Change-Id: I3253469dd285e094f339a17e73c406b008a66195
Reviewed-on: http://review.whamcloud.com/3600
Tested-by: Hudson
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: Niu Yawei <niu@whamcloud.com>
Reviewed-by: Johann Lombardi <johann.lombardi@intel.com>
12 years agoLU-1789 protocol: reserve connect flag for lightweight conn
Johann Lombardi [Tue, 4 Sep 2012 07:52:00 +0000 (09:52 +0200)]
LU-1789 protocol: reserve connect flag for lightweight conn

Reserve connection flag for lightweight connection support.
Although this feature will never be supported on 2.1, it still avoids
flag conflicts.

Signed-off-by: Johann Lombardi <johann@whamcloud.com>
Change-Id: I1b943579d8c46cce0d70a5df45d35874bf63b29b
Reviewed-on: http://review.whamcloud.com/3853
Tested-by: Hudson
Reviewed-by: Niu Yawei <niu@whamcloud.com>
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: Oleg Drokin <green@whamcloud.com>
12 years agoLU-2237 osd: skip OI ops for local objects
Fan Yong [Sat, 27 Oct 2012 07:45:27 +0000 (00:45 -0700)]
LU-2237 osd: skip OI ops for local objects

We should not add the FID mapping in the OI file for local object.
Otherwise it will cause OI lookup to return non-exist local object
when the local object lost for system crash, and it also prevents
new local object to be created, and then causes the server cannot
mountup.

This issue has been fixed in OI scrub project, which will be back
ported to lustre-2.1 soon. This is a temporary patch to allow the
current lustre-2.1 to be workable.

Signed-off-by: Fan Yong <yong.fan@whamcloud.com>
Change-Id: Ibe2861818aaef3842605fb7d4e24cc02dad22104
Reviewed-on: http://review.whamcloud.com/4395
Tested-by: Hudson
Reviewed-by: Andreas Dilger <adilger@whamcloud.com>
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: Alex Zhuravlev <bzzz@whamcloud.com>
12 years agoLU-1779 tests: fix run_one_logged() to log SKIP status
Yu Jian [Fri, 7 Sep 2012 02:58:35 +0000 (10:58 +0800)]
LU-1779 tests: fix run_one_logged() to log SKIP status

In the current test framework, only those tests which are in the
$ALWAYS_EXCEPT list are logged with SKIP status, other skipped
tests are all logged with PASS status.

This patch fixes the above issue by setting the SKIP status in
pass() and logging the status in run_one_logged().

Signed-off-by: Yu Jian <yujian@whamcloud.com>
Change-Id: I7d4e66982a04d8759e887d88d0e406da719c03bf
Reviewed-on: http://review.whamcloud.com/3899
Tested-by: Hudson
Reviewed-by: Andreas Dilger <adilger@whamcloud.com>
Tested-by: Maloo <whamcloud.maloo@gmail.com>