Skip to content

jffs2: fix lockdep warning

Helmut Grohne linux-mtd 1 version In review

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest3 patches 3 repliesIn review

Patches in v1 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/3jffs2: avoid unnecessarily taking f->semfs/jffs2/readinode.c+1 -31 reply
2/3jffs2: jffs2_iget defer f->sem acquisitionfs/jffs2/fs.c+3 -11 reply
3/3jffs2: put directories f->sem on a separate lockdep classfs/jffs2/fs.c fs/jffs2/super.c+15 -04 replies

Reviews trailers per patch

2 of 3 patches have a Reviewed-by or Acked-by.

PatchRichard Weinberger
1/3R-b
2/3R-b
3/3

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.

Read the v1 thread

Cover letter [PATCH 0/3] jffs2: fix lockdep warning

This patch series essentially fixes a false positive lockdep warning. The issue is known since 2012, referenced and fixed in the third patch. The other two patches subtly change the locking semantics on order to make the third patch feasible.

I would like thank Peter Zijlstra for his precise guiding on where to look. His first guess on what is going nailed it.

Helmut Grohne (3):
jffs2: avoid unnecessarily taking f->sem
jffs2: jffs2_iget defer f->sem acquisition
jffs2: put directories f->sem on a separate lockdep class

 fs/jffs2/fs.c        | 16 +++++++++++++++-
 fs/jffs2/readinode.c |  4 +---
 fs/jffs2/super.c     |  3 +++
 3 files changed, 19 insertions(+), 4 deletions(-)
Signature · 1 line
2.11.0