Date: Wed, 26 Sep 2012 09:54:18 -0700 From: John Polstra <jdp@polstra.com> To: Pawel Jakub Dawidek <pjd@FreeBSD.org> Cc: svn-src-head@FreeBSD.org, svn-src-all@FreeBSD.org, src-committers@FreeBSD.org, "Kenneth D. Merry" <ken@FreeBSD.org>, phk@FreeBSD.org Subject: Re: svn commit: r240822 - head/sys/geom Message-ID: <0E55091C-5960-4961-BFDA-6484304FAB1D@polstra.com> In-Reply-To: <20120926072005.GH1391@garage.freebsd.pl> References: <201209221241.q8MCfnhJ067937@svn.freebsd.org> <20120925233712.GA26920@nargothrond.kdm.org> <20120926072005.GH1391@garage.freebsd.pl>
next in thread | previous in thread | raw e-mail | index | archive | help
On Sep 26, 2012, at 12:20 AM, Pawel Jakub Dawidek wrote: > We might be able to change the topology lock to LIST_FOREACH_SAFE(), as > g_wither_provider() can only destroy current provider. This is because > CAM own the geom and nobody should be able to mess with its provider's > list, apart from CAM itself. So I'd need to know how CAM ensures that > two disk_gone() cannot be called for one geom. The answer might be that > those geoms have always only one provider, but then I cannot explain why > we have loop there. Maybe jdp@ will remember why. No, I'm sorry but I don't remember. It's been a long time! John
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?0E55091C-5960-4961-BFDA-6484304FAB1D>