From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:33840) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Wn9oG-0007FF-DV for qemu-devel@nongnu.org; Wed, 21 May 2014 12:56:17 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Wn9o7-00043F-DG for qemu-devel@nongnu.org; Wed, 21 May 2014 12:56:08 -0400 Message-ID: <537CDA97.6090204@gmail.com> Date: Wed, 21 May 2014 11:55:51 -0500 From: Tom Musta MIME-Version: 1.0 References: <1400653228-31540-1-git-send-email-aik@ozlabs.ru> <1400653228-31540-2-git-send-email-aik@ozlabs.ru> In-Reply-To: <1400653228-31540-2-git-send-email-aik@ozlabs.ru> Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH 1/9] target-ppc: Rename MMCR0/1 contants List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Alexey Kardashevskiy , qemu-devel@nongnu.org Cc: qemu-ppc@nongnu.org, Alexander Graf On 5/21/2014 1:20 AM, Alexey Kardashevskiy wrote: > MMCR0 and MMCR1 have different numbers for 32 and 64 bit POWERPC. > We are going to support 64bit versions too so let's rename 32bit ones > to avoid confusion. > > 604 uses same SPR number for MMCR0 so it is included in this patch too. I'm not sure that it is a 32-bit vs. 64-bit thing as much as it is an evolution of Performance Monitor from individual Book IVs into the ISA. Saying differently, the SPR number for MMCR0 is now specified in the ISA and is the same for 32 and 64 bit implementations ... and is also the same for embedded versus server implementations. When the 604 came out, the performance monitor was not yet part of the ISA. Other than that ... Reviewed-by: Tom Musta