Messages in this thread | | | Date | Fri, 22 May 2009 16:36:25 +0200 | From | Stanislaw Gruszka <> | Subject | Re: [PATCH resend4 3/3] itimers: simplify arm_timer() code a bit |
| |
On Fri, 22 May 2009 16:30:09 +0200 (CEST) Thomas Gleixner <tglx@linutronix.de> wrote:
> On Fri, 22 May 2009, Stanislaw Gruszka wrote: > > case CPUCLOCK_PROF: > > - if (cputime_eq(p->cputime_expires.prof_exp, > > - cputime_zero) || > > - cputime_gt(p->cputime_expires.prof_exp, > > - nt->expires.cpu)) > > - p->cputime_expires.prof_exp = > > - nt->expires.cpu; > > + if (expires_le(p->cputime_expires.prof_exp, > > + exp->cpu)) > > + break; > > Why the reverse logic and the extra break ?
Just to use helper and to make similar logic in all cases in this funcion.
Thanks Stanislaw
| |