Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> writes: > Skip trying to stop unused fork helpers to avoid the various asserts > that they were running and didn't die early. > > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=109468 > Signed-off-by: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> > --- > lib/igt_core.c | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/lib/igt_core.c b/lib/igt_core.c > index 49fbf70de..0794e11a0 100644 > --- a/lib/igt_core.c > +++ b/lib/igt_core.c > @@ -1678,6 +1678,9 @@ void igt_stop_helper(struct igt_helper_process *proc) > { > int status; > > + if (!proc->running) /* never even started */ > + return; > + The comment above mentions that it is error to call this on a helper process which hasn't been spawned yet. So as we relax the requirements, remove that comment. With that, Reviewed-by: Mika Kuoppala <mika.kuoppala@xxxxxxxxxxxxxxx> > /* failure here means the pid is already dead and so waiting is safe */ > kill(proc->pid, proc->use_SIGKILL ? SIGKILL : SIGTERM); > > -- > 2.20.1 > > _______________________________________________ > Intel-gfx mailing list > Intel-gfx@xxxxxxxxxxxxxxxxxxxxx > https://lists.freedesktop.org/mailman/listinfo/intel-gfx _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
