linux-user: Small fix for copy_file_range

[PATCH 0/2] linux-user: Small fix for copy_file_range

LemonBoy posted 2 patches 2 years, 11 months ago

Download series mbox

Test checkpatch passed

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20210503174159.54302-1-thatlemon@gmail.com

Maintainers: Laurent Vivier <laurent@vivier.eu>

linux-user/strace.list | 3 +++
linux-user/syscall.c   | 3 ++-
2 files changed, 5 insertions(+), 1 deletion(-)

Expand all Fold all

[PATCH 0/2] linux-user: Small fix for copy_file_range

Posted by LemonBoy 2 years, 11 months ago

From: Giuseppe Musacchio <thatlemon@gmail.com>

An unwanted sign-extension is currently making the host kernel return EOVERFLOW
when calling copy_file_range with a count of 0xffffffff, despite the host being
a 64bit platform.

The fix is only affecting 32bit targets on 64bit hosts, it's a no-op when the
target bitness matches the host one.

As a bonus add a pretty-printer for copy_file_range syscall.

Giuseppe Musacchio (2):
  linux-user: Add copy_file_range to strace.list
  linux-user: Fix erroneous conversion in copy_file_range

 linux-user/strace.list | 3 +++
 linux-user/syscall.c   | 3 ++-
 2 files changed, 5 insertions(+), 1 deletion(-)

-- 
2.30.2

Re: [PATCH 0/2] linux-user: Small fix for copy_file_range

Posted by Laurent Vivier 2 years, 11 months ago

Le 03/05/2021 à 19:41, LemonBoy a écrit :
> From: Giuseppe Musacchio <thatlemon@gmail.com>
> 
> An unwanted sign-extension is currently making the host kernel return EOVERFLOW
> when calling copy_file_range with a count of 0xffffffff, despite the host being
> a 64bit platform.
> 
> The fix is only affecting 32bit targets on 64bit hosts, it's a no-op when the
> target bitness matches the host one.
> 
> As a bonus add a pretty-printer for copy_file_range syscall.
> 
> Giuseppe Musacchio (2):
>   linux-user: Add copy_file_range to strace.list
>   linux-user: Fix erroneous conversion in copy_file_range
> 
>  linux-user/strace.list | 3 +++
>  linux-user/syscall.c   | 3 ++-
>  2 files changed, 5 insertions(+), 1 deletion(-)
> 


Applied to my linux-user-for-6.1 branch.

Thanks,
Laurent