Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 10 Aug 2023 08:35:16 GMT
From:      Martin Matuska <mm@FreeBSD.org>
To:        src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-main@FreeBSD.org
Subject:   git: 28d2e3b5dedf - main - zfs: cherry-pick fix from openzfs
Message-ID:  <202308100835.37A8ZGqx042588@gitrepo.freebsd.org>

next in thread | raw e-mail | index | archive | help
The branch main has been updated by mm:

URL: https://cgit.FreeBSD.org/src/commit/?id=28d2e3b5dedf24938e5ec0806084941510621f13

commit 28d2e3b5dedf24938e5ec0806084941510621f13
Author:     Martin Matuska <mm@FreeBSD.org>
AuthorDate: 2023-08-10 07:59:39 +0000
Commit:     Martin Matuska <mm@FreeBSD.org>
CommitDate: 2023-08-10 08:11:52 +0000

    zfs: cherry-pick fix from openzfs
    
    Vendor PR:
      #15103 Remove zl_issuer_lock from zil_suspend()
    
    Obtained from:  OpenZFS
    OpenZFS commit: 2848de11e516a2ef2e6baa574a60d77a6fb47023
    
    Note: full vendor imports will continue when stable/14 has been branched
---
 sys/contrib/openzfs/module/zfs/zil.c | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/sys/contrib/openzfs/module/zfs/zil.c b/sys/contrib/openzfs/module/zfs/zil.c
index af7137faaccf..be5b9edf6ede 100644
--- a/sys/contrib/openzfs/module/zfs/zil.c
+++ b/sys/contrib/openzfs/module/zfs/zil.c
@@ -3941,13 +3941,11 @@ zil_suspend(const char *osname, void **cookiep)
 		return (error);
 	zilog = dmu_objset_zil(os);
 
-	mutex_enter(&zilog->zl_issuer_lock);
 	mutex_enter(&zilog->zl_lock);
 	zh = zilog->zl_header;
 
 	if (zh->zh_flags & ZIL_REPLAY_NEEDED) {		/* unplayed log */
 		mutex_exit(&zilog->zl_lock);
-		mutex_exit(&zilog->zl_issuer_lock);
 		dmu_objset_rele(os, suspend_tag);
 		return (SET_ERROR(EBUSY));
 	}
@@ -3961,7 +3959,6 @@ zil_suspend(const char *osname, void **cookiep)
 	if (cookiep == NULL && !zilog->zl_suspending &&
 	    (zilog->zl_suspend > 0 || BP_IS_HOLE(&zh->zh_log))) {
 		mutex_exit(&zilog->zl_lock);
-		mutex_exit(&zilog->zl_issuer_lock);
 		dmu_objset_rele(os, suspend_tag);
 		return (0);
 	}
@@ -3970,7 +3967,6 @@ zil_suspend(const char *osname, void **cookiep)
 	dsl_pool_rele(dmu_objset_pool(os), suspend_tag);
 
 	zilog->zl_suspend++;
-	mutex_exit(&zilog->zl_issuer_lock);
 
 	if (zilog->zl_suspend > 1) {
 		/*



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?202308100835.37A8ZGqx042588>