03cc58ff2a
This improves several things and addresses comments up to the diff [11] in this stack. - Simplify many functions to receive less parameters that they can identify easily - Create Storage classes for Trace and TraceIntelPT that can make it easier to reason about what can change with live process refreshes and what cannot. - Don't cache the perf zero conversion numbers in lldb-server to make sure we get the most up-to-date numbers. - Move the thread identifaction from context switches to the bundle parser, to leave TraceIntelPT simpler. This also makes sure that the constructor of TraceIntelPT is invoked when the entire data has been checked to be correct. - Normalize all bundle paths before the Processes, Threads and Modules are created, so that they can assume that all paths are correct and absolute - Fix some issues in the tests. Now they all pass. - return the specific instance when constructing PerThread and MultiCore processor tracers. - Properly implement IntelPTMultiCoreTrace::TraceStart. - Improve some comments. - Use the typedef ContextSwitchTrace more often for clarity. - Move CreateContextSwitchTracePerfEvent to Perf.h as a utility function. - Synchronize better the state of the context switch and the intel pt perf events. - Use a booblean instead of an enum for the PerfEvent state. Differential Revision: https://reviews.llvm.org/D127456
54 lines
1.7 KiB
C++
54 lines
1.7 KiB
C++
//===-- IntelPTProcessTrace.h --------------------------------- -*- C++ -*-===//
|
|
//
|
|
// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
|
|
// See https://llvm.org/LICENSE.txt for license information.
|
|
// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
|
|
//
|
|
//===----------------------------------------------------------------------===//
|
|
|
|
#ifndef liblldb_IntelPTProcessTrace_H_
|
|
#define liblldb_IntelPTProcessTrace_H_
|
|
|
|
#include "lldb/Utility/TraceIntelPTGDBRemotePackets.h"
|
|
|
|
#include <memory>
|
|
|
|
namespace lldb_private {
|
|
namespace process_linux {
|
|
|
|
/// Interface to be implemented by each 'process trace' strategy (per core, per
|
|
/// thread, etc).
|
|
class IntelPTProcessTrace {
|
|
public:
|
|
virtual ~IntelPTProcessTrace() = default;
|
|
|
|
virtual void ProcessDidStop() {}
|
|
|
|
virtual void ProcessWillResume() {}
|
|
|
|
/// Construct a minimal jLLDBTraceGetState response for this process trace.
|
|
virtual TraceIntelPTGetStateResponse GetState() = 0;
|
|
|
|
virtual bool TracesThread(lldb::tid_t tid) const = 0;
|
|
|
|
/// \copydoc IntelPTThreadTraceCollection::TraceStart()
|
|
virtual llvm::Error TraceStart(lldb::tid_t tid) = 0;
|
|
|
|
/// \copydoc IntelPTThreadTraceCollection::TraceStop()
|
|
virtual llvm::Error TraceStop(lldb::tid_t tid) = 0;
|
|
|
|
/// \return
|
|
/// \b None if this instance doesn't support the requested data, an \a
|
|
/// llvm::Error if this isntance supports it but fails at fetching it, and
|
|
/// \b Error::success() otherwise.
|
|
virtual llvm::Expected<llvm::Optional<std::vector<uint8_t>>>
|
|
TryGetBinaryData(const TraceGetBinaryDataRequest &request) = 0;
|
|
};
|
|
|
|
using IntelPTProcessTraceUP = std::unique_ptr<IntelPTProcessTrace>;
|
|
|
|
} // namespace process_linux
|
|
} // namespace lldb_private
|
|
|
|
#endif // liblldb_IntelPTProcessTrace_H_
|