From owner-cvs-src@FreeBSD.ORG Sat Sep 20 23:11:55 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 88B8E1065670; Sat, 20 Sep 2008 23:11:55 +0000 (UTC) (envelope-from marius@alchemy.franken.de) Received: from alchemy.franken.de (alchemy.franken.de [194.94.249.214]) by mx1.freebsd.org (Postfix) with ESMTP id EDB828FC1A; Sat, 20 Sep 2008 23:11:54 +0000 (UTC) (envelope-from marius@alchemy.franken.de) Received: from alchemy.franken.de (localhost [127.0.0.1]) by alchemy.franken.de (8.14.3/8.14.3/ALCHEMY.FRANKEN.DE) with ESMTP id m8KNBrpe008984; Sun, 21 Sep 2008 01:11:53 +0200 (CEST) (envelope-from marius@alchemy.franken.de) Received: (from marius@localhost) by alchemy.franken.de (8.14.3/8.14.3/Submit) id m8KNBrTk008983; Sun, 21 Sep 2008 01:11:53 +0200 (CEST) (envelope-from marius) Date: Sun, 21 Sep 2008 01:11:53 +0200 From: Marius Strobl To: Marcel Moolenaar Message-ID: <20080920231152.GA67442@alchemy.franken.de> References: <200809181356.m8IDuaxT089888@repoman.freebsd.org> <200809181027.51997.jhb@freebsd.org> <20080918191947.GX94638@alchemy.franken.de> <89B9A8BE-05F2-4DB2-B7B2-AB240AA9F0DD@mac.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <89B9A8BE-05F2-4DB2-B7B2-AB240AA9F0DD@mac.com> User-Agent: Mutt/1.4.2.3i Cc: cvs-src@freebsd.org, src-committers@freebsd.org, cvs-all@freebsd.org, John Baldwin , arch@freebsd.org Subject: removal of ipi_all() and ipi_self() [Re: cvs commit: src/sys/sparc64/include smp.h src/sys/sparc64/sparc64 genassym.c mp_machdep.c] 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: Sat, 20 Sep 2008 23:11:55 -0000 On Thu, Sep 18, 2008 at 12:48:52PM -0700, Marcel Moolenaar wrote: > > On Sep 18, 2008, at 12:19 PM, Marius Strobl wrote: > > >On Thu, Sep 18, 2008 at 10:27:51AM -0400, John Baldwin wrote: > >>On Thursday 18 September 2008 09:56:30 am Marius Strobl wrote: > >>>marius 2008-09-18 13:56:30 UTC > >>> > >>> FreeBSD src repository > >>> > >>> Modified files: > >>> sys/sparc64/include smp.h > >>> sys/sparc64/sparc64 genassym.c mp_machdep.c > >>> Log: > >>> SVN rev 183142 on 2008-09-18 13:56:30Z by marius > >>> > >>> - Newer firmware versions no longer provide SUNW,stop-self so just > >>> disable interrupts and loop forever with these. > >>> - Hide all MP-related bits in underneath #ifdef > >>>SMP. > >>> - Inline ipi_all_but_self(9) and ipi_selected(9). We don't expose > >>>any > >>> additional bits but save a few cycles by doing so. > >>> - Remove ipi_all(9), which actually only called panic(9). It > >>>can't be > >>> implemented natively anyway and having it removed at least causes > >>> MI users to fail already fail when linking. > >> > >>Should we just remove ipi_all() completely? > >> > > > >Well, grepping in the CVS repository shows that there never was > >an actually consumer of ipi_all() (only #ifdef'ed out ones in > >ironically the sparc64 code) so it seems to be a good candidate > >for axing. Generally I can't think of a reason why MI code would > >want a CPU to send an IPI to itself. Actually, ipi_self() also > >isn't and never was used in MI code, only in ia64 and powerpc > >code for testing purposes. > > That's DS (=developer-specific) code rather than MI or MD code :-) > > Sending a test IPI to 'self' helps with bring-up or porting, but > serves no real purpose (other than maybe a POST-like purpose) > once IPIs are known to work... > Okay, I take these as a call for removing ipi_all() and ipi_self() along with the ia64 and powerpc test IPI code completely. A patch doing just that and which passes a universe build just fine is at: http://people.freebsd.org/~marius/nuke_ipi_all_self.diff Does anybody object to committing it? Should __FreeBSD_version be bumped for this? Marius