[lttng-dev] [PATCH] Remove remaining semicolons in TP_fast_assign blocks

Mathieu Desnoyers mathieu.desnoyers at efficios.com
Sun Nov 25 16:14:26 EST 2012


* Andrew Gabbasov (andrew_gabbasov at mentor.com) wrote:
> Signed-off-by: Andrew Gabbasov <andrew_gabbasov at mentor.com>

Merged, thanks!

Mathieu

> ---
>  instrumentation/events/lttng-module/jbd2.h |   22 +++++++++++-----------
>  instrumentation/events/lttng-module/sock.h |    2 +-
>  2 files changed, 12 insertions(+), 12 deletions(-)
> 
> diff --git a/instrumentation/events/lttng-module/jbd2.h b/instrumentation/events/lttng-module/jbd2.h
> index 91e2e6c..c7992c0 100644
> --- a/instrumentation/events/lttng-module/jbd2.h
> +++ b/instrumentation/events/lttng-module/jbd2.h
> @@ -25,8 +25,8 @@ TRACE_EVENT(jbd2_checkpoint,
>  	),
>  
>  	TP_fast_assign(
> -		tp_assign(dev, journal->j_fs_dev->bd_dev);
> -		tp_assign(result, result);
> +		tp_assign(dev, journal->j_fs_dev->bd_dev)
> +		tp_assign(result, result)
>  	),
>  
>  	TP_printk("dev %d,%d result %d",
> @@ -46,9 +46,9 @@ DECLARE_EVENT_CLASS(jbd2_commit,
>  	),
>  
>  	TP_fast_assign(
> -		tp_assign(dev, journal->j_fs_dev->bd_dev);
> -		tp_assign(sync_commit, commit_transaction->t_synchronous_commit);
> -		tp_assign(transaction, commit_transaction->t_tid);
> +		tp_assign(dev, journal->j_fs_dev->bd_dev)
> +		tp_assign(sync_commit, commit_transaction->t_synchronous_commit)
> +		tp_assign(transaction, commit_transaction->t_tid)
>  	),
>  
>  	TP_printk("dev %d,%d transaction %d sync %d",
> @@ -97,10 +97,10 @@ TRACE_EVENT(jbd2_end_commit,
>  	),
>  
>  	TP_fast_assign(
> -		tp_assign(dev, journal->j_fs_dev->bd_dev);
> -		tp_assign(sync_commit, commit_transaction->t_synchronous_commit);
> -		tp_assign(transaction, commit_transaction->t_tid);
> -		tp_assign(head, journal->j_tail_sequence);
> +		tp_assign(dev, journal->j_fs_dev->bd_dev)
> +		tp_assign(sync_commit, commit_transaction->t_synchronous_commit)
> +		tp_assign(transaction, commit_transaction->t_tid)
> +		tp_assign(head, journal->j_tail_sequence)
>  	),
>  
>  	TP_printk("dev %d,%d transaction %d sync %d head %d",
> @@ -119,8 +119,8 @@ TRACE_EVENT(jbd2_submit_inode_data,
>  	),
>  
>  	TP_fast_assign(
> -		tp_assign(dev, inode->i_sb->s_dev);
> -		tp_assign(ino, inode->i_ino);
> +		tp_assign(dev, inode->i_sb->s_dev)
> +		tp_assign(ino, inode->i_ino)
>  	),
>  
>  	TP_printk("dev %d,%d ino %lu",
> diff --git a/instrumentation/events/lttng-module/sock.h b/instrumentation/events/lttng-module/sock.h
> index c4e689a..b0c7411 100644
> --- a/instrumentation/events/lttng-module/sock.h
> +++ b/instrumentation/events/lttng-module/sock.h
> @@ -48,7 +48,7 @@ TRACE_EVENT(sock_exceed_buf_limit,
>  		tp_assign(sysctl_mem, prot->sysctl_mem)
>  		tp_assign(allocated, allocated)
>  		tp_assign(sysctl_rmem, prot->sysctl_rmem[0])
> -		tp_assign(rmem_alloc, atomic_read(&sk->sk_rmem_alloc));
> +		tp_assign(rmem_alloc, atomic_read(&sk->sk_rmem_alloc))
>  	),
>  
>  	TP_printk("proto:%s sysctl_mem=%ld,%ld,%ld allocated=%ld "
> -- 
> 1.7.10.4
> 
> 
> _______________________________________________
> lttng-dev mailing list
> lttng-dev at lists.lttng.org
> http://lists.lttng.org/cgi-bin/mailman/listinfo/lttng-dev

-- 
Mathieu Desnoyers
Operating System Efficiency R&D Consultant
EfficiOS Inc.
http://www.efficios.com



More information about the lttng-dev mailing list