Commit 1a562c0d authored by Ian Rogers's avatar Ian Rogers Committed by Namhyung Kim
Browse files

tools subcmd: Add a no exec function call option 



Tools like perf fork tests in case they crash, but they don't want to
exec a full binary. Add an option to call a function rather than do an
exec. The child process exits with the result of the function call and
is passed the struct of the run_command, things like container_of can
then allow the child process function to determine additional
arguments.

Signed-off-by: default avatarIan Rogers <irogers@google.com>
Cc: James Clark <james.clark@arm.com>
Cc: Justin Stitt <justinstitt@google.com>
Cc: Bill Wendling <morbo@google.com>
Cc: Nick Desaulniers <ndesaulniers@google.com>
Cc: Yang Jihong <yangjihong1@huawei.com>
Cc: Nathan Chancellor <nathan@kernel.org>
Cc: Kan Liang <kan.liang@linux.intel.com>
Cc: Athira Jajeev <atrajeev@linux.vnet.ibm.com>
Cc: llvm@lists.linux.dev
Signed-off-by: default avatarNamhyung Kim <namhyung@kernel.org>
Link: https://lore.kernel.org/r/20240221034155.1500118-5-irogers@google.com
parent 526f2ac9

tools/lib/subcmd/run-command.c

+2 −0
Original line number Diff line number Diff line
@@ -122,6 +122,8 @@ int start_command(struct child_process *cmd) 
		}

		if (cmd->preexec_cb)

			cmd->preexec_cb();

		if (cmd->no_exec_cmd)

			exit(cmd->no_exec_cmd(cmd));

		if (cmd->exec_cmd) {

			execv_cmd(cmd->argv);

		} else {

tools/lib/subcmd/run-command.h

+2 −0
Original line number Diff line number Diff line
@@ -47,6 +47,8 @@ struct child_process { 
	unsigned exec_cmd:1; /* if this is to be external sub-command */

	unsigned stdout_to_stderr:1;

	void (*preexec_cb)(void);

	 /* If set, call function in child rather than doing an exec. */

	int (*no_exec_cmd)(struct child_process *process);

};



int start_command(struct child_process *);