From owner-cvs-src@FreeBSD.ORG Sun May 6 19:44:37 2007 Return-Path: X-Original-To: cvs-src@FreeBSD.org Delivered-To: cvs-src@FreeBSD.org Received: from mx1.freebsd.org (mx1.freebsd.org [69.147.83.52]) by hub.freebsd.org (Postfix) with ESMTP id EE85A16A402; Sun, 6 May 2007 19:44:37 +0000 (UTC) (envelope-from piso@newluxor.wired.org) Received: from mail.oltrelinux.com (krisma.oltrelinux.com [194.242.226.43]) by mx1.freebsd.org (Postfix) with ESMTP id 7AEC713C48C; Sun, 6 May 2007 19:44:37 +0000 (UTC) (envelope-from piso@newluxor.wired.org) Received: from newluxor.wired.org (ip-145-199.sn3.eutelia.it [213.136.145.199]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by mail.oltrelinux.com (Postfix) with ESMTP id DA0F411AE43; Sun, 6 May 2007 21:10:49 +0200 (CEST) Received: (from piso@localhost) by newluxor.wired.org (8.13.8/8.13.8/Submit) id l46JAT21002101; Sun, 6 May 2007 21:10:29 +0200 (CEST) (envelope-from piso) Date: Sun, 6 May 2007 21:10:27 +0200 From: Paolo Pisati To: John-Mark Gurney Message-ID: <20070506191027.GA2050@tin.it> References: <200705061702.l46H2oVK057260@repoman.freebsd.org> <20070506174426.GO17958@funkthat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20070506174426.GO17958@funkthat.com> User-Agent: Mutt/1.4.2.2i X-Virus-Scanned: by amavisd-new-20030616-p10 (Debian) at krisma.oltrelinux.com Cc: cvs-src@FreeBSD.org, src-committers@FreeBSD.org, cvs-all@FreeBSD.org, Paolo Pisati Subject: Re: cvs commit: src/sys/amd64/amd64 intr_machdep.c src/sys/i386/i386 intr_machdep.c src/sys/kern kern_intr.c src/sys/sys interrupt.h 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: Sun, 06 May 2007 19:44:38 -0000 On Sun, May 06, 2007 at 10:44:26AM -0700, John-Mark Gurney wrote: > Paolo Pisati wrote this message on Sun, May 06, 2007 at 17:02 +0000: > > Modified files: > > sys/amd64/amd64 intr_machdep.c > > sys/i386/i386 intr_machdep.c > > sys/kern kern_intr.c > > sys/sys interrupt.h > > Log: > > Bring in the reminaing bits to make interrupt filtering work: > > When are we going to see man page updates to document this new feature? didn't prepare any patch for man pages cause the support for some archs is still missing, but i can do that if requested. > Also, you forgot to update sys/sys/bus.h's comments? or did you fix > the code to behave like the comments? you mean this comment: --------8<--------8<--------8<--------8<--------8<--------8<-------- /** * @brief Driver interrupt filter return values * * If a driver provides an interrupt filter routine it must return an * integer consisting of oring together zero or more of the following * flags: * * FILTER_STRAY - this device did not trigger the interrupt * FILTER_HANDLED - the interrupt has been fully handled and can be EOId * FILTER_SCHEDULE_THREAD - the threaded interrupt handler should be * scheduled to execute * * If the driver does not provide a filter, then the interrupt code will * act is if the filter had returned FILTER_SCHEDULE_THREAD. Note that it * is illegal to specify any other flag with FILTER_STRAY and that it is * illegal to not specify either of FILTER_HANDLED or FILTER_SCHEDULE_THREAD * if FILTER_STRAY is not specified. */ #define FILTER_STRAY 0x01 #define FILTER_HANDLED 0x02 #define FILTER_SCHEDULE_THREAD 0x04 --------8<--------8<--------8<--------8<--------8<--------8<-------- except for the horrible english, the semantic is correct. BTW, kris@ just told me that sun4v is broken since my first intr filtering commit: can we sat down and try to fix it or should we consider it officialy dead? bye, P.