From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Subject: Re: [PATCH] KVM: Optimize kvm_arch_vcpu_ioctl_run function References: <20200413034523.110548-1-tianjia.zhang@linux.alibaba.com> <875ze2ywhy.fsf@vitty.brq.redhat.com> <87a73dxgk6.fsf@vitty.brq.redhat.com> <9e122372-249d-3d93-99ed-a670fff33936@redhat.com> From: Tianjia Zhang Message-ID: <4843f690-7071-aa4f-cc9d-d9cc2321e669@linux.alibaba.com> Date: Thu, 16 Apr 2020 10:00:46 +0800 MIME-Version: 1.0 In-Reply-To: <9e122372-249d-3d93-99ed-a670fff33936@redhat.com> Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: To: Paolo Bonzini , Vitaly Kuznetsov Cc: kvm@vger.kernel.org, linux-arm-kernel@lists.infradead.org, kvmarm@lists.cs.columbia.edu, linux-kernel@vger.kernel.org, sean.j.christopherson@intel.com, wanpengli@tencent.com, jmattson@google.com, joro@8bytes.org, tglx@linutronix.de, mingo@redhat.com, bp@alien8.de, x86@kernel.org, hpa@zytor.com, maz@kernel.org, james.morse@arm.com, julien.thierry.kdev@gmail.com, suzuki.poulose@arm.com 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 should be _instead_ of Tianjia's patch rather > than on top. > > Paolo > Thank you very much for the comments of Vitaly and Paolo, I will make a v2 patch. Thanks and best, Tianjia