HID: samples: fix the 2 struct_ops definitions
authorBenjamin Tissoires <bentiss@kernel.org>
Fri, 5 Jul 2024 12:09:25 +0000 (14:09 +0200)
committerBenjamin Tissoires <bentiss@kernel.org>
Mon, 8 Jul 2024 09:22:38 +0000 (11:22 +0200)
Turns out that this is not compiling anymore because the hid_bpf_ops
struct_ops definition had a change during the revisions.

Fixes: e342d6f6f7d8 ("HID: samples: convert the 2 HID-BPF samples into struct_ops")
Link: https://patch.msgid.link/20240705-for-6-11-bpf-v1-1-1960e3165c9e@kernel.org
Signed-off-by: Benjamin Tissoires <bentiss@kernel.org>
samples/hid/hid_mouse.bpf.c
samples/hid/hid_surface_dial.bpf.c

index bd901fa855c93b56196ccfaf11ebadf9348c4c1d..f7f722dcf56dbd92d49e38b9f772e6b7d970bfd9 100644 (file)
@@ -67,7 +67,7 @@ static int hid_x_event(struct hid_bpf_ctx *hctx)
        return 0;
 }
 
-SEC("struct_ops/device_event")
+SEC("struct_ops/hid_device_event")
 int BPF_PROG(hid_event, struct hid_bpf_ctx *hctx, enum hid_report_type type)
 {
        int ret = hid_y_event(hctx);
@@ -79,7 +79,7 @@ int BPF_PROG(hid_event, struct hid_bpf_ctx *hctx, enum hid_report_type type)
 }
 
 
-SEC("struct_ops/rdesc_fixup")
+SEC("struct_ops/hid_rdesc_fixup")
 int BPF_PROG(hid_rdesc_fixup, struct hid_bpf_ctx *hctx)
 {
        __u8 *data = hid_bpf_get_data(hctx, 0 /* offset */, 4096 /* size */);
@@ -121,8 +121,8 @@ int BPF_PROG(hid_rdesc_fixup, struct hid_bpf_ctx *hctx)
 
 SEC(".struct_ops.link")
 struct hid_bpf_ops mouse_invert = {
-       .rdesc_fixup = (void *)hid_rdesc_fixup,
-       .device_event = (void *)hid_event,
+       .hid_rdesc_fixup = (void *)hid_rdesc_fixup,
+       .hid_device_event = (void *)hid_event,
 };
 
 char _license[] SEC("license") = "GPL";
index d8d0fb07391faadac600c37a3e87d9ed0ab8c97f..527d584812ab1fd52282f2b1fbaf89f86dd25dd2 100644 (file)
@@ -10,7 +10,7 @@
 #define HID_UP_BUTTON          0x0009
 #define HID_GD_WHEEL           0x0038
 
-SEC("struct_ops/device_event")
+SEC("struct_ops/hid_device_event")
 int BPF_PROG(hid_event, struct hid_bpf_ctx *hctx)
 {
        __u8 *data = hid_bpf_get_data(hctx, 0 /* offset */, 9 /* size */);
@@ -101,7 +101,7 @@ int set_haptic(struct haptic_syscall_args *args)
 }
 
 /* Convert REL_DIAL into REL_WHEEL */
-SEC("struct_ops/rdesc_fixup")
+SEC("struct_ops/hid_rdesc_fixup")
 int BPF_PROG(hid_rdesc_fixup, struct hid_bpf_ctx *hctx)
 {
        __u8 *data = hid_bpf_get_data(hctx, 0 /* offset */, 4096 /* size */);
@@ -132,8 +132,8 @@ int BPF_PROG(hid_rdesc_fixup, struct hid_bpf_ctx *hctx)
 
 SEC(".struct_ops.link")
 struct hid_bpf_ops surface_dial = {
-       .rdesc_fixup = (void *)hid_rdesc_fixup,
-       .device_event = (void *)hid_event,
+       .hid_rdesc_fixup = (void *)hid_rdesc_fixup,
+       .hid_device_event = (void *)hid_event,
 };
 
 char _license[] SEC("license") = "GPL";