[Mesa-dev] [ANNOUNCE] Mesa 18.3.3 release candidate

Eric Anholt eric at anholt.net
Tue Feb 5 00:31:57 UTC 2019


Carsten Haitzler <raster at rasterman.com> writes:

> On Fri, 1 Feb 2019 11:08:07 +0000 Emil Velikov <emil.l.velikov at gmail.com> said:
>
>> Hi Carsten,
>> 
>> On 2019/01/31, Carsten Haitzler wrote:
>> > On Wed, 30 Jan 2019 18:33:35 +0000 Emil Velikov <emil.l.velikov at gmail.com>
>> > said:
>> > 
>> > You might want to hold off on this. My bugfix was actually patched out by
>> > partly removing some of it. The void ptr math should never have been there
>> > and wasn't in the final patch.
>> > 
>> > I'm talking about:
>> > 
>> > +                void *cpu2 = cpu + 8;
>> > 
>> > In 300d3ae8b1445b5060f92c77c0f577f4b7b2c7d6
>> > 
>> > At least with gcc8 mesa is a dud on Raspberry Pi (can't upload/downlaod
>> > textures without crashing) without the fixes. I moved the secondary ptr math
>> > into the ASM chunk because the C compiler seemed to just mess up cpu2 ptr
>> > content/value for me on gcc8 (it also kept the parameter inputs/outputs
>> > cleaner and consistent with other ASM chunks). Keeping this as void ptr
>> > math alone is just wrong and asking for trouble and as it unfixed a fix I
>> > already had in submitted patches.
>> > 
>> > Being at FOSDEM I now no longer have access to my OS image with all of this
>> > set up to test and won't until next week. I can't dig in and verify.
>> > Without my fixes at all it's a dead man walking with gcc8, and thus Arch
>> > Linux is broken entirely on Rpi without it (and has been for a while now).

FWIW, my testing was done on gcc 8.30 on raspberry pi.

I skipped the part of moving the C expression into the asm because it
didn't make sense, and appeared in the series before the part that
actually fixed the asm clobbers bug, so it (like the .fpu neon part)
looked like random hacks.

>> Thus from stable POV, we're safe, since nothing has regressed per se. We will
>> apply the extra patches for the next release.
>> 
>> Thanks
>> Emil
>> 
>> P.S. How did you submit the patches - I cannot see them neither on mesa-dev
>> mailing list nor gitlab MR.
>
> i mailed them to eric as he was listed as the maintainer.

I suggested an MR but said I'd accept mail, since raster didn't seem to
want to use normal channels.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 832 bytes
Desc: not available
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20190204/9f341895/attachment.sig>


More information about the mesa-dev mailing list