<div dir="ltr">Hi, Chih-Wei, Emil,<div> These series of patches was originally developed on the 11.0 branch, and later ported to 11.2. I don't think softpipe was working when I worked on this, Actually the first few patches is from when I tried to make softpipe work. There are some memory issue in mesa that cause softpipe/llvmpipe to crash when booting. I assumed it was due to jemalloc and ptmalloc difference. </div><div> Regarding the TASK and Change-Id sections in the comment, sorry about that. I should have removed them when I send out the patches, you can just remove them. </div></div><div class="gmail_extra"> <div class="gmail_quote">2016-04-28 23:52 GMT+08:00 Chih-Wei Huang <<a href="mailto:cwhuang@android-x86.org" target="_blank">cwhuang@android-x86.org</a>>: <blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">2016-04-28 22:22 GMT+08:00 Emil Velikov <<a href="mailto:emil.l.velikov@gmail.com">emil.l.velikov@gmail.com</a>>: > Hi Chih-Wei, > > Thanks for getting these out to the community. > > On 28 April 2016 at 08:34, Chih-Wei Huang <<a href="mailto:cwhuang@android-x86.org">cwhuang@android-x86.org</a>> wrote: >> This is a series of patches developed by Jide Technolody to enable >> the llvmpipe for software rendering of Android. >> It makes a device without a Mesa supported GPU could run most modern >> Android apps. >> > Afaict one should only need the extra Android.mk files to get llvmpipe > considering that softpipe already works. > Have you/the Jide folks tried the latter already ? Does it work > without these patches ? Hmm, interesting point. Did you mean just adding Android.mk for llvmpipe is enough? >> These patches are mainly developed and tested on the 11.0 and 11.2 >> branches. They might not work with the Mesa master branch. >> > Humble request - please always aim for master. Doing this will get you > the latest stable branch for free. > If you're targeting some old stable branch then you'll will have to > duplicate the effort to land things in master. And new functionality > goes _only_ in master I clearly understand this point. Actually I've spent several days to try to make it work on the master branch. That's why it was delayed -- I supposed to send them in the last week. However, the master branch is always broken for android. There are a lot of build break I need to fix and workaround or I can't test it. After fixed all the errors and built it OK, however, it didn't work as expected. The system boots to Home but all display is garbled. I'm not sure if I made some mistakes on fixing the building errors or there are some changes that really broke these patches. (the latest commit I've tried in the master is 32cb7d61) I finally decide to give it up and send them as the current status. (otherwise it will take too much of my time and delay my other pending tasks) Unfortunately the situation is most mesa developers don't care android so they usually break android build or functions. Unless the situation is changed it's very hard for us to follow the master branch closely. >> The patches depend on some patches developed by Varad Gautam which >> have not been merged in Mesa master yet, say >> >> fc40946 egl: fixup: define droid_image_loader_extension >> d15901d egl: android: populate dri2_surf->window early >> cff1928 egl: android: use __DRI_IMAGE_LOADER to get color buffers >> b556be4 egl: android: experimental dma-buf fd support >> >> The dependency may be removed but we haven't tested that yet. >> > Afaict none of Varad's work should be required here. It adds an > alternative (better) method of the already existing functionality. I also guess that but it need more time to verify that. > Related: iirc things have gone wrong during the rebase of Varad's work > in Android-x86. Rob H recently sent some patches (based of Android-x86 > ?) which has some strange/extra code in them. Yes I notice that but again it need time to figure what patches are really needed. However due to the master branch status is horrible for android so I gave up. If possible, I'll ask Mauro to follow the master branch and work with others to fix android stuff for future android release (i.e., N). For marshmallow-x86 we will stay in mesa 11.2 and I'll move my time to other pending tasks for a stable release. >> WuZhen (7): >> st/dri: fix double free of dri_drawable >> tgsi: fix stack allocated struct may not be initialized >> gallium/swrast: fix dri_sw_dt->data free func not matching alloc func >> android: print debug info to logcat >> android: enable dlopen >> android: enable x86 asm and sse4 for x86 and x86_64 >> android: support swrast > > A couple of high level suggestions: > - Please split patches appropriately (more). Some patches are great > while others should become 3-4 separate ones. Actually I think the first 6 patches are already good. The 7th patch is bigger and could probably be split. Could you suggest how to do it? > - Drop the NO_REF_TASK, tested and Change-Id tags. Sadly these don't > bring much, plus non-Jide people are unlikely to have access to the > Gerrit and other instances where these are used. These are Jide's internal comments I just kept it as. I've no opinion to keep or remove them. Let WuZhen decide. > I'll try to look into the individual patches shortly. Thanks! -- Chih-Wei Android-x86 project <a href="http://www.android-x86.org" rel="noreferrer" target="_blank">http://www.android-x86.org</a> -- You received this message because you are subscribed to the Google Groups "Android-x86 development" group. To unsubscribe from this group and stop receiving emails from it, send an email to <a href="mailto:android-x86-devel%2Bunsubscribe@googlegroups.com">android-x86-devel+unsubscribe@googlegroups.com</a>. To post to this group, send email to <a href="mailto:android-x86-devel@googlegroups.com">android-x86-devel@googlegroups.com</a>. To view this discussion on the web visit <a href="https://groups.google.com/d/msgid/android-x86-devel/CAKc24n0zF-Po%3DCXwLt7wHr5ogbV-LGOsDZWqkk%3D3b_pGfc2H-A%40mail.gmail.com" rel="noreferrer" target="_blank">https://groups.google.com/d/msgid/android-x86-devel/CAKc24n0zF-Po%3DCXwLt7wHr5ogbV-LGOsDZWqkk%3D3b_pGfc2H-A%40mail.gmail.com</a>. <div class="HOEnZb"><div class="h5">For more options, visit <a href="https://groups.google.com/d/optout" rel="noreferrer" target="_blank">https://groups.google.com/d/optout</a>. </div></div></blockquote></div> </div>