From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-6.8 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, INCLUDES_PATCH,MAILING_LIST_MULTI,SIGNED_OFF_BY,SPF_HELO_NONE,SPF_PASS autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 31255C0650F for ; Tue, 30 Jul 2019 13:28:07 +0000 (UTC) Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 0795F20644 for ; Tue, 30 Jul 2019 13:28:07 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 0795F20644 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=redhat.com Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=qemu-devel-bounces+qemu-devel=archiver.kernel.org@nongnu.org Received: from localhost ([::1]:32926 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.86_2) (envelope-from ) id 1hsSAg-0007PB-Bl for qemu-devel@archiver.kernel.org; Tue, 30 Jul 2019 09:28:06 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:47566) by lists.gnu.org with esmtp (Exim 4.86_2) (envelope-from ) id 1hsS8i-0005KB-Ii for qemu-devel@nongnu.org; Tue, 30 Jul 2019 09:26:05 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1hsS8h-0008O9-Bo for qemu-devel@nongnu.org; Tue, 30 Jul 2019 09:26:04 -0400 Received: from mx1.redhat.com ([209.132.183.28]:57172) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1hsS8h-0008Nh-3G; Tue, 30 Jul 2019 09:26:03 -0400 Received: from smtp.corp.redhat.com (int-mx08.intmail.prod.int.phx2.redhat.com [10.5.11.23]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id 2211E3082AEF; Tue, 30 Jul 2019 13:26:02 +0000 (UTC) Received: from Igors-MacBook-Pro (ovpn-204-67.brq.redhat.com [10.40.204.67]) by smtp.corp.redhat.com (Postfix) with ESMTP id 1FE2E19C70; Tue, 30 Jul 2019 13:26:00 +0000 (UTC) Date: Tue, 30 Jul 2019 15:25:58 +0200 From: Igor Mammedov To: "Dr. David Alan Gilbert" Message-ID: <20190730152558.2e1aa23a@Igors-MacBook-Pro> In-Reply-To: <20190729175315.GK2756@work-vm> References: <20190729145229.4333-1-imammedo@redhat.com> <20190729145229.4333-2-imammedo@redhat.com> <20190729175315.GK2756@work-vm> MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit X-Scanned-By: MIMEDefang 2.84 on 10.5.11.23 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.45]); Tue, 30 Jul 2019 13:26:02 +0000 (UTC) X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-Received-From: 209.132.183.28 Subject: Re: [Qemu-devel] [PATCH RFC 1/2] memory: make MemoryRegion alias migratable X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: pbonzini@redhat.com, qemu-s390x@nongnu.org, qemu-devel@nongnu.org Errors-To: qemu-devel-bounces+qemu-devel=archiver.kernel.org@nongnu.org Sender: "Qemu-devel" On Mon, 29 Jul 2019 18:53:15 +0100 "Dr. David Alan Gilbert" wrote: > * Igor Mammedov (imammedo@redhat.com) wrote: > > use qemu_ram_alloc_from_ptr() to create aliased RAMBlock > > to the part of original memory region. > > > > Signed-off-by: Igor Mammedov > > --- > > exec.c | 7 ++++--- > > memory.c | 5 +++++ > > 2 files changed, 9 insertions(+), 3 deletions(-) > > > > diff --git a/exec.c b/exec.c > > index 3e78de3b8f..daef0cd54f 100644 > > --- a/exec.c > > +++ b/exec.c > > @@ -2313,7 +2313,7 @@ static void ram_block_add(RAMBlock *new_block, Error **errp, bool shared) > > new_block->used_length, > > DIRTY_CLIENTS_ALL); > > > > - if (new_block->host) { > > + if (new_block->host && !new_block->mr->alias) { > > qemu_ram_setup_dump(new_block->host, new_block->max_length); > > qemu_madvise(new_block->host, new_block->max_length, QEMU_MADV_HUGEPAGE); > > /* MADV_DONTFORK is also needed by KVM in absence of synchronous MMU */ > > @@ -2671,7 +2671,8 @@ RAMBlock *qemu_ram_block_from_host(void *ptr, bool round_offset, > > > > rcu_read_lock(); > > block = atomic_rcu_read(&ram_list.mru_block); > > - if (block && block->host && host - block->host < block->max_length) { > > + if (block && !block->mr->alias && block->host && > > + host - block->host < block->max_length) { > > goto found; > > } > > > > @@ -2680,7 +2681,7 @@ RAMBlock *qemu_ram_block_from_host(void *ptr, bool round_offset, > > if (block->host == NULL) { > > continue; > > } > > - if (host - block->host < block->max_length) { > > + if (!block->mr->alias && host - block->host < block->max_length) { > > goto found; > > } > > } > > diff --git a/memory.c b/memory.c > > index 5d8c9a9234..d710c17a26 100644 > > --- a/memory.c > > +++ b/memory.c > > @@ -1678,6 +1678,11 @@ void memory_region_init_alias(MemoryRegion *mr, > > memory_region_init(mr, owner, name, size); > > mr->alias = orig; > > mr->alias_offset = offset; > > + if (orig->ram_block && size) { > > + mr->ram_block = qemu_ram_alloc_from_ptr(size, > > + orig->ram_block->host + offset, > > + mr, &error_fatal); > > + } > > } > > Doesn't this cause new memory regions to be created in other existing > machines, e.g. x86's mem-smram, or the various PCI vga hacks? I'd guess you've meant RAMBlocks instead of memory regions, if that's it then yes, every alias pointing to RAM backed memory region will have RAMBlock that's points to aliased part of aliased memory region. (I didn't not intended to limit it to s390 only as it potentially could be used for x86 initial memory refactoring as well) If it's issue we probably can address that by adding additional API memory_region_alias_make_migratable() and call it on selected aliases, but the less APIs we have the better so I went for reusing exiting vmstate_register_ram_global(). (I admit faking RAMBlock is a hack (albeit working one), but my attempt to teach migration code to accept aliases directly didn't work for me well and was very fragile, I don't remember specifics any more and probably code as well since it was utter failure) PS: this patch is by no means a complete one (I probably cut corners on cleanup path) but should be sufficient to demo idea and see if it's acceptable. > Dave > > > > void memory_region_init_rom_nomigrate(MemoryRegion *mr, > > -- > > 2.18.1 > > > -- > Dr. David Alan Gilbert / dgilbert@redhat.com / Manchester, UK