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
_______________________________________________
freebsd-current@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-current
To unsubscribe, send any mail to "freebsd-current-unsubscr...@freebsd.org"
