From 49bb83e8c99f74f72aaf4e85a604204cdfa3fbc4 Mon Sep 17 00:00:00 2001 From: Isaku Yamahata Date: Wed, 23 Jan 2019 10:43:12 -0800 Subject: [PATCH] [LibOS] Remove redundant variable pcargs in clone_implementation_wrapper() --- LibOS/shim/src/sys/shim_clone.c | 13 ++++++------- 1 file changed, 6 insertions(+), 7 deletions(-) diff --git a/LibOS/shim/src/sys/shim_clone.c b/LibOS/shim/src/sys/shim_clone.c index 5a157daa..c60a7b55 100644 --- a/LibOS/shim/src/sys/shim_clone.c +++ b/LibOS/shim/src/sys/shim_clone.c @@ -94,13 +94,12 @@ int clone_implementation_wrapper(struct clone_args * arg) //PAL allocated stack. We need to switch the stack to use //the user provided stack. - struct clone_args *pcargs = arg; int stack_allocated = 0; - object_wait_with_retry(pcargs->create_event); - DkObjectClose(pcargs->create_event); + object_wait_with_retry(arg->create_event); + DkObjectClose(arg->create_event); - struct shim_thread * my_thread = pcargs->thread; + struct shim_thread * my_thread = arg->thread; assert(my_thread); get_thread(my_thread); @@ -121,8 +120,8 @@ int clone_implementation_wrapper(struct clone_args * arg) if (my_thread->set_child_tid) *(my_thread->set_child_tid) = my_thread->tid; - void * stack = pcargs->stack; - void * return_pc = pcargs->return_pc; + void * stack = arg->stack; + void * return_pc = arg->return_pc; struct shim_vma_val vma; lookup_vma(ALIGN_DOWN(stack), &vma); @@ -134,7 +133,7 @@ int clone_implementation_wrapper(struct clone_args * arg) set_as_child(arg->parent, my_thread); /* Don't signal the initialize event until we are actually init-ed */ - DkEventSet(pcargs->initialize_event); + DkEventSet(arg->initialize_event); /***** From here down, we are switching to the user-provided stack ****/