Re: [PATCH KERNEL] xen/events: use xen_vcpu_id mapping for EVTCHNOP_status

[Juergen Gross]

On 23/11/16 13:38, Vitaly Kuznetsov wrote:
> EVTCHNOP_status hypercall returns Xen's idea of vcpu id so we need to
> compare it against xen_vcpu_id mapping, not the Linux cpu id.
> 
> Suggested-by: Radim Krcmar 
> Signed-off-by: Vitaly Kuznetsov 

Applied to xen/tip.git for-linus-4.10


Juergen

Re: [PATCH KERNEL] xen/events: use xen_vcpu_id mapping for EVTCHNOP_status

[Juergen Gross]

On 23/11/16 13:38, Vitaly Kuznetsov wrote:
> EVTCHNOP_status hypercall returns Xen's idea of vcpu id so we need to
> compare it against xen_vcpu_id mapping, not the Linux cpu id.
> 
> Suggested-by: Radim Krcmar 
> Signed-off-by: Vitaly Kuznetsov 

Applied to xen/tip.git for-linus-4.10


Juergen

Re: [PATCH KERNEL] xen/events: use xen_vcpu_id mapping for EVTCHNOP_status

[Boris Ostrovsky]

On 11/23/2016 07:38 AM, Vitaly Kuznetsov wrote:
> EVTCHNOP_status hypercall returns Xen's idea of vcpu id so we need to
> compare it against xen_vcpu_id mapping, not the Linux cpu id.
>
> Suggested-by: Radim Krcmar 
> Signed-off-by: Vitaly Kuznetsov 

Reviewed-by: Boris Ostrovsky 


> ---
>  drivers/xen/events/events_base.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/xen/events/events_base.c 
> b/drivers/xen/events/events_base.c
> index 9ecfcdc..137bd0e 100644
> --- a/drivers/xen/events/events_base.c
> +++ b/drivers/xen/events/events_base.c
> @@ -948,7 +948,7 @@ static int find_virq(unsigned int virq, unsigned int cpu)
>   continue;
>   if (status.status != EVTCHNSTAT_virq)
>   continue;
> - if (status.u.virq == virq && status.vcpu == cpu) {
> + if (status.u.virq == virq && status.vcpu == xen_vcpu_nr(cpu)) {
>   rc = port;
>   break;
>   }

Re: [PATCH KERNEL] xen/events: use xen_vcpu_id mapping for EVTCHNOP_status

[Boris Ostrovsky]

On 11/23/2016 07:38 AM, Vitaly Kuznetsov wrote:
> EVTCHNOP_status hypercall returns Xen's idea of vcpu id so we need to
> compare it against xen_vcpu_id mapping, not the Linux cpu id.
>
> Suggested-by: Radim Krcmar 
> Signed-off-by: Vitaly Kuznetsov 

Reviewed-by: Boris Ostrovsky 


> ---
>  drivers/xen/events/events_base.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/xen/events/events_base.c 
> b/drivers/xen/events/events_base.c
> index 9ecfcdc..137bd0e 100644
> --- a/drivers/xen/events/events_base.c
> +++ b/drivers/xen/events/events_base.c
> @@ -948,7 +948,7 @@ static int find_virq(unsigned int virq, unsigned int cpu)
>   continue;
>   if (status.status != EVTCHNSTAT_virq)
>   continue;
> - if (status.u.virq == virq && status.vcpu == cpu) {
> + if (status.u.virq == virq && status.vcpu == xen_vcpu_nr(cpu)) {
>   rc = port;
>   break;
>   }

[PATCH KERNEL] xen/events: use xen_vcpu_id mapping for EVTCHNOP_status

[Vitaly Kuznetsov]

EVTCHNOP_status hypercall returns Xen's idea of vcpu id so we need to
compare it against xen_vcpu_id mapping, not the Linux cpu id.

Suggested-by: Radim Krcmar 
Signed-off-by: Vitaly Kuznetsov 
---
 drivers/xen/events/events_base.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/xen/events/events_base.c b/drivers/xen/events/events_base.c
index 9ecfcdc..137bd0e 100644
--- a/drivers/xen/events/events_base.c
+++ b/drivers/xen/events/events_base.c
@@ -948,7 +948,7 @@ static int find_virq(unsigned int virq, unsigned int cpu)
continue;
if (status.status != EVTCHNSTAT_virq)
continue;
-   if (status.u.virq == virq && status.vcpu == cpu) {
+   if (status.u.virq == virq && status.vcpu == xen_vcpu_nr(cpu)) {
rc = port;
break;
}
-- 
2.7.4

[PATCH KERNEL] xen/events: use xen_vcpu_id mapping for EVTCHNOP_status

[Vitaly Kuznetsov]

EVTCHNOP_status hypercall returns Xen's idea of vcpu id so we need to
compare it against xen_vcpu_id mapping, not the Linux cpu id.

Suggested-by: Radim Krcmar 
Signed-off-by: Vitaly Kuznetsov 
---
 drivers/xen/events/events_base.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/xen/events/events_base.c b/drivers/xen/events/events_base.c
index 9ecfcdc..137bd0e 100644
--- a/drivers/xen/events/events_base.c
+++ b/drivers/xen/events/events_base.c
@@ -948,7 +948,7 @@ static int find_virq(unsigned int virq, unsigned int cpu)
continue;
if (status.status != EVTCHNSTAT_virq)
continue;
-   if (status.u.virq == virq && status.vcpu == cpu) {
+   if (status.u.virq == virq && status.vcpu == xen_vcpu_nr(cpu)) {
rc = port;
break;
}
-- 
2.7.4