[PATCH] octeon: false positive timeout
Ralf Baechle
ralf at linux-mips.org
Wed Aug 26 16:09:55 UTC 2009
On Wed, Aug 26, 2009 at 08:45:10AM -0700, David Daney wrote:
> Roel Kluin wrote:
>> If we reach the test just below the loop with a `timeout' value of 0,
>> this does not mean that the timeout caused the loop to end, but rather
>> the `smi_rd.s.pending', in the last iteration. If timeout caused the
>> loop to end, then `timeout' is -1, not 0.
>>
>> Since this can occur only in the last iteration, it is not very likely
>> to be a problem. By changing the post- to prefix decrement we ensure
>> that a timeout of 0 does mean it timed out.
>>
>> Signed-off-by: Roel Kluin <roel.kluin at gmail.com>
>
> Acked-by: David Daney <ddaney at caviumnetworks.com>
Thanks, applied.
Ralf
More information about the devel
mailing list