Re: [PATCH 2/3] perf test: Cleanup daemon if test is interrupted.

2021-03-11 Thread Jiri Olsa
On Thu, Mar 11, 2021 at 08:18:27AM -0800, Ian Rogers wrote: > On Thu, Mar 11, 2021 at 2:38 AM Jiri Olsa wrote: > > > > On Wed, Mar 10, 2021 at 12:41:17PM -0800, Ian Rogers wrote: > > > Reorder daemon_start and daemon_exit as the trap handler is added in > > > daemon_start referencing daemon_exit.

Re: [PATCH 2/3] perf test: Cleanup daemon if test is interrupted.

2021-03-11 Thread Ian Rogers
On Thu, Mar 11, 2021 at 2:38 AM Jiri Olsa wrote: > > On Wed, Mar 10, 2021 at 12:41:17PM -0800, Ian Rogers wrote: > > Reorder daemon_start and daemon_exit as the trap handler is added in > > daemon_start referencing daemon_exit. > > makes sense, minor comments below > > > > > Signed-off-by: Ian Rog

Re: [PATCH 2/3] perf test: Cleanup daemon if test is interrupted.

2021-03-11 Thread Jiri Olsa
On Wed, Mar 10, 2021 at 12:41:17PM -0800, Ian Rogers wrote: > Reorder daemon_start and daemon_exit as the trap handler is added in > daemon_start referencing daemon_exit. makes sense, minor comments below > > Signed-off-by: Ian Rogers > --- > tools/perf/tests/shell/daemon.sh | 34 +

[PATCH 2/3] perf test: Cleanup daemon if test is interrupted.

2021-03-10 Thread Ian Rogers
Reorder daemon_start and daemon_exit as the trap handler is added in daemon_start referencing daemon_exit. Signed-off-by: Ian Rogers --- tools/perf/tests/shell/daemon.sh | 34 +++- 1 file changed, 20 insertions(+), 14 deletions(-) diff --git a/tools/perf/tests/shell/