cluster-devel.redhat.com archive mirror
 help / color / mirror / Atom feed
From: Bob Peterson <rpeterso@redhat.com>
To: cluster-devel.redhat.com
Subject: [Cluster-devel] [[GFS2 PATCH] 0/3] Patches for gfs2_lookup_by_inum deadlock
Date: Wed, 15 Jun 2016 09:46:49 -0500	[thread overview]
Message-ID: <1466002012-12115-1-git-send-email-rpeterso@redhat.com> (raw)

Hi,

This is a set of three patches from Andreas Gruenbacher that fix the
gfs2_lookup_by_inum deadlock problem. I've been working with Andreas
for a while now, and we've both made several attempts to fix this
problem in the past, in regard to the transition of dinodes from the
"unlinked" to the "free" state. This is the latest attempt, and it
seems to be working well.

Our previous attempt made a change to vfs, but Al Viro didn't like
that, so it was scrapped in favor of this one, which is simpler and
confined to GFS2. It's similar in concept to the patch set I posted
on 18 December 2015.

It also fixes a problem for 32-bit architecture that was introduced
by a recent patch related to the same problem.

Signed-off-by: Andreas Gruenbacher <agruenba@redhat.com>
Signed-off-by: Bob Peterson <rpeterso@redhat.com>
---
Andreas Gruenbacher (3):
  gfs2: Fix gfs2_lookup_by_inum lock inversion
  gfs2: Get rid of gfs2_ilookup
  gfs2: Large-filesystem fix for 32-bit systems

 fs/gfs2/dir.c        |   3 +-
 fs/gfs2/export.c     |  11 ------
 fs/gfs2/glock.c      |   9 +----
 fs/gfs2/inode.c      | 103 ++++++++++++++++++++++++++++++++++++---------------
 fs/gfs2/inode.h      |   4 +-
 fs/gfs2/ops_fstype.c |   3 +-
 6 files changed, 81 insertions(+), 52 deletions(-)

-- 
2.5.5



             reply	other threads:[~2016-06-15 14:46 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-06-15 14:46 Bob Peterson [this message]
2016-06-15 14:46 ` [Cluster-devel] [[GFS2 PATCH] 1/3] gfs2: Fix gfs2_lookup_by_inum lock inversion Bob Peterson
2016-06-15 14:46 ` [Cluster-devel] [[GFS2 PATCH] 2/3] gfs2: Get rid of gfs2_ilookup Bob Peterson
2016-06-16 15:51   ` Steven Whitehouse
2016-06-15 14:46 ` [Cluster-devel] [[GFS2 PATCH] 3/3] gfs2: Large-filesystem fix for 32-bit systems Bob Peterson
2016-06-16 15:48   ` Steven Whitehouse
2016-06-17  9:40 ` [Cluster-devel] [PATCH] gfs2: Initialize iopen glock holder for new inodes Andreas Gruenbacher
2016-06-17 13:41   ` Bob Peterson
2016-06-27 15:20 ` [Cluster-devel] [[GFS2 PATCH] 0/3] Patches for gfs2_lookup_by_inum deadlock Bob Peterson

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=1466002012-12115-1-git-send-email-rpeterso@redhat.com \
    --to=rpeterso@redhat.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).