Re: [PATCH] [232/275] nfsd: wrong index used in inner loop

From: Michael Tokarev
Date: Wed Mar 30 2011 - 18:04:49 EST


31.03.2011 01:56, Michael Tokarev ÐÐÑÐÑ:
> 31.03.2011 01:07, Andi Kleen wrote:
>> 2.6.35-longterm review patch. If anyone has any objections, please let me know.
>>
>> ------------------
>> From: Mi Jinlong <mijinlong@xxxxxxxxxxxxxx>
>>
>> commit 5a02ab7c3c4580f94d13c683721039855b67cda6 upstream.
>>
>> We must not use dummy for index.
>> After the first index, READ32(dummy) will change dummy!!!!
>
> This is also in patch 180/275 (this one is 232/275).

Doh, so this 232/275 basically reverts that 180 and makes
other change down the line, but keeps the subject intact.
Especially useful that the two patches are so far away
from each other (more than 50 patches in-between).
Confusing :)

So both are actually needed, apparently.

Thanks!

/mjt

--
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/