Re: [Replicant] [PATCH 0/7] Per process libagl/llvmpipe

2020-01-02 Thread Denis 'GNUtoo' Carikli
On Mon, 23 Dec 2019 21:36:39 +0100 Denis 'GNUtoo' Carikli wrote: > According to Adfeno on IRC, that patch is supposed to make the stock > graphical interface about as fast than it was before. > > With libagl being switched to "ro.libagl=0" with the scripts to do > that, the graphical interface i

Re: [Replicant] [PATCH 0/7] Per process libagl/llvmpipe

2019-12-23 Thread Denis 'GNUtoo' Carikli
On Thu, 12 Dec 2019 23:52:20 + dllud wrote: > Thanks a lot for preparing the patch Denis. > > It is missing one important thing though. Please set "ro.libagl=0" in > vendor/replicant/config/common.mk line 242. Like this: > [...] > Particularly, this patch places some files that make bootani

Re: [Replicant] [PATCH 0/7] Per process libagl/llvmpipe

2019-12-12 Thread dllud
Thanks a lot for preparing the patch Denis. It is missing one important thing though. Please set "ro.libagl=0" in vendor/replicant/config/common.mk line 242. Like this: - # use Android's software renderer by default - PRODUCT_PROPERTY_OVERRIDES += ro.libagl=1 + # This makes sure that Mesa (llvmpi

Re: [Replicant] [PATCH 0/7] Per process libagl/llvmpipe

2019-12-12 Thread Denis 'GNUtoo' Carikli
On Thu, 12 Dec 2019 02:35:34 + dllud wrote: > Sorry, my bad, as Joonas correctly reviewed, the last patch shouldn't > be merged. > Instead, "ro.libagl=0" should be set in > vendor/replicant/config/common.mk in order to have llvmpipe as > default. Thanks. During testing, I was wondering why i

Re: [Replicant] [PATCH 0/7] Per process libagl/llvmpipe

2019-12-11 Thread dllud
Sorry, my bad, as Joonas correctly reviewed, the last patch shouldn't be merged. Instead, "ro.libagl=0" should be set in vendor/replicant/config/common.mk in order to have llvmpipe as default. Along with that we need the two commits from adjtm that I mentioned previously, otherwise the system will

Re: [Replicant] [PATCH 0/7] Per process libagl/llvmpipe

2019-12-11 Thread dllud
The last patch (PATCH 7/7) should be pushed alongside with a new patch that includes the content of the following two commits available on adjtm's personal repo: https://github.com/andresdju/vendor_replicant/commit/19fe4dddff99ba0a12b4ff2545df65562170847b https://github.com/andresdju/vendor_replica

Re: [Replicant] [PATCH 0/7] Per process libagl/llvmpipe

2019-10-14 Thread Joonas Kylmälä
Hi, pushed all except the last patch. Joonas Kylmälä: > Hi, > > thanks for the reply! > > I'm planning to push these patches tomorrow so if you have any > objections please do that now. > > Joonas > > Andrés Domínguez: >> Hi. >> >> As Joonas says, that last patch should not be pushed, without

Re: [Replicant] [PATCH 0/7] Per process libagl/llvmpipe

2019-10-13 Thread Joonas Kylmälä
Hi, thanks for the reply! I'm planning to push these patches tomorrow so if you have any objections please do that now. Joonas Andrés Domínguez: > Hi. > > As Joonas says, that last patch should not be pushed, without it we have > the expected behavior: it uses the system default opengles rende

Re: [Replicant] [PATCH 0/7] Per process libagl/llvmpipe

2019-10-13 Thread Andrés Domínguez
Hi. As Joonas says, that last patch should not be pushed, without it we have the expected behavior: it uses the system default opengles renderer. Andrés Domínguez El jue., 10 oct. 2019 a las 20:07, Joonas Kylmälä () escribió: > Hi, > > so we didn't get anyone to review these patches except the

Re: [Replicant] [PATCH 0/7] Per process libagl/llvmpipe

2019-10-10 Thread Joonas Kylmälä
Hi, so we didn't get anyone to review these patches except the last one: . The last patch turned out to be not needed as my testing showed and reading the patch's logic it is doing nothing except forcing llvmpipe which is not wanted. I suggest we p

Re: [Replicant] [PATCH 0/7] Per process libagl/llvmpipe

2019-10-09 Thread Joonas Kylmälä
Hi, I got this setup working successfully if I don't apply the last patch. Without the last patch I can choose the default renderer with instructions at https://redmine.replicant.us/projects/replicant/wiki/Graphics#Enabling-llvmpipe-as-software-renderer. And then if the system default renderer is

Re: [Replicant] [PATCH 0/7] Per process libagl/llvmpipe

2019-09-06 Thread Denis 'GNUtoo' Carikli
On Thu, 15 Aug 2019 21:04:26 +0200 Andrés Domínguez wrote: > Selection of libagl or llvmpipe at EGL link time. It's based > on the work of Jookia and JeremyRand, but it uses files for > configuration instead of properties or symlinks. Hi, I tried to review the patches but I lack enough knowledge

[Replicant] [PATCH 0/7] Per process libagl/llvmpipe

2019-08-15 Thread Andrés Domínguez
Selection of libagl or llvmpipe at EGL link time. It's based on the work of Jookia and JeremyRand, but it uses files for configuration instead of properties or symlinks. First it looks for files with the names libGLES_mesa and libGLES_android (in this order) inside /data/data//, it links to the f