summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorReinUsesLisp <reinuseslisp@airmail.cc>2019-05-08 02:48:02 +0200
committerReinUsesLisp <reinuseslisp@airmail.cc>2019-06-21 02:36:12 +0200
commit2b30000a1ed1972e0701a8525182104b4544caa4 (patch)
treee66b712f76e6ebedf1da25a233736cf6765a3b38
parentcopy_params: Use constructor instead of C-like initialization (diff)
downloadyuzu-2b30000a1ed1972e0701a8525182104b4544caa4.tar
yuzu-2b30000a1ed1972e0701a8525182104b4544caa4.tar.gz
yuzu-2b30000a1ed1972e0701a8525182104b4544caa4.tar.bz2
yuzu-2b30000a1ed1972e0701a8525182104b4544caa4.tar.lz
yuzu-2b30000a1ed1972e0701a8525182104b4544caa4.tar.xz
yuzu-2b30000a1ed1972e0701a8525182104b4544caa4.tar.zst
yuzu-2b30000a1ed1972e0701a8525182104b4544caa4.zip
-rw-r--r--src/video_core/texture_cache/surface_base.cpp34
-rw-r--r--src/video_core/texture_cache/surface_base.h35
2 files changed, 37 insertions, 32 deletions
diff --git a/src/video_core/texture_cache/surface_base.cpp b/src/video_core/texture_cache/surface_base.cpp
index 0de0bc656..5e994cf08 100644
--- a/src/video_core/texture_cache/surface_base.cpp
+++ b/src/video_core/texture_cache/surface_base.cpp
@@ -18,17 +18,19 @@ MICROPROFILE_DEFINE(GPU_Flush_Texture, "GPU", "Texture Flush", MP_RGB(128, 192,
using Tegra::Texture::ConvertFromGuestToHost;
using VideoCore::MortonSwizzleMode;
-SurfaceBaseImpl::SurfaceBaseImpl(const GPUVAddr gpu_vaddr, const SurfaceParams& params)
- : gpu_addr{gpu_vaddr}, params{params}, mipmap_sizes{params.num_levels},
- mipmap_offsets{params.num_levels}, layer_size{params.GetGuestLayerSize()},
- memory_size{params.GetGuestSizeInBytes()}, host_memory_size{params.GetHostSizeInBytes()} {
- u32 offset = 0;
- mipmap_offsets.resize(params.num_levels);
- mipmap_sizes.resize(params.num_levels);
- for (u32 i = 0; i < params.num_levels; i++) {
- mipmap_offsets[i] = offset;
- mipmap_sizes[i] = params.GetGuestMipmapSize(i);
- offset += mipmap_sizes[i];
+SurfaceBaseImpl::SurfaceBaseImpl(GPUVAddr gpu_addr, const SurfaceParams& params)
+ : params{params}, gpu_addr{gpu_addr}, layer_size{params.GetGuestLayerSize()},
+ guest_memory_size{params.GetGuestSizeInBytes()}, host_memory_size{
+ params.GetHostSizeInBytes()} {
+ mipmap_offsets.reserve(params.num_levels);
+ mipmap_sizes.reserve(params.num_levels);
+
+ std::size_t offset = 0;
+ for (u32 level = 0; level < params.num_levels; ++level) {
+ const std::size_t mipmap_size{params.GetGuestMipmapSize(level)};
+ mipmap_sizes.push_back(mipmap_size);
+ mipmap_offsets.push_back(offset);
+ offset += mipmap_size;
}
}
@@ -44,7 +46,7 @@ void SurfaceBaseImpl::SwizzleFunc(MortonSwizzleMode mode, u8* memory, const Surf
std::size_t host_offset{0};
const std::size_t guest_stride = layer_size;
const std::size_t host_stride = params.GetHostLayerSize(level);
- for (u32 layer = 0; layer < params.depth; layer++) {
+ for (u32 layer = 0; layer < params.depth; ++layer) {
MortonSwizzle(mode, params.pixel_format, width, block_height, height, block_depth, 1,
params.tile_width_spacing, buffer + host_offset, memory + guest_offset);
guest_offset += guest_stride;
@@ -60,12 +62,12 @@ void SurfaceBaseImpl::SwizzleFunc(MortonSwizzleMode mode, u8* memory, const Surf
void SurfaceBaseImpl::LoadBuffer(Tegra::MemoryManager& memory_manager,
std::vector<u8>& staging_buffer) {
MICROPROFILE_SCOPE(GPU_Load_Texture);
- auto host_ptr = memory_manager.GetPointer(gpu_addr);
+ const auto host_ptr{memory_manager.GetPointer(gpu_addr)};
if (params.is_tiled) {
ASSERT_MSG(params.block_width == 1, "Block width is defined as {} on texture target {}",
params.block_width, static_cast<u32>(params.target));
for (u32 level = 0; level < params.num_levels; ++level) {
- const u32 host_offset = params.GetHostMipmapLevelOffset(level);
+ const std::size_t host_offset{params.GetHostMipmapLevelOffset(level)};
SwizzleFunc(MortonSwizzleMode::MortonToLinear, host_ptr, params,
staging_buffer.data() + host_offset, level);
}
@@ -91,7 +93,7 @@ void SurfaceBaseImpl::LoadBuffer(Tegra::MemoryManager& memory_manager,
}
for (u32 level = 0; level < params.num_levels; ++level) {
- const u32 host_offset = params.GetHostMipmapLevelOffset(level);
+ const std::size_t host_offset{params.GetHostMipmapLevelOffset(level)};
ConvertFromGuestToHost(staging_buffer.data() + host_offset, params.pixel_format,
params.GetMipWidth(level), params.GetMipHeight(level),
params.GetMipDepth(level), true, true);
@@ -105,7 +107,7 @@ void SurfaceBaseImpl::FlushBuffer(Tegra::MemoryManager& memory_manager,
if (params.is_tiled) {
ASSERT_MSG(params.block_width == 1, "Block width is defined as {}", params.block_width);
for (u32 level = 0; level < params.num_levels; ++level) {
- const u32 host_offset = params.GetHostMipmapLevelOffset(level);
+ const std::size_t host_offset{params.GetHostMipmapLevelOffset(level)};
SwizzleFunc(MortonSwizzleMode::LinearToMorton, host_ptr, params,
staging_buffer.data() + host_offset, level);
}
diff --git a/src/video_core/texture_cache/surface_base.h b/src/video_core/texture_cache/surface_base.h
index 029cfb055..7cc122158 100644
--- a/src/video_core/texture_cache/surface_base.h
+++ b/src/video_core/texture_cache/surface_base.h
@@ -78,7 +78,7 @@ public:
void SetCacheAddr(const CacheAddr new_addr) {
cache_addr = new_addr;
- cache_addr_end = new_addr + memory_size;
+ cache_addr_end = new_addr + guest_memory_size;
}
const SurfaceParams& GetSurfaceParams() const {
@@ -86,7 +86,7 @@ public:
}
std::size_t GetSizeInBytes() const {
- return memory_size;
+ return guest_memory_size;
}
std::size_t GetHostSizeInBytes() const {
@@ -135,17 +135,19 @@ public:
}
std::optional<std::pair<u32, u32>> GetLayerMipmap(const GPUVAddr candidate_gpu_addr) const {
- if (candidate_gpu_addr < gpu_addr)
+ if (candidate_gpu_addr < gpu_addr) {
return {};
- const GPUVAddr relative_address = candidate_gpu_addr - gpu_addr;
- const u32 layer = relative_address / layer_size;
+ }
+ const auto relative_address{static_cast<GPUVAddr>(candidate_gpu_addr - gpu_addr)};
+ const auto layer{static_cast<u32>(relative_address / layer_size)};
const GPUVAddr mipmap_address = relative_address - layer_size * layer;
const auto mipmap_it =
binary_find(mipmap_offsets.begin(), mipmap_offsets.end(), mipmap_address);
- if (mipmap_it != mipmap_offsets.end()) {
- return {{layer, std::distance(mipmap_offsets.begin(), mipmap_it)}};
+ if (mipmap_it == mipmap_offsets.end()) {
+ return {};
}
- return {};
+ const auto level{static_cast<u32>(std::distance(mipmap_offsets.begin(), mipmap_it))};
+ return std::make_pair(layer, level);
}
std::vector<CopyParams> BreakDown(const SurfaceParams& in_params) const {
@@ -169,7 +171,7 @@ public:
} else {
result.reserve(mipmaps);
- for (std::size_t level = 0; level < mipmaps; level++) {
+ for (u32 level = 0; level < mipmaps; level++) {
const u32 width{std::min(params.GetMipWidth(level), in_params.GetMipWidth(level))};
const u32 height{
std::min(params.GetMipHeight(level), in_params.GetMipHeight(level))};
@@ -181,21 +183,22 @@ public:
}
protected:
- explicit SurfaceBaseImpl(const GPUVAddr gpu_vaddr, const SurfaceParams& params);
+ explicit SurfaceBaseImpl(GPUVAddr gpu_addr, const SurfaceParams& params);
~SurfaceBaseImpl() = default;
virtual void DecorateSurfaceName() = 0;
const SurfaceParams params;
- GPUVAddr gpu_addr{};
- std::vector<u32> mipmap_sizes;
- std::vector<u32> mipmap_offsets;
const std::size_t layer_size;
- const std::size_t memory_size;
+ const std::size_t guest_memory_size;
const std::size_t host_memory_size;
- CacheAddr cache_addr;
+ GPUVAddr gpu_addr{};
+ CacheAddr cache_addr{};
CacheAddr cache_addr_end{};
- VAddr cpu_addr;
+ VAddr cpu_addr{};
+
+ std::vector<std::size_t> mipmap_sizes;
+ std::vector<std::size_t> mipmap_offsets;
private:
void SwizzleFunc(MortonSwizzleMode mode, u8* memory, const SurfaceParams& params, u8* buffer,