From: David Gibson <david@gibson.dropbear.id.au>
To: Alexander Graf <agraf@suse.de>
Cc: qemu-ppc@nongnu.org, qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] [Qemu-ppc] [PATCH 02/16] target-ppc: Rework storage of VPA registration state
Date: Fri, 2 Nov 2012 13:07:23 +1100 [thread overview]
Message-ID: <20121102020723.GK27695@truffula.fritz.box> (raw)
In-Reply-To: <CC1C1CEC-99E9-404E-BA6C-081DFC668A3F@suse.de>
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 <david@gibson.dropbear.id.au>
>
> 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
next prev parent reply other threads:[~2012-11-02 2:07 UTC|newest]
Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-10-30 3:24 [Qemu-devel] [0/16] pseries and PPC pending patches David Gibson
2012-10-30 3:24 ` [Qemu-devel] [PATCH 01/16] Revert "PPC: pseries: Remove hack for PIO window" David Gibson
2012-10-30 3:24 ` [Qemu-devel] [PATCH 02/16] target-ppc: Rework storage of VPA registration state David Gibson
2012-11-01 10:10 ` Alexander Graf
2012-11-02 2:07 ` David Gibson [this message]
2012-10-30 3:24 ` [Qemu-devel] [PATCH 03/16] target-ppc: Extend FPU state for newer POWER CPUs David Gibson
2012-10-30 3:25 ` [Qemu-devel] [PATCH 04/16] pseries: Clean up inconsistent variable name in xics.c David Gibson
2012-10-30 3:25 ` [Qemu-devel] [PATCH 05/16] pseries: Use #define for XICS base irq number David Gibson
2012-11-01 10:15 ` Alexander Graf
2012-11-02 2:12 ` David Gibson
2012-10-30 3:25 ` [Qemu-devel] [PATCH 06/16] pseries: Cleanup duplications of ics_valid_irq() code David Gibson
2012-10-30 3:25 ` [Qemu-devel] [PATCH 07/16] pseries: Move XICS initialization before cpu initialization David Gibson
2012-11-01 10:22 ` Alexander Graf
2012-10-30 3:25 ` [Qemu-devel] [PATCH 08/16] pseries: Return the token when we register an RTAS call David Gibson
2012-10-30 3:25 ` [Qemu-devel] [PATCH 09/16] pseries: Allow RTAS tokens without a qemu handler David Gibson
2012-10-30 3:25 ` [Qemu-devel] [PATCH 10/16] pseries: Add tracepoints to the XICS interrupt controller David Gibson
2012-10-30 3:25 ` [Qemu-devel] [PATCH 11/16] pseries: Split xics irq configuration from state information David Gibson
2012-10-30 3:25 ` [Qemu-devel] [PATCH 12/16] target-pcc: Convert ppcemb_tlb_t to use fixed 64-bit RPN David Gibson
2012-10-30 3:25 ` [Qemu-devel] [PATCH 13/16] pseries: Implement PAPR NVRAM David Gibson
2012-10-30 3:25 ` [Qemu-devel] [PATCH 14/16] pseries: Update SLOF for NVRAM support David Gibson
2012-10-30 3:25 ` [Qemu-devel] [PATCH 15/16] pseries: Fix bug in PCI MSI allocation David Gibson
2012-10-30 3:25 ` [Qemu-devel] [PATCH 16/16] pseries: Generate unique LIOBNs for PCI host bridges David Gibson
2012-11-01 10:26 ` [Qemu-devel] [0/16] pseries and PPC pending patches Alexander Graf
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20121102020723.GK27695@truffula.fritz.box \
--to=david@gibson.dropbear.id.au \
--cc=agraf@suse.de \
--cc=qemu-devel@nongnu.org \
--cc=qemu-ppc@nongnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).