public inbox for binutils@sourceware.org
 help / color / mirror / Atom feed
* [PATCH v1 1/1] ld: microblaze: ignore rwx segments
@ 2023-09-28  5:57 Neal Frager
  2023-10-04 15:39 ` Michael Eager
  0 siblings, 1 reply; 2+ messages in thread
From: Neal Frager @ 2023-09-28  5:57 UTC (permalink / raw)
  To: binutils
  Cc: ibai.erkiaga-elorza, nagaraju.mekala, mark.hatle,
	sadanand.mutyala, appa.rao.nali, vidhumouli.hunsigida,
	luca.ceresoli, Neal Frager

The linker will generate warnings if it is creating an executable
stack or a segment with all three read, write and execute permissions.
These settings are not appropriate for all targets including
MicroBlaze.

Signed-off-by: Neal Frager <neal.frager@amd.com>
---
 ld/configure.tgt | 1 +
 1 file changed, 1 insertion(+)

diff --git a/ld/configure.tgt b/ld/configure.tgt
index ff0aaaae257..3aa595ef130 100644
--- a/ld/configure.tgt
+++ b/ld/configure.tgt
@@ -55,6 +55,7 @@ if test "${ac_default_ld_warn_rwx_segments}" = unset; then
     cris-*-* | crisv32-*-* | \
     hppa*-*-* | \
     mips*-*-* | \
+    microblaze*-*-* | \
     sparc*-*-* | \
     v850*-*-*)
       ac_default_ld_warn_rwx_segments=0
-- 
2.25.1


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

* Re: [PATCH v1 1/1] ld: microblaze: ignore rwx segments
  2023-09-28  5:57 [PATCH v1 1/1] ld: microblaze: ignore rwx segments Neal Frager
@ 2023-10-04 15:39 ` Michael Eager
  0 siblings, 0 replies; 2+ messages in thread
From: Michael Eager @ 2023-10-04 15:39 UTC (permalink / raw)
  To: Neal Frager, binutils
  Cc: ibai.erkiaga-elorza, nagaraju.mekala, mark.hatle,
	sadanand.mutyala, appa.rao.nali, vidhumouli.hunsigida,
	luca.ceresoli

On 9/27/23 22:57, Neal Frager via Binutils wrote:
> The linker will generate warnings if it is creating an executable
> stack or a segment with all three read, write and execute permissions.
> These settings are not appropriate for all targets including
> MicroBlaze.
> 
> Signed-off-by: Neal Frager <neal.frager@amd.com>
> ---
>   ld/configure.tgt | 1 +
>   1 file changed, 1 insertion(+)
> 
> diff --git a/ld/configure.tgt b/ld/configure.tgt
> index ff0aaaae257..3aa595ef130 100644
> --- a/ld/configure.tgt
> +++ b/ld/configure.tgt
> @@ -55,6 +55,7 @@ if test "${ac_default_ld_warn_rwx_segments}" = unset; then
>       cris-*-* | crisv32-*-* | \
>       hppa*-*-* | \
>       mips*-*-* | \
> +    microblaze*-*-* | \
>       sparc*-*-* | \
>       v850*-*-*)
>         ac_default_ld_warn_rwx_segments=0

Committed.

-- 
Michael Eager


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

end of thread, other threads:[~2023-10-04 15:39 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-09-28  5:57 [PATCH v1 1/1] ld: microblaze: ignore rwx segments Neal Frager
2023-10-04 15:39 ` Michael Eager

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).