[LTP] [PATCH v3 3/4] lib: ignore SIGINT in _tst_kill_test

Joerg Vehlow
2021-05-12T10:34:29Z

Hi Li,
>> A possible fix could be using SIGTERM instead of SIGINT. This signal
>> does not seem to have some "intelligent" handling for background processes.
> I agree. Can you make a patch to replace that INT?
>
> (and this is only a timeout issue, so patch merging may be delayed due
> to LTP new release)
I'd like to supply the patch, I've placed it on my todo list. I will
probably not finish it before the release,
but since it will probably not be included anyway, it doesn't matter.
I do not know why LTP used SIGINT in the first place. My first thought
>> would have been to use SIGTERM. It is the way to "politely ask
>> processes to terminate"
> Yes, but that not strange to me, the possible reason is just to
> stop(ctrl ^c) the LTP test manually for debugging, so we went
> too far for using SIGINT but forget the original purpose :).
Ok sounds reasonable. The nice think about changing timeout to SIGTERM
would be,
that abort using CTRL+C is clearly distinguishable from a timeout.

J?rg