Re: [GIT PATCH] USB fixes for 2.6.26-rc4

[Leonid]

Hi Alan,

Not to be overly pedantic, but you weren't so wrong to be confused on
the issue of "regression".  Here's the scoop:

1) Kernel versions <= 2.6.24: latency of 2/6/2/6/... uframes pattern
observed between packets on the bus for nVIDIA HCs.

2) Kernel versions >= 2.6.26: latency of 8/8/8/8/... uframes pattern
observed between packets on the bus for nVIDIA HCs.

3) Post-patch: latency of 1/1/1/1/... uframes on all kernels versions
and all HC chipsets.

So basically there was a peripheral issue in the EHCI driver between
2.6.24 and 2.6.25 that exacerbated the problem.  But the underlying
issue was there long before 2.6.25.  Dave is quite right that this
isn't technically a "regression."

The issue itself was a bug (as opposed to a "feature" ;-) ) since the
original comments in the code clearly had the right intention, but the
code itself did not execute as intended in certain situations.  In
fact, thinking about it now, this issue could even cause significant
performance degradation on nVIDIA HCs if one has multiple devices
using the bus at any given time.  Given this, it might be best to
label the patch "USB: EHCI: fix performance issue" instead of using
the word regression.  But regardless, we are just worrying about
symmantics at this point! :)

Hope that helps clarify and put to rest this titling issue once in for all.

-Leonid


On Thu, May 29, 2008 at 6:44 PM, Alan Stern <stern@xxxxxxxxxxxxxxxxxxx> wrote:
> On Thu, 29 May 2008, Greg KH wrote:
>> Well, I was misinformed by Alan Stern about this then, sorry for missing
>> that you didn't ack this one (you acked the other patches in this series
>> though.)
>
> The notion that this was a regression came about by mistake.  It was
> stated that 2.6.24 worked better than 2.6.25; apparently that remark
> was wrong.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/