[v2] linux-user: Fix shmat(NULL) for h != g

[PATCH v2 0/4] linux-user: Fix shmat(NULL) for h != g

Ilya Leoshkevich posted 4 patches 1 month ago

Diff against v1
Download series mbox

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20240325192436.561154-1-iii@linux.ibm.com

Maintainers: Laurent Vivier <laurent@vivier.eu>, "Alex Bennée" <alex.bennee@linaro.org>

linux-user/mmap.c                            |  2 +-
linux-user/strace.c                          | 10 ++----
tests/tcg/multiarch/linux/linux-shmat-null.c | 38 ++++++++++++++++++++
3 files changed, 42 insertions(+), 8 deletions(-)
create mode 100644 tests/tcg/multiarch/linux/linux-shmat-null.c

Expand all Fold all

[PATCH v2 0/4] linux-user: Fix shmat(NULL) for h != g

Posted by Ilya Leoshkevich 1 month ago

v1: https://lore.kernel.org/qemu-devel/20240325153313.526888-1-iii@linux.ibm.com/
v1 -> v2: Remove an unnecessary ifdef, add R-Bs (Richard).

Hi,

I noticed that while shmat() now works with /proc/self/maps,
shmat(NULL) got broken. This series fixes that along with two related
strace issues, and adds a test.

Best regards,
Ilya


Ilya Leoshkevich (4):
  linux-user: Fix semctl() strace
  linux-user: Fix shmat() strace
  linux-user: Fix shmat(NULL) for h != g
  tests/tcg: Test shmat(NULL)

 linux-user/mmap.c                            |  2 +-
 linux-user/strace.c                          | 10 ++----
 tests/tcg/multiarch/linux/linux-shmat-null.c | 38 ++++++++++++++++++++
 3 files changed, 42 insertions(+), 8 deletions(-)
 create mode 100644 tests/tcg/multiarch/linux/linux-shmat-null.c

-- 
2.44.0

Re: [PATCH v2 0/4] linux-user: Fix shmat(NULL) for h != g

Posted by Richard Henderson 1 month ago

On 3/25/24 09:22, Ilya Leoshkevich wrote:
> v1: https://lore.kernel.org/qemu-devel/20240325153313.526888-1-iii@linux.ibm.com/
> v1 -> v2: Remove an unnecessary ifdef, add R-Bs (Richard).
> 
> Hi,
> 
> I noticed that while shmat() now works with /proc/self/maps,
> shmat(NULL) got broken. This series fixes that along with two related
> strace issues, and adds a test.
> 
> Best regards,
> Ilya
> 
> 
> Ilya Leoshkevich (4):
>    linux-user: Fix semctl() strace
>    linux-user: Fix shmat() strace
>    linux-user: Fix shmat(NULL) for h != g
>    tests/tcg: Test shmat(NULL)
> 
>   linux-user/mmap.c                            |  2 +-
>   linux-user/strace.c                          | 10 ++----
>   tests/tcg/multiarch/linux/linux-shmat-null.c | 38 ++++++++++++++++++++
>   3 files changed, 42 insertions(+), 8 deletions(-)
>   create mode 100644 tests/tcg/multiarch/linux/linux-shmat-null.c
> 

Queued, thanks.


r~