From owner-cvs-src@FreeBSD.ORG Fri Apr 11 09:50:24 2008 Return-Path: Delivered-To: cvs-src@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 887AB1065670 for ; Fri, 11 Apr 2008 09:50:24 +0000 (UTC) (envelope-from jroberson@jroberson.net) Received: from wf-out-1314.google.com (wf-out-1314.google.com [209.85.200.169]) by mx1.freebsd.org (Postfix) with ESMTP id 601A38FC15 for ; Fri, 11 Apr 2008 09:50:24 +0000 (UTC) (envelope-from jroberson@jroberson.net) Received: by wf-out-1314.google.com with SMTP id 25so416621wfa.7 for ; Fri, 11 Apr 2008 02:50:24 -0700 (PDT) Received: by 10.142.157.15 with SMTP id f15mr775235wfe.275.1207907424033; Fri, 11 Apr 2008 02:50:24 -0700 (PDT) Received: from ?10.0.1.199? ( [24.94.72.120]) by mx.google.com with ESMTPS id 32sm5098180wfa.13.2008.04.11.02.50.22 (version=SSLv3 cipher=OTHER); Fri, 11 Apr 2008 02:50:23 -0700 (PDT) Date: Thu, 10 Apr 2008 23:51:31 -1000 (HST) From: Jeff Roberson X-X-Sender: jroberson@desktop To: Marcel Moolenaar In-Reply-To: <9B8DD956-E197-4540-A7F3-367B1E277456@mac.com> Message-ID: <20080410235123.K43186@desktop> References: <200804110326.m3B3QfjL095474@repoman.freebsd.org> <20080410190028.U43186@desktop> <9B8DD956-E197-4540-A7F3-367B1E277456@mac.com> MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII; format=flowed Cc: cvs-src@FreeBSD.org, src-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/amd64/amd64 intr_machdep.c src/sys/arm/arm intr.c src/sys/i386/i386 intr_machdep.c src/sys/ia64/ia64 interrupt.c src/sys/kern kern_cpuset.c kern_intr.c src/sys/powerpc/powerpc intr_machdep.c src/sys/sparc64/sparc64 ... X-BeenThere: cvs-src@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: CVS commit messages for the src tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 11 Apr 2008 09:50:24 -0000 On Thu, 10 Apr 2008, Marcel Moolenaar wrote: > > On Apr 10, 2008, at 10:01 PM, Jeff Roberson wrote: >> On Thu, 10 Apr 2008, Marcel Moolenaar wrote: >> >>> >>> On Apr 10, 2008, at 8:26 PM, Jeff Roberson wrote: >>>> jeff 2008-04-11 03:26:41 UTC >>>> FreeBSD src repository >>>> Modified files: >>>> sys/amd64/amd64 intr_machdep.c >>>> sys/arm/arm intr.c >>>> sys/i386/i386 intr_machdep.c >>>> sys/ia64/ia64 interrupt.c >>>> sys/kern kern_cpuset.c kern_intr.c >>>> sys/powerpc/powerpc intr_machdep.c >>>> sys/sparc64/sparc64 intr_machdep.c >>>> sys/sun4v/sun4v intr_machdep.c >>>> sys/sys cpuset.h interrupt.h >>>> Log: >>>> - Add the interrupt vector number to intr_event_create so MI code can >>>> lookup hard interrupt events by number. Ignore the irq# for soft intrs. >>> >>> JFYI: vectors are not the same as hard interrupt numbers. >>> On ia64 they imply priority, which means that vector >>> numbers should be selected for their priority and not >>> have a 1-to-1 mapping with IRQ number. >>> On powerpc they mean absolutely nothing other than being >>> an index into an array. >>> >>> It's better to use IRQs and not vectors to lookup int. >>> events by number. >> >> I think I mixed my terminology in the commit. I meant for all >> architectures to pass in the number that would appear in vmstat -i. Can >> you verify that this is what I did for powerpc and ia64? > > powerpc correctly uses irq. > ia64 incorrectly uses vector. > > Would you mind changing ia64 to use irq as well? Index: ia64/ia64/interrupt.c =================================================================== RCS file: /home/ncvs/src/sys/ia64/ia64/interrupt.c,v retrieving revision 1.66 diff -r1.66 interrupt.c 343c343 < 0, vector, ia64_intr_mask, ia64_intr_unmask, ia64_intr_eoi, --- > 0, irq, ia64_intr_mask, ia64_intr_unmask, ia64_intr_eoi, Does that look right? > > Thanks, > > -- > Marcel Moolenaar > xcllnt@mac.com >