Re: [LTP] [PATCH v2 1/2] shell: add optional OOM protection

> > +	if [ "$TST_OOM_PROTECTION" = 1 -a "$_TST_OOM_PROTECTION" != 0 ]; then
> > +		_tst_run_oom_protected "$@"
> > +	fi
> 
> In the C library, OOM protection for the LTP library and harness
> is enabled by default. But in such Shell implementation, we have to
> explicitly enable it via 'TST_OOM_PROTECTION=1'.

Say precisely: only protect LTP library not contains upper harness.
The upper harness can to be handled out from LTP.

See:

  commit 8e5fe1d5f2 ("lib: enable OOM protection for ltp lib process")


> If we keep them consistent and remove the TST_OOM_PROTECTION
> variable, things will become easier I guess :).

-- 
Regards,
Li Wang

-- 
Mailing list info: https://lists.linux.it/listinfo/ltp