linux-kernel - Re: [PATCH 2/2] time: allow gcc to fold constants when possible

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Tue, 2 Jun 2015 23:23:28 +0200 (CEST)
From:	Thomas Gleixner <tglx@...utronix.de>
To:	Joe Perches <joe@...ches.com>
cc:	Nicholas Mc Guire <hofrat@...dl.org>,
	Michal Marek <mmarek@...e.cz>,
	Masahiro Yamada <yamada.m@...panasonic.com>,
	Sam Ravnborg <sam@...nborg.org>,
	"H. Peter Alvin" <hpa@...or.com>,
	John Stultz <john.stultz@...aro.org>,
	Andrew Hunter <ahh@...gle.com>, Paul Turner <pjt@...gle.com>,
	linux-kernel@...r.kernel.org
Subject: Re: [PATCH 2/2] time: allow gcc to fold constants when possible

On Thu, 28 May 2015, Joe Perches wrote:
> On Thu, 2015-05-28 at 19:09 +0200, Nicholas Mc Guire wrote:
> > To allow constant folding in usecs_to_jiffies() conditionally calls
> > the HZ dependent _usecs_to_jiffies() helpers or, when gcc can not
> > figure out constant folding, __usecs_to_jiffies, which is the renamed
> > original usecs_to_jiffies() function.
> 
> Hi Nicholas.
> 
> Thanks for following through with this.
> 
> The subject might be nicer if it referenced usecs_to_jiffies
> instead of being a bit generic.
> 
> Maybe something like:
> 
> time: allow gcc to fold usecs_to_jiffies(constant)

And for correctness sake the first letter after the colon, i.e. the
one starting the sentence wants to be uppercase.

I'm going to fix that up all together when applying the lot. No need
for resend.

Thanks,

	tglx
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/