[PATCH 1/1] Staging: hv: Fix a bug in get_vp_index()

[PATCH 1/1] Staging: hv: Fix a bug in get_vp_index()

[K. Y. Srinivasan]

Linux' notion of cpuid is different from the Host's notion of CPUID. In the
call to bind the channel interrupts, we should use the host's notion of
CPU Ids. Fix this bug.

Signed-off-by: K. Y. Srinivasan <kys@microsoft.com>
Cc: Stable <stable@vger.kernel.org> (V3.9)
---
 drivers/hv/channel_mgmt.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/drivers/hv/channel_mgmt.c b/drivers/hv/channel_mgmt.c
index bad8128..21ef689 100644
--- a/drivers/hv/channel_mgmt.c
+++ b/drivers/hv/channel_mgmt.c
@@ -329,7 +329,7 @@ static u32 get_vp_index(uuid_le *type_guid)
 		return 0;
 	}
 	cur_cpu = (++next_vp % max_cpus);
-	return cur_cpu;
+	return hv_context.vp_index[cur_cpu];
 }
 
 /*
-- 
1.7.4.1

Re: [PATCH 1/1] Staging: hv: Fix a bug in get_vp_index()

[Greg KH]

On Fri, May 10, 2013 at 08:47:01AM -0700, K. Y. Srinivasan wrote:
> Linux' notion of cpuid is different from the Host's notion of CPUID. In the
> call to bind the channel interrupts, we should use the host's notion of
> CPU Ids. Fix this bug.
> 
> Signed-off-by: K. Y. Srinivasan <kys@microsoft.com>
> Cc: Stable <stable@vger.kernel.org> (V3.9)
> ---
>  drivers/hv/channel_mgmt.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)

Why do you have "Staging" in the Subject:?  This isn't a staging driver
anymore, right?

greg k-h

RE: [PATCH 1/1] Staging: hv: Fix a bug in get_vp_index()

[KY Srinivasan]

> -----Original Message-----
> From: Greg KH [mailto:gregkh@linuxfoundation.org]
> Sent: Thursday, May 16, 2013 7:00 PM
> To: KY Srinivasan
> Cc: linux-kernel@vger.kernel.org; devel@linuxdriverproject.org; olaf@aepfle.de;
> apw@canonical.com; jasowang@redhat.com; Stable
> Subject: Re: [PATCH 1/1] Staging: hv: Fix a bug in get_vp_index()
> 
> On Fri, May 10, 2013 at 08:47:01AM -0700, K. Y. Srinivasan wrote:
> > Linux' notion of cpuid is different from the Host's notion of CPUID. In the
> > call to bind the channel interrupts, we should use the host's notion of
> > CPU Ids. Fix this bug.
> >
> > Signed-off-by: K. Y. Srinivasan <kys@microsoft.com>
> > Cc: Stable <stable@vger.kernel.org> (V3.9)
> > ---
> >  drivers/hv/channel_mgmt.c |    2 +-
> >  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> Why do you have "Staging" in the Subject:?  This isn't a staging driver
> anymore, right?

Sorry, my mistake; do you want me to resend this.

Regards,

K. Y
> 
> greg k-h
>