Summary: | Signals being emitted from wrong thread | ||
---|---|---|---|
Product: | nice | Reporter: | Youness Alaoui <kakaroto> |
Component: | General | Assignee: | Olivier Crête <olivier.crete> |
Status: | RESOLVED MOVED | QA Contact: | |
Severity: | normal | ||
Priority: | medium | ||
Version: | unspecified | ||
Hardware: | Other | ||
OS: | All | ||
Whiteboard: | |||
i915 platform: | i915 features: |
Description
Youness Alaoui
2014-07-30 15:33:39 UTC
I tried to do that, but I ran into complications, I just can't remember what they were. I think if you run different parts of libnice in different threads, then the rest of your code must be thread safe... That said, with the new signal-after-unlock, it would be quite easy to defer the signals to a different thread. (In reply to comment #1) > I tried to do that, but I ran into complications, I just can't remember what > they were. I think if you run different parts of libnice in different > threads, then the rest of your code must be thread safe... That said, with > the new signal-after-unlock, it would be quite easy to defer the signals to > a different thread. Yeah, I was thinking of doing a if !g_main_context_is_owner, g_main_context_invoke should be fairly easy to achieve. If you can remember what issues you ran into, let me know. I've implemented it in : https://github.com/kakaroto/libnice/commit/970aad4f58aca96a22946916fee7d89bb9483d76 For some reason though the test-io-stream-closing-read unit test fails, so I disabled the change in the code. I'm unable to figure out the cause for that unit test to fail. Had any chance to figure out why it was breaking the unit tests ? Nope, I never got time to look into this any more, as I'm very busy lately (and will be for a while). If you get the time for it, I'd appreciate it if you debugged it instead (if not, it will just have to wait). Migrated to Phabricator: http://phabricator.freedesktop.org/T103 |
Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.