From: Andrew Stubbs <ams@codesourcery.com>
To: "gcc-patches@gcc.gnu.org" <gcc-patches@gcc.gnu.org>
Subject: [committed] amdgcn: Pass -mstack-size through to runtime
Date: Mon, 6 Feb 2023 17:22:31 +0000 [thread overview]
Message-ID: <03830517-e578-4ee7-b652-e82e6b5a6614@codesourcery.com> (raw)
[-- Attachment #1: Type: text/plain, Size: 755 bytes --]
The -mstack-size option has been marked obsolete in favour of setting an
environment variable at runtime ("GCN_STACK_SIZE"), but some testcases
still need the option set or they have stack overflow. I could change
them to use the envvar, but my testing setup uses remote execute which
doesn't support that yet, and means I would skip my own tests (not ideal).
This patch causes the testcase to automatically set GCN_STACK_SIZE
itself, in a hidden constructor (only if it's not already set), and
therefore bypasses the problem. I'm leaving the documentation saying
-mstack-size is obsolete because this fix only works for offload tests,
and it's still not the preferred user solution.
Committed to mainline. I'll backport to OG12 shortly.
Andrew
[-- Attachment #2: 230206-mstack-size-passthrough.patch --]
[-- Type: text/plain, Size: 1713 bytes --]
amdgcn: Pass -mstack-size through to runtime
But only for the offload case.
gcc/ChangeLog:
* config/gcn/mkoffload.cc (gcn_stack_size): New global variable.
(process_asm): Create a constructor for GCN_STACK_SIZE.
(main): Parse the -mstack-size option.
diff --git a/gcc/config/gcn/mkoffload.cc b/gcc/config/gcn/mkoffload.cc
index 9c262f974be..9493f89fec3 100644
--- a/gcc/config/gcn/mkoffload.cc
+++ b/gcc/config/gcn/mkoffload.cc
@@ -117,6 +117,8 @@ uint32_t elf_arch = EF_AMDGPU_MACH_AMDGCN_GFX803; // Default GPU architecture.
uint32_t elf_flags =
(EF_AMDGPU_FEATURE_XNACK_ANY_V4 | EF_AMDGPU_FEATURE_SRAMECC_ANY_V4);
+static int gcn_stack_size = 0; /* Zero means use default. */
+
/* Delete tempfiles. */
void
@@ -662,6 +664,18 @@ process_asm (FILE *in, FILE *out, FILE *cfile)
}
fprintf (cfile, "\n};\n\n");
+ /* Set the stack size if the user configured a value. */
+ if (gcn_stack_size)
+ fprintf (cfile,
+ "static __attribute__((constructor))\n"
+ "void configure_stack_size (void)\n"
+ "{\n"
+ " const char *val = getenv (\"GCN_STACK_SIZE\");\n"
+ " if (!val || val[0] == '\\0')\n"
+ " setenv (\"GCN_STACK_SIZE\", \"%d\", true);\n"
+ "}\n\n",
+ gcn_stack_size);
+
obstack_free (&fns_os, NULL);
for (i = 0; i < dims_count; i++)
free (dims[i].name);
@@ -920,6 +934,10 @@ main (int argc, char **argv)
elf_arch = EF_AMDGPU_MACH_AMDGCN_GFX908;
else if (strcmp (argv[i], "-march=gfx90a") == 0)
elf_arch = EF_AMDGPU_MACH_AMDGCN_GFX90a;
+#define STR "-mstack-size="
+ else if (startswith (argv[i], STR))
+ gcn_stack_size = atoi (argv[i] + strlen (STR));
+#undef STR
}
if (!(fopenacc ^ fopenmp))
next reply other threads:[~2023-02-06 17:22 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-02-06 17:22 Andrew Stubbs [this message]
2023-02-07 7:40 ` Tobias Burnus
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=03830517-e578-4ee7-b652-e82e6b5a6614@codesourcery.com \
--to=ams@codesourcery.com \
--cc=gcc-patches@gcc.gnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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).