<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Jan 28, 2014 at 2:13 PM, Ian Romanick <span dir="ltr"><<a href="mailto:idr@freedesktop.org" target="_blank">idr@freedesktop.org</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 01/28/2014 02:51 PM, Mark Mueller wrote:<br>
> This patch could cause the i965 driver to not load if Mesa was built on<br>
> a system without libudev devel present. For example on Fedora one should<br>
> install systemd-devel before configuring and building Mesa drivers<br>
> subsequent to this change.<br>
<br>
</div>I don't think so.  loader_get_driver_for_fd should return NULL in that<br>
case, and the loader will fallback to the old behavior.<br></blockquote><div><br></div><div>Except it doesn't; it goes and grabs the kernel module name. For example on an i965 it will try to open the i915 driver, because that's the name of the kernel module.</div>

<div><br></div><div>The erroneous assumption in this patch is that dri2 implies libudev is present (this isn't checked by ./configure). Although that is true with dri3, it's not the case with dri2. So in that respect the patch is incorrect.</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">
<div class="im"><br>
> On Sun, Jan 26, 2014 at 4:02 PM, Keith Packard <<a href="mailto:keithp@keithp.com">keithp@keithp.com</a><br>
</div>> <mailto:<a href="mailto:keithp@keithp.com">keithp@keithp.com</a>>> wrote:<br>
<div class="im">><br>
>     Eric Anholt <<a href="mailto:eric@anholt.net">eric@anholt.net</a> <mailto:<a href="mailto:eric@anholt.net">eric@anholt.net</a>>> writes:<br>
><br>
>     > This allows Mesa to choose to rename driver .sos (or split drivers),<br>
>     > without needing a flag day with the corresponding 2D driver.<br>
><br>
>     Reviewed-by: Keith Packard <<a href="mailto:keithp@keithp.com">keithp@keithp.com</a><br>
</div>>     <mailto:<a href="mailto:keithp@keithp.com">keithp@keithp.com</a>>><br>
><br>
>     --<br>
>     <a href="mailto:keith.packard@intel.com">keith.packard@intel.com</a> <mailto:<a href="mailto:keith.packard@intel.com">keith.packard@intel.com</a>><br>
><br>
>     _______________________________________________<br>
>     mesa-dev mailing list<br>
>     <a href="mailto:mesa-dev@lists.freedesktop.org">mesa-dev@lists.freedesktop.org</a> <mailto:<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 class="HOEnZb"><div class="h5">><br>
><br>
><br>
><br>
> _______________________________________________<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>
><br>
<br>
_______________________________________________<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>