[LTP] [PATCH] kvm_pagefault01: Do not write into tdp_mmu sysfile
Petr Vorel
pvorel@suse.cz
Tue May 28 11:37:52 CEST 2024
> On 28. 05. 24 0:35, Petr Vorel wrote:
> > > +
> > > + if (read_bool_sys_param(TDP_MMU_SYSFILE) > 0)
> > > + tst_res(TINFO, "tdp_mmu is enabled, beware of false negatives");
> > Wouldn't it be better to add "WARNING:" to make it more visible?
> > tst_res(TINFO, "WARNING: tdp_mmu is enabled, beware of false negatives");
> > (if you agree, I can change it before merge)
> I thought about it for a while and I see good reasons for both TINFO and
> TWARN. It shouldn't matter for our tests because tdp_mmu defaults to off on
> SLE-15SP4 and SLE-15SP5. If other reviewers agree, feel free to change it to
> TWARN.
I slightly prefer tst_res(TINFO, "WARNING: ...", than TWARN. Although false
negative is serious, still failing the test based on suspicion of false
positives is not good.
@Li, @Jan, @Metan: any opinion on it?
Also, I send a patch to add TINFO_WARN flag (yeah, ugly name) to print TINFO
but slightly more visible due the color.
https://patchwork.ozlabs.org/project/ltp/list/?series=408394&state=*
Kind regards,
Petr
More information about the ltp
mailing list