vm: fixed review comments

Signed-off-by: philippe <beliasossim@gmail.com>

Author: beliosien

analyses/org.eclipse.tracecompass.incubator.executioncomparison.core/src/org/eclipse/tracecompass/incubator/internal/executioncomparison/core/DifferentialSeqCallGraphAnalysis.java

@@ -97,9 +97,6 @@ public DifferentialSeqCallGraphAnalysis() {
         // Adding VM/Native analysis
         fCallStackAnalysisMap.put("org.eclipse.linuxtools.lttng2.kernel.tracetype",  //$NON-NLS-1$
                 "org.eclipse.tracecompass.incubator.internal.virtual.machine.analysis.core.flow.analysis.vm.native.callstack"); //$NON-NLS-1$
-
-        /*fCallStackAnalysisMap.put("org.eclipse.tracecompass.tmf.core.experiment",  //$NON-NLS-1$
-                "org.eclipse.tracecompass.incubator.overhead.core.analysis.vm.native.callstack"); //$NON-NLS-1$*/
     }
 
     /**

analyses/org.eclipse.tracecompass.incubator.overhead.core.tests/.project

@@ -36,5 +36,6 @@ Export-Package: org.eclipse.tracecompass.incubator.internal.virtual.machine.anal
  org.eclipse.tracecompass.incubator.internal.virtual.machine.analysis.core.virtual.resources.handlers;x-internal:=true
 Import-Package: com.google.common.collect,
  com.google.common.hash,
- com.google.common.primitives
+ com.google.common.primitives,
+ org.eclipse.tracecompass.internal.lttng2.kernel.core.trace.layout
 Automatic-Module-Name: org.eclipse.tracecompass.incubator.virtual.machine.analysis.core

analyses/org.eclipse.tracecompass.incubator.overhead.core/.project

@@ -63,17 +63,10 @@
                class="org.eclipse.tracecompass.tmf.core.trace.TmfTrace">
          </tracetype>
       </module>
-      <module
-            analysis_module="org.eclipse.tracecompass.incubator.internal.virtual.machine.analysis.core.flow.analysis.VMNativeComparisonAnalysis"
-            id="org.eclipse.tracecompass.incubator.internal.virtual.machine.analysis.core.flow.analysis.vmcomparison"
-            name="VM vs Native Comparison">
-         <tracetype
-               class="org.eclipse.tracecompass.tmf.core.trace.experiment.TmfExperiment">
-         </tracetype>
-      </module>
       <module
             analysis_module="org.eclipse.tracecompass.incubator.internal.virtual.machine.analysis.core.flow.analysis.VMNativeCallStackAnalysis"
             applies_experiment="true"
+            automatic="false"
             id="org.eclipse.tracecompass.incubator.internal.virtual.machine.analysis.core.flow.analysis.vm.native.callstack"
             name="VM Native CallStack">
          <tracetype
@@ -84,12 +77,8 @@
    <extension
          point="org.eclipse.tracecompass.tmf.core.dataprovider">
       <dataProviderFactory
-            class="org.eclipse.tracecompass.incubator.internal.virtual.machine.analysis.core.flow.analysis.core.data.provider.KvmExitDataProviderFactory"
+            class="org.eclipse.tracecompass.incubator.internal.virtual.machine.analysis.core.flow.analysis.core.data.provider.KvmExitRateDataProviderFactory"
             id="org.eclipse.incubator.overhead.xy.rate.dataprovider">
       </dataProviderFactory>
-      <dataProviderFactory
-            class="org.eclipse.tracecompass.incubator.internal.virtual.machine.analysis.core.flow.analysis.core.data.provider.VMNativeDataProviderFactory"
-            id="org.eclipse.incubator.overhead.timegraph.dataprovider">
-      </dataProviderFactory>
    </extension>
 </plugin>

analyses/org.eclipse.tracecompass.incubator.overhead.ui.swtbot.tests/.project

@@ -1,16 +1,22 @@
+/*******************************************************************************
+ * Copyright (c) 2026 École Polytechnique de Montréal
+ *
+ * All rights reserved. This program and the accompanying materials are
+ * made available under the terms of the Eclipse Public License 2.0 which
+ * accompanies this distribution, and is available at
+ * https://www.eclipse.org/legal/epl-2.0/
+ *
+ * SPDX-License-Identifier: EPL-2.0
+ *******************************************************************************/
 package org.eclipse.tracecompass.incubator.internal.virtual.machine.analysis.core.flow.analysis;
 
 import java.util.ArrayList;
 import java.util.List;
-import java.io.File;
-import java.io.FileOutputStream;
-import java.io.IOException;
-import java.io.PrintWriter;
 
 /**
  * Represents a complete execution sequence in a virtualized environment:
  * <pre>
- * Guest → VM Exit → Hypervisor (Host) → VM Entry
+ * VM ENTRY → GUEST → VM EXIT → Hypervisor (Host) → VM Entry
  * </pre>
  * <p>
  * This class aggregates events occurring in both the guest and the hypervisor,
@@ -82,78 +88,6 @@ void setVmEntry(FlowEvent event) {
         this.vmEntry = event;
     }
 
-    /**
-     * Prints the execution sequence for debugging purposes.
-     * <p>
-     * The sequence is printed both to the standard output and appended
-     * to a file on disk.
-     * </p>
-     *
-     * @throws IOException
-     *             if an I/O error occurs while writing to the file
-     */
-    void printSequence() throws IOException {
-        try (
-            FileOutputStream fos = new FileOutputStream(
-                    new File("/home/philippe/Desktop/virtualized_flow.txt"), true); //$NON-NLS-1$
-            PrintWriter writer = new PrintWriter(fos)) {
-
-            // Print guest events
-            for (FlowEvent guestEvent : guestEvents) {
-                KernelEventInfo evt = guestEvent.kernelEvent;
-                System.out.printf("    [GUEST] %s (TID:%d, CPU:%d)\n", evt.name, evt.tid, evt.cpuid); //$NON-NLS-1$
-                writer.printf("    [GUEST] %s (TID:%d, CPU:%d)\n", evt.name, evt.tid, evt.cpuid); //$NON-NLS-1$
-            }
-
-            // Print VM exit
-            if (vmExit != null) {
-                System.out.printf("    ↓ [VM_EXIT] %s (CPU:%d, VCPU:%d, exit_reason:%s)\n", //$NON-NLS-1$
-                        vmExit.kernelEvent.name,
-                        vmExit.kernelEvent.cpuid,
-                        vmExit.kernelEvent.vcpuid,
-                        vmExit.kernelEvent.exitReason);
-
-                writer.printf("    ↓ [VM_EXIT] %s (CPU:%d, VCPU:%d, exit_reason:%s)\n", //$NON-NLS-1$
-                        vmExit.kernelEvent.name,
-                        vmExit.kernelEvent.cpuid,
-                        vmExit.kernelEvent.vcpuid,
-                        vmExit.kernelEvent.exitReason);
-            }
-
-            // Print hypervisor events
-            for (FlowEvent hypervisorEvent : hypervisorEvents) {
-                KernelEventInfo evt = hypervisorEvent.kernelEvent;
-                System.out.printf("      [HOST] %s (PID:%d, CPU:%d)\n", //$NON-NLS-1$
-                        evt.name, evt.pid, evt.cpuid);
-
-                writer.printf("      [HOST] %s (PID:%d, CPU:%d)\n", //$NON-NLS-1$
-                        evt.name, evt.pid, evt.cpuid);
-            }
-
-            // Print VM entry
-            if (vmEntry != null) {
-                System.out.printf("    ↑ [VM_ENTRY] %s (CPU:%d, VCPU:%d)\n", //$NON-NLS-1$
-                        vmEntry.kernelEvent.name,
-                        vmEntry.kernelEvent.cpuid,
-                        vmEntry.kernelEvent.vcpuid);
-
-                writer.printf("    ↑ [VM_ENTRY] %s (CPU:%d, VCPU:%d)\n", //$NON-NLS-1$
-                        vmEntry.kernelEvent.name,
-                        vmEntry.kernelEvent.cpuid,
-                        vmEntry.kernelEvent.vcpuid);
-            }
-
-            // Print timing summary
-            if (!guestEvents.isEmpty() && vmEntry != null) {
-                long totalDuration = vmEntry.kernelEvent.timestamp
-                        - guestEvents.get(0).kernelEvent.timestamp;
-
-                System.out.printf("    Total sequence duration: %d µs\n", totalDuration / 1000); //$NON-NLS-1$
-                writer.printf("    Total sequence duration: %d µs\n", totalDuration / 1000); //$NON-NLS-1$
-            }
-        }
-    }
-
     /**
      * Indicates whether the execution sequence is complete.
      * <p>