From: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
To: James Simmons <jsimmons@infradead.org>
Cc: devel@driverdev.osuosl.org,
Andreas Dilger <andreas.dilger@intel.com>,
Oleg Drokin <oleg.drokin@intel.com>,
Sebastien Buisson <sbuisson@ddn.com>,
Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
Lustre Development List <lustre-devel@lists.lustre.org>
Subject: [lustre-devel] [PATCH 01/17] staging: lustre: osc: fix race issues thanks to oap_lock
Date: Mon, 11 Apr 2016 21:05:08 -0700 [thread overview]
Message-ID: <20160412040508.GA2578@kroah.com> (raw)
In-Reply-To: <1460294020-2111-2-git-send-email-jsimmons@infradead.org>
On Sun, Apr 10, 2016 at 09:13:24AM -0400, James Simmons wrote:
> From: Sebastien Buisson <sbuisson@ddn.com>
>
> Fix 'data race condition' defects found by Coverity version 6.5.0:
>
> Data race condition (MISSING_LOCK)
>
> Accessing variable without holding lock. Elsewhere,
> this variable is accessed with lock held.
>
> This patch is dedicated to code fragments involving oap_lock.
>
> Signed-off-by: Sebastien Buisson <sbuisson@ddn.com>
> Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-2744
> Reviewed-on: http://review.whamcloud.com/6572
> Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
> Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
> Signed-off-by: James Simmons <jsimmons@infradead.org>
> ---
> drivers/staging/lustre/lustre/osc/osc_cache.c | 8 ++++++++
> drivers/staging/lustre/lustre/osc/osc_io.c | 2 ++
> 2 files changed, 10 insertions(+), 0 deletions(-)
This patch doesn't apply at all. Can you rebase the whole series and
resend them?
thanks,
greg k-h
WARNING: multiple messages have this Message-ID (diff)
From: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
To: James Simmons <jsimmons@infradead.org>
Cc: devel@driverdev.osuosl.org,
Andreas Dilger <andreas.dilger@intel.com>,
Oleg Drokin <oleg.drokin@intel.com>,
Sebastien Buisson <sbuisson@ddn.com>,
Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
Lustre Development List <lustre-devel@lists.lustre.org>
Subject: Re: [PATCH 01/17] staging: lustre: osc: fix race issues thanks to oap_lock
Date: Mon, 11 Apr 2016 21:05:08 -0700 [thread overview]
Message-ID: <20160412040508.GA2578@kroah.com> (raw)
In-Reply-To: <1460294020-2111-2-git-send-email-jsimmons@infradead.org>
On Sun, Apr 10, 2016 at 09:13:24AM -0400, James Simmons wrote:
> From: Sebastien Buisson <sbuisson@ddn.com>
>
> Fix 'data race condition' defects found by Coverity version 6.5.0:
>
> Data race condition (MISSING_LOCK)
>
> Accessing variable without holding lock. Elsewhere,
> this variable is accessed with lock held.
>
> This patch is dedicated to code fragments involving oap_lock.
>
> Signed-off-by: Sebastien Buisson <sbuisson@ddn.com>
> Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-2744
> Reviewed-on: http://review.whamcloud.com/6572
> Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
> Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
> Signed-off-by: James Simmons <jsimmons@infradead.org>
> ---
> drivers/staging/lustre/lustre/osc/osc_cache.c | 8 ++++++++
> drivers/staging/lustre/lustre/osc/osc_io.c | 2 ++
> 2 files changed, 10 insertions(+), 0 deletions(-)
This patch doesn't apply at all. Can you rebase the whole series and
resend them?
thanks,
greg k-h
next prev parent reply other threads:[~2016-04-12 4:05 UTC|newest]
Thread overview: 48+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-04-10 13:13 [lustre-devel] [PATCH 00/17] staging : lustre : rest of missing patches from 2.5.0 release James Simmons
2016-04-10 13:13 ` James Simmons
2016-04-10 13:13 ` [lustre-devel] [PATCH 01/17] staging: lustre: osc: fix race issues thanks to oap_lock James Simmons
2016-04-10 13:13 ` James Simmons
2016-04-12 4:05 ` Greg Kroah-Hartman [this message]
2016-04-12 4:05 ` Greg Kroah-Hartman
2016-04-10 13:13 ` [lustre-devel] [PATCH 02/17] staging: lustre: clio: incorrect assertions in 'enable-invariants' James Simmons
2016-04-10 13:13 ` James Simmons
2016-04-10 13:13 ` [lustre-devel] [PATCH 03/17] staging: lustre: ldlm: Fix a race during FLock handling James Simmons
2016-04-10 13:13 ` James Simmons
2016-04-10 13:13 ` [lustre-devel] [PATCH 04/17] staging: lustre: ldlm: refine LU-2665 patch for POSIX compliance James Simmons
2016-04-10 13:13 ` James Simmons
2016-04-10 13:13 ` [lustre-devel] [PATCH 05/17] staging: lustre: llite: variable rename in namei.c James Simmons
2016-04-10 13:13 ` James Simmons
2016-04-10 13:13 ` [lustre-devel] [PATCH 06/17] staging: lustre: llite: speedup in unlink/rmdir James Simmons
2016-04-10 13:13 ` James Simmons
2016-04-10 13:13 ` [lustre-devel] [PATCH 07/17] staging: lustre: llite: error setting max_cache_mb at mount time James Simmons
2016-04-10 13:13 ` James Simmons
2016-04-10 13:13 ` [lustre-devel] [PATCH 08/17] staging: lustre: obd: MDT mount fails on MDS w/o MGS on it James Simmons
2016-04-10 13:13 ` James Simmons
2016-04-10 13:13 ` [lustre-devel] [PATCH 09/17] staging: lustre: ptlrpc: return a meaningful status from ptlrpcd_init() James Simmons
2016-04-10 13:13 ` James Simmons
2016-04-10 13:13 ` [lustre-devel] [PATCH 10/17] staging: lustre: llite: Truncate to restore file James Simmons
2016-04-10 13:13 ` James Simmons
2016-04-10 13:13 ` [lustre-devel] [PATCH 11/17] staging: lustre: osc: osc_extent_wait() shouldn't be interruptible James Simmons
2016-04-10 13:13 ` James Simmons
2016-04-10 13:13 ` [lustre-devel] [PATCH 12/17] staging: lustre: lprocfs: implement log2 using bitops James Simmons
2016-04-10 13:13 ` James Simmons
2016-04-10 13:13 ` [lustre-devel] [PATCH 13/17] staging: lustre: lloop: Fix build failure on ppc64 James Simmons
2016-04-10 13:13 ` James Simmons
2016-04-10 13:22 ` [lustre-devel] " Jeff Mahoney
2016-04-10 13:22 ` Jeff Mahoney
2016-04-10 14:04 ` [lustre-devel] " James Simmons
2016-04-10 14:04 ` James Simmons
2016-04-10 15:35 ` [lustre-devel] " Greg Kroah-Hartman
2016-04-10 15:35 ` Greg Kroah-Hartman
2016-04-10 15:52 ` [lustre-devel] " James Simmons
2016-04-10 15:52 ` James Simmons
2016-04-10 21:13 ` [lustre-devel] " Jeff Mahoney
2016-04-10 21:13 ` Jeff Mahoney
2016-04-10 13:13 ` [lustre-devel] [PATCH 14/17] staging: lustre: lov: return minimal FIEMAP for released files James Simmons
2016-04-10 13:13 ` James Simmons
2016-04-10 13:13 ` [lustre-devel] [PATCH 15/17] staging: lustre: lov: Don't wait for active target with OBD_STATFS_NODELAY James Simmons
2016-04-10 13:13 ` James Simmons
2016-04-10 13:13 ` [lustre-devel] [PATCH 16/17] staging: lustre: hsm: permission checks for HSM ioctl operations James Simmons
2016-04-10 13:13 ` James Simmons
2016-04-10 13:13 ` [lustre-devel] [PATCH 17/17] staging: lustre: hsm: don't use real suppgid James Simmons
2016-04-10 13:13 ` James Simmons
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20160412040508.GA2578@kroah.com \
--to=gregkh@linuxfoundation.org \
--cc=andreas.dilger@intel.com \
--cc=devel@driverdev.osuosl.org \
--cc=jsimmons@infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=lustre-devel@lists.lustre.org \
--cc=oleg.drokin@intel.com \
--cc=sbuisson@ddn.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.