Commit 33c0fb85 authored by Benjamin Tissoires's avatar Benjamin Tissoires

HID: bpf: make part of struct hid_device writable

It is useful to change the name, the phys and/or the uniq of a
struct hid_device during .rdesc_fixup().

For example, hid-uclogic.ko changes the uniq to store the firmware version
to differentiate between 2 devices sharing the same PID. In the same
way, changing the device name is useful when the device export 3 nodes,
all with the same name.

Link: https://lore.kernel.org/r/20240608-hid_bpf_struct_ops-v3-16-6ac6ade58329@kernel.orgSigned-off-by: default avatarBenjamin Tissoires <bentiss@kernel.org>
parent f1a5fb6c
...@@ -74,6 +74,9 @@ static int hid_bpf_ops_btf_struct_access(struct bpf_verifier_log *log, ...@@ -74,6 +74,9 @@ static int hid_bpf_ops_btf_struct_access(struct bpf_verifier_log *log,
const struct hid_bpf_offset_write_range write_ranges[] = { const struct hid_bpf_offset_write_range write_ranges[] = {
WRITE_RANGE(hid_bpf_ctx, retval, false), WRITE_RANGE(hid_bpf_ctx, retval, false),
WRITE_RANGE(hid_device, name, true),
WRITE_RANGE(hid_device, uniq, true),
WRITE_RANGE(hid_device, phys, true),
}; };
#undef WRITE_RANGE #undef WRITE_RANGE
const struct btf_type *state = NULL; const struct btf_type *state = NULL;
......
...@@ -43,7 +43,7 @@ struct hid_device; ...@@ -43,7 +43,7 @@ struct hid_device;
* ``hid`` and ``allocated_size`` are read-only, ``size`` and ``retval`` are read-write. * ``hid`` and ``allocated_size`` are read-only, ``size`` and ``retval`` are read-write.
*/ */
struct hid_bpf_ctx { struct hid_bpf_ctx {
const struct hid_device *hid; struct hid_device *hid;
__u32 allocated_size; __u32 allocated_size;
union { union {
__s32 retval; __s32 retval;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment