On Wednesday 15 July 2009 6:49:35 am Roman Divacky wrote:
quoted text
> hi
> 
> the isa_dma* functions seem to require Giant. ppc(4) uses those
> but does not seem to lock Giant or anything. is this ok?
> is it a bug?

Doh, that is a bug.

quoted text
> I looked at the i386/isa/isa_dma.c and it seems pretty selfcontained
> (ie. it operates on hw and it's own variables). would it be sufficient
> to just introduce a new lock and lock with it all the isa_dma* functions
> ? or is there some indirect dependency on Giant that I cant see?

That would probably work.  If it can be self-contained in the isa_dma API that 
would so that consumers do not need fixing that would be ideal.

-- 
John Baldwin
_______________________________________________
freebsd-current@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-current
To unsubscribe, send any mail to "freebsd-current-unsubscribe@freebsd.org"