diff options
| author |   Eric W. Biederman <ebiederm@xmission.com> | 2020-06-24 16:34:57 -0500 |
|---|---|---|
| committer | Eric W. Biederman <ebiederm@xmission.com> | 2020-07-04 09:33:40 -0500 |
| commit | 5fec25f2cb959cb5f189d7f6127bee3efc782530 (patch) | |
| tree | 32bd4073caaf2c279e0d97e1e3dce5084ebc8a7f /include/linux/umh.h | |
| parent | Linux 5.8-rc1 (diff) | |
| download | linux-dev-5fec25f2cb959cb5f189d7f6127bee3efc782530.tar.xz linux-dev-5fec25f2cb959cb5f189d7f6127bee3efc782530.zip | |
umh: Capture the pid in umh_pipe_setup
The pid in struct subprocess_info is only used by umh_clean_and_save_pid to
write the pid into umh_info.
Instead always capture the pid on struct umh_info in umh_pipe_setup, removing
code that is specific to user mode drivers from the common user path of
user mode helpers.
v1: https://lkml.kernel.org/r/87h7uygf9i.fsf_-_@x220.int.ebiederm.org
v2: https://lkml.kernel.org/r/875zb97iix.fsf_-_@x220.int.ebiederm.org
Link: https://lkml.kernel.org/r/20200702164140.4468-1-ebiederm@xmission.com
Reviewed-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Acked-by: Alexei Starovoitov <ast@kernel.org>
Tested-by: Alexei Starovoitov <ast@kernel.org>
Signed-off-by: "Eric W. Biederman" <ebiederm@xmission.com>
Diffstat (limited to 'include/linux/umh.h')
| -rw-r--r-- | include/linux/umh.h | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/include/linux/umh.h b/include/linux/umh.h index 0c08de356d0d..aae16a0ebd0f 100644 --- a/include/linux/umh.h +++ b/include/linux/umh.h @@ -25,7 +25,6 @@ struct subprocess_info { struct file *file; int wait; int retval; - pid_t pid; int (*init)(struct subprocess_info *info, struct cred *new); void (*cleanup)(struct subprocess_info *info); void *data; |