-
Notifications
You must be signed in to change notification settings - Fork 7
-
Notifications
You must be signed in to change notification settings - Fork 7
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
Updating device causes mir-kiosk to crash #60
Comments
Hm. That looks like Mir querying details of a udev device that has gone away. But I can't think of any relevant code that has changed. Perhaps unrelated changes have changed the timing and exposed a race? |
@RAOF as part of the update, the device would go away and reappear so I think your assessment is correct. Please let me know if there are any other details I can provide that might help with this issue. |
I concur with @RAOF: there are no recent changes in this code, so I suspect the change is unrelated to the 2.4 release. The root cause of the crash would appear to be that the exception shown is not being handled. Unfortunately, the above log snippet doesn't indicate the context in which the "Udev device does not exist". Is there anything that might be interesting in the log immediately before or after this? |
Here are the rest of the logs I could pull out starting from when the update happens. Those unbind and bind failures are fairly common even before this issue. A few other thoughts: I've reverted to a snap version that was heavily tested and working fine before the latest mir-kisok snap was released. However, I haven't yet done the work to check if one of our Canonical maintained snaps has been updated in the same window. It's possible that a change to another snap might be the root cause. I will be exploring that further.
|
@daleknauss @RAOF came up with a likely fix that I've incorporated into a new "2.4.1~rc2-snap124" build on the If you have got an environment where you can reproduce the problem as it happens for you, please give this version a try. It's not yet available for all architectures, but it should be there soon. I'm reaching EOD so letting you know now. |
@AlanGriffiths I'm still seeing that crash on 2.4.1~rc2-snap124 (7239):
|
Thanks. I'll take another look armed with that additional information. (I'm pretty sure we fixed a problem, a shame it wasn't the problem you're seeing.) |
Oops: Overzealous automated close! |
@daleknauss there's now an I'm still failing to reproduce the actual problem you're seeing. Any hints you might have on how to reproduce would be welcome. |
@AlanGriffiths 2.4.1~rc3-snap124 appears to have resolved the issue. Thank you for resolving this so quickly. Roughly how long should we expect these fixes to make it onto stable? |
Thanks @daleknauss, I need to complete some tests and then I'll tag for release. The time to stable then depends on schedules I don't control, but should be of the order of two weeks. Meanwhile, I'd like to confirm our understanding of what went wrong. You should be getting some "warning" messages in the log (instead of a crash) with the text "Failed to handle UDev". Can you confirm that and share the log for a few lines before and after? |
@daleknauss one further thought: were you aware of the "Call for testing"? |
@daleknauss FYI I have reproduced. As a result, I found the log warning message leaves something to be desired (canonical/mir#2095), but fixing that can wait for the next full release. |
Thanks Alan. Definitely aware of calls for testing but don't always catch them when they're happening. Thank you. I'll try to see if I can get any more details on our end. |
Hello,
As part of our system, we apply firmware updates to a number of attached devices. Since around the time of the 2.4.0-snap124 snap release, we have started to see mir-kiosk crash when applying an update. Here are the relevant logs:
Any ideas?
The text was updated successfully, but these errors were encountered: