Re: [OE-core] [PATCH v6] mesa: Upgrade to 19.2.1

2019-11-12 Thread Alexander Kanavin
On Tue, 12 Nov 2019 at 19:16, Khem Raj  wrote:

> > > in this case, texrel will endup causing sigsegv, this is a mesa issue,
> > > primarily we have been working it around so far.
> >
> > I sent a new patch, it probably doesn't fix this issue though.
> >
>
> sure, and I think we should be able to address it with a patch or some
> feature disable/enable a mechanism
> I wonder what changed in new build system that it cant be knobbed out
> as it used to be with autotools
>

It is probably upstream's removal of the 'asm' configuration switch which
was previously set to false and now is always on.
Seems like switching that on enables poorly written assembly code that
causes the qa issue?

Also there should be a runtime test for all of this. There is a test for
kmscube over virgl but only for glibc.

Alex
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


Re: [OE-core] [PATCH v6] mesa: Upgrade to 19.2.1

2019-11-12 Thread Khem Raj
On Tue, Nov 12, 2019 at 10:07 AM Alistair Francis  wrote:
>
> On Tue, Nov 12, 2019 at 8:19 AM Khem Raj  wrote:
> >
> > On Sat, Nov 9, 2019 at 7:36 AM Alexander Kanavin  
> > wrote:
> > >
> > > On Sat, 9 Nov 2019 at 00:02, Alistair Francis  
> > > wrote:
> > >>
> > >> > right this means glx-tls is not working anymore, and it will fail on
> > >> > musl at runtime
> > >> > see
> > >> > https://gitlab.freedesktop.org/mesa/mesa/issues/966
> > >>
> > >> So what do we do here?
> > >>
> > >> There are some patches in that issue, but they don't cleanly apply and
> > >> seem hacky anyway. Can we have two versions of mesa? One for musl and
> > >> one for others until this is fixed upstream?
> > >
> > >
> > > Maybe we can silence the warning for musl only, via 
> > > INSANE_SKIP_..._libc-musl = "textrel"?
> > > ffmpeg does the same already.
> >
> > in this case, texrel will endup causing sigsegv, this is a mesa issue,
> > primarily we have been working it around so far.
>
> I sent a new patch, it probably doesn't fix this issue though.
>

sure, and I think we should be able to address it with a patch or some
feature disable/enable a mechanism
I wonder what changed in new build system that it cant be knobbed out
as it used to be with autotools

> We can't just not update mesa because of this.

We can certainly take time to fix it, we are early in release cycle.

What about having two
> versions, or for musl and one for glibc?
>

Probably a bad idea

> Alistair
>
> >
> > >
> > > Alex
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


Re: [OE-core] [PATCH v6] mesa: Upgrade to 19.2.1

2019-11-12 Thread Alistair Francis
On Tue, Nov 12, 2019 at 8:19 AM Khem Raj  wrote:
>
> On Sat, Nov 9, 2019 at 7:36 AM Alexander Kanavin  
> wrote:
> >
> > On Sat, 9 Nov 2019 at 00:02, Alistair Francis  wrote:
> >>
> >> > right this means glx-tls is not working anymore, and it will fail on
> >> > musl at runtime
> >> > see
> >> > https://gitlab.freedesktop.org/mesa/mesa/issues/966
> >>
> >> So what do we do here?
> >>
> >> There are some patches in that issue, but they don't cleanly apply and
> >> seem hacky anyway. Can we have two versions of mesa? One for musl and
> >> one for others until this is fixed upstream?
> >
> >
> > Maybe we can silence the warning for musl only, via 
> > INSANE_SKIP_..._libc-musl = "textrel"?
> > ffmpeg does the same already.
>
> in this case, texrel will endup causing sigsegv, this is a mesa issue,
> primarily we have been working it around so far.

I sent a new patch, it probably doesn't fix this issue though.

We can't just not update mesa because of this. What about having two
versions, or for musl and one for glibc?

Alistair

>
> >
> > Alex
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


Re: [OE-core] [PATCH v6] mesa: Upgrade to 19.2.1

2019-11-12 Thread Khem Raj
On Sat, Nov 9, 2019 at 7:36 AM Alexander Kanavin  wrote:
>
> On Sat, 9 Nov 2019 at 00:02, Alistair Francis  wrote:
>>
>> > right this means glx-tls is not working anymore, and it will fail on
>> > musl at runtime
>> > see
>> > https://gitlab.freedesktop.org/mesa/mesa/issues/966
>>
>> So what do we do here?
>>
>> There are some patches in that issue, but they don't cleanly apply and
>> seem hacky anyway. Can we have two versions of mesa? One for musl and
>> one for others until this is fixed upstream?
>
>
> Maybe we can silence the warning for musl only, via INSANE_SKIP_..._libc-musl 
> = "textrel"?
> ffmpeg does the same already.

in this case, texrel will endup causing sigsegv, this is a mesa issue,
primarily we have been working it around so far.

>
> Alex
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


Re: [OE-core] [PATCH v6] mesa: Upgrade to 19.2.1

2019-11-12 Thread Adrian Bunk
On Fri, Nov 08, 2019 at 10:13:07AM -0800, Khem Raj wrote:
> On Fri, Nov 8, 2019 at 2:32 AM Ross Burton  wrote:
> > On 04/11/2019 22:48, Alistair Francis wrote:
> > > From: Alistair Francis 
> > >
> > > Upgrade mesa and mesa-gl to 19.2.1.
> > >
> > > The license hash change was a trivial new line removal.
> > >
> > > The glx-tls option was removed as it isn't included in the meson.build
> > > file. It has been replaced with 'use-elf-tls' instead.
> >
> > I think this has regressed something, this is a new warning on musl builds:
> >
> > do_package_qa: QA Issue: ELF binary
> > '[...]libgles2-mesa/usr/lib/libGLESv2.so.2.0.0' has relocations in .text
> > [textrel]
> >
> > (ditto for libGLESv1_CM.so.1.1.0 libGL.so.1.2.0 libglapi.so.0.0.0)
> 
> right this means glx-tls is not working anymore, and it will fail on
> musl at runtime
> see
> https://gitlab.freedesktop.org/mesa/mesa/issues/966

glx-tls is already PACKAGECONFIG_remove_libc-musl.

That this is no longer sufficient is the regression Ross mentioned.

cu
Adrian

-- 

   "Is there not promise of rain?" Ling Tan asked suddenly out
of the darkness. There had been need of rain for many days.
   "Only a promise," Lao Er said.
   Pearl S. Buck - Dragon Seed

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


Re: [OE-core] [PATCH v6] mesa: Upgrade to 19.2.1

2019-11-09 Thread Alexander Kanavin
On Sat, 9 Nov 2019 at 00:02, Alistair Francis  wrote:

> > right this means glx-tls is not working anymore, and it will fail on
> > musl at runtime
> > see
> > https://gitlab.freedesktop.org/mesa/mesa/issues/966
>
> So what do we do here?
>
> There are some patches in that issue, but they don't cleanly apply and
> seem hacky anyway. Can we have two versions of mesa? One for musl and
> one for others until this is fixed upstream?
>

Maybe we can silence the warning for musl only, via
INSANE_SKIP_..._libc-musl = "textrel"?
ffmpeg does the same already.

Alex
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


Re: [OE-core] [PATCH v6] mesa: Upgrade to 19.2.1

2019-11-08 Thread Alistair Francis
On Fri, Nov 8, 2019 at 10:13 AM Khem Raj  wrote:
>
> On Fri, Nov 8, 2019 at 2:32 AM Ross Burton  wrote:
> >
> > On 04/11/2019 22:48, Alistair Francis wrote:
> > > From: Alistair Francis 
> > >
> > > Upgrade mesa and mesa-gl to 19.2.1.
> > >
> > > The license hash change was a trivial new line removal.
> > >
> > > The glx-tls option was removed as it isn't included in the meson.build
> > > file. It has been replaced with 'use-elf-tls' instead.
> >
> > I think this has regressed something, this is a new warning on musl builds:
> >
> > do_package_qa: QA Issue: ELF binary
> > '[...]libgles2-mesa/usr/lib/libGLESv2.so.2.0.0' has relocations in .text
> > [textrel]
> >
> > (ditto for libGLESv1_CM.so.1.1.0 libGL.so.1.2.0 libglapi.so.0.0.0)
> >
>
> right this means glx-tls is not working anymore, and it will fail on
> musl at runtime
> see
> https://gitlab.freedesktop.org/mesa/mesa/issues/966

So what do we do here?

There are some patches in that issue, but they don't cleanly apply and
seem hacky anyway. Can we have two versions of mesa? One for musl and
one for others until this is fixed upstream?

Alistair

>
> > Ross
> > --
> > ___
> > Openembedded-core mailing list
> > Openembedded-core@lists.openembedded.org
> > http://lists.openembedded.org/mailman/listinfo/openembedded-core
> --
> ___
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


Re: [OE-core] [PATCH v6] mesa: Upgrade to 19.2.1

2019-11-08 Thread Khem Raj
On Fri, Nov 8, 2019 at 2:32 AM Ross Burton  wrote:
>
> On 04/11/2019 22:48, Alistair Francis wrote:
> > From: Alistair Francis 
> >
> > Upgrade mesa and mesa-gl to 19.2.1.
> >
> > The license hash change was a trivial new line removal.
> >
> > The glx-tls option was removed as it isn't included in the meson.build
> > file. It has been replaced with 'use-elf-tls' instead.
>
> I think this has regressed something, this is a new warning on musl builds:
>
> do_package_qa: QA Issue: ELF binary
> '[...]libgles2-mesa/usr/lib/libGLESv2.so.2.0.0' has relocations in .text
> [textrel]
>
> (ditto for libGLESv1_CM.so.1.1.0 libGL.so.1.2.0 libglapi.so.0.0.0)
>

right this means glx-tls is not working anymore, and it will fail on
musl at runtime
see
https://gitlab.freedesktop.org/mesa/mesa/issues/966

> Ross
> --
> ___
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


Re: [OE-core] [PATCH v6] mesa: Upgrade to 19.2.1

2019-11-08 Thread Ross Burton

On 04/11/2019 22:48, Alistair Francis wrote:

From: Alistair Francis 

Upgrade mesa and mesa-gl to 19.2.1.

The license hash change was a trivial new line removal.

The glx-tls option was removed as it isn't included in the meson.build
file. It has been replaced with 'use-elf-tls' instead.


I think this has regressed something, this is a new warning on musl builds:

do_package_qa: QA Issue: ELF binary 
'[...]libgles2-mesa/usr/lib/libGLESv2.so.2.0.0' has relocations in .text 
[textrel]


(ditto for libGLESv1_CM.so.1.1.0 libGL.so.1.2.0 libglapi.so.0.0.0)

Ross
--
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core