RE: Patch - support e500-specific: Performance monitor

2014-09-04 Thread bharat.bhus...@freescale.com


> -Original Message-
> From: kvm-ppc-ow...@vger.kernel.org [mailto:kvm-ppc-ow...@vger.kernel.org] On
> Behalf Of Amit Tomar
> Sent: Thursday, September 04, 2014 8:34 PM
> To: ag...@suse.de; kvm-ppc@vger.kernel.org; Caraman Mihai Claudiu-B02008;
> pbonz...@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 majord...@vger.kernel.org More majordomo info at
> http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe kvm-ppc" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Patch - support e500-specific: Performance monitor

2014-09-04 Thread 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.
--
To unsubscribe from this list: send the line "unsubscribe kvm-ppc" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html