Re: [Qemu-devel] [PATCH v2] monitor: add PPC BookE SPRs

[Scott Wood <scottwood@freescale.com>]

On Tue, 3 May 2011 13:02:40 +0200
Alexander Graf <agraf@suse.de> wrote:

> 
> On 30.04.2011, at 00:10, Scott Wood wrote:
> 
> > Read them via KVM_GET_SREGS in kvm_arch_get_registers(),
> > and display them in "info registers".
> > 
> > Also get CR and PID from the existing KVM_GET_REGS.
> > 
> > Signed-off-by: Scott Wood <scottwood@freescale.com>
> > ---
> > v2: fix a couple style oversights, and cache kvm caps as requested
> 
> In fact, I'll just squash this patch with yours, if that's ok for you:

OK.

> diff --git a/target-ppc/kvm.c b/target-ppc/kvm.c
> index a4ec4bc..5a1b6cb 100644
> --- a/target-ppc/kvm.c
> +++ b/target-ppc/kvm.c
> @@ -45,7 +45,9 @@ const KVMCapabilityInfo kvm_arch_required_capabilities[] = {
>  static int cap_interrupt_unset = false;
>  static int cap_interrupt_level = false;
>  static int cap_segstate;
> +#ifdef KVM_CAP_PPC_BOOKE_SREGS
>  static int cap_booke_sregs;
> +#endif
>  
>  /* XXX We have a race condition where we actually have a level triggered
>   *     interrupt, but the infrastructure can't expose that yet, so the guest
> @@ -94,13 +96,9 @@ static int kvm_arch_sync_sregs(CPUState *cenv)
>      if (cenv->excp_model == POWERPC_EXCP_BOOKE) {
>          return 0;
>      } else {
> -#ifdef KVM_CAP_PPC_SEGSTATE
>          if (!cap_segstate) {
>              return 0;
>          }
> -#else
> -        return 0;
> -#endif

So there are no headers out there that we need to support building with
that don't have KVM_CAP_PPC_SEGSTATE?

-Scott