[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <5a4c581d0802111531qf6fe2d0tf4878caefef958d3@mail.gmail.com>
Date: Tue, 12 Feb 2008 00:31:21 +0100
From: "Alessandro Suardi" <alessandro.suardi@...il.com>
To: "Rafael J. Wysocki" <rjw@...k.pl>
Cc: "Ingo Molnar" <mingo@...e.hu>,
"Linus Torvalds" <torvalds@...ux-foundation.org>,
LKML <linux-kernel@...r.kernel.org>,
"Andrew Morton" <akpm@...ux-foundation.org>,
"Peter Zijlstra" <a.p.zijlstra@...llo.nl>
Subject: Re: 2.6.24-git2: Oracle 11g VKTM process enters R state on startup and is unkillable [still broken in 2.6.25-rc1]
On Feb 12, 2008 12:12 AM, Rafael J. Wysocki <rjw@...k.pl> wrote:
> On Monday, 11 of February 2008, Ingo Molnar wrote:
> >
> > * Ingo Molnar <mingo@...e.hu> wrote:
> >
> > > no, they were not lost, they just didnt pass QA here (they crashed on
> > > a particularly hard to debug 8-way box i have) and Peter worked on
> > > that queue of fixes up until today to get it really correct. Could you
> > > check:
> > >
> > > git://git.kernel.org/pub/scm/linux/kernel/git/mingo/linux-2.6-sched.git
> > >
> > > combo patch below as well - whichever you prefer. The shortlog can be
> > > found below as well - but i dont yet consider this pullable, i'd like
> > > it to see pass a full night of randconfig tests on my test-systems.
> >
> > ok, we just found the reason for the 8-way crash, the delta fix from
> > Peter is below if any of you have tried the previous combo patch.
> > Updated sched.git as well, new HEAD is
> > fec13e45305d69fd0bd23b30bd05a0a42cf341f8.
>
> With the previous patch and this patch applied, the issue is not reproducible
> here.
>
> Thanks,
> Rafael
The problem is fixed for me as well with the previous patch + the patch
below, VKTM now enters S state and Oracle shuts down properly again.
Thanks !
> > Index: linux-2.6/kernel/sched.c
> > ===================================================================
> > --- linux-2.6.orig/kernel/sched.c
> > +++ linux-2.6/kernel/sched.c
> > @@ -219,6 +219,10 @@ static void start_rt_bandwidth(struct rt
> > if (rt_b->rt_runtime == RUNTIME_INF)
> > return;
> >
> > + if (hrtimer_active(&rt_b->rt_period_timer))
> > + return;
> > +
> > + spin_lock(&rt_b->rt_runtime_lock);
> > for (;;) {
> > if (hrtimer_active(&rt_b->rt_period_timer))
> > break;
> > @@ -229,6 +233,7 @@ static void start_rt_bandwidth(struct rt
> > rt_b->rt_period_timer.expires,
> > HRTIMER_MODE_ABS);
> > }
> > + spin_unlock(&rt_b->rt_runtime_lock);
> > }
> >
> > #ifdef CONFIG_RT_GROUP_SCHED
--alessandro
"We act as though comfort and luxury were the chief requirements
of life, when all that we need to make us really happy is
something to be enthusiastic about."
(Charles Kingsley)
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists