[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