[ltt-dev] [PATCH] Fix initial state bug

[Yannick Brosseau]

On 11/25/2010 10:29 AM, Francis Giraldeau wrote:
> This patch fixes a bug with the initial state in the control flow 
> viewer. When
> a process was started before starting the trace, in some situation all 
> the
> states of this process was wrong, because of a unknown initial state.

+/* get a given quark from lttng module enum value */
+#define enum_quark(e, f, names) (g_quark_from_string(names[ltt_event_get_unsigned(e, f)]))
+
  /* Channel Quarks */

The enum_quark function name sounds too generic to me. Maybe ltt_event_enum_quark or ltt_enum_quark


+/*
+ * Quark strings that matches lttng module enums found in statedump
+ * Matching enums are from lttng-modules/ltt-statedump.c
+ */
+static const char *const lttng_thread_type_names[] = {
+        "USER_THREAD",     // LTTNG_USER_THREAD
+        "KERNEL_THREAD"    // LTTNG_KERNEL_THREAD
+};

Maybe these should be in their own header file to ease the maintenance of the list ?


Yannick