<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Jun 15, 2016 at 12:18 PM, Jason Ekstrand <span dir="ltr"><<a href="mailto:jason@jlekstrand.net" target="_blank">jason@jlekstrand.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote"><div><div class="h5">On Wed, Jun 15, 2016 at 12:16 PM, Jason Ekstrand <span dir="ltr"><<a href="mailto:jason@jlekstrand.net" target="_blank">jason@jlekstrand.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div><br><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Jun 15, 2016 at 10:34 AM, Chad Versace <span dir="ltr"><<a href="mailto:chad.versace@intel.com" target="_blank">chad.versace@intel.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span>On Tue 14 Jun 2016, Jason Ekstrand wrote:<br>
> On Jun 14, 2016 4:23 PM, "Chad Versace" <<a href="mailto:chad.versace@intel.com" target="_blank">chad.versace@intel.com</a>> wrote:<br>
> ><br>
> > anv_pipeline_binding::index is a uint8_t, but some code assigned to it<br>
> > UINT16_MAX.<br>
> > ---<br>
> >  src/intel/vulkan/anv_pipeline.c | 2 +-<br>
> >  1 file changed, 1 insertion(+), 1 deletion(-)<br>
> ><br>
> > diff --git a/src/intel/vulkan/anv_pipeline.c<br>
> b/src/intel/vulkan/anv_pipeline.c<br>
> > index 60b7c6b..b41e11e 100644<br>
> > --- a/src/intel/vulkan/anv_pipeline.c<br>
> > +++ b/src/intel/vulkan/anv_pipeline.c<br>
> > @@ -664,7 +664,7 @@ anv_pipeline_compile_fs(struct anv_pipeline *pipeline,<br>
> >           rt_bindings[0] = (struct anv_pipeline_binding) {<br>
> >              .set = ANV_DESCRIPTOR_SET_COLOR_ATTACHMENTS,<br>
> >              .binding = 0,<br>
> > -            .index = UINT16_MAX,<br>
> > +            .index = UINT8_MAX,<br>
><br>
> I believe we have a descriptive #define specifically for render targets.<br>
> Probably better to use that.<br>
<br>
</span>What #define did you have in mind? The only relevant one I found was<br>
<br>
  #define MAX_RTS 8<br>
</blockquote></div><br></div></div></div><div class="gmail_extra">ANV_DESCRIPTOR_SET_COLOR_ATTACHMENTS<br><br></div><div class="gmail_extra">Sorry I didn't give it to you last night, I was looking at patches from my phone.<span><font color="#888888"><br></font></span></div><span><font color="#888888"><div class="gmail_extra">--Jason<br></div></font></span></div></blockquote><div><br></div></div></div><div>Wait... Never mind.  We need to add a new one probably.  Or UINT8_MAX works too.  Really, it just needs to be a value that fails the binding->index < subpass->color_count check in anv_cmd_buffer.c:815.  Sorry for the noise.<span class="HOEnZb"><font color="#888888"><br></font></span></div></div></div></div></blockquote><div><br></div><div>In case you couldn't tell.  I'm fine with you leaving it as-is<br><br></div><div>Reviewd-by: Jason Ekstrand <<a href="mailto:jason@jlekstrand.net">jason@jlekstrand.net</a>><br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div><span class="HOEnZb"><font color="#888888"></font></span></div><span class="HOEnZb"><font color="#888888"><div>--Jason <br></div></font></span></div><br></div></div>
</blockquote></div><br></div></div>