From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from [140.186.70.92] (port=41661 helo=eggs.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1Pl3TD-0004Rz-7X for qemu-devel@nongnu.org; Thu, 03 Feb 2011 12:59:55 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Pl2pB-0008AC-1u for qemu-devel@nongnu.org; Thu, 03 Feb 2011 12:18:30 -0500 Received: from goliath.siemens.de ([192.35.17.28]:34534) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Pl2pA-00089Q-Os for qemu-devel@nongnu.org; Thu, 03 Feb 2011 12:18:29 -0500 Message-ID: <4D4AE360.1050905@siemens.com> Date: Thu, 03 Feb 2011 18:18:24 +0100 From: Jan Kiszka MIME-Version: 1.0 Subject: Re: [Qemu-devel] [0.14?][PATCH 3/4] ioapic: Prepare for base address relocation References: <0072079efad1c31da849cff7ad2cb426aeb6c29f.1296744934.git.jan.kiszka@siemens.com> In-Reply-To: Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Blue Swirl Cc: Anthony Liguori , Gleb Natapov , "kvm@vger.kernel.org" , Marcelo Tosatti , "qemu-devel@nongnu.org" , Alexander Graf , Avi Kivity On 2011-02-03 18:03, Blue Swirl wrote: > On Thu, Feb 3, 2011 at 2:55 PM, Jan Kiszka wrote: >> The registers of real IOAPICs can be relocated during runtime (via >> chipset registers). We don't support this yet, but qemu-kvm carries the >> current base address in its version 2 vmstate. >> >> To align both implementations for migratability, add the proper >> infrastructure to accept initial as well as updated base addresses and >> include the current address in the vmstate. This is done in a way that >> will also allow multiple IOAPICs in the future. > > Nack, the addresses should be device properties. Hmm.... we could make default_base_address a property. Will change that. But current_base_address is just the same as apicbase and can't be a property. Jan -- Siemens AG, Corporate Technology, CT T DE IT 1 Corporate Competence Center Embedded Linux