While playing with hid_bpf_hw_output_report() on a device without report IDs, I realized I wasn't able to send proper reports to the USB endpoint.
This is because the __hid_bpf_hw_check_params() check wasn't taking into account the report ID reserved space, and it was trimming down one element in the report. Of course, things are not simple, because sending an input report is not taking this extra space into account, so we need to account for the type of request as well. Cheers, Benjamin Signed-off-by: Benjamin Tissoires <[email protected]> --- Benjamin Tissoires (3): selftests/hid: add define for commonly used buf size HID: bpf: fix __hid_bpf_hw_check_params report length selftests/hid: add unnumbered variant to the hid_bpf tests drivers/hid/bpf/hid_bpf_dispatch.c | 12 +++-- tools/testing/selftests/hid/hid_bpf.c | 91 ++++++++++++++++++++++---------- tools/testing/selftests/hid/hid_common.h | 25 ++++++++- tools/testing/selftests/hid/progs/hid.c | 2 +- 4 files changed, 96 insertions(+), 34 deletions(-) --- base-commit: a93f3bf4e1d60777b1659b812c9e818cfc53b449 change-id: 20260904-wip-bpf-check_report-f490de16b864 Best regards, -- Benjamin Tissoires <[email protected]>
