X-Git-Url: http://git.liburcu.org/?a=blobdiff_plain;f=liblttng-ust-java-agent%2Fjava%2Flttng-ust-agent-common%2Forg%2Flttng%2Fust%2Fagent%2Fclient%2FSessiondEnableEventCommand.java;h=5b36ac5d10a006c0e2ed1365a83b628f7dd631fe;hb=59e3be47eee9ed871cc60f3cee5acb3e6f327938;hp=5f0fbd948ce5801c89eab64400dc8076ebc94a6e;hpb=301a3ddb302c9c2767f41f3b47d2f3e8ca8b9067;p=lttng-ust.git

diff --git a/liblttng-ust-java-agent/java/lttng-ust-agent-common/org/lttng/ust/agent/client/SessiondEnableEventCommand.java b/liblttng-ust-java-agent/java/lttng-ust-agent-common/org/lttng/ust/agent/client/SessiondEnableEventCommand.java
index 5f0fbd94..5b36ac5d 100644
--- a/liblttng-ust-java-agent/java/lttng-ust-agent-common/org/lttng/ust/agent/client/SessiondEnableEventCommand.java
+++ b/liblttng-ust-java-agent/java/lttng-ust-agent-common/org/lttng/ust/agent/client/SessiondEnableEventCommand.java
@@ -21,7 +21,8 @@ package org.lttng.ust.agent.client;
 import java.nio.ByteBuffer;
 import java.nio.ByteOrder;
 
-import org.lttng.ust.agent.AbstractLttngAgent;
+import org.lttng.ust.agent.session.EventRule;
+import org.lttng.ust.agent.session.LogLevelSelector;
 
 /**
  * Session daemon command indicating to the Java agent that some events were
@@ -30,29 +31,48 @@ import org.lttng.ust.agent.AbstractLttngAgent;
  * @author Alexandre Montplaisir
  * @author David Goulet
  */
-class SessiondEnableEventCommand implements ISessiondCommand {
+class SessiondEnableEventCommand extends SessiondCommand {
 
-	private static final int INT_SIZE = 4;
+	/** Fixed event name length. Value defined by the lttng agent protocol. */
+	private static final int EVENT_NAME_LENGTH = 256;
 
-	/** Event name to enable in the tracing session */
+	private final boolean commandIsValid;
+
+	/* Parameters of the event rule being enabled */
 	private final String eventName;
+	private final LogLevelSelector logLevelFilter;
+	private final String filterString;
 
 	public SessiondEnableEventCommand(byte[] data) {
 		if (data == null) {
 			throw new IllegalArgumentException();
 		}
-		int dataOffset = INT_SIZE * 2;
-
 		ByteBuffer buf = ByteBuffer.wrap(data);
-		buf.order(ByteOrder.LITTLE_ENDIAN);
-		buf.getInt(); // logLevel, currently unused
-		buf.getInt(); // logLevelType, currently unused
-		eventName = new String(data, dataOffset, data.length - dataOffset).trim();
+		buf.order(ByteOrder.BIG_ENDIAN);
+		int logLevel = buf.getInt();
+		int logLevelType = buf.getInt();
+		logLevelFilter = new LogLevelSelector(logLevel, logLevelType);
+
+		/* Read the event name */
+		byte[] eventNameBytes = new byte[EVENT_NAME_LENGTH];
+		buf.get(eventNameBytes);
+		eventName = new String(eventNameBytes).trim();
+
+		/* Read the filter string */
+		filterString = readNextString(buf);
+
+		/* The command was invalid if the string could not be read correctly */
+		commandIsValid = (filterString != null);
 	}
 
 	@Override
-	public ILttngAgentResponse execute(AbstractLttngAgent<?> agent) {
-		boolean success = agent.eventEnabled(this.eventName);
-		return (success ? ILttngAgentResponse.SUCESS_RESPONSE : ILttngAgentResponse.FAILURE_RESPONSE);
+	public LttngAgentResponse execute(ILttngTcpClientListener agent) {
+		if (!commandIsValid) {
+			return LttngAgentResponse.FAILURE_RESPONSE;
+		}
+
+		EventRule rule = new EventRule(eventName, logLevelFilter, filterString);
+		boolean success = agent.eventEnabled(rule);
+		return (success ? LttngAgentResponse.SUCESS_RESPONSE : LttngAgentResponse.FAILURE_RESPONSE);
 	}
 }