[lttng-dev] [PATCH v2 lttng-tools master, 2.10-rc] Introduce "--blocking-timeout" channel parameter

Mathieu Desnoyers mathieu.desnoyers at efficios.com
Mon Jun 12 21:57:40 UTC 2017



----- On Jun 12, 2017, at 5:28 PM, Jeremie Galarneau jeremie.galarneau at efficios.com wrote:


>>
>> +       /* Retry timeout in usec */
> 
> The unit conversion is confusing in this patch.
> 
> The user-provided (CLI) argument is in usec and is then converted to
> msec by the client, before being used as msec with liblttng-ctl
> functions. Then, it is retrieved and output as-is (in msec) here and
> in the human-readable listing.
> 
> I'll move the unit conversion to a single place (before we hand-off
> the delay to lttng-ust) since the current convention is to store and
> expose all timers in usec.
> 

As discussed, the CLI only does an overflow check, not a conversion.
Let's simply add this overflow check in the lib too, so we can
return "invalid parameter" to the caller.

> I'll also open an issue to add this property as part of the
> "save/load" functionality before the realease.

OK,

Thanks!

Mathieu

-- 
Mathieu Desnoyers
EfficiOS Inc.
http://www.efficios.com


More information about the lttng-dev mailing list