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

KIYOHARA Takashi <[email protected]>
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
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.