Re: [Mesa-dev] [PATCH 1/2] anv: Fix vkCmdFillBuffer()

2017-10-02 Thread Lionel Landwerlin
Thanks, I modified the CTS slightly to reproduce the problem (will try to send it upstream). Reviewed-by: Lionel Landwerlin On 02/10/17 20:41, Józef Kucia wrote: On Mon, Oct 2, 2017 at 9:19 PM, Lionel Landwerlin wrote: I guess it makes things clearer. Does this fix something? Yes, it fixes

Re: [Mesa-dev] [PATCH 1/2] anv: Fix vkCmdFillBuffer()

2017-10-02 Thread Józef Kucia
On Mon, Oct 2, 2017 at 9:19 PM, Lionel Landwerlin wrote: > I guess it makes things clearer. > Does this fix something? Yes, it fixes buffer fills. After the change buffers are filled with uint32_t values instead of uint8_t or uint16_t values. Before the change, when the fill value is 0xabcdef bu

Re: [Mesa-dev] [PATCH 1/2] anv: Fix vkCmdFillBuffer()

2017-10-02 Thread Lionel Landwerlin
I guess it makes things clearer. Does this fix something? On 02/10/17 18:58, Józef Kucia wrote: The vkCmdFillBuffer() command fills a buffer with an uint32_t value. --- src/intel/vulkan/anv_blorp.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/intel/vulkan/anv_bl

[Mesa-dev] [PATCH 1/2] anv: Fix vkCmdFillBuffer()

2017-10-02 Thread Józef Kucia
The vkCmdFillBuffer() command fills a buffer with an uint32_t value. --- src/intel/vulkan/anv_blorp.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/intel/vulkan/anv_blorp.c b/src/intel/vulkan/anv_blorp.c index 8d0c275248..a8ebdb384c 100644 --- a/src/intel/vulkan/anv_b