Re: [PATCH 0/3 RFC v2] selftests: Basic framework for tests

From: Daniel Borkmann
Date: Thu Apr 25 2013 - 07:27:29 EST

On 04/25/2013 01:04 PM, Alexandru Copot wrote:
This series adds a generic test abstraction that can make
writing testcases easier. A generic_test structure is
used to define a test and its methods: prepare, run, cleanup.

This is a generic implementation so it was placed in selftests/lib.

The second patch updates the socket tests to use the
new framework and the third patch creates new tests
for [set/get]sockopt with some IPV6_* options.

This already looks better than the previous one. I will probably go
a bit more in depth through the code in the evening. A couple of minor
general items I can already tell you:

Signed-of-by Alexandru Copot <alex.mihai.c@xxxxxxxxx>

It's: Signed-off-by: Alexandru Copot <alex.mihai.c@xxxxxxxxx>

You have this wrong in all your patches.

Also, in patches 1-3 a proper commit message would be nice, i.e. in
patch 1 when you add the library that everyone should use. It could be
that your cover letter will not go into the Git history, thus people
will only see you empty commit bodies.

Cc: Daniel Baluta <dbaluta@xxxxxxxxxxx>

Changes since v1:
- moved the implementation to selftests/lib
- use goto instead of directly returning

Alexandru Copot (3):
selftests: introduce testing abstractions
selftests/net: update socket test to use new testing framework
selftests/net: add socket options test with IPv6 testcases

tools/testing/selftests/Makefile | 3 +-
tools/testing/selftests/lib/Makefile | 14 ++
tools/testing/selftests/lib/selftests.c | 57 +++++++++
tools/testing/selftests/lib/selftests.h | 67 ++++++++++
tools/testing/selftests/net/Makefile | 17 ++-
tools/testing/selftests/net/run_netsocktests | 10 ++
tools/testing/selftests/net/socket.c | 108 +++++++++++-----
tools/testing/selftests/net/sockopt.c | 185 +++++++++++++++++++++++++++
8 files changed, 425 insertions(+), 36 deletions(-)
create mode 100644 tools/testing/selftests/lib/Makefile
create mode 100644 tools/testing/selftests/lib/selftests.c
create mode 100644 tools/testing/selftests/lib/selftests.h
create mode 100644 tools/testing/selftests/net/sockopt.c
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at
Please read the FAQ at