* rtmutex's wait_lock in 2.6.18-rt7
@ 2006-10-23 9:55 Esben Nielsen
2006-10-24 9:46 ` Thomas Gleixner
0 siblings, 1 reply; 3+ messages in thread
From: Esben Nielsen @ 2006-10-23 9:55 UTC (permalink / raw)
To: Ingo Molnar; +Cc: linux-kernel
Hi,
I see that in 2.6.18-rt7 the rtmutex's wait_lock is sudden interrupt
disabling. I don't see the need as no (hard) interrupt-handlers should be
touching any mutex.
Esben
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: rtmutex's wait_lock in 2.6.18-rt7
2006-10-23 9:55 rtmutex's wait_lock in 2.6.18-rt7 Esben Nielsen
@ 2006-10-24 9:46 ` Thomas Gleixner
2006-10-24 12:19 ` Esben Nielsen
0 siblings, 1 reply; 3+ messages in thread
From: Thomas Gleixner @ 2006-10-24 9:46 UTC (permalink / raw)
To: Esben Nielsen; +Cc: Ingo Molnar, linux-kernel
On Mon, 2006-10-23 at 11:55 +0200, Esben Nielsen wrote:
> Hi,
> I see that in 2.6.18-rt7 the rtmutex's wait_lock is sudden interrupt
> disabling. I don't see the need as no (hard) interrupt-handlers should be
> touching any mutex.
It does not touch mutexes, but the dynamic priority adjustment of the
hrtimer softirq needs it.
The correct solution will be moving the timer callback into the process
context, as it will be woken up anyway, but that's more complex to do
than it looks in the first place.
tglx
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: rtmutex's wait_lock in 2.6.18-rt7
2006-10-24 9:46 ` Thomas Gleixner
@ 2006-10-24 12:19 ` Esben Nielsen
0 siblings, 0 replies; 3+ messages in thread
From: Esben Nielsen @ 2006-10-24 12:19 UTC (permalink / raw)
To: Thomas Gleixner; +Cc: Esben Nielsen, Ingo Molnar, linux-kernel
On Tue, 24 Oct 2006, Thomas Gleixner wrote:
> On Mon, 2006-10-23 at 11:55 +0200, Esben Nielsen wrote:
>> Hi,
>> I see that in 2.6.18-rt7 the rtmutex's wait_lock is sudden interrupt
>> disabling. I don't see the need as no (hard) interrupt-handlers should be
>> touching any mutex.
>
> It does not touch mutexes, but the dynamic priority adjustment of the
> hrtimer softirq needs it.
>
> The correct solution will be moving the timer callback into the process
> context, as it will be woken up anyway, but that's more complex to do
> than it looks in the first place.
>
I have send out patches doing the correct priority adjustment without
touching the wait_lock. Why not use that?
I found it in the archives:
http://www.uwsg.iu.edu/hypermail/linux/kernel/0610.0/0049.html
(or more specific in
http://www.uwsg.iu.edu/hypermail/linux/kernel/0610.0/0051.html, look for
changes to sched.c)
It is very bad to do PI traversal in interrupt context. In the general
case, where there are user-space locks, that operation unbounded. I
know that in your case you can only traverse kernel locks, but I think it
is bad to open for such posibilities if it can be avoided.
Esben
> tglx
>
>
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2006-10-24 12:19 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2006-10-23 9:55 rtmutex's wait_lock in 2.6.18-rt7 Esben Nielsen
2006-10-24 9:46 ` Thomas Gleixner
2006-10-24 12:19 ` Esben Nielsen
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®