<div dir="ltr">Op donderdag 31 maart 2016 13:02:36 UTC+2 schreef Chih-Wei Huang:<blockquote class="gmail_quote" style="margin: 0;margin-left: 0.8ex;border-left: 1px #ccc solid;padding-left: 1ex;">2016-03-31 0:09 GMT+08:00 Rob Herring <<a href="javascript:" target="_blank" gdf-obfuscated-mailto="zmjnEBtZEQAJ" rel="nofollow">ro...@kernel.org</a>>:
<br>> On Tue, Mar 29, 2016 at 2:49 PM, Jaap Jan Meijer <<a href="javascript:" target="_blank" gdf-obfuscated-mailto="zmjnEBtZEQAJ" rel="nofollow">jjmei...@gmail.com</a>> wrote:
<br>>>
<br>>> First of all, thanks for the hard work!
<br>>>
<br>>> Only one problem I noticed so far: when enabling virgl in drm_gralloc but
<br>>> without support in MESA, MESA / drm_gralloc (?) will just crash during
<br>>> initialization. I don't think it's that important but when I'm only using
<br>>> i915 it shouldn't really need to crash because of this?
<br>>
<br>> The only thing I can see is if load_pipe_screen is NULL, but
<br>> gallium_dri.so is present. Not completely sure if that is possible,
<br>> but I've pushed a fix to check load_pipe_screen to my android-m
<br>> branch.
<br></blockquote><div><br><br>Thanks for the patch, I tested it on both x86 and x64 kernels and it doesn't crash anymore, previously it did on both.<br> <br><br></div><blockquote class="gmail_quote" style="margin: 0;margin-left: 0.8ex;border-left: 1px #ccc solid;padding-left: 1ex;">Thanks for the fix. I'll apply it.
<br>But Jaap uses i915 which is a classical driver.
<br>I doubt if it's related to the gallium_dri.so.
<br>
<br>Jaap, did you use a 64-bit kernel to test?
<br></blockquote><div><br>You are right but because of the driver selection strategy chosen in gralloc_drm.c it will always first try the pipe driver whenever it is enabled.<br></div></div>