From mboxrd@z Thu Jan 1 00:00:00 1970 From: Paolo Bonzini Subject: Re: [Qemu-devel] [PATCH v5 10/12] hw/mips: malta: Add KVM support Date: Fri, 20 Jun 2014 13:11:16 +0200 Message-ID: <53A416D4.1020902@redhat.com> References: <1403043037-1271-1-git-send-email-james.hogan@imgtec.com> <1403043037-1271-11-git-send-email-james.hogan@imgtec.com> <20140619162748.GO7398@ohm.rr44.fr> <577D5B21-4F8B-4A9A-BD25-A76C76987544@kymasys.com> <20140619214733.GJ24904@hall.aurel32.net> <53A3FE04.9030703@imgtec.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-15; format=flowed Content-Transfer-Encoding: 7bit Cc: qemu-devel@nongnu.org, Peter Maydell , kvm@vger.kernel.org, Gleb Natapov To: James Hogan , Aurelien Jarno , Sanjay Lal Return-path: Received: from mail-we0-f170.google.com ([74.125.82.170]:45186 "EHLO mail-we0-f170.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S966811AbaFTLLV (ORCPT ); Fri, 20 Jun 2014 07:11:21 -0400 Received: by mail-we0-f170.google.com with SMTP id w61so3682175wes.1 for ; Fri, 20 Jun 2014 04:11:20 -0700 (PDT) In-Reply-To: <53A3FE04.9030703@imgtec.com> Sender: kvm-owner@vger.kernel.org List-ID: Il 20/06/2014 11:25, James Hogan ha scritto: > On 19/06/14 22:47, Aurelien Jarno wrote: >> On Thu, Jun 19, 2014 at 12:34:24PM -0700, Sanjay Lal wrote: >>> >>> On Jun 19, 2014, at 9:27 AM, Aurelien Jarno wrote: >>> >>>> On Tue, Jun 17, 2014 at 11:10:35PM +0100, James Hogan wrote: >>>>> In KVM mode the bootrom is loaded and executed from the last 1MB of >>>>> DRAM. >>>> >>>> What is the reason for that? I am not opposed to that, but if it is >>>> really needed, it means that loading a bootloader into the flash area >>>> (for example YAMON) won't work and that this should be forbidden to the >>>> user. >>>> >>> >>> In trap and emulate mode, both the kernel and userland run in user mode on the processor. Virtual addresses >= 0x80000000 are only accessible in kernel mode, and the default flash area (VA: 0xbfc00000/PA: 0x1fc00000) falls in this range. >>> >>> We therefore decided to relocate the bootloader to the last 1MB of RAM. This area is excluded from the RAM ranges supplied to the kernel, so it should not be accessible to the user. >>> >> >> Thanks for the explanation. It means we should disable the support for >> booting from the flash (using -pflash) in KVM mode, as it would simply >> not work. >> > > Hi Aurelien, > > Is this fixup to the malta patch the sort of thing you had in mind? If > so I'll generate a v6 patchset with it. It looks like this. No hurry, it can go in after the main series; just git-send-email it so it gets noticed and has a proper commit message. Paolo