Thanks for the patch. John, since you're going to be doing more QEMU work in 
future I'd encourage you to go through the process of submitting it to 
upstream's mailing list and shepherding it through the patch review process.  
Upstream's patch submission guidelines are here: 
http://wiki.qemu.org/Contribute/SubmitAPatch. A couple of remarks about this 
patch specifically:
 * it's going to need a signed-off-by: line
 * it's fixing two different bugs (the actual futex bug and the strace error) 
and they will need to be in two different patches
 * you don't need to treat FUTEX_WAIT and _WAIT_BITSET separately, you can just 
always pass val3 to sys_futex in both cases (if the op is not FUTEX_WAIT_BITSET 
the kernel will just ignore the extra parameter)

Optional but nice: try the futex related bits of the LTP
(http://wiki.qemu.org/Testing/LTP) and see if more of them pass now (or
at least that we don't regress).

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1129571

Title:
  libreoffice armhf FTBFS

To manage notifications about this bug go to:
https://bugs.launchpad.net/qemu/+bug/1129571/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to