From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:49151) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1TU6fV-0008Cb-GJ for qemu-devel@nongnu.org; Thu, 01 Nov 2012 22:07:34 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1TU6fU-0000PM-8b for qemu-devel@nongnu.org; Thu, 01 Nov 2012 22:07:33 -0400 Date: Fri, 2 Nov 2012 13:07:23 +1100 From: David Gibson Message-ID: <20121102020723.GK27695@truffula.fritz.box> References: <1351567512-17278-1-git-send-email-david@gibson.dropbear.id.au> <1351567512-17278-3-git-send-email-david@gibson.dropbear.id.au> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Subject: Re: [Qemu-devel] [Qemu-ppc] [PATCH 02/16] target-ppc: Rework storage of VPA registration state List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Alexander Graf Cc: qemu-ppc@nongnu.org, qemu-devel@nongnu.org On Thu, Nov 01, 2012 at 11:10:33AM +0100, Alexander Graf wrote: > > On 30.10.2012, at 04:24, David Gibson wrote: > > > With PAPR guests, hypercalls allow registration of the Virtual Processor > > Area (VPA), SLB shadow and dispatch trace log (DTL), each of which allow > > for certain communication between the guest and hypervisor. Currently, we > > store the addresses of the three areas and the size of the dtl in > > CPUPPCState. > > > > The SLB shadow and DTL are variable sized, with the size being retrieved > > from within the registered memory area at the hypercall time. This size > > can later be overwritten with other information, however, so we need to > > save the size as of registration time. We already do this for the DTL, > > but not for the SLB shadow, so this patch fixes that. > > > > In addition, we change the storage of the VPA information to use fixed > > size integer types which will make life easier for syncing this data with > > KVM, which we will need in future. > > > > Signed-off-by: David Gibson > > The commit message seems to not match the patch contents. I'll fix > that up. Sorry, thanks. I think the described change was in an earlier version, but I'm not sure wuite when it fell out. -- David Gibson | I'll have my music baroque, and my code david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_ | _way_ _around_! http://www.ozlabs.org/~dgibson