[strace/strace] Add lirc ioctl decoding (PR #208)

esyr notifications at github.com
Thu Feb 10 04:36:20 UTC 2022


On Mon, Feb 07, 2022 at 11:00:43AM -0800, Sean Young via Strace-devel wrote:
> Ready for review again.

Looks great, thank you.

> diff --git a/tests/ioctl_lirc.c b/tests/ioctl_lirc.c
> new file mode 100644
> index 0000000..df601e9
> --- /dev/null
> +++ b/tests/ioctl_lirc.c

> +	value = 40000;
> +	do_ioctl(LIRC_SET_REC_CARRIER_RANGE, &value);
> +	printf("ioctl(-1, IPMICTL_SET_MAINTENANCE_MODE_CMD or LIRC_SET_REC_CARRIER_RANGE, [40000]) = %s\n",
> +	       errstr);

In strace, string literals tend to be wrapped to be under 80 columns
as well:

	printf("ioctl(-1, IPMICTL_SET_MAINTENANCE_MODE_CMD"
	       " or LIRC_SET_REC_CARRIER_RANGE, [40000]) = %s\n",
	       errstr);

> +	printf("ioctl(-1, LIRC_GET_FEATURES, [LIRC_CAN_SEND_PULSE|LIRC_CAN_SET_SEND_DUTY_CYCLE|LIRC_CAN_USE_WIDEBAND_RECEIVER|LIRC_CAN_GET_REC_RESOLUTION|0x8000000]) = %s\n",
> +	       errstr);

	printf("ioctl(-1, LIRC_GET_FEATURES, [LIRC_CAN_SEND_PULSE"
	       "|LIRC_CAN_SET_SEND_DUTY_CYCLE|LIRC_CAN_USE_WIDEBAND_RECEIVER"
	       "|LIRC_CAN_GET_REC_RESOLUTION|0x8000000]) = %s\n",
	       errstr);

> +	do_ioctl(LIRC_GET_FEATURES, (unsigned int*)0x40000);

It is usually formatted as follows in strace:

	do_ioctl(LIRC_GET_FEATURES, (unsigned int *) 0x40000);

Since these are just minor formatting nits,

Reviewed-by: Eugene Syromyatnikov ***@***.***>

> 
> -- 
> Reply to this email directly or view it on GitHub:
> https://github.com/strace/strace/pull/208#issuecomment-1031810043
> You are receiving this because you are subscribed to this thread.
> 
> Message ID: ***@***.***>

> -- 
> Strace-devel mailing list
> ***@***.***
> https://lists.strace.io/mailman/listinfo/strace-devel


-- 
Reply to this email directly or view it on GitHub:
https://github.com/strace/strace/pull/208#issuecomment-1034484453
You are receiving this because you are subscribed to this thread.

Message ID: <strace/strace/pull/208/c1034484453 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.strace.io/pipermail/strace-devel/attachments/20220209/47d111de/attachment.htm>


More information about the Strace-devel mailing list