<div dir="ltr"><div>drmVersion::name = amdgpu, radeon, intel, etc.<br></div><div>drmVersion::desc = vega10, vega12, vega20, ...</div><div><br></div><div>The common Mesa code will use name and desc to select the driver.<br></div><div><br></div><div>The AMD-specific Mesa code will use desc to identify the chip.</div><div><br></div><div>Mesa won't receive any PCI IDs for future chips.<br></div><div><br></div><div>Marek<br></div><div><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Sep 17, 2019 at 10:33 AM Michel Dänzer <<a href="mailto:michel@daenzer.net">michel@daenzer.net</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On 2019-09-17 1:20 p.m., Christian König wrote:<br>
> Am 17.09.19 um 11:27 schrieb Michel Dänzer:<br>
>> On 2019-09-17 11:23 a.m., Michel Dänzer wrote:<br>
>>> On 2019-09-17 10:23 a.m., Koenig, Christian wrote:<br>
>>>> Am 17.09.19 um 10:17 schrieb Daniel Vetter:<br>
>>>>> On Tue, Sep 17, 2019 at 10:12 AM Christian König<br>
>>>>> <<a href="mailto:ckoenig.leichtzumerken@gmail.com" target="_blank">ckoenig.leichtzumerken@gmail.com</a>> wrote:<br>
>>>>>> Am 17.09.19 um 07:47 schrieb Jani Nikula:<br>
>>>>>>> On Mon, 16 Sep 2019, Marek Olšák <<a href="mailto:maraeo@gmail.com" target="_blank">maraeo@gmail.com</a>> wrote:<br>
>>>>>>>> The purpose is to get rid of all PCI ID tables for all drivers in<br>
>>>>>>>> userspace. (or at least stop updating them)<br>
>>>>>>>><br>
>>>>>>>> Mesa common code and modesetting will use this.<br>
>>>>>>> I'd think this would warrant a high level description of what you<br>
>>>>>>> want<br>
>>>>>>> to achieve in the commit message.<br>
>>>>>> And maybe explicitly call it uapi_name or even uapi_driver_name.<br>
>>>>> If it's uapi_name, then why do we need a new one for every generation?<br>
>>>>> Userspace drivers tend to span a lot more than just 1 generation. And<br>
>>>>> if you want to have per-generation data from the kernel to userspace,<br>
>>>>> then imo that's much better suited in some amdgpu ioctl, instead of<br>
>>>>> trying to encode that into the driver name.<br>
>>>> Well we already have an IOCTL for that, but I thought the intention<br>
>>>> here<br>
>>>> was to get rid of the PCI-ID tables in userspace to figure out which<br>
>>>> driver to load.<br>
>>> That's just unrealistic in general, I'm afraid. See e.g. the ongoing<br>
>>> transition from i965 to iris for recent Intel hardware. How is the<br>
>>> kernel supposed to know which driver is to be used?<br>
> <br>
> Well how is userspace currently handling that? The kernel should NOT say<br>
> which driver to use in userspace, but rather which one is used in the<br>
> kernel.<br>
<br>
Would that really help though? E.g. the radeon kernel driver supports<br>
radeon/r200/r300/r600/radeonsi DRI drivers, the i915 one i915/i965/iris<br>
(and the amdgpu one radeonsi/amdgpu).<br>
<br>
The HW generation identifier proposed in these patches might be useful,<br>
but I suspect there'll always be cases where userspace needs to know<br>
more precisely.<br>
<br>
<br>
> Mapping that information to an userspace driver still needs to be done<br>
> somewhere else, but the difference is that you don't need to add all<br>
> PCI-IDs twice.<br>
<br>
It should only really be necessary in Mesa.<br>
<br>
<br>
On 2019-09-17 1:32 p.m., Daniel Vetter wrote:<br>
> How are other compositors solving this? I don't expect they have a<br>
> pciid table like modesetting copied to all of them ...<br>
<br>
They don't need any of this. The Xorg modesetting driver only did for<br>
determining the client driver name to advertise via the DRI2 extension.<br>
<br>
<br>
-- <br>
Earthling Michel Dänzer               |               <a href="https://redhat.com" rel="noreferrer" target="_blank">https://redhat.com</a><br>
Libre software enthusiast             |             Mesa and X developer<br>
_______________________________________________<br>
dri-devel mailing list<br>
<a href="mailto:dri-devel@lists.freedesktop.org" target="_blank">dri-devel@lists.freedesktop.org</a><br>
<a href="https://lists.freedesktop.org/mailman/listinfo/dri-devel" rel="noreferrer" target="_blank">https://lists.freedesktop.org/mailman/listinfo/dri-devel</a></blockquote></div>