<div dir="auto">Sorry, I missed this part of change and thought there's only<div dir="auto">kmsro_dri.so. Then there's no concern on my side.</div><div dir="auto"><br></div><div dir="auto">Thanks,</div><div dir="auto">Qiang</div></div><br><div class="gmail_quote"><div dir="ltr">Rob Herring <<a href="mailto:robh@kernel.org">robh@kernel.org</a>> 于 2019年1月27日周日 上午3:43写道：<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Fri, Jan 25, 2019 at 9:00 PM Qiang Yu <<a href="mailto:yuq825@gmail.com" target="_blank" rel="noreferrer">yuq825@gmail.com</a>> wrote:<br>
><br>
> Thanks Rob, I'm OK with this kmsro approach.<br>
><br>
> But I have to point out that this will break XServer AIGLX:<br>
> 1. modesetting DDX will report the display drm driver name like meson<br>
>   as DRI2 driver name<br>
> 2. libglx.so used by xserver will look after meson_dri.so for dlopen<br>
> 3. then dlsym __driDriverGetExtensions_meson for init<br>
<br>
2 and 3 are now the only things you have to add to add a platform. Now<br>
it is a 2 line patch to add a platform. Specifically what you have to<br>
add are shown in the lima branch I sent you.<br>
<br>
Rob<br>
</blockquote></div>