mbox series

[v3,0/2] add KSM performance tests

Message ID cover.1629386192.git.zhansayabagdaulet@gmail.com
Headers show
Series add KSM performance tests | expand

Message

Zhansaya Bagdauletkyzy Aug. 19, 2021, 3:33 p.m. UTC 
Extend KSM self tests with a performance benchmark. These tests are not
part of regular regression testing, as they are mainly intended to be
used by developers making changes to the memory management subsystem.
This patchset is a respin of the previous series:
v2: https://lkml.org/lkml/2021/8/6/422
v1: https://lkml.org/lkml/2021/8/1/130

Zhansaya Bagdauletkyzy (2):
  selftests: vm: add KSM merging time test
  selftests: vm: add COW time test for KSM pages

v2 -> v3:
 - address COW test review comments

v1 -> v2:
 - replace MB with MiB
 - address COW test review comments

 tools/testing/selftests/vm/ksm_tests.c | 154 ++++++++++++++++++++++++-
 1 file changed, 150 insertions(+), 4 deletions(-)

Comments

Pasha Tatashin Aug. 20, 2021, 5:19 p.m. UTC | #1 
On Thu, Aug 19, 2021 at 12:29 PM Tyler Hicks
<tyhicks@linux.microsoft.com> wrote:
>

> On 2021-08-19 21:33:41, Zhansaya Bagdauletkyzy wrote:

> > Add ksm_merge_time() function to determine speed and time needed for

> > merging. The total spent time is shown in seconds while speed is

> > in MiB/s. User must specify the size of duplicated memory area (in MiB)

> > before running the test.

> >

> > The test is run as follows: ./ksm_tests -P -s 100

> > The output:

> >       Total size:    100 MiB

> >       Total time:    0.201106786 s

> >       Average speed:  497.248 MiB/s

> >

> > Signed-off-by: Zhansaya Bagdauletkyzy <zhansayabagdaulet@gmail.com>

> > ---

> > v2 -> v3:

> >  - no changes

>

> There were no changes to this patch since v2 so my previous review still

> stands:

>

>  Reviewed-by: Tyler Hicks <tyhicks@linux.microsoft.com>


Thank you for addressing my comments in v2.

Reviewed-by: Pavel Tatashin <pasha.tatashin@soleen.com>