diff options
| -rw-r--r-- | dist/languages/.tx/config | 1 | ||||
| -rw-r--r-- | src/common/typed_address.h | 82 | ||||
| -rw-r--r-- | src/core/debugger/gdbstub_arch.cpp | 4 | ||||
| -rw-r--r-- | src/core/hle/service/cmif_serialization.h | 6 | ||||
| -rw-r--r-- | src/core/hle/service/cmif_types.h | 27 | ||||
| -rw-r--r-- | src/core/memory/cheat_engine.cpp | 46 | ||||
| -rw-r--r-- | src/core/memory/cheat_engine.h | 8 | ||||
| -rw-r--r-- | src/core/memory/dmnt_cheat_types.h | 2 | ||||
| -rw-r--r-- | src/core/memory/dmnt_cheat_vm.cpp | 74 | ||||
| -rw-r--r-- | src/core/memory/dmnt_cheat_vm.h | 22 | ||||
| -rw-r--r-- | src/yuzu/configuration/configure_per_game.cpp | 3 | 
11 files changed, 181 insertions, 94 deletions
diff --git a/dist/languages/.tx/config b/dist/languages/.tx/config index cca7b3d67..fbdfbe017 100644 --- a/dist/languages/.tx/config +++ b/dist/languages/.tx/config @@ -11,3 +11,4 @@ type = QT  file_filter = ../../src/android/app/src/main/res/values-<lang>/strings.xml  source_file = ../../src/android/app/src/main/res/values/strings.xml  type = ANDROID +lang_map = ja_JP:ja, ko_KR:ko, pt_BR:pt-rBR, pt_PT:pt-rPT, ru_RU:ru, vi_VN:vi, zh_CN:zh-rCN, zh_TW:zh-rTW diff --git a/src/common/typed_address.h b/src/common/typed_address.h index 64f4a07c2..d5e743583 100644 --- a/src/common/typed_address.h +++ b/src/common/typed_address.h @@ -186,68 +186,68 @@ static_assert(std::is_trivially_destructible_v<PhysicalAddress>);  static_assert(std::is_trivially_destructible_v<VirtualAddress>);  static_assert(std::is_trivially_destructible_v<ProcessAddress>); -static_assert(Null<uint64_t> == 0); +static_assert(Null<uint64_t> == 0U);  static_assert(Null<PhysicalAddress> == Null<uint64_t>);  static_assert(Null<VirtualAddress> == Null<uint64_t>);  static_assert(Null<ProcessAddress> == Null<uint64_t>);  // Constructor/assignment validations.  static_assert([] { -    const PhysicalAddress a(5); +    const PhysicalAddress a(5U);      PhysicalAddress b(a);      return b; -}() == PhysicalAddress(5)); +}() == PhysicalAddress(5U));  static_assert([] { -    const PhysicalAddress a(5); -    PhysicalAddress b(10); +    const PhysicalAddress a(5U); +    PhysicalAddress b(10U);      b = a;      return b; -}() == PhysicalAddress(5)); +}() == PhysicalAddress(5U));  // Arithmetic validations. -static_assert(PhysicalAddress(10) + 5 == PhysicalAddress(15)); -static_assert(PhysicalAddress(10) - 5 == PhysicalAddress(5)); +static_assert(PhysicalAddress(10U) + 5U == PhysicalAddress(15U)); +static_assert(PhysicalAddress(10U) - 5U == PhysicalAddress(5U));  static_assert([] { -    PhysicalAddress v(10); -    v += 5; +    PhysicalAddress v(10U); +    v += 5U;      return v; -}() == PhysicalAddress(15)); +}() == PhysicalAddress(15U));  static_assert([] { -    PhysicalAddress v(10); -    v -= 5; +    PhysicalAddress v(10U); +    v -= 5U;      return v; -}() == PhysicalAddress(5)); -static_assert(PhysicalAddress(10)++ == PhysicalAddress(10)); -static_assert(++PhysicalAddress(10) == PhysicalAddress(11)); -static_assert(PhysicalAddress(10)-- == PhysicalAddress(10)); -static_assert(--PhysicalAddress(10) == PhysicalAddress(9)); +}() == PhysicalAddress(5U)); +static_assert(PhysicalAddress(10U)++ == PhysicalAddress(10U)); +static_assert(++PhysicalAddress(10U) == PhysicalAddress(11U)); +static_assert(PhysicalAddress(10U)-- == PhysicalAddress(10U)); +static_assert(--PhysicalAddress(10U) == PhysicalAddress(9U));  // Logical validations. -static_assert((PhysicalAddress(0b11111111) >> 1) == 0b01111111); -static_assert((PhysicalAddress(0b10101010) >> 1) == 0b01010101); -static_assert((PhysicalAddress(0b11111111) << 1) == 0b111111110); -static_assert((PhysicalAddress(0b01010101) << 1) == 0b10101010); -static_assert((PhysicalAddress(0b11111111) & 0b01010101) == 0b01010101); -static_assert((PhysicalAddress(0b11111111) & 0b10101010) == 0b10101010); -static_assert((PhysicalAddress(0b01010101) & 0b10101010) == 0b00000000); -static_assert((PhysicalAddress(0b00000000) | 0b01010101) == 0b01010101); -static_assert((PhysicalAddress(0b11111111) | 0b01010101) == 0b11111111); -static_assert((PhysicalAddress(0b10101010) | 0b01010101) == 0b11111111); +static_assert((PhysicalAddress(0b11111111U) >> 1) == 0b01111111U); +static_assert((PhysicalAddress(0b10101010U) >> 1) == 0b01010101U); +static_assert((PhysicalAddress(0b11111111U) << 1) == 0b111111110U); +static_assert((PhysicalAddress(0b01010101U) << 1) == 0b10101010U); +static_assert((PhysicalAddress(0b11111111U) & 0b01010101U) == 0b01010101U); +static_assert((PhysicalAddress(0b11111111U) & 0b10101010U) == 0b10101010U); +static_assert((PhysicalAddress(0b01010101U) & 0b10101010U) == 0b00000000U); +static_assert((PhysicalAddress(0b00000000U) | 0b01010101U) == 0b01010101U); +static_assert((PhysicalAddress(0b11111111U) | 0b01010101U) == 0b11111111U); +static_assert((PhysicalAddress(0b10101010U) | 0b01010101U) == 0b11111111U);  // Comparisons. -static_assert(PhysicalAddress(0) == PhysicalAddress(0)); -static_assert(PhysicalAddress(0) != PhysicalAddress(1)); -static_assert(PhysicalAddress(0) < PhysicalAddress(1)); -static_assert(PhysicalAddress(0) <= PhysicalAddress(1)); -static_assert(PhysicalAddress(1) > PhysicalAddress(0)); -static_assert(PhysicalAddress(1) >= PhysicalAddress(0)); - -static_assert(!(PhysicalAddress(0) == PhysicalAddress(1))); -static_assert(!(PhysicalAddress(0) != PhysicalAddress(0))); -static_assert(!(PhysicalAddress(1) < PhysicalAddress(0))); -static_assert(!(PhysicalAddress(1) <= PhysicalAddress(0))); -static_assert(!(PhysicalAddress(0) > PhysicalAddress(1))); -static_assert(!(PhysicalAddress(0) >= PhysicalAddress(1))); +static_assert(PhysicalAddress(0U) == PhysicalAddress(0U)); +static_assert(PhysicalAddress(0U) != PhysicalAddress(1U)); +static_assert(PhysicalAddress(0U) < PhysicalAddress(1U)); +static_assert(PhysicalAddress(0U) <= PhysicalAddress(1U)); +static_assert(PhysicalAddress(1U) > PhysicalAddress(0U)); +static_assert(PhysicalAddress(1U) >= PhysicalAddress(0U)); + +static_assert(!(PhysicalAddress(0U) == PhysicalAddress(1U))); +static_assert(!(PhysicalAddress(0U) != PhysicalAddress(0U))); +static_assert(!(PhysicalAddress(1U) < PhysicalAddress(0U))); +static_assert(!(PhysicalAddress(1U) <= PhysicalAddress(0U))); +static_assert(!(PhysicalAddress(0U) > PhysicalAddress(1U))); +static_assert(!(PhysicalAddress(0U) >= PhysicalAddress(1U)));  } // namespace Common diff --git a/src/core/debugger/gdbstub_arch.cpp b/src/core/debugger/gdbstub_arch.cpp index f2a407dc8..452f565be 100644 --- a/src/core/debugger/gdbstub_arch.cpp +++ b/src/core/debugger/gdbstub_arch.cpp @@ -383,7 +383,7 @@ std::string GDBStubA32::RegRead(const Kernel::KThread* thread, size_t id) const      } else if (id == CPSR_REGISTER) {          return ValueToHex(context.pstate);      } else if (id >= D0_REGISTER && id < Q0_REGISTER) { -        return ValueToHex(fprs[id - D0_REGISTER][0]); +        return ValueToHex(fprs[(id - D0_REGISTER) / 2][(id - D0_REGISTER) % 2]);      } else if (id >= Q0_REGISTER && id < FPSCR_REGISTER) {          return ValueToHex(fprs[id - Q0_REGISTER]);      } else if (id == FPSCR_REGISTER) { @@ -406,7 +406,7 @@ void GDBStubA32::RegWrite(Kernel::KThread* thread, size_t id, std::string_view v      } else if (id == CPSR_REGISTER) {          context.pstate = HexToValue<u32>(value);      } else if (id >= D0_REGISTER && id < Q0_REGISTER) { -        fprs[id - D0_REGISTER] = {HexToValue<u64>(value), 0}; +        fprs[(id - D0_REGISTER) / 2][(id - D0_REGISTER) % 2] = HexToValue<u64>(value);      } else if (id >= Q0_REGISTER && id < FPSCR_REGISTER) {          fprs[id - Q0_REGISTER] = HexToValue<u128>(value);      } else if (id == FPSCR_REGISTER) { diff --git a/src/core/hle/service/cmif_serialization.h b/src/core/hle/service/cmif_serialization.h index 315475e71..e985fe317 100644 --- a/src/core/hle/service/cmif_serialization.h +++ b/src/core/hle/service/cmif_serialization.h @@ -115,6 +115,11 @@ struct ArgumentTraits {      static constexpr ArgumentType Type = ArgumentType::InData;  }; +template <typename... Ts> +consteval bool ConstIfReference() { +    return ((!std::is_reference_v<Ts> || std::is_const_v<std::remove_reference_t<Ts>>) && ... && true); +} +  struct RequestLayout {      u32 copy_handle_count;      u32 move_handle_count; @@ -435,6 +440,7 @@ void CmifReplyWrapImpl(HLERequestContext& ctx, T& t, Result (T::*f)(A...)) {      }      const bool is_domain = Domain ? ctx.GetManager()->IsDomain() : false; +    static_assert(ConstIfReference<A...>(), "Arguments taken by reference must be const");      using MethodArguments = std::tuple<std::remove_cvref_t<A>...>;      OutTemporaryBuffers buffers{}; diff --git a/src/core/hle/service/cmif_types.h b/src/core/hle/service/cmif_types.h index dc06169f4..84f4c2456 100644 --- a/src/core/hle/service/cmif_types.h +++ b/src/core/hle/service/cmif_types.h @@ -4,10 +4,9 @@  #pragma once  #include <memory> +#include <span> -#include "common/common_funcs.h"  #include "common/common_types.h" -#include "core/hle/service/hle_ipc.h"  namespace Service { @@ -22,8 +21,10 @@ class Out {  public:      using Type = T; +    /* implicit */ Out(const Out& t) : raw(t.raw) {}      /* implicit */ Out(AutoOut<Type>& t) : raw(&t.raw) {}      /* implicit */ Out(Type* t) : raw(t) {} +    Out& operator=(const Out&) = delete;      Type* Get() const {          return raw; @@ -37,6 +38,10 @@ public:          return raw;      } +    operator Type*() const { +        return raw; +    } +  private:      Type* raw;  }; @@ -113,8 +118,10 @@ class OutCopyHandle {  public:      using Type = T*; +    /* implicit */ OutCopyHandle(const OutCopyHandle& t) : raw(t.raw) {}      /* implicit */ OutCopyHandle(AutoOut<Type>& t) : raw(&t.raw) {}      /* implicit */ OutCopyHandle(Type* t) : raw(t) {} +    OutCopyHandle& operator=(const OutCopyHandle&) = delete;      Type* Get() const {          return raw; @@ -128,6 +135,10 @@ public:          return raw;      } +    operator Type*() const { +        return raw; +    } +  private:      Type* raw;  }; @@ -137,8 +148,10 @@ class OutMoveHandle {  public:      using Type = T*; +    /* implicit */ OutMoveHandle(const OutMoveHandle& t) : raw(t.raw) {}      /* implicit */ OutMoveHandle(AutoOut<Type>& t) : raw(&t.raw) {}      /* implicit */ OutMoveHandle(Type* t) : raw(t) {} +    OutMoveHandle& operator=(const OutMoveHandle&) = delete;      Type* Get() const {          return raw; @@ -152,6 +165,10 @@ public:          return raw;      } +    operator Type*() const { +        return raw; +    } +  private:      Type* raw;  }; @@ -248,8 +265,10 @@ public:      static constexpr BufferAttr Attr = static_cast<BufferAttr>(A | BufferAttr_In | BufferAttr_FixedSize);      using Type = T; +    /* implicit */ OutLargeData(const OutLargeData& t) : raw(t.raw) {}      /* implicit */ OutLargeData(Type* t) : raw(t) {}      /* implicit */ OutLargeData(AutoOut<T>& t) : raw(&t.raw) {} +    OutLargeData& operator=(const OutLargeData&) = delete;      Type* Get() const {          return raw; @@ -263,6 +282,10 @@ public:          return raw;      } +    operator Type*() const { +        return raw; +    } +  private:      Type* raw;  }; diff --git a/src/core/memory/cheat_engine.cpp b/src/core/memory/cheat_engine.cpp index 96fa7fa3a..14d1a3840 100644 --- a/src/core/memory/cheat_engine.cpp +++ b/src/core/memory/cheat_engine.cpp @@ -9,6 +9,7 @@  #include "core/core_timing.h"  #include "core/hle/kernel/k_page_table.h"  #include "core/hle/kernel/k_process.h" +#include "core/hle/kernel/k_process_page_table.h"  #include "core/hle/service/hid/hid_server.h"  #include "core/hle/service/sm/sm.h"  #include "core/memory.h" @@ -46,12 +47,23 @@ StandardVmCallbacks::StandardVmCallbacks(System& system_, const CheatProcessMeta  StandardVmCallbacks::~StandardVmCallbacks() = default; -void StandardVmCallbacks::MemoryRead(VAddr address, void* data, u64 size) { -    system.ApplicationMemory().ReadBlock(SanitizeAddress(address), data, size); +void StandardVmCallbacks::MemoryReadUnsafe(VAddr address, void* data, u64 size) { +    // Return zero on invalid address +    if (!IsAddressInRange(address) || !system.ApplicationMemory().IsValidVirtualAddress(address)) { +        std::memset(data, 0, size); +        return; +    } + +    system.ApplicationMemory().ReadBlock(address, data, size);  } -void StandardVmCallbacks::MemoryWrite(VAddr address, const void* data, u64 size) { -    system.ApplicationMemory().WriteBlock(SanitizeAddress(address), data, size); +void StandardVmCallbacks::MemoryWriteUnsafe(VAddr address, const void* data, u64 size) { +    // Skip invalid memory write address +    if (!IsAddressInRange(address) || !system.ApplicationMemory().IsValidVirtualAddress(address)) { +        return; +    } + +    system.ApplicationMemory().WriteBlock(address, data, size);  }  u64 StandardVmCallbacks::HidKeysDown() { @@ -81,21 +93,25 @@ void StandardVmCallbacks::CommandLog(std::string_view data) {                data.back() == '\n' ? data.substr(0, data.size() - 1) : data);  } -VAddr StandardVmCallbacks::SanitizeAddress(VAddr in) const { +bool StandardVmCallbacks::IsAddressInRange(VAddr in) const {      if ((in < metadata.main_nso_extents.base ||           in >= metadata.main_nso_extents.base + metadata.main_nso_extents.size) &&          (in < metadata.heap_extents.base || -         in >= metadata.heap_extents.base + metadata.heap_extents.size)) { -        LOG_ERROR(CheatEngine, +         in >= metadata.heap_extents.base + metadata.heap_extents.size) && +        (in < metadata.alias_extents.base || +         in >= metadata.heap_extents.base + metadata.alias_extents.size) && +        (in < metadata.aslr_extents.base || +         in >= metadata.heap_extents.base + metadata.aslr_extents.size)) { +        LOG_DEBUG(CheatEngine,                    "Cheat attempting to access memory at invalid address={:016X}, if this "                    "persists, "                    "the cheat may be incorrect. However, this may be normal early in execution if "                    "the game has not properly set up yet.",                    in); -        return 0; ///< Invalid addresses will hard crash +        return false; ///< Invalid addresses will hard crash      } -    return in; +    return true;  }  CheatParser::~CheatParser() = default; @@ -211,16 +227,14 @@ void CheatEngine::Initialize() {          .base = GetInteger(page_table.GetHeapRegionStart()),          .size = page_table.GetHeapRegionSize(),      }; - -    metadata.address_space_extents = { -        .base = GetInteger(page_table.GetAddressSpaceStart()), -        .size = page_table.GetAddressSpaceSize(), -    }; - -    metadata.alias_extents = { +    metadata.aslr_extents = {          .base = GetInteger(page_table.GetAliasCodeRegionStart()),          .size = page_table.GetAliasCodeRegionSize(),      }; +    metadata.alias_extents = { +        .base = GetInteger(page_table.GetAliasRegionStart()), +        .size = page_table.GetAliasRegionSize(), +    };      is_pending_reload.exchange(true);  } diff --git a/src/core/memory/cheat_engine.h b/src/core/memory/cheat_engine.h index ced2168d1..619cabaa2 100644 --- a/src/core/memory/cheat_engine.h +++ b/src/core/memory/cheat_engine.h @@ -27,17 +27,17 @@ public:      StandardVmCallbacks(System& system_, const CheatProcessMetadata& metadata_);      ~StandardVmCallbacks() override; -    void MemoryRead(VAddr address, void* data, u64 size) override; -    void MemoryWrite(VAddr address, const void* data, u64 size) override; +    void MemoryReadUnsafe(VAddr address, void* data, u64 size) override; +    void MemoryWriteUnsafe(VAddr address, const void* data, u64 size) override;      u64 HidKeysDown() override;      void DebugLog(u8 id, u64 value) override;      void CommandLog(std::string_view data) override;  private: -    VAddr SanitizeAddress(VAddr address) const; +    bool IsAddressInRange(VAddr address) const;      const CheatProcessMetadata& metadata; -    System& system; +    Core::System& system;  };  // Intermediary class that parses a text file or other disk format for storing cheats into a diff --git a/src/core/memory/dmnt_cheat_types.h b/src/core/memory/dmnt_cheat_types.h index c6b40e505..64c072d3d 100644 --- a/src/core/memory/dmnt_cheat_types.h +++ b/src/core/memory/dmnt_cheat_types.h @@ -18,7 +18,7 @@ struct CheatProcessMetadata {      MemoryRegionExtents main_nso_extents{};      MemoryRegionExtents heap_extents{};      MemoryRegionExtents alias_extents{}; -    MemoryRegionExtents address_space_extents{}; +    MemoryRegionExtents aslr_extents{};      std::array<u8, 0x20> main_nso_build_id{};  }; diff --git a/src/core/memory/dmnt_cheat_vm.cpp b/src/core/memory/dmnt_cheat_vm.cpp index 31ffc4fbb..8bc81e72d 100644 --- a/src/core/memory/dmnt_cheat_vm.cpp +++ b/src/core/memory/dmnt_cheat_vm.cpp @@ -322,8 +322,9 @@ bool DmntCheatVm::DecodeNextOpcode(CheatVmOpcode& out) {      } break;      case CheatVmOpcodeType::EndConditionalBlock: {          // 20000000 -        // There's actually nothing left to process here! -        opcode.opcode = EndConditionalOpcode{}; +        opcode.opcode = EndConditionalOpcode{ +            .is_else = ((first_dword >> 24) & 0xf) == 1, +        };      } break;      case CheatVmOpcodeType::ControlLoop: {          // 300R0000 VVVVVVVV @@ -555,6 +556,18 @@ bool DmntCheatVm::DecodeNextOpcode(CheatVmOpcode& out) {              .idx = first_dword & 0xF,          };      } break; +    case CheatVmOpcodeType::PauseProcess: { +        /* FF0????? */ +        /* FF0 = opcode 0xFF0 */ +        /* Pauses the current process. */ +        opcode.opcode = PauseProcessOpcode{}; +    } break; +    case CheatVmOpcodeType::ResumeProcess: { +        /* FF0????? */ +        /* FF0 = opcode 0xFF0 */ +        /* Pauses the current process. */ +        opcode.opcode = ResumeProcessOpcode{}; +    } break;      case CheatVmOpcodeType::DebugLog: {          // FFFTIX##          // FFFTI0Ma aaaaaaaa @@ -621,7 +634,7 @@ bool DmntCheatVm::DecodeNextOpcode(CheatVmOpcode& out) {      return valid;  } -void DmntCheatVm::SkipConditionalBlock() { +void DmntCheatVm::SkipConditionalBlock(bool is_if) {      if (condition_depth > 0) {          // We want to continue until we're out of the current block.          const std::size_t desired_depth = condition_depth - 1; @@ -637,8 +650,12 @@ void DmntCheatVm::SkipConditionalBlock() {              // We also support nesting of conditional blocks, and Gateway does not.              if (skip_opcode.begin_conditional_block) {                  condition_depth++; -            } else if (std::holds_alternative<EndConditionalOpcode>(skip_opcode.opcode)) { -                condition_depth--; +            } else if (auto end_cond = std::get_if<EndConditionalOpcode>(&skip_opcode.opcode)) { +                if (!end_cond->is_else) { +                    condition_depth--; +                } else if (is_if && condition_depth - 1 == desired_depth) { +                    break; +                }              }          }      } else { @@ -675,6 +692,10 @@ u64 DmntCheatVm::GetCheatProcessAddress(const CheatProcessMetadata& metadata,          return metadata.main_nso_extents.base + rel_address;      case MemoryAccessType::Heap:          return metadata.heap_extents.base + rel_address; +    case MemoryAccessType::Alias: +        return metadata.alias_extents.base + rel_address; +    case MemoryAccessType::Aslr: +        return metadata.aslr_extents.base + rel_address;      }  } @@ -682,7 +703,6 @@ void DmntCheatVm::ResetState() {      registers.fill(0);      saved_values.fill(0);      loop_tops.fill(0); -    static_registers.fill(0);      instruction_ptr = 0;      condition_depth = 0;      decode_success = true; @@ -753,7 +773,7 @@ void DmntCheatVm::Execute(const CheatProcessMetadata& metadata) {              case 2:              case 4:              case 8: -                callbacks->MemoryWrite(dst_address, &dst_value, store_static->bit_width); +                callbacks->MemoryWriteUnsafe(dst_address, &dst_value, store_static->bit_width);                  break;              }          } else if (auto begin_cond = std::get_if<BeginConditionalOpcode>(&cur_opcode.opcode)) { @@ -766,7 +786,7 @@ void DmntCheatVm::Execute(const CheatProcessMetadata& metadata) {              case 2:              case 4:              case 8: -                callbacks->MemoryRead(src_address, &src_value, begin_cond->bit_width); +                callbacks->MemoryReadUnsafe(src_address, &src_value, begin_cond->bit_width);                  break;              }              // Check against condition. @@ -794,13 +814,18 @@ void DmntCheatVm::Execute(const CheatProcessMetadata& metadata) {              }              // Skip conditional block if condition not met.              if (!cond_met) { -                SkipConditionalBlock(); +                SkipConditionalBlock(true);              } -        } else if (std::holds_alternative<EndConditionalOpcode>(cur_opcode.opcode)) { -            // Decrement the condition depth. -            // We will assume, graciously, that mismatched conditional block ends are a nop. -            if (condition_depth > 0) { -                condition_depth--; +        } else if (auto end_cond = std::get_if<EndConditionalOpcode>(&cur_opcode.opcode)) { +            if (end_cond->is_else) { +                /* Skip to the end of the conditional block. */ +                this->SkipConditionalBlock(false); +            } else { +                /* Decrement the condition depth. */ +                /* We will assume, graciously, that mismatched conditional block ends are a nop. */ +                if (condition_depth > 0) { +                    condition_depth--; +                }              }          } else if (auto ctrl_loop = std::get_if<ControlLoopOpcode>(&cur_opcode.opcode)) {              if (ctrl_loop->start_loop) { @@ -832,8 +857,8 @@ void DmntCheatVm::Execute(const CheatProcessMetadata& metadata) {              case 2:              case 4:              case 8: -                callbacks->MemoryRead(src_address, ®isters[ldr_memory->reg_index], -                                      ldr_memory->bit_width); +                callbacks->MemoryReadUnsafe(src_address, ®isters[ldr_memory->reg_index], +                                            ldr_memory->bit_width);                  break;              }          } else if (auto str_static = std::get_if<StoreStaticToAddressOpcode>(&cur_opcode.opcode)) { @@ -849,7 +874,7 @@ void DmntCheatVm::Execute(const CheatProcessMetadata& metadata) {              case 2:              case 4:              case 8: -                callbacks->MemoryWrite(dst_address, &dst_value, str_static->bit_width); +                callbacks->MemoryWriteUnsafe(dst_address, &dst_value, str_static->bit_width);                  break;              }              // Increment register if relevant. @@ -908,7 +933,7 @@ void DmntCheatVm::Execute(const CheatProcessMetadata& metadata) {              // Check for keypress.              if ((begin_keypress_cond->key_mask & kDown) != begin_keypress_cond->key_mask) {                  // Keys not pressed. Skip conditional block. -                SkipConditionalBlock(); +                SkipConditionalBlock(true);              }          } else if (auto perform_math_reg =                         std::get_if<PerformArithmeticRegisterOpcode>(&cur_opcode.opcode)) { @@ -1007,7 +1032,7 @@ void DmntCheatVm::Execute(const CheatProcessMetadata& metadata) {              case 2:              case 4:              case 8: -                callbacks->MemoryWrite(dst_address, &dst_value, str_register->bit_width); +                callbacks->MemoryWriteUnsafe(dst_address, &dst_value, str_register->bit_width);                  break;              } @@ -1086,7 +1111,8 @@ void DmntCheatVm::Execute(const CheatProcessMetadata& metadata) {                  case 2:                  case 4:                  case 8: -                    callbacks->MemoryRead(cond_address, &cond_value, begin_reg_cond->bit_width); +                    callbacks->MemoryReadUnsafe(cond_address, &cond_value, +                                                begin_reg_cond->bit_width);                      break;                  }              } @@ -1116,7 +1142,7 @@ void DmntCheatVm::Execute(const CheatProcessMetadata& metadata) {              // Skip conditional block if condition not met.              if (!cond_met) { -                SkipConditionalBlock(); +                SkipConditionalBlock(true);              }          } else if (auto save_restore_reg =                         std::get_if<SaveRestoreRegisterOpcode>(&cur_opcode.opcode)) { @@ -1178,6 +1204,10 @@ void DmntCheatVm::Execute(const CheatProcessMetadata& metadata) {                  // Store a register to a static register.                  static_registers[rw_static_reg->static_idx] = registers[rw_static_reg->idx];              } +        } else if (std::holds_alternative<PauseProcessOpcode>(cur_opcode.opcode)) { +            // TODO: Pause cheat process +        } else if (std::holds_alternative<ResumeProcessOpcode>(cur_opcode.opcode)) { +            // TODO: Resume cheat process          } else if (auto debug_log = std::get_if<DebugLogOpcode>(&cur_opcode.opcode)) {              // Read value from memory.              u64 log_value = 0; @@ -1224,7 +1254,7 @@ void DmntCheatVm::Execute(const CheatProcessMetadata& metadata) {                  case 2:                  case 4:                  case 8: -                    callbacks->MemoryRead(val_address, &log_value, debug_log->bit_width); +                    callbacks->MemoryReadUnsafe(val_address, &log_value, debug_log->bit_width);                      break;                  }              } diff --git a/src/core/memory/dmnt_cheat_vm.h b/src/core/memory/dmnt_cheat_vm.h index 641cb09c4..fed6a24ad 100644 --- a/src/core/memory/dmnt_cheat_vm.h +++ b/src/core/memory/dmnt_cheat_vm.h @@ -42,12 +42,16 @@ enum class CheatVmOpcodeType : u32 {      DoubleExtendedWidth = 0xF0,      // Double-extended width opcodes. +    PauseProcess = 0xFF0, +    ResumeProcess = 0xFF1,      DebugLog = 0xFFF,  };  enum class MemoryAccessType : u32 {      MainNso = 0,      Heap = 1, +    Alias = 2, +    Aslr = 3,  };  enum class ConditionalComparisonType : u32 { @@ -131,7 +135,9 @@ struct BeginConditionalOpcode {      VmInt value{};  }; -struct EndConditionalOpcode {}; +struct EndConditionalOpcode { +    bool is_else; +};  struct ControlLoopOpcode {      bool start_loop{}; @@ -222,6 +228,10 @@ struct ReadWriteStaticRegisterOpcode {      u32 idx{};  }; +struct PauseProcessOpcode {}; + +struct ResumeProcessOpcode {}; +  struct DebugLogOpcode {      u32 bit_width{};      u32 log_id{}; @@ -244,8 +254,8 @@ struct CheatVmOpcode {                   PerformArithmeticStaticOpcode, BeginKeypressConditionalOpcode,                   PerformArithmeticRegisterOpcode, StoreRegisterToAddressOpcode,                   BeginRegisterConditionalOpcode, SaveRestoreRegisterOpcode, -                 SaveRestoreRegisterMaskOpcode, ReadWriteStaticRegisterOpcode, DebugLogOpcode, -                 UnrecognizedInstruction> +                 SaveRestoreRegisterMaskOpcode, ReadWriteStaticRegisterOpcode, PauseProcessOpcode, +                 ResumeProcessOpcode, DebugLogOpcode, UnrecognizedInstruction>          opcode{};  }; @@ -256,8 +266,8 @@ public:      public:          virtual ~Callbacks(); -        virtual void MemoryRead(VAddr address, void* data, u64 size) = 0; -        virtual void MemoryWrite(VAddr address, const void* data, u64 size) = 0; +        virtual void MemoryReadUnsafe(VAddr address, void* data, u64 size) = 0; +        virtual void MemoryWriteUnsafe(VAddr address, const void* data, u64 size) = 0;          virtual u64 HidKeysDown() = 0; @@ -296,7 +306,7 @@ private:      std::array<std::size_t, NumRegisters> loop_tops{};      bool DecodeNextOpcode(CheatVmOpcode& out); -    void SkipConditionalBlock(); +    void SkipConditionalBlock(bool is_if);      void ResetState();      // For implementing the DebugLog opcode. diff --git a/src/yuzu/configuration/configure_per_game.cpp b/src/yuzu/configuration/configure_per_game.cpp index 9d38ab812..4dbe801a9 100644 --- a/src/yuzu/configuration/configure_per_game.cpp +++ b/src/yuzu/configuration/configure_per_game.cpp @@ -73,8 +73,11 @@ ConfigurePerGame::ConfigurePerGame(QWidget* parent, u64 title_id_, const std::st      ui->tabWidget->addTab(graphics_advanced_tab.get(), tr("Adv. Graphics"));      ui->tabWidget->addTab(audio_tab.get(), tr("Audio"));      ui->tabWidget->addTab(input_tab.get(), tr("Input Profiles")); +      // Only show Linux tab on Unix +    linux_tab->setVisible(false);  #ifdef __unix__ +    linux_tab->setVisible(true);      ui->tabWidget->addTab(linux_tab.get(), tr("Linux"));  #endif  | 
