Re: [PATCH v3 1/2] selftests: vm: add KSM merging time test

From: Pavel Tatashin
Date: Fri Aug 20 2021 - 13:20:03 EST


On Thu, Aug 19, 2021 at 12:29 PM Tyler Hicks
<tyhicks@xxxxxxxxxxxxxxxxxxx> 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@xxxxxxxxx>
> > ---
> > v2 -> v3:
> > - no changes
>
> There were no changes to this patch since v2 so my previous review still
> stands:
>
> Reviewed-by: Tyler Hicks <tyhicks@xxxxxxxxxxxxxxxxxxx>

Thank you for addressing my comments in v2.

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