https://github.com/DhruvSrivastavaX updated https://github.com/llvm/llvm-project/pull/118160
>From 03a290e9c748540b69ca6df7fa9c4481f9cdd3d0 Mon Sep 17 00:00:00 2001 From: Dhruv-Srivastava <dhruv.srivast...@ibm.com> Date: Sat, 30 Nov 2024 01:14:15 -0600 Subject: [PATCH 1/4] Added base files for NativeProcess for AIX --- .../source/Plugins/Process/AIX/CMakeLists.txt | 14 + .../Plugins/Process/AIX/NativeProcessAIX.cpp | 288 ++++++++++++++++++ .../Plugins/Process/AIX/NativeProcessAIX.h | 134 ++++++++ lldb/source/Plugins/Process/CMakeLists.txt | 3 + 4 files changed, 439 insertions(+) create mode 100644 lldb/source/Plugins/Process/AIX/CMakeLists.txt create mode 100644 lldb/source/Plugins/Process/AIX/NativeProcessAIX.cpp create mode 100644 lldb/source/Plugins/Process/AIX/NativeProcessAIX.h diff --git a/lldb/source/Plugins/Process/AIX/CMakeLists.txt b/lldb/source/Plugins/Process/AIX/CMakeLists.txt new file mode 100644 index 00000000000000..4fabbe93022870 --- /dev/null +++ b/lldb/source/Plugins/Process/AIX/CMakeLists.txt @@ -0,0 +1,14 @@ +add_lldb_library(lldbPluginProcessAIX + NativeProcessAIX.cpp + + LINK_LIBS + lldbCore + lldbHost + lldbSymbol + lldbTarget + lldbUtility + lldbPluginProcessPOSIX + lldbPluginProcessUtility + LINK_COMPONENTS + Support + ) diff --git a/lldb/source/Plugins/Process/AIX/NativeProcessAIX.cpp b/lldb/source/Plugins/Process/AIX/NativeProcessAIX.cpp new file mode 100644 index 00000000000000..6661693b2fc02b --- /dev/null +++ b/lldb/source/Plugins/Process/AIX/NativeProcessAIX.cpp @@ -0,0 +1,288 @@ +//===-- NativeProcessAIX.cpp --------------------------------------------===// +// +// 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 +// +//===----------------------------------------------------------------------===// + +#include "NativeProcessAIX.h" + +#include <cerrno> +#include <cstdint> +#include <cstring> +#include <unistd.h> +#include <fstream> +#include <mutex> +#include <optional> +#include <sstream> +#include <string> +#include <unordered_map> +#include "NativeThreadAIX.h" +#include "Plugins/Process/POSIX/ProcessPOSIXLog.h" +#include "lldb/Core/ModuleSpec.h" +#include "lldb/Host/Host.h" +#include "lldb/Host/HostProcess.h" +#include "lldb/Host/ProcessLaunchInfo.h" +#include "lldb/Host/PseudoTerminal.h" +#include "lldb/Host/ThreadLauncher.h" +#include "lldb/Host/common/NativeRegisterContext.h" +#include "lldb/Host/aix/Ptrace.h" +#include "lldb/Host/posix/ProcessLauncherPosixFork.h" +#include "lldb/Symbol/ObjectFile.h" +#include "lldb/Target/Process.h" +#include "lldb/Target/Target.h" +#include "lldb/Utility/LLDBAssert.h" +#include "lldb/Utility/LLDBLog.h" +#include "lldb/Utility/RegisterValue.h" +#include "lldb/Utility/State.h" +#include "lldb/Utility/Status.h" +#include "lldb/Utility/StringExtractor.h" +#include "llvm/ADT/ScopeExit.h" +#include "llvm/Support/Errno.h" +#include "llvm/Support/Error.h" +#include "llvm/Support/FileSystem.h" +#include "llvm/Support/Threading.h" +#include <sys/reg.h> +#include <sys/ptrace.h> +#include <sys/ldr.h> +#include <sys/socket.h> +#include <sys/types.h> +#include <sys/user.h> +#include <sys/wait.h> +#include <sys/mman.h> +#ifdef __aarch64__ +#include <asm/hwcap.h> +#include <sys/auxv.h> +#endif + +// Support hardware breakpoints in case it has not been defined +#ifndef TRAP_HWBKPT +#define TRAP_HWBKPT 4 +#endif + +#ifndef HWCAP2_MTE +#define HWCAP2_MTE (1 << 18) +#endif + +using namespace lldb; +using namespace lldb_private; +using namespace lldb_private::process_aix; +using namespace llvm; + +static constexpr unsigned k_ptrace_word_size = sizeof(void *); +static_assert(sizeof(long) >= k_ptrace_word_size, + "Size of long must be larger than ptrace word size"); + +// Simple helper function to ensure flags are enabled on the given file +// descriptor. +static Status EnsureFDFlags(int fd, int flags) { + Status error; + + int status = fcntl(fd, F_GETFL); + if (status == -1) { + error = Status::FromErrno(); + return error; + } + + if (fcntl(fd, F_SETFL, status | flags) == -1) { + error = Status::FromErrno(); + return error; + } + + return error; +} + +NativeProcessAIX::Manager::Manager(MainLoop &mainloop) + : NativeProcessProtocol::Manager(mainloop) { + Status status; + m_sigchld_handle = mainloop.RegisterSignal( + SIGCHLD, [this](MainLoopBase &) { SigchldHandler(); }, status); + assert(m_sigchld_handle && status.Success()); +} + +// Public Static Methods + +llvm::Expected<std::unique_ptr<NativeProcessProtocol>> +NativeProcessAIX::Manager::Launch(ProcessLaunchInfo &launch_info, + NativeDelegate &native_delegate) { + Log *log = GetLog(POSIXLog::Process); + + Status status; + ::pid_t pid = ProcessLauncherPosixFork() + .LaunchProcess(launch_info, status) + .GetProcessId(); + LLDB_LOG(log, "pid = {0:x}", pid); + if (status.Fail()) { + LLDB_LOG(log, "failed to launch process: {0}", status); + return status.ToError(); + } + + // Wait for the child process to trap on its call to execve. + int wstatus = 0; + ::pid_t wpid = llvm::sys::RetryAfterSignal(-1, ::waitpid, pid, &wstatus, 0); + assert(wpid == pid); + UNUSED_IF_ASSERT_DISABLED(wpid); + if (!WIFSTOPPED(wstatus)) { + LLDB_LOG(log, "Could not sync with inferior process: wstatus={1}", + WaitStatus::Decode(wstatus)); + return llvm::make_error<StringError>("Could not sync with inferior process", + llvm::inconvertibleErrorCode()); + } + LLDB_LOG(log, "inferior started, now in stopped state"); + + ProcessInstanceInfo Info; + if (!Host::GetProcessInfo(pid, Info)) { + return llvm::make_error<StringError>("Cannot get process architectrue", + llvm::inconvertibleErrorCode()); + } + + // Set the architecture to the exe architecture. + LLDB_LOG(log, "pid = {0}, detected architecture {1}", pid, + Info.GetArchitecture().GetArchitectureName()); + + return std::unique_ptr<NativeProcessAIX>(new NativeProcessAIX( + pid, launch_info.GetPTY().ReleasePrimaryFileDescriptor(), native_delegate, + Info.GetArchitecture(), *this, {pid})); +} + +llvm::Expected<std::unique_ptr<NativeProcessProtocol>> +NativeProcessAIX::Manager::Attach( + lldb::pid_t pid, NativeProcessProtocol::NativeDelegate &native_delegate) { + Log *log = GetLog(POSIXLog::Process); + LLDB_LOG(log, "pid = {0:x}", pid); + + ProcessInstanceInfo Info; + if (!Host::GetProcessInfo(pid, Info)) { + return llvm::make_error<StringError>("Cannot get process architectrue", + llvm::inconvertibleErrorCode()); + } + auto tids_or = NativeProcessAIX::Attach(pid); + if (!tids_or) + return tids_or.takeError(); + + return std::unique_ptr<NativeProcessAIX>( + new NativeProcessAIX(pid, -1, native_delegate, Info.GetArchitecture(), *this, *tids_or)); +} + +NativeProcessAIX::Extension +NativeProcessAIX::Manager::GetSupportedExtensions() const { + NativeProcessAIX::Extension supported = + Extension::multiprocess | Extension::fork | Extension::vfork | + Extension::pass_signals | Extension::auxv | Extension::libraries_svr4 | + Extension::siginfo_read; + + return supported; +} + +void NativeProcessAIX::Manager::SigchldHandler() { +} + +void NativeProcessAIX::Manager::CollectThread(::pid_t tid) { +} + +// Public Instance Methods + +NativeProcessAIX::NativeProcessAIX(::pid_t pid, int terminal_fd, + NativeDelegate &delegate, + const ArchSpec &arch, Manager &manager, + llvm::ArrayRef<::pid_t> tids) + : NativeProcessProtocol(pid, terminal_fd, delegate), m_manager(manager), + m_arch(arch) { + manager.AddProcess(*this); + if (m_terminal_fd != -1) { + Status status = EnsureFDFlags(m_terminal_fd, O_NONBLOCK); + assert(status.Success()); + } + + // Let our process instance know the thread has stopped. + SetCurrentThreadID(tids[0]); + SetState(StateType::eStateStopped, false); +} + +llvm::Expected<std::vector<::pid_t>> NativeProcessAIX::Attach(::pid_t pid) { + + Status status; + if ((status = PtraceWrapper(PT_ATTACH, pid)).Fail()) { + return status.ToError(); + } + + std::vector<::pid_t> tids; + tids.push_back(pid); + return std::move(tids); +} + +void NativeProcessAIX::MonitorSIGTRAP(const WaitStatus status, + NativeThreadAIX &thread) { +} + +void NativeProcessAIX::MonitorBreakpoint(NativeThreadAIX &thread) { +} + +bool NativeProcessAIX::SupportHardwareSingleStepping() const { + return false; +} + +Status NativeProcessAIX::Resume(const ResumeActionList &resume_actions) { + return Status(); +} + +Status NativeProcessAIX::Halt() { + Status error; + return error; +} + +Status NativeProcessAIX::Detach() { + Status error; + return error; +} + +Status NativeProcessAIX::Signal(int signo) { + Status error; + return error; +} + +Status NativeProcessAIX::Interrupt() { + return Status(); +} + +Status NativeProcessAIX::Kill() { + Status error; + return error; +} + +Status NativeProcessAIX::SetBreakpoint(lldb::addr_t addr, uint32_t size, + bool hardware) { + if (hardware) + return SetHardwareBreakpoint(addr, size); + else + return SetSoftwareBreakpoint(addr, size); +} + +Status NativeProcessAIX::RemoveBreakpoint(lldb::addr_t addr, bool hardware) { + if (hardware) + return RemoveHardwareBreakpoint(addr); + else + return NativeProcessProtocol::RemoveBreakpoint(addr); +} + +int8_t NativeProcessAIX::GetSignalInfo(WaitStatus wstatus) const { + return wstatus.status; +} + +Status NativeProcessAIX::Detach(lldb::tid_t tid) { + if (tid == LLDB_INVALID_THREAD_ID) + return Status(); + + return PtraceWrapper(PT_DETACH, tid); +} + +// Wrapper for ptrace to catch errors and log calls. Note that ptrace sets +// errno on error because -1 can be a valid result (i.e. for PTRACE_PEEK*) +Status NativeProcessAIX::PtraceWrapper(int req, lldb::pid_t pid, void *addr, + void *data, size_t data_size, + long *result) { + Status error; + return error; +} + diff --git a/lldb/source/Plugins/Process/AIX/NativeProcessAIX.h b/lldb/source/Plugins/Process/AIX/NativeProcessAIX.h new file mode 100644 index 00000000000000..2c07ba420d3fee --- /dev/null +++ b/lldb/source/Plugins/Process/AIX/NativeProcessAIX.h @@ -0,0 +1,134 @@ +//===-- NativeProcessAIX.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_NativeProcessAIX_H_ +#define liblldb_NativeProcessAIX_H_ + +#include <csignal> +#include <unordered_set> + +#include "lldb/Host/Debug.h" +#include "lldb/Host/HostThread.h" +#include "lldb/Target/MemoryRegionInfo.h" +#include "lldb/Utility/ArchSpec.h" +#include "lldb/Utility/FileSpec.h" +#include "lldb/lldb-types.h" +#include "llvm/ADT/SmallPtrSet.h" +#include "lldb/Host/aix/Support.h" + +#include "NativeThreadAIX.h" +#include "lldb/Host/common/NativeProcessProtocol.h" +#include "Plugins/Process/Utility/NativeProcessSoftwareSingleStep.h" + +namespace lldb_private { +class Status; +class Scalar; + +namespace process_aix { +/// \class NativeProcessAIX +/// Manages communication with the inferior (debugee) process. +/// +/// Upon construction, this class prepares and launches an inferior process +/// for debugging. +/// +/// Changes in the inferior process state are broadcasted. +class NativeProcessAIX : public NativeProcessProtocol, + private NativeProcessSoftwareSingleStep { +public: + class Manager : public NativeProcessProtocol::Manager { + public: + Manager(MainLoop &mainloop); + + llvm::Expected<std::unique_ptr<NativeProcessProtocol>> + Launch(ProcessLaunchInfo &launch_info, + NativeDelegate &native_delegate) override; + + llvm::Expected<std::unique_ptr<NativeProcessProtocol>> + Attach(lldb::pid_t pid, NativeDelegate &native_delegate) override; + + Extension GetSupportedExtensions() const override; + + void AddProcess(NativeProcessAIX &process) { + m_processes.insert(&process); + } + + void RemoveProcess(NativeProcessAIX &process) { + m_processes.erase(&process); + } + + // Collect an event for the given tid, waiting for it if necessary. + void CollectThread(::pid_t tid); + + private: + MainLoop::SignalHandleUP m_sigchld_handle; + + llvm::SmallPtrSet<NativeProcessAIX *, 2> m_processes; + + void SigchldHandler(); + }; + + // NativeProcessProtocol Interface + + ~NativeProcessAIX() override { m_manager.RemoveProcess(*this); } + + Status Resume(const ResumeActionList &resume_actions) override; + + Status Halt() override; + + Status Detach() override; + + Status Signal(int signo) override; + + Status Interrupt() override; + + Status Kill() override; + + const ArchSpec &GetArchitecture() const override { return m_arch; } + + Status SetBreakpoint(lldb::addr_t addr, uint32_t size, + bool hardware) override; + + Status RemoveBreakpoint(lldb::addr_t addr, bool hardware = false) override; + + Status GetFileLoadAddress(const llvm::StringRef &file_name, + lldb::addr_t &load_addr) override; + + static Status PtraceWrapper(int req, lldb::pid_t pid, void *addr = nullptr, + void *data = nullptr, size_t data_size = 0, + long *result = nullptr); + +// Wrapper for ptrace to catch errors and log calls. Note that ptrace sets + +private: + Manager &m_manager; + /*MainLoop::SignalHandleUP m_sigchld_handle;*/ + ArchSpec m_arch; + /*MainLoop& m_main_loop;*/ + + // Private Instance Methods + NativeProcessAIX(::pid_t pid, int terminal_fd, NativeDelegate &delegate, + const ArchSpec &arch, Manager &manager, + llvm::ArrayRef<::pid_t> tids); + + // Returns a list of process threads that we have attached to. + static llvm::Expected<std::vector<::pid_t>> Attach(::pid_t pid); + + void MonitorSIGTRAP(const WaitStatus status, NativeThreadAIX &thread); + + void MonitorBreakpoint(NativeThreadAIX &thread); + + Status Detach(lldb::tid_t tid); + + void SigchldHandler(); + +}; + +} // namespace process_aix +} // namespace lldb_private + +#endif // #ifndef liblldb_NativeProcessAIX_H_ diff --git a/lldb/source/Plugins/Process/CMakeLists.txt b/lldb/source/Plugins/Process/CMakeLists.txt index a51d0f7afd1759..01bb5f462eba44 100644 --- a/lldb/source/Plugins/Process/CMakeLists.txt +++ b/lldb/source/Plugins/Process/CMakeLists.txt @@ -7,6 +7,9 @@ elseif (CMAKE_SYSTEM_NAME MATCHES "FreeBSD") elseif (CMAKE_SYSTEM_NAME MATCHES "NetBSD") add_subdirectory(NetBSD) add_subdirectory(POSIX) +elseif (CMAKE_SYSTEM_NAME MATCHES "AIX") + add_subdirectory(AIX) + add_subdirectory(POSIX) elseif (CMAKE_SYSTEM_NAME MATCHES "Windows") add_subdirectory(Windows/Common) elseif (CMAKE_SYSTEM_NAME MATCHES "Darwin") >From 3462b9263d4f268a89baf4b956b8dc851858fe0a Mon Sep 17 00:00:00 2001 From: Dhruv-Srivastava <dhruv.srivast...@ibm.com> Date: Sat, 30 Nov 2024 01:32:53 -0600 Subject: [PATCH 2/4] Added base files for NativeProcess for AIX --- .../Plugins/Process/AIX/NativeProcessAIX.cpp | 31 ------------------- 1 file changed, 31 deletions(-) diff --git a/lldb/source/Plugins/Process/AIX/NativeProcessAIX.cpp b/lldb/source/Plugins/Process/AIX/NativeProcessAIX.cpp index 6661693b2fc02b..a8d54483239ad2 100644 --- a/lldb/source/Plugins/Process/AIX/NativeProcessAIX.cpp +++ b/lldb/source/Plugins/Process/AIX/NativeProcessAIX.cpp @@ -7,7 +7,6 @@ //===----------------------------------------------------------------------===// #include "NativeProcessAIX.h" - #include <cerrno> #include <cstdint> #include <cstring> @@ -20,50 +19,20 @@ #include <unordered_map> #include "NativeThreadAIX.h" #include "Plugins/Process/POSIX/ProcessPOSIXLog.h" -#include "lldb/Core/ModuleSpec.h" #include "lldb/Host/Host.h" #include "lldb/Host/HostProcess.h" #include "lldb/Host/ProcessLaunchInfo.h" -#include "lldb/Host/PseudoTerminal.h" -#include "lldb/Host/ThreadLauncher.h" -#include "lldb/Host/common/NativeRegisterContext.h" -#include "lldb/Host/aix/Ptrace.h" -#include "lldb/Host/posix/ProcessLauncherPosixFork.h" #include "lldb/Symbol/ObjectFile.h" #include "lldb/Target/Process.h" #include "lldb/Target/Target.h" #include "lldb/Utility/LLDBAssert.h" #include "lldb/Utility/LLDBLog.h" -#include "lldb/Utility/RegisterValue.h" #include "lldb/Utility/State.h" #include "lldb/Utility/Status.h" #include "lldb/Utility/StringExtractor.h" -#include "llvm/ADT/ScopeExit.h" #include "llvm/Support/Errno.h" #include "llvm/Support/Error.h" #include "llvm/Support/FileSystem.h" -#include "llvm/Support/Threading.h" -#include <sys/reg.h> -#include <sys/ptrace.h> -#include <sys/ldr.h> -#include <sys/socket.h> -#include <sys/types.h> -#include <sys/user.h> -#include <sys/wait.h> -#include <sys/mman.h> -#ifdef __aarch64__ -#include <asm/hwcap.h> -#include <sys/auxv.h> -#endif - -// Support hardware breakpoints in case it has not been defined -#ifndef TRAP_HWBKPT -#define TRAP_HWBKPT 4 -#endif - -#ifndef HWCAP2_MTE -#define HWCAP2_MTE (1 << 18) -#endif using namespace lldb; using namespace lldb_private; >From e76b37b681150f7cf7ad498f405e9535d6be234e Mon Sep 17 00:00:00 2001 From: Dhruv-Srivastava <dhruv.srivast...@ibm.com> Date: Sat, 30 Nov 2024 01:55:36 -0600 Subject: [PATCH 3/4] Added base files for NativeProcess for AIX --- .../Plugins/Process/AIX/NativeProcessAIX.cpp | 79 ++++++++----------- 1 file changed, 35 insertions(+), 44 deletions(-) diff --git a/lldb/source/Plugins/Process/AIX/NativeProcessAIX.cpp b/lldb/source/Plugins/Process/AIX/NativeProcessAIX.cpp index a8d54483239ad2..28f56238538785 100644 --- a/lldb/source/Plugins/Process/AIX/NativeProcessAIX.cpp +++ b/lldb/source/Plugins/Process/AIX/NativeProcessAIX.cpp @@ -7,16 +7,6 @@ //===----------------------------------------------------------------------===// #include "NativeProcessAIX.h" -#include <cerrno> -#include <cstdint> -#include <cstring> -#include <unistd.h> -#include <fstream> -#include <mutex> -#include <optional> -#include <sstream> -#include <string> -#include <unordered_map> #include "NativeThreadAIX.h" #include "Plugins/Process/POSIX/ProcessPOSIXLog.h" #include "lldb/Host/Host.h" @@ -33,6 +23,16 @@ #include "llvm/Support/Errno.h" #include "llvm/Support/Error.h" #include "llvm/Support/FileSystem.h" +#include <cerrno> +#include <cstdint> +#include <cstring> +#include <fstream> +#include <mutex> +#include <optional> +#include <sstream> +#include <string> +#include <unistd.h> +#include <unordered_map> using namespace lldb; using namespace lldb_private; @@ -74,7 +74,7 @@ NativeProcessAIX::Manager::Manager(MainLoop &mainloop) llvm::Expected<std::unique_ptr<NativeProcessProtocol>> NativeProcessAIX::Manager::Launch(ProcessLaunchInfo &launch_info, - NativeDelegate &native_delegate) { + NativeDelegate &native_delegate) { Log *log = GetLog(POSIXLog::Process); Status status; @@ -102,13 +102,13 @@ NativeProcessAIX::Manager::Launch(ProcessLaunchInfo &launch_info, ProcessInstanceInfo Info; if (!Host::GetProcessInfo(pid, Info)) { - return llvm::make_error<StringError>("Cannot get process architectrue", - llvm::inconvertibleErrorCode()); - } + return llvm::make_error<StringError>("Cannot get process architectrue", + llvm::inconvertibleErrorCode()); + } - // Set the architecture to the exe architecture. - LLDB_LOG(log, "pid = {0}, detected architecture {1}", pid, - Info.GetArchitecture().GetArchitectureName()); + // Set the architecture to the exe architecture. + LLDB_LOG(log, "pid = {0}, detected architecture {1}", pid, + Info.GetArchitecture().GetArchitectureName()); return std::unique_ptr<NativeProcessAIX>(new NativeProcessAIX( pid, launch_info.GetPTY().ReleasePrimaryFileDescriptor(), native_delegate, @@ -117,21 +117,21 @@ NativeProcessAIX::Manager::Launch(ProcessLaunchInfo &launch_info, llvm::Expected<std::unique_ptr<NativeProcessProtocol>> NativeProcessAIX::Manager::Attach( - lldb::pid_t pid, NativeProcessProtocol::NativeDelegate &native_delegate) { + lldb::pid_t pid, NativeProcessProtocol::NativeDelegate &native_delegate) { Log *log = GetLog(POSIXLog::Process); LLDB_LOG(log, "pid = {0:x}", pid); ProcessInstanceInfo Info; if (!Host::GetProcessInfo(pid, Info)) { - return llvm::make_error<StringError>("Cannot get process architectrue", - llvm::inconvertibleErrorCode()); - } + return llvm::make_error<StringError>("Cannot get process architectrue", + llvm::inconvertibleErrorCode()); + } auto tids_or = NativeProcessAIX::Attach(pid); if (!tids_or) return tids_or.takeError(); - return std::unique_ptr<NativeProcessAIX>( - new NativeProcessAIX(pid, -1, native_delegate, Info.GetArchitecture(), *this, *tids_or)); + return std::unique_ptr<NativeProcessAIX>(new NativeProcessAIX( + pid, -1, native_delegate, Info.GetArchitecture(), *this, *tids_or)); } NativeProcessAIX::Extension @@ -144,18 +144,16 @@ NativeProcessAIX::Manager::GetSupportedExtensions() const { return supported; } -void NativeProcessAIX::Manager::SigchldHandler() { -} +void NativeProcessAIX::Manager::SigchldHandler() {} -void NativeProcessAIX::Manager::CollectThread(::pid_t tid) { -} +void NativeProcessAIX::Manager::CollectThread(::pid_t tid) {} // Public Instance Methods NativeProcessAIX::NativeProcessAIX(::pid_t pid, int terminal_fd, - NativeDelegate &delegate, - const ArchSpec &arch, Manager &manager, - llvm::ArrayRef<::pid_t> tids) + NativeDelegate &delegate, + const ArchSpec &arch, Manager &manager, + llvm::ArrayRef<::pid_t> tids) : NativeProcessProtocol(pid, terminal_fd, delegate), m_manager(manager), m_arch(arch) { manager.AddProcess(*this); @@ -182,15 +180,11 @@ llvm::Expected<std::vector<::pid_t>> NativeProcessAIX::Attach(::pid_t pid) { } void NativeProcessAIX::MonitorSIGTRAP(const WaitStatus status, - NativeThreadAIX &thread) { -} + NativeThreadAIX &thread) {} -void NativeProcessAIX::MonitorBreakpoint(NativeThreadAIX &thread) { -} +void NativeProcessAIX::MonitorBreakpoint(NativeThreadAIX &thread) {} -bool NativeProcessAIX::SupportHardwareSingleStepping() const { - return false; -} +bool NativeProcessAIX::SupportHardwareSingleStepping() const { return false; } Status NativeProcessAIX::Resume(const ResumeActionList &resume_actions) { return Status(); @@ -211,9 +205,7 @@ Status NativeProcessAIX::Signal(int signo) { return error; } -Status NativeProcessAIX::Interrupt() { - return Status(); -} +Status NativeProcessAIX::Interrupt() { return Status(); } Status NativeProcessAIX::Kill() { Status error; @@ -221,7 +213,7 @@ Status NativeProcessAIX::Kill() { } Status NativeProcessAIX::SetBreakpoint(lldb::addr_t addr, uint32_t size, - bool hardware) { + bool hardware) { if (hardware) return SetHardwareBreakpoint(addr, size); else @@ -249,9 +241,8 @@ Status NativeProcessAIX::Detach(lldb::tid_t tid) { // Wrapper for ptrace to catch errors and log calls. Note that ptrace sets // errno on error because -1 can be a valid result (i.e. for PTRACE_PEEK*) Status NativeProcessAIX::PtraceWrapper(int req, lldb::pid_t pid, void *addr, - void *data, size_t data_size, - long *result) { + void *data, size_t data_size, + long *result) { Status error; return error; } - >From 1284f0b9099fcd9bb89bcfed29af3ab9904e69e8 Mon Sep 17 00:00:00 2001 From: Dhruv-Srivastava <dhruv.srivast...@ibm.com> Date: Sat, 30 Nov 2024 02:08:55 -0600 Subject: [PATCH 4/4] added missing format --- .../Plugins/Process/AIX/NativeProcessAIX.h | 18 +++++++----------- 1 file changed, 7 insertions(+), 11 deletions(-) diff --git a/lldb/source/Plugins/Process/AIX/NativeProcessAIX.h b/lldb/source/Plugins/Process/AIX/NativeProcessAIX.h index 2c07ba420d3fee..3532f1c1cacd4a 100644 --- a/lldb/source/Plugins/Process/AIX/NativeProcessAIX.h +++ b/lldb/source/Plugins/Process/AIX/NativeProcessAIX.h @@ -19,11 +19,10 @@ #include "lldb/Utility/FileSpec.h" #include "lldb/lldb-types.h" #include "llvm/ADT/SmallPtrSet.h" -#include "lldb/Host/aix/Support.h" #include "NativeThreadAIX.h" -#include "lldb/Host/common/NativeProcessProtocol.h" #include "Plugins/Process/Utility/NativeProcessSoftwareSingleStep.h" +#include "lldb/Host/common/NativeProcessProtocol.h" namespace lldb_private { class Status; @@ -38,7 +37,7 @@ namespace process_aix { /// /// Changes in the inferior process state are broadcasted. class NativeProcessAIX : public NativeProcessProtocol, - private NativeProcessSoftwareSingleStep { + private NativeProcessSoftwareSingleStep { public: class Manager : public NativeProcessProtocol::Manager { public: @@ -46,16 +45,14 @@ class NativeProcessAIX : public NativeProcessProtocol, llvm::Expected<std::unique_ptr<NativeProcessProtocol>> Launch(ProcessLaunchInfo &launch_info, - NativeDelegate &native_delegate) override; + NativeDelegate &native_delegate) override; llvm::Expected<std::unique_ptr<NativeProcessProtocol>> Attach(lldb::pid_t pid, NativeDelegate &native_delegate) override; Extension GetSupportedExtensions() const override; - void AddProcess(NativeProcessAIX &process) { - m_processes.insert(&process); - } + void AddProcess(NativeProcessAIX &process) { m_processes.insert(&process); } void RemoveProcess(NativeProcessAIX &process) { m_processes.erase(&process); @@ -102,7 +99,7 @@ class NativeProcessAIX : public NativeProcessProtocol, void *data = nullptr, size_t data_size = 0, long *result = nullptr); -// Wrapper for ptrace to catch errors and log calls. Note that ptrace sets + // Wrapper for ptrace to catch errors and log calls. Note that ptrace sets private: Manager &m_manager; @@ -112,8 +109,8 @@ class NativeProcessAIX : public NativeProcessProtocol, // Private Instance Methods NativeProcessAIX(::pid_t pid, int terminal_fd, NativeDelegate &delegate, - const ArchSpec &arch, Manager &manager, - llvm::ArrayRef<::pid_t> tids); + const ArchSpec &arch, Manager &manager, + llvm::ArrayRef<::pid_t> tids); // Returns a list of process threads that we have attached to. static llvm::Expected<std::vector<::pid_t>> Attach(::pid_t pid); @@ -125,7 +122,6 @@ class NativeProcessAIX : public NativeProcessProtocol, Status Detach(lldb::tid_t tid); void SigchldHandler(); - }; } // namespace process_aix _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
