Re: minor GEOM disk API change coming

From: Alexander Motin <mav_at_FreeBSD.org>
Date: Fri, 22 Jun 2012 19:50:01 +0300
Hi.

I understand problem you are going to fix and I think your patch should 
do it. What I don't very like is addition of new GEOM method. Now GEOM 
doesn't need it because all internal open/close operations and provider 
destructions there protected by the topology SX lock. Unluckily that 
lock doesn't cover g_wither_provider(), called by disk_gone() while 
holding CAM SIM lock. If not that SIM lock, it would be enough to just 
grab and drop GEOM topology lock to ensure that no new open() calls will 
follow. Indirect way to do it could be to post GEOM event that would 
drop the reference as soon as it will be handled and can obtain the 
topology lock. Unluckily it uses malloc() for event storage and also can 
be unreliable if called from under the SIM mutex lock. So it seems many 
things would be much easier if it was possible to drop SIM lock inside 
periph invalidate method, but now it is unsafe

That is not an objection, just some thoughts about.

-- 
Alexander Motin
Received on Fri Jun 22 2012 - 14:50:06 UTC

This archive was generated by hypermail 2.4.0 : Wed May 19 2021 - 11:40:28 UTC