From owner-svn-src-head@FreeBSD.ORG Sun Apr 12 13:06:01 2009 Return-Path: Delivered-To: svn-src-head@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 24EDB106564A; Sun, 12 Apr 2009 13:06:00 +0000 (UTC) (envelope-from kabaev@gmail.com) Received: from mail-qy0-f132.google.com (mail-qy0-f132.google.com [209.85.221.132]) by mx1.freebsd.org (Postfix) with ESMTP id 75ADA8FC18; Sun, 12 Apr 2009 13:05:59 +0000 (UTC) (envelope-from kabaev@gmail.com) Received: by qyk38 with SMTP id 38so227845qyk.3 for ; Sun, 12 Apr 2009 06:05:58 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=domainkey-signature:received:received:date:from:to:cc:subject :message-id:in-reply-to:references:x-mailer:mime-version :content-type; bh=1IzA/mCHO5mB51EVB8++CRdU6Ye7VtK/oN5IHP7R3uQ=; b=QeZLQtscTIB7suFoMGNNR5cV5jOzcdWr6stHOdcRQOs8bRix7SdCntJM3jksRk3Hjk 8konMmW2y9xgMb47yxBIIOP/uPyOcuN444diyVB60swN8oQcXViZVLtk4vSf64MUSA8c SKuYO2TB3O6XY+dHeN7FjqfxDoBUKKQbGxU3w= DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=date:from:to:cc:subject:message-id:in-reply-to:references:x-mailer :mime-version:content-type; b=KEdJk4dYatMyER3QeHCxAbgwe4kr9le+jQerM7ZLmV+gBQcbMEmm7wABTdh0/P/U3a 8lmR3DL1hjTPAf2wiHJoBqXh0xBaG04Ez5aKHBLJoxdWx6F+sUlPNi17pEK+kJ/9zf0k 8QrxPfgP3nLBVwMoI8O4W+7T2z/nz5g0D9OUo= Received: by 10.224.11.136 with SMTP id t8mr5313629qat.228.1239541558740; Sun, 12 Apr 2009 06:05:58 -0700 (PDT) Received: from kan.dnsalias.net (c-98-217-224-113.hsd1.ma.comcast.net [98.217.224.113]) by mx.google.com with ESMTPS id 7sm6258171qwb.31.2009.04.12.06.05.57 (version=SSLv3 cipher=RC4-MD5); Sun, 12 Apr 2009 06:05:58 -0700 (PDT) Date: Sun, 12 Apr 2009 09:05:51 -0400 From: Alexander Kabaev To: Christoph Mallon Message-ID: <20090412090551.176f327c@kan.dnsalias.net> In-Reply-To: <49E1E01C.90704@gmx.de> References: <200904111401.n3BE1108088009@svn.freebsd.org> <20090411163528.GC46526@troutmask.apl.washington.edu> <20090411124335.0600a72f@kan.dnsalias.net> <49E1E01C.90704@gmx.de> X-Mailer: Claws Mail 3.7.1 (GTK+ 2.14.7; i386-portbld-freebsd8.0) Mime-Version: 1.0 Content-Type: multipart/signed; boundary="Sig_/sq08_vM1+Gby3ShgmOFNVC+"; protocol="application/pgp-signature"; micalg=PGP-SHA1 Cc: svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org, Ed Schouten Subject: Re: svn commit: r190919 - in head/sys: amd64/amd64 amd64/include i386/i386 i386/include X-BeenThere: svn-src-head@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: SVN commit messages for the src tree for head/-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 12 Apr 2009 13:06:01 -0000 --Sig_/sq08_vM1+Gby3ShgmOFNVC+ Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: quoted-printable On Sun, 12 Apr 2009 14:35:40 +0200 Christoph Mallon wrote: > Alexander Kabaev schrieb: > > On Sat, 11 Apr 2009 09:35:28 -0700 > > Steve Kargl wrote: > >=20 > >> On Sat, Apr 11, 2009 at 02:01:01PM +0000, Ed Schouten wrote: > >>> Author: ed > >>> Date: Sat Apr 11 14:01:01 2009 > >>> New Revision: 190919 > >>> URL: http://svn.freebsd.org/changeset/base/190919 > >>> > >>> Log: > >>> Simplify in/out functions (for i386 and AMD64). > >>> =20 > >>> Remove a hack to generate more efficient code for port numbers > >>> below 0x100, which has been obsolete for at least ten years, > >>> because GCC has an asm constraint to specify that. > >>> =20 > >>> Submitted by: Christoph Mallon > >>> > >> I thought Christoph and bde were still hashing out the correctness > >> of this patch. > >> > >> http://lists.freebsd.org/pipermail/freebsd-amd64/2009-April/012064.html > >> > >> --=20 > >> Steve > >=20 > > The patch is inconsistent in regards to usage of volatile vs. > > __volatile even within itself. I think the code is sloppy and was > > not ready to be committed yet. Please fix or back out. >=20 > Backing it out because of two underscores (!) would be=20 > counterproductive: It removes about 150 lines of hard to read hacks,=20 > which are unnecessary for at least a decade. GCC 2.95, which was=20 > released in 1999 supports the "N" constraint for inline asm. Perhaps=20 > olders do, too, but you cannot get older GCCs from the official site. > Attached is a patch, which replaces all __inline and __volatile in > the touched headers by thir ISO equivalents - again there hasn't been > a reason to use the alternate GCC keywords for at least a decade. > Also "inline" and "volatile" are already used hundreds of times in > sys/. The patch is simply the result of > %s/\<__\(inline\|volatile\)\>/\1/. >=20 > Christoph Underscores as the sole reason for backing out might as well be counterproductive. The real objection was not about underscored though, but about consistency. Your patch was not consistent itself and it left files it touched with unholy mixture of plain and underscored versions sprinkled all over with no apparent system. The way to handle this change was: 1. Prepare functional patch, commit. 2. Prepare underscore removal patch, commit. Both steps require proper review and all reviewer concerns should have been addressed or reviewer should be persuaded to withdraw his objections if any. -- Alexander Kabaev --Sig_/sq08_vM1+Gby3ShgmOFNVC+ Content-Type: application/pgp-signature; name=signature.asc Content-Disposition: attachment; filename=signature.asc -----BEGIN PGP SIGNATURE----- Version: GnuPG v2.0.11 (FreeBSD) iD8DBQFJ4ec0Q6z1jMm+XZYRApaVAKCcArQ01hWBe530HLrB8DXm0JERkQCfZxRY I9Bidwp3p54apZrxnw78K5A= =KAS/ -----END PGP SIGNATURE----- --Sig_/sq08_vM1+Gby3ShgmOFNVC+--