From mboxrd@z Thu Jan 1 00:00:00 1970 From: Pavel Fedin Subject: RE: [PATCH v2 3/3] KVM: arm64: Introduce default dummy save/restore functions Date: Tue, 14 Jul 2015 16:02:39 +0300 Message-ID: <020d01d0be35$5d3bab00$17b30100$@samsung.com> References: <248fcfc798e243b618c5275eef8aa333253fd4da.1436874248.git.p.fedin@samsung.com> <55A5007A.4040802@arm.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: 'Marc Zyngier' , kvmarm@lists.cs.columbia.edu, kvm@vger.kernel.org Return-path: In-reply-to: <55A5007A.4040802@arm.com> Content-language: ru List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: kvmarm-bounces@lists.cs.columbia.edu Sender: kvmarm-bounces@lists.cs.columbia.edu List-Id: kvm.vger.kernel.org Hello! > You may want to check commit 8a14849, which makes this whole patch > irrelevant. Thank you very much for pointing out, i will recheck. My patches are based on linux-stable.git, should i base them on something else? > Also, we do check whether or not the GIC is enabled for a given VM, and > not touching the registers if they are not. What is wrong with that? Look at arch/arm64/kvm/hyp.S, __kvm_vcpu_run function. It unconditionally calls entries in __vgic_sr_vectors table. If vGIC initialization did not complete, this table stays unset and contains zeroes. That was the problem. Ok, this is 64-bit-specific, i can fix this up later. My patch set can be used without 0003, in this case it will fix only ARM32 architecture. So if you would like to accept the rest, you can do it. Kind regards, Pavel Fedin Expert Engineer Samsung Electronics Research center Russia