[OE-core] [PATCH] mesa: enable virgl driver for qemux86/qemux86-64

[Mark Hatle]

On 9/20/18 6:37 PM, Burton, Ross wrote:
> I'm assuming you have a local config to turn on gallium, as that isn't
> enabled by default.

This appears to be changing the basehash for mesa.

Fro bitbake-diffsigs:

Hash for dependent task mesa/mesa_18.1.9.bb.do_configure changed from
cd484f650abebbd8126040df82218cb8 to cf71872a3ad0ccea0933b914d1eef467

basehash changed from 8f512789ad94158da66c949e97a15ae8 to
3a5c711c799c7ba6edd399ebf84fcc65

Variable GALLIUMDRIVERS value changed:

@@ -1,4 +1,4 @@
-swrast${@bb.utils.contains('PACKAGECONFIG', 'etnaviv', ',etnaviv', '',
d)}${@bb.utils.contains('PACKAGECONFIG', 'imx', ',imx', '',
d)}${@bb.utils.contains('PACKAGECONFIG', 'gallium-llvm',
',${GALLIUMDRIVERS_LLVM}', '', d)}
+swrast${@bb.utils.contains('PACKAGECONFIG', 'etnaviv', ',etnaviv', '',
d)}${@bb.utils.contains('PACKAGECONFIG', 'imx', ',imx', '',
d)}${@bb.utils.contains('PACKAGECONFIG', 'gallium-llvm',
',${GALLIUMDRIVERS_LLVM}', '', d)},virgl
PACKAGECONFIG{etnaviv} = Unset
PACKAGECONFIG{gallium-llvm} = Unset
PACKAGECONFIG{imx} = Unset

I'm finding this when I try to do a genericx86-64 and qemux86-64 build in the
same project.  The package, and everything depending on mesa keeps building due
to this.

I'm wondering if virgl should be enabled by default, and then disabled as a
distro config if the user really doesn't want it?

--Mark

> Ross
> On Wed, 19 Sep 2018 at 04:28, <liu.ming50 at gmail.com> wrote:
>>
>> From: Ming Liu <liu.ming50 at gmail.com>
>>
>> virgl driver should be enabled for qemux86/qemux86-64 machines, or else
>> it will cause runtime issue due to virtio_gpu driver missing.
>>
>> Signed-off-by: Ming Liu <liu.ming50 at gmail.com>
>> ---
>>  meta/recipes-graphics/mesa/mesa.inc | 2 ++
>>  1 file changed, 2 insertions(+)
>>
>> diff --git a/meta/recipes-graphics/mesa/mesa.inc b/meta/recipes-graphics/mesa/mesa.inc
>> index 5afd0db..8d0e2cb 100644
>> --- a/meta/recipes-graphics/mesa/mesa.inc
>> +++ b/meta/recipes-graphics/mesa/mesa.inc
>> @@ -91,6 +91,8 @@ GALLIUMDRIVERS_LLVM33_ENABLED = "${@oe.utils.version_less_or_equal('MESA_LLVM_RE
>>  GALLIUMDRIVERS_LLVM = "r300,svga,nouveau${@',${GALLIUMDRIVERS_LLVM33}' if ${GALLIUMDRIVERS_LLVM33_ENABLED} else ''}"
>>  GALLIUMDRIVERS_append_x86 = "${@bb.utils.contains('PACKAGECONFIG', 'gallium-llvm', ',${GALLIUMDRIVERS_LLVM}', '', d)}"
>>  GALLIUMDRIVERS_append_x86-64 = "${@bb.utils.contains('PACKAGECONFIG', 'gallium-llvm', ',${GALLIUMDRIVERS_LLVM}', '', d)}"
>> +GALLIUMDRIVERS_append_qemux86 = ",virgl"
>> +GALLIUMDRIVERS_append_qemux86-64 = ",virgl"
>>  # keep --with-gallium-drivers separate, because when only one of gallium versions is enabled, other 2 were adding --without-gallium-drivers
>>  PACKAGECONFIG[gallium]      = "--enable-texture-float --with-gallium-drivers=${GALLIUMDRIVERS}, --without-gallium-drivers"
>>  MESA_LLVM_RELEASE ?= "6.0"
>> --
>> 2.7.4
>>
>> --
>> _______________________________________________
>> Openembedded-core mailing list
>> Openembedded-core at lists.openembedded.org
>> http://lists.openembedded.org/mailman/listinfo/openembedded-core