From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756771Ab2HPJDK (ORCPT ); Thu, 16 Aug 2012 05:03:10 -0400 Received: from mx1.redhat.com ([209.132.183.28]:47183 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756512Ab2HPJDI (ORCPT ); Thu, 16 Aug 2012 05:03:08 -0400 Message-ID: <502CB745.8060709@redhat.com> Date: Thu, 16 Aug 2012 12:03:01 +0300 From: Avi Kivity User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:14.0) Gecko/20120717 Thunderbird/14.0 MIME-Version: 1.0 To: Marcelo Tosatti CC: Xiao Guangrong , LKML , KVM Subject: Re: [PATCH v5 00/12] KVM: introduce readonly memslot References: <5020E423.9080004@linux.vnet.ibm.com> <20120810181422.GA14892@amt.cnet> <502A5A01.6040505@redhat.com> <20120814155106.GA22249@amt.cnet> <502B7D7E.20902@redhat.com> <20120815175338.GA18452@amt.cnet> In-Reply-To: <20120815175338.GA18452@amt.cnet> Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 08/15/2012 08:53 PM, Marcelo Tosatti wrote: > On Wed, Aug 15, 2012 at 01:44:14PM +0300, Avi Kivity wrote: >> On 08/14/2012 06:51 PM, Marcelo Tosatti wrote: >> >> >> >> Userspace may want to modify the ROM (for example, when programming a >> >> flash device). It is also possible to map an hva range rw through one >> >> slot and ro through another. >> > >> > Right, can do that with multiple userspace maps to the same anonymous >> > memory region (see other email). >> >> Yes it's possible. It requires that we move all memory allocation to be >> fd based, since userspace can't predict what memory will be dual-mapped >> (at least if emulated hardware allows this). > > It can: > - Create named memory object, with associated fd. > - Copy data from large anonymous memory region to named memory. That doesn't work if dma is in progress (assigned device). It also doubles the amount of memory in use. > - Unmap region that must be dual-mapped from large anonymous memory chunk. > - Map named memory object at address. > > The last step can be replaced by adjusting KVM memory slots. > > The disadvantage of protection information in memory slots > is that it duplicates functionality that is handled by > userspace mappings. Agree. So does the memory slots mechanism, and even dirty logging. > > Moreover, multiple memory maps are necessary for any > split-qemu-into-smaller-pieces solutions. Complex users can use complex mechanism, but let's keep the simple stuff simple. > >> Is this a reasonable >> requirement? Do ksm/thp/autonuma work with this? > > As mentioned, only memory used for ROM purposes must be dual mapped. > > I don't think there is any way to create multiple mappings > to one anonymous memory object ATM, but POSIX defines it > (posix_typed_mem_open). > > The limitation of thp/ksm on shared memory also affects any other user > of shared memory, so it should be fixed there. > > Also, QEMU ROM is allocated separately from RAM, correct? > Correct. But the chipset is also able to to write-protect some ranges in the 0xc0000-0x100000 area via the PAM. It is able to write-protect both RAM and PCI memory (usually mapped to flash). -- error compiling committee.c: too many arguments to function