Re: [PATCH 0/3] staging: dwc2: 2nd try at uframe scheduler patch

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel




[Index of Archives]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux