Date: Mon, 24 Aug 2015 13:29:03 +0300 From: Andriy Gapon <avg@FreeBSD.org> To: Fabian Keil <freebsd-listen@fabiankeil.de> Cc: "svn-src-head@freebsd.org" <svn-src-head@FreeBSD.org> Subject: Re: svn commit: r285021 - head/sys/cddl/contrib/opensolaris/uts/common/fs/zfs Message-ID: <55DAF1EF.2080900@FreeBSD.org> In-Reply-To: <25a69eb6.5dfe13af@fabiankeil.de> References: <201507020832.t628W3WJ002944@repo.freebsd.org> <CAHM0Q_OcLqT_VuxmL4tFrDyEhAEprm_ivP8j_mrow6ZG1FLcsg@mail.gmail.com> <55BF6A84.1040808@FreeBSD.org> <25a69eb6.5dfe13af@fabiankeil.de>
next in thread | previous in thread | raw e-mail | index | archive | help
On 03/08/2015 16:29, Fabian Keil wrote: > Andriy Gapon <avg@FreeBSD.org> wrote: > >> On 30/07/2015 10:24, K. Macy wrote: >>> Just FYI this change introduces a deadlock with with the >>> spa_namespace_lock. Mount will be holding this lock while trying to >>> acquire the spa_namespace_lock. zfskern on the other hand holds the >>> spa_namespace_lock when calling zfs_freebsd_access which in turn >>> tries to acquire the teardown lock. >> >> I missed the fact that zpool.cache file is being written with spa_namespace_lock >> held. >> I'll try to either resolve the problem in the next day or I will revert the change. > > BTW, there's another (unrelated) spa_namespace_lock issue in spa_import_rootpool(): > https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=198563 Committed. Thank you! -- Andriy Gapon
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?55DAF1EF.2080900>