Whamcloud - gitweb
fs/lustre-release.git
4 years agoLU-8210 osd-ldiskfs: fix setting pages PageUptodate state 78/20478/6
Wang Shilong [Fri, 27 May 2016 12:40:26 +0000 (21:40 +0900)]
LU-8210 osd-ldiskfs: fix setting pages PageUptodate state

In our VM autotest
sanity:156 test failed in rhel7 plataform for writecache
disable and readcache enabled.

After debugging this is because of we miss to set PageUptodate
after reading bio finish, this will make read cache total
unusable for rhel7, because osd_read_prep() will think pages
always not uptodate and triger bio submitting for every reading
request.

Problems come from pages iterating, since bio_for_each_segment_all
introduced, bio_for_each_segment will no longer work right
for pages interating anymore, we should use new interface
for rhel7.

So this problems only exist for some specific Drivers:
From Documentation/block/biodoc.txt

-------------------------------
Drivers which can't process a large bio in one shot can
use the bi_idx field to keep track of the next bio_vec entry
to process.(e.g a 1MB bio_vec needs to be handled in max 128kB
chunks for IDE) [TBD: Should preferably also have a bi_voffset
and bi_vlen to avoid modifying bi_offset an len fields]
-------------------------------

This might explain why intel autotest could not reproduce it
but our clustered can always reproduce the problem.

Signed-off-by: Wang Shilong <wshilong@ddn.com>
Change-Id: I5d3de5bf5facbee8d9450c15b8ab7189d7f2fbfe
Reviewed-on: http://review.whamcloud.com/20478
Tested-by: Jenkins
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Li Xi <lixi@ddn.com>
Reviewed-by: Alex Zhuravlev <alexey.zhuravlev@intel.com>
4 years agoLU-8100 lmv: Correctly generate target_obd 36/20336/4
Giuseppe Di Natale [Thu, 12 May 2016 20:39:36 +0000 (13:39 -0700)]
LU-8100 lmv: Correctly generate target_obd

The target_obd proc file was not being generated correctly
in cases where nonconsecutive MDT indices were used when
generating a filesystem.

Signed-off-by: Giuseppe Di Natale <dinatale2@llnl.gov>
Change-Id: I4bd3aaffd56707ea29b9b2d4366de5e20b3d3760
Reviewed-on: http://review.whamcloud.com/20336
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
Reviewed-by: Lai Siyao <lai.siyao@intel.com>
Reviewed-by: James Simmons <uja.ornl@yahoo.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-8126 kernel: new kernel RHEL 6.8 [2.6.32-642.el6] 06/20106/10
Bob Glossman [Tue, 10 May 2016 17:08:59 +0000 (10:08 -0700)]
LU-8126 kernel: new kernel RHEL 6.8 [2.6.32-642.el6]

Changes to support new kernel version in new el6.8 release distro
New kernel-config and target files.
Revised base kernel and ldiskfs patches.
Small adjustments to lbuild and autoconf.

Test-Parameters: clientdistro=el6.8 mdsdistro=el6.8 ossdistro=el6.8

Signed-off-by: Bob Glossman <bob.glossman@intel.com>
Change-Id: I94c0c06f43ca2973133a1dabb0184079290dbc6a
Reviewed-on: http://review.whamcloud.com/20106
Tested-by: Jenkins
Reviewed-by: Minh Diep <minh.diep@intel.com>
Reviewed-by: Yang Sheng <yang.sheng@intel.com>
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-8071 ldiskfs: handle system freeze protection 62/20062/8
Fan Yong [Sun, 24 Apr 2016 06:28:02 +0000 (14:28 +0800)]
LU-8071 ldiskfs: handle system freeze protection

For RHEL6 2.6.32, or kernel 3.6 or newer, it supports wrapped
functions for filesystem freeze protection, such as sb_{start,
stop}_write(). Usually, they are called by VFS. Since Lustre
uses ldiskfs backend bypass VFS layer, it is the OSD's duty to
handle filesystem freeze protection by calling related wrapped
filesystem freeze protection functions. As for other kernel,
such as 3.0.101, related logic has been handled inside ldiskfs
via vfs_check_frozen().

Signed-off-by: Fan Yong <fan.yong@intel.com>
Change-Id: Idb2b4ca02d569ec818a06b534e857b6ee44f89a2
Reviewed-on: http://review.whamcloud.com/20062
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
Reviewed-by: Wang Shilong <wshilong@ddn.com>
4 years agoLU-7185 ldlm: Restore connect flags on failure 50/16950/5
Jeremy Filizetti [Mon, 26 Oct 2015 23:47:38 +0000 (19:47 -0400)]
LU-7185 ldlm: Restore connect flags on failure

Restore connect flags on failure of ptlrpc_connect_import()
to prevent an LBUG due to flags mismatch.

Signed-off-by: Jeremy Filizetti <jeremy.filizetti@gmail.com>
Change-Id: I9149c8d2f512c1f920b3bdfa8f3435b03dc72ab2
Reviewed-on: http://review.whamcloud.com/16950
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
Tested-by: Jenkins
Reviewed-by: Sebastien Buisson <sebastien.buisson@bull.net>
Reviewed-by: John L. Hammond <john.hammond@intel.com>
Tested-by: Maloo <hpdd-maloo@intel.com>
4 years agoLU-8147 osd-zfs: fix osd_mount error path 09/20309/3
Yang Sheng [Wed, 18 May 2016 15:52:29 +0000 (23:52 +0800)]
LU-8147 osd-zfs: fix osd_mount error path

The osd_oi_fini will grab osl_seq_list_lock for cleanup.
But this lock is not initilized if osd_mount return and
failed before osd_oi_init.

Signed-off-by: Yang Sheng <yang.sheng@intel.com>
Change-Id: I8f76a87e22d131e6ab9034d8ecc42b47e301ff25
Reviewed-on: http://review.whamcloud.com/20309
Tested-by: Jenkins
Reviewed-by: Mike Pershin <mike.pershin@intel.com>
Reviewed-by: John L. Hammond <john.hammond@intel.com>
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-5092 nodemap: transfer nodemaps between MGS, MDTs, and OSTs 30/11830/48
Kit Westneat [Sat, 20 Feb 2016 22:11:25 +0000 (17:11 -0500)]
LU-5092 nodemap: transfer nodemaps between MGS, MDTs, and OSTs

This creates a new config lock on the MGS for transferring the
nodemaps.  The target MGCs can enqueue this lock, which is then
revoked by the MGS when the nodemaps are modified. The MGCs then
issue a read config RPC on the nodemap config lock, which causes
the MGS to transfer the nodemap index file to the MGCs. The MGCs
then process the index file, similarly to the way the MGS
processes it on start.

Signed-off-by: Kit Westneat <kit.westneat@gmail.com>
Change-Id: I99a34773298484f70a912761f5831b75196d41d8
Reviewed-on: http://review.whamcloud.com/11830
Reviewed-by: John L. Hammond <john.hammond@intel.com>
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: James Simmons <uja.ornl@yahoo.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-7906 osd: no blocksize on non-OST objects 86/19286/4
Alex Zhuravlev [Fri, 1 Apr 2016 11:22:53 +0000 (14:22 +0300)]
LU-7906 osd: no blocksize on non-OST objects

osd-zfs should not set blocksize on non-OST objects.

Change-Id: Idafe2236a49ffcb86bf06971d3687c8fceb516a2
Signed-off-by: Alex Zhuravlev <alexey.zhuravlev@intel.com>
Reviewed-on: http://review.whamcloud.com/19286
Tested-by: Jenkins
Reviewed-by: Nathaniel Clark <nathaniel.l.clark@intel.com>
Reviewed-by: Jinshan Xiong <jinshan.xiong@intel.com>
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
Tested-by: Maloo <hpdd-maloo@intel.com>
4 years agoLU-7728 osp: soft lockup in osp_precreate_reserve() 44/18244/3
Andriy Skulysh [Mon, 1 Feb 2016 09:54:57 +0000 (11:54 +0200)]
LU-7728 osp: soft lockup in osp_precreate_reserve()

LU-4313 commit 3a2073d3bf9fab33fa26c5c5f0da2b0821185131
treats -ENOTCONN as recoverable error, so it is needed
to add it to osp_precreate_ready_condition() to wait
for an event instead of busy-looping.

Seagate-bug-id: MRP-3247
Change-Id: I8a0a77a7369be0325fbeb9d0a2c321950905afbb
Signed-off-by: Andriy Skulysh <andriy.skulysh@seagate.com>
Reviewed-on: http://review.whamcloud.com/18244
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Lai Siyao <lai.siyao@intel.com>
Reviewed-by: Mike Pershin <mike.pershin@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-4330 lnet: Allocate MEs and small MDs in own kmem_caches 86/18586/12
Bruno Faccini [Tue, 23 Feb 2016 19:38:47 +0000 (20:38 +0100)]
LU-4330 lnet: Allocate MEs and small MDs in own kmem_caches

As part of LU-3848 and LU-4330, it has been discovered that LNET
MEs and small MDs (<=128 Bytes) are allocated in <size-128> kmem_cache
and thus can suffer quite frequent corruptions, from other modules or
Kernel parts, that occur there. To avoid this, MEs and small-MDs
specific kmem_cache have been created.

Also, lnet_libmd struct fields have been re-ordered to optimize its
memory foot-print.

Signed-off-by: Bruno Faccini <bruno.faccini@intel.com>
Change-Id: I6cfbed0c3f0a4c2447fe22b2d0d30dba8e3f3173
Reviewed-on: http://review.whamcloud.com/18586
Tested-by: Jenkins
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
Reviewed-by: Doug Oucharek <doug.s.oucharek@intel.com>
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-7975 lod: fix delayed stripe error path & Client resend 02/19302/2
Bruno Faccini [Fri, 1 Apr 2016 22:11:36 +0000 (00:11 +0200)]
LU-7975 lod: fix delayed stripe error path & Client resend

This patch is a follow-on to LU-4260/LU-4971/LU-5346 to fix
cleanup in error path during layout/stripe creation.
It is intended to fix special case when layout creation has been
delayed and will be with a non-0 size (ie truncate).

Problem has been unveiled due to the current limitation where a
delayed layout creation with a non-0 size/truncate will not be
resent if connection with OST is down, but is rather returning
-EAGAIN/-EWOULDBLOCK upon next open, and finally MDS will crash for
"(lod_object.c:700:lod_ah_init()) ASSERTION( lc->ldo_stripenr == 0 )"
LBUG.

Thus, patch also fixes this limitation by instead returning
-EINPROGRESS,this to avoid using costly resend operation from MDS
(will keep a ptlrpc thread busy) but offload this responsibility to
Client side (in after_reply()) which will handle this process in a
much less agressive way.

Specific sanity/test_27F sub-test has been created to test both
layout creation error path correct cleanup and Client resend.

Signed-off-by: Bruno Faccini <bruno.faccini@intel.com>
Change-Id: Ibc1734f52c071c66cb5974e3fe21d8819d725c2a
Reviewed-on: http://review.whamcloud.com/19302
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Alex Zhuravlev <alexey.zhuravlev@intel.com>
Reviewed-by: Mike Pershin <mike.pershin@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-8141 tests: fix for acl test 98/20198/3
Bob Glossman [Tue, 10 Mar 2015 17:28:36 +0000 (10:28 -0700)]
LU-8141 tests: fix for acl test

Replace earlier fix for LU-3227 with a better version.
The old fix still allows some failures on sanity, test 103a

Lustre-commit: 30ee7bf159e53fc5b19a4a4605387d09191436e6
Lustre-change: http://review.whamcloud.com/14036

Signed-off-by: Yang Sheng <yang.sheng@intel.com>
Signed-off-by: Bob Glossman <bob.glossman@intel.com>
Reviewed-by: James Simmons <uja.ornl@gmail.com>
Reviewed-by: Jian Yu <jian.yu@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
Change-Id: I4d240eefc4f5d5364edd776aef76397202c36f51
Reviewed-on: http://review.whamcloud.com/20198
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Minh Diep <minh.diep@intel.com>
4 years agoLU-8030 hsm: prevent duplicated HSM requests 35/19635/8
Frank Zago [Mon, 18 Apr 2016 17:53:00 +0000 (13:53 -0400)]
LU-8030 hsm: prevent duplicated HSM requests

It is possible to insert identical HSM requests in the HSM action
list. The following command

  lfs hsm-archive foo foo foo foo

results in 4 new archive requests in the "actions" list for the same
file.

Identify such requests and suppress the duplicates.

Signed-off-by: frank zago <fzago@cray.com>
Change-Id: If83d2855948578b94babca96dc72c33f96ae4f71
Reviewed-on: http://review.whamcloud.com/19635
Tested-by: Jenkins
Reviewed-by: jacques-Charles Lafoucriere <jacques-charles.lafoucriere@cea.fr>
Reviewed-by: John L. Hammond <john.hammond@intel.com>
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-7134 utils: Ensure hostid set for ZFS during mkfs 11/16611/12
Nathaniel Clark [Tue, 15 Sep 2015 19:26:53 +0000 (15:26 -0400)]
LU-7134 utils: Ensure hostid set for ZFS during mkfs

Ensure import protection in ZFS.  spl_hostid must be set to a
non-zero value for it to work correctly.  Error out during mkfs and
tunefs unless override is passed.  Also print error during mount.

Ensure hostid is set in conf-sanity.sh

Signed-off-by: Nathaniel Clark <nathaniel.l.clark@intel.com>
Change-Id: If37318c20e4502a643992e6bc21ac4bc4ad20ed7
Reviewed-on: http://review.whamcloud.com/16611
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
Reviewed-by: Alex Zhuravlev <alexey.zhuravlev@intel.com>
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
4 years agoLU-5092 nodemap: users of ted_nodemap should take ref 74/19674/16
Kit Westneat [Wed, 20 Apr 2016 14:12:42 +0000 (10:12 -0400)]
LU-5092 nodemap: users of ted_nodemap should take ref

In order to quickly map clients, the client's assigned nodemap is
saved on the export data struct. When mapping occurs, a reference to
that nodemap is used for mapping. If no reference is taken during
mapping and the nodemap is put elsewhere, then the nodemap could be
destroyed before the mapping is finished. This patch ensures that a
reference is taken before mapping.

Signed-off-by: Kit Westneat <kit.westneat@gmail.com>
Change-Id: I5c4d707970b1a86fb061a501240182ff3dc7be51
Reviewed-on: http://review.whamcloud.com/19674
Reviewed-by: John L. Hammond <john.hammond@intel.com>
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
Tested-by: Jenkins
Reviewed-by: James Simmons <uja.ornl@yahoo.com>
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-8193 ptlrpc: set proper mbits for EINPROGRESS resend 77/20377/2
Niu Yawei [Mon, 23 May 2016 05:49:26 +0000 (01:49 -0400)]
LU-8193 ptlrpc: set proper mbits for EINPROGRESS resend

Set mbits for EINPROGRESS resend in ptl_send_rpc().

Signed-off-by: Niu Yawei <yawei.niu@intel.com>
Change-Id: If9db4a67530371ce5c2504c60ea79f9c9db6ce15
Reviewed-on: http://review.whamcloud.com/20377
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Liang Zhen <liang.zhen@intel.com>
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-7422 llite: don't panic when fid is insane 85/17985/3
Sergey Cheremencev [Mon, 16 Nov 2015 12:54:25 +0000 (15:54 +0300)]
LU-7422 llite: don't panic when fid is insane

LASSERT should never be done on data that is
received to over the network. Return EINVAL
when server returns invalid fid despite of
it_status == 0.

Change-Id: I4341d1002761ab4c86bd7af4d0ca5718fe2a3623
Signed-off-by: Sergey Cheremencev <sergey.cheremencev@seagate.com>
Seagate-bug-id: MRP-3073
Reviewed-on: http://review.whamcloud.com/17985
Tested-by: Jenkins
Reviewed-by: John L. Hammond <john.hammond@intel.com>
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-8022 lnet: Don't access NULL NI on failure path 14/19614/4
Doug Oucharek [Fri, 15 Apr 2016 18:37:54 +0000 (11:37 -0700)]
LU-8022 lnet: Don't access NULL NI on failure path

In kiblnd_passive_connect(), if we are failing the connection
attempt because we cannot find a valid NI (we have a NULL NI),
we were coring after the "goto fail" because the failure
path was assuming non-NULL NI.

This patch ensures we don't dereference a NULL NI on that
failure path.

Signed-off-by: Doug Oucharek <doug.s.oucharek@intel.com>
Change-Id: I4f3deb78b4d3e9d81551ba915ceee852ec3b0792
Reviewed-on: http://review.whamcloud.com/19614
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Dmitry Eremin <dmitry.eremin@intel.com>
Reviewed-by: James Simmons <uja.ornl@yahoo.com>
Reviewed-by: Matt Ezell <ezellma@ornl.gov>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-7848 target: Do not fail MDT-MDT connection 00/18800/5
Di Wang [Fri, 4 Mar 2016 18:01:48 +0000 (13:01 -0500)]
LU-7848 target: Do not fail MDT-MDT connection

When one MDT restarts, and try to connect to another
MDT with the new cookie, let's do not fail it, otherwise
it might cause eviction between MDT.

Signed-off-by: Di Wang <di.wang@intel.com>
Change-Id: I3d2e0712ee0ed05bdf46c7781e190206a6dcfb1c
Reviewed-on: http://review.whamcloud.com/18800
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
Reviewed-by: Mike Pershin <mike.pershin@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-4781 test: wait for dbench to stop 71/20471/2
John L. Hammond [Thu, 26 May 2016 19:34:42 +0000 (14:34 -0500)]
LU-4781 test: wait for dbench to stop

In lustre-rsync-test.sh after sending a SIGSTOP to dbench, wait for
all dbench processes to show a stopped state.

Test-Parameters: testlist=lustre-rsync-test,lustre-rsync-test,lustre-rsync-test,lustre-rsync-test,lustre-rsync-test,lustre-rsync-test
Signed-off-by: John L. Hammond <john.hammond@intel.com>
Change-Id: I22767b5002ca4e1b5b1d596380591dc5130ee11d
Reviewed-on: http://review.whamcloud.com/20471
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
Reviewed-by: Sebastien Buisson <sbuisson@ddn.com>
4 years agoLU-8155 tests: Fix timing of sanity-hsm/16 71/20271/4
Nathaniel Clark [Tue, 17 May 2016 15:46:22 +0000 (11:46 -0400)]
LU-8155 tests: Fix timing of sanity-hsm/16

sanity-hsm/16 occationally fails due to a timing issue.
The measured duration is 19s when the expected duration
is 20s, this is most likely due to rounding of half seconds.

Test-Parameters: trivial testlist=sanity-hsm

Signed-off-by: Nathaniel Clark <nathaniel.l.clark@intel.com>
Change-Id: Iff8b4dec6d9d562b1560cc0101f251bf4b039508
Reviewed-on: http://review.whamcloud.com/20271
Reviewed-by: John L. Hammond <john.hammond@intel.com>
Reviewed-by: Sebastien Buisson <sbuisson@ddn.com>
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
4 years agoLU-7352 tests: don't fail conf-sanity test_78 on ENOSPC 66/20166/7
Andreas Dilger [Fri, 13 May 2016 03:38:45 +0000 (21:38 -0600)]
LU-7352 tests: don't fail conf-sanity test_78 on ENOSPC

Add debugging to conf-sanity.sh test_78 to see why this test is
regularly reporting ENOSPC creating 100x 1MB test files, when
there should be many GB of space available during the test.

Don't fail the test if the OST runs out of space, or if the MDS
refuses to create more files when OST has no available blocks,
since the main goal of this test is to verify ldiskfs resizing.

Test-Parameters: trivial testlist=conf-sanity,conf-sanity,conf-sanity
Test-Parameters: testlist=conf-sanity,conf-sanity,conf-sanity,conf-sanity

Signed-off-by: Andreas Dilger <andreas.dilger@intel.com>
Change-Id: Id5a7049aa434c794a5a8f8aa5398bf7ca9bcab07
Reviewed-on: http://review.whamcloud.com/20166
Tested-by: Jenkins
Reviewed-by: Wang Shilong <wshilong@ddn.com>
Reviewed-by: Emoly Liu <emoly.liu@intel.com>
4 years agoLU-8049 obdclass: use __BIG_ENDIAN to detect local endianness 12/19812/3
Fan Yong [Fri, 8 Apr 2016 13:12:33 +0000 (21:12 +0800)]
LU-8049 obdclass: use __BIG_ENDIAN to detect local endianness

Originally, we used LUSTRE_MSG_MAGIC to detect the local endianness
as "if (LUSTRE_MSG_MAGIC != cpu_to_le32(LUSTRE_MSG_MAGIC))". But it
is not a good way. It is gratuitously confusing for no reason not
only because it has nothing specific to do with LUSTRE_MSG_MAGIC,
but also because this is a compile-time check but looks like runtime
check. This patch fixes it by checking "__BIG_ENDIAN".

Signed-off-by: Fan Yong <fan.yong@intel.com>
Change-Id: I7fc15890119b15c178da22b6dc6eebe102403734
Reviewed-on: http://review.whamcloud.com/19812
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Niu Yawei <yawei.niu@intel.com>
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
4 years agoLU-8134 kernel: kernel update RHEL7.2 [3.10.0-327.18.2.el7] 06/20206/4
Bob Glossman [Thu, 12 May 2016 14:00:08 +0000 (07:00 -0700)]
LU-8134 kernel: kernel update RHEL7.2 [3.10.0-327.18.2.el7]

Update RHEL7.2 kernel to 3.10.0-327.18.2.el7

Signed-off-by: Bob Glossman <bob.glossman@intel.com>
Change-Id: I3b587d11e9ad8051665f38c879c45f1ec5727213
Reviewed-on: http://review.whamcloud.com/20206
Reviewed-by: Minh Diep <minh.diep@intel.com>
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: John L. Hammond <john.hammond@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-8014 hsm: remove invalid kuc_free in coordinator 69/19869/3
Frank Zago [Thu, 28 Apr 2016 21:07:16 +0000 (17:07 -0400)]
LU-8014 hsm: remove invalid kuc_free in coordinator

Commit 9091a934fc ("LU-8014 hsm: remove kuc_ispayload") moved the KUC
allocation in mdt_hsm_agent_send() and I left a kuc_free in an error
path in mdt_coordinator(). The HAL is no longer a KUC and it is freed
outside the for loop. That kuc_free is incorrect.

Signed-off-by: frank zago <fzago@cray.com>
Change-Id: If0894d1a8bcbc7960b728e33a34bddb95806acf8
Reviewed-on: http://review.whamcloud.com/19869
Tested-by: Jenkins
Reviewed-by: Henri Doreau <henri.doreau@cea.fr>
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: John L. Hammond <john.hammond@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-8080 utils: Replace calls to signal with sigaction 55/19855/10
Quentin Bouget [Fri, 13 May 2016 07:47:33 +0000 (09:47 +0200)]
LU-8080 utils: Replace calls to signal with sigaction

The man page for signal (2) states that sigaction should replace
signal for portability. It also states:
"The effects of signal() in a multithreaded process are unspecified."

This patch affects utils/lhsmtool_posix.c for the second reason and
utils/liblustreapi_hsm.c for completeness even though: "The only
portable use of signal() is to set a signal’s disposition to
SIG_DFL or SIG_IGN." (which is what is done in liblustreapi_hsm.c)

Signed-off-by: Quentin Bouget <quentin.bouget.ocre@cea.fr>
Change-Id: Ib247fab937e3aac514f418c7260cdd67fc9e442c
Reviewed-on: http://review.whamcloud.com/19855
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
Tested-by: Jenkins
Reviewed-by: John L. Hammond <john.hammond@intel.com>
Reviewed-by: Henri Doreau <henri.doreau@cea.fr>
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-8078 iokit: correct obdfilter-survey output data format 41/19841/6
Emoly Liu [Thu, 28 Apr 2016 03:11:48 +0000 (11:11 +0800)]
LU-8078 iokit: correct obdfilter-survey output data format

A space is needed between min and max status numbers in the
obdfilter-survey script, otherwise plot-obdfilter can't parse
the summary file correctly, so that the related .dat files and
graphs can't be generated correctly.

Test-Parameters: trivial testlist=obdfilter-survey
Signed-off-by: Emoly Liu <emoly.liu@intel.com>
Change-Id: I5bf4790f2db1ee11cb6a7cc95d5527cb0d68e35a
Reviewed-on: http://review.whamcloud.com/19841
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Bobi Jam <bobijam@hotmail.com>
Reviewed-by: Jian Yu <jian.yu@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-7905 osd: pin OI objects 03/19103/8
Alex Zhuravlev [Wed, 23 Mar 2016 19:30:03 +0000 (22:30 +0300)]
LU-7905 osd: pin OI objects

so that corresponing dnodes are kept in memory.

Change-Id: Ic312bfc57e3adc1032cefdd97efcf8f819418dc8
Signed-off-by: Alex Zhuravlev <alexey.zhuravlev@intel.com>
Reviewed-on: http://review.whamcloud.com/19103
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Fan Yong <fan.yong@intel.com>
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-7795 quota: tuneable soft least qunit 96/18896/9
Niu Yawei [Mon, 14 Mar 2016 07:01:09 +0000 (03:01 -0400)]
LU-7795 quota: tuneable soft least qunit

Introduce a tuneable least qunit for soft block quota, which is
used when soft limit is exceeded to retain an acceptable write
performance.

Remove the stale declarations of quota proc function.

Signed-off-by: Niu Yawei <yawei.niu@intel.com>
Change-Id: I172d1d08494418a6f7bc261f022c587da34b784d
Reviewed-on: http://review.whamcloud.com/18896
Tested-by: Jenkins
Reviewed-by: Fan Yong <fan.yong@intel.com>
Reviewed-by: Wang Shilong <wshilong@ddn.com>
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-5810 tests: add client hostname to lctl mark 13/13113/4
Andreas Dilger [Tue, 16 Dec 2014 02:32:37 +0000 (19:32 -0700)]
LU-5810 tests: add client hostname to lctl mark

Improve debug messages to include the originating hostname.

Signed-off-by: Andreas Dilger <andreas.dilger@intel.com>
Change-Id: I441bf8294c38135276a5a0f0853dbebf4358c563
Reviewed-on: http://review.whamcloud.com/13113
Tested-by: Jenkins
Reviewed-by: Emoly Liu <emoly.liu@intel.com>
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: James Nunez <james.a.nunez@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-6215 lnet: split struct ib_send_wr 68/19168/10
Dmitry Eremin [Tue, 17 May 2016 13:22:15 +0000 (09:22 -0400)]
LU-6215 lnet: split struct ib_send_wr

In v4.4 Linux kernel split up struct ib_send_wr so that all non-trivial
verbs use their own structure which embeds struct ib_send_wr.

Linux-commit: e622f2f4ad2142d2a613a57fb85f8cf737935ef5
Change-Id: Iea2599ea57a576a4c061841be47a989aba097cc6
Signed-off-by: Dmitry Eremin <dmitry.eremin@intel.com>
Reviewed-on: http://review.whamcloud.com/19168
Reviewed-by: Doug Oucharek <doug.s.oucharek@intel.com>
Reviewed-by: James Simmons <uja.ornl@yahoo.com>
Reviewed-by: Li Dongyang <dongyang.li@anu.edu.au>
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-7469 test: add missing init to lustre-rsync-test:test_1a 55/17355/4
HemaHarish [Wed, 25 Nov 2015 11:25:54 +0000 (16:55 +0530)]
LU-7469 test: add missing init to lustre-rsync-test:test_1a

Target directory was not created for first time, so failed to write to
target location.

Seagate-bug-id: MRP-2382
Signed-off-by: Hema Yarramilli <hema.yarramilli@seagate.com>
Change-Id: I1192c413c8c173abdea3929dbaec19f1f5d20c92
Reviewed-on: http://review.whamcloud.com/17355
Reviewed-by: Nathaniel Clark <nathaniel.l.clark@intel.com>
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Saurabh Tandan <saurabh.tandan@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoRevert "LU-5050 libcfs: default CPT matches NUMA topology" 76/20376/2
Oleg Drokin [Mon, 23 May 2016 01:58:05 +0000 (01:58 +0000)]
Revert "LU-5050 libcfs: default CPT matches NUMA topology"

This causes rhel7 to crash on start in libcfs init
100% of the time.

This reverts commit 6b929502063ac0fa8e75a3e09259e9b2b0d0ee6e.

Change-Id: I9b6276380b580e9d76ae21bf389f5e7edc1a4f18
Reviewed-on: http://review.whamcloud.com/20376
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
Tested-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-7346 tests: Reintroduce SLOW tests to review process 43/17143/9
Saurabh Tandan [Thu, 12 Nov 2015 19:41:26 +0000 (12:41 -0700)]
LU-7346 tests: Reintroduce SLOW tests to review process

Tests running for five minutes and more are considered to
be eligible to be marked as SLOW. After running tests
multiple times it was found, there are certain tests
which take under five minutes and hence can be
reintroduced in the normal review process.

Signed-off-by: Saurabh Tandan <saurabh.tandan@intel.com>
Change-Id: If638e41a8bf6c9418d69257b00cad964d0810a02
Reviewed-on: http://review.whamcloud.com/17143
Tested-by: Jenkins
Reviewed-by: James Nunez <james.a.nunez@intel.com>
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Wei Liu <wei3.liu@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-8116 build: Shrink multi-line pclmulqdq warnings 54/20054/2
Christopher J. Morrone [Sat, 7 May 2016 01:53:07 +0000 (18:53 -0700)]
LU-8116 build: Shrink multi-line pclmulqdq warnings

The multi-line pclmulqdq warning in the configure script
are excessive.  Reduce them to a single line each.

Change-Id: Ifb92d257e41099204bda1392fa6fcfd044f4a5c1
Signed-off-by: Christopher J. Morrone <morrone2@llnl.gov>
Reviewed-on: http://review.whamcloud.com/20054
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Minh Diep <minh.diep@intel.com>
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
Reviewed-by: Bob Glossman <bob.glossman@intel.com>
Reviewed-by: Alexander Boyko <alexander.boyko@seagate.com>
4 years agoLU-7667 tests: Write to unique files in $TMP 61/18261/4
James Nunez [Tue, 2 Feb 2016 17:50:47 +0000 (10:50 -0700)]
LU-7667 tests: Write to unique files in $TMP

sanity uses $TMP/files to store the output of 'lfs getstripe'
and does not clean up the $TMP directory. When conf-sanity tries
to create $TMP/files, it fails. These tests need to use a unique
file, $tfile, or directory, $tdir, name in $TMP and should
clean $TMP out before test exits.

Tests should use $TMP instead of /tmp. $TMP defaults to /tmp.

Remove sanity test 51d from the ALWAYS_EXCEPT list.

Signed-off-by: James Nunez <james.a.nunez@intel.com>
Change-Id: I09a0a2be1d88c3f4a0042e3956bbc54f9df61f55
Reviewed-on: http://review.whamcloud.com/18261
Tested-by: Jenkins
Reviewed-by: Wei Liu <wei3.liu@intel.com>
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Saurabh Tandan <saurabh.tandan@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-5050 libcfs: default CPT matches NUMA topology 24/17824/3
Liang Zhen [Fri, 29 Apr 2016 17:45:39 +0000 (13:45 -0400)]
LU-5050 libcfs: default CPT matches NUMA topology

Change default value of CPT pattern and make it match NUMA topology

Signed-off-by: Liang Zhen <liang.zhen@intel.com>
Change-Id: Ia837ee4b1c52046fcac4a5c05d1c159001fbe810
Reviewed-on: http://review.whamcloud.com/17824
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: James Simmons <uja.ornl@yahoo.com>
Reviewed-by: Dmitry Eremin <dmitry.eremin@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-7403 dlm: const qualify struct lustre_handle * params 71/17071/3
John L. Hammond [Fri, 6 Nov 2015 17:17:54 +0000 (11:17 -0600)]
LU-7403 dlm: const qualify struct lustre_handle * params

Add a const qualifier to several struct lustre_handle * parameters in
the LDLM interface.

Signed-off-by: John L. Hammond <john.hammond@intel.com>
Change-Id: I65bc949af4ee78118ce9118801434b906dddf49b
Reviewed-on: http://review.whamcloud.com/17071
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
Reviewed-by: James Simmons <uja.ornl@yahoo.com>
4 years agoLU-7403 llite: change it_data to it_request 70/17070/5
John L. Hammond [Fri, 6 Nov 2015 16:43:37 +0000 (10:43 -0600)]
LU-7403 llite: change it_data to it_request

Change the void *it_data member of struct lookup_intent to struct
ptlrpc_request *it_request.

Signed-off-by: John L. Hammond <john.hammond@intel.com>
Change-Id: If3eb215620dfd376a18e4b15d7ab9f346468add0
Reviewed-on: http://review.whamcloud.com/17070
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
Reviewed-by: James Simmons <uja.ornl@yahoo.com>
4 years agoLU-8097 mdt: remove assertion for lock resent 74/19974/2
Di Wang [Mon, 25 Apr 2016 21:34:39 +0000 (17:34 -0400)]
LU-8097 mdt: remove assertion for lock resent

Do not assert for missing lock for resent, because the lock
might be destroyed by other thread in the mean time.

Signed-off-by: Di Wang <di.wang@intel.com>
Change-Id: If184684ec734e7162e570d3044d9ece5f3ac3567
Reviewed-on: http://review.whamcloud.com/19974
Tested-by: Jenkins
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Mike Pershin <mike.pershin@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-8098 doc: add "lfs mdts" to lfs main man page 80/20080/4
Emoly Liu [Thu, 12 May 2016 01:57:15 +0000 (09:57 +0800)]
LU-8098 doc: add "lfs mdts" to lfs main man page

Add "lfs mdts" to lfs.1.

Test-Parameters: trivial
Signed-off-by: Emoly Liu <emoly.liu@intel.com>
Change-Id: I0da79a56025b58bd32bdcee35c36e7021d69194f
Reviewed-on: http://review.whamcloud.com/20080
Reviewed-by: Giuseppe Di Natale <dinatale2@llnl.gov>
Reviewed-by: Bob Glossman <bob.glossman@intel.com>
Tested-by: Jenkins
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-8139 tests: skip sanity test 101g until fix is landed 41/20241/2
Andreas Dilger [Mon, 16 May 2016 17:46:00 +0000 (11:46 -0600)]
LU-8139 tests: skip sanity test 101g until fix is landed

The landing of 16MB RPC and lprocfs value parsing were tested
independently, but had conflicting changes that are causing
sanity.sh test_101g to fail continuously.  Disable the test
until a fix is made.

Test-Parameters: trivial
Signed-off-by: Andreas Dilger <andreas.dilger@intel.com>
Change-Id: I93e82cb5606400f39b6da7a6e35c580826c2d106
Reviewed-on: http://review.whamcloud.com/20241
Reviewed-by: James Nunez <james.a.nunez@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
Tested-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-8091 mount: error out properly in server_lsi2mti() 33/19933/2
Niu Yawei [Tue, 3 May 2016 07:37:41 +0000 (03:37 -0400)]
LU-8091 mount: error out properly in server_lsi2mti()

server_lsi2mti() should return an error if no proper NID is
initialized, otherwise, target will register successfully with
an invalid empty NID.

Signed-off-by: Niu Yawei <yawei.niu@intel.com>
Change-Id: Ia5fd78e4df6344764109486ce1082099054ebf1e
Reviewed-on: http://review.whamcloud.com/19933
Tested-by: Jenkins
Reviewed-by: Bobi Jam <bobijam@hotmail.com>
Reviewed-by: Lai Siyao <lai.siyao@intel.com>
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-8079 llog: Remove llog_cat_init_and_process 50/19850/4
Henri Doreau [Thu, 28 Apr 2016 11:01:16 +0000 (13:01 +0200)]
LU-8079 llog: Remove llog_cat_init_and_process

Unlike name suggests, this function does no longer process
the log but only initialize it. Replace it by direct calls
to the function it wraps.

Signed-off-by: Henri Doreau <henri.doreau@cea.fr>
Change-Id: Ib48e7d4cb15a2433cc07c6760e25450a9aa0a3b3
Reviewed-on: http://review.whamcloud.com/19850
Tested-by: Jenkins
Reviewed-by: John L. Hammond <john.hammond@intel.com>
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: James Nunez <james.a.nunez@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-6215 o2iblnd: port to new fast reg API introduced in 4.4 86/19186/5
Li Dongyang [Wed, 4 May 2016 17:11:15 +0000 (13:11 -0400)]
LU-6215 o2iblnd: port to new fast reg API introduced in 4.4

Remove the allocation of fastreg page list, as the page
vector is now private to the provider. Just pass tx_frags
to ib_map_mr_sg() and construct ib_reg_wr.
Defer the conversion of tx_frags to tx_pages and only do
it when the new API is not available.

Linux-commit: 4c67e2bfc8b7121d51434362fa7c2d012f8bcf1b
Linux-commit: 39bfc271bd687be2c8e396e976c0fb9a97963400

Signed-off-by: Li Dongyang <dongyang.li@anu.edu.au>
Change-Id: I5c62b0370ad6ddcc93102a29343491968e4446d0
Reviewed-on: http://review.whamcloud.com/19186
Tested-by: Jenkins
Reviewed-by: James Simmons <uja.ornl@yahoo.com>
Reviewed-by: Dmitry Eremin <dmitry.eremin@intel.com>
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-7888 obdclass: not hold global lock when lwp callback 34/19034/5
Fan Yong [Wed, 13 Apr 2016 09:37:18 +0000 (17:37 +0800)]
LU-7888 obdclass: not hold global lock when lwp callback

Originally, the lwp_notify_main() thread handled the every
registered lwp callback in turn with holding the global
lwp_register_list_lock. Because some lwp callback, such as
mdt_register_lwp_callback() may be blocked, then it will
cause others to be blocked when try to obtain such lock for
lustre_register_lwp_item(), and then block the mount.

This patch introduces reference count on the lwp_register_item
that will prevent the lwp item being freed during the callback,
so allow the lwp_notify_main() thread to do the real callback
without holding the global lock lwp_register_list_lock.

Intel-bug-Id: LDEV-323

Signed-off-by: Fan Yong <fan.yong@intel.com>
Change-Id: I1061205d5781e0699478d293e223567a0ebea286
Reviewed-on: http://review.whamcloud.com/19034
Tested-by: Jenkins
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Lai Siyao <lai.siyao@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-7733 ptlrpc: print times in microseconds 35/18335/5
Bob Glossman [Fri, 5 Feb 2016 20:47:48 +0000 (12:47 -0800)]
LU-7733 ptlrpc: print times in microseconds

report times and time diffs in seconds.microseconds instead of seconds

Signed-off-by: Bob Glosman <bob.glossman@intel.com>
Change-Id: I272626012a7162072fb3ad73912a56403afb6b39
Reviewed-on: http://review.whamcloud.com/18335
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
Reviewed-by: Minh Diep <minh.diep@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-8010 mdt: fix orphan layout_lock cases for restore 10/19710/7
Bruno Faccini [Thu, 21 Apr 2016 13:56:28 +0000 (15:56 +0200)]
LU-8010 mdt: fix orphan layout_lock cases for restore

Previously to this patch layout was not unlocked when a restore
was failed before being sent to CT, leading to a situation where
other requestors hang and also to an orphan restore_handle to be
kept on CDT's list of registered restore actions.
Only way to clear situation then, was to stop CDT.
This situation could at least occur if a restore was canceled
but the CT does not handle cancel operation, allowing the restore
to complete but also to have new restore requests to be
registered in the mean time and then to be failed due to
incompatible (no longer released) file state.

Also fix similar deadlock cases where layout lock was taken for
previously started restore requests upon CDT restart, by forcing
their replay/restart. This should also strengthen overall HSM
recovery process.

Signed-off-by: Bruno Faccini <bruno.faccini@intel.com>
Change-Id: Ib1ba9156793a230d256ff80d74372813f10b0321
Reviewed-on: http://review.whamcloud.com/19710
Tested-by: Jenkins
Reviewed-by: John L. Hammond <john.hammond@intel.com>
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: jacques-Charles Lafoucriere <jacques-charles.lafoucriere@cea.fr>
Reviewed-by: Aurelien Degremont <aurelien.degremont@cea.fr>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-8127 lfsck: remove XATTR_NAME_LFSCK_NAMESPACE_OLD 12/20112/4
Fan Yong [Tue, 19 Apr 2016 06:12:00 +0000 (14:12 +0800)]
LU-8127 lfsck: remove XATTR_NAME_LFSCK_NAMESPACE_OLD

We do not support the LFSCK compatibility between Lustre-2.9
and Lustre-2.6 any more. So drop related conditional compile
logic for XATTR_NAME_LFSCK_NAMESPACE_OLD.

Signed-off-by: Fan Yong <fan.yong@intel.com>
Change-Id: I3374ed357fc675b382fbb6703d0ba65c2d1420d9
Reviewed-on: http://review.whamcloud.com/20112
Reviewed-by: John L. Hammond <john.hammond@intel.com>
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
Tested-by: Jenkins
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
Tested-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-8093 kernel: kernel update RHEL 6.7 [2.6.32-573.26.1.el6] 84/19984/2
Bob Glossman [Wed, 4 May 2016 15:17:02 +0000 (08:17 -0700)]
LU-8093 kernel: kernel update RHEL 6.7 [2.6.32-573.26.1.el6]

Update RHEL 6.7 kernel to 2.6.32-573.26.1.el6

Signed-off-by: Bob Glossman <bob.glossman@intel.com>
Change-Id: I1ff675f734cb651d10860956c9d2a7edf191cbf2
Reviewed-on: http://review.whamcloud.com/19984
Reviewed-by: Yang Sheng <yang.sheng@intel.com>
Tested-by: Jenkins
Reviewed-by: Minh Diep <minh.diep@intel.com>
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-8094 ldlm: restore missing newlines 60/19960/2
Bob Glossman [Wed, 4 May 2016 01:27:04 +0000 (18:27 -0700)]
LU-8094 ldlm: restore missing newlines

Recent commit f28cc2 landed for LU-7521 removed too many newlines
in debug messages, leading to lots of noise in syslog.
This mod puts some back in.

Signed-off-by: Bob Glossman <bob.glossman@intel.com>
Change-Id: I4d3eb59a5db3ed548b03aa29ec036b4b2f899a86
Reviewed-on: http://review.whamcloud.com/19960
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
Tested-by: Jenkins
Reviewed-by: Alex Zhuravlev <alexey.zhuravlev@intel.com>
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: John L. Hammond <john.hammond@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-8041 mdd: increasing only atime update on close 32/19932/3
Niu Yawei [Tue, 3 May 2016 07:18:17 +0000 (03:18 -0400)]
LU-8041 mdd: increasing only atime update on close

The atime update on close should be increasing only, otherwise,
the atime could be overwritten by stale client's atime.

This patch also changed client to clear out the ATTR_ATIME
flag fro close RPC when atime == 0, so that patch client won't
overwrite the unpatched server with 0 atime.

A test case sanity 39p is added to verify that close won't
overwrite atime with zero, and sanity 39l is adjusted as well.

Signed-off-by: Niu Yawei <yawei.niu@intel.com>
Change-Id: I40d1103ddfea184d82170a6795f89ce303a5c14e
Reviewed-on: http://review.whamcloud.com/19932
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
Tested-by: Jenkins
Reviewed-by: Jinshan Xiong <jinshan.xiong@intel.com>
Tested-by: Maloo <hpdd-maloo@intel.com>
4 years agoLU-8035 obd: rename md_getstatus() to md_get_root() 24/19824/3
John L. Hammond [Fri, 29 Apr 2016 12:00:33 +0000 (07:00 -0500)]
LU-8035 obd: rename md_getstatus() to md_get_root()

Finish the partial renaming of the the OBD MD method md_getstatus() to
md_get_root().

Signed-off-by: John L. Hammond <john.hammond@intel.com>
Change-Id: Iadd785ce774377d3dba40f64dbddaaf68cfc8087
Reviewed-on: http://review.whamcloud.com/19824
Tested-by: Jenkins
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
Reviewed-by: Wang Shilong <wshilong@ddn.com>
Reviewed-by: James Simmons <uja.ornl@yahoo.com>
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-8027 llite: ensure obd is effective in onu_upcall 97/19597/3
Yang Sheng [Thu, 14 Apr 2016 20:05:24 +0000 (04:05 +0800)]
LU-8027 llite: ensure obd is effective in onu_upcall

The watched obd device may still not setup while onu_upcall
invoked. So we need verify it in cl_ocd_update.

Signed-off-by: Yang Sheng <yang.sheng@intel.com>
Change-Id: Iddda6837127f1066ede8430f355d8fd04ed3b71c
Reviewed-on: http://review.whamcloud.com/19597
Tested-by: Jenkins
Reviewed-by: Niu Yawei <yawei.niu@intel.com>
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Lai Siyao <lai.siyao@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-7870 osd-zfs: Build against SPL/ZFS 0.6.5.6 20/18920/9
Nathaniel Clark [Tue, 15 Mar 2016 15:37:32 +0000 (11:37 -0400)]
LU-7870 osd-zfs: Build against SPL/ZFS 0.6.5.6

Build against SPL/ZFS 0.6.5.6.

Fix issue with rpm's debugedit, where it can't account for path
canonicalization causes the path to shrink by only one characater.  So
a trailing slash will cause it choke.

ZFS Changelogs since 0.6.4.3:
https://github.com/zfsonlinux/zfs/releases/tag/zfs-0.6.5.6
https://github.com/zfsonlinux/zfs/releases/tag/zfs-0.6.5.5
https://github.com/zfsonlinux/zfs/releases/tag/zfs-0.6.5.4
https://github.com/zfsonlinux/zfs/releases/tag/zfs-0.6.5.3
https://github.com/zfsonlinux/zfs/releases/tag/zfs-0.6.5.2
https://github.com/zfsonlinux/zfs/releases/tag/zfs-0.6.5.1
https://github.com/zfsonlinux/zfs/releases/tag/zfs-0.6.5

Signed-off-by: Nathaniel Clark <nathaniel.l.clark@intel.com>
Change-Id: I26569a038df605b7b6a37e2806d477029d68fca0
Reviewed-on: http://review.whamcloud.com/18920
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Christopher J. Morrone <morrone2@llnl.gov>
Reviewed-by: Alex Zhuravlev <alexey.zhuravlev@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-7656 tests: tar fix for replay-single/70c 32/18732/3
Noopur Maheshwari [Tue, 1 Mar 2016 08:25:13 +0000 (13:55 +0530)]
LU-7656 tests: tar fix for replay-single/70c

The tar process runs in an infinite loop, and the removal/cleanup
of files interferes in the process and causes tar to fail.

The code flow could take two paths:
1. Mdt recovery has completed, catch exit signal.
In this case, put a check and confirm if the mdt recovery has
completed, if yes, then exit and perform cleanup
commands (remove files).

2. Main process code flow.
Wait for tar process to complete and then perform cleanup commands.

Also, instead of removing files within the tar process, keep the
removal of files in the cleanup part.

Seagate-bug-id: MRP-2925
Signed-off-by: Noopur Maheshwari <noopur.maheshwari@seagate.com>
Change-Id: Ia1c840dde554a2f1487aa9aebb70e5dcea29b806
Reviewed-on: http://review.whamcloud.com/18732
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Ujjwal Lanjewar <ujjwal@yahoo.com>
Reviewed-by: Wei Liu <wei3.liu@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-7334 lprocfs: Refactored string to value helpers 30/16930/13
Giuseppe Di Natale [Thu, 1 Oct 2015 22:51:19 +0000 (15:51 -0700)]
LU-7334 lprocfs: Refactored string to value helpers

Refactored the string parsing helpers to correct
inconsistent behavior and added handling for s64
overflow/underflow. Removed the old helper
functions and refactored functions that called
them.

Signed-off-by: Giuseppe Di Natale <dinatale2@llnl.gov>
Change-Id: I84f3b5d46d4e20bd09fe223589e1be96dc8f7468
Reviewed-on: http://review.whamcloud.com/16930
Reviewed-by: Christopher J. Morrone <morrone2@llnl.gov>
Tested-by: Jenkins
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-8048 mgc: use the same lu_env for mgc_fs_{set,clean}up 92/19692/2
Fan Yong [Sun, 3 Apr 2016 13:10:48 +0000 (21:10 +0800)]
LU-8048 mgc: use the same lu_env for mgc_fs_{set,clean}up

Originally, the mgc_fs_setup() and mgc_fs_cleanup use each own
lu_env. Sometimes the mgc_fs_cleanup() may fail to prepare the
lu_env. Such failure will cause local_oid_storage_fini() to be
skipped as to object leak found when osd_device_free().

This patch prepares the lu_env before calling mgs_fs_setup(),
then mgc_fs_setup() and mgc_fs_cleanup() can share the lu_env
and no need to worry about the failures in mgc_fs_cleanup().

Signed-off-by: Fan Yong <fan.yong@intel.com>
Change-Id: I7062abd670c53e93f456a8d8f1f6131d7ee92320
Reviewed-on: http://review.whamcloud.com/19692
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Dmitry Eremin <dmitry.eremin@intel.com>
Reviewed-by: Niu Yawei <yawei.niu@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-8127 lfsck: compile XATTR_NAME_LFSCK_NAMESPACE_OLD 09/20109/2
Fan Yong [Mon, 18 Apr 2016 10:45:02 +0000 (18:45 +0800)]
LU-8127 lfsck: compile XATTR_NAME_LFSCK_NAMESPACE_OLD

The logic related with XATTR_NAME_LFSCK_NAMESPACE_OLD is only
compiled for the version less than OBD_OCD_VERSION(2, 8, 53, 0).

Signed-off-by: Fan Yong <fan.yong@intel.com>
Change-Id: Ie0f9c0901e69465fe73051c16fa5e7f5997ef082
Reviewed-on: http://review.whamcloud.com/20109
Tested-by: Jenkins
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
4 years agoNew tag 2.8.53 2.8.53 v2_8_53 v2_8_53_0
Oleg Drokin [Wed, 11 May 2016 02:48:18 +0000 (22:48 -0400)]
New tag 2.8.53

Change-Id: I3f5aa724e3c629a97f51bdd885ca9b28db5de6ae
Signed-off-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-6937 test: use ps in get_agent_uuid() 93/20093/2
John L. Hammond [Tue, 10 May 2016 14:36:42 +0000 (09:36 -0500)]
LU-6937 test: use ps in get_agent_uuid()

Use 'ps -C $HSMTOOL_BASE -o args=' rather than 'pgrep -fl
$HSMTOOL_BASE' to get the copytool command line.

Signed-off-by: John L. Hammond <john.hammond@intel.com>
Change-Id: Idb1238966de1a8bea5d5c225632936d8c0a88529
Reviewed-on: http://review.whamcloud.com/20093
Reviewed-by: Minh Diep <minh.diep@intel.com>
Tested-by: Jenkins
Reviewed-by: Bob Glossman <bob.glossman@intel.com>
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-6844 tests: disable DNE testing of dbench 22/20022/2
Andreas Dilger [Fri, 6 May 2016 05:03:54 +0000 (23:03 -0600)]
LU-6844 tests: disable DNE testing of dbench

Temporarily disable DNE recovery testing with dbench until LU-6844
and related failures are fixed.

Test-Parameters: testlist=replay-single,replay-single,replay-single,replay-single,replay-single,replay-single
Signed-off-by: Andreas Dilger <andreas.dilger@intel.com>
Change-Id: I628642acb119121457ffb83d654f02e7ac40d056
Reviewed-on: http://review.whamcloud.com/20022
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: James Nunez <james.a.nunez@intel.com>
Reviewed-by: Sebastien Buisson <sbuisson@ddn.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-6215 llite: remove obsolete conditional code 90/19890/2
Andreas Dilger [Fri, 29 Apr 2016 20:48:52 +0000 (14:48 -0600)]
LU-6215 llite: remove obsolete conditional code

Remove conditional flock/aops code that was only for out-of-tree
vendor kernels but is not relevant anymore.

Linux-commit: 3746e6f93bbf28a25d2d69350ab6bfba02e14654

Signed-off-by: Andreas Dilger <andreas.dilger@intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Change-Id: If119a2da2bbc30a48d872029bd048ea90a3ebbe5
Reviewed-on: http://review.whamcloud.com/19890
Tested-by: Jenkins
Reviewed-by: James Simmons <uja.ornl@yahoo.com>
Reviewed-by: Frank Zago <fzago@cray.com>
Reviewed-by: John L. Hammond <john.hammond@intel.com>
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Dmitry Eremin <dmitry.eremin@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-7152 hsm: sync volatile file before setting times 41/19441/2
John L. Hammond [Mon, 11 Apr 2016 14:07:34 +0000 (09:07 -0500)]
LU-7152 hsm: sync volatile file before setting times

In the restore case of llapi_hsm_action_end() call fsync() on the
volatile file before updating its timestamps. This prevents spurious
{a,m,c}times from fsync(). Correct a related comment in sanity-hsm
test_24a().

Signed-off-by: John L. Hammond <john.hammond@intel.com>
Change-Id: I982dfd49cbb7d6f9cd7d569e834e4e0581e4bdc9
Reviewed-on: http://review.whamcloud.com/19441
Tested-by: Jenkins
Reviewed-by: Bob Glossman <bob.glossman@intel.com>
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Faccini Bruno <bruno.faccini@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-8017 obd: report correct health state of a node 37/19537/3
James Simmons [Fri, 29 Apr 2016 16:47:38 +0000 (12:47 -0400)]
LU-8017 obd: report correct health state of a node

The patch 16933 that changed the handling of seq_printf
to work with newer kernels introduced a bug in the proc
file that reports the health of a node. The original
code returned immediately after a healthy state was
reported but the bug prevented the immediate return
so a false unhealthy state was always reported instead.

Change-Id: I414d43d59f0df413ea955acf74c91d2402543228
Signed-off-by: James Simmons <uja.ornl@yahoo.com>
Reviewed-on: http://review.whamcloud.com/19537
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Frank Zago <fzago@cray.com>
Reviewed-by: John L. Hammond <john.hammond@intel.com>
4 years agoLU-8073 build: Eliminate lustre-source binary package 06/19806/3
Christopher J. Morrone [Tue, 26 Apr 2016 21:54:51 +0000 (14:54 -0700)]
LU-8073 build: Eliminate lustre-source binary package

The lustre source code is already available in tarball form
and on rpm systems in .src.rpm form.  The source code is
also incidentally available in the lustre-debuginfo package.

The reasons for ever having a lustre-source binary package
are probably lost to history.

But this is a really odd thing to do, so we remove that package
from the build system.

Change-Id: I4a175c556d70a380eb550e5ec2830f4a8d3cd04a
Signed-off-by: Christopher J. Morrone <morrone2@llnl.gov>
Reviewed-on: http://review.whamcloud.com/19806
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Brian J. Murrell <brian.murrell@intel.com>
Reviewed-by: James Simmons <uja.ornl@yahoo.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-8059 ldiskfs: wrong ldiskfs patch 47/19747/8
Andrew Perepechko [Sat, 23 Apr 2016 05:03:05 +0000 (08:03 +0300)]
LU-8059 ldiskfs: wrong ldiskfs patch

Something went wrong with rebasing the ldiskfs patch in
http://review.whamcloud.com/#/c/16360/ LU-7132.

This patch includes the corrected version.

Change-Id: Ice4319f3ebedcdea781bd2198e8b62d010a304de
Signed-off-by: Andrew Perepechko <andrew.perepechko@seagate.com>
Seagate-bug-id: MRP-2688
Reviewed-on: http://review.whamcloud.com/19747
Tested-by: Jenkins
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: James Simmons <uja.ornl@yahoo.com>
4 years agoLU-4423 libcfs: use ahash for libcfs crypto layer 16/19716/5
Herbert Xu [Mon, 25 Apr 2016 16:49:34 +0000 (12:49 -0400)]
LU-4423 libcfs: use ahash for libcfs crypto layer

This patch replaces uses of the long obsolete hash interface with
ahash.

Linux-commit : 6dae10001e84253992142c7343fdae16a13f3d73
Linux-commit : 9dffdb38d864ae89e16ff7b3a09451270736e35b

Change-Id: I9fba47cc05f755ebfd2350c7881708745283d666
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Signed-off-by: James Simmons <uja.ornl@yahoo.com>
Acked-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Reviewed-on: http://review.whamcloud.com/19716
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Alexander Boyko <alexander.boyko@seagate.com>
Reviewed-by: Frank Zago <fzago@cray.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-7990 clio: revise readahead to support 16MB IO 68/19368/12
Jinshan Xiong [Thu, 4 Sep 2014 16:56:25 +0000 (09:56 -0700)]
LU-7990 clio: revise readahead to support 16MB IO

The major work is to adjust read ahead policy to issue
read ahead RPC by underlying RPC size

Test case is backport from Andrew Perepechko's
"LU-7140 llite: too few 4 MiB RPCs from readahead"
http://review.whamcloud.com/16374

Signed-off-by: Jinshan Xiong <jinshan.xiong@intel.com>
Signed-off-by: Gu Zheng <gzheng@ddn.com>
Change-Id: Ib25ff37712389cca7fcee86996124e7c9dd4ec9a
Reviewed-on: http://review.whamcloud.com/19368
Tested-by: Jenkins
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Li Xi <lixi@ddn.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-7904 osd: honor LOC_F_NEW 89/19089/7
Alex Zhuravlev [Wed, 23 Mar 2016 12:37:57 +0000 (15:37 +0300)]
LU-7904 osd: honor LOC_F_NEW

the caller passes this flag to ensure OSD that the object being
initialized is new so OSD can skip save on lookup in OI.

Change-Id: Iaebc23071420f812571c7652a9f29a7e9fc7b67e
Signed-off-by: Alex Zhuravlev <alexey.zhuravlev@intel.com>
Reviewed-on: http://review.whamcloud.com/19089
Tested-by: Jenkins
Reviewed-by: Fan Yong <fan.yong@intel.com>
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
Reviewed-by: Lai Siyao <lai.siyao@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-4423 mem: remove GFP_IOFS 56/18956/4
Mel Gorman [Fri, 1 Apr 2016 23:37:57 +0000 (19:37 -0400)]
LU-4423 mem: remove GFP_IOFS

GFP_IOFS was intended to be shorthand for clearing two flags, not a set of
allocation flags.  There is only one user of this flag combination now and
there appears to be no reason why Lustre had to be protected from reclaim
stalls. As none of the sites appear to be atomic, this patch simply
deletes GFP_IOFS and converts Lustre to using GFP_KERNEL, GFP_NOFS or
GFP_NOIO as appropriate.

Linux-commit : 40113370836e8e79befa585277296ed42781ef31

Change-Id: Ie17d225583f9b48ec2558a85bb6e4e9957010c14
Signed-off-by: Mel Gorman <mgorman@techsingularity.net>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: James Simmons <uja.ornl@yahoo.com>
Reviewed-on: http://review.whamcloud.com/18956
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Lai Siyao <lai.siyao@intel.com>
Reviewed-by: John L. Hammond <john.hammond@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-8072 build: Restore module debuginfo 05/19805/2
Christopher J. Morrone [Tue, 26 Apr 2016 21:41:02 +0000 (14:41 -0700)]
LU-8072 build: Restore module debuginfo

In commit 3ce96f162a weaccidentally dropped the chmod that makes kernel
modules available to the find-debuginfo.sh.

Reintroducing this chmod allows the lustre debuginfo package to contain
debuginfo for the kenrel modules.

Change-Id: I57e2a6dc248aaccf573e68a49a981bf92c7fbb62
Signed-off-by: Christopher J. Morrone <morrone2@llnl.gov>
Reviewed-on: http://review.whamcloud.com/19805
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Dmitry Eremin <dmitry.eremin@intel.com>
Reviewed-by: James Simmons <uja.ornl@yahoo.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-6245 libcfs: Remove kp30.h 60/19460/2
James Simmons [Mon, 11 Apr 2016 21:42:00 +0000 (17:42 -0400)]
LU-6245 libcfs: Remove kp30.h

The header kp30.h contains various wrappers to handle things
like module parameter handling. We can now remove this
abstraction.

Signed-off-by: James Simmons <uja.ornl@yahoo.com>
Change-Id: I7f5fec4372023146215c740e287529a05dfceba6
Reviewed-on: http://review.whamcloud.com/19460
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: John L. Hammond <john.hammond@intel.com>
Reviewed-by: Dmitry Eremin <dmitry.eremin@intel.com>
Reviewed-by: Frank Zago <fzago@cray.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-6245 libcfs: remove sysctl module handling 59/19459/7
James Simmons [Tue, 26 Apr 2016 00:54:50 +0000 (20:54 -0400)]
LU-6245 libcfs: remove sysctl module handling

Remove the sysctl handling we have for module parameters.
This code only gets built if CFS_SYSFS_MODULE_PARM is not
1 which is never the case. So this code is never built.
Plus this code has long been replaced by the sysfs module
parameters handling.

Signed-off-by: James Simmons <uja.ornl@yahoo.com>
Change-Id: I5635bfd75a53bdd25af51c65fb1c2550d3cbbbaa
Reviewed-on: http://review.whamcloud.com/19459
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: John L. Hammond <john.hammond@intel.com>
Reviewed-by: Chris Horn <hornc@cray.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-7999 utils: fix double free the pointer 09/19409/4
Dmitry Eremin [Fri, 8 Apr 2016 15:32:34 +0000 (18:32 +0300)]
LU-7999 utils: fix double free the pointer

Object 'uuids' was freed at line 1895 after being freed by
calling 'realloc' at line 1892.

Change-Id: I2549f2cb0d2e1a2e4707c4d54d37c90ac099f375
Signed-off-by: Dmitry Eremin <dmitry.eremin@intel.com>
Reviewed-on: http://review.whamcloud.com/19409
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: John L. Hammond <john.hammond@intel.com>
Reviewed-by: James Simmons <uja.ornl@yahoo.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-7957 kernel: kernel update [SLES11 SP4 3.0.101-71] 15/19215/8
Bob Glossman [Wed, 30 Mar 2016 15:03:20 +0000 (08:03 -0700)]
LU-7957 kernel: kernel update [SLES11 SP4 3.0.101-71]

Update SLES11 SP4 kernel to 3.0.101-71

Test-Parameters: mdsdistro=sles11sp4 ossdistro=sles11sp4 \
  clientdistro=sles11sp4 mdsfilesystemtype=ldiskfs \
  mdtfilesystemtype=ldiskfs ostfilesystemtype=ldiskfs \
  testgroup=review-ldiskfs

Signed-off-by: Bob Glossman <bob.glossman@intel.com>
Change-Id: I1e7ff0e2abb2274176a04e8a16a33fe298e77bc5
Reviewed-on: http://review.whamcloud.com/19215
Tested-by: Jenkins
Reviewed-by: Minh Diep <minh.diep@intel.com>
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Yang Sheng <yang.sheng@intel.com>
Reviewed-by: Jian Yu <jian.yu@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-7521 ldlm: LDLM_DEBUG() shouldn't be passed \n 94/17494/8
Alex Zhuravlev [Mon, 7 Dec 2015 06:42:24 +0000 (09:42 +0300)]
LU-7521 ldlm: LDLM_DEBUG() shouldn't be passed \n

as it adds own \n, so any extra \n break log format.

Change-Id: Ie78d0969421b744db6789d7bd46505ba7097a46b
Signed-off-by: Alex Zhuravlev <alexey.zhuravlev@intel.com>
Reviewed-on: http://review.whamcloud.com/17494
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: James Simmons <uja.ornl@yahoo.com>
Reviewed-by: John L. Hammond <john.hammond@intel.com>
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
4 years agoLU-8064 lctl: fix a typo in lustre_cfg.c 59/19759/3
Emoly Liu [Mon, 25 Apr 2016 07:41:40 +0000 (15:41 +0800)]
LU-8064 lctl: fix a typo in lustre_cfg.c

When checking overflow issue in param_dispaly(), we should use rc2
not rc, because rc2 is the return value of pathname string size.

Signed-off-by: Emoly Liu <emoly.liu@intel.com>
Change-Id: I35820b00c7a4e264444e5574967d56c54b4a469b
Reviewed-on: http://review.whamcloud.com/19759
Reviewed-by: Bob Glossman <bob.glossman@intel.com>
Tested-by: Jenkins
Reviewed-by: James Simmons <uja.ornl@yahoo.com>
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-8051 tests: Skipping sanity tests 300[a-d] for old servers. 98/19698/3
Kirtankumar Krishna Shetty [Thu, 21 Apr 2016 08:48:01 +0000 (14:18 +0530)]
LU-8051 tests: Skipping sanity tests 300[a-d] for old servers.

The tests 300[a-i] were added in version 2.7.0 for the newer
version of the code to test striped directory related features
and hence the tests 300[a,b,c,d] used to fail for the v2.7.0
and above client with older server versions. Added skip
statement for the tests if server is of an older versions.

Seagate-bug-id: MRP-3393
Change-Id: I26f271c1a4aa62c8f41c391e0c595f1ccaf18f8b
Signed-off-by: Kirtankumar Krishna Shetty <kirtan.shetty@seagate.com>
Reviewed-on: http://review.whamcloud.com/19698
Tested-by: Jenkins
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
Reviewed-by: Jian Yu <jian.yu@intel.com>
Tested-by: Maloo <hpdd-maloo@intel.com>
4 years agoLU-8005 osc: set lock data for readahead lock 53/19453/2
Jinshan Xiong [Mon, 11 Apr 2016 17:56:37 +0000 (10:56 -0700)]
LU-8005 osc: set lock data for readahead lock

If osc_io_readahead() finds a lock that belongs to the previous
instance of osc_object, the lock data pointer will be null. It has
to instantiate with new instance otherwise those pages won't be
destroyed at lock cancel, and then finally hit the assertion in
osc_req_attr_set().

This patch revised dlmlock_at_pgoff() to call osc_match_base() to
find caching locks for readahead. And new osc_object will be set
to the lock if it doesn't have one yet.

Signed-off-by: Jinshan Xiong <jinshan.xiong@intel.com>
Change-Id: I6e95499bf0c1e501f3f55d25e0c2b3fbc61ed0da
Reviewed-on: http://review.whamcloud.com/19453
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Bobi Jam <bobijam@hotmail.com>
Reviewed-by: John L. Hammond <john.hammond@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-7972 tests: use $REFORMAT in test-framework.sh correctly 76/19276/6
Emoly Liu [Thu, 21 Apr 2016 04:00:31 +0000 (12:00 +0800)]
LU-7972 tests: use $REFORMAT in test-framework.sh correctly

$REFORMAT is always used incorrectly in test-framework.sh, because
as long as $REFORMAT is not null, format will be done, even if
REFORMAT=no.
This patch fixes this issue by setting/judging $REFORMAT=yes/no
explicitly.

Signed-off-by: Emoly Liu <emoly.liu@intel.com>
Change-Id: Iaabea34e0ad453b3f9f986009fa1f8bdec8a98ff
Reviewed-on: http://review.whamcloud.com/19276
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Bobi Jam <bobijam@hotmail.com>
4 years agoLU-7859 tests: Use getent instead of /etc/passwd 46/18846/4
Nathaniel Clark [Wed, 9 Mar 2016 22:14:15 +0000 (17:14 -0500)]
LU-7859 tests: Use getent instead of /etc/passwd

Use getent passwd instead of directly reading form /etc/passwd
to prevent missing user checks if users come from ldap or another
auth source that isn't /etc/passwd.

Signed-off-by: Nathaniel Clark <nathaniel.l.clark@intel.com>
Change-Id: Ia2831c777c7c3f8bae5a634fa897744954d14c1e
Reviewed-on: http://review.whamcloud.com/18846
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: James Simmons <uja.ornl@yahoo.com>
Reviewed-by: Dmitry Eremin <dmitry.eremin@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-6512 llite:ll_dir_ioctl cleanup of redundant comparisons 27/18027/7
Parinay Kondekar [Mon, 18 Jan 2016 03:29:35 +0000 (08:59 +0530)]
LU-6512 llite:ll_dir_ioctl cleanup of redundant comparisons

In ll_dir_ioctl() two identical comparisions are present for
return code (rc) of ll_dir_getstripe(). This patch removes
the other inside if( ) condition which is not necessary.

Change-Id: Ia32dd5ac5937d30dd3676094ef99a623e5cca608
Signed-off-by: Parinay Kondekar <parinay.kondekar@seagate.com>
Reviewed-on: http://review.whamcloud.com/18027
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Bobi Jam <bobijam@hotmail.com>
Reviewed-by: James Simmons <uja.ornl@yahoo.com>
Tested-by: Jenkins
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-7403 llite: flatten struct lookup_intent 69/17069/4
John L. Hammond [Fri, 6 Nov 2015 16:05:32 +0000 (10:05 -0600)]
LU-7403 llite: flatten struct lookup_intent

Replace the union in struct lookup_intent with the members of struct
lustre_indent_data. Remove the then unused struct lustre_intent_data.

Signed-off-by: John L. Hammond <john.hammond@intel.com>
Change-Id: I47d815f7c8e51bcff33ea7dba6efbc08337ca509
Reviewed-on: http://review.whamcloud.com/17069
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: James Simmons <uja.ornl@yahoo.com>
Reviewed-by: Dmitry Eremin <dmitry.eremin@intel.com>
Reviewed-by: Frank Zago <fzago@cray.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-7981 llite: take trunc_sem only at vvp layer 15/19315/4
Patrick Farrell [Tue, 5 Apr 2016 15:48:57 +0000 (10:48 -0500)]
LU-7981 llite: take trunc_sem only at vvp layer

The lli_trunc_sem is taken in 'read' mode in both
ll_page_mkwrite and vvp_io_fault_start. This can lead to a
deadlock with another thread which asks for the semaphore
in write mode between thse two read calls.

Since all users of lli_trunc_sem are in the vvp layer, we
can satisfy the requirement to exclude truncate by taking
the semaphore only in vvp_io_fault_start.

Signed-off-by: Patrick Farrell <paf@cray.com>
Change-Id: I7341d179f550835ccebfb1bb2d90260c1d5d0ff8
Reviewed-on: http://review.whamcloud.com/19315
Tested-by: Jenkins
Reviewed-by: Jinshan Xiong <jinshan.xiong@intel.com>
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Andriy Skulsyh
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-6722 ldiskfs: fix credits at ldiskfs_delete_inode 32/19732/3
Alexander Boyko [Fri, 22 Apr 2016 13:00:40 +0000 (16:00 +0300)]
LU-6722 ldiskfs: fix credits at ldiskfs_delete_inode

extra_credits was added to take account of quota, but later
journal extend still use 3 blocks. The patch changes
this 3 credits to extra_credits.

Signed-off-by: Alexander Boyko <alexander.boyko@seagate.com>
Seagate-bug-id: MRP-3441
Change-Id: Ib098f3181790367cbc384f25072588b08ea624e4
Reviewed-on: http://review.whamcloud.com/19732
Tested-by: Jenkins
Reviewed-by: Alex Zhuravlev <alexey.zhuravlev@intel.com>
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Fan Yong <fan.yong@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-7699 build: Convert version underscores to dashes for dpkg 88/19488/5
Christopher J. Morrone [Mon, 25 Apr 2016 14:36:49 +0000 (10:36 -0400)]
LU-7699 build: Convert version underscores to dashes for dpkg

For rpm, we can't have dashes in the version string so we use
underscores.  For dpkg, the requirements are reversed.

In the "debs" make target, we convert underscores in the version
string to dashes.  We also trim down the comment that no longer
really makes sense with the new versioning scheme.

Change-Id: I6d600fc4a16be60c664ef0662b482c31877fbfea
Signed-off-by: Christopher J. Morrone <morrone2@llnl.gov>
Reviewed-on: http://review.whamcloud.com/19488
Tested-by: Jenkins
Reviewed-by: Dmitry Eremin <dmitry.eremin@intel.com>
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: James Simmons <uja.ornl@yahoo.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-7558 import: don't reconnect during connect interpret 12/19312/2
Mikhal Pershin [Mon, 4 Apr 2016 10:56:35 +0000 (13:56 +0300)]
LU-7558 import: don't reconnect during connect interpret

The import connect flags might be cleared by ptlrpc_connect_import()
wrongly if there is still connect interpret function is running.

Use imp_connected boolean variable to indicate that we are still
interpretting connect reply and don't try to reconnect until it ends.

Signed-off-by: Mikhal Pershin <mike.pershin@intel.com>
Change-Id: I77cf70820835dc06b33a75353c8ef71ba4ed4fad
Reviewed-on: http://review.whamcloud.com/19312
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: John L. Hammond <john.hammond@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoRevert "LU-7791 ldlm: signal vs CP callback race" 98/19798/2
Oleg Drokin [Tue, 26 Apr 2016 17:46:09 +0000 (17:46 +0000)]
Revert "LU-7791 ldlm: signal vs CP callback race"

This causes frequent failures tracked in LU-8067

This reverts commit a880f385164547837a34df92acd6cdc8e9a9698d.

Change-Id: Ib247bb924dabe4635edd17f9a2dca391135f558f
Reviewed-on: http://review.whamcloud.com/19798
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
Tested-by: Oleg Drokin <oleg.drokin@intel.com>
Tested-by: Jenkins
4 years agoLU-7982 libcfs: memory allocation without CPT for binheap 17/19317/5
Li Xi [Tue, 21 Jul 2015 09:12:16 +0000 (17:12 +0800)]
LU-7982 libcfs: memory allocation without CPT for binheap

In order to use heap in cases which are not tied to a specific CPT,
memory allocation should support normal memory allocation. This
patch adds support for it.

Signed-off-by: Li Xi <lixi@ddn.com>
Change-Id: I1daeb76b708489b3adf09251fe1fbf2ddeab35aa
Reviewed-on: http://review.whamcloud.com/19317
Tested-by: Jenkins
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
Reviewed-by: Nikitas Angelinas <nikitasangelinas@gmail.com>
Reviewed-by: Emoly Liu <emoly.liu@intel.com>
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: James Simmons <uja.ornl@yahoo.com>
4 years agoLU-7434 ptlrpc: Early Reply vs Reply MDunlink 34/18934/2
Vitaly Fertman [Sun, 21 Feb 2016 01:35:20 +0000 (04:35 +0300)]
LU-7434 ptlrpc: Early Reply vs Reply MDunlink

A race between unregister_reply & early reply.
When buffers are busy for the early transfer, they cannon be unlinked
by unregister_reply, so the RPC gets into UNREGISTERING state. The
coming reply_in_callback for the early RPC already has unlinked flag
set due to previous mdunlink attempt, but we handle it properly only
for UNILNK event, whereas this is PUT in this case.

Signed-off-by: Vitaly Fertman <vitaly.fertman@seagate.com>
Change-Id: I98a939e54d5d9298f217761c8c7289843e46162d
Seagate-bug-id: MRP-3323
Reviewed-by: Alexey Leonidovich Lyashkov <alexey.lyashkov@seagate.com>
Reviewed-by: Andriy Skulysh <andriy.skulysh@seagate.com>
Tested-by: Parinay Vijayprakash Kondekar <parinay.kondekar@seagate.com>
Reviewed-on: http://review.whamcloud.com/18934
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Chris Horn <hornc@cray.com>
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-7980 ldiskfs: always pre-allocate max depth for path 49/19349/7
Bruno Faccini [Wed, 6 Apr 2016 14:38:05 +0000 (16:38 +0200)]
LU-7980 ldiskfs: always pre-allocate max depth for path

Upon very specific meta-data and I/O pattern a situation can be
encountered where ext_depth() will concurrently (due to i_data_sem
release) grow outside of currently used ext4_ext_path[] array
boundary.
This can lead to potential Slab overrun if current ext_depth()
is used to index previously sized+allocated ext4_ext_path[] array.
This patch fixes this by already pre-allocating _ext_path[]
array of the max possible depth.

This is an alternate way to avoid racy cases, that can lead to
slab overrun, with current and unsafe implementation that causes
ext4_ext_path[] array re-[sizing,allocation] based on current depth.

Now unnecessay free an realloc upon depth change detection prior to
reuse already allocated path space have also been removed.

Signed-off-by: Bruno Faccini <bruno.faccini@intel.com>
Change-Id: I0ae955c5eccf032cb02ef55c6381964a5eaf82c2
Reviewed-on: http://review.whamcloud.com/19349
Tested-by: Jenkins
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Alex Zhuravlev <alexey.zhuravlev@intel.com>
4 years agoLU-6215 o2iblnd: rdma_create_id() takes extra parameter 59/18759/6
Li Dongyang [Sat, 2 Apr 2016 00:26:46 +0000 (20:26 -0400)]
LU-6215 o2iblnd: rdma_create_id() takes extra parameter

4.4 kernel added network namespace parameter to rdma_create_id().
This patch handles the API change.

Linux-commit: fa20105e09e97e81aadf02f722c31195e4a75c84

Signed-off-by: Li Dongyang <dongyang.li@anu.edu.au>
Change-Id: I3f3792e70e91ac3b3d9fefdd58cc9e7e4d1c25c6
Reviewed-on: http://review.whamcloud.com/18759
Reviewed-by: James Simmons <uja.ornl@yahoo.com>
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Sebastien Buisson <sbuisson@ddn.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-6215 o2iblnd: ib_alloc_fast_reg_mr() removed in 4.3 85/19185/2
Li Dongyang [Tue, 29 Mar 2016 03:57:57 +0000 (14:57 +1100)]
LU-6215 o2iblnd: ib_alloc_fast_reg_mr() removed in 4.3

ib_alloc_fast_reg_mr() is replaced by a more generic
ib_alloc_mr() since kernel 4.3

Linux-commit: d9f272c523db47a56a64942eb6f25361c400de66

Signed-off-by: Li Dongyang <dongyang.li@anu.edu.au>
Change-Id: I53f4e7c395c83d731aaa342f308f878b07b54d61
Reviewed-on: http://review.whamcloud.com/19185
Reviewed-by: Dmitry Eremin <dmitry.eremin@intel.com>
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: James Simmons <uja.ornl@yahoo.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-6215 gss: key->payload.data is an array in 4.4+ kernels 31/18731/11
Li Dongyang [Thu, 21 Apr 2016 14:25:28 +0000 (10:25 -0400)]
LU-6215 gss: key->payload.data is an array in 4.4+ kernels

Kernel 4.4 merged the type-specific data with the payload data
for keys, as a result, payload.data is now an array.
This patch handles the code change.

Linux-commit: 146aa8b1453bd8f1ff2304ffb71b4ee0eb9acdcc

Signed-off-by: Li Dongyang <dongyang.li@anu.edu.au>
Signed-off-by: James Simmons <uja.ornl@yahoo.com>
Change-Id: I1036ab927514f0efa073d9f0136e299be8950461
Reviewed-on: http://review.whamcloud.com/18731
Reviewed-by: Dmitry Eremin <dmitry.eremin@intel.com>
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Sebastien Buisson <sbuisson@ddn.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-8052 test: stop using unknonwn sed option 14/19714/5
Bob Glossman [Thu, 21 Apr 2016 17:18:16 +0000 (10:18 -0700)]
LU-8052 test: stop using unknonwn sed option

Avoid the use of the -E command line option for sed.
It isn't known in all versions.

Signed-off-by: Bob Glossman <bob.glossman@intel.com>
Change-Id: I66f58aa04ecc751f4b3d61475cafe0c478ce6206
Reviewed-on: http://review.whamcloud.com/19714
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
Tested-by: Jenkins
Reviewed-by: James Nunez <james.a.nunez@intel.com>
Tested-by: Maloo <hpdd-maloo@intel.com>
4 years agoLU-8023 lbuild: add find-requires parameter to rpmbuild 50/19650/5
Dmitry Eremin [Tue, 19 Apr 2016 15:18:41 +0000 (18:18 +0300)]
LU-8023 lbuild: add find-requires parameter to rpmbuild

If kernel-devel unpacked in not default directory we need to
pass modified version of find-requires which can find it.

Test-Parameters: mdsdistro=sles11sp4 ossdistro=sles11sp4 \
  clientdistro=sles11sp4 mdsfilesystemtype=ldiskfs \
  mdtfilesystemtype=ldiskfs ostfilesystemtype=ldiskfs \
  testgroup=review-ldiskfs

Change-Id: Id30744fe596b1bf690611e9f66dd804219b20fe3
Signed-off-by: Dmitry Eremin <dmitry.eremin@intel.com>
Reviewed-on: http://review.whamcloud.com/19650
Reviewed-by: Bob Glossman <bob.glossman@intel.com>
Tested-by: Jenkins
Tested-by: Andreas Dilger <andreas.dilger@intel.com>
Reviewed-by: Minh Diep <minh.diep@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoRevert "LU-7434 ptlrpc: lost bulk leads to a hang" 78/19778/2
Oleg Drokin [Tue, 26 Apr 2016 00:35:34 +0000 (00:35 +0000)]
Revert "LU-7434 ptlrpc: lost bulk leads to a hang"

This causes widespread testing failures reported in LU-8062

This reverts commit 55f8520817a31dabf19fe0a8ac2492b85d039c38.

Change-Id: Ifcdf0bf569f6ea43cf625095e77ba347649c37df
Reviewed-on: http://review.whamcloud.com/19778
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
Tested-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-8014 hsm: remove kuc_ispayload 94/19494/3
Frank Zago [Tue, 12 Apr 2016 21:31:07 +0000 (17:31 -0400)]
LU-8014 hsm: remove kuc_ispayload

That function takes a pointer, decrements it and dereference the new
address. However we have no idea if the page it's in is
readable or even exists. Such dereference would cause an oops.

That function is only called from mdt_hsm_agent_send, which has itself
2 callers. In the most common case, from mdt_coordinator(), the buffer
passed is already a KUC, but in the less used case, from
hsm_cancel_all_actions(), it is not. Allocating the KUC buffer in
mdt_coordinator() doesn't buy us anything. So remove that code from
the coordinator, which means mdt_hsm_agent_send() is now always called
with a regular buffer, and kuc_ispayload can go.

Signed-off-by: frank zago <fzago@cray.com>
Change-Id: I99c29d4223c14b6fb778215032cd07b6e4676990
Reviewed-on: http://review.whamcloud.com/19494
Reviewed-by: John L. Hammond <john.hammond@intel.com>
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Henri Doreau <henri.doreau@cea.fr>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-7990 ofd: add a parameter to adjust preferred brw size 67/19367/8
Jinshan Xiong [Fri, 9 May 2014 01:24:19 +0000 (18:24 -0700)]
LU-7990 ofd: add a parameter to adjust preferred brw size

Add parameter ofd_device::ofd_brw_size to set the
preferred BRW size of an OST.

Signed-off-by: Jinshan Xiong <jinshan.xiong@intel.com>
Signed-off-by: Gu Zheng <gzheng@ddn.com>
Change-Id: Iab9b26fb1e1c1af8de10d2c96da69151a0a654f5
Reviewed-on: http://review.whamcloud.com/19367
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Bobi Jam <bobijam@hotmail.com>
Reviewed-by: Li Xi <lixi@ddn.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>