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

Bug 959 #55

Closed
wants to merge 2 commits into from
Closed

Bug 959 #55

wants to merge 2 commits into from

Conversation

PSRCode
Copy link
Contributor

@PSRCode PSRCode commented Oct 26, 2015

Add test to validate bug 959 and a fix.

This can be applied as is on stable 2.7 and require some minor work for stable 2.6.

Cheers!

@jgalar jgalar added this to the LTTng 2.8 milestone Feb 8, 2016
Refs #959

Signed-off-by: Jonathan Rajotte <jonathan.rajotte-julien@efficios.com>
This bug fix is a workaround due to the limitation of lttng_disable_event_ext
regarding the disabling of events with similar name but different
characteristics. Although lttng_disable_event_ext provide support for
disabling by name and filter string it does not support exclusion.

The loading of events is cut in 3 phases.

1 - Create all events regardless of their state.
2 - Disable all events.
3 - Enable only the events with the 'enabled' state.

Fixes #959

Signed-off-by: Jonathan Rajotte <jonathan.rajotte-julien@efficios.com>
@jgalar
Copy link
Member

jgalar commented May 16, 2016

Merged in master and stable-2.8.

Thanks!
Jérémie

@jgalar jgalar closed this May 16, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants