Hi Paul,

> Matthijs' concern about periodic endpoints with bInterval=1 seems
> to be unfounded. I tried a webcam, which uses a bInterval=1 isoc
> endpoint, on my PCI-based dev board, and it still works fine with
> this patch applied.
For the record, I still think this concern actually exists, but you will
need to completely fill up the available bandwidth to notice the
problem, since the scheduler is now too optimistic about how much
bandwidth is availble when devices with bInterval < 8 are present (but
pessimistic for devices with bInterval > 8).

However, this is something I think we can easily fix later, so getting
this series in is a step forward anyway.

I haven't got time to properly review and test these patches now, since
I'm about to leave for a two-week vacation though. I did a quick scan
just now and replied with some comments that occured to me.

Gr.

Matthijs
_______________________________________________
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Reply via email to