[ovs-dev] [PATCH] ovs-lib: Add time stamps to Valgrind log messages.

[Justin Pettit]

Looks good.

--Justin


On Jun 20, 2012, at 10:29 AM, Ben Pfaff wrote:

> Sometimes it's easier to interpret Valgrind warnings when you can
> correlate them with other events.
> 
> Suggested-by: James Schmidt <jschmidt at nicira.com>
> Signed-off-by: Ben Pfaff <blp at nicira.com>
> ---
> utilities/ovs-lib.in |    2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/utilities/ovs-lib.in b/utilities/ovs-lib.in
> index 50a5950..f8e2609 100644
> --- a/utilities/ovs-lib.in
> +++ b/utilities/ovs-lib.in
> @@ -110,7 +110,7 @@ start_daemon () {
>     case $wrapper in
>         valgrind)
>             if (valgrind --version) > /dev/null 2>&1; then
> -                set valgrind -q --leak-check=full \
> +                set valgrind -q --leak-check=full --time-stamp=yes \
>                     --log-file="$logdir/$daemon.valgrind.log.%p" "$@"
>             else
>                 log_failure_msg "valgrind not installed, running $daemon without it"
> -- 
> 1.7.2.5
> 
> _______________________________________________
> dev mailing list
> dev at openvswitch.org
> http://openvswitch.org/mailman/listinfo/dev