From: george anzinger <george@mvista.com>
To: Andrew Morton <akpm@digeo.com>
Cc: tim@physik3.uni-rostock.de, linux-kernel@vger.kernel.org
Subject: Re: [PATCH] fix nanosleep() granularity bumps
Date: Wed, 19 Mar 2003 10:08:38 -0800 [thread overview]
Message-ID: <3E78B226.6050908@mvista.com> (raw)
In-Reply-To: <20030319014230.3412298e.akpm@digeo.com>
Andrew Morton wrote:
> george anzinger <george@mvista.com> wrote:
>
>>The attached patch is for 2.5.65. As of this moment, the bk patch has
>>not been posted to the snapshots directory. I will wait for that to
>>update.
>
>
> Don't use the snapshots directory. Use
>
> http://www.kernel.org/pub/linux/kernel/v2.5/testing/cset/
but then I need to pull each patch ... :)
-g
>
> It is more up to date.
>
>
>>For what its worth, can someone explain how the add_timer call from
>>run_timers was causing a problem. The code looks right to me, unless
>>the caller is so nasty as to continue to do the same thing (which
>>would loop forever).
>
>
> That was the problem.
>
>
>>In this case, the simple fix is to bump the
>>base->timer_jiffies at the beginning of the loop rather than the end.
>> This would cause the new timer to be put in the next jiffie instead
>>of the current one AND it is free!
>
>
> Didn't think of that - I just ported up Andrea's fix.
>
>
>
--
George Anzinger george@mvista.com
High-res-timers: http://sourceforge.net/projects/high-res-timers/
Preemption patch: http://www.kernel.org/pub/linux/kernel/people/rml
next prev parent reply other threads:[~2003-03-19 17:58 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <Pine.LNX.4.33.0303181251130.28123-100000@gans.physik3.uni-rostock.de>
2003-03-18 20:26 ` [PATCH] fix nanosleep() granularity bumps Tim Schmielau
2003-03-19 2:08 ` george anzinger
2003-03-19 4:31 ` Andrew Morton
2003-03-19 2:37 ` george anzinger
2003-03-19 2:59 ` Andrew Morton
2003-03-19 7:51 ` Tim Schmielau
2003-03-19 8:35 ` george anzinger
2003-03-19 9:28 ` george anzinger
2003-03-19 9:40 ` Tim Schmielau
2003-03-19 21:30 ` george anzinger
[not found] ` <20030319155258.64cbc43d.akpm@digeo.com>
2003-03-19 22:25 ` [PATCH] Remove defered timer list in favor of moving the list time update george anzinger
2003-03-20 7:36 ` [PATCH] fix nanosleep() granularity bumps Tim Schmielau
2003-03-19 9:42 ` Andrew Morton
2003-03-19 18:08 ` george anzinger [this message]
2003-03-19 18:51 ` Andrew Morton
2003-03-19 20:29 ` george anzinger
2003-03-17 19:42 [BUG & WORKAROUND] nanosleep() granularity bumps up in 2.5.64 Tim Schmielau
2003-03-18 9:05 ` [PATCH] fix nanosleep() granularity bumps Tim Schmielau
2003-03-24 12:06 ` Finn Arne Gangstad
2003-03-24 12:10 ` Tim Schmielau
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=3E78B226.6050908@mvista.com \
--to=george@mvista.com \
--cc=akpm@digeo.com \
--cc=linux-kernel@vger.kernel.org \
--cc=tim@physik3.uni-rostock.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox