lkml.org 
[lkml]   [2002]   [Jun]   [18]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRe: [PATCH] Replace timer_bh with tasklet
Date
Hello!

> > But this is impossible, timers must not race with another BHs,
> > all the code using BHs depends on this. That's why they are BHs.
>
> If indeed they do "race" the old code had the timer_bh being first.
> So does this patch.

I do not understand what you mean here.

I feel you misunderstand my comment. I said the patch is one pure big bug,
because tasklets are not serialized wrt BHs. Timer MUST. If you are going
to get rid of this must, start from editing all the code which makes this
assumption.


> This is one of the most hard to control paths in the system as ALL it
> does is execute functions that are unknown as to size, duration, etc.
> One would hope that they never run for long, but...

Pardon, your code has an effect only after this "but..." happens
and this effect is insane.


> Not really. One REALLY expects timers to expire in timed order :) Using
> a separate procedure to deliver a timer just because it is of a different
> resolution opens one up to a world of pathology.

Are you going to mix use of hires and lores timers for one task?

Alexey
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2005-03-22 13:26    [W:2.275 / U:0.100 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and my Meterkast|Read the blog