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@redhat.com Closes: https://datawarehouse.cki-project.org/kcidb/tests/redhat:1413102084-x86_64-k...

Signed-off-by: Ryan Sullivan rysulliv@redhat.com

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@suse.com

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.