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: > > From: Andrei Vagin <avagin@xxxxxxxxxx> > > > > 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 <avagin@xxxxxxxxxx> > > --- > > 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. -- To unsubscribe from this list: send the line "unsubscribe linux-api" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
