On 2020/6/23 17:42, Paolo Bonzini wrote:
On 27/04/20 06:35, Tianjia Zhang wrote:
In the current kvm version, 'kvm_run' has been included in the 'kvm_vcpu'
structure. For historical reasons, many kvm-related function parameters
retain the 'kvm_run' and 'kv
aining redundant parameters.
Signed-off-by: Tianjia Zhang
Reviewed-by: Huacai Chen
---
arch/mips/include/asm/kvm_host.h | 28 +---
arch/mips/kvm/emulate.c | 59 ++--
arch/mips/kvm/mips.c | 11 ++-
arch/mips/kvm/trap_emul.c| 114 +++
aining redundant parameters.
Signed-off-by: Tianjia Zhang
Reviewed-by: Huacai Chen
---
arch/mips/include/asm/kvm_host.h | 4 ++--
arch/mips/kvm/entry.c| 21 -
arch/mips/kvm/mips.c | 3 ++-
arch/mips/kvm/trap_emul.c| 2 +-
arch/mips/kvm/vz.
aining redundant parameters.
Signed-off-by: Tianjia Zhang
---
arch/powerpc/include/asm/kvm_ppc.h| 2 +-
arch/powerpc/kvm/book3s_interrupts.S | 22 ++
arch/powerpc/kvm/book3s_pr.c | 9 -
arch/powerpc/kvm/booke.c | 9 -
ar
sting `vcpu->run` in the function body unchanged.
v2 change:
s390 retains the original variable name and minimizes modification.
Tianjia Zhang (5):
KVM: s390: clean up redundant 'kvm_run' parameters
KVM: arm64: clean up redundant 'kvm_run' parameters
KVM: PPC: clean u
aining redundant parameters.
Signed-off-by: Tianjia Zhang
Reviewed-by: Vitaly Kuznetsov
---
arch/s390/kvm/kvm-s390.c | 23 +++
1 file changed, 15 insertions(+), 8 deletions(-)
diff --git a/arch/s390/kvm/kvm-s390.c b/arch/s390/kvm/kvm-s390.c
index d47c19718615..f5f96dc337
aining redundant parameters.
Signed-off-by: Tianjia Zhang
Reviewed-by: Vitaly Kuznetsov
---
arch/arm64/include/asm/kvm_coproc.h | 12 +-
arch/arm64/include/asm/kvm_host.h | 11 -
arch/arm64/include/asm/kvm_mmu.h| 2 +-
arch/arm64/kvm/arm.c| 6 ++---
On 2020/6/23 23:31, Christian Borntraeger wrote:
On 23.06.20 15:14, Tianjia Zhang wrote:
In the current kvm version, 'kvm_run' has been included in the 'kvm_vcpu'
structure. For historical reasons, many kvm-related function parameters
retain the 'kvm_run' an
Hi Paolo,
Any opinion on this series patches? Can I help with this patchset ?
Thanks and best,
Tianjia
On 2020/6/23 21:14, Tianjia Zhang wrote:
In the current kvm version, 'kvm_run' has been included in the 'kvm_vcpu'
structure. For historical reasons, many kvm-related
On 2020/5/26 13:59, Paul Mackerras wrote:
On Mon, Apr 27, 2020 at 12:35:12PM +0800, Tianjia Zhang wrote:
In the current kvm version, 'kvm_run' has been included in the 'kvm_vcpu'
structure. For historical reasons, many kvm-related function parameters
retain the &#x
kvm_arch_vcpu_ioctl_run() is only called in the file kvm_main.c,
where vcpu->run is the kvm_run parameter, so it has been replaced.
Signed-off-by: Tianjia Zhang
---
arch/x86/kvm/x86.c | 8
virt/kvm/arm/arm.c | 2 +-
2 files changed, 5 insertions(+), 5 deletions(-)
diff --git a/a
On 2020/4/13 16:56, Marc Zyngier wrote:
Tianjia,
On 2020-04-13 04:45, Tianjia Zhang wrote:
kvm_arch_vcpu_ioctl_run() is only called in the file kvm_main.c,
where vcpu->run is the kvm_run parameter, so it has been replaced.
Signed-off-by: Tianjia Zhang
---
arch/x86/kvm/x86.c
On 2020/4/14 22:26, Vitaly Kuznetsov wrote:
Tianjia Zhang writes:
kvm_arch_vcpu_ioctl_run() is only called in the file kvm_main.c,
where vcpu->run is the kvm_run parameter, so it has been replaced.
Signed-off-by: Tianjia Zhang
---
arch/x86/kvm/x86.c | 8
virt/kvm/arm/ar
On 2020/4/16 16:50, Cornelia Huck wrote:
On Thu, 16 Apr 2020 16:45:33 +0800
Tianjia Zhang wrote:
On 2020/4/16 16:28, Marc Zyngier wrote:
On 2020-04-16 08:03, Vitaly Kuznetsov wrote:
Tianjia Zhang writes:
In earlier versions of kvm, 'kvm_run' is an independent structure
On 2020/4/16 16:28, Marc Zyngier wrote:
On 2020-04-16 08:03, Vitaly Kuznetsov wrote:
Tianjia Zhang writes:
In earlier versions of kvm, 'kvm_run' is an independent structure
and is not included in the vcpu structure. At present, 'kvm_run'
is already included in the vc
On 2020/4/15 22:53, Paolo Bonzini wrote:
On 15/04/20 11:07, Vitaly Kuznetsov wrote:
In case this is no longer needed I'd suggest we drop 'kvm_run' parameter
and extract it from 'struct kvm_vcpu' when needed. This looks like a
natural add-on to your cleanup patch.
I agree, though I think it
the extra
'kvm_run' parameter, and extract it from the 'kvm_vcpu' structure
if necessary.
Signed-off-by: Tianjia Zhang
---
v2 change:
remove 'kvm_run' parameter and extract it from 'kvm_vcpu'
arch/mips/kvm/mips.c | 3 ++-
arch/powerpc/kvm/power
undant structure
pointer of 'kvm_run' in 'vcpu_arch', which has also been cleaned
separately.
Thanks and best.
Tianjia Zhang (7):
KVM: s390: clean up redundant 'kvm_run' parameters
KVM: arm64: clean up redundant 'kvm_run' parameters
KVM: PPC: Remove redundant
of these remaining redundant parameters.
Signed-off-by: Tianjia Zhang
---
arch/arm64/include/asm/kvm_coproc.h | 12 +-
arch/arm64/include/asm/kvm_host.h | 11 -
arch/arm64/include/asm/kvm_mmu.h| 2 +-
arch/arm64/kvm/handle_exit.c| 36 ++---
of these remaining redundant parameters.
Signed-off-by: Tianjia Zhang
---
arch/s390/kvm/kvm-s390.c | 127 +--
1 file changed, 67 insertions(+), 60 deletions(-)
diff --git a/arch/s390/kvm/kvm-s390.c b/arch/s390/kvm/kvm-s390.c
index 443af3ead739..cf420d013ba3 100
of these remaining redundant parameters.
Signed-off-by: Tianjia Zhang
---
arch/powerpc/include/asm/kvm_book3s.h| 16 +++---
arch/powerpc/include/asm/kvm_ppc.h | 27 +
arch/powerpc/kvm/book3s.c| 4 +-
arch/powerpc/kvm/book3s.h| 2 +-
arch/powerpc/kvm
of these remaining redundant parameters.
Signed-off-by: Tianjia Zhang
---
arch/powerpc/include/asm/kvm_ppc.h| 2 +-
arch/powerpc/kvm/book3s_interrupts.S | 17 -
arch/powerpc/kvm/book3s_pr.c | 9 -
arch/powerpc/kvm/booke.c | 9 -
ar
of these remaining redundant parameters.
Signed-off-by: Tianjia Zhang
---
arch/mips/include/asm/kvm_host.h | 28 +---
arch/mips/kvm/emulate.c | 59 ++--
arch/mips/kvm/mips.c | 11 ++-
arch/mips/kvm/trap_emul.c| 114 ++-
The 'kvm_run' field already exists in the 'vcpu' structure, which
is the same structure as the 'kvm_run' in the 'vcpu_arch' and
should be deleted.
Signed-off-by: Tianjia Zhang
---
arch/powerpc/include/asm/kvm_host.h | 1 -
arch/powerpc/kvm/book3s_
reason is that the 'pagemap.h' header file is not included.
Signed-off-by: Tianjia Zhang
---
arch/x86/kvm/svm/sev.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/arch/x86/kvm/svm/sev.c b/arch/x86/kvm/svm/sev.c
index 0e3fc311d7da..3ef99e87c1db 100644
--- a/arch/x86/kvm/svm/sev.c
+++
of these remaining redundant parameters.
Signed-off-by: Tianjia Zhang
---
arch/mips/include/asm/kvm_host.h | 4 ++--
arch/mips/kvm/entry.c| 15 +--
arch/mips/kvm/mips.c | 3 ++-
arch/mips/kvm/trap_emul.c| 2 +-
arch/mips/kvm/vz.c | 2 +
On 2020/4/19 16:24, Xiaoyao Li wrote:
On 4/19/2020 3:30 PM, Tianjia Zhang wrote:
The compiler reported the following compilation errors:
arch/x86/kvm/svm/sev.c: In function ‘sev_pin_memory’:
arch/x86/kvm/svm/sev.c:361:3: error: implicit declaration of function
‘release_pages’ [-Werror
On 2020/4/20 15:07, Christian Borntraeger wrote:
On 19.04.20 09:51, Tianjia Zhang wrote:
In the current kvm version, 'kvm_run' has been included in the 'kvm_vcpu'
structure. Earlier than historical reasons, many kvm-related function
parameters retain the 'kvm_run&#
On 2020/4/20 18:32, maobibo wrote:
On 04/19/2020 03:51 PM, Tianjia Zhang wrote:
In the current kvm version, 'kvm_run' has been included in the 'kvm_vcpu'
structure. Earlier than historical reasons, many kvm-related function
parameters retain the 'kvm_run' an
of these remaining redundant parameters.
Signed-off-by: Tianjia Zhang
---
arch/arm64/include/asm/kvm_coproc.h | 12 +-
arch/arm64/include/asm/kvm_host.h | 11 -
arch/arm64/include/asm/kvm_mmu.h| 2 +-
arch/arm64/kvm/handle_exit.c| 36 ++---
undant structure
pointer of 'kvm_run' in 'vcpu_arch', which has also been cleaned
separately.
---
v2 change:
s390 retains the original variable name and minimizes modification.
Tianjia Zhang (7):
KVM: s390: clean up redundant 'kvm_run' parameters
KVM: arm64: clean up re
of these remaining redundant parameters.
Signed-off-by: Tianjia Zhang
---
arch/s390/kvm/kvm-s390.c | 37 ++---
1 file changed, 22 insertions(+), 15 deletions(-)
diff --git a/arch/s390/kvm/kvm-s390.c b/arch/s390/kvm/kvm-s390.c
index e335a7e5ead7..d7bb2e7a07ff 100644
-
The 'kvm_run' field already exists in the 'vcpu' structure, which
is the same structure as the 'kvm_run' in the 'vcpu_arch' and
should be deleted.
Signed-off-by: Tianjia Zhang
---
arch/powerpc/include/asm/kvm_host.h | 1 -
arch/powerpc/kvm/book3s_
of these remaining redundant parameters.
Signed-off-by: Tianjia Zhang
---
arch/mips/include/asm/kvm_host.h | 28 +---
arch/mips/kvm/emulate.c | 59 ++--
arch/mips/kvm/mips.c | 11 ++-
arch/mips/kvm/trap_emul.c| 114 ++-
of these remaining redundant parameters.
Signed-off-by: Tianjia Zhang
---
arch/mips/include/asm/kvm_host.h | 4 ++--
arch/mips/kvm/entry.c| 15 +--
arch/mips/kvm/mips.c | 3 ++-
arch/mips/kvm/trap_emul.c| 2 +-
arch/mips/kvm/vz.c | 2 +
of these remaining redundant parameters.
Signed-off-by: Tianjia Zhang
---
arch/powerpc/include/asm/kvm_ppc.h| 2 +-
arch/powerpc/kvm/book3s_interrupts.S | 17 -
arch/powerpc/kvm/book3s_pr.c | 9 -
arch/powerpc/kvm/booke.c | 9 -
ar
of these remaining redundant parameters.
Signed-off-by: Tianjia Zhang
---
arch/powerpc/include/asm/kvm_book3s.h| 16 +++---
arch/powerpc/include/asm/kvm_ppc.h | 27 +
arch/powerpc/kvm/book3s.c| 4 +-
arch/powerpc/kvm/book3s.h| 2 +-
arch/powerpc/kvm
On 2020/4/22 21:45, Cornelia Huck wrote:
On Wed, 22 Apr 2020 20:58:04 +0800
Tianjia Zhang wrote:
In the current kvm version, 'kvm_run' has been included in the 'kvm_vcpu'
structure. Earlier than historical reasons, many kvm-related function
s/Earlier than/For/ ?
On 2020/4/23 18:39, Cornelia Huck wrote:
On Thu, 23 Apr 2020 11:01:43 +0800
Tianjia Zhang wrote:
On 2020/4/23 0:04, Cornelia Huck wrote:
On Wed, 22 Apr 2020 17:58:04 +0200
Christian Borntraeger wrote:
On 22.04.20 15:45, Cornelia Huck wrote:
On Wed, 22 Apr 2020 20:58:04 +0800
On 2020/4/23 19:00, Christian Borntraeger wrote:
On 23.04.20 12:58, Tianjia Zhang wrote:
On 2020/4/23 18:39, Cornelia Huck wrote:
On Thu, 23 Apr 2020 11:01:43 +0800
Tianjia Zhang wrote:
On 2020/4/23 0:04, Cornelia Huck wrote:
On Wed, 22 Apr 2020 17:58:04 +0200
Christian Borntraeger
On 2020/4/22 23:58, Christian Borntraeger wrote:
On 22.04.20 15:45, Cornelia Huck wrote:
On Wed, 22 Apr 2020 20:58:04 +0800
Tianjia Zhang wrote:
In the current kvm version, 'kvm_run' has been included in the 'kvm_vcpu'
structure. Earlier than historical reaso
On 2020/4/23 0:04, Cornelia Huck wrote:
On Wed, 22 Apr 2020 17:58:04 +0200
Christian Borntraeger wrote:
On 22.04.20 15:45, Cornelia Huck wrote:
On Wed, 22 Apr 2020 20:58:04 +0800
Tianjia Zhang wrote:
In the current kvm version, 'kvm_run' has been included in the 'kvm_
The 'kvm_run' field already exists in the 'vcpu' structure, which
is the same structure as the 'kvm_run' in the 'vcpu_arch' and
should be deleted.
Signed-off-by: Tianjia Zhang
---
arch/powerpc/include/asm/kvm_host.h | 1 -
arch/powerpc/kvm/book3s_
The 'kvm_run' field already exists in the 'vcpu' structure, which
is the same structure as the 'kvm_run' in the 'vcpu_arch' and
should be deleted.
Signed-off-by: Tianjia Zhang
---
arch/powerpc/include/asm/kvm_host.h | 1 -
arch/powerpc/kvm/book3s_
aining redundant parameters.
Signed-off-by: Tianjia Zhang
---
arch/s390/kvm/kvm-s390.c | 23 +++
1 file changed, 15 insertions(+), 8 deletions(-)
diff --git a/arch/s390/kvm/kvm-s390.c b/arch/s390/kvm/kvm-s390.c
index e335a7e5ead7..c0d94eaa00d7 100644
--- a/arch/s390/kvm/k
On 2020/4/27 11:51, Huacai Chen wrote:
Hi, Tianjia,
On Sun, Apr 26, 2020 at 8:40 PM Tianjia Zhang
wrote:
In the current kvm version, 'kvm_run' has been included in the 'kvm_vcpu'
structure. Earlier than historical reasons, many kvm-related function
parameters reta
of these remaining redundant parameters.
Signed-off-by: Tianjia Zhang
---
arch/powerpc/include/asm/kvm_book3s.h| 16 +++---
arch/powerpc/include/asm/kvm_ppc.h | 27 +
arch/powerpc/kvm/book3s.c| 4 +-
arch/powerpc/kvm/book3s.h| 2 +-
arch/powerpc/kvm
aining redundant parameters.
Signed-off-by: Tianjia Zhang
---
arch/powerpc/include/asm/kvm_book3s.h| 16 +++---
arch/powerpc/include/asm/kvm_ppc.h | 27 +
arch/powerpc/kvm/book3s.c| 4 +-
arch/powerpc/kvm/book3s.h| 2 +-
arch/powerpc/kvm/book3s_
of these remaining redundant parameters.
Signed-off-by: Tianjia Zhang
---
arch/powerpc/include/asm/kvm_ppc.h| 2 +-
arch/powerpc/kvm/book3s_interrupts.S | 17 -
arch/powerpc/kvm/book3s_pr.c | 9 -
arch/powerpc/kvm/booke.c | 9 -
ar
aining redundant parameters.
Signed-off-by: Tianjia Zhang
---
arch/powerpc/include/asm/kvm_ppc.h| 2 +-
arch/powerpc/kvm/book3s_interrupts.S | 17 -
arch/powerpc/kvm/book3s_pr.c | 9 -
arch/powerpc/kvm/booke.c | 9 -
arch/powerpc/kvm/b
undant structure
pointer of 'kvm_run' in 'vcpu_arch', which has also been cleaned
separately.
---
v3 change:
Keep the existing `vcpu->run` in the function body unchanged.
v2 change:
s390 retains the original variable name and minimizes modification.
Tianjia Zhang (7):
KVM: s3
of these remaining redundant parameters.
Signed-off-by: Tianjia Zhang
---
arch/s390/kvm/kvm-s390.c | 23 +++
1 file changed, 15 insertions(+), 8 deletions(-)
diff --git a/arch/s390/kvm/kvm-s390.c b/arch/s390/kvm/kvm-s390.c
index e335a7e5ead7..c0d94eaa00d7 100644
--- a/arch/s39
cture
pointer of 'kvm_run' in 'vcpu_arch', which has also been cleaned
separately.
---
v4 change:
mips: fixes two errors in entry.c.
v3 change:
Keep the existing `vcpu->run` in the function body unchanged.
v2 change:
s390 retains the original variable name and minimizes mo
aining redundant parameters.
Signed-off-by: Tianjia Zhang
---
arch/arm64/include/asm/kvm_coproc.h | 12 +-
arch/arm64/include/asm/kvm_host.h | 11 -
arch/arm64/include/asm/kvm_mmu.h| 2 +-
arch/arm64/kvm/handle_exit.c| 36 ++---
of these remaining redundant parameters.
Signed-off-by: Tianjia Zhang
---
arch/mips/include/asm/kvm_host.h | 4 ++--
arch/mips/kvm/entry.c| 15 +--
arch/mips/kvm/mips.c | 3 ++-
arch/mips/kvm/trap_emul.c| 2 +-
arch/mips/kvm/vz.c | 2 +
aining redundant parameters.
Signed-off-by: Tianjia Zhang
---
arch/mips/include/asm/kvm_host.h | 28 +---
arch/mips/kvm/emulate.c | 59 ++--
arch/mips/kvm/mips.c | 11 ++-
arch/mips/kvm/trap_emul.c| 114 ++-
arch/mips
of these remaining redundant parameters.
Signed-off-by: Tianjia Zhang
---
arch/arm64/include/asm/kvm_coproc.h | 12 +-
arch/arm64/include/asm/kvm_host.h | 11 -
arch/arm64/include/asm/kvm_mmu.h| 2 +-
arch/arm64/kvm/handle_exit.c| 36 ++---
aining redundant parameters.
Signed-off-by: Tianjia Zhang
---
arch/mips/include/asm/kvm_host.h | 4 ++--
arch/mips/kvm/entry.c| 21 -
arch/mips/kvm/mips.c | 3 ++-
arch/mips/kvm/trap_emul.c| 2 +-
arch/mips/kvm/vz.c | 2 +
of these remaining redundant parameters.
Signed-off-by: Tianjia Zhang
---
arch/mips/include/asm/kvm_host.h | 28 +---
arch/mips/kvm/emulate.c | 59 ++--
arch/mips/kvm/mips.c | 11 ++-
arch/mips/kvm/trap_emul.c| 114 ++-
On 2020/4/26 20:59, Thomas Huth wrote:
On 23/04/2020 13.00, Christian Borntraeger wrote:
On 23.04.20 12:58, Tianjia Zhang wrote:
On 2020/4/23 18:39, Cornelia Huck wrote:
On Thu, 23 Apr 2020 11:01:43 +0800
Tianjia Zhang wrote:
On 2020/4/23 0:04, Cornelia Huck wrote:
On Wed, 22 Apr
Paolo Bonzini, any opinion on this?
Thanks and best,
Tianjia
On 2020/4/27 12:35, Tianjia Zhang wrote:
In the current kvm version, 'kvm_run' has been included in the 'kvm_vcpu'
structure. For historical reasons, many kvm-related function parameters
retain the 'kvm_run&#
On 2020/5/5 16:39, Marc Zyngier wrote:
Hi Tianjia,
On 2020-04-27 05:35, Tianjia Zhang wrote:
In the current kvm version, 'kvm_run' has been included in the 'kvm_vcpu'
structure. For historical reasons, many kvm-related function parameters
retain the 'kvm_run' an
On 2020/5/27 12:20, Paul Mackerras wrote:
On Mon, Apr 27, 2020 at 12:35:10PM +0800, Tianjia Zhang wrote:
The 'kvm_run' field already exists in the 'vcpu' structure, which
is the same structure as the 'kvm_run' in the 'vcpu_arch' and
should be deleted.
On 2020/4/27 13:40, Huacai Chen wrote:
Reviewed-by: Huacai Chen
On Mon, Apr 27, 2020 at 12:35 PM Tianjia Zhang
wrote:
In the current kvm version, 'kvm_run' has been included in the 'kvm_vcpu'
structure. For historical reasons, many kvm-related function parameters
reta
On 2020/5/29 17:48, Paolo Bonzini wrote:
On 27/05/20 08:24, Tianjia Zhang wrote:
Hi Huacai,
These two patches(6/7 and 7/7) should be merged into the tree of the
mips architecture separately. At present, there seems to be no good way
to merge the whole architecture patchs.
For this
65 matches
Mail list logo
