RE: Patch - support e500-specific: Performance monitor

RE: Patch - support e500-specific: Performance monitor

[Bharat.Bhushan]

> -----Original Message-----
> From: kvm-ppc-owner@vger.kernel.org [mailto:kvm-ppc-owner@vger.kernel.org] On
> Behalf Of Amit Tomar
> Sent: Thursday, September 04, 2014 8:34 PM
> To: agraf@suse.de; kvm-ppc@vger.kernel.org; Caraman Mihai Claudiu-B02008;
> pbonzini@redhat.com
> Subject: Patch - support e500-specific: Performance monitor
> 
> Is There specific any reason not to copy extra handler IOVR 35 for e500?

Because we do not support Performance Monitor for guest.
Why you want to add this, any specific requirement?

Thanks
-Bharat

> 
> 
> --- a/arch/powerpc/kvm/e500.c
> +++ b/arch/powerpc/kvm/e500.c
> @@ -527,7 +527,7 @@ static struct kvmppc_ops kvm_ops_e500 = {
>   static int __init kvmppc_e500_init(void)
>   {
>          int r, i;
> -       unsigned long ivor[3];
> +      unsigned long ivor[4];
>          /* Process remaining handlers above the generic first 16 */
>          unsigned long *handler = &kvmppc_booke_handler_addr[16];
>          unsigned long handler_len;
> @@ -545,7 +545,8 @@ static int __init kvmppc_e500_init(void)
>          ivor[0] = mfspr(SPRN_IVOR32);
>          ivor[1] = mfspr(SPRN_IVOR33);
>          ivor[2] = mfspr(SPRN_IVOR34);
> -       for (i = 0; i < 3; i++) {
> +        ivor[3] = mfspr(SPRN_IVOR35);
> +       for (i = 0; i < 4; i++) {
>                  if (ivor[i] > ivor[max_ivor])
>                          max_ivor = i;
> 
> 
> Please ignore this patch if find it irrelevant or been submitted.
> 
> Thanks,
> Amit Tomar.
> --
> To unsubscribe from this list: send the line "unsubscribe kvm-ppc" in the body
> of a message to majordomo@vger.kernel.org More majordomo info at
> http://vger.kernel.org/majordomo-info.html

Patch - support e500-specific: Performance monitor

[Amit Tomar]

Is There specific any reason not to copy extra handler IOVR 35 for e500?


--- a/arch/powerpc/kvm/e500.c
+++ b/arch/powerpc/kvm/e500.c
@@ -527,7 +527,7 @@ static struct kvmppc_ops kvm_ops_e500 = {
  static int __init kvmppc_e500_init(void)
  {
         int r, i;
-       unsigned long ivor[3];
+      unsigned long ivor[4];
         /* Process remaining handlers above the generic first 16 */
         unsigned long *handler = &kvmppc_booke_handler_addr[16];
         unsigned long handler_len;
@@ -545,7 +545,8 @@ static int __init kvmppc_e500_init(void)
         ivor[0] = mfspr(SPRN_IVOR32);
         ivor[1] = mfspr(SPRN_IVOR33);
         ivor[2] = mfspr(SPRN_IVOR34);
-       for (i = 0; i < 3; i++) {
+        ivor[3] = mfspr(SPRN_IVOR35);
+       for (i = 0; i < 4; i++) {
                 if (ivor[i] > ivor[max_ivor])
                         max_ivor = i;


Please ignore this patch if find it irrelevant or been submitted.

Thanks,
Amit Tomar.