<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Nov 7, 2013 at 1:22 PM, Dave Airlie <span dir="ltr"><<a href="mailto:airlied@gmail.com" target="_blank">airlied@gmail.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On Sat, Oct 12, 2013 at 8:10 AM, Ian Romanick <<a href="mailto:idr@freedesktop.org">idr@freedesktop.org</a>> wrote:<br>


> From: Ian Romanick <<a href="mailto:ian.d.romanick@intel.com">ian.d.romanick@intel.com</a>><br>
><br>
> The enumerated values are currently allocated from Intel's range.<br>
<br>
</div>Some highlevel comments below,<br>
<div class="im"><br>
<br>
> +    GLX renderer attribute         number     description<br>
> +                                  of values<br>
> +    ----------------------        ---------   -----------<br>
> +    GLX_RENDERER_VENDOR_ID_MESA   1           PCI ID of the device vendor<br>
> +    GLX_RENDERER_DEVICE_ID_MESA   1           PCI ID of the device<br>
<br>
</div>Okay PCI IDs might seem useful but we have a lot of situations where<br>
they aren't available,<br></blockquote><div><br></div><div>I was thinking the same, but then noticed that it's an int, and a pci device or vendor id is 16 bit, so you could implement this by returning the pciid if you have it, or something > 0xFFFF if you don't have it (and sort of make sure that those don't overlap).</div>

<div><br></div><div>Stéphane</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
a) ARM devices generally don't have them<br>
b) nvidia generally espouse using PCI IDs in favour of a chip family<br>
ID they bake into the GPUs,<br>
nouveau doesn't even know the actual PCI IDs in userspace, we could<br>
find them but it does take a bit more work, like new kernel interface.<br>
<div class="im"><br>
> +<br>
> +    GLX renderer attribute        description<br>
> +    ----------------------        -----------<br>
> +    GLX_RENDERER_VENDOR_ID_MESA   Name of the renderer provider.  This may<br>
> +                                  differ from the vendor name of the<br>
> +                                  underlying hardware.<br>
> +    GLX_RENDERER_DEVICE_ID_MESA   Name of the renderer.  This may differ from<br>
> +                                  the name of the underlying hardware (e.g.,<br>
> +                                  for a software renderer).<br>
> +<br>
> +    If <attribute> is not a recognized value, NULL is returned, but no GLX<br>
> +    error is generated.<br>
> +<br>
> +    The string returned for GLX_RENDERER_VENDOR_ID_MESA will have the same<br>
> +    format as the string that would be returned by glGetString of GL_VENDOR.<br>
> +    It may, however, have a different value.<br>
<br>
</div>Any reason for the value to be different here? Like a use case?<br>
<div class="im">> +    14) Why not make the queries from issue #14 GL functions (instead of GLX)?<br>
> +<br>
<br>
</div>from issue 14? this is issue 14<br>
<span class="HOEnZb"><font color="#888888"><br>
Dave.<br>
</font></span><div class="HOEnZb"><div class="h5">_______________________________________________<br>
mesa-dev mailing list<br>
<a href="mailto:mesa-dev@lists.freedesktop.org">mesa-dev@lists.freedesktop.org</a><br>
<a href="http://lists.freedesktop.org/mailman/listinfo/mesa-dev" target="_blank">http://lists.freedesktop.org/mailman/listinfo/mesa-dev</a><br>
</div></div></blockquote></div><br></div></div>