Re: [PATCH -rt 4/9] ifdef raise_softirq_irqoff wakeup

[Ingo Molnar]

* Daniel Walker <[EMAIL PROTECTED]> wrote:

> On Mon, 2007-07-30 at 11:27 +0200, Ingo Molnar wrote:
> > * Daniel Walker <[EMAIL PROTECTED]> wrote:
> > 
> > > @@ -508,7 +508,9 @@ inline fastcall void raise_softirq_irqof
> > >  {
> > >   __do_raise_softirq_irqoff(nr);
> > >  
> > > +#ifdef CONFIG_PREEMPT_SOFTIRQS
> > >   wakeup_softirqd(nr);
> > > +#endif
> > 
> > thanks, applied. People rarely run the -rt kernel just to turn off 
> > PREEMPT_RT, that's why this bug was there ;-)
> 
> Ultimately, all modes should function correctly right?

yes, of course - especially once any of the components nears upstream 
integration ;-)

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

Re: [PATCH -rt 4/9] ifdef raise_softirq_irqoff wakeup

[Ingo Molnar]

* Daniel Walker [EMAIL PROTECTED] wrote:

 On Mon, 2007-07-30 at 11:27 +0200, Ingo Molnar wrote:
  * Daniel Walker [EMAIL PROTECTED] wrote:
  
   @@ -508,7 +508,9 @@ inline fastcall void raise_softirq_irqof
{
 __do_raise_softirq_irqoff(nr);

   +#ifdef CONFIG_PREEMPT_SOFTIRQS
 wakeup_softirqd(nr);
   +#endif
  
  thanks, applied. People rarely run the -rt kernel just to turn off 
  PREEMPT_RT, that's why this bug was there ;-)
 
 Ultimately, all modes should function correctly right?

yes, of course - especially once any of the components nears upstream 
integration ;-)

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

Re: [PATCH -rt 4/9] ifdef raise_softirq_irqoff wakeup

[Daniel Walker]

On Mon, 2007-07-30 at 11:27 +0200, Ingo Molnar wrote:
> * Daniel Walker <[EMAIL PROTECTED]> wrote:
> 
> > @@ -508,7 +508,9 @@ inline fastcall void raise_softirq_irqof
> >  {
> > __do_raise_softirq_irqoff(nr);
> >  
> > +#ifdef CONFIG_PREEMPT_SOFTIRQS
> > wakeup_softirqd(nr);
> > +#endif
> 
> thanks, applied. People rarely run the -rt kernel just to turn off 
> PREEMPT_RT, that's why this bug was there ;-)

Ultimately, all modes should function correctly right?

Daniel

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

Re: [PATCH -rt 4/9] ifdef raise_softirq_irqoff wakeup

[Ingo Molnar]

* Daniel Walker <[EMAIL PROTECTED]> wrote:

> @@ -508,7 +508,9 @@ inline fastcall void raise_softirq_irqof
>  {
>   __do_raise_softirq_irqoff(nr);
>  
> +#ifdef CONFIG_PREEMPT_SOFTIRQS
>   wakeup_softirqd(nr);
> +#endif

thanks, applied. People rarely run the -rt kernel just to turn off 
PREEMPT_RT, that's why this bug was there ;-)

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

Re: [PATCH -rt 4/9] ifdef raise_softirq_irqoff wakeup

[Ingo Molnar]

* Daniel Walker [EMAIL PROTECTED] wrote:

 @@ -508,7 +508,9 @@ inline fastcall void raise_softirq_irqof
  {
   __do_raise_softirq_irqoff(nr);
  
 +#ifdef CONFIG_PREEMPT_SOFTIRQS
   wakeup_softirqd(nr);
 +#endif

thanks, applied. People rarely run the -rt kernel just to turn off 
PREEMPT_RT, that's why this bug was there ;-)

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

Re: [PATCH -rt 4/9] ifdef raise_softirq_irqoff wakeup

[Daniel Walker]

On Mon, 2007-07-30 at 11:27 +0200, Ingo Molnar wrote:
 * Daniel Walker [EMAIL PROTECTED] wrote:
 
  @@ -508,7 +508,9 @@ inline fastcall void raise_softirq_irqof
   {
  __do_raise_softirq_irqoff(nr);
   
  +#ifdef CONFIG_PREEMPT_SOFTIRQS
  wakeup_softirqd(nr);
  +#endif
 
 thanks, applied. People rarely run the -rt kernel just to turn off 
 PREEMPT_RT, that's why this bug was there ;-)

Ultimately, all modes should function correctly right?

Daniel

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