diff options
| -rw-r--r-- | .lgtm.yml | 13 | ||||
| m--------- | externals/dynarmic | 0 | ||||
| -rw-r--r-- | src/audio_core/renderer/system.cpp | 5 | ||||
| -rw-r--r-- | src/audio_core/sink/sdl2_sink.cpp | 11 | ||||
| -rw-r--r-- | src/common/address_space.inc | 2 | ||||
| -rw-r--r-- | src/common/vector_math.h | 14 | ||||
| -rw-r--r-- | src/core/hid/emulated_controller.cpp | 10 | ||||
| -rw-r--r-- | src/core/hid/emulated_controller.h | 1 | ||||
| -rw-r--r-- | src/core/hid/motion_input.cpp | 36 | ||||
| -rw-r--r-- | src/core/hid/motion_input.h | 2 | ||||
| -rw-r--r-- | src/input_common/input_engine.cpp | 2 | ||||
| -rw-r--r-- | src/video_core/shader_cache.cpp | 4 | ||||
| -rw-r--r-- | src/video_core/shader_environment.cpp | 16 | ||||
| -rw-r--r-- | src/video_core/shader_environment.h | 6 | ||||
| -rw-r--r-- | src/yuzu/configuration/configure_input_player_widget.cpp | 91 | ||||
| -rw-r--r-- | src/yuzu/configuration/configure_input_player_widget.h | 5 | ||||
| -rw-r--r-- | src/yuzu_cmd/config.cpp | 12 | 
17 files changed, 177 insertions, 53 deletions
| diff --git a/.lgtm.yml b/.lgtm.yml deleted file mode 100644 index 7cd3f9926..000000000 --- a/.lgtm.yml +++ /dev/null @@ -1,13 +0,0 @@ -# SPDX-FileCopyrightText: 2020 yuzu Emulator Project -# SPDX-License-Identifier: GPL-2.0-or-later - -path_classifiers: -  library: "externals" -extraction: -  cpp: -    prepare: -      packages: -      - "libsdl2-dev" -      - "qtmultimedia5-dev" -      - "libtbb-dev" -      - "libjack-jackd2-dev" diff --git a/externals/dynarmic b/externals/dynarmic -Subproject c08c5a9362bb224dc343c2f616c24df027dfdf1 +Subproject f9e6a3df5c84bcc74be46c289a74a78e5e28d62 diff --git a/src/audio_core/renderer/system.cpp b/src/audio_core/renderer/system.cpp index ad869facb..53b258c4f 100644 --- a/src/audio_core/renderer/system.cpp +++ b/src/audio_core/renderer/system.cpp @@ -436,10 +436,7 @@ void System::Stop() {      }      if (execution_mode == ExecutionMode::Auto) { -        // Should wait for the system to terminate here, but core timing (should have) already -        // stopped, so this isn't needed. Find a way to make this definite. - -        // terminate_event.Wait(); +        terminate_event.Wait();      }  } diff --git a/src/audio_core/sink/sdl2_sink.cpp b/src/audio_core/sink/sdl2_sink.cpp index ee1a0652f..c1529d1f9 100644 --- a/src/audio_core/sink/sdl2_sink.cpp +++ b/src/audio_core/sink/sdl2_sink.cpp @@ -3,6 +3,7 @@  #include <span>  #include <vector> +#include <SDL.h>  #include "audio_core/common/common.h"  #include "audio_core/sink/sdl2_sink.h" @@ -10,16 +11,6 @@  #include "common/logging/log.h"  #include "core/core.h" -// Ignore -Wimplicit-fallthrough due to https://github.com/libsdl-org/SDL/issues/4307 -#ifdef __clang__ -#pragma clang diagnostic push -#pragma clang diagnostic ignored "-Wimplicit-fallthrough" -#endif -#include <SDL.h> -#ifdef __clang__ -#pragma clang diagnostic pop -#endif -  namespace AudioCore::Sink {  /**   * SDL sink stream, responsible for sinking samples to hardware. diff --git a/src/common/address_space.inc b/src/common/address_space.inc index 2195dabd5..c97dc8651 100644 --- a/src/common/address_space.inc +++ b/src/common/address_space.inc @@ -72,7 +72,7 @@ MAP_MEMBER(void)::MapLocked(VaType virt, PaType phys, VaType size, ExtraBlockInf                  }              }()}; -            if (block_end_predecessor->virt >= virt) { +            if (block_end_predecessor != blocks.begin() && block_end_predecessor->virt >= virt) {                  // If this block's start would be overlapped by the map then reuse it as a tail                  // block                  block_end_predecessor->virt = virt_end; diff --git a/src/common/vector_math.h b/src/common/vector_math.h index 0e2095c45..b4885835d 100644 --- a/src/common/vector_math.h +++ b/src/common/vector_math.h @@ -259,6 +259,20 @@ public:          return *this;      } +    void RotateFromOrigin(float roll, float pitch, float yaw) { +        float temp = y; +        y = std::cos(roll) * y - std::sin(roll) * z; +        z = std::sin(roll) * temp + std::cos(roll) * z; + +        temp = x; +        x = std::cos(pitch) * x + std::sin(pitch) * z; +        z = -std::sin(pitch) * temp + std::cos(pitch) * z; + +        temp = x; +        x = std::cos(yaw) * x - std::sin(yaw) * y; +        y = std::sin(yaw) * temp + std::cos(yaw) * y; +    } +      [[nodiscard]] constexpr T Length2() const {          return x * x + y * y + z * z;      } diff --git a/src/core/hid/emulated_controller.cpp b/src/core/hid/emulated_controller.cpp index a70f8807c..ecab85893 100644 --- a/src/core/hid/emulated_controller.cpp +++ b/src/core/hid/emulated_controller.cpp @@ -376,6 +376,7 @@ void EmulatedController::ReloadInput() {          motion.accel = emulated_motion.GetAcceleration();          motion.gyro = emulated_motion.GetGyroscope();          motion.rotation = emulated_motion.GetRotations(); +        motion.euler = emulated_motion.GetEulerAngles();          motion.orientation = emulated_motion.GetOrientation();          motion.is_at_rest = !emulated_motion.IsMoving(motion_sensitivity);      } @@ -551,6 +552,8 @@ void EmulatedController::EnableSystemButtons() {  void EmulatedController::DisableSystemButtons() {      std::scoped_lock lock{mutex};      system_buttons_enabled = false; +    controller.home_button_state.raw = 0; +    controller.capture_button_state.raw = 0;  }  void EmulatedController::ResetSystemButtons() { @@ -734,6 +737,8 @@ void EmulatedController::SetButton(const Common::Input::CallbackStatus& callback      if (is_configuring) {          controller.npad_button_state.raw = NpadButton::None;          controller.debug_pad_button_state.raw = 0; +        controller.home_button_state.raw = 0; +        controller.capture_button_state.raw = 0;          lock.unlock();          TriggerOnChange(ControllerTriggerType::Button, false);          return; @@ -976,14 +981,11 @@ void EmulatedController::SetMotion(const Common::Input::CallbackStatus& callback      emulated.UpdateOrientation(raw_status.delta_timestamp);      force_update_motion = raw_status.force_update; -    if (is_configuring) { -        return; -    } -      auto& motion = controller.motion_state[index];      motion.accel = emulated.GetAcceleration();      motion.gyro = emulated.GetGyroscope();      motion.rotation = emulated.GetRotations(); +    motion.euler = emulated.GetEulerAngles();      motion.orientation = emulated.GetOrientation();      motion.is_at_rest = !emulated.IsMoving(motion_sensitivity);  } diff --git a/src/core/hid/emulated_controller.h b/src/core/hid/emulated_controller.h index 429655355..6e01f4e12 100644 --- a/src/core/hid/emulated_controller.h +++ b/src/core/hid/emulated_controller.h @@ -106,6 +106,7 @@ struct ControllerMotion {      Common::Vec3f accel{};      Common::Vec3f gyro{};      Common::Vec3f rotation{}; +    Common::Vec3f euler{};      std::array<Common::Vec3f, 3> orientation{};      bool is_at_rest{};  }; diff --git a/src/core/hid/motion_input.cpp b/src/core/hid/motion_input.cpp index 0dd66c1cc..b60478dbb 100644 --- a/src/core/hid/motion_input.cpp +++ b/src/core/hid/motion_input.cpp @@ -1,6 +1,8 @@  // SPDX-FileCopyrightText: Copyright 2020 yuzu Emulator Project  // SPDX-License-Identifier: GPL-2.0-or-later +#include <cmath> +  #include "common/math_util.h"  #include "core/hid/motion_input.h" @@ -51,6 +53,20 @@ void MotionInput::SetQuaternion(const Common::Quaternion<f32>& quaternion) {      quat = quaternion;  } +void MotionInput::SetEulerAngles(const Common::Vec3f& euler_angles) { +    const float cr = std::cos(euler_angles.x * 0.5f); +    const float sr = std::sin(euler_angles.x * 0.5f); +    const float cp = std::cos(euler_angles.y * 0.5f); +    const float sp = std::sin(euler_angles.y * 0.5f); +    const float cy = std::cos(euler_angles.z * 0.5f); +    const float sy = std::sin(euler_angles.z * 0.5f); + +    quat.w = cr * cp * cy + sr * sp * sy; +    quat.xyz.x = sr * cp * cy - cr * sp * sy; +    quat.xyz.y = cr * sp * cy + sr * cp * sy; +    quat.xyz.z = cr * cp * sy - sr * sp * cy; +} +  void MotionInput::SetGyroBias(const Common::Vec3f& bias) {      gyro_bias = bias;  } @@ -222,6 +238,26 @@ Common::Vec3f MotionInput::GetRotations() const {      return rotations;  } +Common::Vec3f MotionInput::GetEulerAngles() const { +    // roll (x-axis rotation) +    const float sinr_cosp = 2 * (quat.w * quat.xyz.x + quat.xyz.y * quat.xyz.z); +    const float cosr_cosp = 1 - 2 * (quat.xyz.x * quat.xyz.x + quat.xyz.y * quat.xyz.y); + +    // pitch (y-axis rotation) +    const float sinp = std::sqrt(1 + 2 * (quat.w * quat.xyz.y - quat.xyz.x * quat.xyz.z)); +    const float cosp = std::sqrt(1 - 2 * (quat.w * quat.xyz.y - quat.xyz.x * quat.xyz.z)); + +    // yaw (z-axis rotation) +    const float siny_cosp = 2 * (quat.w * quat.xyz.z + quat.xyz.x * quat.xyz.y); +    const float cosy_cosp = 1 - 2 * (quat.xyz.y * quat.xyz.y + quat.xyz.z * quat.xyz.z); + +    return { +        std::atan2(sinr_cosp, cosr_cosp), +        2 * std::atan2(sinp, cosp) - Common::PI / 2, +        std::atan2(siny_cosp, cosy_cosp), +    }; +} +  void MotionInput::ResetOrientation() {      if (!reset_enabled || only_accelerometer) {          return; diff --git a/src/core/hid/motion_input.h b/src/core/hid/motion_input.h index 9f3fc1cf7..482719359 100644 --- a/src/core/hid/motion_input.h +++ b/src/core/hid/motion_input.h @@ -35,6 +35,7 @@ public:      void SetAcceleration(const Common::Vec3f& acceleration);      void SetGyroscope(const Common::Vec3f& gyroscope);      void SetQuaternion(const Common::Quaternion<f32>& quaternion); +    void SetEulerAngles(const Common::Vec3f& euler_angles);      void SetGyroBias(const Common::Vec3f& bias);      void SetGyroThreshold(f32 threshold); @@ -54,6 +55,7 @@ public:      [[nodiscard]] Common::Vec3f GetGyroBias() const;      [[nodiscard]] Common::Vec3f GetRotations() const;      [[nodiscard]] Common::Quaternion<f32> GetQuaternion() const; +    [[nodiscard]] Common::Vec3f GetEulerAngles() const;      [[nodiscard]] bool IsMoving(f32 sensitivity) const;      [[nodiscard]] bool IsCalibrated(f32 sensitivity) const; diff --git a/src/input_common/input_engine.cpp b/src/input_common/input_engine.cpp index 91aa96aa7..49f5e7f54 100644 --- a/src/input_common/input_engine.cpp +++ b/src/input_common/input_engine.cpp @@ -58,6 +58,8 @@ void InputEngine::SetHatButton(const PadIdentifier& identifier, int button, u8 v  }  void InputEngine::SetAxis(const PadIdentifier& identifier, int axis, f32 value) { +    value /= 2.0f; +    value -= 0.5f;      {          std::scoped_lock lock{mutex};          ControllerData& controller = controller_list.at(identifier); diff --git a/src/video_core/shader_cache.cpp b/src/video_core/shader_cache.cpp index d9482371b..c5213875b 100644 --- a/src/video_core/shader_cache.cpp +++ b/src/video_core/shader_cache.cpp @@ -228,14 +228,14 @@ const ShaderInfo* ShaderCache::MakeShaderInfo(GenericEnvironment& env, VAddr cpu      auto info = std::make_unique<ShaderInfo>();      if (const std::optional<u64> cached_hash{env.Analyze()}) {          info->unique_hash = *cached_hash; -        info->size_bytes = env.CachedSize(); +        info->size_bytes = env.CachedSizeBytes();      } else {          // Slow path, not really hit on commercial games          // Build a control flow graph to get the real shader size          Shader::ObjectPool<Shader::Maxwell::Flow::Block> flow_block;          Shader::Maxwell::Flow::CFG cfg{env, flow_block, env.StartAddress()};          info->unique_hash = env.CalculateHash(); -        info->size_bytes = env.ReadSize(); +        info->size_bytes = env.ReadSizeBytes();      }      const size_t size_bytes{info->size_bytes};      const ShaderInfo* const result{info.get()}; diff --git a/src/video_core/shader_environment.cpp b/src/video_core/shader_environment.cpp index 574760f80..c7cb56243 100644 --- a/src/video_core/shader_environment.cpp +++ b/src/video_core/shader_environment.cpp @@ -170,15 +170,19 @@ std::optional<u64> GenericEnvironment::Analyze() {  void GenericEnvironment::SetCachedSize(size_t size_bytes) {      cached_lowest = start_address;      cached_highest = start_address + static_cast<u32>(size_bytes); -    code.resize(CachedSize()); +    code.resize(CachedSizeWords());      gpu_memory->ReadBlock(program_base + cached_lowest, code.data(), code.size() * sizeof(u64));  } -size_t GenericEnvironment::CachedSize() const noexcept { -    return cached_highest - cached_lowest + INST_SIZE; +size_t GenericEnvironment::CachedSizeWords() const noexcept { +    return CachedSizeBytes() / INST_SIZE;  } -size_t GenericEnvironment::ReadSize() const noexcept { +size_t GenericEnvironment::CachedSizeBytes() const noexcept { +    return static_cast<size_t>(cached_highest) - cached_lowest + INST_SIZE; +} + +size_t GenericEnvironment::ReadSizeBytes() const noexcept {      return read_highest - read_lowest + INST_SIZE;  } @@ -187,7 +191,7 @@ bool GenericEnvironment::CanBeSerialized() const noexcept {  }  u64 GenericEnvironment::CalculateHash() const { -    const size_t size{ReadSize()}; +    const size_t size{ReadSizeBytes()};      const auto data{std::make_unique<char[]>(size)};      gpu_memory->ReadBlock(program_base + read_lowest, data.get(), size);      return Common::CityHash64(data.get(), size); @@ -198,7 +202,7 @@ void GenericEnvironment::Dump(u64 hash) {  }  void GenericEnvironment::Serialize(std::ofstream& file) const { -    const u64 code_size{static_cast<u64>(CachedSize())}; +    const u64 code_size{static_cast<u64>(CachedSizeBytes())};      const u64 num_texture_types{static_cast<u64>(texture_types.size())};      const u64 num_texture_pixel_formats{static_cast<u64>(texture_pixel_formats.size())};      const u64 num_cbuf_values{static_cast<u64>(cbuf_values.size())}; diff --git a/src/video_core/shader_environment.h b/src/video_core/shader_environment.h index d75987a52..a0f61cbda 100644 --- a/src/video_core/shader_environment.h +++ b/src/video_core/shader_environment.h @@ -48,9 +48,11 @@ public:      void SetCachedSize(size_t size_bytes); -    [[nodiscard]] size_t CachedSize() const noexcept; +    [[nodiscard]] size_t CachedSizeWords() const noexcept; -    [[nodiscard]] size_t ReadSize() const noexcept; +    [[nodiscard]] size_t CachedSizeBytes() const noexcept; + +    [[nodiscard]] size_t ReadSizeBytes() const noexcept;      [[nodiscard]] bool CanBeSerialized() const noexcept; diff --git a/src/yuzu/configuration/configure_input_player_widget.cpp b/src/yuzu/configuration/configure_input_player_widget.cpp index c287220fc..fe1ee2289 100644 --- a/src/yuzu/configuration/configure_input_player_widget.cpp +++ b/src/yuzu/configuration/configure_input_player_widget.cpp @@ -180,6 +180,10 @@ void PlayerControlPreview::ControllerUpdate(Core::HID::ControllerTriggerType typ          battery_values = controller->GetBatteryValues();          needs_redraw = true;          break; +    case Core::HID::ControllerTriggerType::Motion: +        motion_values = controller->GetMotions(); +        needs_redraw = true; +        break;      default:          break;      } @@ -313,6 +317,15 @@ void PlayerControlPreview::DrawLeftController(QPainter& p, const QPointF center)          DrawRawJoystick(p, center + QPointF(-140, 90), QPointF(0, 0));      } +    { +        // Draw motion cubes +        using namespace Settings::NativeMotion; +        p.setPen(colors.outline); +        p.setBrush(colors.transparent); +        Draw3dCube(p, center + QPointF(-140, 90), +                   motion_values[Settings::NativeMotion::MotionLeft].euler, 20.0f); +    } +      using namespace Settings::NativeButton;      // D-pad constants @@ -435,6 +448,15 @@ void PlayerControlPreview::DrawRightController(QPainter& p, const QPointF center          DrawRawJoystick(p, QPointF(0, 0), center + QPointF(140, 90));      } +    { +        // Draw motion cubes +        using namespace Settings::NativeMotion; +        p.setPen(colors.outline); +        p.setBrush(colors.transparent); +        Draw3dCube(p, center + QPointF(140, 90), +                   motion_values[Settings::NativeMotion::MotionRight].euler, 20.0f); +    } +      using namespace Settings::NativeButton;      // Face buttons constants @@ -555,6 +577,17 @@ void PlayerControlPreview::DrawDualController(QPainter& p, const QPointF center)          DrawRawJoystick(p, center + QPointF(-180, 90), center + QPointF(180, 90));      } +    { +        // Draw motion cubes +        using namespace Settings::NativeMotion; +        p.setPen(colors.outline); +        p.setBrush(colors.transparent); +        Draw3dCube(p, center + QPointF(-180, -5), +                   motion_values[Settings::NativeMotion::MotionLeft].euler, 20.0f); +        Draw3dCube(p, center + QPointF(180, -5), +                   motion_values[Settings::NativeMotion::MotionRight].euler, 20.0f); +    } +      using namespace Settings::NativeButton;      // Face buttons constants @@ -647,6 +680,15 @@ void PlayerControlPreview::DrawHandheldController(QPainter& p, const QPointF cen          DrawRawJoystick(p, center + QPointF(-50, 0), center + QPointF(50, 0));      } +    { +        // Draw motion cubes +        using namespace Settings::NativeMotion; +        p.setPen(colors.outline); +        p.setBrush(colors.transparent); +        Draw3dCube(p, center + QPointF(0, -115), +                   motion_values[Settings::NativeMotion::MotionLeft].euler, 15.0f); +    } +      using namespace Settings::NativeButton;      // Face buttons constants @@ -750,6 +792,15 @@ void PlayerControlPreview::DrawProController(QPainter& p, const QPointF center)          DrawRawJoystick(p, center + QPointF(-50, 105), center + QPointF(50, 105));      } +    { +        // Draw motion cubes +        using namespace Settings::NativeMotion; +        p.setPen(colors.button); +        p.setBrush(colors.transparent); +        Draw3dCube(p, center + QPointF(0, -100), +                   motion_values[Settings::NativeMotion::MotionLeft].euler, 15.0f); +    } +      using namespace Settings::NativeButton;      // Face buttons constants @@ -2871,6 +2922,46 @@ void PlayerControlPreview::DrawArrow(QPainter& p, const QPointF center, const Di      DrawPolygon(p, arrow_symbol);  } +// Draw motion functions +void PlayerControlPreview::Draw3dCube(QPainter& p, QPointF center, const Common::Vec3f& euler, +                                      float size) { +    std::array<Common::Vec3f, 8> cube{ +        Common::Vec3f{-1, -1, -1}, +        {-1, 1, -1}, +        {1, 1, -1}, +        {1, -1, -1}, +        {-1, -1, 1}, +        {-1, 1, 1}, +        {1, 1, 1}, +        {1, -1, 1}, +    }; + +    for (Common::Vec3f& point : cube) { +        point.RotateFromOrigin(euler.x, euler.y, euler.z); +        point *= size; +    } + +    const std::array<QPointF, 4> front_face{ +        center + QPointF{cube[0].x, cube[0].y}, +        center + QPointF{cube[1].x, cube[1].y}, +        center + QPointF{cube[2].x, cube[2].y}, +        center + QPointF{cube[3].x, cube[3].y}, +    }; +    const std::array<QPointF, 4> back_face{ +        center + QPointF{cube[4].x, cube[4].y}, +        center + QPointF{cube[5].x, cube[5].y}, +        center + QPointF{cube[6].x, cube[6].y}, +        center + QPointF{cube[7].x, cube[7].y}, +    }; + +    DrawPolygon(p, front_face); +    DrawPolygon(p, back_face); +    p.drawLine(center + QPointF{cube[0].x, cube[0].y}, center + QPointF{cube[4].x, cube[4].y}); +    p.drawLine(center + QPointF{cube[1].x, cube[1].y}, center + QPointF{cube[5].x, cube[5].y}); +    p.drawLine(center + QPointF{cube[2].x, cube[2].y}, center + QPointF{cube[6].x, cube[6].y}); +    p.drawLine(center + QPointF{cube[3].x, cube[3].y}, center + QPointF{cube[7].x, cube[7].y}); +} +  template <size_t N>  void PlayerControlPreview::DrawPolygon(QPainter& p, const std::array<QPointF, N>& polygon) {      p.drawPolygon(polygon.data(), static_cast<int>(polygon.size())); diff --git a/src/yuzu/configuration/configure_input_player_widget.h b/src/yuzu/configuration/configure_input_player_widget.h index 267d134de..a16943c3c 100644 --- a/src/yuzu/configuration/configure_input_player_widget.h +++ b/src/yuzu/configuration/configure_input_player_widget.h @@ -9,6 +9,7 @@  #include "common/input.h"  #include "common/settings_input.h" +#include "common/vector_math.h"  #include "core/hid/emulated_controller.h"  #include "core/hid/hid_types.h" @@ -193,6 +194,9 @@ private:      void DrawSymbol(QPainter& p, QPointF center, Symbol symbol, float icon_size);      void DrawArrow(QPainter& p, QPointF center, Direction direction, float size); +    // Draw motion functions +    void Draw3dCube(QPainter& p, QPointF center, const Common::Vec3f& euler, float size); +      // Draw primitive types      template <size_t N>      void DrawPolygon(QPainter& p, const std::array<QPointF, N>& polygon); @@ -222,4 +226,5 @@ private:      Core::HID::SticksValues stick_values{};      Core::HID::TriggerValues trigger_values{};      Core::HID::BatteryValues battery_values{}; +    Core::HID::MotionState motion_values{};  }; diff --git a/src/yuzu_cmd/config.cpp b/src/yuzu_cmd/config.cpp index 605280949..352e6a4c7 100644 --- a/src/yuzu_cmd/config.cpp +++ b/src/yuzu_cmd/config.cpp @@ -4,18 +4,8 @@  #include <memory>  #include <optional>  #include <sstream> - -// Ignore -Wimplicit-fallthrough due to https://github.com/libsdl-org/SDL/issues/4307 -#ifdef __clang__ -#pragma clang diagnostic push -#pragma clang diagnostic ignored "-Wimplicit-fallthrough" -#endif -#include <SDL.h> -#ifdef __clang__ -#pragma clang diagnostic pop -#endif -  #include <INIReader.h> +#include <SDL.h>  #include "common/fs/file.h"  #include "common/fs/fs.h"  #include "common/fs/path_util.h" | 
