[Mesa-dev] [PATCH 1/2] radv: Remove some intel comments from the resolve code.

Kenneth Graunke kenneth at whitecape.org
Fri Aug 25 06:30:16 UTC 2017


On Thursday, August 24, 2017 4:17:25 PM PDT Bas Nieuwenhuizen wrote:
> These are clearly not applicable to radv.
> ---
>  src/amd/vulkan/radv_meta_resolve.c    | 7 -------
>  src/amd/vulkan/radv_meta_resolve_cs.c | 7 -------
>  src/amd/vulkan/radv_meta_resolve_fs.c | 7 -------
>  3 files changed, 21 deletions(-)
> 
> diff --git a/src/amd/vulkan/radv_meta_resolve.c b/src/amd/vulkan/radv_meta_resolve.c
> index 6023e0f8999..dd811c25142 100644
> --- a/src/amd/vulkan/radv_meta_resolve.c
> +++ b/src/amd/vulkan/radv_meta_resolve.c
> @@ -612,13 +612,6 @@ radv_cmd_buffer_resolve_subpass(struct radv_cmd_buffer *cmd_buffer)
>  
>  		radv_cmd_buffer_set_subpass(cmd_buffer, &resolve_subpass, false);
>  
> -		/* Subpass resolves must respect the render area. We can ignore the
> -		 * render area here because vkCmdBeginRenderPass set the render area
> -		 * with 3DSTATE_DRAWING_RECTANGLE.
> -		 *
> -		 * XXX(chadv): Does the hardware really respect
> -		 * 3DSTATE_DRAWING_RECTANGLE when draing a 3DPRIM_RECTLIST?
> -		 */
>  		emit_resolve(cmd_buffer,
>  			     &(VkOffset2D) { 0, 0 },
>  			     &(VkExtent2D) { fb->width, fb->height });
> diff --git a/src/amd/vulkan/radv_meta_resolve_cs.c b/src/amd/vulkan/radv_meta_resolve_cs.c
> index d20d04231ed..da6ca76b6d4 100644
> --- a/src/amd/vulkan/radv_meta_resolve_cs.c
> +++ b/src/amd/vulkan/radv_meta_resolve_cs.c
> @@ -529,13 +529,6 @@ radv_cmd_buffer_resolve_subpass_cs(struct radv_cmd_buffer *cmd_buffer)
>  
>  		radv_cmd_buffer_set_subpass(cmd_buffer, &resolve_subpass, false);
>  
> -		/* Subpass resolves must respect the render area. We can ignore the
> -		 * render area here because vkCmdBeginRenderPass set the render area
> -		 * with 3DSTATE_DRAWING_RECTANGLE.
> -		 *
> -		 * XXX(chadv): Does the hardware really respect
> -		 * 3DSTATE_DRAWING_RECTANGLE when draing a 3DPRIM_RECTLIST?
> -		 */
>  		emit_resolve(cmd_buffer,
>  			     src_iview,
>  			     dst_iview,
> diff --git a/src/amd/vulkan/radv_meta_resolve_fs.c b/src/amd/vulkan/radv_meta_resolve_fs.c
> index 2f745f0ea09..373dd9665a7 100644
> --- a/src/amd/vulkan/radv_meta_resolve_fs.c
> +++ b/src/amd/vulkan/radv_meta_resolve_fs.c
> @@ -633,13 +633,6 @@ radv_cmd_buffer_resolve_subpass_fs(struct radv_cmd_buffer *cmd_buffer)
>  
>  		radv_cmd_buffer_set_subpass(cmd_buffer, &resolve_subpass, false);
>  
> -		/* Subpass resolves must respect the render area. We can ignore the
> -		 * render area here because vkCmdBeginRenderPass set the render area
> -		 * with 3DSTATE_DRAWING_RECTANGLE.
> -		 *
> -		 * XXX(chadv): Does the hardware really respect
> -		 * 3DSTATE_DRAWING_RECTANGLE when draing a 3DPRIM_RECTLIST?
> -		 */
>  		emit_resolve(cmd_buffer,
>  			     src_iview,
>  			     dest_iview,
> 

Yeah!  Down with the 3DSTATE_DRAWING_RECTANGLE comments, and bring on the
S_028B50_DONUT_SPLIT and V_028B6C_DISTRIBUTION_MODE_DONUTS comments! :)

Reviewed-by: Kenneth Graunke <kenneth at whitecape.org>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: This is a digitally signed message part.
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20170824/60822333/attachment.sig>


More information about the mesa-dev mailing list