mbox series

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

Message ID 20240325192436.561154-1-iii@linux.ibm.com
Headers show
Series linux-user: Fix shmat(NULL) for h != g | expand

Message

Ilya Leoshkevich March 25, 2024, 7:22 p.m. UTC 
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

Comments

Richard Henderson March 27, 2024, 5:55 a.m. UTC | #1 
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~