Dears,
The check line still included in iso_stream_schedule(). in linux
2.6.39. I check the code from linux archive
[linux/kernel/git/stable/linux-2.6.39.y.git] / drivers / usb / host /
ehci-sched.c, line 1488 ~1496
===============================================
1488 /* Tried to schedule too far into the future? */
1489 if (unlikely(start - now + span - period
1490 >= mod - 2 * SCHEDULE_SLOP)) {
1491 ehci_dbg(ehci, "request %p would overflow (%d+%d >= %d)\n",
1492 urb, start - now, span - period,
1493 mod - 2 * SCHEDULE_SLOP);
1494 status = -EFBIG;
1495 goto fail;
1496 }
================================================
and
http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-2.6.git;a=blob_plain;f=drivers/usb/host/ehci-sched.c;hb=4a7df24ddc7097cd789a7527187826552ea0765e
==================================================
1492 /* Tried to schedule too far into the future? */
1493 if (unlikely(start - now + span - period
1494 >= mod - 2 * SCHEDULE_SLOP)) {
1495 ehci_dbg(ehci, "request %p would overflow (%d+%d >= %d)\n",
1496 urb, start - now, span - period,
1497 mod - 2 * SCHEDULE_SLOP);
1498 status = -EFBIG;
1499 goto fail;
1500 }
==================================================
The check line will get error "EFBIG" for urb_sumbit.
Could you kindly help confirm?
Thanks a lot!
Best Regards,
Soho
2011/5/25 Alan Stern <stern@xxxxxxxxxxxxxxxxxxx>:
> On Wed, 25 May 2011, Soho Soho123 wrote:
>
>> Dears,
>>
>> we have apply patch from new kernel for our 2.6.34 kernel:
>> the list of patch we apply:
>> USB: EHCI: remove PCI assumption(as1405)
>> USB: EHCI: simplify remainder computations(as1406)
>> USB: EHCI: add missing frame -> microframe conversion(as1407)
>> USB: EHCI: reorganize isochronous scheduler routine(as1408)
>> USB: EHCI: remove dead code in the periodic scheduler(as1409)
>>
>> the result is identical.
>
> It can't be identical, because one of those patches (as1408) removes
> the line where you found the error was occurring.
>
>> is it necessary to update uvc driver ?
>> or
>> how about the error handliing for uvs driver?
>
> I don't know.
>
> Alan Stern
>
>
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
