| Message ID | 20240822173122.14760-1-rysulliv@redhat.com |
|---|---|
| State | Superseded |
| Headers | show |
| Series | selftests/livepatch: wait for atomic replace to occur | expand |
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-kernel_upt_28 > 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.
Changes from v2: Adds: Reported-by: CKI Project <cki-project@redhat.com> Closes: https://datawarehouse.cki-project.org/kcidb/tests/redhat:1413102084-x86_64-kernel_upt_28
Hi, JFYI, I have committed the patch into livepatching.git, branch for-6.11/selftests-fixup. Few nits below :-) On Fri 2024-08-23 09:09:26, Ryan B. Sullivan wrote: > Changes from v2: > > Adds: > Reported-by: CKI Project <cki-project@redhat.com> > Closes: https://datawarehouse.cki-project.org/kcidb/tests/redhat:1413102084-x86_64-kernel_upt_28 The "changelog" should have been added after the "---" line. It allows to send the patch as the main part of the mail (no attachment). The lines below the "---" gets automatically removed when the "mail" gets applied using "git am". Best Regards, Petr
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
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. Signed-off-by: Ryan Sullivan <rysulliv@redhat.com> --- tools/testing/selftests/livepatch/test-livepatch.sh | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-)