Skip site navigation (1)Skip section navigation (2)
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>