Project

General

Profile

Actions

Bug #357

closed

Lttng event name length became one byte shorter when with filter.

Added by huafeng Wang over 11 years ago. Updated over 11 years ago.

Status:
Resolved
Priority:
Low
Target version:
-
Start date:
10/01/2012
Due date:
% Done:

0%

Estimated time:

Description

1.(provider:event) 127:127 without filter, it succeeded.

SC-1:~ # lttng enable-event this_domain_name_has_127_characters_it_does_not_follow_the_naming_convention_and_it_is_used_for_testing_purposes_only_yeah__127:this_event_name_has_127_characters_which_is_the_maximum_number_of_character_supported_man_i_still_need_some_more_characters_127 -u
UST event this_domain_name_has_127_characters_it_does_not_follow_the_naming_convention_and_it_is_used_for_testing_purposes_only_yeah__127:this_event_name_has_127_characters_which_is_the_maximum_number_of_character_supported_man_i_still_need_some_more_characters_127 created in channel channel0

2.(provider:event) 127:127 with filter, it failed.

SC-1:~ # lttng enable-event this_domain_name_has_127_characters_it_does_not_follow_the_naming_convention_and_it_is_used_for_testing_purposes_only_yeah__127:this_event_name_has_127_characters_which_is_the_maximum_number_of_character_supported_man_i_still_need_some_more_characters_127 -u --filter 'TenPs>3'
UST event this_domain_name_has_127_characters_it_does_not_follow_the_naming_convention_and_it_is_used_for_testing_purposes_only_yeah__127:this_event_name_has_127_characters_which_is_the_maximum_number_of_character_supported_man_i_still_need_some_more_characters_127 created in channel channel0
Error: Error setting filter
LTTng Trace Control 2.1.0-rc3 - Basse Messe

3.(provider:event) 127:126 with filter, it succeeded.

SC-1:~ # lttng enable-event this_domain_name_has_127_characters_it_does_not_follow_the_naming_convention_and_it_is_used_for_testing_purposes_only_yeah__127:this_event_name_has_127_characters_which_is_the_maximum_number_of_character_supported_man_i_still_need_some_more_characters_12 -u --filter 'TenPs>3'
UST event this_domain_name_has_127_characters_it_does_not_follow_the_naming_convention_and_it_is_used_for_testing_purposes_only_yeah__127:this_event_name_has_127_characters_which_is_the_maximum_number_of_character_supported_man_i_still_need_some_more_characters_12 created in channel channel0

Why we are short of char in eventName when we have filter. ?

Actions #1

Updated by Mathieu Desnoyers over 11 years ago

  • Project changed from LTTng-UST to LTTng-tools
  • Status changed from New to Confirmed
  • Assignee set to Mathieu Desnoyers
Actions #2

Updated by Mathieu Desnoyers over 11 years ago

fixed by commit:

commit db8870edf473e2a2f69e488375d32405ea324017
Author: Mathieu Desnoyers <>
Date: Mon Oct 1 19:19:29 2012 -0400

Fix: off-by-one in comm proto between lttng-ctl and sessiond
Mostly affects setting a filter on an event with large event name.
This fix breaks the ABI between lttng-ctl and lttng-sessiond, both
should be updated together, and sessiond must be restarted.
Fixes #357
Signed-off-by: Christian Babeux &lt;&gt;
Signed-off-by: Mathieu Desnoyers &lt;
Actions #3

Updated by Mathieu Desnoyers over 11 years ago

  • Status changed from Confirmed to Resolved
Actions

Also available in: Atom PDF