[Cluster-devel] [[GFS2 PATCH] 0/3] Patches for gfs2_lookup_by_inum deadlock

[Bob Peterson]

----- Original Message -----
| 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 at redhat.com>
| Signed-off-by: Bob Peterson <rpeterso at 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
| 
| 
Hi,

Thanks. These are now applied to the for-next branch of the linux-gfs2 tree:
https://git.kernel.org/cgit/linux/kernel/git/gfs2/linux-gfs2.git/commit/fs?h=for-next&id=3ce37b2cb4917674fa5b776e857dcea94c0e0835
https://git.kernel.org/cgit/linux/kernel/git/gfs2/linux-gfs2.git/commit/fs?h=for-next&id=ec5ec66ba48bd3163110599359797858ac38e79b
https://git.kernel.org/cgit/linux/kernel/git/gfs2/linux-gfs2.git/commit/fs?h=for-next&id=cda9dd4207aeb29d0aa2298085cc2d1ebcb87e04

Regards,

Bob Peterson
Red Hat File Systems