I think we should actually disable dri (non-gallium) drivers altogether.
They're only useful for ancient x86 hardware, and upstream is talking about
removing them from mainline branch and onto a legacy branch in 21.2
timeframe.
Alex
On Fri, 2 Apr 2021 at 19:30, Khem Raj <raj.k...@gmail.com> wrote:
> yeah we need to start using nouveau dri driver perhaps instead of swrast
>
> On Fri, Apr 2, 2021 at 3:15 AM Martin Jansa <martin.ja...@gmail.com>
> wrote:
> >
> > Unfortunately it still fails the same for qemux86* with this change.
> >
> > Enabling wayland PACKAGECONFIG doesn't help as well.
> >
> > Building for qemuarm with empty DRIDRIVERS does pass the do_configure
> with:
> > EXTRA_OEMESON=" -Dshared-glapi=enabled
> -Dgallium-opencl=disabled -Dglx-read-only-text=true
> -Dplatforms='wayland' -Ddri-drivers= -Ddri3=disabled -Degl=disabled
> -Delf-tls=false -Dgallium-drivers=swrast,swrast -Dllvm=disabled
> -Dgbm=disabled -D
> > gles1=disabled -Dgles2=disabled -Dopengl=true -Dosmesa=true
> -Dlibunwind=disabled -Dgallium-va=disabled -Dgallium-vdpau=disabled
> -Dvulkan-drivers='' -Dglx=disabled -Dgallium-xa=disabled
> -Dgallium-xvmc=disabled"
> >
> > which is a bit surprising as mesa-gl_21.0.0.bb comment says:
> > # When NOT using X11, we need to make sure we have at least one driver
> enabled
> > and now the only build which isn't broken by the 21 upgrade is the one
> without any driver.
> >
> > and x86 and x86-64 always enable some DRIDRIVERS in mesa.inc:
> > DRIDRIVERS_class-native = "nouveau"
> > DRIDRIVERS_class-nativesdk = "nouveau"
> > DRIDRIVERS_append_x86_class-target = ",r100,r200,nouveau,i965,i915"
> > DRIDRIVERS_append_x86-64_class-target = ",r100,r200,nouveau,i965,i915"
> >
> > Maybe this upgrade to development version 21.0.0 should be reverted and
> 21.0.1 should be merged after this meta-gl breakage is resolved somehow?
> >
> > On Thu, Apr 1, 2021 at 3:58 PM Khem Raj <raj.k...@gmail.com> wrote:
> >>
> >> On Thu, Apr 1, 2021 at 3:19 AM Martin Jansa <martin.ja...@gmail.com>
> wrote:
> >> >
> >> > On Wed, Mar 31, 2021 at 03:50:51PM -0700, Khem Raj wrote:
> >> > > Fixes
> >> > > ../mesa-21.0.0/meson.build:21:0: ERROR: Options "swrast" are not in
> allowed choices: "auto, i915, i965, r100, r200, nouveau"
> >> > >
> >> > > Signed-off-by: Khem Raj <raj.k...@gmail.com>
> >> > > Cc: Martin Jansa <martin.ja...@gmail.com>
> >> > > ---
> >> > > meta/recipes-graphics/mesa/mesa-gl_21.0.0.bb | 2 +-
> >> > > 1 file changed, 1 insertion(+), 1 deletion(-)
> >> > >
> >> > > diff --git a/meta/recipes-graphics/mesa/mesa-gl_21.0.0.bb
> b/meta/recipes-graphics/mesa/mesa-gl_21.0.0.bb
> >> > > index e50782be1c..fc8b4f7504 100644
> >> > > --- a/meta/recipes-graphics/mesa/mesa-gl_21.0.0.bb
> >> > > +++ b/meta/recipes-graphics/mesa/mesa-gl_21.0.0.bb
> >> > > @@ -12,4 +12,4 @@ PACKAGECONFIG ??= "opengl dri
> ${@bb.utils.contains('DISTRO_FEATURES', 'x11', 'x1
> >> > > PACKAGECONFIG_class-target = "opengl dri
> ${@bb.utils.contains('DISTRO_FEATURES', 'x11', 'x11', 'osmesa', d)}"
> >> > >
> >> > > # When NOT using X11, we need to make sure we have swrast
> available.
> >> > > -DRIDRIVERS_append = "${@bb.utils.contains('DISTRO_FEATURES',
> 'x11', '', ',swrast', d)}"
> >> > > +GALLIUMDRIVERS_append = "${@bb.utils.contains('DISTRO_FEATURES',
> 'x11', '', ',swrast', d)}"
> >> >
> >> > With:
> >> > DISTRO_FEATURES_remove = "x11"
> >> > PREFERRED_PROVIDER_virtual/libgl = "mesa-gl"
> >> > PREFERRED_PROVIDER_virtual/mesa = "mesa-gl"
> >> > in local.conf this is unfortunately still failing, now with:
> >> >
> >> > ../mesa-21.0.0/meson.build:519:4: ERROR: Problem encountered:
> building dri drivers require at least one windowing system
> >> >
> >> > whole log:
> >> > http://errors.yoctoproject.org/Errors/Details/575265/
> >> >
> >> > adding nouveau to DRIDRIVERS like normal mesa doesn't help in this
> case,
> >> > because I was building for qemux86-64 which already has bunch of
> >> > DRIDRIVERS added by default (but adding at least one DRIDRIVER might
> be still
> >> > needed for other architectures).
> >> >
> >>
> >> classic OSMesa is removed and swrast gallium drivers are needed for
> osmsa to run
> >> could you also add below patch and try. ?
> >>
> >>
> >> diff --git a/meta/recipes-graphics/mesa/mesa-gl_21.0.0.bb
> >> b/meta/recipes-graphics/mesa/mesa-gl_21.0.0.bb
> >> index fc8b4f7504..47b14fb97f 100644
> >> --- a/meta/recipes-graphics/mesa/mesa-gl_21.0.0.bb
> >> +++ b/meta/recipes-graphics/mesa/mesa-gl_21.0.0.bb
> >> @@ -8,8 +8,8 @@ S = "${WORKDIR}/mesa-${PV}"
> >>
> >> # At least one DRI rendering engine is required to build mesa.
> >> # When no X11 is available, use osmesa for the rendering engine.
> >> -PACKAGECONFIG ??= "opengl dri ${@bb.utils.contains('DISTRO_FEATURES',
> >> 'x11', 'x11', 'osmesa', d)}"
> >> -PACKAGECONFIG_class-target = "opengl dri
> >> ${@bb.utils.contains('DISTRO_FEATURES', 'x11', 'x11', 'osmesa', d)}"
> >> +PACKAGECONFIG ??= "opengl dri ${@bb.utils.contains('DISTRO_FEATURES',
> >> 'x11', 'x11', 'osmesa gallium', d)}"
> >> +PACKAGECONFIG_class-target = "opengl dri
> >> ${@bb.utils.contains('DISTRO_FEATURES', 'x11', 'x11', 'osmesa
> >> gallium', d)}"
> >>
> >> # When NOT using X11, we need to make sure we have swrast available.
> >> GALLIUMDRIVERS_append = "${@bb.utils.contains('DISTRO_FEATURES',
> >> 'x11', '', ',swrast', d)}"
> >>
> >>
> >> > # $DRIDRIVERS [4 operations]
> >> > # _append[x86_class-target]
> /OE/build/oe-core/openembedded-core/meta/recipes-graphics/mesa/mesa.inc:107
> >> > # ",r100,r200,nouveau,i965,i915"
> >> > # _append[x86-64_class-target]
> /OE/build/oe-core/openembedded-core/meta/recipes-graphics/mesa/mesa.inc:108
> >> > # ",r100,r200,nouveau,i965,i915"
> >> > # override[class-native]:set
> /OE/build/oe-core/openembedded-core/meta/recipes-graphics/mesa/mesa.inc:105
> >> > # "nouveau"
> >> > # override[class-nativesdk]:set
> /OE/build/oe-core/openembedded-core/meta/recipes-graphics/mesa/mesa.inc:106
> >> > # "nouveau"
> >> > # pre-expansion value:
> >> > # ",r100,r200,nouveau,i965,i915"
> >> > DRIDRIVERS=",r100,r200,nouveau,i965,i915"
> >> >
> >> > Reverting Alex's upgrade to 21.0.0 allows to build mesa-gl again and
> it should surely pass AB build :).
>
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#150184):
https://lists.openembedded.org/g/openembedded-core/message/150184
Mute This Topic: https://lists.openembedded.org/mt/81763397/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-
