Re: [PATCH v7 08/14] KVM: Rename mmu_notifier_*

2023-05-24 Thread Kautuk Consul
On 2023-05-24 22:33:36, Peter Zijlstra wrote: > On Wed, May 24, 2023 at 01:16:03PM -0700, Sean Christopherson wrote: > > > Atomics aren't memory barriers on all architectures, e.g. see the various > > definitions of smp_mb__after_atomic(). > > > > Even if atomic operations did provide barriers,

Re: [PATCH v7 08/14] KVM: Rename mmu_notifier_*

2023-05-24 Thread Kautuk Consul
On 2023-05-23 07:19:43, Sean Christopherson wrote: > On Tue, May 23, 2023, Kautuk Consul wrote: > > On 2022-07-06 16:20:10, Chao Peng wrote: > > > diff --git a/include/linux/kvm_host.h b/include/linux/kvm_host.h > > > index e9153b54e2a4..c262ebb168a7 100644 > >

Re: [PATCH v7 08/14] KVM: Rename mmu_notifier_*

2023-05-23 Thread Kautuk Consul
On 2022-07-06 16:20:10, Chao Peng wrote: > The sync mechanism between mmu_notifier and page fault handler employs > fields mmu_notifier_seq/count and mmu_notifier_range_start/end. For the > to be added private memory, there is the same mechanism needed but not > rely on mmu_notifier (It uses new

Re: [PATCH v5] tests/avocado/tuxrun_baselines.py: improve code coverage for ppc64

2023-04-24 Thread Kautuk Consul
On 2023-04-24 09:53:23, Alex Bennée wrote: > > Kautuk Consul writes: > > > Commit c0c8687ef0fd990db8db1655a8a6c5a5e35dd4bb disabled the > > boot_linux.py test-case due to which the code coverage for ppc > > decreased by around 2%. As per the discussion on > > h

Re: [PATCH v4 0/2] Improve code coverage for ppc64

2023-04-23 Thread Kautuk Consul
On 2023-04-21 14:29:38, Alex Bennée wrote: > > Kautuk Consul writes: > > > Commit c0c8687ef0fd990db8db1655a8a6c5a5e35dd4bb disabled the > > boot_linux.py test-case due to which the code coverage for ppc > > decreased by around 2%. As per the discussion on > > h

[PATCH v5] tests/avocado/tuxrun_baselines.py: improve code coverage for ppc64

2023-04-23 Thread Kautuk Consul
..: 20.7% (7640 of 36993 functions) branches...: 8.4% (20223 of 240611 branches) Rebased on Alex Benee's testing/next branch: https://gitlab.com/stsquad/qemu/-/tree/testing/next Signed-off-by: Kautuk Consul Reported-by: Alex Bennée Reviewed-by: Harsh Prateek Bora Reviewed-by: Philippe Mathieu

Re: [PATCH v4 2/2] tests/avocado/tuxrun_baselines.py: improve code coverage for ppc64

2023-04-21 Thread Kautuk Consul
Adding Harsh Prateek Bora . On 2023-04-20 23:23:22, Kautuk Consul wrote: > Commit c0c8687ef0fd990db8db1655a8a6c5a5e35dd4bb disabled the > boot_linux.py test-case due to which the code coverage for ppc > decreased by around 2%. As per the discussion on > https://lore.kernel.or

Re: [PATCH v4 1/2] avocado_qemu/__init__.py: factor out the qemu-img finding

2023-04-21 Thread Kautuk Consul
Adding Harsh Prateel Bora. On 2023-04-20 23:23:21, Kautuk Consul wrote: > Factor out the code that finds the qemu-img binary in the > QemuSystemTest class and create a new get_qemu_img() function > with it. This function will get called also from the new code > in tuxrun_baselines.py

Re: [PATCH v4 0/2] Improve code coverage for ppc64

2023-04-21 Thread Kautuk Consul
Adding Harsh Prateek Bora. On 2023-04-20 23:23:20, Kautuk Consul wrote: > Commit c0c8687ef0fd990db8db1655a8a6c5a5e35dd4bb disabled the > boot_linux.py test-case due to which the code coverage for ppc > decreased by around 2%. As per the discussion on > https://lore.kernel.or

Re: [PATCH v3 2/2] tests/avocado/tuxrun_baselines.py: improve code coverage for ppc64

2023-04-20 Thread Kautuk Consul
On 2023-04-21 09:23:52, Harsh Prateek Bora wrote: > > > On 4/21/23 09:12, Kautuk Consul wrote: > > Hi, > > > > On 2023-04-20 19:20:40, Harsh Prateek Bora wrote: > > > Since we are optimising code a lot, one suggestion below: > > > > > >

[PATCH v4 1/2] avocado_qemu/__init__.py: factor out the qemu-img finding

2023-04-20 Thread Kautuk Consul
Factor out the code that finds the qemu-img binary in the QemuSystemTest class and create a new get_qemu_img() function with it. This function will get called also from the new code in tuxrun_baselines.py avocado test-case. Signed-off-by: Kautuk Consul --- tests/avocado/avocado_qemu/__init__.py

[PATCH v4 2/2] tests/avocado/tuxrun_baselines.py: improve code coverage for ppc64

2023-04-20 Thread Kautuk Consul
..: 20.7% (7640 of 36993 functions) branches...: 8.4% (20223 of 240611 branches) Signed-off-by: Kautuk Consul Reported-by: Alex Bennée --- tests/avocado/tuxrun_baselines.py | 68 +-- 1 file changed, 64 insertions(+), 4 deletions(-) diff --git a/tests/avocado

[PATCH v4 0/2] Improve code coverage for ppc64

2023-04-20 Thread Kautuk Consul
and call that from the ppc64 and ppc64le test case routines. Kautuk Consul (2): avocado_qemu/__init__.py: factor out the qemu-img finding tests/avocado/tuxrun_baselines.py: improve code coverage for ppc64 tests/avocado/avocado_qemu/__init__.py | 27 +- tests/avocado/tuxrun_baselines.py

Re: [PATCH v3 0/2] Improve avocado code coverage for ppc64

2023-04-20 Thread Kautuk Consul
On 2023-04-20 16:01:45, Alex Bennée wrote: > > Kautuk Consul writes: > > > Commit c0c8687ef0fd990db8db1655a8a6c5a5e35dd4bb disabled the > > boot_linux.py test-case due to which the code coverage for > > powerpc decreased by 2%. This patchset aims to make u

Re: [PATCH v3 2/2] tests/avocado/tuxrun_baselines.py: improve code coverage for ppc64

2023-04-20 Thread Kautuk Consul
Hi, On 2023-04-20 19:20:40, Harsh Prateek Bora wrote: > Since we are optimising code a lot, one suggestion below: > > > On Thu, Apr 20, 2023 at 6:23 PM Kautuk Consul > wrote: > > > Commit c0c8687ef0fd990db8db1655a8a6c5a5e35dd4bb disabled the > > boot_linux.py t

Re: [PATCH v2] tests/avocado/tuxrun_baselines.py: improve code coverage for ppc64

2023-04-20 Thread Kautuk Consul
On 2023-04-20 10:12:37, Thomas Huth wrote: > On 20/04/2023 09.57, Philippe Mathieu-Daud޸ wrote: > > Hi Kautuk, > > > > On 19/4/23 11:22, Kautuk Consul wrote: > > > Commit c0c8687ef0fd990db8db1655a8a6c5a5e35dd4bb disabled the > > > boot_linux.py test-case

[PATCH v3 2/2] tests/avocado/tuxrun_baselines.py: improve code coverage for ppc64

2023-04-20 Thread Kautuk Consul
..: 20.7% (7640 of 36993 functions) branches...: 8.4% (20223 of 240611 branches) Signed-off-by: Kautuk Consul Reported-by: Alex Bennée --- tests/avocado/tuxrun_baselines.py | 120 +- 1 file changed, 116 insertions(+), 4 deletions(-) diff --git a/tests/avocado

[PATCH v3 0/2] Improve avocado code coverage for ppc64

2023-04-20 Thread Kautuk Consul
that creates the temporary qcow2 image that the tuxrun_baselines.py is now using. - Factored out code in avocado_qemu/__init__.py to create the get_qemu_img() function that will now get called from tuxrun_baselines.py. Kautuk Consul (2): avocado_qemu/__init__.py: factor out the qemu-img

[PATCH v3 1/2] avocado_qemu/__init__.py: factor out the qemu-img finding

2023-04-20 Thread Kautuk Consul
Factor out the code that finds the qemu-img binary in the QemuSystemTest class and create a new get_qemu_img() function with it. This function will get called also from the new code in tuxrun_baselines.py avocado test-case. Signed-off-by: Kautuk Consul --- tests/avocado/avocado_qemu/__init__.py

[PATCH v2] tests/avocado/tuxrun_baselines.py: improve code coverage for ppc64

2023-04-19 Thread Kautuk Consul
..: 20.7% (7640 of 36993 functions) branches...: 8.4% (20223 of 240611 branches) Signed-off-by: Kautuk Consul Reported-by: Alex Bennée --- tests/avocado/tuxrun_baselines.py | 124 +- 1 file changed, 122 insertions(+), 2 deletions(-) diff --git a/tests/avocado

Re: [PATCH] tests/avocado/tuxrun_baselines.py: improve code coverage for ppc64

2023-04-19 Thread Kautuk Consul
On 2023-04-18 09:07:53, Thomas Huth wrote: > On 18/04/2023 07.53, Kautuk Consul wrote: > > Commit c0c8687ef0fd990db8db1655a8a6c5a5e35dd4bb disabled the > > boot_linux.py test-case due to which the code coverage for ppc > > decreased by around 2%. As per the di

[PATCH] tests/avocado/tuxrun_baselines.py: improve code coverage for ppc64

2023-04-17 Thread Kautuk Consul
..: 20.7% (7640 of 36993 functions) branches...: 8.4% (20223 of 240611 branches) Signed-off-by: Kautuk Consul Reported-by: Alex Bennée --- tests/avocado/tuxrun_baselines.py | 98 ++- 1 file changed, 96 insertions(+), 2 deletions(-) diff --git a/tests/avocado

Re: [PATCH 1/2] tests/requirements.txt: bump up avocado-framework version to 101.0

2023-03-31 Thread Kautuk Consul
On 2023-03-31 11:19:18, Alex Bennée wrote: > > Kautuk Consul writes: > > > Hi, > > On 2023-03-27 07:50:29, Kautuk Consul wrote: > >> Avocado version 101.0 has a fix to re-compute the checksum > >> of an asset file if the algorithm used in the *-CHECKSUM &g

Re: [PATCH 1/2] tests/requirements.txt: bump up avocado-framework version to 101.0

2023-03-30 Thread Kautuk Consul
Hi, On 2023-03-27 07:50:29, Kautuk Consul wrote: > Avocado version 101.0 has a fix to re-compute the checksum > of an asset file if the algorithm used in the *-CHECKSUM > file isn't the same as the one being passed to it by the > avocado user (i.e. the avocado_qemu python module). >

Re: [PATCH 2/2] tests/avocado/boot_linux.py: re-enable test-case for ppc64

2023-03-28 Thread Kautuk Consul
On 2023-03-27 17:07:30, Alex Bennée wrote: > > Kautuk Consul writes: > > > Fixes c0c8687ef0("tests/avocado: disable BootLinuxPPC64 test in CI"). > > > > Commit c0c8687ef0fd990db8db1655a8a6c5a5e35dd4bb disabled the test-case > > for PPC64. On i

[PATCH 2/2] tests/avocado/boot_linux.py: re-enable test-case for ppc64

2023-03-27 Thread Kautuk Consul
test-case timeout. Re-enable this test-case by setting the timeout to 360 seconds just before launching the downloaded VM image. Signed-off-by: Kautuk Consul Reported-by: Alex Bennée Tested-by: Hariharan T S hariharan...@linux.vnet.ibm.com --- tests/avocado/boot_linux.py | 6 +- 1 file

[PATCH 0/2] Re-enabling tests/avocado/boot_linux.py for PPC64

2023-03-27 Thread Kautuk Consul
execution timeout in boot_linux.py. Kautuk Consul (2): tests/requirements.txt: bump up avocado-framework version to 101.0 tests/avocado/boot_linux.py: re-enable test-case for ppc64 tests/avocado/boot_linux.py | 6 +- tests/requirements.txt | 2 +- 2 files changed, 6

[PATCH 1/2] tests/requirements.txt: bump up avocado-framework version to 101.0

2023-03-27 Thread Kautuk Consul
if the checksum wouldn't match the earlier checksum (calculated by a different algorithm), the avocado code would start downloading a fresh image from the internet URL thus making the test-cases take longer to execute. Bump up the avocado-framework version to 101.0. Signed-off-by: Kautuk Consul Tested