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
next 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).