The branch stable/13 has been updated by kib: URL: https://cgit.FreeBSD.org/src/commit/?id=773d9d7e40dbd1d37517ba20d983fc7d31f5cc8b
commit 773d9d7e40dbd1d37517ba20d983fc7d31f5cc8b Author: Konstantin Belousov <[email protected]> AuthorDate: 2022-01-23 00:33:18 +0000 Commit: Konstantin Belousov <[email protected]> CommitDate: 2022-02-07 09:38:50 +0000 ffs_snapblkfree(): add a comment explaining lockmgr invocation (cherry picked from commit 4559700a0a99376640a8913d08bdf0cfb0124d4a) --- sys/ufs/ffs/ffs_snapshot.c | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/sys/ufs/ffs/ffs_snapshot.c b/sys/ufs/ffs/ffs_snapshot.c index a0d16d9eccef..cb5c4f5e2481 100644 --- a/sys/ufs/ffs/ffs_snapshot.c +++ b/sys/ufs/ffs/ffs_snapshot.c @@ -1807,9 +1807,15 @@ retry: VI_UNLOCK(devvp); return (0); } + + /* + * Use LK_SLEEPFAIL because sn might be freed under us while + * both devvp interlock and snaplk are not owned. + */ if (lockmgr(&sn->sn_lock, LK_INTERLOCK | LK_EXCLUSIVE | LK_SLEEPFAIL, VI_MTX(devvp)) != 0) goto retry; + TAILQ_FOREACH(ip, &sn->sn_head, i_nextsnap) { vp = ITOV(ip); if (DOINGSOFTDEP(vp))
