From mboxrd@z Thu Jan 1 00:00:00 1970 From: "H. Peter Anvin" Subject: Re: [PATCH 1/2] Add C99-style constructor macros for specific-sized integers Date: Sun, 02 Mar 2008 20:50:01 -0800 Message-ID: <47CB8379.5050203@zytor.com> References: <4a8566f54ff84f498c5c0c0340076c296b1840e6@tazenda.hos.anvin.org> <7e0381d9932a37ad78f0c38b2a3c6a03@kernel.crashing.org> <200803022014.11535.david-b@pacbell.net> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <200803022014.11535.david-b-yBeKhBN/0LDR7s880joybQ@public.gmane.org> Sender: linux-arch-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-ID: To: David Brownell Cc: Segher Boessenkool , Andrew Morton , Linus Torvalds , Linux Arch Mailing List , Linux Kernel Mailing List David Brownell wrote: > On Sunday 02 March 2008, Segher Boessenkool wrote: > >> Do these new macros really buy anything over just writing LL in the >> few places that 64-bit constants are used? > > The original workaround was to make the offending *SEC_TO_HZ_ADJ32 > constants use "ULL" suffixes. Certainly making the Perl script emit > those would be less intrusive. > "Less intrusive" in the sense of smaller line count for this particular patch, perhaps, but it makes it less obvious when reading the code what's going on (a size override in an entirely different file). Additionally, I'm hoping these kind of things can be used elsewhere; the C99 facilities are really quite nice once you get over the rather ugly look of them, and I for one definitely miss having access to the in the kernel. -hpa -- To unsubscribe from this list: send the line "unsubscribe linux-arch" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from terminus.zytor.com ([198.137.202.10]:52410 "EHLO terminus.zytor.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753180AbYCCExR (ORCPT ); Sun, 2 Mar 2008 23:53:17 -0500 Message-ID: <47CB8379.5050203@zytor.com> Date: Sun, 02 Mar 2008 20:50:01 -0800 From: "H. Peter Anvin" MIME-Version: 1.0 Subject: Re: [PATCH 1/2] Add C99-style constructor macros for specific-sized integers References: <4a8566f54ff84f498c5c0c0340076c296b1840e6@tazenda.hos.anvin.org> <7e0381d9932a37ad78f0c38b2a3c6a03@kernel.crashing.org> <200803022014.11535.david-b@pacbell.net> In-Reply-To: <200803022014.11535.david-b@pacbell.net> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-arch-owner@vger.kernel.org List-ID: To: David Brownell Cc: Segher Boessenkool , Andrew Morton , Linus Torvalds , Linux Arch Mailing List , Linux Kernel Mailing List Message-ID: <20080303045001.Q6jTlWjNoqITW_ltGT38pxGeSHGk88oXPm_Q5YCGJ3c@z> David Brownell wrote: > On Sunday 02 March 2008, Segher Boessenkool wrote: > >> Do these new macros really buy anything over just writing LL in the >> few places that 64-bit constants are used? > > The original workaround was to make the offending *SEC_TO_HZ_ADJ32 > constants use "ULL" suffixes. Certainly making the Perl script emit > those would be less intrusive. > "Less intrusive" in the sense of smaller line count for this particular patch, perhaps, but it makes it less obvious when reading the code what's going on (a size override in an entirely different file). Additionally, I'm hoping these kind of things can be used elsewhere; the C99 facilities are really quite nice once you get over the rather ugly look of them, and I for one definitely miss having access to the in the kernel. -hpa