public inbox for gentoo-dev@lists.gentoo.org
 help / color / mirror / Atom feed
Search results ordered by [date|relevance]  view[summary|nested|Atom feed]
thread overview below | download: 
* [gentoo-dev] Re: [PATCH] To enable ssp default in Gcc the toolchain.eclass need some changes.
  @ 2014-01-10 21:56 99%                 ` Ryan Hill
  0 siblings, 0 replies; 1+ results
From: Ryan Hill @ 2014-01-10 21:56 UTC (permalink / raw
  To: gentoo-dev

[-- Attachment #1: Type: text/plain, Size: 1748 bytes --]

On Fri, 10 Jan 2014 15:08:02 -0500
"Anthony G. Basile" <basile@opensource.dyc.edu> wrote:

> On 01/10/2014 10:50 AM, Ryan Hill wrote:
> > Having slept on it I'm starting to agree.  My first argument was that on
> > hardened ssp is -fstack-protector-all, which is much more expensive, and it
> > adds -fstack-check and -z,now to the linker by default as well.  The pie
> > half
> 
> I'm pretty sure we're not adding -fstack-check unless something has 
> changed.  Where are you seeing that?
> 
> The reason I'm concerned is because of situations like bug #471756. 
> stack-check incumbers a register which in some situations (like the asm 
> in ffmpeg) can get you into trouble with not enough GENERAL_REGS.

40_all_gcc48_config_esp.patch:

  54 +   #if defined ( EFAULT_SSP ) || defined ( EFAULT_PIE_SSP )
  55 +       #define ESP_OPTIONS_SSP_SPEC \
  56 +           "%{nostdlib|nodefaultlibs|ffreestanding|fno-stack-protector| \
  57 +           fstack-protector|fstack-protector-all:;:-fstack-protector-all} \
  58 +           %{fstack-check|fstack-check=*:;: -fstack-check}"

> > It might be useful to have these flags so we can mask them on archs that
> > don't support ssp/pie.  But that's always been true and it looks like sh is
> > the only place we've bothered for some reason.
> 
> Yes please.  I had this issue on mips where gcc didn't support ssp for 
> early versions of gcc 4.x.

There's a list of arches in every gcc ebuild that support PIE and SSP for
both glibc and uclibc.  AFAICT you would just have to remove mips from that
list.


-- 
Ryan Hill                        psn: dirtyepic_sk
   gcc-porting/toolchain/wxwidgets @ gentoo.org

47C3 6D62 4864 0E49 8E9E  7F92 ED38 BD49 957A 8463

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 490 bytes --]

^ permalink raw reply	[relevance 99%]

Results 1-1 of 1 | reverse | options above
-- pct% links below jump to the message on this page, permalinks otherwise --
2014-01-09 20:58     [gentoo-dev] [PATCH] To enable ssp default in Gcc the toolchain.eclass need some changes Magnus Granberg
2014-01-09 21:11     ` Rick "Zero_Chaos" Farina
2014-01-09 23:26       ` [gentoo-dev] " Ryan Hill
2014-01-09 23:30         ` Andreas K. Huettel
2014-01-09 23:41           ` William Hubbs
2014-01-10  0:12             ` Ryan Hill
2014-01-10  6:35               ` Rick "Zero_Chaos" Farina
2014-01-10 15:50                 ` Ryan Hill
2014-01-10 20:08                   ` Anthony G. Basile
2014-01-10 21:56 99%                 ` Ryan Hill

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox