Re: [Mesa-dev] [PATCH] anv: Add gen11 to anv_genX_call

2018-03-23 Thread Lionel Landwerlin 

Reviewed-by: Lionel Landwerlin 

On 22/03/18 20:58, Jordan Justen wrote:

Signed-off-by: Jordan Justen 
---
  src/intel/vulkan/anv_cmd_buffer.c | 3 +++
  1 file changed, 3 insertions(+)

diff --git a/src/intel/vulkan/anv_cmd_buffer.c 
b/src/intel/vulkan/anv_cmd_buffer.c
index 8f4bf3f0bb9..33687920a38 100644
--- a/src/intel/vulkan/anv_cmd_buffer.c
+++ b/src/intel/vulkan/anv_cmd_buffer.c
@@ -332,6 +332,9 @@ VkResult anv_ResetCommandBuffer(
 case 10:\
gen10_##func(__VA_ARGS__);   \
break;   \
+   case 11:\
+  gen11_##func(__VA_ARGS__);   \
+  break;   \
 default:\
assert(!"Unknown hardware generation");  \
 }



___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Re: [Mesa-dev] [PATCH] anv: Add gen11 to anv_genX_call

2018-03-23 Thread Jordan Justen 
On 2018-03-22 14:19:47, Lionel Landwerlin wrote:
> There is another macro anv_genX_call() in anv_cmd_buffer.c & anv_blorp.c

This patch was changing anv_genX_call in anv_cmd_buffer.c.

anv_blorp.c appeared to have gen11 handled.

> Also a switch in anv_device.c in anv_device_init_dispatch.

You're right. I replied to your email with another patch for
anv_device.c.

-Jordan

> If you just knock those off in the same patch, that would be great.
> 
> Thanks!
> 
> -
> Lionel
> 
> On 22/03/18 20:58, Jordan Justen wrote:
> > Signed-off-by: Jordan Justen 
> > ---
> >   src/intel/vulkan/anv_cmd_buffer.c | 3 +++
> >   1 file changed, 3 insertions(+)
> >
> > diff --git a/src/intel/vulkan/anv_cmd_buffer.c 
> > b/src/intel/vulkan/anv_cmd_buffer.c
> > index 8f4bf3f0bb9..33687920a38 100644
> > --- a/src/intel/vulkan/anv_cmd_buffer.c
> > +++ b/src/intel/vulkan/anv_cmd_buffer.c
> > @@ -332,6 +332,9 @@ VkResult anv_ResetCommandBuffer(
> >  case 10:\
> > gen10_##func(__VA_ARGS__);   \
> > break;   \
> > +   case 11:\
> > +  gen11_##func(__VA_ARGS__);   \
> > +  break;   \
> >  default:\
> > assert(!"Unknown hardware generation");  \
> >  }
> 
> 
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Re: [Mesa-dev] [PATCH] anv: Add gen11 to anv_genX_call

2018-03-22 Thread Lionel Landwerlin 

There is another macro anv_genX_call() in anv_cmd_buffer.c & anv_blorp.c
Also a switch in anv_device.c in anv_device_init_dispatch.

If you just knock those off in the same patch, that would be great.

Thanks!

-
Lionel

On 22/03/18 20:58, Jordan Justen wrote:

Signed-off-by: Jordan Justen 
---
  src/intel/vulkan/anv_cmd_buffer.c | 3 +++
  1 file changed, 3 insertions(+)

diff --git a/src/intel/vulkan/anv_cmd_buffer.c 
b/src/intel/vulkan/anv_cmd_buffer.c
index 8f4bf3f0bb9..33687920a38 100644
--- a/src/intel/vulkan/anv_cmd_buffer.c
+++ b/src/intel/vulkan/anv_cmd_buffer.c
@@ -332,6 +332,9 @@ VkResult anv_ResetCommandBuffer(
 case 10:\
gen10_##func(__VA_ARGS__);   \
break;   \
+   case 11:\
+  gen11_##func(__VA_ARGS__);   \
+  break;   \
 default:\
assert(!"Unknown hardware generation");  \
 }



___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

[Mesa-dev] [PATCH] anv: Add gen11 to anv_genX_call

2018-03-22 Thread Jordan Justen 
Signed-off-by: Jordan Justen 
---
 src/intel/vulkan/anv_cmd_buffer.c | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/src/intel/vulkan/anv_cmd_buffer.c 
b/src/intel/vulkan/anv_cmd_buffer.c
index 8f4bf3f0bb9..33687920a38 100644
--- a/src/intel/vulkan/anv_cmd_buffer.c
+++ b/src/intel/vulkan/anv_cmd_buffer.c
@@ -332,6 +332,9 @@ VkResult anv_ResetCommandBuffer(
case 10:\
   gen10_##func(__VA_ARGS__);   \
   break;   \
+   case 11:\
+  gen11_##func(__VA_ARGS__);   \
+  break;   \
default:\
   assert(!"Unknown hardware generation");  \
}
-- 
2.16.2

___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev