diff options
author | Richard Purdie <richard.purdie@linuxfoundation.org> | 2021-02-16 19:04:42 +0000 |
---|---|---|
committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2021-02-17 11:01:14 +0000 |
commit | c68349a3334542e51f6c1c23f4f4342e4fce2b81 (patch) | |
tree | 06c02e8b5ad5d372b842fe4e2e175b04c14a91ce | |
parent | 84d3a90557444d8cd83d780b1c3f2b278ab07af2 (diff) | |
download | openembedded-core-contrib-c68349a3334542e51f6c1c23f4f4342e4fce2b81.tar.gz |
qemu: Refresh mmap fixes patch status/content
One mmap patch was merged upstream, the other is in progress but after
discussion has been amended and resent. Update the patches as such.
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
-rw-r--r-- | meta/recipes-devtools/qemu/qemu/mmap.patch | 2 | ||||
-rw-r--r-- | meta/recipes-devtools/qemu/qemu/mmap2.patch | 26 |
2 files changed, 21 insertions, 7 deletions
diff --git a/meta/recipes-devtools/qemu/qemu/mmap.patch b/meta/recipes-devtools/qemu/qemu/mmap.patch index 0f7d2ce04c..edd9734f30 100644 --- a/meta/recipes-devtools/qemu/qemu/mmap.patch +++ b/meta/recipes-devtools/qemu/qemu/mmap.patch @@ -9,7 +9,7 @@ avoid asserting binaries when reserved_va is set. This meant a test case now gives the same behaviour regardless of whether reserved_va is set or not. -Upstream-Status: Pending +Upstream-Status: Backport [https://github.com/qemu/qemu/commit/ccc5ccc17f8cfbfd87d9aede5d12a2d47c56e712] Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org Index: qemu-5.2.0/linux-user/mmap.c diff --git a/meta/recipes-devtools/qemu/qemu/mmap2.patch b/meta/recipes-devtools/qemu/qemu/mmap2.patch index 9d40565938..1652131757 100644 --- a/meta/recipes-devtools/qemu/qemu/mmap2.patch +++ b/meta/recipes-devtools/qemu/qemu/mmap2.patch @@ -6,21 +6,35 @@ if it only sees ENOMEM and only exits when it hits EFAULT. According to the docs, trying to mremap outside the address space can/should return EFAULT and changing this allows the build to succeed. -Whether this should be fixed in qemu and/or musl, not sure. -Upstream-Status: Pending +A better return value for the other cases of invalid addresses is EINVAL +rather than ENOMEM so adjust the other part of the test to this. + +Upstream-Status: Submitted [https://lists.gnu.org/archive/html/qemu-devel/2021-01/msg01355.html] Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org Index: qemu-5.2.0/linux-user/mmap.c =================================================================== --- qemu-5.2.0.orig/linux-user/mmap.c +++ qemu-5.2.0/linux-user/mmap.c -@@ -727,7 +727,7 @@ abi_long target_mremap(abi_ulong old_add - !guest_range_valid(new_addr, new_size)) || - ((flags & MREMAP_MAYMOVE) == 0 && - !guest_range_valid(old_addr, new_size))) { +@@ -722,12 +722,14 @@ abi_long target_mremap(abi_ulong old_add + int prot; + void *host_addr; + +- if (!guest_range_valid(old_addr, old_size) || +- ((flags & MREMAP_FIXED) && +- !guest_range_valid(new_addr, new_size)) || +- ((flags & MREMAP_MAYMOVE) == 0 && +- !guest_range_valid(old_addr, new_size))) { - errno = ENOMEM; ++ if (!guest_range_valid(old_addr, old_size)) { + errno = EFAULT; ++ return -1; ++ } ++ ++ if (((flags & MREMAP_FIXED) && !guest_range_valid(new_addr, new_size)) || ++ ((flags & MREMAP_MAYMOVE) == 0 && !guest_range_valid(old_addr, new_size))) { ++ errno = EINVAL; return -1; } |