mirror of
				https://git.zaroz.cloud/nintendo-back-up/yuzu/yuzu-mainline.git
				synced 2025-03-21 01:53:15 +00:00 
			
		
		
		
	Merge pull request #401 from lioncash/gdbstub
core/gdbstub: Move logging macros to new fmt-compatible ones
This commit is contained in:
		
						commit
						3dd3cdeafd
					
				@ -6,7 +6,6 @@
 | 
			
		||||
 | 
			
		||||
#include <algorithm>
 | 
			
		||||
#include <atomic>
 | 
			
		||||
#include <cinttypes>
 | 
			
		||||
#include <climits>
 | 
			
		||||
#include <csignal>
 | 
			
		||||
#include <cstdarg>
 | 
			
		||||
@ -180,7 +179,7 @@ static u8 HexCharToValue(u8 hex) {
 | 
			
		||||
        return hex - 'A' + 0xA;
 | 
			
		||||
    }
 | 
			
		||||
 | 
			
		||||
    LOG_ERROR(Debug_GDBStub, "Invalid nibble: %c (%02x)\n", hex, hex);
 | 
			
		||||
    NGLOG_ERROR(Debug_GDBStub, "Invalid nibble: {} ({:02X})", hex, hex);
 | 
			
		||||
    return 0;
 | 
			
		||||
}
 | 
			
		||||
 | 
			
		||||
@ -320,7 +319,7 @@ static u8 ReadByte() {
 | 
			
		||||
    u8 c;
 | 
			
		||||
    size_t received_size = recv(gdbserver_socket, reinterpret_cast<char*>(&c), 1, MSG_WAITALL);
 | 
			
		||||
    if (received_size != 1) {
 | 
			
		||||
        LOG_ERROR(Debug_GDBStub, "recv failed : %ld", received_size);
 | 
			
		||||
        NGLOG_ERROR(Debug_GDBStub, "recv failed: {}", received_size);
 | 
			
		||||
        Shutdown();
 | 
			
		||||
    }
 | 
			
		||||
 | 
			
		||||
@ -361,9 +360,8 @@ static void RemoveBreakpoint(BreakpointType type, PAddr addr) {
 | 
			
		||||
 | 
			
		||||
    auto bp = p.find(static_cast<u64>(addr));
 | 
			
		||||
    if (bp != p.end()) {
 | 
			
		||||
        LOG_DEBUG(Debug_GDBStub,
 | 
			
		||||
                  "gdb: removed a breakpoint: %016" PRIx64 " bytes at %016" PRIx64 " of type %d\n",
 | 
			
		||||
                  bp->second.len, bp->second.addr, static_cast<int>(type));
 | 
			
		||||
        NGLOG_DEBUG(Debug_GDBStub, "gdb: removed a breakpoint: {:016X} bytes at {:016X} of type {}",
 | 
			
		||||
                    bp->second.len, bp->second.addr, static_cast<int>(type));
 | 
			
		||||
        p.erase(static_cast<u64>(addr));
 | 
			
		||||
    }
 | 
			
		||||
}
 | 
			
		||||
@ -408,10 +406,10 @@ bool CheckBreakpoint(PAddr addr, BreakpointType type) {
 | 
			
		||||
        }
 | 
			
		||||
 | 
			
		||||
        if (bp->second.active && (addr >= bp->second.addr && addr < bp->second.addr + len)) {
 | 
			
		||||
            LOG_DEBUG(Debug_GDBStub,
 | 
			
		||||
                      "Found breakpoint type %d @ %016" PRIx64 ", range: %016" PRIx64
 | 
			
		||||
                      " - %016" PRIx64 " (%" PRIx64 " bytes)\n",
 | 
			
		||||
                      static_cast<int>(type), addr, bp->second.addr, bp->second.addr + len, len);
 | 
			
		||||
            NGLOG_DEBUG(Debug_GDBStub,
 | 
			
		||||
                        "Found breakpoint type {} @ {:016X}, range: {:016X}"
 | 
			
		||||
                        " - {:016X} ({:X} bytes)",
 | 
			
		||||
                        static_cast<int>(type), addr, bp->second.addr, bp->second.addr + len, len);
 | 
			
		||||
            return true;
 | 
			
		||||
        }
 | 
			
		||||
    }
 | 
			
		||||
@ -427,7 +425,7 @@ bool CheckBreakpoint(PAddr addr, BreakpointType type) {
 | 
			
		||||
static void SendPacket(const char packet) {
 | 
			
		||||
    size_t sent_size = send(gdbserver_socket, &packet, 1, 0);
 | 
			
		||||
    if (sent_size != 1) {
 | 
			
		||||
        LOG_ERROR(Debug_GDBStub, "send failed");
 | 
			
		||||
        NGLOG_ERROR(Debug_GDBStub, "send failed");
 | 
			
		||||
    }
 | 
			
		||||
}
 | 
			
		||||
 | 
			
		||||
@ -445,7 +443,7 @@ static void SendReply(const char* reply) {
 | 
			
		||||
 | 
			
		||||
    command_length = static_cast<u32>(strlen(reply));
 | 
			
		||||
    if (command_length + 4 > sizeof(command_buffer)) {
 | 
			
		||||
        LOG_ERROR(Debug_GDBStub, "command_buffer overflow in SendReply");
 | 
			
		||||
        NGLOG_ERROR(Debug_GDBStub, "command_buffer overflow in SendReply");
 | 
			
		||||
        return;
 | 
			
		||||
    }
 | 
			
		||||
 | 
			
		||||
@ -462,7 +460,7 @@ static void SendReply(const char* reply) {
 | 
			
		||||
    while (left > 0) {
 | 
			
		||||
        int sent_size = send(gdbserver_socket, reinterpret_cast<char*>(ptr), left, 0);
 | 
			
		||||
        if (sent_size < 0) {
 | 
			
		||||
            LOG_ERROR(Debug_GDBStub, "gdb: send failed");
 | 
			
		||||
            NGLOG_ERROR(Debug_GDBStub, "gdb: send failed");
 | 
			
		||||
            return Shutdown();
 | 
			
		||||
        }
 | 
			
		||||
 | 
			
		||||
@ -473,7 +471,7 @@ static void SendReply(const char* reply) {
 | 
			
		||||
 | 
			
		||||
/// Handle query command from gdb client.
 | 
			
		||||
static void HandleQuery() {
 | 
			
		||||
    LOG_DEBUG(Debug_GDBStub, "gdb: query '%s'\n", command_buffer + 1);
 | 
			
		||||
    NGLOG_DEBUG(Debug_GDBStub, "gdb: query '{}'", command_buffer + 1);
 | 
			
		||||
 | 
			
		||||
    const char* query = reinterpret_cast<const char*>(command_buffer + 1);
 | 
			
		||||
 | 
			
		||||
@ -512,8 +510,8 @@ static void SendSignal(u32 signal) {
 | 
			
		||||
 | 
			
		||||
    latest_signal = signal;
 | 
			
		||||
 | 
			
		||||
    std::string buffer = Common::StringFromFormat("T%02x", latest_signal);
 | 
			
		||||
    LOG_DEBUG(Debug_GDBStub, "Response: %s", buffer.c_str());
 | 
			
		||||
    std::string buffer = fmt::format("T{:02x}", latest_signal);
 | 
			
		||||
    NGLOG_DEBUG(Debug_GDBStub, "Response: {}", buffer);
 | 
			
		||||
    SendReply(buffer.c_str());
 | 
			
		||||
}
 | 
			
		||||
 | 
			
		||||
@ -527,18 +525,18 @@ static void ReadCommand() {
 | 
			
		||||
        // ignore ack
 | 
			
		||||
        return;
 | 
			
		||||
    } else if (c == 0x03) {
 | 
			
		||||
        LOG_INFO(Debug_GDBStub, "gdb: found break command\n");
 | 
			
		||||
        NGLOG_INFO(Debug_GDBStub, "gdb: found break command");
 | 
			
		||||
        halt_loop = true;
 | 
			
		||||
        SendSignal(SIGTRAP);
 | 
			
		||||
        return;
 | 
			
		||||
    } else if (c != GDB_STUB_START) {
 | 
			
		||||
        LOG_DEBUG(Debug_GDBStub, "gdb: read invalid byte %02x\n", c);
 | 
			
		||||
        NGLOG_DEBUG(Debug_GDBStub, "gdb: read invalid byte {:02X}", c);
 | 
			
		||||
        return;
 | 
			
		||||
    }
 | 
			
		||||
 | 
			
		||||
    while ((c = ReadByte()) != GDB_STUB_END) {
 | 
			
		||||
        if (command_length >= sizeof(command_buffer)) {
 | 
			
		||||
            LOG_ERROR(Debug_GDBStub, "gdb: command_buffer overflow\n");
 | 
			
		||||
            NGLOG_ERROR(Debug_GDBStub, "gdb: command_buffer overflow");
 | 
			
		||||
            SendPacket(GDB_STUB_NACK);
 | 
			
		||||
            return;
 | 
			
		||||
        }
 | 
			
		||||
@ -551,9 +549,10 @@ static void ReadCommand() {
 | 
			
		||||
    u8 checksum_calculated = CalculateChecksum(command_buffer, command_length);
 | 
			
		||||
 | 
			
		||||
    if (checksum_received != checksum_calculated) {
 | 
			
		||||
        LOG_ERROR(Debug_GDBStub,
 | 
			
		||||
                  "gdb: invalid checksum: calculated %02x and read %02x for $%s# (length: %d)\n",
 | 
			
		||||
                  checksum_calculated, checksum_received, command_buffer, command_length);
 | 
			
		||||
        NGLOG_ERROR(
 | 
			
		||||
            Debug_GDBStub,
 | 
			
		||||
            "gdb: invalid checksum: calculated {:02X} and read {:02X} for ${}# (length: {})",
 | 
			
		||||
            checksum_calculated, checksum_received, command_buffer, command_length);
 | 
			
		||||
 | 
			
		||||
        command_length = 0;
 | 
			
		||||
 | 
			
		||||
@ -580,7 +579,7 @@ static bool IsDataAvailable() {
 | 
			
		||||
    t.tv_usec = 0;
 | 
			
		||||
 | 
			
		||||
    if (select(gdbserver_socket + 1, &fd_socket, nullptr, nullptr, &t) < 0) {
 | 
			
		||||
        LOG_ERROR(Debug_GDBStub, "select failed");
 | 
			
		||||
        NGLOG_ERROR(Debug_GDBStub, "select failed");
 | 
			
		||||
        return false;
 | 
			
		||||
    }
 | 
			
		||||
 | 
			
		||||
@ -693,7 +692,7 @@ static void ReadMemory() {
 | 
			
		||||
    u64 len =
 | 
			
		||||
        HexToLong(start_offset, static_cast<u64>((command_buffer + command_length) - start_offset));
 | 
			
		||||
 | 
			
		||||
    LOG_DEBUG(Debug_GDBStub, "gdb: addr: %016lx len: %016lx\n", addr, len);
 | 
			
		||||
    NGLOG_DEBUG(Debug_GDBStub, "gdb: addr: {:016X} len: {:016X}", addr, len);
 | 
			
		||||
 | 
			
		||||
    if (len * 2 > sizeof(reply)) {
 | 
			
		||||
        SendReply("E01");
 | 
			
		||||
@ -781,8 +780,8 @@ static bool CommitBreakpoint(BreakpointType type, PAddr addr, u64 len) {
 | 
			
		||||
    breakpoint.len = len;
 | 
			
		||||
    p.insert({addr, breakpoint});
 | 
			
		||||
 | 
			
		||||
    LOG_DEBUG(Debug_GDBStub, "gdb: added %d breakpoint: %016" PRIx64 " bytes at %016" PRIx64 "\n",
 | 
			
		||||
              static_cast<int>(type), breakpoint.len, breakpoint.addr);
 | 
			
		||||
    NGLOG_DEBUG(Debug_GDBStub, "gdb: added {} breakpoint: {:016X} bytes at {:016X}",
 | 
			
		||||
                static_cast<int>(type), breakpoint.len, breakpoint.addr);
 | 
			
		||||
 | 
			
		||||
    return true;
 | 
			
		||||
}
 | 
			
		||||
@ -889,7 +888,7 @@ void HandlePacket() {
 | 
			
		||||
        return;
 | 
			
		||||
    }
 | 
			
		||||
 | 
			
		||||
    LOG_DEBUG(Debug_GDBStub, "Packet: %s", command_buffer);
 | 
			
		||||
    NGLOG_DEBUG(Debug_GDBStub, "Packet: {}", command_buffer);
 | 
			
		||||
 | 
			
		||||
    switch (command_buffer[0]) {
 | 
			
		||||
    case 'q':
 | 
			
		||||
@ -903,7 +902,7 @@ void HandlePacket() {
 | 
			
		||||
        break;
 | 
			
		||||
    case 'k':
 | 
			
		||||
        Shutdown();
 | 
			
		||||
        LOG_INFO(Debug_GDBStub, "killed by gdb");
 | 
			
		||||
        NGLOG_INFO(Debug_GDBStub, "killed by gdb");
 | 
			
		||||
        return;
 | 
			
		||||
    case 'g':
 | 
			
		||||
        ReadRegisters();
 | 
			
		||||
@ -982,7 +981,7 @@ static void Init(u16 port) {
 | 
			
		||||
    breakpoints_write.clear();
 | 
			
		||||
 | 
			
		||||
    // Start gdb server
 | 
			
		||||
    LOG_INFO(Debug_GDBStub, "Starting GDB server on port %d...", port);
 | 
			
		||||
    NGLOG_INFO(Debug_GDBStub, "Starting GDB server on port {}...", port);
 | 
			
		||||
 | 
			
		||||
    sockaddr_in saddr_server = {};
 | 
			
		||||
    saddr_server.sin_family = AF_INET;
 | 
			
		||||
@ -995,28 +994,28 @@ static void Init(u16 port) {
 | 
			
		||||
 | 
			
		||||
    int tmpsock = static_cast<int>(socket(PF_INET, SOCK_STREAM, 0));
 | 
			
		||||
    if (tmpsock == -1) {
 | 
			
		||||
        LOG_ERROR(Debug_GDBStub, "Failed to create gdb socket");
 | 
			
		||||
        NGLOG_ERROR(Debug_GDBStub, "Failed to create gdb socket");
 | 
			
		||||
    }
 | 
			
		||||
 | 
			
		||||
    // Set socket to SO_REUSEADDR so it can always bind on the same port
 | 
			
		||||
    int reuse_enabled = 1;
 | 
			
		||||
    if (setsockopt(tmpsock, SOL_SOCKET, SO_REUSEADDR, (const char*)&reuse_enabled,
 | 
			
		||||
                   sizeof(reuse_enabled)) < 0) {
 | 
			
		||||
        LOG_ERROR(Debug_GDBStub, "Failed to set gdb socket option");
 | 
			
		||||
        NGLOG_ERROR(Debug_GDBStub, "Failed to set gdb socket option");
 | 
			
		||||
    }
 | 
			
		||||
 | 
			
		||||
    const sockaddr* server_addr = reinterpret_cast<const sockaddr*>(&saddr_server);
 | 
			
		||||
    socklen_t server_addrlen = sizeof(saddr_server);
 | 
			
		||||
    if (bind(tmpsock, server_addr, server_addrlen) < 0) {
 | 
			
		||||
        LOG_ERROR(Debug_GDBStub, "Failed to bind gdb socket");
 | 
			
		||||
        NGLOG_ERROR(Debug_GDBStub, "Failed to bind gdb socket");
 | 
			
		||||
    }
 | 
			
		||||
 | 
			
		||||
    if (listen(tmpsock, 1) < 0) {
 | 
			
		||||
        LOG_ERROR(Debug_GDBStub, "Failed to listen to gdb socket");
 | 
			
		||||
        NGLOG_ERROR(Debug_GDBStub, "Failed to listen to gdb socket");
 | 
			
		||||
    }
 | 
			
		||||
 | 
			
		||||
    // Wait for gdb to connect
 | 
			
		||||
    LOG_INFO(Debug_GDBStub, "Waiting for gdb to connect...\n");
 | 
			
		||||
    NGLOG_INFO(Debug_GDBStub, "Waiting for gdb to connect...");
 | 
			
		||||
    sockaddr_in saddr_client;
 | 
			
		||||
    sockaddr* client_addr = reinterpret_cast<sockaddr*>(&saddr_client);
 | 
			
		||||
    socklen_t client_addrlen = sizeof(saddr_client);
 | 
			
		||||
@ -1027,9 +1026,9 @@ static void Init(u16 port) {
 | 
			
		||||
        halt_loop = false;
 | 
			
		||||
        step_loop = false;
 | 
			
		||||
 | 
			
		||||
        LOG_ERROR(Debug_GDBStub, "Failed to accept gdb client");
 | 
			
		||||
        NGLOG_ERROR(Debug_GDBStub, "Failed to accept gdb client");
 | 
			
		||||
    } else {
 | 
			
		||||
        LOG_INFO(Debug_GDBStub, "Client connected.\n");
 | 
			
		||||
        NGLOG_INFO(Debug_GDBStub, "Client connected.");
 | 
			
		||||
        saddr_client.sin_addr.s_addr = ntohl(saddr_client.sin_addr.s_addr);
 | 
			
		||||
    }
 | 
			
		||||
 | 
			
		||||
@ -1048,7 +1047,7 @@ void Shutdown() {
 | 
			
		||||
        return;
 | 
			
		||||
    }
 | 
			
		||||
 | 
			
		||||
    LOG_INFO(Debug_GDBStub, "Stopping GDB ...");
 | 
			
		||||
    NGLOG_INFO(Debug_GDBStub, "Stopping GDB ...");
 | 
			
		||||
    if (gdbserver_socket != -1) {
 | 
			
		||||
        shutdown(gdbserver_socket, SHUT_RDWR);
 | 
			
		||||
        gdbserver_socket = -1;
 | 
			
		||||
@ -1058,7 +1057,7 @@ void Shutdown() {
 | 
			
		||||
    WSACleanup();
 | 
			
		||||
#endif
 | 
			
		||||
 | 
			
		||||
    LOG_INFO(Debug_GDBStub, "GDB stopped.");
 | 
			
		||||
    NGLOG_INFO(Debug_GDBStub, "GDB stopped.");
 | 
			
		||||
}
 | 
			
		||||
 | 
			
		||||
bool IsServerEnabled() {
 | 
			
		||||
 | 
			
		||||
		Loading…
	
		Reference in New Issue
	
	Block a user