linux-kernel - Re: [PATCH 01/12] Force always inline for gcc 4.5 when optimizing for size

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] [day] [month] [year] [list]

Date:	Sun, 22 May 2011 21:20:42 +0100 (IST)
From:	Dave Airlie <airlied@...ux.ie>
To:	Andi Kleen <andi@...stfloor.org>
cc:	linux-kernel@...r.kernel.org, akpm@...ux-foundation.org,
	Andi Kleen <ak@...ux.intel.com>
Subject: Re: [PATCH 01/12] Force always inline for gcc 4.5 when optimizing
 for size


> > Oh these look like a good plan, one of our contributors has an inline
> > fetish and it got cut-n-pasted into a few places particular in radeon.
> >
> > But we won't be turning DRM debugging off, its what we use to debug things
> > in the field, I did get a patch recently to reduce some of its overhead
> 
> Why can't it be a CONFIG option like all other debugging code?

Not sure which part of debug things in the field doesn't explain it, 
having people remove the debugging code would sort of work against fixing 
problems. Having a config option might be possible, but then I'd have to 
deal with people who think they know what they are doing turning it off, 
so I'd rather not. Next think you know some distro turns it off and you 
have users who you can't help anymore.

> If you want to keep it on would need to drop even more inlines to get
> the 50k overhead down.

Dave.
--
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/