* Dave Airlie <[EMAIL PROTECTED]> wrote:

> I can't find the lock from a quick code inspection.. I'll add the
> reschedule to a test drm and I'll build my kernel with all the
> debugging on...

there doesnt seem to be any further lock other than the BKL:

 00000001 0.558ms (+0.000ms): delay_tsc (__delay)
 00000001 0.560ms (+0.001ms): __const_udelay (radeon_do_wait_for_fifo)
 00000001 0.560ms (+0.000ms): __delay (radeon_do_wait_for_fifo)
 00000001 0.560ms (+0.000ms): delay_tsc (__delay)
 00000001 0.561ms (+0.001ms): __const_udelay (radeon_do_wait_for_fifo)
 00000001 0.561ms (+0.000ms): __delay (radeon_do_wait_for_fifo)
 00000001 0.561ms (+0.000ms): delay_tsc (__delay)

so unless the BKL is being relied on, it's safe to add the conditional
reschedule.

        Ingo


-------------------------------------------------------
SF.Net email is sponsored by Shop4tech.com-Lowest price on Blank Media
100pk Sonic DVD-R 4x for only $29 -100pk Sonic DVD+R for only $33
Save 50% off Retail on Ink & Toner - Free Shipping and Free Gift.
http://www.shop4tech.com/z/Inkjet_Cartridges/9_108_r285
--
_______________________________________________
Dri-devel mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/dri-devel

Reply via email to