lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: 
 <SA3PR13MB6372229283A34550FD52F913FD1C2@SA3PR13MB6372.namprd13.prod.outlook.com>
Date: Mon, 6 May 2024 18:30:02 +0000
From: "Bird, Tim" <Tim.Bird@...y.com>
To: Wen Yang <wen.yang@...ux.dev>, Shuah Khan <skhan@...uxfoundation.org>,
        Christian Brauner <brauner@...nel.org>,
        Andrew Morton
	<akpm@...ux-foundation.org>
CC: SShuah Khan <shuah@...nel.org>, Andrei Vagin <avagin@...gle.com>,
        Mathieu
 Desnoyers <mathieu.desnoyers@...icios.com>,
        Steven Rostedt
	<rostedt@...dmis.org>, Dave Young <dyoung@...hat.com>,
        "linux-kselftest@...r.kernel.org" <linux-kselftest@...r.kernel.org>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: RE: [PATCH] selftests: introduce additional eventfd test coverage



> -----Original Message-----
> From: Wen Yang <wen.yang@...ux.dev>
> Add several new test cases which assert corner cases on the eventfd
> mechanism, for example, the supplied buffer is less than 8 bytes,
> attempting to write a value that is too large, etc.
> 
> 	./eventfd_test
> 	# Starting 9 tests from 1 test cases.
> 	#  RUN           global.eventfd01 ...
> 	#            OK  global.eventfd01
> 	ok 1 global.eventfd01

These are horrible test case names.  Can you please use strings which indicate
what is being tested, that are useful to a human?
 -- Tim

> 	#  RUN           global.eventfd02 ...
> 	#            OK  global.eventfd02
> 	ok 2 global.eventfd02
> 	#  RUN           global.eventfd03 ...
> 	#            OK  global.eventfd03
> 	ok 3 global.eventfd03
> 	#  RUN           global.eventfd04 ...
> 	#            OK  global.eventfd04
> 	ok 4 global.eventfd04
> 	#  RUN           global.eventfd05 ...
> 	#            OK  global.eventfd05
> 	ok 5 global.eventfd05
> 	#  RUN           global.eventfd06 ...
> 	#            OK  global.eventfd06
> 	ok 6 global.eventfd06
> 	#  RUN           global.eventfd07 ...
> 	#            OK  global.eventfd07
> 	ok 7 global.eventfd07
> 	#  RUN           global.eventfd08 ...
> 	#            OK  global.eventfd08
> 	ok 8 global.eventfd08
> 	#  RUN           global.eventfd09 ...
> 	#            OK  global.eventfd09
> 	ok 9 global.eventfd09
> 	# PASSED: 9 / 9 tests passed.
> 	# Totals: pass:9 fail:0 xfail:0 xpass:0 skip:0 error:0
> 
> Signed-off-by: Wen Yang <wen.yang@...ux.dev>
> Cc: SShuah Khan <shuah@...nel.org>
> Cc: Christian Brauner <brauner@...nel.org>
> Cc: Andrei Vagin <avagin@...gle.com>
> Cc: Mathieu Desnoyers <mathieu.desnoyers@...icios.com>
> Cc: Steven Rostedt <rostedt@...dmis.org>
> Cc: Andrew Morton <akpm@...ux-foundation.org>
> Cc: Dave Young <dyoung@...hat.com>
> Cc: linux-kselftest@...r.kernel.org
> Cc: linux-kernel@...r.kernel.org
> ---
>  .../filesystems/eventfd/eventfd_test.c        | 116 ++++++++++++++++++
>  1 file changed, 116 insertions(+)
> 
> diff --git a/tools/testing/selftests/filesystems/eventfd/eventfd_test.c b/tools/testing/selftests/filesystems/eventfd/eventfd_test.c
> index f142a137526c..eeab8df5b1b5 100644
> --- a/tools/testing/selftests/filesystems/eventfd/eventfd_test.c
> +++ b/tools/testing/selftests/filesystems/eventfd/eventfd_test.c
> @@ -183,4 +183,120 @@ TEST(eventfd05)
>  	close(fd);
>  }
> 
> +/*
> + * A write(2) fails with the error EINVAL if the size of the supplied buffer
> + * is less than 8 bytes, or if an attempt is made to write the value
> + * 0xffffffffffffffff.
> + */
> +TEST(eventfd06)
> +{
> +	uint64_t value = 1;
> +	ssize_t size;
> +	int fd;
> +
> +	fd = sys_eventfd2(0, 0);
> +	ASSERT_GE(fd, 0);
> +
> +	size = write(fd, &value, sizeof(int));
> +	EXPECT_EQ(size, -1);
> +	EXPECT_EQ(errno, EINVAL);
> +
> +	value = (uint64_t)-1;
> +	size = write(fd, &value, sizeof(value));
> +	EXPECT_EQ(size, -1);
> +	EXPECT_EQ(errno, EINVAL);
> +
> +	close(fd);
> +}
> +
> +/*
> + * A read(2) fails with the error EINVAL if the size of the supplied buffer is
> + * less than 8 bytes.
> + */
> +TEST(eventfd07)
> +{
> +	int value = 0;
> +	ssize_t size;
> +	int fd;
> +
> +	fd = sys_eventfd2(0, 0);
> +	ASSERT_GE(fd, 0);
> +
> +	size = write(fd, &value, sizeof(value));
> +	EXPECT_EQ(size, -1);
> +	EXPECT_EQ(errno, EINVAL);
> +
> +	close(fd);
> +}
> +
> +/*
> + * If EFD_SEMAPHORE was not specified and the eventfd counter has a nonzero
> + * value, then a read(2) returns 8 bytes containing that value, and the
> + * counter's value is reset to zero.
> + * If the eventfd counter is zero at the time of the call to read(2), then the
> + * call fails with the error EAGAIN if the file descriptor has been made nonblocking.
> + */
> +TEST(eventfd08)
> +{
> +	uint64_t value;
> +	ssize_t size;
> +	int fd;
> +	int i;
> +
> +	fd = sys_eventfd2(0, EFD_NONBLOCK);
> +	ASSERT_GE(fd, 0);
> +
> +	value = 1;
> +	for (i = 0; i < 10000000; i++) {
> +		size = write(fd, &value, sizeof(value));
> +		EXPECT_EQ(size, sizeof(value));
> +	}
> +
> +	size = read(fd, &value, sizeof(value));
> +	EXPECT_EQ(size, sizeof(uint64_t));
> +	EXPECT_EQ(value, 10000000);
> +
> +	size = read(fd, &value, sizeof(value));
> +	EXPECT_EQ(size, -1);
> +	EXPECT_EQ(errno, EAGAIN);
> +
> +	close(fd);
> +}
> +
> +/*
> + * If EFD_SEMAPHORE was specified and the eventfd counter has a nonzero value,
> + * then a read(2) returns 8 bytes containing the value 1, and the counter's
> + * value is decremented by 1.
> + * If the eventfd counter is zero at the time of the call to read(2), then the
> + * call fails with the error EAGAIN if the file descriptor has been made nonblocking.
> + */
> +TEST(eventfd09)
> +{
> +	uint64_t value;
> +	ssize_t size;
> +	int fd;
> +	int i;
> +
> +	fd = sys_eventfd2(0, EFD_SEMAPHORE|EFD_NONBLOCK);
> +	ASSERT_GE(fd, 0);
> +
> +	value = 1;
> +	for (i = 0; i < 10000000; i++) {
> +		size = write(fd, &value, sizeof(value));
> +		EXPECT_EQ(size, sizeof(value));
> +	}
> +
> +	for (i = 0; i < 10000000; i++) {
> +		size = read(fd, &value, sizeof(value));
> +		EXPECT_EQ(size, sizeof(value));
> +		EXPECT_EQ(value, 1);
> +	}
> +
> +	size = read(fd, &value, sizeof(value));
> +	EXPECT_EQ(size, -1);
> +	EXPECT_EQ(errno, EAGAIN);
> +
> +	close(fd);
> +}
> +
>  TEST_HARNESS_MAIN
> --
> 2.25.1
> 


Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ