PR/52301 CVS commit: src/sys/ufs/lfs

["Maya Rashish" <maya%netbsd.org@localhost>]

The following reply was made to PR kern/52301; it has been noted by GNATS.

From: "Maya Rashish" <maya%netbsd.org@localhost>
To: gnats-bugs%gnats.NetBSD.org@localhost
Cc: 
Subject: PR/52301 CVS commit: src/sys/ufs/lfs
Date: Thu, 15 Jun 2017 14:37:31 +0000

 Module Name:	src
 Committed By:	maya
 Date:		Thu Jun 15 14:37:31 UTC 2017
 
 Modified Files:
 	src/sys/ufs/lfs: lfs_segment.c
 
 Log Message:
 It isn't safe to drain dirops with seglock held, it'll deadlock if there
 are any dirops. drain before grabbing seglock.
 
 lfs_dirops == 0 is always true (as we already drained dirops), so omit
 that part of the comparison.
 
 Fixes a lot of LFS deadlocks. PR kern/52301
 
 Many thanks to dholland for help analyzing coredumps
 
 
 To generate a diff of this commit:
 cvs rdiff -u -r1.271 -r1.272 src/sys/ufs/lfs/lfs_segment.c
 
 Please note that diffs are not public domain; they are subject to the
 copyright notices on the relevant files.