| Message ID | cover.1631737524.git.skhan@linuxfoundation.org (mailing list archive) |
|---|---|
| Headers | show |
| Series | selftests: kvm: fscanf warn fixes and cleanups | expand |
On 15/09/21 23:28, Shuah Khan wrote: > This patch series fixes fscanf() ignoring return value warnings. > Consolidates get_run_delay() duplicate defines moving it to > common library. > > Shuah Khan (4): > selftests:kvm: fix get_warnings_count() ignoring fscanf() return warn > selftests:kvm: fix get_trans_hugepagesz() ignoring fscanf() return > warn > selftests: kvm: move get_run_delay() into lib/test_util > selftests: kvm: fix get_run_delay() ignoring fscanf() return warn > > .../testing/selftests/kvm/include/test_util.h | 3 +++ > tools/testing/selftests/kvm/lib/test_util.c | 22 ++++++++++++++++++- > tools/testing/selftests/kvm/steal_time.c | 16 -------------- > .../selftests/kvm/x86_64/mmio_warning_test.c | 3 ++- > .../selftests/kvm/x86_64/xen_shinfo_test.c | 15 ------------- > 5 files changed, 26 insertions(+), 33 deletions(-) > Acked-by: Paolo Bonzini <pbonzini@redhat.com> Thanks Shuah! Paolo
On 9/15/21 4:34 PM, Paolo Bonzini wrote: > On 15/09/21 23:28, Shuah Khan wrote: >> This patch series fixes fscanf() ignoring return value warnings. >> Consolidates get_run_delay() duplicate defines moving it to >> common library. >> >> Shuah Khan (4): >> selftests:kvm: fix get_warnings_count() ignoring fscanf() return warn >> selftests:kvm: fix get_trans_hugepagesz() ignoring fscanf() return >> warn >> selftests: kvm: move get_run_delay() into lib/test_util >> selftests: kvm: fix get_run_delay() ignoring fscanf() return warn >> >> .../testing/selftests/kvm/include/test_util.h | 3 +++ >> tools/testing/selftests/kvm/lib/test_util.c | 22 ++++++++++++++++++- >> tools/testing/selftests/kvm/steal_time.c | 16 -------------- >> .../selftests/kvm/x86_64/mmio_warning_test.c | 3 ++- >> .../selftests/kvm/x86_64/xen_shinfo_test.c | 15 ------------- >> 5 files changed, 26 insertions(+), 33 deletions(-) >> > > Acked-by: Paolo Bonzini <pbonzini@redhat.com> > > Thanks Shuah! > Thank you. I can take these through linux-kselftest - let me know if that causes issues for kvm tree. thanks, -- Shuah
On 16/09/21 02:02, Shuah Khan wrote: > On 9/15/21 4:34 PM, Paolo Bonzini wrote: >> On 15/09/21 23:28, Shuah Khan wrote: >>> This patch series fixes fscanf() ignoring return value warnings. >>> Consolidates get_run_delay() duplicate defines moving it to >>> common library. >>> >>> Shuah Khan (4): >>> selftests:kvm: fix get_warnings_count() ignoring fscanf() return warn >>> selftests:kvm: fix get_trans_hugepagesz() ignoring fscanf() return >>> warn >>> selftests: kvm: move get_run_delay() into lib/test_util >>> selftests: kvm: fix get_run_delay() ignoring fscanf() return warn >>> >>> .../testing/selftests/kvm/include/test_util.h | 3 +++ >>> tools/testing/selftests/kvm/lib/test_util.c | 22 ++++++++++++++++++- >>> tools/testing/selftests/kvm/steal_time.c | 16 -------------- >>> .../selftests/kvm/x86_64/mmio_warning_test.c | 3 ++- >>> .../selftests/kvm/x86_64/xen_shinfo_test.c | 15 ------------- >>> 5 files changed, 26 insertions(+), 33 deletions(-) >>> >> >> Acked-by: Paolo Bonzini <pbonzini@redhat.com> >> >> Thanks Shuah! >> > > Thank you. I can take these through linux-kselftest - let me know > if that causes issues for kvm tree. Go ahead if it's for 5.15-rc, I don't have any selftests patches pending. Paolo
On 9/15/21 11:06 PM, Paolo Bonzini wrote: > On 16/09/21 02:02, Shuah Khan wrote: >> On 9/15/21 4:34 PM, Paolo Bonzini wrote: >>> On 15/09/21 23:28, Shuah Khan wrote: >>>> This patch series fixes fscanf() ignoring return value warnings. >>>> Consolidates get_run_delay() duplicate defines moving it to >>>> common library. >>>> >>>> Shuah Khan (4): >>>> selftests:kvm: fix get_warnings_count() ignoring fscanf() return warn >>>> selftests:kvm: fix get_trans_hugepagesz() ignoring fscanf() return >>>> warn >>>> selftests: kvm: move get_run_delay() into lib/test_util >>>> selftests: kvm: fix get_run_delay() ignoring fscanf() return warn >>>> >>>> .../testing/selftests/kvm/include/test_util.h | 3 +++ >>>> tools/testing/selftests/kvm/lib/test_util.c | 22 ++++++++++++++++++- >>>> tools/testing/selftests/kvm/steal_time.c | 16 -------------- >>>> .../selftests/kvm/x86_64/mmio_warning_test.c | 3 ++- >>>> .../selftests/kvm/x86_64/xen_shinfo_test.c | 15 ------------- >>>> 5 files changed, 26 insertions(+), 33 deletions(-) >>>> >>> >>> Acked-by: Paolo Bonzini <pbonzini@redhat.com> >>> >>> Thanks Shuah! >>> >> >> Thank you. I can take these through linux-kselftest - let me know >> if that causes issues for kvm tree. > > Go ahead if it's for 5.15-rc, I don't have any selftests patches pending. > Thanks Paolo. I will apply these for rc3. thanks, -- Shuah
This patch series fixes fscanf() ignoring return value warnings. Consolidates get_run_delay() duplicate defines moving it to common library. Shuah Khan (4): selftests:kvm: fix get_warnings_count() ignoring fscanf() return warn selftests:kvm: fix get_trans_hugepagesz() ignoring fscanf() return warn selftests: kvm: move get_run_delay() into lib/test_util selftests: kvm: fix get_run_delay() ignoring fscanf() return warn .../testing/selftests/kvm/include/test_util.h | 3 +++ tools/testing/selftests/kvm/lib/test_util.c | 22 ++++++++++++++++++- tools/testing/selftests/kvm/steal_time.c | 16 -------------- .../selftests/kvm/x86_64/mmio_warning_test.c | 3 ++- .../selftests/kvm/x86_64/xen_shinfo_test.c | 15 ------------- 5 files changed, 26 insertions(+), 33 deletions(-)