[PATCH 3/8] futex: Some refinement to unknown command argument formatting

Dmitry V. Levin ldv at altlinux.org
Mon Aug 29 17:35:33 UTC 2016


On Mon, Aug 29, 2016 at 06:12:15PM +0300, Eugene Syromyatnikov wrote:
> Several changes have been performed:
>  * alternate form is used for timeout value since it is pointer half
>    of the time. And half of the time it is not, which can be confusing
>    without number base prefix.
>  * uaddr is pointer across all existing operations, so it's rather expected
>    that it would be the same in possible future operations.
>  * val3 numeric base is also different in different commands, so it's
>    better to use alternate form for it too.
> 
> * futex.c (SYS_FUNC(futex)): update formatting in unknown command case.
> ---
>  futex.c |    5 ++++-
>  1 file changed, 4 insertions(+), 1 deletion(-)
> 
> diff --git a/futex.c b/futex.c
> index 44b90e0..6bd739f 100644
> --- a/futex.c
> +++ b/futex.c
> @@ -107,7 +107,10 @@ SYS_FUNC(futex)
>  	case FUTEX_TRYLOCK_PI:
>  		break;
>  	default:
> -		tprintf(", %lx, %lx, %x", timeout, uaddr2, val3);
> +		tprintf(", %#lx", timeout);
> +		tprints(", ");
> +		printaddr(uaddr2);
> +		tprintf(", %#x", val3);
>  		break;
>  	}
>  

Looks fine.


-- 
ldv
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://lists.strace.io/pipermail/strace-devel/attachments/20160829/8256f771/attachment.bin>


More information about the Strace-devel mailing list