X-Git-Url: http://git.lttng.org/?a=blobdiff_plain;f=include%2Flttng%2Ftrigger%2Ftrigger-internal.h;h=6fba6668491d71dd95f9f706a818e7e915c76d1e;hb=94dbd8e4ed88cd56829159e1fef374a16fddd593;hp=87368476958f08186e5f907545764bc3a6376086;hpb=5c504c41e4d470012cb98c849aa82086d1ba1bde;p=lttng-tools.git

diff --git a/include/lttng/trigger/trigger-internal.h b/include/lttng/trigger/trigger-internal.h
index 873684769..6fba66684 100644
--- a/include/lttng/trigger/trigger-internal.h
+++ b/include/lttng/trigger/trigger-internal.h
@@ -8,13 +8,14 @@
 #ifndef LTTNG_TRIGGER_INTERNAL_H
 #define LTTNG_TRIGGER_INTERNAL_H
 
-#include <lttng/trigger/trigger.h>
 #include <common/credentials.h>
 #include <common/dynamic-array.h>
 #include <common/macros.h>
 #include <common/optional.h>
-#include <stdint.h>
+#include <lttng/lttng.h>
+#include <pthread.h>
 #include <stdbool.h>
+#include <stdint.h>
 #include <sys/types.h>
 #include <urcu/ref.h>
 
@@ -30,17 +31,31 @@ struct lttng_trigger {
 	char *name;
 	/* For now only the uid portion of the credentials is used. */
 	struct lttng_credentials creds;
-	struct {
-		enum lttng_trigger_firing_policy type;
-		uint64_t threshold;
-		uint64_t current_count;
-	} firing_policy;
 	/*
 	 * Internal use only.
 	 * The unique token passed to the tracer to identify an event-rule
 	 * notification.
 	 */
 	LTTNG_OPTIONAL(uint64_t) tracer_token;
+
+	/*
+	 * Is the trigger registered?
+	 *
+	 * This is necessary since a reference holder might be interested in the
+	 * overall state of the trigger from the point of view of its owner.
+	 *
+	 * The main user is the action executor since we want to prevent the
+	 * execution of actions related to a trigger that is unregistered.
+	 *
+	 * Not considered for `is_equal`.
+	 */
+	bool registered;
+
+	/*
+	 * The lock is used to protect against concurrent trigger execution and
+	 * trigger removal.
+	 */
+	pthread_mutex_t lock;
 };
 
 struct lttng_triggers {
@@ -60,10 +75,6 @@ struct lttng_trigger_comm {
 	uint32_t length;
 	/* Includes '\0' terminator. */
 	uint32_t name_length;
-	/* Firing policy. */
-	/* Maps to enum lttng_trigger_firing_policy. */
-	uint8_t firing_policy_type;
-	uint64_t firing_policy_threshold;
 	/* A null-terminated name, a condition, and an action follow. */
 	char payload[];
 } LTTNG_PACKED;
@@ -84,15 +95,7 @@ int lttng_trigger_serialize(const struct lttng_trigger *trigger,
 		struct lttng_payload *payload);
 
 LTTNG_HIDDEN
-const struct lttng_condition *lttng_trigger_get_const_condition(
-		const struct lttng_trigger *trigger);
-
-LTTNG_HIDDEN
-const struct lttng_action *lttng_trigger_get_const_action(
-		const struct lttng_trigger *trigger);
-
-LTTNG_HIDDEN
-bool lttng_trigger_validate(struct lttng_trigger *trigger);
+bool lttng_trigger_validate(const struct lttng_trigger *trigger);
 
 LTTNG_HIDDEN
 int lttng_trigger_assign_name(
@@ -170,18 +173,73 @@ LTTNG_HIDDEN
 void lttng_trigger_set_credentials(struct lttng_trigger *trigger,
 		const struct lttng_credentials *creds);
 
+/*
+ * Return the type of any underlying domain restriction. If no particular
+ * requirement is present, returns LTTNG_DOMAIN_NONE.
+ */
+LTTNG_HIDDEN
+enum lttng_domain_type lttng_trigger_get_underlying_domain_type_restriction(
+		const struct lttng_trigger *trigger);
 
 /*
- * Fire the trigger.
- * Increments the occurrence count.
+ * Generate any bytecode related to the trigger.
+ * On success LTTNG_OK. On error, returns lttng_error code.
  */
 LTTNG_HIDDEN
-void lttng_trigger_fire(struct lttng_trigger *trigger);
+enum lttng_error_code lttng_trigger_generate_bytecode(
+		struct lttng_trigger *trigger,
+		const struct lttng_credentials *creds);
 
 /*
- * Check if the trigger would fire.
+ * Note that the trigger object is not locked by "copy" as it is const and
+ * used with a number of 'const' triggers. If the trigger could be shared at
+ * the moment of the copy, it is the caller's responsability to lock it for
+ * the duration of the copy.
  */
 LTTNG_HIDDEN
-bool lttng_trigger_should_fire(const struct lttng_trigger *trigger);
+struct lttng_trigger *lttng_trigger_copy(const struct lttng_trigger *trigger);
+
+/*
+ * A given trigger needs a tracer notifier if
+ *  it has an event-rule condition,
+ *  AND
+ *  it has one or more sessiond-execution action.
+ */
+LTTNG_HIDDEN
+bool lttng_trigger_needs_tracer_notifier(const struct lttng_trigger *trigger);
+
+LTTNG_HIDDEN
+void lttng_trigger_set_as_registered(struct lttng_trigger *trigger);
+
+LTTNG_HIDDEN
+void lttng_trigger_set_as_unregistered(struct lttng_trigger *trigger);
+
+/*
+ * The trigger must be locked before calling lttng_trigger_is_registered.
+ *
+ * The lock is necessary since a trigger can be unregistered at any time.
+ *
+ * Manipulations requiring that the trigger be registered must always acquire
+ * the trigger lock for the duration of the manipulation using
+ * `lttng_trigger_lock` and `lttng_trigger_unlock`.
+ */
+LTTNG_HIDDEN
+bool lttng_trigger_is_registered(struct lttng_trigger *trigger);
+
+LTTNG_HIDDEN
+void lttng_trigger_lock(struct lttng_trigger *trigger);
+
+LTTNG_HIDDEN
+void lttng_trigger_unlock(struct lttng_trigger *trigger);
+
+LTTNG_HIDDEN
+enum lttng_trigger_status lttng_trigger_add_error_results(
+		const struct lttng_trigger *trigger,
+		struct lttng_error_query_results *results);
+
+LTTNG_HIDDEN
+enum lttng_trigger_status lttng_trigger_add_action_error_query_results(
+		struct lttng_trigger *trigger,
+		struct lttng_error_query_results *results);
 
 #endif /* LTTNG_TRIGGER_INTERNAL_H */