From owner-svn-src-projects@FreeBSD.ORG Fri Aug 2 19:14:48 2013 Return-Path: Delivered-To: svn-src-projects@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [8.8.178.115]) (using TLSv1 with cipher ADH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTP id 43563B7C; Fri, 2 Aug 2013 19:14:48 +0000 (UTC) (envelope-from kostikbel@gmail.com) Received: from kib.kiev.ua (kib.kiev.ua [IPv6:2001:470:d5e7:1::1]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.freebsd.org (Postfix) with ESMTPS id BB5FF269F; Fri, 2 Aug 2013 19:14:47 +0000 (UTC) Received: from tom.home (kostik@localhost [127.0.0.1]) by kib.kiev.ua (8.14.7/8.14.7) with ESMTP id r72JEgPm034151; Fri, 2 Aug 2013 22:14:42 +0300 (EEST) (envelope-from kostikbel@gmail.com) DKIM-Filter: OpenDKIM Filter v2.8.3 kib.kiev.ua r72JEgPm034151 Received: (from kostik@localhost) by tom.home (8.14.7/8.14.7/Submit) id r72JEgIJ034150; Fri, 2 Aug 2013 22:14:42 +0300 (EEST) (envelope-from kostikbel@gmail.com) X-Authentication-Warning: tom.home: kostik set sender to kostikbel@gmail.com using -f Date: Fri, 2 Aug 2013 22:14:42 +0300 From: Konstantin Belousov To: Jung-uk Kim Subject: Re: svn commit: r253891 - in projects/atomic64/sys: amd64/include i386/include Message-ID: <20130802191442.GD4972@kib.kiev.ua> References: <201308021832.r72IW2pR034687@svn.freebsd.org> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="NEAn2rYUJijllS+S" Content-Disposition: inline In-Reply-To: <201308021832.r72IW2pR034687@svn.freebsd.org> User-Agent: Mutt/1.5.21 (2010-09-15) X-Spam-Status: No, score=-2.0 required=5.0 tests=ALL_TRUSTED,BAYES_00, DKIM_ADSP_CUSTOM_MED,FREEMAIL_FROM,NML_ADSP_CUSTOM_MED autolearn=no version=3.3.2 X-Spam-Checker-Version: SpamAssassin 3.3.2 (2011-06-06) on tom.home Cc: svn-src-projects@freebsd.org, src-committers@freebsd.org X-BeenThere: svn-src-projects@freebsd.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: "SVN commit messages for the src " projects" tree" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 02 Aug 2013 19:14:48 -0000 --NEAn2rYUJijllS+S Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Fri, Aug 02, 2013 at 06:32:02PM +0000, Jung-uk Kim wrote: > Author: jkim > Date: Fri Aug 2 18:32:01 2013 > New Revision: 253891 > URL: http://svnweb.freebsd.org/changeset/base/253891 >=20 > Log: > Allow an immediate for the bit number. > =20 > Suggested by: >=20 > Modified: > projects/atomic64/sys/amd64/include/atomic.h > projects/atomic64/sys/i386/include/atomic.h >=20 > Modified: projects/atomic64/sys/amd64/include/atomic.h > =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D > --- projects/atomic64/sys/amd64/include/atomic.h Fri Aug 2 18:05:42 2013= (r253890) > +++ projects/atomic64/sys/amd64/include/atomic.h Fri Aug 2 18:32:01 2013= (r253891) > @@ -225,7 +225,7 @@ atomic_testandset_int(volatile u_int *p, > "# atomic_testandset_int" > : "=3Dr" (res), /* 0 */ > "=3Dm" (*p) /* 1 */ > - : "r" (v), /* 2 */ > + : "Ir" (v % 32), /* 2 */ > "m" (*p) /* 3 */ > : "cc"); > return (res); > @@ -243,7 +243,7 @@ atomic_testandset_long(volatile u_long * > "# atomic_testandset_long" > : "=3Dr" (res), /* 0 */ > "=3Dm" (*p) /* 1 */ > - : "r" ((u_long)v), /* 2 */ > + : "Jr" ((u_long)v % 64), /* 2 */ > "m" (*p) /* 3 */ > : "cc"); > return (res); >=20 > Modified: projects/atomic64/sys/i386/include/atomic.h > =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D > --- projects/atomic64/sys/i386/include/atomic.h Fri Aug 2 18:05:42 2013 = (r253890) > +++ projects/atomic64/sys/i386/include/atomic.h Fri Aug 2 18:32:01 2013 = (r253891) > @@ -364,9 +364,9 @@ atomic_testandset_int(volatile u_int *p, > " btsl %2, %1 ; " > " setc %0 ; " > "# atomic_testandset_int" > - : "=3Dr" (res), /* 0 (result) */ > + : "=3Dr" (res), /* 0 */ > "=3Dm" (*p) /* 1 */ > - : "r" (v), /* 2 */ > + : "Ir" (v % 32), /* 2 */ > "m" (*p) /* 3 */ > : "cc"); > return (res); You should also adjust the pointer then, to compensate the clamp of the bit number. Bit operations apparently work on arbitrary-length bitstrings. --NEAn2rYUJijllS+S Content-Type: application/pgp-signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v2.0.20 (FreeBSD) iQIcBAEBAgAGBQJR/AUiAAoJEJDCuSvBvK1BMUQP/iXHMoCOb7tqPrwCe2glUlyI n2Rcbb4TirC2gVAUddwAga8+d3RufKZMvuVY4QocNM5Bb6jtm7t5pPNCHwAPdETT 3v0ulbDSg1POkUZNi6FtOuaOrkYpsFnfq9sfwBQjxidWWGvE/vacScwfgcoQDb9M ReoQ9e83Tt9TlfUDC2KbDf1i4ZUq6v04dD/FcJBniNddWRQtjiUvbm/uuRXToK92 hnosDgKdwzYPMkDeoew9iII9qg9nXIaasJ3LguuInPHO1xvB51mM/WxYKlw/OKJM SImXNfokgaK38Xwk27sfw0BpQoXxNlN0yMHIiTK+fU1N5aIfCFZk8V9Mihcotya6 mZkIuVlkAUBqHyVUarPKwiEpRjS3qJSVlxj1aIg5q/wynGxAgSTpJ4+3Iy4dT9Uy 0ksQkCSOefLrcHbbg77gpdsErhIM9JcPeJ5sd8eZ+JPM3VlaLbbunSPF80ng4Sk5 hVWXxMMVuyiM++hARNurqdCZAF4mD88pE4TsUDRQ2f4wYe4/+IGEyejZwbeE7/Hr JAyDU2kXG9bzcVavY+SisKL/Jv7H9aiztaUv3D8ww6nto3+nCvcp8B8Hk+WVjXhd RFrnCYsxgDELE3ZsIGrf56M4koF39BdRTJyA7Y/lge930WIpbbzIYaB/033rbrmC sm+fyCWJWdRO7WOmcqu+ =RyM8 -----END PGP SIGNATURE----- --NEAn2rYUJijllS+S--