<div dir="ltr"><div class="gmail_quote"><div dir="ltr">On Thu, Oct 18, 2018 at 3:24 PM Jason Ekstrand <<a href="mailto:jason@jlekstrand.net">jason@jlekstrand.net</a>> wrote:<br></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_quote"><div dir="ltr">On Thu, Oct 18, 2018 at 3:17 PM Jan Vesely <<a href="mailto:jan.vesely@rutgers.edu" target="_blank">jan.vesely@rutgers.edu</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi,<br>
<br>
I think this patch breaks the build:<br>
<a href="https://travis-ci.org/jvesely/mesa/jobs/443356781" rel="noreferrer" target="_blank">https://travis-ci.org/jvesely/mesa/jobs/443356781</a><br></blockquote><div><br></div><div>Interesting.  It appears that it breaks when you have one of the WSI extensions disabled.  I'll take a look. <br></div></div></div></blockquote><div><br></div><div>Yeah, in retrospect, it's pretty bogus. I've reverted the patch.  Thanks for reporting!  It was nice while it lasted because it pointed out an entrypoint we forgot to implement but it looks like it was a bad idea in the long term.  Maybe one day, I can come up with something more clever but not today.</div><div><br></div><div>--jason</div><div><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_quote"><div></div><div>--Jason</div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Jan<br>
<br>
On Tue, 2018-10-16 at 08:18 -0500, Jason Ekstrand wrote:<br>
> FYI, patch 1 is required for this patch to build.  It also means this patch <br>
> found a nice little bug.  I'll respond to patch 1 in more detail after the <br>
> SI call tomorrow.<br>
> <br>
> --Jason<br>
> <br>
> <br>
> On October 16, 2018 06:49:35 Lionel Landwerlin <br>
> <<a href="mailto:lionel.g.landwerlin@intel.com" target="_blank">lionel.g.landwerlin@intel.com</a>> wrote:<br>
> <br>
> > Reviewed-by: Lionel Landwerlin <<a href="mailto:lionel.g.landwerlin@intel.com" target="_blank">lionel.g.landwerlin@intel.com</a>><br>
> > <br>
> > On 15/10/2018 04:47, Jason Ekstrand wrote:<br>
> > > We don't need weak references to instance entrypoints because we never<br>
> > > have more than one of each so we don't need the NULL fall-back.  This<br>
> > > also helps us avoid forgetting things because we now get link errors for<br>
> > > missing instance entrypoints.<br>
> > > ---<br>
> > >   src/intel/vulkan/anv_entrypoints_gen.py | 13 -------------<br>
> > >   1 file changed, 13 deletions(-)<br>
> > > <br>
> > > diff --git a/src/intel/vulkan/anv_entrypoints_gen.py <br>
> > > b/src/intel/vulkan/anv_entrypoints_gen.py<br>
> > > index beb658b8660..25a532fd706 100644<br>
> > > --- a/src/intel/vulkan/anv_entrypoints_gen.py<br>
> > > +++ b/src/intel/vulkan/anv_entrypoints_gen.py<br>
> > > @@ -227,19 +227,6 @@ ${strmap(device_strmap, 'device')}<br>
> > >    * either pick the correct entry point.<br>
> > >    */<br>
> > > <br>
> > > -% for e in instance_entrypoints:<br>
> > > -  % if e.alias:<br>
> > > -    <% continue %><br>
> > > -  % endif<br>
> > > -  % if e.guard is not None:<br>
> > > -#ifdef ${e.guard}<br>
> > > -  % endif<br>
> > > -  ${e.return_type} ${e.prefixed_name('anv')}(${e.decl_params()}) <br>
> > > __attribute__ ((weak));<br>
> > > -  % if e.guard is not None:<br>
> > > -#endif // ${e.guard}<br>
> > > -  % endif<br>
> > > -% endfor<br>
> > > -<br>
> > >   const struct anv_instance_dispatch_table anv_instance_dispatch_table = {<br>
> > >   % for e in instance_entrypoints:<br>
> > >     % if e.guard is not None:<br>
> > <br>
> > <br>
> > _______________________________________________<br>
> > mesa-dev mailing list<br>
> > <a href="mailto:mesa-dev@lists.freedesktop.org" target="_blank">mesa-dev@lists.freedesktop.org</a><br>
> > <a href="https://lists.freedesktop.org/mailman/listinfo/mesa-dev" rel="noreferrer" target="_blank">https://lists.freedesktop.org/mailman/listinfo/mesa-dev</a><br>
> <br>
> <br>
> <br>
> _______________________________________________<br>
> mesa-dev mailing list<br>
> <a href="mailto:mesa-dev@lists.freedesktop.org" target="_blank">mesa-dev@lists.freedesktop.org</a><br>
> <a href="https://lists.freedesktop.org/mailman/listinfo/mesa-dev" rel="noreferrer" target="_blank">https://lists.freedesktop.org/mailman/listinfo/mesa-dev</a><br>
<br>
</blockquote></div></div>
</blockquote></div></div>