Re: powerpc/pic/intr.c IPL_HIGH vs IPL_NONE

[KIYOHARA Takashi <kiyohara%kk.iij4u.or.jp@localhost>]

Hi! Frank,


From: Frank Wille <frank%phoenix.owl.de@localhost>
Subject: Re: powerpc/pic/intr.c IPL_HIGH vs IPL_NONE
Date: Tue, 01 Jan 2013 21:11:31 +0100

> KIYOHARA Takashi wrote:
> 
> >> Can you compile the kernel with PIC_DEBUG, to get some more
> >> information about the irqs, virqs and masks in your system?
> >
> > The attached ipl.log is after r1.23:
> >
> >   ih->ih_ipl = ipl;
> >
> > And attached maxipl.log is before r1.22:
> >
> >   ih->ih_ipl = maxipl;
> 
> You probably mean "is->is_ipl = maxipl"? I'm sure this assignment is
> correct.

I very confused this problem.  X-<
I apologize truly.
Change of r1.23 was not related to ih->ih_ipl.
I will commit this change.

Index: intr.c
===================================================================
RCS file: /cvsroot/src/sys/arch/powerpc/pic/intr.c,v
retrieving revision 1.23
diff -u -r1.23 intr.c
--- intr.c      31 Aug 2012 13:12:52 -0000      1.23
+++ intr.c      3 Jan 2013 10:57:03 -0000
@@ -213,7 +213,7 @@
        ih->ih_fun = ih_fun;
        ih->ih_arg = ih_arg;
        ih->ih_next = NULL;
-       ih->ih_ipl = ipl;
+       ih->ih_ipl = maxipl;
        ih->ih_virq = virq;
        *p = ih;
 

Thanks,
--
kiyohara