svn commit: r206497 - in head: sbin/geom/class sbin/geom/class/sched sys/geom/sched sys/modules/geom sys/modules/geom/geom_sched sys/modules/geom/geom_sched/gs_sched sys/modules/geom/geom_sched/gsc...

[Luigi Rizzo]

On Wed, Apr 14, 2010 at 11:01:17PM +0200, Pawel Jakub Dawidek wrote:
> On Wed, Apr 14, 2010 at 06:05:54PM +0200, Fabio Checconi wrote:
> >   - Some classes (from a quick look I've seen only g_mirror, but I'm
> >     pretty sure there were some other ones) cache pointers to their
> >     providers.  With these classes this implementation does not work.
> 
> This can be easly changed to go though geom to provider. Many of my
> classes do that.

Absolutely. The above was just to explain why we only implemented
insert/remove within geom_sched instead of globally. If/when we
make the feature globally available, we also need to inspect all
classes to make sure that there are no surprises.

cheers
luigi

> 
> -- 
> Pawel Jakub Dawidek                       http://www.wheelsystems.com
> pjd at FreeBSD.org                           http://www.FreeBSD.org
> FreeBSD committer                         Am I Evil? Yes, I Am!