Thread (5 messages) 5 messages, 3 authors, 2d ago

Re: [PATCH v5 1/2] serial: qcom-geni: trace: Drop redundant len field from geni_serial_data

From: Konrad Dybcio <hidden>
Date: 2026-06-18 08:55:20
Also in: linux-arm-msm, linux-serial, lkml

On 6/15/26 4:16 PM, Praveen Talari wrote:
The dynamic array stored in the ring buffer already carries its own
length in the array metadata. There is no need to also store it as a
separate scalar field in the entry struct.

Drop __field(unsigned int, len) and the corresponding __entry->len
assignment, and use __get_dynamic_array_len(data) in the TP_printk for
both the len=%u format argument and the __print_hex() size argument.
This saves 4 bytes per event on the ring buffer.

Signed-off-by: Praveen Talari <redacted>
---
Suggested-by: Steven Rostedt <rostedt@goodmis.org>
Reviewed-by: Konrad Dybcio <redacted>

Konrad
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help