linux-kernel - Re: [RFC PATCH v1 0/9] Tests for sync infrastructure

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: <56E04B8C.7000504@osg.samsung.com>
Date:	Wed, 9 Mar 2016 09:13:00 -0700
From:	Shuah Khan <shuahkh@....samsung.com>
To:	Emilio López <emilio.lopez@...labora.co.uk>
Cc:	devel@...verdev.osuosl.org,
	Gustavo Padovan <gustavo.padovan@...labora.co.uk>,
	Riley Andrews <riandrews@...roid.com>,
	Daniel Vetter <daniel.vetter@...ll.ch>,
	John Harrison <John.C.Harrison@...el.com>,
	linux-kernel@...r.kernel.org, dri-devel@...ts.freedesktop.org,
	Greg Hackmann <ghackmann@...gle.com>,
	Rob Clark <robdclark@...il.com>,
	linux-kselftest@...r.kernel.org,
	Maarten Lankhorst <maarten.lankhorst@...ux.intel.com>,
	Daniel Stone <daniels@...labora.com>,
	Arve Hjønnevåg <arve@...roid.com>,
	Shuah Khan <shuahkh@....samsung.com>
Subject: Re: [RFC PATCH v1 0/9] Tests for sync infrastructure

On 03/09/2016 08:28 AM, Emilio López wrote:
> Hello everyone,
> 
> This is a series of tests to exercise the sync kernel infrastructure. It is
> meant to be a test suite for the work Gustavo has been doing to destage it,
> see [0] for his latest series to date.
> 
> These tests were originally part of a battery of tests shipping with
> Android's libsync that were rewritten to use the new userspace interfaces.
> 
> As usual, all comments are welcome.
> 
> Cheers!
> Emilio
> 
> [0] https://lists.freedesktop.org/archives/dri-devel/2016-March/102204.html

Good to see this suite added to Kselftest. Thanks for
being thorough and including .gitignore.

Are there any destructive and/or longer time tests
in this suite. If so could you please make them
optional to run. Please see selftests/timers as an
example for excluding.

More comments on individual patches to follow.

thanks,
-- Shuah

> 
> 
> Emilio López (9):
>   selftest: sync: basic tests for sw_sync framework
>   selftest: sync: fence tests for sw_sync framework
>   selftest: sync: merge tests for sw_sync framework
>   selftest: sync: wait tests for sw_sync framework
>   selftest: sync: destruction tests for sw_sync framework
>   selftest: sync: stress test for parallelism
>   selftest: sync: stress consumer/producer test
>   selftest: sync: stress test for merges
>   selftest: sync: disable tests that rely on not yet defined behaviour
> 
>  tools/testing/selftests/Makefile                   |   1 +
>  tools/testing/selftests/sync/.gitignore            |   1 +
>  tools/testing/selftests/sync/Makefile              |  28 +++
>  tools/testing/selftests/sync/sw_sync.h             |  46 +++++
>  tools/testing/selftests/sync/sync.c                | 203 +++++++++++++++++++++
>  tools/testing/selftests/sync/sync.h                | 119 ++++++++++++
>  tools/testing/selftests/sync/sync_alloc.c          |  74 ++++++++
>  tools/testing/selftests/sync/sync_destroyed.c      |  90 +++++++++
>  tools/testing/selftests/sync/sync_fence.c          | 134 ++++++++++++++
>  tools/testing/selftests/sync/sync_merge.c          |  60 ++++++
>  .../testing/selftests/sync/sync_stress_consumer.c  | 185 +++++++++++++++++++
>  tools/testing/selftests/sync/sync_stress_merge.c   | 115 ++++++++++++
>  .../selftests/sync/sync_stress_parallelism.c       | 111 +++++++++++
>  tools/testing/selftests/sync/sync_test.c           |  87 +++++++++
>  tools/testing/selftests/sync/sync_wait.c           |  95 ++++++++++
>  tools/testing/selftests/sync/synctest.h            |  69 +++++++
>  16 files changed, 1418 insertions(+)
>  create mode 100644 tools/testing/selftests/sync/.gitignore
>  create mode 100644 tools/testing/selftests/sync/Makefile
>  create mode 100644 tools/testing/selftests/sync/sw_sync.h
>  create mode 100644 tools/testing/selftests/sync/sync.c
>  create mode 100644 tools/testing/selftests/sync/sync.h
>  create mode 100644 tools/testing/selftests/sync/sync_alloc.c
>  create mode 100644 tools/testing/selftests/sync/sync_destroyed.c
>  create mode 100644 tools/testing/selftests/sync/sync_fence.c
>  create mode 100644 tools/testing/selftests/sync/sync_merge.c
>  create mode 100644 tools/testing/selftests/sync/sync_stress_consumer.c
>  create mode 100644 tools/testing/selftests/sync/sync_stress_merge.c
>  create mode 100644 tools/testing/selftests/sync/sync_stress_parallelism.c
>  create mode 100644 tools/testing/selftests/sync/sync_test.c
>  create mode 100644 tools/testing/selftests/sync/sync_wait.c
>  create mode 100644 tools/testing/selftests/sync/synctest.h
> 


-- 
Shuah Khan
Sr. Linux Kernel Developer
Open Source Innovation Group
Samsung Research America (Silicon Valley)
shuahkh@....samsung.com | (970) 217-8978