Code review comment for ~rafaeldtinoco/ubuntu/+source/qemu:lp1805256-bionic-refix

Revision history for this message
dann frazier (dannf) wrote :

On Tue, Jul 21, 2020 at 10:40 PM Rafael David Tinoco
<email address hidden> wrote:
>
> Right,
>
> so TARGET_ARM was only defined in some parts of the build and wasn't appropriate for async code (also because it would include armhf and that wasn't needed based in our observations).

I don't know that we actually did any direct testing on armhf.
Unfortunately we don't have any hw in the lab that both reproduces
this problem and supports 32-bit. But I spun up a Graviton2 metal
instance, and found that it does both. I'm running the test case in a
loop in an armhf chroot to see if it can be reproduced there.

  -dann

« Back to merge proposal