2020-06-17 11:33:52 |
Christian Ehrhardt |
bug |
|
|
added bug |
2020-06-17 11:34:01 |
Christian Ehrhardt |
nominated for series |
|
Ubuntu Bionic |
|
2020-06-17 11:34:01 |
Christian Ehrhardt |
bug task added |
|
binutils (Ubuntu Bionic) |
|
2020-06-17 11:34:07 |
Christian Ehrhardt |
binutils (Ubuntu): status |
New |
Fix Released |
|
2020-06-17 11:49:40 |
Frode Nordahl |
bug |
|
|
added subscriber Frode Nordahl |
2020-06-17 12:49:42 |
Harry van Haaren (Intel) |
bug |
|
|
added subscriber Harry van Haaren |
2020-06-17 15:01:57 |
Bruce Richardson (Intel) |
bug |
|
|
added subscriber Bruce Richardson (Intel) |
2020-06-18 14:38:44 |
Christian Ehrhardt |
bug |
|
|
added subscriber Luca Boccassi |
2020-06-19 07:24:43 |
Launchpad Janitor |
merge proposal linked |
|
https://code.launchpad.net/~paelzer/ubuntu/+source/binutils/+git/binutils/+merge/386073 |
|
2020-06-19 07:30:30 |
Christian Ehrhardt |
description |
Hi,
DPDK has run into some issues in the past
https://bugs.dpdk.org/show_bug.cgi?id=97
https://bugs.dpdk.org/show_bug.cgi?id=249
Eventually the issues got resolved in binutils via
https://sourceware.org/bugzilla/show_bug.cgi?id=23465
After binutils is fixed people rebuilding DPDK themselve can use
http://patches.dpdk.org/patch/71679/
to gain more performance while on Bionics bintuils level.
Note: Bionic is on DPDK 17.11.x which will not get further stable release afaik. But quite often people build their own DPDK. In fact this came up as a request from Openvswitch upstream/Intel to allow such builds on Bionic.
I'd ping those people about the bug and ask them to participate in the verification if this becomes an SRU. |
[Impact]
* the assembler scales non 8 bit cases which was identified
to break e.g. some AVX512 code. It is nasty as it isn't a compile/link/
time error. Instead the instructions might silently be corrupted until
running. Things might even work on some but fail on other systems if
e.g. the AVX code paths only run on newer chips.
* The fix is upstream for a while and not re-changed again. Furthermore
it is in several Ubuntu releases without bugs due to that, which should
make the backport rather safe.
[Test Case]
* Simple example to trigger the bug:
echo "vmovaps 0x40(,%rax,1),%zmm0" | as --64 -o avx.o && objdump -d avx.o | grep vmovaps
The expected output is that the objdump output matches the vmovaps instruction input. When using binutils with the bug, the initial 0x40 will be incorrect.
Working:
$ echo "vmovaps 0x40(,%rax,1),%zmm0" | as --64 -o avx.o && objdump -d avx.o | grep vmovaps
0: 62 f1 7c 48 28 04 05 vmovaps 0x40(,%rax,1),%zmm0
Failing:
$ echo "vmovaps 0x40(,%rax,1),%zmm0" | as --64 -o avx.o && objdump -d avx.o | grep vmovaps
0: 62 f1 7c 48 28 04 05 vmovaps 0x1(,%rax,1),%zmm0
[Regression Potential]
* Well, this is a double edged sword. On one hand this is fortunately a
small change and only affects something formerly clearly broken. So it
should be good and only change cases formerly being bad.
But OTOH binutils areused in so many cases that I feel unable to say
"nothing will happen". The change goes to the gnu assembler, so that is
the place to look out for regressions.
[Other Info]
* needs a sponsor experienced with binutils to check potential pitfalls
---
Hi,
DPDK has run into some issues in the past
https://bugs.dpdk.org/show_bug.cgi?id=97
https://bugs.dpdk.org/show_bug.cgi?id=249
Eventually the issues got resolved in binutils via
https://sourceware.org/bugzilla/show_bug.cgi?id=23465
After binutils is fixed people rebuilding DPDK themselve can use
http://patches.dpdk.org/patch/71679/
to gain more performance while on Bionics bintuils level.
Note: Bionic is on DPDK 17.11.x which will not get further stable release afaik. But quite often people build their own DPDK. In fact this came up as a request from Openvswitch upstream/Intel to allow such builds on Bionic.
I'd ping those people about the bug and ask them to participate in the verification if this becomes an SRU. |
|
2020-06-19 07:33:40 |
Christian Ehrhardt |
binutils (Ubuntu Bionic): assignee |
|
Matthias Klose (doko) |
|
2020-06-24 05:25:34 |
Christian Ehrhardt |
binutils (Ubuntu Bionic): status |
New |
In Progress |
|
2020-06-24 05:25:37 |
Christian Ehrhardt |
binutils (Ubuntu Bionic): assignee |
Matthias Klose (doko) |
Christian Ehrhardt (paelzer) |
|
2020-06-24 05:25:45 |
Christian Ehrhardt |
tags |
|
server-next |
|
2020-06-24 05:25:58 |
Christian Ehrhardt |
bug |
|
|
added subscriber Ubuntu Server |
2020-07-15 15:56:34 |
Robie Basak |
binutils (Ubuntu Bionic): status |
In Progress |
Fix Committed |
|
2020-07-15 15:56:36 |
Robie Basak |
bug |
|
|
added subscriber Ubuntu Stable Release Updates Team |
2020-07-15 15:56:39 |
Robie Basak |
bug |
|
|
added subscriber SRU Verification |
2020-07-15 15:56:44 |
Robie Basak |
tags |
server-next |
server-next verification-needed verification-needed-bionic |
|
2020-07-16 13:18:29 |
Christian Ehrhardt |
tags |
server-next verification-needed verification-needed-bionic |
server-next verification-done verification-done-bionic |
|
2020-07-16 14:38:33 |
Christian Ehrhardt |
bug watch added |
|
https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=963702 |
|
2020-07-16 14:38:33 |
Christian Ehrhardt |
bug task added |
|
binutils (Debian) |
|
2020-07-20 20:30:44 |
Bug Watch Updater |
binutils (Debian): status |
Unknown |
Confirmed |
|
2020-08-04 07:45:02 |
Łukasz Zemczak |
removed subscriber Ubuntu Stable Release Updates Team |
|
|
|
2020-08-04 07:45:01 |
Launchpad Janitor |
binutils (Ubuntu Bionic): status |
Fix Committed |
Fix Released |
|
2023-06-03 02:21:08 |
Bug Watch Updater |
binutils (Debian): status |
Confirmed |
Fix Released |
|