----- On May 15, 2019, at 12:31 PM, Jonathan Rajotte
jonathan.rajotte-jul...@efficios.com wrote:
>> @@ -168,8 +187,7 @@ skip $isroot "Root access is needed to set bandwith
>> limits.
>> Skipping all tests."
>> {
>>
>> # Catch sigint and try to cleanup limits
>> - trap interrupt_cleanup SIGTERM
>> - trap interrupt_cleanup SIGINT
>> + trap interrupt_cleanup SIGTERM SIGINT
>>
>> BW_LIMITS=(3200 1600 800 400 200 100 50 25)
>> for BW in ${BW_LIMITS[@]};
>> @@ -177,6 +195,7 @@ skip $isroot "Root access is needed to set bandwith
>> limits.
>> Skipping all tests."
>> diag "Test high-throughput with bandwidth limit set to
>> ${BW}kbits"
>>
>> set_bw_limit $BW
>> + ok $? "Setting bandwidth limit"
>
> In case of failure here we should force a reset since it could lead to long
> timeout/hang depending on which tc command failed?
>
> This could also be done inside set_bw_limit at each stage.
I'll do this. Will be in v2 of this patch.
Thanks,
Mathieu
>
>>
>> start_lttng_sessiond
>> start_lttng_relayd "-o $TRACE_PATH"
>> @@ -185,5 +204,6 @@ skip $isroot "Root access is needed to set bandwith
>> limits.
>> Skipping all tests."
>> stop_lttng_relayd
>> stop_lttng_sessiond
>> reset_bw_limit
>> + ok $? "Reset bandwith limits"
>> done
>> }
>> --
>> 2.11.0
>>
>
> --
> Jonathan Rajotte-Julien
> EfficiOS
--
Mathieu Desnoyers
EfficiOS Inc.
http://www.efficios.com
_______________________________________________
lttng-dev mailing list
lttng-dev@lists.lttng.org
https://lists.lttng.org/cgi-bin/mailman/listinfo/lttng-dev
