Re: [PATCH] KVM: arm: irqfd: fix value returned by kvm_irq_map_gsi

[Christoffer Dall]

On Mon, Apr 13, 2015 at 03:01:59PM +0200, Eric Auger wrote:
 irqfd/arm curently does not support routing. kvm_irq_map_gsi is
 supposed to return all the routing entries associated with the
 provided gsi and return the number of those entries. We should
 return 0 at this point.
 
 Signed-off-by: Eric Auger eric.au...@linaro.org
 ---
  virt/kvm/arm/vgic.c | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)
 
 diff --git a/virt/kvm/arm/vgic.c b/virt/kvm/arm/vgic.c
 index 9ad074e..9b4f7d4 100644
 --- a/virt/kvm/arm/vgic.c
 +++ b/virt/kvm/arm/vgic.c
 @@ -1947,7 +1947,7 @@ int kvm_irq_map_gsi(struct kvm *kvm,
   struct kvm_kernel_irq_routing_entry *entries,
   int gsi)
  {
 - return gsi;
 + return 0;
  }
  
  int kvm_irq_map_chip_pin(struct kvm *kvm, unsigned irqchip, unsigned pin)
 -- 

Acked-by: Christoffer Dall christoffer.d...@linaro.org
___
kvmarm mailing list
kvmarm@lists.cs.columbia.edu
https://lists.cs.columbia.edu/mailman/listinfo/kvmarm

[PATCH] KVM: arm: irqfd: fix value returned by kvm_irq_map_gsi

[Eric Auger]

irqfd/arm curently does not support routing. kvm_irq_map_gsi is
supposed to return all the routing entries associated with the
provided gsi and return the number of those entries. We should
return 0 at this point.

Signed-off-by: Eric Auger eric.au...@linaro.org
---
 virt/kvm/arm/vgic.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/virt/kvm/arm/vgic.c b/virt/kvm/arm/vgic.c
index 9ad074e..9b4f7d4 100644
--- a/virt/kvm/arm/vgic.c
+++ b/virt/kvm/arm/vgic.c
@@ -1947,7 +1947,7 @@ int kvm_irq_map_gsi(struct kvm *kvm,
struct kvm_kernel_irq_routing_entry *entries,
int gsi)
 {
-   return gsi;
+   return 0;
 }
 
 int kvm_irq_map_chip_pin(struct kvm *kvm, unsigned irqchip, unsigned pin)
-- 
1.9.1

___
kvmarm mailing list
kvmarm@lists.cs.columbia.edu
https://lists.cs.columbia.edu/mailman/listinfo/kvmarm