[Mesa-dev] [PATCH v2] radv/gfx10: fix intensity formats by setting ALPHA_IS_ON_MSB
Bas Nieuwenhuizen
bas at basnieuwenhuizen.nl
Thu Jul 25 09:50:34 UTC 2019
On Wed, Jul 24, 2019 at 4:47 PM Samuel Pitoiset
<samuel.pitoiset at gmail.com> wrote:
>
> This fixes
> dEQP-VK.rasterization.primitive_size.points.point_size_*
>
> This also fixes some black squares with the Sascha SSAO demo.
>
> v2: - do not set for multiple channels
> - call vi_alpha_is_on_msb() for pre-GFX10
> - remove unused 'swap'
>
> Signed-off-by: Samuel Pitoiset <samuel.pitoiset at gmail.com>
> ---
> src/amd/vulkan/radv_image.c | 17 +++++++++++------
> 1 file changed, 11 insertions(+), 6 deletions(-)
>
> diff --git a/src/amd/vulkan/radv_image.c b/src/amd/vulkan/radv_image.c
> index 09413333cbb..d46946269e6 100644
> --- a/src/amd/vulkan/radv_image.c
> +++ b/src/amd/vulkan/radv_image.c
> @@ -617,6 +617,15 @@ static unsigned gfx9_border_color_swizzle(const enum vk_swizzle swizzle[4])
> return bc_swizzle;
> }
>
> +static bool vi_alpha_is_on_msb(struct radv_device *device, VkFormat format)
> +{
> + const struct vk_format_description *desc = vk_format_description(format);
> +
> + if (device->physical_device->rad_info.chip_class >= GFX10 && desc->nr_channels == 1)
> + return desc->swizzle[3] == VK_SWIZZLE_X;
In vulkan we never have an alpha-only format, so this will always be false.
r-b anyway.
> +
> + return radv_translate_colorswap(format, false) <= 1;
> +}
> /**
> * Build the sampler view descriptor for a texture (GFX10).
> */
> @@ -691,11 +700,9 @@ gfx10_make_texture_descriptor(struct radv_device *device,
> state[7] = 0;
>
> if (radv_dcc_enabled(image, first_level)) {
> - unsigned swap = radv_translate_colorswap(vk_format, FALSE);
> -
> state[6] |= S_00A018_MAX_UNCOMPRESSED_BLOCK_SIZE(V_028C78_MAX_BLOCK_SIZE_256B) |
> S_00A018_MAX_COMPRESSED_BLOCK_SIZE(V_028C78_MAX_BLOCK_SIZE_128B) |
> - S_00A018_ALPHA_IS_ON_MSB(swap <= 1);
> + S_00A018_ALPHA_IS_ON_MSB(vi_alpha_is_on_msb(device, vk_format));
> }
>
> /* Initialize the sampler view for FMASK. */
> @@ -849,9 +856,7 @@ si_make_texture_descriptor(struct radv_device *device,
> state[5] |= S_008F24_LAST_ARRAY(last_layer);
> }
> if (image->dcc_offset) {
> - unsigned swap = radv_translate_colorswap(vk_format, FALSE);
> -
> - state[6] = S_008F28_ALPHA_IS_ON_MSB(swap <= 1);
> + state[6] = S_008F28_ALPHA_IS_ON_MSB(vi_alpha_is_on_msb(device, vk_format));
> } else {
> /* The last dword is unused by hw. The shader uses it to clear
> * bits in the first dword of sampler state.
> --
> 2.22.0
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
More information about the mesa-dev
mailing list