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

KIYOHARA Takashi <[email protected]> Thu, 03 Jan 2013 19:59:34 +0900 (JST)
Newsgroups gmane.os.netbsd.ports.bebox,gmane.os.netbsd.ports.powerpc,gmane.os.netbsd.current
Message-ID <[email protected]>
Hi! Frank,


From: Frank Wille <[email protected]>
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