Re: [Mesa-dev] [PATCH 2/2] anv/image: fix offset's alignment to the surface alignment

2019-02-18 Thread Jason Ekstrand
This confuses me. When is this ever a problem? I suspect that we're doing something wing with alignments now. --Jason On February 18, 2019 09:02:39 Lionel Landwerlin wrote: On 15/02/2019 14:43, Samuel Iglesias Gonsálvez wrote: Signed-off-by: Samuel Iglesias Gonsálvez Hey Samuel,

Re: [Mesa-dev] [PATCH 2/2] anv/image: fix offset's alignment to the surface alignment

2019-02-18 Thread Lionel Landwerlin
On 15/02/2019 14:43, Samuel Iglesias Gonsálvez wrote: Signed-off-by: Samuel Iglesias Gonsálvez Hey Samuel, Thanks for this change. Would you mind changing the align_u32 in the if() branch too? It won't fix anything but that's just to be consistent. With that : Reviewed-by: Lionel

[Mesa-dev] [PATCH 2/2] anv/image: fix offset's alignment to the surface alignment

2019-02-15 Thread Samuel Iglesias Gonsálvez
Signed-off-by: Samuel Iglesias Gonsálvez --- src/intel/vulkan/anv_image.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/intel/vulkan/anv_image.c b/src/intel/vulkan/anv_image.c index 3999c7399d0..f4a65044a3b 100644 --- a/src/intel/vulkan/anv_image.c +++

[Mesa-dev] [PATCH 2/2] anv/image: fix offset's alignment to the surface alignment

2019-02-15 Thread Samuel Iglesias Gonsálvez
Signed-off-by: Samuel Iglesias Gonsálvez --- src/intel/vulkan/anv_image.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/intel/vulkan/anv_image.c b/src/intel/vulkan/anv_image.c index 3999c7399d0..f4a65044a3b 100644 --- a/src/intel/vulkan/anv_image.c +++