Messages in this thread | | | Date | Fri, 13 Mar 2015 16:21:29 -0700 (PDT) | From | David Rientjes <> | Subject | Re: [PATCH 01/32] do_fork(): Rename 'stack_size' argument to reflect actual use |
| |
On Fri, 13 Mar 2015, josh@joshtriplett.org wrote:
> On Fri, Mar 13, 2015 at 08:04:16PM +0200, Alex Dowad wrote: > > The 'stack_size' argument is never used to pass a stack size. It's only used when > > forking a kernel thread, in which case it is an argument which should be passed > > to the 'main' function which the kernel thread executes. Hence, rename it to > > 'kthread_arg'. > > That's not the only use of stack_size. Take a look at the clone2 system > call (very minimally documented in the clone manpage) and the > implementation of copy_thread on ia64, which does use stack_size in the > non-kthread path. >
Exactly, and it seems like Alex just disregarded this early feedback when this was first raised that suggested it just be named "arg" and to comment the individual usage in the functions that get called with the formal.
| |