diff options
author | Vasant Hegde <hegdevasant@linux.vnet.ibm.com> | 2018-08-07 16:37:13 +0530 |
---|---|---|
committer | Stewart Smith <stewart@linux.ibm.com> | 2018-08-13 23:48:19 -0500 |
commit | 15880d514e1f27e4380eaaf0b7de5ac90d35da66 (patch) | |
tree | 7c6e3c2c4f53c63631b0d9bb7424ec1195a31a4c /core/cpu.c | |
parent | 8bb4a1cd9af489f8e4a89fd77d89fecc6e387cc8 (diff) | |
download | skiboot-15880d514e1f27e4380eaaf0b7de5ac90d35da66.zip skiboot-15880d514e1f27e4380eaaf0b7de5ac90d35da66.tar.gz skiboot-15880d514e1f27e4380eaaf0b7de5ac90d35da66.tar.bz2 |
core/cpu: Call memset with proper cpu_thread offset
"cpu_thread *t + value" vs "(void *)t + val"
Fixes: cfe9d441 (core/cpu: Prevent clobbering of stack guard for boot-cpu)
CC: stable <skiboot@lists.ozlabs.org> # v6.0+
CC: Vaibhav Jain <vaibhav@linux.vnet.ibm.com>
CC: Nicholas Piggin <npiggin@gmail.com>
CC: Mahesh Salgaonkar <mahesh@linux.vnet.ibm.com>
Acked-by: Mahesh Salgaonkar <mahesh@linux.vnet.ibm.com>
Signed-off-by: Vasant Hegde <hegdevasant@linux.vnet.ibm.com>
Reviewed-by: Vaibhav Jain<vaibhav@linux.ibm.com>
Signed-off-by: Stewart Smith <stewart@linux.ibm.com>
Diffstat (limited to 'core/cpu.c')
-rw-r--r-- | core/cpu.c | 2 |
1 files changed, 1 insertions, 1 deletions
@@ -889,7 +889,7 @@ static void init_cpu_thread(struct cpu_thread *t, const size_t guard_skip = container_off_var(t, stack_guard) + sizeof(t->stack_guard); - memset(t + guard_skip, 0, sizeof(struct cpu_thread) - guard_skip); + memset(((void *)t) + guard_skip, 0, sizeof(struct cpu_thread) - guard_skip); init_lock(&t->dctl_lock); init_lock(&t->job_lock); list_head_init(&t->job_queue); |