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

Added API to PluginKit so plugins can trigger hardware detection. #404

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

godbyk
Copy link
Contributor

@godbyk godbyk commented Mar 24, 2016

This PR addresses issue #155.

@rpavlik
Copy link
Member

rpavlik commented Sep 23, 2016

Also needs rebasing by now (sorry) and possibly analysis/commenting of thread safety - may need lock guard in case it's called from an async plugin, etc.

@godbyk godbyk force-pushed the plugin-trigger-hardware-detect branch from 2e4328a to 84b3a12 Compare September 23, 2016 17:12
@godbyk
Copy link
Contributor Author

godbyk commented Sep 23, 2016

I rebranched from master and applied this PR patch again so it should merge cleanly now.

As for thread safety, I think we're okay there because we just set a trigger-hardware-detect flag to true (in ServerImpl) and the triggering is actually executed the next time the server loops through.

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

Successfully merging this pull request may close these issues.

None yet

2 participants