Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

kernel 6.8.0 compile error eventfd_signal #23

Open
robreardon opened this issue Mar 12, 2024 · 5 comments
Open

kernel 6.8.0 compile error eventfd_signal #23

robreardon opened this issue Mar 12, 2024 · 5 comments

Comments

@robreardon
Copy link

Recent change to 6.8.0 kernel requires a minor change to gasket_interrupt.c:

diff --git a/src/gasket_interrupt.c b/src/gasket_interrupt.c
index 531d94e..8dba182 100644
--- a/src/gasket_interrupt.c
+++ b/src/gasket_interrupt.c
@@ -158,7 +158,7 @@ gasket_handle_interrupt(struct gasket_interrupt_data *interrupt_data,
        read_lock(&interrupt_data->eventfd_ctx_lock);
        ctx = interrupt_data->eventfd_ctxs[interrupt_index];
        if (ctx)
-               eventfd_signal(ctx, 1);
+               eventfd_signal(ctx);
        read_unlock(&interrupt_data->eventfd_ctx_lock);

        ++(interrupt_data->interrupt_counts[interrupt_index]);
@hbh7
Copy link

hbh7 commented Apr 19, 2024

I'm on 6.8.4-2-pve with the PCIE Coral, and this worked for me with Frigate, at least as far as I can tell.

@taylorjonl
Copy link

That commit only applies the change to version >=6.8.0, I had the same issue on 6.7.9 and 6.7.12, so I changed my local copy to compare that version and it works. Not sure how to find what version of the linux kernel change the signature but it was likely lower than 6.8.0 or Debian backported it.

@jrabbit
Copy link

jrabbit commented Oct 10, 2024

When will this be addressed? 24.04.01 dropped already 6.8.x is being installed in LTS installs across the world.

@robertzaage
Copy link
Contributor

When will this be addressed? 24.04.01 dropped already 6.8.x is being installed in LTS installs across the world.

its already upstream. see #26

@EraYaN
Copy link

EraYaN commented Dec 8, 2024

Can we get this in the actual package repos?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

6 participants