From mboxrd@z Thu Jan 1 00:00:00 1970 From: Pavel Fedin Subject: RE: [PATCH v2 1/3] KVM: arm: Fix NULL pointer dereference if KVM is used without in-kernel irqchip Date: Sun, 19 Jul 2015 17:19:09 +0300 Message-ID: <00e901d0c22d$e1b0c680$a5125380$@samsung.com> References: <644bce2849b89bc8520c7efcb7752c92e31f9ccd.1436874248.git.p.fedin@samsung.com> <55A931B7.4000905@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: <55A931B7.4000905@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! > I'd appreciate a better commit log. Which patch broke it, why is that > necessary. It was broken by 662d9715840aef44dcb573b0f9fab9e8319c868a. Actually, runtime detection was never implemented. The patch message just says that it can be done. And in order to run KVM on my lame board i had to disable VGIC and VTIMER in kernel .config. The actual breakage happened because without these definitions kvm_vgic_map_resources() was just a do-nothing inline. vGIC can perfectly be emulated by userspace. And even generic timer can, with some tricks. So i decided to restore this functionality. Kind regards, Pavel Fedin Expert Engineer Samsung Electronics Research center Russia