Hi, this is 2nd version of the patch. There should have been used [PATCH v2] in the Subject to make it clear in the mailbox. On Thu 2024-08-22 13:31:22, Ryan Sullivan wrote: > On some machines with a large number of CPUs there is a sizable delay > between an atomic replace occurring and when sysfs updates accordingly. > This fix uses 'loop_until' to wait for the atomic replace to unload all > previous livepatches. > I think that Joe suggested to add: Reported-by: CKI Project <cki-project@xxxxxxxxxx> Closes: https://datawarehouse.cki-project.org/kcidb/tests/redhat:1413102084-x86_64-kernel_upt_28 > Signed-off-by: Ryan Sullivan <rysulliv@xxxxxxxxxx> > --- Also it is a good practice to summarize changes between versions. In this case it would have been something like: Changes against v1: - Cleaned the commit message. > tools/testing/selftests/livepatch/test-livepatch.sh | 7 ++----- > 1 file changed, 2 insertions(+), 5 deletions(-) > > diff --git a/tools/testing/selftests/livepatch/test-livepatch.sh b/tools/testing/selftests/livepatch/test-livepatch.sh > index 65c9c058458d..bd13257bfdfe 100755 > --- a/tools/testing/selftests/livepatch/test-livepatch.sh > +++ b/tools/testing/selftests/livepatch/test-livepatch.sh > @@ -139,11 +139,8 @@ load_lp $MOD_REPLACE replace=1 > grep 'live patched' /proc/cmdline > /dev/kmsg > grep 'live patched' /proc/meminfo > /dev/kmsg > > -mods=(/sys/kernel/livepatch/*) > -nmods=${#mods[@]} > -if [ "$nmods" -ne 1 ]; then > - die "Expecting only one moduled listed, found $nmods" > -fi > +loop_until 'mods=(/sys/kernel/livepatch/*); nmods=${#mods[@]}; [[ "$nmods" -eq 1 ]]' || > + die "Expecting only one moduled listed, found $nmods" > > # These modules were disabled by the atomic replace > for mod in $MOD_LIVEPATCH3 $MOD_LIVEPATCH2 $MOD_LIVEPATCH1; do Otherwise, it looks good to me. With the added references: Reviewed-by: Petr Mladek <pmladek@xxxxxxxx> Best Regards, Petr PS: No need to resend the patch. I would add the references when committing. I am going to wait few more days before committing.
