From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:46000) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1TakWn-00008P-AQ for qemu-devel@nongnu.org; Tue, 20 Nov 2012 04:54:06 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1TakWh-0006Op-He for qemu-devel@nongnu.org; Tue, 20 Nov 2012 04:54:01 -0500 Received: from mail-ia0-f173.google.com ([209.85.210.173]:50058) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1TakWh-0006OW-Ct for qemu-devel@nongnu.org; Tue, 20 Nov 2012 04:53:55 -0500 Received: by mail-ia0-f173.google.com with SMTP id w21so2769816iac.4 for ; Tue, 20 Nov 2012 01:53:54 -0800 (PST) MIME-Version: 1.0 In-Reply-To: <3F40C7F7-0687-4730-89D8-E2E414820117@suse.de> References: <1352774820-22804-1-git-send-email-david@gibson.dropbear.id.au> <1352774820-22804-9-git-send-email-david@gibson.dropbear.id.au> <69634422-E5DA-48E6-807D-5C0333424EB9@suse.de> <20121119224823.GC18362@truffula.fritz.box> <3F40C7F7-0687-4730-89D8-E2E414820117@suse.de> Date: Tue, 20 Nov 2012 09:53:53 +0000 Message-ID: From: Peter Maydell Content-Type: text/plain; charset=UTF-8 Subject: Re: [Qemu-devel] [Qemu-ppc] [PATCH 08/12] target-ppc: Convert ppcemb_tlb_t to use fixed 64-bit RPN List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Alexander Graf Cc: qemu-ppc@nongnu.org, qemu-devel@nongnu.org, David Gibson On 20 November 2012 09:29, Alexander Graf wrote: > On 19.11.2012, at 23:48, David Gibson wrote: >> On Mon, Nov 19, 2012 at 05:26:45PM +0100, Alexander Graf wrote: >>> On 13.11.2012, at 03:46, David Gibson wrote: >>>> This patch therefore changes ppcemb_tlb_t to use a fixed 64-bit integer >>>> which we know is sufficient for all the machines which use this structure. >>> >>> hwaddr is always defined to 64bit by now. >> >> I know, but there aren't state save helpers for hwaddr, and there are >> objections to creating them. (previous discussion on this point: https://lists.gnu.org/archive/html/qemu-devel/2012-10/msg01456.html ) > Sure, but you can just use the 64bit save helpers now that hwaddr == uint64_t, no? That would be one approach. I'm a bit sceptical about putting hwaddr fields in CPU state, though -- it's suggestive that something's not modelled right. hwaddr is conceptually "big enough for the biggest bus in the system", and no single component should have internal state whose size depends on that. -- PMM