summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorCraig Hesling <hesling@chromium.org>2022-09-27 14:07:36 -0700
committerChromeos LUCI <chromeos-scoped@luci-project-accounts.iam.gserviceaccount.com>2022-09-29 20:50:17 +0000
commit170dff8dc9b2f5a5ff1e0cdae1393b2358f46366 (patch)
tree69b24d20e8ac73379caf0cec2aa33e0ac6f5e397
parent17e0734e88c26d63658952c8cad73ef847a7b277 (diff)
downloadchrome-ec-170dff8dc9b2f5a5ff1e0cdae1393b2358f46366.tar.gz
test: Change always_memset extra stack to all 's'
This allows for easier identification of stack usage, since 0 is a common value to be written to the stack. BRANCH=none BUG=b:176500425 TEST=make run-always_memset V=1 TEST=make CLANG_BOARDS=host run-always_memset V=1 Signed-off-by: Craig Hesling <hesling@chromium.org> Change-Id: Id88cb5daafede6be17aae558b2bf214a52facb67 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3922445 Reviewed-by: Tom Hughes <tomhughes@chromium.org>
-rw-r--r--test/always_memset.c8
1 files changed, 6 insertions, 2 deletions
diff --git a/test/always_memset.c b/test/always_memset.c
index 03437bc651..40a68d883b 100644
--- a/test/always_memset.c
+++ b/test/always_memset.c
@@ -88,7 +88,9 @@ test_static void exercise_memset(char **p)
*
* Without volatile, space will be optimized out.
*/
- volatile char __unused space[EXTRA_STACK_SIZE] = { 0 };
+ volatile char __unused space[EXTRA_STACK_SIZE] = {
+ [0 ... EXTRA_STACK_SIZE - 1] = 's'
+ };
char buf[] = UNIQUE_STRING;
*p = buf;
@@ -142,7 +144,9 @@ test_static void exercise_always_memset(char **p)
*
* Without volatile, space will be optimized out.
*/
- volatile char __unused space[EXTRA_STACK_SIZE] = { 0 };
+ volatile char __unused space[EXTRA_STACK_SIZE] = {
+ [0 ... EXTRA_STACK_SIZE - 1] = 's'
+ };
char buf[] = UNIQUE_STRING;
*p = buf;