mirror/gcc
2
0
Fork 0
mirror of git://gcc.gnu.org/git/gcc.git synced 2025-04-06 23:51:26 +08:00
Code Issues Packages Projects Releases Wiki Activity

Set default values for stack-clash and do basic validation in back-end.

Browse Source 
This patch enforces that the default guard size for stack-clash protection for
AArch64 be 64KB unless the user has overriden it via configure in which case
the user value is used as long as that value is within the valid range.

It also does some basic validation to ensure that the guard size is only 4KB or
64KB and also enforces that for aarch64 the stack-clash probing interval is
equal to the guard size.

gcc/

	PR target/86486
	* config/aarch64/aarch64.c (aarch64_override_options_internal):
	Add validation for stack-clash parameters and set defaults.

From-SVN: r264753
This commit is contained in:
Tamar Christina 2018-10-01 13:03:31 +00:00 committed by Tamar Christina
parent 630b1e3a18
commit fbe9af5042
2 changed files with 37 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
gcc/ChangeLog
View File

@ -1,3 +1,9 @@
2018-10-01 Tamar Christina <tamar.christina@arm.com>
PR target/86486
* config/aarch64/aarch64.c (aarch64_override_options_internal):
Add validation for stack-clash parameters and set defaults.
2018-10-01 Tamar Christina <tamar.christina@arm.com>
PR target/86486

31
gcc/config/aarch64/aarch64.c
View File

@ -10992,6 +10992,37 @@ aarch64_override_options_internal (struct gcc_options *opts)
opts->x_param_values,
global_options_set.x_param_values);
/* If the user hasn't changed it via configure then set the default to 64 KB
for the backend. */
maybe_set_param_value (PARAM_STACK_CLASH_PROTECTION_GUARD_SIZE,
DEFAULT_STK_CLASH_GUARD_SIZE == 0
? 16 : DEFAULT_STK_CLASH_GUARD_SIZE,
opts->x_param_values,
global_options_set.x_param_values);
/* Validate the guard size. */
int guard_size = PARAM_VALUE (PARAM_STACK_CLASH_PROTECTION_GUARD_SIZE);
if (guard_size != 12 && guard_size != 16)
error ("only values 12 (4 KB) and 16 (64 KB) are supported for guard "
"size. Given value %d (%llu KB) is out of range.",
guard_size, (1ULL << guard_size) / 1024ULL);
/* Enforce that interval is the same size as size so the mid-end does the
right thing. */
maybe_set_param_value (PARAM_STACK_CLASH_PROTECTION_PROBE_INTERVAL,
guard_size,
opts->x_param_values,
global_options_set.x_param_values);
/* The maybe_set calls won't update the value if the user has explicitly set
one. Which means we need to validate that probing interval and guard size
are equal. */
int probe_interval
= PARAM_VALUE (PARAM_STACK_CLASH_PROTECTION_PROBE_INTERVAL);
if (guard_size != probe_interval)
error ("stack clash guard size '%d' must be equal to probing interval "
"'%d'", guard_size, probe_interval);
/* Enable sw prefetching at specified optimization level for
CPUS that have prefetch. Lower optimization level threshold by 1
when profiling is enabled. */