diff options
| -rw-r--r-- | src/core/file_sys/disk_filesystem.cpp | 21 | ||||
| -rw-r--r-- | src/core/file_sys/filesystem.cpp | 6 | ||||
| -rw-r--r-- | src/core/file_sys/partition_filesystem.cpp | 3 | ||||
| -rw-r--r-- | src/core/file_sys/program_metadata.cpp | 39 | ||||
| -rw-r--r-- | src/core/file_sys/romfs_factory.cpp | 6 | ||||
| -rw-r--r-- | src/core/file_sys/romfs_filesystem.cpp | 38 | ||||
| -rw-r--r-- | src/core/file_sys/savedata_factory.cpp | 9 | ||||
| -rw-r--r-- | src/core/file_sys/sdmc_factory.cpp | 5 | ||||
| -rw-r--r-- | src/core/memory.cpp | 4 | ||||
| -rw-r--r-- | src/video_core/command_processor.cpp | 27 | ||||
| -rw-r--r-- | src/video_core/engines/fermi_2d.cpp | 61 | ||||
| -rw-r--r-- | src/video_core/engines/fermi_2d.h | 89 | ||||
| -rw-r--r-- | src/video_core/engines/maxwell_3d.cpp | 20 | ||||
| -rw-r--r-- | src/video_core/engines/maxwell_3d.h | 21 | ||||
| -rw-r--r-- | src/video_core/gpu.cpp | 14 | ||||
| -rw-r--r-- | src/video_core/gpu.h | 10 | ||||
| -rw-r--r-- | src/video_core/textures/decoders.cpp | 5 | ||||
| -rw-r--r-- | src/video_core/textures/decoders.h | 4 | 
18 files changed, 262 insertions, 120 deletions
| diff --git a/src/core/file_sys/disk_filesystem.cpp b/src/core/file_sys/disk_filesystem.cpp index 4d00249fa..8aa0e0aa4 100644 --- a/src/core/file_sys/disk_filesystem.cpp +++ b/src/core/file_sys/disk_filesystem.cpp @@ -80,19 +80,19 @@ ResultCode Disk_FileSystem::RenameFile(const std::string& src_path,  }  ResultCode Disk_FileSystem::DeleteDirectory(const Path& path) const { -    LOG_WARNING(Service_FS, "(STUBBED) called"); +    NGLOG_WARNING(Service_FS, "(STUBBED) called");      // TODO(wwylele): Use correct error code      return ResultCode(-1);  }  ResultCode Disk_FileSystem::DeleteDirectoryRecursively(const Path& path) const { -    LOG_WARNING(Service_FS, "(STUBBED) called"); +    NGLOG_WARNING(Service_FS, "(STUBBED) called");      // TODO(wwylele): Use correct error code      return ResultCode(-1);  }  ResultCode Disk_FileSystem::CreateFile(const std::string& path, u64 size) const { -    LOG_WARNING(Service_FS, "(STUBBED) called"); +    NGLOG_WARNING(Service_FS, "(STUBBED) called");      std::string full_path = base_directory + path;      if (size == 0) { @@ -107,7 +107,7 @@ ResultCode Disk_FileSystem::CreateFile(const std::string& path, u64 size) const          return RESULT_SUCCESS;      } -    LOG_ERROR(Service_FS, "Too large file"); +    NGLOG_ERROR(Service_FS, "Too large file");      // TODO(Subv): Find out the correct error code      return ResultCode(-1);  } @@ -120,13 +120,13 @@ ResultCode Disk_FileSystem::CreateDirectory(const std::string& path) const {          return RESULT_SUCCESS;      } -    LOG_CRITICAL(Service_FS, "(unreachable) Unknown error creating %s", full_path.c_str()); +    NGLOG_CRITICAL(Service_FS, "(unreachable) Unknown error creating {}", full_path);      // TODO(wwylele): Use correct error code      return ResultCode(-1);  }  ResultCode Disk_FileSystem::RenameDirectory(const Path& src_path, const Path& dest_path) const { -    LOG_WARNING(Service_FS, "(STUBBED) called"); +    NGLOG_WARNING(Service_FS, "(STUBBED) called");      // TODO(wwylele): Use correct error code      return ResultCode(-1);  } @@ -146,7 +146,7 @@ ResultVal<std::unique_ptr<DirectoryBackend>> Disk_FileSystem::OpenDirectory(  }  u64 Disk_FileSystem::GetFreeSpaceSize() const { -    LOG_WARNING(Service_FS, "(STUBBED) called"); +    NGLOG_WARNING(Service_FS, "(STUBBED) called");      return 0;  } @@ -163,14 +163,14 @@ ResultVal<FileSys::EntryType> Disk_FileSystem::GetEntryType(const std::string& p  }  ResultVal<size_t> Disk_Storage::Read(const u64 offset, const size_t length, u8* buffer) const { -    LOG_TRACE(Service_FS, "called offset=%llu, length=%zu", offset, length); +    NGLOG_TRACE(Service_FS, "called offset={}, length={}", offset, length);      file->Seek(offset, SEEK_SET);      return MakeResult<size_t>(file->ReadBytes(buffer, length));  }  ResultVal<size_t> Disk_Storage::Write(const u64 offset, const size_t length, const bool flush,                                        const u8* buffer) const { -    LOG_WARNING(Service_FS, "(STUBBED) called"); +    NGLOG_WARNING(Service_FS, "(STUBBED) called");      file->Seek(offset, SEEK_SET);      size_t written = file->WriteBytes(buffer, length);      if (flush) { @@ -204,8 +204,7 @@ u64 Disk_Directory::Read(const u64 count, Entry* entries) {          const std::string& filename = file.virtualName;          Entry& entry = entries[entries_read]; -        LOG_TRACE(Service_FS, "File %s: size=%llu dir=%d", filename.c_str(), file.size, -                  file.isDirectory); +        NGLOG_TRACE(Service_FS, "File {}: size={} dir={}", filename, file.size, file.isDirectory);          // TODO(Link Mauve): use a proper conversion to UTF-16.          for (size_t j = 0; j < FILENAME_LENGTH; ++j) { diff --git a/src/core/file_sys/filesystem.cpp b/src/core/file_sys/filesystem.cpp index 82fdb3c46..87083878b 100644 --- a/src/core/file_sys/filesystem.cpp +++ b/src/core/file_sys/filesystem.cpp @@ -71,7 +71,7 @@ std::string Path::AsString() const {      case Binary:      default:          // TODO(yuriks): Add assert -        LOG_ERROR(Service_FS, "LowPathType cannot be converted to string!"); +        NGLOG_ERROR(Service_FS, "LowPathType cannot be converted to string!");          return {};      }  } @@ -87,7 +87,7 @@ std::u16string Path::AsU16Str() const {      case Invalid:      case Binary:          // TODO(yuriks): Add assert -        LOG_ERROR(Service_FS, "LowPathType cannot be converted to u16string!"); +        NGLOG_ERROR(Service_FS, "LowPathType cannot be converted to u16string!");          return {};      } @@ -115,7 +115,7 @@ std::vector<u8> Path::AsBinary() const {      case Invalid:      default:          // TODO(yuriks): Add assert -        LOG_ERROR(Service_FS, "LowPathType cannot be converted to binary!"); +        NGLOG_ERROR(Service_FS, "LowPathType cannot be converted to binary!");          return {};      }  } diff --git a/src/core/file_sys/partition_filesystem.cpp b/src/core/file_sys/partition_filesystem.cpp index 4a58a9291..808254ecc 100644 --- a/src/core/file_sys/partition_filesystem.cpp +++ b/src/core/file_sys/partition_filesystem.cpp @@ -2,7 +2,6 @@  // Licensed under GPLv2 or any later version  // Refer to the license.txt file included. -#include <cinttypes>  #include <utility>  #include "common/file_util.h"  #include "common/logging/log.h" @@ -40,7 +39,7 @@ Loader::ResultStatus PartitionFilesystem::Load(const std::string& file_path, siz      Loader::ResultStatus result = Load(file_data);      if (result != Loader::ResultStatus::Success) -        LOG_ERROR(Service_FS, "Failed to load PFS from file %s!", file_path.c_str()); +        NGLOG_ERROR(Service_FS, "Failed to load PFS from file {}!", file_path);      return result;  } diff --git a/src/core/file_sys/program_metadata.cpp b/src/core/file_sys/program_metadata.cpp index a6dcebcc3..1f5ded514 100644 --- a/src/core/file_sys/program_metadata.cpp +++ b/src/core/file_sys/program_metadata.cpp @@ -2,7 +2,6 @@  // Licensed under GPLv2 or any later version  // Refer to the license.txt file included. -#include <cinttypes>  #include "common/file_util.h"  #include "common/logging/log.h"  #include "core/file_sys/program_metadata.h" @@ -22,7 +21,7 @@ Loader::ResultStatus ProgramMetadata::Load(const std::string& file_path) {      Loader::ResultStatus result = Load(file_data);      if (result != Loader::ResultStatus::Success) -        LOG_ERROR(Service_FS, "Failed to load NPDM from file %s!", file_path.c_str()); +        NGLOG_ERROR(Service_FS, "Failed to load NPDM from file {}!", file_path);      return result;  } @@ -77,14 +76,14 @@ u64 ProgramMetadata::GetFilesystemPermissions() const {  }  void ProgramMetadata::Print() const { -    LOG_DEBUG(Service_FS, "Magic:                  %.4s", npdm_header.magic.data()); -    LOG_DEBUG(Service_FS, "Main thread priority:   0x%02x", npdm_header.main_thread_priority); -    LOG_DEBUG(Service_FS, "Main thread core:       %u", npdm_header.main_thread_cpu); -    LOG_DEBUG(Service_FS, "Main thread stack size: 0x%x bytes", npdm_header.main_stack_size); -    LOG_DEBUG(Service_FS, "Process category:       %u", npdm_header.process_category); -    LOG_DEBUG(Service_FS, "Flags:                  %02x", npdm_header.flags); -    LOG_DEBUG(Service_FS, " > 64-bit instructions: %s", -              npdm_header.has_64_bit_instructions ? "YES" : "NO"); +    NGLOG_DEBUG(Service_FS, "Magic:                  {:.4}", npdm_header.magic.data()); +    NGLOG_DEBUG(Service_FS, "Main thread priority:   {:#04X}", npdm_header.main_thread_priority); +    NGLOG_DEBUG(Service_FS, "Main thread core:       {}", npdm_header.main_thread_cpu); +    NGLOG_DEBUG(Service_FS, "Main thread stack size: {:#X} bytes", npdm_header.main_stack_size); +    NGLOG_DEBUG(Service_FS, "Process category:       {}", npdm_header.process_category); +    NGLOG_DEBUG(Service_FS, "Flags:                  {:02X}", npdm_header.flags); +    NGLOG_DEBUG(Service_FS, " > 64-bit instructions: {}", +                npdm_header.has_64_bit_instructions ? "YES" : "NO");      auto address_space = "Unknown";      switch (npdm_header.address_space_type) { @@ -96,19 +95,19 @@ void ProgramMetadata::Print() const {          break;      } -    LOG_DEBUG(Service_FS, " > Address space:       %s\n", address_space); +    NGLOG_DEBUG(Service_FS, " > Address space:       {}\n", address_space);      // Begin ACID printing (potential perms, signed) -    LOG_DEBUG(Service_FS, "Magic:                  %.4s", acid_header.magic.data()); -    LOG_DEBUG(Service_FS, "Flags:                  %02x", acid_header.flags); -    LOG_DEBUG(Service_FS, " > Is Retail:           %s", acid_header.is_retail ? "YES" : "NO"); -    LOG_DEBUG(Service_FS, "Title ID Min:           %016" PRIX64, acid_header.title_id_min); -    LOG_DEBUG(Service_FS, "Title ID Max:           %016" PRIX64, acid_header.title_id_max); -    LOG_DEBUG(Service_FS, "Filesystem Access:      %016" PRIX64 "\n", acid_file_access.permissions); +    NGLOG_DEBUG(Service_FS, "Magic:                  {:.4}", acid_header.magic.data()); +    NGLOG_DEBUG(Service_FS, "Flags:                  {:02X}", acid_header.flags); +    NGLOG_DEBUG(Service_FS, " > Is Retail:           {}", acid_header.is_retail ? "YES" : "NO"); +    NGLOG_DEBUG(Service_FS, "Title ID Min:           {:016X}", acid_header.title_id_min); +    NGLOG_DEBUG(Service_FS, "Title ID Max:           {:016X}", acid_header.title_id_max); +    NGLOG_DEBUG(Service_FS, "Filesystem Access:      {:016X}\n", acid_file_access.permissions);      // Begin ACI0 printing (actual perms, unsigned) -    LOG_DEBUG(Service_FS, "Magic:                  %.4s", aci_header.magic.data()); -    LOG_DEBUG(Service_FS, "Title ID:               %016" PRIX64, aci_header.title_id); -    LOG_DEBUG(Service_FS, "Filesystem Access:      %016" PRIX64 "\n", aci_file_access.permissions); +    NGLOG_DEBUG(Service_FS, "Magic:                  {:.4}", aci_header.magic.data()); +    NGLOG_DEBUG(Service_FS, "Title ID:               {:016X}", aci_header.title_id); +    NGLOG_DEBUG(Service_FS, "Filesystem Access:      {:016X}\n", aci_file_access.permissions);  }  } // namespace FileSys diff --git a/src/core/file_sys/romfs_factory.cpp b/src/core/file_sys/romfs_factory.cpp index b21427948..dc7591aca 100644 --- a/src/core/file_sys/romfs_factory.cpp +++ b/src/core/file_sys/romfs_factory.cpp @@ -14,7 +14,7 @@ namespace FileSys {  RomFS_Factory::RomFS_Factory(Loader::AppLoader& app_loader) {      // Load the RomFS from the app      if (Loader::ResultStatus::Success != app_loader.ReadRomFS(romfs_file, data_offset, data_size)) { -        LOG_ERROR(Service_FS, "Unable to read RomFS!"); +        NGLOG_ERROR(Service_FS, "Unable to read RomFS!");      }  } @@ -24,13 +24,13 @@ ResultVal<std::unique_ptr<FileSystemBackend>> RomFS_Factory::Open(const Path& pa  }  ResultCode RomFS_Factory::Format(const Path& path) { -    LOG_ERROR(Service_FS, "Unimplemented Format archive %s", GetName().c_str()); +    NGLOG_ERROR(Service_FS, "Unimplemented Format archive {}", GetName());      // TODO(bunnei): Find the right error code for this      return ResultCode(-1);  }  ResultVal<ArchiveFormatInfo> RomFS_Factory::GetFormatInfo(const Path& path) const { -    LOG_ERROR(Service_FS, "Unimplemented GetFormatInfo archive %s", GetName().c_str()); +    NGLOG_ERROR(Service_FS, "Unimplemented GetFormatInfo archive {}", GetName());      // TODO(bunnei): Find the right error code for this      return ResultCode(-1);  } diff --git a/src/core/file_sys/romfs_filesystem.cpp b/src/core/file_sys/romfs_filesystem.cpp index b9982e6fa..8e2bce687 100644 --- a/src/core/file_sys/romfs_filesystem.cpp +++ b/src/core/file_sys/romfs_filesystem.cpp @@ -21,74 +21,72 @@ ResultVal<std::unique_ptr<StorageBackend>> RomFS_FileSystem::OpenFile(const std:  }  ResultCode RomFS_FileSystem::DeleteFile(const std::string& path) const { -    LOG_CRITICAL(Service_FS, "Attempted to delete a file from an ROMFS archive (%s).", -                 GetName().c_str()); +    NGLOG_CRITICAL(Service_FS, "Attempted to delete a file from an ROMFS archive ({}).", GetName());      // TODO(bunnei): Use correct error code      return ResultCode(-1);  }  ResultCode RomFS_FileSystem::RenameFile(const std::string& src_path,                                          const std::string& dest_path) const { -    LOG_CRITICAL(Service_FS, "Attempted to rename a file within an ROMFS archive (%s).", -                 GetName().c_str()); +    NGLOG_CRITICAL(Service_FS, "Attempted to rename a file within an ROMFS archive ({}).", +                   GetName());      // TODO(wwylele): Use correct error code      return ResultCode(-1);  }  ResultCode RomFS_FileSystem::DeleteDirectory(const Path& path) const { -    LOG_CRITICAL(Service_FS, "Attempted to delete a directory from an ROMFS archive (%s).", -                 GetName().c_str()); +    NGLOG_CRITICAL(Service_FS, "Attempted to delete a directory from an ROMFS archive ({}).", +                   GetName());      // TODO(wwylele): Use correct error code      return ResultCode(-1);  }  ResultCode RomFS_FileSystem::DeleteDirectoryRecursively(const Path& path) const { -    LOG_CRITICAL(Service_FS, "Attempted to delete a directory from an ROMFS archive (%s).", -                 GetName().c_str()); +    NGLOG_CRITICAL(Service_FS, "Attempted to delete a directory from an ROMFS archive ({}).", +                   GetName());      // TODO(wwylele): Use correct error code      return ResultCode(-1);  }  ResultCode RomFS_FileSystem::CreateFile(const std::string& path, u64 size) const { -    LOG_CRITICAL(Service_FS, "Attempted to create a file in an ROMFS archive (%s).", -                 GetName().c_str()); +    NGLOG_CRITICAL(Service_FS, "Attempted to create a file in an ROMFS archive ({}).", GetName());      // TODO(bunnei): Use correct error code      return ResultCode(-1);  }  ResultCode RomFS_FileSystem::CreateDirectory(const std::string& path) const { -    LOG_CRITICAL(Service_FS, "Attempted to create a directory in an ROMFS archive (%s).", -                 GetName().c_str()); +    NGLOG_CRITICAL(Service_FS, "Attempted to create a directory in an ROMFS archive ({}).", +                   GetName());      // TODO(wwylele): Use correct error code      return ResultCode(-1);  }  ResultCode RomFS_FileSystem::RenameDirectory(const Path& src_path, const Path& dest_path) const { -    LOG_CRITICAL(Service_FS, "Attempted to rename a file within an ROMFS archive (%s).", -                 GetName().c_str()); +    NGLOG_CRITICAL(Service_FS, "Attempted to rename a file within an ROMFS archive ({}).", +                   GetName());      // TODO(wwylele): Use correct error code      return ResultCode(-1);  }  ResultVal<std::unique_ptr<DirectoryBackend>> RomFS_FileSystem::OpenDirectory(      const std::string& path) const { -    LOG_WARNING(Service_FS, "Opening Directory in a ROMFS archive"); +    NGLOG_WARNING(Service_FS, "Opening Directory in a ROMFS archive");      return MakeResult<std::unique_ptr<DirectoryBackend>>(std::make_unique<ROMFSDirectory>());  }  u64 RomFS_FileSystem::GetFreeSpaceSize() const { -    LOG_WARNING(Service_FS, "Attempted to get the free space in an ROMFS archive"); +    NGLOG_WARNING(Service_FS, "Attempted to get the free space in an ROMFS archive");      return 0;  }  ResultVal<FileSys::EntryType> RomFS_FileSystem::GetEntryType(const std::string& path) const { -    LOG_CRITICAL(Service_FS, "Called within an ROMFS archive (path %s).", path.c_str()); +    NGLOG_CRITICAL(Service_FS, "Called within an ROMFS archive (path {}).", path);      // TODO(wwylele): Use correct error code      return ResultCode(-1);  }  ResultVal<size_t> RomFS_Storage::Read(const u64 offset, const size_t length, u8* buffer) const { -    LOG_TRACE(Service_FS, "called offset=%llu, length=%zu", offset, length); +    NGLOG_TRACE(Service_FS, "called offset={}, length={}", offset, length);      romfs_file->Seek(data_offset + offset, SEEK_SET);      size_t read_length = (size_t)std::min((u64)length, data_size - offset); @@ -97,7 +95,7 @@ ResultVal<size_t> RomFS_Storage::Read(const u64 offset, const size_t length, u8*  ResultVal<size_t> RomFS_Storage::Write(const u64 offset, const size_t length, const bool flush,                                         const u8* buffer) const { -    LOG_ERROR(Service_FS, "Attempted to write to ROMFS file"); +    NGLOG_ERROR(Service_FS, "Attempted to write to ROMFS file");      // TODO(Subv): Find error code      return MakeResult<size_t>(0);  } @@ -107,7 +105,7 @@ u64 RomFS_Storage::GetSize() const {  }  bool RomFS_Storage::SetSize(const u64 size) const { -    LOG_ERROR(Service_FS, "Attempted to set the size of an ROMFS file"); +    NGLOG_ERROR(Service_FS, "Attempted to set the size of an ROMFS file");      return false;  } diff --git a/src/core/file_sys/savedata_factory.cpp b/src/core/file_sys/savedata_factory.cpp index 14868fed2..c1be8fee4 100644 --- a/src/core/file_sys/savedata_factory.cpp +++ b/src/core/file_sys/savedata_factory.cpp @@ -2,11 +2,9 @@  // Licensed under GPLv2 or any later version  // Refer to the license.txt file included. -#include <cinttypes>  #include <memory>  #include "common/common_types.h"  #include "common/logging/log.h" -#include "common/string_util.h"  #include "core/core.h"  #include "core/file_sys/disk_filesystem.h"  #include "core/file_sys/savedata_factory.h" @@ -30,7 +28,7 @@ ResultVal<std::unique_ptr<FileSystemBackend>> SaveData_Factory::Open(const Path&  }  ResultCode SaveData_Factory::Format(const Path& path) { -    LOG_WARNING(Service_FS, "Format archive %s", GetName().c_str()); +    NGLOG_WARNING(Service_FS, "Format archive {}", GetName());      // Create the save data directory.      if (!FileUtil::CreateFullPath(GetFullPath())) {          // TODO(Subv): Find the correct error code. @@ -41,7 +39,7 @@ ResultCode SaveData_Factory::Format(const Path& path) {  }  ResultVal<ArchiveFormatInfo> SaveData_Factory::GetFormatInfo(const Path& path) const { -    LOG_ERROR(Service_FS, "Unimplemented GetFormatInfo archive %s", GetName().c_str()); +    NGLOG_ERROR(Service_FS, "Unimplemented GetFormatInfo archive {}", GetName());      // TODO(bunnei): Find the right error code for this      return ResultCode(-1);  } @@ -50,8 +48,7 @@ std::string SaveData_Factory::GetFullPath() const {      u64 title_id = Core::CurrentProcess()->program_id;      // TODO(Subv): Somehow obtain this value.      u32 user = 0; -    return Common::StringFromFormat("%ssave/%016" PRIX64 "/%08X/", nand_directory.c_str(), title_id, -                                    user); +    return fmt::format("{}save/{:016X}/{:08X}/", nand_directory, title_id, user);  }  } // namespace FileSys diff --git a/src/core/file_sys/sdmc_factory.cpp b/src/core/file_sys/sdmc_factory.cpp index 00e80d2a7..59ac3e0be 100644 --- a/src/core/file_sys/sdmc_factory.cpp +++ b/src/core/file_sys/sdmc_factory.cpp @@ -2,7 +2,6 @@  // Licensed under GPLv2 or any later version  // Refer to the license.txt file included. -#include <cinttypes>  #include <memory>  #include "common/common_types.h"  #include "common/logging/log.h" @@ -26,13 +25,13 @@ ResultVal<std::unique_ptr<FileSystemBackend>> SDMC_Factory::Open(const Path& pat  }  ResultCode SDMC_Factory::Format(const Path& path) { -    LOG_ERROR(Service_FS, "Unimplemented Format archive %s", GetName().c_str()); +    NGLOG_ERROR(Service_FS, "Unimplemented Format archive {}", GetName());      // TODO(Subv): Find the right error code for this      return ResultCode(-1);  }  ResultVal<ArchiveFormatInfo> SDMC_Factory::GetFormatInfo(const Path& path) const { -    LOG_ERROR(Service_FS, "Unimplemented GetFormatInfo archive %s", GetName().c_str()); +    NGLOG_ERROR(Service_FS, "Unimplemented GetFormatInfo archive {}", GetName());      // TODO(bunnei): Find the right error code for this      return ResultCode(-1);  } diff --git a/src/core/memory.cpp b/src/core/memory.cpp index d7c0080fa..2afa0916d 100644 --- a/src/core/memory.cpp +++ b/src/core/memory.cpp @@ -659,6 +659,10 @@ void CopyBlock(const Kernel::Process& process, VAddr dest_addr, VAddr src_addr,      }  } +void CopyBlock(VAddr dest_addr, VAddr src_addr, size_t size) { +    CopyBlock(*Core::CurrentProcess(), dest_addr, src_addr, size); +} +  boost::optional<PAddr> TryVirtualToPhysicalAddress(const VAddr addr) {      if (addr == 0) {          return 0; diff --git a/src/video_core/command_processor.cpp b/src/video_core/command_processor.cpp index f6a88f031..2eaece298 100644 --- a/src/video_core/command_processor.cpp +++ b/src/video_core/command_processor.cpp @@ -24,10 +24,7 @@ namespace Tegra {  enum class BufferMethods {      BindObject = 0, -    SetGraphMacroCode = 0x45, -    SetGraphMacroCodeArg = 0x46, -    SetGraphMacroEntry = 0x47, -    CountBufferMethods = 0x100, +    CountBufferMethods = 0x40,  };  void GPU::WriteReg(u32 method, u32 subchannel, u32 value, u32 remaining_params) { @@ -36,28 +33,6 @@ void GPU::WriteReg(u32 method, u32 subchannel, u32 value, u32 remaining_params)                    "{:08X} remaining params {}",                    method, subchannel, value, remaining_params); -    if (method == static_cast<u32>(BufferMethods::SetGraphMacroEntry)) { -        // Prepare to upload a new macro, reset the upload counter. -        NGLOG_DEBUG(HW_GPU, "Uploading GPU macro {:08X}", value); -        current_macro_entry = value; -        current_macro_code.clear(); -        return; -    } - -    if (method == static_cast<u32>(BufferMethods::SetGraphMacroCodeArg)) { -        // Append a new code word to the current macro. -        current_macro_code.push_back(value); - -        // There are no more params remaining, submit the code to the 3D engine. -        if (remaining_params == 0) { -            maxwell_3d->SubmitMacroCode(current_macro_entry, std::move(current_macro_code)); -            current_macro_entry = InvalidGraphMacroEntry; -            current_macro_code.clear(); -        } - -        return; -    } -      if (method == static_cast<u32>(BufferMethods::BindObject)) {          // Bind the current subchannel to the desired engine id.          NGLOG_DEBUG(HW_GPU, "Binding subchannel {} to engine {}", subchannel, value); diff --git a/src/video_core/engines/fermi_2d.cpp b/src/video_core/engines/fermi_2d.cpp index 7aab163dc..9019f2504 100644 --- a/src/video_core/engines/fermi_2d.cpp +++ b/src/video_core/engines/fermi_2d.cpp @@ -2,12 +2,71 @@  // Licensed under GPLv2 or any later version  // Refer to the license.txt file included. +#include "core/memory.h"  #include "video_core/engines/fermi_2d.h" +#include "video_core/textures/decoders.h"  namespace Tegra {  namespace Engines { -void Fermi2D::WriteReg(u32 method, u32 value) {} +Fermi2D::Fermi2D(MemoryManager& memory_manager) : memory_manager(memory_manager) {} + +void Fermi2D::WriteReg(u32 method, u32 value) { +    ASSERT_MSG(method < Regs::NUM_REGS, +               "Invalid Fermi2D register, increase the size of the Regs structure"); + +    regs.reg_array[method] = value; + +    switch (method) { +    case FERMI2D_REG_INDEX(trigger): { +        HandleSurfaceCopy(); +        break; +    } +    } +} + +void Fermi2D::HandleSurfaceCopy() { +    NGLOG_WARNING(HW_GPU, "Requested a surface copy with operation {}", +                  static_cast<u32>(regs.operation)); + +    const GPUVAddr source = regs.src.Address(); +    const GPUVAddr dest = regs.dst.Address(); + +    // TODO(Subv): Only same-format and same-size copies are allowed for now. +    ASSERT(regs.src.format == regs.dst.format); +    ASSERT(regs.src.width * regs.src.height == regs.dst.width * regs.dst.height); + +    // TODO(Subv): Only raw copies are implemented. +    ASSERT(regs.operation == Regs::Operation::SrcCopy); + +    const VAddr source_cpu = *memory_manager.GpuToCpuAddress(source); +    const VAddr dest_cpu = *memory_manager.GpuToCpuAddress(dest); + +    u32 src_bytes_per_pixel = RenderTargetBytesPerPixel(regs.src.format); +    u32 dst_bytes_per_pixel = RenderTargetBytesPerPixel(regs.dst.format); + +    if (regs.src.linear == regs.dst.linear) { +        // If the input layout and the output layout are the same, just perform a raw copy. +        Memory::CopyBlock(dest_cpu, source_cpu, +                          src_bytes_per_pixel * regs.dst.width * regs.dst.height); +        return; +    } + +    u8* src_buffer = Memory::GetPointer(source_cpu); +    u8* dst_buffer = Memory::GetPointer(dest_cpu); + +    if (!regs.src.linear && regs.dst.linear) { +        // If the input is tiled and the output is linear, deswizzle the input and copy it over. +        Texture::CopySwizzledData(regs.src.width, regs.src.height, src_bytes_per_pixel, +                                  dst_bytes_per_pixel, src_buffer, dst_buffer, true, +                                  regs.src.block_height); +    } else { +        // If the input is linear and the output is tiled, swizzle the input and copy it over. +        Texture::CopySwizzledData(regs.src.width, regs.src.height, src_bytes_per_pixel, +                                  dst_bytes_per_pixel, dst_buffer, src_buffer, false, +                                  regs.dst.block_height); +    } +}  } // namespace Engines  } // namespace Tegra diff --git a/src/video_core/engines/fermi_2d.h b/src/video_core/engines/fermi_2d.h index 8967ddede..0c5b413cc 100644 --- a/src/video_core/engines/fermi_2d.h +++ b/src/video_core/engines/fermi_2d.h @@ -4,19 +4,106 @@  #pragma once +#include <array> +#include "common/assert.h" +#include "common/bit_field.h" +#include "common/common_funcs.h"  #include "common/common_types.h" +#include "video_core/gpu.h" +#include "video_core/memory_manager.h"  namespace Tegra {  namespace Engines { +#define FERMI2D_REG_INDEX(field_name)                                                              \ +    (offsetof(Tegra::Engines::Fermi2D::Regs, field_name) / sizeof(u32)) +  class Fermi2D final {  public: -    Fermi2D() = default; +    explicit Fermi2D(MemoryManager& memory_manager);      ~Fermi2D() = default;      /// Write the value to the register identified by method.      void WriteReg(u32 method, u32 value); + +    struct Regs { +        static constexpr size_t NUM_REGS = 0x258; + +        struct Surface { +            RenderTargetFormat format; +            BitField<0, 1, u32> linear; +            union { +                BitField<0, 4, u32> block_depth; +                BitField<4, 4, u32> block_height; +                BitField<8, 4, u32> block_width; +            }; +            u32 depth; +            u32 layer; +            u32 pitch; +            u32 width; +            u32 height; +            u32 address_high; +            u32 address_low; + +            GPUVAddr Address() const { +                return static_cast<GPUVAddr>((static_cast<GPUVAddr>(address_high) << 32) | +                                             address_low); +            } +        }; +        static_assert(sizeof(Surface) == 0x28, "Surface has incorrect size"); + +        enum class Operation : u32 { +            SrcCopyAnd = 0, +            ROPAnd = 1, +            Blend = 2, +            SrcCopy = 3, +            ROP = 4, +            SrcCopyPremult = 5, +            BlendPremult = 6, +        }; + +        union { +            struct { +                INSERT_PADDING_WORDS(0x80); + +                Surface dst; + +                INSERT_PADDING_WORDS(2); + +                Surface src; + +                INSERT_PADDING_WORDS(0x15); + +                Operation operation; + +                INSERT_PADDING_WORDS(0x9); + +                // TODO(Subv): This is only a guess. +                u32 trigger; + +                INSERT_PADDING_WORDS(0x1A3); +            }; +            std::array<u32, NUM_REGS> reg_array; +        }; +    } regs{}; + +    MemoryManager& memory_manager; + +private: +    /// Performs the copy from the source surface to the destination surface as configured in the +    /// registers. +    void HandleSurfaceCopy();  }; +#define ASSERT_REG_POSITION(field_name, position)                                                  \ +    static_assert(offsetof(Fermi2D::Regs, field_name) == position * 4,                             \ +                  "Field " #field_name " has invalid position") + +ASSERT_REG_POSITION(dst, 0x80); +ASSERT_REG_POSITION(src, 0x8C); +ASSERT_REG_POSITION(operation, 0xAB); +ASSERT_REG_POSITION(trigger, 0xB5); +#undef ASSERT_REG_POSITION +  } // namespace Engines  } // namespace Tegra diff --git a/src/video_core/engines/maxwell_3d.cpp b/src/video_core/engines/maxwell_3d.cpp index 2acbb9cd6..4306b894f 100644 --- a/src/video_core/engines/maxwell_3d.cpp +++ b/src/video_core/engines/maxwell_3d.cpp @@ -22,10 +22,6 @@ constexpr u32 MacroRegistersStart = 0xE00;  Maxwell3D::Maxwell3D(MemoryManager& memory_manager)      : memory_manager(memory_manager), macro_interpreter(*this) {} -void Maxwell3D::SubmitMacroCode(u32 entry, std::vector<u32> code) { -    uploaded_macros[entry * 2 + MacroRegistersStart] = std::move(code); -} -  void Maxwell3D::CallMacroMethod(u32 method, std::vector<u32> parameters) {      auto macro_code = uploaded_macros.find(method);      // The requested macro must have been uploaded already. @@ -37,9 +33,6 @@ void Maxwell3D::CallMacroMethod(u32 method, std::vector<u32> parameters) {  }  void Maxwell3D::WriteReg(u32 method, u32 value, u32 remaining_params) { -    ASSERT_MSG(method < Regs::NUM_REGS, -               "Invalid Maxwell3D register, increase the size of the Regs structure"); -      auto debug_context = Core::System::GetInstance().GetGPUDebugContext();      // It is an error to write to a register other than the current macro's ARG register before it @@ -68,6 +61,9 @@ void Maxwell3D::WriteReg(u32 method, u32 value, u32 remaining_params) {          return;      } +    ASSERT_MSG(method < Regs::NUM_REGS, +               "Invalid Maxwell3D register, increase the size of the Regs structure"); +      if (debug_context) {          debug_context->OnEvent(Tegra::DebugContext::Event::MaxwellCommandLoaded, nullptr);      } @@ -75,6 +71,10 @@ void Maxwell3D::WriteReg(u32 method, u32 value, u32 remaining_params) {      regs.reg_array[method] = value;      switch (method) { +    case MAXWELL3D_REG_INDEX(macros.data): { +        ProcessMacroUpload(value); +        break; +    }      case MAXWELL3D_REG_INDEX(code_address.code_address_high):      case MAXWELL3D_REG_INDEX(code_address.code_address_low): {          // Note: For some reason games (like Puyo Puyo Tetris) seem to write 0 to the CODE_ADDRESS @@ -141,6 +141,12 @@ void Maxwell3D::WriteReg(u32 method, u32 value, u32 remaining_params) {      }  } +void Maxwell3D::ProcessMacroUpload(u32 data) { +    // Store the uploaded macro code to interpret them when they're called. +    auto& macro = uploaded_macros[regs.macros.entry * 2 + MacroRegistersStart]; +    macro.push_back(data); +} +  void Maxwell3D::ProcessQueryGet() {      GPUVAddr sequence_address = regs.query.QueryAddress();      // Since the sequence address is given as a GPU VAddr, we have to convert it to an application diff --git a/src/video_core/engines/maxwell_3d.h b/src/video_core/engines/maxwell_3d.h index a022665eb..5cf62fb01 100644 --- a/src/video_core/engines/maxwell_3d.h +++ b/src/video_core/engines/maxwell_3d.h @@ -31,7 +31,7 @@ public:      /// Register structure of the Maxwell3D engine.      /// TODO(Subv): This structure will need to be made bigger as more registers are discovered.      struct Regs { -        static constexpr size_t NUM_REGS = 0xE36; +        static constexpr size_t NUM_REGS = 0xE00;          static constexpr size_t NumRenderTargets = 8;          static constexpr size_t NumViewports = 16; @@ -322,7 +322,15 @@ public:          union {              struct { -                INSERT_PADDING_WORDS(0x200); +                INSERT_PADDING_WORDS(0x45); + +                struct { +                    INSERT_PADDING_WORDS(1); +                    u32 data; +                    u32 entry; +                } macros; + +                INSERT_PADDING_WORDS(0x1B8);                  struct {                      u32 address_high; @@ -605,7 +613,7 @@ public:                      u32 size[MaxShaderStage];                  } tex_info_buffers; -                INSERT_PADDING_WORDS(0x102); +                INSERT_PADDING_WORDS(0xCC);              };              std::array<u32, NUM_REGS> reg_array;          }; @@ -637,9 +645,6 @@ public:      /// Write the value to the register identified by method.      void WriteReg(u32 method, u32 value, u32 remaining_params); -    /// Uploads the code for a GPU macro program associated with the specified entry. -    void SubmitMacroCode(u32 entry, std::vector<u32> code); -      /// Returns a list of enabled textures for the specified shader stage.      std::vector<Texture::FullTextureInfo> GetStageTextures(Regs::ShaderStage stage) const; @@ -670,6 +675,9 @@ private:       */      void CallMacroMethod(u32 method, std::vector<u32> parameters); +    /// Handles writes to the macro uploading registers. +    void ProcessMacroUpload(u32 data); +      /// Handles a write to the QUERY_GET register.      void ProcessQueryGet(); @@ -687,6 +695,7 @@ private:      static_assert(offsetof(Maxwell3D::Regs, field_name) == position * 4,                           \                    "Field " #field_name " has invalid position") +ASSERT_REG_POSITION(macros, 0x45);  ASSERT_REG_POSITION(rt, 0x200);  ASSERT_REG_POSITION(viewport_transform[0], 0x280);  ASSERT_REG_POSITION(viewport, 0x300); diff --git a/src/video_core/gpu.cpp b/src/video_core/gpu.cpp index 9463cd5d6..9eb143918 100644 --- a/src/video_core/gpu.cpp +++ b/src/video_core/gpu.cpp @@ -12,7 +12,7 @@ namespace Tegra {  GPU::GPU() {      memory_manager = std::make_unique<MemoryManager>();      maxwell_3d = std::make_unique<Engines::Maxwell3D>(*memory_manager); -    fermi_2d = std::make_unique<Engines::Fermi2D>(); +    fermi_2d = std::make_unique<Engines::Fermi2D>(*memory_manager);      maxwell_compute = std::make_unique<Engines::MaxwellCompute>();  } @@ -22,4 +22,16 @@ const Tegra::Engines::Maxwell3D& GPU::Get3DEngine() const {      return *maxwell_3d;  } +u32 RenderTargetBytesPerPixel(RenderTargetFormat format) { +    ASSERT(format != RenderTargetFormat::NONE); + +    switch (format) { +    case RenderTargetFormat::RGBA8_UNORM: +    case RenderTargetFormat::RGB10_A2_UNORM: +        return 4; +    default: +        UNIMPLEMENTED_MSG("Unimplemented render target format %u", static_cast<u32>(format)); +    } +} +  } // namespace Tegra diff --git a/src/video_core/gpu.h b/src/video_core/gpu.h index 2888daedc..f168a5171 100644 --- a/src/video_core/gpu.h +++ b/src/video_core/gpu.h @@ -21,6 +21,9 @@ enum class RenderTargetFormat : u32 {      RGBA8_SRGB = 0xD6,  }; +/// Returns the number of bytes per pixel of each rendertarget format. +u32 RenderTargetBytesPerPixel(RenderTargetFormat format); +  class DebugContext;  /** @@ -86,8 +89,6 @@ public:      }  private: -    static constexpr u32 InvalidGraphMacroEntry = 0xFFFFFFFF; -      /// Writes a single register in the engine bound to the specified subchannel      void WriteReg(u32 method, u32 subchannel, u32 value, u32 remaining_params); @@ -100,11 +101,6 @@ private:      std::unique_ptr<Engines::Fermi2D> fermi_2d;      /// Compute engine      std::unique_ptr<Engines::MaxwellCompute> maxwell_compute; - -    /// Entry of the macro that is currently being uploaded -    u32 current_macro_entry = InvalidGraphMacroEntry; -    /// Code being uploaded for the current macro -    std::vector<u32> current_macro_code;  };  } // namespace Tegra diff --git a/src/video_core/textures/decoders.cpp b/src/video_core/textures/decoders.cpp index 9c3ae875c..8b39b2bdf 100644 --- a/src/video_core/textures/decoders.cpp +++ b/src/video_core/textures/decoders.cpp @@ -27,9 +27,8 @@ static u32 GetSwizzleOffset(u32 x, u32 y, u32 image_width, u32 bytes_per_pixel,      return address;  } -static void CopySwizzledData(u32 width, u32 height, u32 bytes_per_pixel, u32 out_bytes_per_pixel, -                             u8* swizzled_data, u8* unswizzled_data, bool unswizzle, -                             u32 block_height) { +void CopySwizzledData(u32 width, u32 height, u32 bytes_per_pixel, u32 out_bytes_per_pixel, +                      u8* swizzled_data, u8* unswizzled_data, bool unswizzle, u32 block_height) {      u8* data_ptrs[2];      for (unsigned y = 0; y < height; ++y) {          for (unsigned x = 0; x < width; ++x) { diff --git a/src/video_core/textures/decoders.h b/src/video_core/textures/decoders.h index a700911cf..2562c4b06 100644 --- a/src/video_core/textures/decoders.h +++ b/src/video_core/textures/decoders.h @@ -17,6 +17,10 @@ namespace Texture {  std::vector<u8> UnswizzleTexture(VAddr address, TextureFormat format, u32 width, u32 height,                                   u32 block_height = TICEntry::DefaultBlockHeight); +/// Copies texture data from a buffer and performs swizzling/unswizzling as necessary. +void CopySwizzledData(u32 width, u32 height, u32 bytes_per_pixel, u32 out_bytes_per_pixel, +                      u8* swizzled_data, u8* unswizzled_data, bool unswizzle, u32 block_height); +  /**   * Decodes an unswizzled texture into a A8R8G8B8 texture.   */ | 
