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

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]

Date:	Thu, 25 Apr 2013 14:36:19 +0300
From:	Alexandru Copot <alex.mihai.c@...il.com>
To:	unlisted-recipients:; (no To-header on input)
Cc:	linux-kernel@...r.kernel.org, netdev@...r.kernel.org,
	akpm@...ux-foundation.org, davem@...emloft.net, willemb@...gle.com,
	ebiederm@...ssion.com, gorcunov@...nvz.org, palewis@...be.com,
	edumazet@...gle.com, Daniel Baluta <dbaluta@...acom.com>
Subject: Re: [PATCH 0/3 RFC v2] selftests: Basic framework for tests

On Thu, Apr 25, 2013 at 2:27 PM, Daniel Borkmann <dborkman@...hat.com> wrote:
>> Signed-of-by Alexandru Copot <alex.mihai.c@...il.com>
>
>
> It's: Signed-off-by: Alexandru Copot <alex.mihai.c@...il.com>
>
> You have this wrong in all your patches.

I know, it was copy-pasted and I saw it just after sending. But for now
it's fine, these are just RFCs.

> 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.

You are right, I just thought the code was small enough to not
require a description.

Thanks!
Alex

On Thu, Apr 25, 2013 at 2:27 PM, Daniel Borkmann <dborkman@...hat.com> wrote:
> 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@...il.com>
>
>
> It's: Signed-off-by: Alexandru Copot <alex.mihai.c@...il.com>
>
> 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@...acom.com>
>>
>> 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@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/