Re: kernel/timer.c: next_timer_interrupt() strange/buggy(?) code (2.6.18-rc1-mm2)

From: Valdis . Kletnieks
Date: Tue Jul 18 2006 - 10:48:32 EST


On Tue, 18 Jul 2006 16:29:27 +0200, Michael Buesch said:

> Continue is equal to:
>
> LOOP {
> /* foo */
> goto continue; /* == continue */
/* What the code actually had: */
goto found; /* Note placement of the label *AFTER* end of loop */
> /* foo */
> continue:
> } LOOP

found: /* out of the loop entirely */

A 'continue' drops you *at* the end of the loop. The 'goto found:' in the
original code drops you *after* the end of the loop. One will potentially go
around for another pass, the other you're *done*.

Attachment: pgp00000.pgp
Description: PGP signature