public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
* [Bug target/102352] New: Add -mstack-protector-guard=... for arm32
@ 2021-09-15 16:23 kees at outflux dot net
  2021-09-15 20:03 ` [Bug target/102352] " pinskia at gcc dot gnu.org
  2021-10-14  9:22 ` ardb at kernel dot org
  0 siblings, 2 replies; 3+ messages in thread
From: kees at outflux dot net @ 2021-09-15 16:23 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=102352

            Bug ID: 102352
           Summary: Add -mstack-protector-guard=... for arm32
           Product: gcc
           Version: unknown
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: target
          Assignee: unassigned at gcc dot gnu.org
          Reporter: kees at outflux dot net
  Target Milestone: ---

As done for powerpc (1b3254e4bbe82245421a55324bc8fe34a99c6e3c), aarch64
(cd0b2d361df82c848dc7e1c3078651bb0624c3c6), riscv
(c931e8d5a96463427040b0d11f9c4352ac22b2b0), and x86
(e1769bdd4cef522ada32aec863feba41116b183a), the arm32 target needs
mstack-protector-guard= support as well, so that the kernel can gain per-thread
stack canaries on arm32:
https://github.com/KSPP/linux/issues/29

^ permalink raw reply	[flat|nested] 3+ messages in thread

* [Bug target/102352] Add -mstack-protector-guard=... for arm32
  2021-09-15 16:23 [Bug target/102352] New: Add -mstack-protector-guard=... for arm32 kees at outflux dot net
@ 2021-09-15 20:03 ` pinskia at gcc dot gnu.org
  2021-10-14  9:22 ` ardb at kernel dot org
  1 sibling, 0 replies; 3+ messages in thread
From: pinskia at gcc dot gnu.org @ 2021-09-15 20:03 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=102352

Andrew Pinski <pinskia at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Severity|normal                      |enhancement

^ permalink raw reply	[flat|nested] 3+ messages in thread

* [Bug target/102352] Add -mstack-protector-guard=... for arm32
  2021-09-15 16:23 [Bug target/102352] New: Add -mstack-protector-guard=... for arm32 kees at outflux dot net
  2021-09-15 20:03 ` [Bug target/102352] " pinskia at gcc dot gnu.org
@ 2021-10-14  9:22 ` ardb at kernel dot org
  1 sibling, 0 replies; 3+ messages in thread
From: ardb at kernel dot org @ 2021-10-14  9:22 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=102352

Ard Biesheuvel <ardb at kernel dot org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |ardb at kernel dot org

--- Comment #1 from Ard Biesheuvel <ardb at kernel dot org> ---
For 32-bit ARM, it would be useful to have

   -mstack-protector-guard=tls
   -mstack-protector-guard-offset=<offset>

and use gen_load_tp_hard() internally. As we also use the TLS register to index
other per-task data (using __builtin_thread_pointer()), we get better code this
way, since the compiler understands that there is no need to load the TLS
register multiple times in the same function.

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2021-10-14  9:22 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-09-15 16:23 [Bug target/102352] New: Add -mstack-protector-guard=... for arm32 kees at outflux dot net
2021-09-15 20:03 ` [Bug target/102352] " pinskia at gcc dot gnu.org
2021-10-14  9:22 ` ardb at kernel dot org

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).