Whamcloud - gitweb
fs/lustre-release.git
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>
4 years agoLU-7922 ldiskfs: correction in ext4_kzalloc 41/19541/2
Lokesh Nagappa Jaliminche [Thu, 14 Apr 2016 08:12:01 +0000 (13:42 +0530)]
LU-7922 ldiskfs: correction in ext4_kzalloc

ext4_kzalloc uses kmalloc instead of kzalloc, this was
introduced by commit 9933fc0. Due to this get_orlov_stats
was not working properly, making orlov allocator completely
random. kmalloc is replaced with kzalloc to get correct
stats.This is corrected in kernel as part of following
commit v3.0-7217-gdb9481c04

Added __GFP_NOWARN to k.alloc calls with v.alloc fallbacks
as per this commit v3.11-rc2-221-g8be04b937.

Seagate-bug-id: MRP-3412
Signed-off-by: Lokesh Nagappa Jaliminche <lokesh.jaliminche@seagate.com>
Change-Id: I15f1a28537cb5e4f32d63527c99508383e12c97f
Reviewed-on: http://review.whamcloud.com/19541
Tested-by: Jenkins
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
Reviewed-by: Yang Sheng <yang.sheng@intel.com>
Tested-by: Maloo <hpdd-maloo@intel.com>
4 years agoLU-7198 clio: remove mtime check in vvp_io_fault_start() 62/19162/4
Bobi Jam [Sat, 26 Mar 2016 14:36:34 +0000 (22:36 +0800)]
LU-7198 clio: remove mtime check in vvp_io_fault_start()

In fault IO initialization, inode's mtime is saved, and after
getting locks, when the IO is about to start, vvp_io_fault_start()
checks the mtime's intactness.

It's a false alarm, since the timestamp from MDS could be stale,
we maintain mtime mainly on OST objects, and if the check in
vvp_io_fault_start() happens before mtime on OST objects are merged,
it will get wrong timestamp from the inode, even the timestamp it
fetched in vvp_io_fault_init() could be wrong in the first place.

This patch remove the mtime check in vvp_io_fault_start().

Signed-off-by: Bobi Jam <bobijam.xu@intel.com>
Change-Id: Iff0a7f3ae789b443927fe2f3487829f357a93a4a
Reviewed-on: http://review.whamcloud.com/19162
Tested-by: Jenkins
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Jinshan Xiong <jinshan.xiong@intel.com>
4 years agoLU-7746 tests: some fixes for sanity.sh with upstream kernel 63/19663/2
Andreas Dilger [Sat, 13 Feb 2016 10:11:50 +0000 (03:11 -0700)]
LU-7746 tests: some fixes for sanity.sh with upstream kernel

sanity.sh:
- delete is_patchless() check, since this is always true today.
- remove test for "lfs getstripe --raw" added in 2.5.57
- minor style cleanups

Signed-off-by: Andreas Dilger <andreas.dilger@intel.com>
Change-Id: I6c808febe078fdc7afd89b82a04be622879cab07
Reviewed-on: http://review.whamcloud.com/19663
Tested-by: Jenkins
Reviewed-by: James Simmons <uja.ornl@yahoo.com>
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Jian Yu <jian.yu@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-8038 tests: test case for lfs migrate --block 45/19645/3
Yang Sheng [Tue, 19 Apr 2016 03:56:26 +0000 (11:56 +0800)]
LU-8038 tests: test case for lfs migrate --block

Add a test case to cover 'lfs migrate --block'.

Signed-off-by: Yang Sheng <yang.sheng@intel.com>
Change-Id: I59bd0e9f881a37f3416ac07063921985f0313342
Reviewed-on: http://review.whamcloud.com/19645
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Bob Glossman <bob.glossman@intel.com>
Reviewed-by: Jian Yu <jian.yu@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-6601 tests: test case validate OSP status op 37/19637/4
Yang Sheng [Mon, 18 Apr 2016 18:43:51 +0000 (02:43 +0800)]
LU-6601 tests: test case validate OSP status op

Add a test case to verify status change is effective for OSP.

Signed-off-by: Yang Sheng <yang.sheng@intel.com>
Change-Id: I11a6968756f66989ba87063424f6618475814901
Reviewed-on: http://review.whamcloud.com/19637
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Bob Glossman <bob.glossman@intel.com>
Reviewed-by: Jian Yu <jian.yu@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-8015 utils: fix lr_link() to get correct src and dest 99/19599/5
Jian Yu [Fri, 15 Apr 2016 07:54:20 +0000 (00:54 -0700)]
LU-8015 utils: fix lr_link() to get correct src and dest

The current codes in lr_link() of lustre_rsync.c can not
find out the correct info->src and info->dest in the
following situations:
1) the source name ends with the destination name
2) the file has two links with the same name but
   in different directories

This patch fixes the above issue by using the path of the
parent FID and target name to construct info->dest, and
comparing the path of the target FID with info->dest to
get the correct info->src.

Test-Parameters: testlist=lustre-rsync-test
Signed-off-by: Jian Yu <jian.yu@intel.com>
Change-Id: Ia92b01c069bde01bd0f094017bd2502292afc7f8
Reviewed-on: http://review.whamcloud.com/19599
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: John L. Hammond <john.hammond@intel.com>
Reviewed-by: Jinshan Xiong <jinshan.xiong@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-7973 llite: lock i_lock/dcache_lock before __d_drop() 87/19287/2
Bruno Faccini [Fri, 1 Apr 2016 12:31:55 +0000 (14:31 +0200)]
LU-7973 llite: lock i_lock/dcache_lock before __d_drop()

There has been several Lustre Client crashes reported by sites
running with Lustre versions 2.1/2.5, all showing the same
dentry->d_hash->next corrupted pointer cause.

This patch fixes a regression that has been introduced since a
long time by commit :
d10c9e05ab593f361fdfd27842766a1924e63e58
(LU-506 kernel: FC15 - support dcache scalability changes.)

where i_lock/dcache_lock protection usage has been removed and
that is likely to cause racy condition during dentry [un]hashing
and to be the root cause of these crashes.

Signed-off-by: Bruno Faccini <bruno.faccini@intel.com>
Change-Id: Ie1239058222f15f7e094d8fae4d37e4a36449707
Reviewed-on: http://review.whamcloud.com/19287
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Lai Siyao <lai.siyao@intel.com>
Reviewed-by: Yang Sheng <yang.sheng@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-5725 ofd: Expose OFD site_stats through proc 47/19247/4
Christopher J. Morrone [Wed, 30 Mar 2016 01:26:00 +0000 (21:26 -0400)]
LU-5725 ofd: Expose OFD site_stats through proc

While exposed for the MDT the lu_object cache site stats are not
available for the OFD.  This small patch makes those same stats
available under /proc/fs/lustre/obdfilter/lustre-OST0001/site_stats.

Change-Id: Ic5207d5471cd0f8eacc5a3363c12975c8880224d
Signed-off-by: Christopher J. Morrone <morrone2@llnl.gov>
Reviewed-on: http://review.whamcloud.com/19247
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Emoly Liu <emoly.liu@intel.com>
Reviewed-by: James Simmons <uja.ornl@yahoo.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-7937 utils: Fix NULL pointer dereference 94/19194/4
Dmitry Eremin [Tue, 29 Mar 2016 15:09:47 +0000 (18:09 +0300)]
LU-7937 utils: Fix NULL pointer dereference

Check the error of malloc() function and handle it properly.

Change-Id: I2763d1b6d9c97c137a2593077ee31eaa3758ad81
Signed-off-by: Dmitry Eremin <dmitry.eremin@intel.com>
Reviewed-on: http://review.whamcloud.com/19194
Tested-by: Jenkins
Reviewed-by: Bob Glossman <bob.glossman@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-7902 osd: no need to set same [acm]time 88/19088/4
Alex Zhuravlev [Wed, 23 Mar 2016 12:32:54 +0000 (15:32 +0300)]
LU-7902 osd: no need to set same [acm]time

osd_attr_set() has [acm]time cached in the objects. so if the caller
wants to set them to the same value, there is no point to call very
expensive SA update procedure.

Change-Id: I4e06306198496170f5d9c486a8d245c0978c0f79
Signed-off-by: Alex Zhuravlev <alexey.zhuravlev@intel.com>
Reviewed-on: http://review.whamcloud.com/19088
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Niu Yawei <yawei.niu@intel.com>
Reviewed-by: Bobi Jam <bobijam@hotmail.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-1118 llite: Invoke file_update_time in page_mkwrite 83/18683/4
Yang Sheng [Thu, 25 Feb 2016 16:16:31 +0000 (00:16 +0800)]
LU-1118 llite: Invoke file_update_time in page_mkwrite

Upstream commit 41c4d25f78c01ede13efee1f2e979f3f35dd26f6
Only update file times if page_mkwrite is not set. So we
need call file_update_time by ourselves. Since all we
supported kernel have included this change. Then config
check is unnecessary.

Signed-off-by: Yang Sheng <yang.sheng@intel.com>
Change-Id: I1477ea6c6576df01af28afdf0dbaa7b012a7238e
Reviewed-on: http://review.whamcloud.com/18683
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
Reviewed-by: Niu Yawei <yawei.niu@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-7402 llite: correct request handling after ll_lookup_it() 68/17068/2
John L. Hammond [Fri, 6 Nov 2015 15:39:17 +0000 (09:39 -0600)]
LU-7402 llite: correct request handling after ll_lookup_it()

In the FIFO cases of ll_atomic_open() and ll_lookup_nd() remove
spurious calls to ptlrpc_req_finished(). Explain that these cases are
unreachable in practice anyway.

Signed-off-by: John L. Hammond <john.hammond@intel.com>
Change-Id: Ic7d677674ed7f9e52d039bb81259f60bc75ba71b
Reviewed-on: http://review.whamcloud.com/17068
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Dmitry Eremin <dmitry.eremin@intel.com>
Reviewed-by: Lai Siyao <lai.siyao@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-8001 nodemap: fix null deref when reclassifying 95/19595/2
Kit Westneat [Fri, 15 Apr 2016 03:05:58 +0000 (23:05 -0400)]
LU-8001 nodemap: fix null deref when reclassifying

There are some situations in which an export's connection can be
NULL. This patch adds a NULL check before dereferencing in
nm_member_reclassify_nodemap. If the connection is null, keep export
in current nodemap since it should get reclassified on reconnect.

Signed-off-by: Kit Westneat <kit.westneat@gmail.com>
Change-Id: I795afdac3b388e883bafe5a79c8237794bc01ef6
Reviewed-on: http://review.whamcloud.com/19595
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>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-8009 utils: Lustre_rsync does not sync files on MDT1 94/19594/2
Tatsushi Takamura [Fri, 15 Apr 2016 01:04:46 +0000 (10:04 +0900)]
LU-8009 utils: Lustre_rsync does not sync files on MDT1

This patch fixes the 3rd parameter of llapi_changelog_start()
to refer a changelog correctly.

Signed-off-by: Tatsushi Takamura <takamr.tatsushi@jp.fujitsu.com>
Change-Id: I84ff553b299648159b8107fd9be486db44829ef2
Reviewed-on: http://review.whamcloud.com/19594
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>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-5070 osd-zfs: get object's FID from its LMA EA 95/18595/2
Fan Yong [Tue, 23 Feb 2016 03:33:55 +0000 (11:33 +0800)]
LU-5070 osd-zfs: get object's FID from its LMA EA

Do not assume that the FID-in-dirent is always valid. If the
FID-in-dirent is absent or invalid, then the osd_dir_lookup
needs to find out the FID from the target object's LMA EA.

It is the similar logic as osd-ldiskfs does.

Signed-off-by: Fan Yong <fan.yong@intel.com>
Change-Id: If75ff4ed03bd75a3385666842e94838ece3f4344
Reviewed-on: http://review.whamcloud.com/18595
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
Reviewed-by: Niu Yawei <yawei.niu@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-7592 osd-ldiskfs: increase supported ldiskfs fs size limit 02/17702/7
Artem Blagodarenko [Tue, 22 Dec 2015 09:15:15 +0000 (12:15 +0300)]
LU-7592 osd-ldiskfs: increase supported ldiskfs fs size limit

Change "force_over_128tb" mount option to "force_over_256tb".

Signed-off-by: Alexander Zarochentsev <alexander.zarochentsev@seagate.com>
Signed-off-by: Artem Blagodarenko <artem.blagodarenko@seagate.com>
Seagate-bug-id: MRP-2840
Reviewed-by: Artem Blagodarenko <artem.blagodarenko@seagate.com>
Reviewed-by: Alexey Leonidovich Lyashkov <alexey.lyashkov@seagate.com>
Tested-by: Elena V. Gryaznova <elena.gryaznova@seagate.com>
Change-Id: Ib53c4375a170823997a5a6b9ef6d832e8b3e0c72
Reviewed-on: http://review.whamcloud.com/17702
Tested-by: Jenkins
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-5070 utils: mkfs.lustre code cleanup 03/15303/8
Fan Yong [Wed, 21 Oct 2015 02:24:58 +0000 (10:24 +0800)]
LU-5070 utils: mkfs.lustre code cleanup

Re-order the options to increase the code readability.
Remove unused code, redundant logic, white spaces,
blank lines, and so on.

Signed-off-by: Fan Yong <fan.yong@intel.com>
Change-Id: I88ee52fb2425e9e256b7751bad4b6cfcd88011ec
Reviewed-on: http://review.whamcloud.com/15303
Tested-by: Jenkins
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Ben Evans <bevans@cray.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-7851 mdt: skip SOM related configuration 34/18834/5
Hongchao Zhang [Tue, 15 Dec 2015 21:04:13 +0000 (05:04 +0800)]
LU-7851 mdt: skip SOM related configuration

the SOM is deprecated, then the old SOM configuration should be
skipped by the newer version when upgrading Lustre.

Change-Id: Ibb32e43501479aac75c8e02a652bea8d8a727445
Signed-off-by: Hongchao Zhang <hongchao.zhang@intel.com>
Reviewed-on: http://review.whamcloud.com/18834
Reviewed-by: Fan Yong <fan.yong@intel.com>
Reviewed-by: Niu Yawei <yawei.niu@intel.com>
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-8037 fileset: reject dotdot as a path component 46/19646/2
Wang Shilong [Tue, 19 Apr 2016 04:19:30 +0000 (12:19 +0800)]
LU-8037 fileset: reject dotdot as a path component

Administrator might access Lustre internal files by
using dotdot path for example:

mount -t lustre ip@tcp:/lustre/.. /mnt/lustre

Signed-off-by: Wang Shilong <wshilong@ddn.com>
Change-Id: I137e12aa3f2feacb8549eb85f966c9dc8426f3af
Reviewed-on: http://review.whamcloud.com/19646
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
Reviewed-by: John L. Hammond <john.hammond@intel.com>
Reviewed-by: Li Xi <lixi@ddn.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-8024 kernel: kernel update [SLES12 SP1 3.12.57-60.35] 93/19593/2
Bob Glossman [Thu, 14 Apr 2016 19:32:34 +0000 (12:32 -0700)]
LU-8024 kernel: kernel update [SLES12 SP1 3.12.57-60.35]

Update target and kernel_config files for new version

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

Signed-off-by: Bob Glossman <bob.glossman@intel.com>
Change-Id: Ic9f33c4877f93249b37726d8ad60c98ee624f719
Reviewed-on: http://review.whamcloud.com/19593
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Yang Sheng <yang.sheng@intel.com>
Reviewed-by: Minh Diep <minh.diep@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-7986 hsm: update actions llog in place 47/19447/2
John L. Hammond [Fri, 5 Jun 2015 21:39:23 +0000 (16:39 -0500)]
LU-7986 hsm: update actions llog in place

Update HSM actions llog entries in place rather than doing an append
and cancel.

Signed-off-by: John L. Hammond <john.hammond@intel.com>
Change-Id: I5c5282d199e43fbc6709d5dc9bc050e0b98fb889
Reviewed-by: Jinshan Xiong <jinshan.xiong@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
Reviewed-on: http://review.whamcloud.com/19447
Tested-by: Jenkins
Reviewed-by: Frank Zago <fzago@cray.com>
Reviewed-by: jacques-Charles Lafoucriere <jacques-charles.lafoucriere@cea.fr>
Tested-by: Maloo <hpdd-maloo@intel.com>
4 years agoLU-7992 hsm: close KUC pipe before unregistering 42/19442/2
John L. Hammond [Thu, 4 Jun 2015 14:48:44 +0000 (09:48 -0500)]
LU-7992 hsm: close KUC pipe before unregistering

In llapi_hsm_copytool_unregister() close the read side of the KUC pipe
before unregistering the copytool to avoid deadlocking with a ldlm_cb
thread.

Signed-off-by: John L. Hammond <john.hammond@intel.com>
Change-Id: I844086b32a3b2958ecaec391accded0ca06f60a2
Reviewed-by: Jinshan Xiong <jinshan.xiong@intel.com>
Reviewed-by: Bob Glossman <bob.glossman@intel.com>
Reviewed-by: Faccini Bruno <bruno.faccini@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
Reviewed-on: http://review.whamcloud.com/19442
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Frank Zago <fzago@cray.com>
4 years agoLU-7998 utils: fix fd lost 08/19408/4
Dmitry Eremin [Fri, 8 Apr 2016 15:20:01 +0000 (18:20 +0300)]
LU-7998 utils: fix fd lost

A file descriptor for "/dev/loop-control" was never closed.

Change-Id: I0dc1cb2453f49322315f00266308c829c04b4792
Signed-off-by: Dmitry Eremin <dmitry.eremin@intel.com>
Reviewed-on: http://review.whamcloud.com/19408
Reviewed-by: John L. Hammond <john.hammond@intel.com>
Reviewed-by: Bob Glossman <bob.glossman@intel.com>
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-7835 checkpatch: Update gerrit_checkpatch for new features. 48/18748/4
Ashley Pittman [Wed, 2 Mar 2016 12:24:12 +0000 (12:24 +0000)]
LU-7835 checkpatch: Update gerrit_checkpatch for new features.

Allow checking of a single patch per invocation rather than running as
daemon. Update wiki URL for coding standards. Supply --notree option
allowing upstream checkpatch.pl to be used. Add linebreaks to output
when reporting multiple errors per line of source

Signed-off-by: Ashley Pittman <ashley.m.pittman@intel.com>
Change-Id: I3823db55073b606fb84b84f520e598e8427b22e7
Signed-off-by: Ashley Pittman <ashley.m.pittman@intel.com>
Reviewed-on: http://review.whamcloud.com/18748
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: John L. Hammond <john.hammond@intel.com>
4 years agoLU-7791 ldlm: signal vs CP callback race 98/18498/5
Andriy Skulysh [Tue, 1 Mar 2016 22:15:55 +0000 (00:15 +0200)]
LU-7791 ldlm: signal vs CP callback race

a lock isn't canceled on the server on reception
of a signal on the client when it is blocked waiting
for a lock to be granted, thus client is evicted for
not responding to LDLM_FL_AST_SENT.

Seagate-bug-id: MRP-3314
Change-Id: Ie025bb58b13fb2d741119f74c87439f917983268
Signed-off-by: Andriy Skulysh <andriy.skulysh@seagate.com>
Reviewed-on: http://review.whamcloud.com/18498
Tested-by: Jenkins
Reviewed-by: Vitaly Fertman <vitaly.fertman@seagate.com>
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-7434 ptlrpc: lost bulk leads to a hang 21/17221/10
Vitaly Fertman [Tue, 1 Mar 2016 23:46:31 +0000 (02:46 +0300)]
LU-7434 ptlrpc: lost bulk leads to a hang

The reverse order of request_out_callback() and reply_in_callback()
puts the RPC into UNREGISTERING state, which is waiting for RPC &
bulk md unlink, whereas only RPC md unlink has been called so far.
If bulk is lost, even expired_set does not check for UNREGISTERING
state.

The same for write if server returns an error.

This phase is ambiguous, split to UNREG_RPC and UNREG_BULK.

Signed-off-by: Vitaly Fertman <vitaly.fertman@seagate.com>
Change-Id: Ib1eeb1777ad1ab4c7ea1c83fe95dc9ae82c1894c
Seagate-bug-id:  MRP-2953, MRP-3206
Reviewed-by: Andriy Skulysh <andriy.skulysh@seagate.com>
Reviewed-by: Alexey Leonidovich Lyashkov <alexey.lyashkov@seagate.com>
Tested-by: Elena V. Gryaznova <elena.gryaznova@seagate.com>
Reviewed-on: http://review.whamcloud.com/17221
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-7323 test: Unknown symbol mb_cache_create (err 0)" on RHEL7 00/16900/7
Parinay Kondekar [Wed, 6 Apr 2016 04:43:45 +0000 (10:13 +0530)]
LU-7323 test: Unknown symbol mb_cache_create (err 0)" on RHEL7

The error message is seen if mbcache module isn't loaded.

Before RHEL7 it was part of ext4, but EL7 have mbcache as
separate module and needs an extra `modprobe mbcache` command
before tests are run from local build dir.

This fixes the function load_modules_local() in test-framework.sh

Seagate-bug-id: MRP-3018
Change-Id: I7f1317355785c191287788bcb177d78b12231c97
Signed-off-by: Alexey Lyashkov <alexey.lyashkov@seagate.com>
Signed-off-by: Parinay Kondekar <parinay.kondekar@seagate.com>
Reviewed-on: http://review.whamcloud.com/16900
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Christopher J. Morrone <morrone2@llnl.gov>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-7130 test: sanity/162 doesn't handle DIR values properly 58/16358/4
Kyrylo Shatskyy [Wed, 13 Apr 2016 06:29:32 +0000 (11:59 +0530)]
LU-7130 test: sanity/162 doesn't handle DIR values properly

The test fails when path values have trailing slash,
e.g. DIR=/mnt/lustre/

The patch adds validation which at the moment processes
these parameters-DIR, DIR1, DIR2, MOUNT, MOUNT1, MOUNT2.

Change-Id: Ief85e8798911cb67b4dafce985a9e4ef56cc26a4
Signed-off-by: Kyrylo Shatskyy <kyrylo.shatskyy@seagate.com>
Signed-off-by: Parinay Kondekar <parinay.kondekar@seagate.com>
Seagate-bug-id: MRP-334
Reviewed-on: http://es-gerrit.xyus.xyratex.com:8080/#/c/269
Tested-by: Jenkins
Reviewed-by: Andrew Perepechko <Andrew_Perepechko@xyratex.com>
Reviewed-by: Alexander Lezhoev <Alexander_Lezhoev@xyratex.com>
Tested-by: Alexander Lezhoev <Alexander_Lezhoev@xyratex.com>
Reviewed-by: Vitaly Fertman <Vitaly_Fertman@xyratex.com>
Reviewed-on: http://review.whamcloud.com/16358
Reviewed-by: James Nunez <james.a.nunez@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-7121 tests: Extra items shown in the stack trace in t-f 38/16338/4
Parinay Kondekar [Wed, 9 Sep 2015 12:09:39 +0000 (17:39 +0530)]
LU-7121 tests: Extra items shown in the stack trace in t-f

The call to functions error_exit(), error_noexit(), error()
in turn call function log_trace_dump().

This call along with important information about where the
particular test failed shows some extra information in the
dump trace.

This changes fix the problem noticed and dumps only the relevant
information instead.

Signed-off-by: Parinay Kondekar <parinay.kondekar@seagate.com>
Change-Id: I5fba23f6e296e77feb0339c5e20815c288b8b751
Seagate-Bug-Id: MRP-891
Reviewed-on: http://es-gerrit.xyus.xyratex.com:8080/721/
Tested-by: Jenkins
Reviewed-by: Elena V. Gryaznova <elena.gryaznova@seagate.com>
Tested-by: Elena V. Gryaznova <elena.gryaznova@seagate.com>
Reviewed-by: Kyrylo Shatskyy <kyrylo_shatskyy@xyratex.com>
Signed-off-by: Kyrylo Shatskyy <kyrylo_shatskyy@xyratex.com>
Reviewed-on: http://review.whamcloud.com/16338
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: James Nunez <james.a.nunez@intel.com>
Reviewed-by: Wei Liu <wei3.liu@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-6245 libcfs: replace CFS_MODULE_PARAM with linux kernel module api 83/19383/6
James Simmons [Mon, 11 Apr 2016 20:01:00 +0000 (16:01 -0400)]
LU-6245 libcfs: replace CFS_MODULE_PARAM with linux kernel module api

Remove the CFS_MODULE_PARM wrapper and use the linux macros
module_param and MODULE_PARM_DESC instead.

Change-Id: If551518b34195caf92435189f98b01895a0b0d24
Signed-off-by: James Simmons <uja.ornl@yahoo.com>
Reviewed-on: http://review.whamcloud.com/19383
Reviewed-by: Frank Zago <fzago@cray.com>
Reviewed-by: Dmitry Eremin <dmitry.eremin@intel.com>
Reviewed-by: John L. Hammond <john.hammond@intel.com>
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-7435 test: managing OST pools properly 26/17326/5
Elena Gryaznova [Sat, 16 Jan 2016 22:57:01 +0000 (00:57 +0200)]
LU-7435 test: managing OST pools properly

ost-pools tests fail on configuration MGS/MDS not combined,
different nodes, configured as failover pair.

Patch fixes ost-pools to execute commands on mgs instead of
$SINGLEMDS.

Seagate-bug-id: MRP-2733
Signed-off-by: Elena Gryaznova <elena.gryaznova@seagate.com>
Change-Id: I8dfe4cc38a30af59f6cf5f4dcd144588b499c3c3
Reviewed-on: http://review.whamcloud.com/17326
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: James Nunez <james.a.nunez@intel.com>
Reviewed-by: Wei Liu <wei3.liu@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-7699 build: Convert lustre_ver.h.in into a static .h file 11/18111/13
Christopher J. Morrone [Sun, 24 Jan 2016 00:44:48 +0000 (16:44 -0800)]
LU-7699 build: Convert lustre_ver.h.in into a static .h file

The AC_LUSTRE_{MAJOR,MINOR,PATCH,FIX} autoconf output values are
no longer necessary.  We can instead use AC_DEFINE to set the
LUSTRE_{MAJOR,MINOR,PATCH,FIX} CPP macros directly.  As a result,
the lustre_ver.h file no longer needs to be generated by autoconf
and it can be made into a static file.

At the same time we set LUSTRE_VERSION_STRING, even though it
is not strictly needed, to avoid additional divergence from
the client code in the upstream Linux kernel staging area.

Change-Id: I6b0364a7af1869beb6acedc63c034565af00ef06
Signed-off-by: Christopher J. Morrone <morrone2@llnl.gov>
Reviewed-on: http://review.whamcloud.com/18111
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-7110 tests: sanity-hsm.sh can erase all files on filesystem 91/16291/7
Parinay Kondekar [Tue, 12 Apr 2016 06:44:03 +0000 (12:14 +0530)]
LU-7110 tests: sanity-hsm.sh can erase all files on filesystem

For SINGLEAGT=client, $HSM_ARCHIVE & $hsm_root is empty.
Thus purging of archive in copytool_setup() will result
in execution of "rm -rf /*", resulting in permanent loss
of files as well as cluster node.

This change fixes it by checking the respective variables
like HSM_ARCHIVE or hsm_root to be non-empty.

Change-Id: I73cdc08a58409ae1f3ba85a9d12077062a38b321
Seagate-Bug-Id: MRP-2659
Signed-off-by: Parinay Kondekar <parinay.kondekar@seagate.com>
Reviewed-on: http://es-gerrit.xyus.xyratex.com:8080/6471
Tested-by:Jenkins
Reviewed-by: Elena V. Gryaznova <elena.gryaznova@seagate.com>
Reviewed-by: Alexander Lezhoev <alexander.lezhoev@seagate.com>
Reviewed-on: http://review.whamcloud.com/16291
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
Reviewed-by: jacques-Charles Lafoucriere <jacques-charles.lafoucriere@cea.fr>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoNew version 2.8.52 2.8.52 v2_8_52 v2_8_52_0
Oleg Drokin [Thu, 21 Apr 2016 01:41:39 +0000 (21:41 -0400)]
New version 2.8.52

Change-Id: I7c5d288c7d2a1290bac5a4c3e179977672813194
Signed-off-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-7220 llog: fix fd lost and correct error reporting 07/19407/3
Dmitry Eremin [Fri, 8 Apr 2016 15:09:23 +0000 (18:09 +0300)]
LU-7220 llog: fix fd lost and correct error reporting

A file descriptor acquired to 'fd' may be lost in case of
error in is_fstype_ext() function.

Fix error code of is_fstype_ext() propagation.

Change-Id: I0894cfad83854c503e2cdcfff78d44f6f2746ef3
Signed-off-by: Dmitry Eremin <dmitry.eremin@intel.com>
Reviewed-on: http://review.whamcloud.com/19407
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Bob Glossman <bob.glossman@intel.com>
Reviewed-by: John L. Hammond <john.hammond@intel.com>
Tested-by: Jenkins
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-7990 rpc: increase bulk size 66/19366/3
Jinshan Xiong [Fri, 9 May 2014 00:13:19 +0000 (17:13 -0700)]
LU-7990 rpc: increase bulk size

To make the ptlrpc be able to size 16MB IO

Signed-off-by: Jinshan Xiong <jinshan.xiong@intel.com>
Signed-off-by: Gu Zheng <gzheng@ddn.com>
Change-Id: I32727beaed08cb0e9048e7fe0e27489e330fce60
Reviewed-on: http://review.whamcloud.com/19366
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-7967 kernel: kernel update RHEL7.2 [3.10.0-327.13.1.el7] 89/19289/4
Bob Glossman [Thu, 31 Mar 2016 15:49:39 +0000 (08:49 -0700)]
LU-7967 kernel: kernel update RHEL7.2 [3.10.0-327.13.1.el7]

Update RHEL7.2 kernel to 3.10.0-327.13.1.el7

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

Signed-off-by: Bob Glossman <bob.glossman@intel.com>
Change-Id: I341473cfef85a687aa2218f457e757186ac923f0
Reviewed-on: http://review.whamcloud.com/19289
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Minh Diep <minh.diep@intel.com>
Reviewed-by: Yang Sheng <yang.sheng@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-4182 tests: enable sanity-scrub test_15 under DNE mode 40/18440/2
Fan Yong [Tue, 1 Dec 2015 15:13:32 +0000 (23:13 +0800)]
LU-4182 tests: enable sanity-scrub test_15 under DNE mode

Drop unnecessary condition check.

Test-Parameters: mdsfilesystemtype=ldiskfs mdtfilesystemtype=ldiskfs ostfilesystemtype=ldiskfs mdtcount=4 mdscount=2 envdefinitions=ONLY=15 testlist=sanity-scrub,sanity-scrub,sanity-scrub
Signed-off-by: Fan Yong <fan.yong@intel.com>
Change-Id: Ic355218b2c449a9979bec22f613e6f6646601870
Reviewed-on: http://review.whamcloud.com/18440
Tested-by: Jenkins
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Jian Yu <jian.yu@intel.com>
4 years agoLU-28 fileset: add fileset mount support 07/5007/39
Lai Siyao [Tue, 3 Mar 2015 12:36:31 +0000 (20:36 +0800)]
LU-28 fileset: add fileset mount support

This patch enables client to mount subdirectory as fileset.

usage: mount -t lustre mgsname:/fsname/subdir /mount/point

* mdt lookup fileset fid and return to client during mount.
* `lfs fid2path` support for fileset.

Test-Parameters: envdefinitions=FILESET=/somedir testlist=sanity
Signed-off-by: Lai Siyao <lai.siyao@intel.com>
Signed-off-by: Kit Westneat <kwestneat@ddn.com>
Signed-off-by: Wang Shilong <wshilong@ddn.com>
Change-Id: I947785de3b053907600d1090c80e8f57396e1e93
Reviewed-on: http://review.whamcloud.com/5007
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-4931 ladvise: Add feature of giving file access advices 29/10029/52
Li Xi [Tue, 22 Mar 2016 00:12:55 +0000 (08:12 +0800)]
LU-4931 ladvise: Add feature of giving file access advices

The fadvise() system call provided by Linux kernel enables
applications to give advices or hints about how a file will be
accessed. However, It is only a client side mechanism which is
not enough for distributed file systems like Lustre, because in
order to tune system-wide cache or read-ahead policies, servers
need to understand the advices too.

This patch adds a new feature named ladvise which provides new
APIs and utils to give advices about the access pattern of Lustre
files with the purpose of performance improvement. It is similar
to Linux fadvise() system call, except it forwards the advices
directly from Lustre client to server. The server side codes will
apply appropriate read-ahead and caching techniques for the
corresponding files.

A typical workload for ladvise is e.g. a bunch of different
clients are doing small random reads of a file, so prefetching
pages into OSS cache with big linear reads before the random IO
is a net benefit. Fetching all that data into each client cache
with fadvise() may not be, due to much more data being sent to
the client.

Signed-off-by: Li Xi <lixi@ddn.com>
Change-Id: I6de14ed87199949963360d393897ff79826c9819
Reviewed-on: http://review.whamcloud.com/10029
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Wang Shilong <wshilong@ddn.com>
Reviewed-by: Jinshan Xiong <jinshan.xiong@intel.com>
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-7699 build: Eliminate lustre_build_version.h 08/18108/11
Christopher J. Morrone [Wed, 23 Mar 2016 23:35:50 +0000 (19:35 -0400)]
LU-7699 build: Eliminate lustre_build_version.h

The lustre_build_version.h is no longer necessary.  With the new
versioning system the lustre version can now be retrieved from
standard autotools defines and/or lustre_ver.h.

To make compatibility easier for the upstream lustre client in
the linux kernel, we use LUSTRE_VERSION_STRING from lustre_ver.h.

Change-Id: I08233d8547bc4af7bf2fb0c15bee8e306f6fa0f0
Signed-off-by: Christopher J. Morrone <morrone2@llnl.gov>
Reviewed-on: http://review.whamcloud.com/18108
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-6601 osp: update obd status while osp status changed 02/19402/2
Yang Sheng [Fri, 8 Apr 2016 04:10:43 +0000 (12:10 +0800)]
LU-6601 osp: update obd status while osp status changed

We should update OBD status when osp import status is
changed. Else the status change will not reflect in
'lctl dl'.

Signed-off-by: Yang Sheng <yang.sheng@intel.com>
Change-Id: Ice10802028b569470027db33a5e5e5e2bcc9dfc0
Reviewed-on: http://review.whamcloud.com/19402
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
Reviewed-by: Niu Yawei <yawei.niu@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-7925 llite: avoid clearing i_nlink for inodes in use 64/19164/4
Andrew Perepechko [Sat, 26 Mar 2016 14:48:25 +0000 (17:48 +0300)]
LU-7925 llite: avoid clearing i_nlink for inodes in use

The patch removes find_cbdata callbacks and clear_nlink
from dentry_iput path, since this piece of code makes
a few races possible.

The test case reproduces one of the possible races
described in LU-7925:

1) two hard links are created for the same file
2) the test calls stat(2) for link #1
3) in the middle of 2) the test opens and closes link #2
4) in the middle of 2) the test drops the ldlm locks and
   forces dentry reclaim via vm.drop_caches=2
5) in the middle of 2) ll_d_iput() clears i_nlink for
   the inode
6) the initial stat(2) continues and copies the wrong
   i_nlink value into st_nlink

Signed-off-by: Andrew Perepechko <andrew.perepechko@seagate.com>
Seagate-bug-id: MRP-3271
Change-Id: I185bed4b875a4fbf66df835cf9161912a6ade415
Reviewed-on: http://review.whamcloud.com/19164
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Wally Wang <wang@cray.com>
Reviewed-by: Lai Siyao <lai.siyao@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-7979 o2iblnd: fix free of IS_ERR pointer 11/19311/2
Dmitry Eremin [Mon, 4 Apr 2016 10:21:18 +0000 (13:21 +0300)]
LU-7979 o2iblnd: fix free of IS_ERR pointer

Explicitly nullify pointers is case of error.

Change-Id: Ic25ef3323b8ef4e8bf1da442b2328e6801b3d252
Signed-off-by: Dmitry Eremin <dmitry.eremin@intel.com>
Reviewed-on: http://review.whamcloud.com/19311
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>
Reviewed-by: Doug Oucharek <doug.s.oucharek@intel.com>
4 years agoLU-7965 utils: fix stack corruption in mkfs.lustre 60/19260/3
Gregoire Pichon [Thu, 31 Mar 2016 11:05:20 +0000 (13:05 +0200)]
LU-7965 utils: fix stack corruption in mkfs.lustre

This patch fixes a stack corruption that randomly happens when
formating a Lustre target. A call to append_unique() routine
was done with a wrong maxbuflen value.

Signed-off-by: Gregoire Pichon <gregoire.pichon@bull.net>
Change-Id: I8838b90a1dcda7c27457c0be0246e216856e7d5d
Reviewed-on: http://review.whamcloud.com/19260
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
Reviewed-by: lokesh.jaliminche <lokesh.jaliminche@seagate.com>
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
4 years agoLU-7962 build: Support builds w/ weak module ZFS 49/19249/4
Christopher J. Morrone [Tue, 4 Aug 2015 01:37:47 +0000 (18:37 -0700)]
LU-7962 build: Support builds w/ weak module ZFS

Now that ZFS supports weak module updates, ZFS does will not
always be built for every specific kernel that is installed.
Unfortunately, some directories that Lustre needs from ZFS and
SPL have a kernel string embedded in them, and now that kernel
string will potentially be different than the kernel that
Lustre is building against.

As a work around we can update the Lustre build scripts to just
select the last subdirectory (after all other methods fail) that
has a kernel name from the ZFS and SPL build directories as a
directory that Lustre should use in its build. Its as reasonable
a guess as I can figure we can easily make.

Change-Id: Id17940b78b99ddb8b589d1f46f85c3282c71a3ad
Signed-off-by: Christopher J. Morrone <morrone2@llnl.gov>
Reviewed-on: http://review.whamcloud.com/19249
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Nathaniel Clark <nathaniel.l.clark@intel.com>
Reviewed-by: Faccini Bruno <bruno.faccini@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-7936 utils: fix resource lost 93/19193/5
Dmitry Eremin [Tue, 29 Mar 2016 14:33:48 +0000 (17:33 +0300)]
LU-7936 utils: fix resource lost

Close file handle before return from function.

Change-Id: I1cea4c1d60672b945cd524367e91fb27df185c87
Signed-off-by: Dmitry Eremin <dmitry.eremin@intel.com>
Reviewed-on: http://review.whamcloud.com/19193
Tested-by: Jenkins
Reviewed-by: Bob Glossman <bob.glossman@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-6401 headers: move seq_range.h to correct directory 63/18863/4
Ben Evans [Thu, 10 Mar 2016 22:09:47 +0000 (16:09 -0600)]
LU-6401 headers: move seq_range.h to correct directory

in change http://review.whamcloud.com/#/c/16339/ seq_range.h
was put in the wrong directory, and was requested to move it
up a level.

Signed-off-by: Ben Evans <bevans@cray.com>
Change-Id: I461273c4d216714eddc7d4d5c7cdb8ac9f8c8678
Reviewed-on: http://review.whamcloud.com/18863
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: James Simmons <uja.ornl@yahoo.com>
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
Reviewed-by: John L. Hammond <john.hammond@intel.com>
4 years agoLU-7772 llog: do not destroy llog twice 62/18362/3
Alex Zhuravlev [Tue, 9 Feb 2016 13:00:04 +0000 (16:00 +0300)]
LU-7772 llog: do not destroy llog twice

if llog_process_thread() finds a llog corrupted, then it cancels
all remaining records which in turn causes destroy of this llog.
but it's still referenced by the catalog's in-code structures
(chd_head list) and llog_cat_close() at umount tries to destroy
this llog again. we can reset LLOG_F_ZAP_WHEN_EMPTY at destroy,
so it never repeats.

Change-Id: I8173d9dd2971fa5cce613e71555ca04e96b5512b
Signed-off-by: Alex Zhuravlev <alexey.zhuravlev@intel.com>
Reviewed-on: http://review.whamcloud.com/18362
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: wangdi <di.wang@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-7055 osd: move dput() out of osd_ost_fini()'s rwlock 89/19189/2
Emoly Liu [Tue, 29 Mar 2016 07:04:44 +0000 (15:04 +0800)]
LU-7055 osd: move dput() out of osd_ost_fini()'s rwlock

osd_ost_fini() calls into osd_seq_free() under write lock, which
in turn calls into dput() that might sleep and cause "BUG: rwlock
wrong CPU on CPU#x, umount/xxx" sort of messages.
To fix this issue, this patch only keeps list_del_init() under
write lock, and moves sleeping function out of there.

Signed-off-by: Emoly Liu <emoly.liu@intel.com>
Change-Id: Ie1eef345d1735bb88aeb016a2409ea649cd20fe5
Reviewed-on: http://review.whamcloud.com/19189
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>
4 years agoLU-6245 libcfs: remove libcfs userland time abstraction 13/19113/3
James Simmons [Tue, 29 Mar 2016 00:30:55 +0000 (20:30 -0400)]
LU-6245 libcfs: remove libcfs userland time abstraction

Instead of creating a time abstraction layer in libcfs
for userland lets just use the userland time APIs
directly.

Change-Id: I5c91b23d4ae6c4f75f7712c28572818f26fe57e2
Signed-off-by: James Simmons <uja.ornl@yahoo.com>
Reviewed-on: http://review.whamcloud.com/19113
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Frank Zago <fzago@cray.com>
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
Reviewed-by: John L. Hammond <john.hammond@intel.com>
4 years agoLU-7759 utils: build mount.lustre with libmount 20/18820/5
Yang Sheng [Tue, 8 Mar 2016 17:21:14 +0000 (01:21 +0800)]
LU-7759 utils: build mount.lustre with libmount

Newer distro, like RHEL7, SLES12 will invoke statfs when
umount. It will block while some OST unavailable. So we
write a entry in utab file to avoid invoke statfs() before
issuing the unmount call.

Signed-off-by: Yang Sheng <yang.sheng@intel.com>
Change-Id: I3930ac3454a568908f0872f4a0d83ab264742f3c
Reviewed-on: http://review.whamcloud.com/18820
Tested-by: Jenkins
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Bob Glossman <bob.glossman@intel.com>
4 years agoLU-4423 lnet: remove LNET_MUTEX_LOCK macro 85/18985/3
Mike Shuey [Thu, 17 Mar 2016 17:20:18 +0000 (13:20 -0400)]
LU-4423 lnet: remove LNET_MUTEX_LOCK macro

LNET_MUTEX_LOCK and LNET_MUTEX_UNLOCK are verbose wrappers to mutex_lock and
mutex_unlock.  Get rid of these.

Change-Id: Icdb029441d57ce080ce4b1e7990d38d4ff0edf26
Signed-off-by: Mike Shuey <shuey@purdue.edu>
Signed-off-by: James Simmons <uja.ornl@yahoo.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Reviewed-on: http://review.whamcloud.com/18985
Tested-by: Jenkins
Reviewed-by: Frank Zago <fzago@cray.com>
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Doug Oucharek <doug.s.oucharek@intel.com>
Reviewed-by: Dmitry Eremin <dmitry.eremin@intel.com>
Reviewed-by: John L. Hammond <john.hammond@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-7101 lnet: per NI map-on-demand value 67/16367/18
Amir Shehata [Tue, 15 Mar 2016 19:39:54 +0000 (15:39 -0400)]
LU-7101 lnet: per NI map-on-demand value

Enables support of different map-on-demand values per NI.  This is
required to support OPA coexistence with MLX5 cards.  MLX5 does not
support FMR, which is enabled via map-on-demand.  However OPA's
performance is greatly enahanced when FMR is enabled.  In order
to enable coexistence of both of these two types of cards we
need to be able to set different map-on-demand values for both
NIs.

This patch also lays the ground work for other per NI tunables to
be added in future patches.

Signed-off-by: Amir Shehata <amir.shehata@intel.com>
Signed-off-by: James Simmons <uja.ornl@yahoo.com>
Change-Id: Ic7617d3d5846e58f83e7c67bb9eb7173700be8d7
Reviewed-on: http://review.whamcloud.com/16367
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Doug Oucharek <doug.s.oucharek@intel.com>
Reviewed-by: Olaf Weber <olaf@sgi.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-7355 osd: supress useless warning about many credits 85/16985/14
Alex Zhuravlev [Thu, 29 Oct 2015 15:26:43 +0000 (18:26 +0300)]
LU-7355 osd: supress useless warning about many credits

on tiny systems (like we used to debug Lustre) journals are
tiny as max.transaction too. it doesn't help to warn about
this. we've got another precise mechanism to catch credits
misuse so it should be OK to suppress these messages.

Change-Id: I9599ae856bedb0a08a8dac01821e2c9a72377cac
Signed-off-by: Alex Zhuravlev <alexey.zhuravlev@intel.com>
Reviewed-on: http://review.whamcloud.com/16985
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-7060 ldev: Added MGS NID substitution to ldev 29/16129/11
Giuseppe Di Natale [Fri, 28 Aug 2015 23:17:24 +0000 (16:17 -0700)]
LU-7060 ldev: Added MGS NID substitution to ldev

For automation purposes, it would be nice to have a
command substitution for the MGS NID of a node.
The ldev perl script now has a %m flag to sub in
the MGS NID. The MGS associated with a node is
determined by which file system the node is a part
of. The NID is obtained by using the current
NID file parsing. Also, MGS is now an acceptable
type to have specified in a label.

Signed-off-by: Giuseppe Di Natale <dinatale2@llnl.gov>
Change-Id: Ifcfad7664ac028e2074030e4b3ba5995ddb5358d
Reviewed-on: http://review.whamcloud.com/16129
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
Reviewed-by: Bob Glossman <bob.glossman@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-7907 kernel: kernel update RHEL 6.7 [2.6.32-573.22.1.el6] 18/19118/2
Bob Glossman [Tue, 22 Mar 2016 22:38:40 +0000 (15:38 -0700)]
LU-7907 kernel: kernel update RHEL 6.7 [2.6.32-573.22.1.el6]

Update RHEL 6.7 kernel to 2.6.32-573.22.1.el6

Signed-off-by: Bob Glossman <bob.glossman@intel.com>
Change-Id: I2736a6390607d552ea171bdf53dd071a8575a4fa
Reviewed-on: http://review.whamcloud.com/19118
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Yang Sheng <yang.sheng@intel.com>
Reviewed-by: Dmitry Eremin <dmitry.eremin@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-4423 lnet: Drop unnecessary wrapper function srpc_post_active_rqtbuf 57/18957/7
Shivani Bhardwaj [Tue, 29 Mar 2016 00:27:24 +0000 (20:27 -0400)]
LU-4423 lnet: Drop unnecessary wrapper function srpc_post_active_rqtbuf

Remove the function srpc_post_active_rqtbuf() and replace its only call
with the function srpc_post_active_rdma() by adding appropriate parameters.

Change-Id: I0fcbfa6bc07f04ae218abac7d25264af7072b902
Signed-off-by: Shivani Bhardwaj <shivanib134@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: James Simmons <uja.ornl@yahoo.com>
Reviewed-on: http://review.whamcloud.com/18957
Reviewed-by: Doug Oucharek <doug.s.oucharek@intel.com>
Tested-by: Jenkins
Reviewed-by: Dmitry Eremin <dmitry.eremin@intel.com>
Reviewed-by: Frank Zago <fzago@cray.com>
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
4 years agoLU-4423 lnet: remove memset(0) after LIBCFS_ALLOC 54/18954/7
Vitaly Osipov [Tue, 29 Mar 2016 00:25:12 +0000 (20:25 -0400)]
LU-4423 lnet: remove memset(0) after LIBCFS_ALLOC

Joe Perches mentioned on driverdev-devel that memset after LIBCFS_ALLOC
is not necessary as it is already done during LIBCFS_ALLOC_POST. This
commit removes these unnecessary memsets. Based on the results of running
a cocci patch along the lines of:

@@
expression E1, E2;
@@

LIBCFS_ALLOC (E1,E2);
...
- memset(E1,0,E2);

Linux-commit : cab80d98f50bbacb7df79bd267d0b2b008b2d84f

Change-Id: I49b48539d7b99e59e3bb56f9ad74efed101d8a4e
Signed-off-by: Vitaly Osipov <vitaly.osipov@gmail.com>
Reviewed-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: James Simmons <uja.ornl@yahoo.com>
Reviewed-on: http://review.whamcloud.com/18954
Reviewed-by: Doug Oucharek <doug.s.oucharek@intel.com>
Tested-by: Jenkins
Reviewed-by: Dmitry Eremin <dmitry.eremin@intel.com>
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Frank Zago <fzago@cray.com>
Reviewed-by: John L. Hammond <john.hammond@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>