Thread (9 messages) 9 messages, 3 authors, 2017-11-23

Re: [PATCH v3 4/4] test: add a test for the process_vmsplice syscall

From: Mike Rapoport <hidden>
Date: 2017-11-23 14:07:25
Also in: linux-fsdevel, linux-mm, lkml

On Thu, Nov 23, 2017 at 09:01:03AM +0100, Greg KH wrote:
On Wed, Nov 22, 2017 at 09:36:31PM +0200, Mike Rapoport wrote:
quoted
From: Andrei Vagin <redacted>

This test checks that process_vmsplice() can splice pages from a remote
process and returns EFAULT, if process_vmsplice() tries to splice pages
by an unaccessiable address.

Signed-off-by: Andrei Vagin <redacted>
---
 tools/testing/selftests/process_vmsplice/Makefile  |   5 +
 .../process_vmsplice/process_vmsplice_test.c       | 188 +++++++++++++++++++++
 2 files changed, 193 insertions(+)
 create mode 100644 tools/testing/selftests/process_vmsplice/Makefile
 create mode 100644 tools/testing/selftests/process_vmsplice/process_vmsplice_test.c
[ ... ]
Shouldn't you check to see if the syscall is even present?  You should
not error if it is not, as this test will then "fail" on kernels/arches
without the syscall enabled, which isn't the nicest.
Sure, will fix.
thanks,

greg k-h
-- 
Sincerely yours,
Mike.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help