[PATCH 1/2] KVM: arm/arm64: Optimize away redundant LR tracking
Currently we use vgic_irq_lr_map in order to track which LRs hold which IRQs, and lr_used bitmap in order to track which LRs are used or free. vgic_irq_lr_map is actually used only for piggy-back optimization, and can be easily replaced by iteration over lr_used. This is good because in future, when LPI support is introduced, number of IRQs will grow up to at least 16384, while numbers from 1024 to 8192 are never going to be used. This would be a huge memory waste. In its turn, lr_used is also completely redundant since ae705930fca6322600690df9dc1c7d0516145a93 ("arm/arm64: KVM: Keep elrsr/aisr in sync with software model"), because together with lr_used we also update elrsr. This allows to easily replace lr_used with elrsr, inverting all conditions (because in elrsr '1' means 'free'). Signed-off-by: Pavel Fedin --- include/kvm/arm_vgic.h | 6 virt/kvm/arm/vgic.c| 74 +++--- 2 files changed, 28 insertions(+), 52 deletions(-) diff --git a/include/kvm/arm_vgic.h b/include/kvm/arm_vgic.h index 4e14dac..d908028 100644 --- a/include/kvm/arm_vgic.h +++ b/include/kvm/arm_vgic.h @@ -296,9 +296,6 @@ struct vgic_v3_cpu_if { }; struct vgic_cpu { - /* per IRQ to LR mapping */ - u8 *vgic_irq_lr_map; - /* Pending/active/both interrupts on this VCPU */ DECLARE_BITMAP( pending_percpu, VGIC_NR_PRIVATE_IRQS); DECLARE_BITMAP( active_percpu, VGIC_NR_PRIVATE_IRQS); @@ -309,9 +306,6 @@ struct vgic_cpu { unsigned long *active_shared; unsigned long *pend_act_shared; - /* Bitmap of used/free list registers */ - DECLARE_BITMAP( lr_used, VGIC_V2_MAX_LRS); - /* Number of list registers on this CPU */ int nr_lr; diff --git a/virt/kvm/arm/vgic.c b/virt/kvm/arm/vgic.c index 6bd1c9b..2f4d25a 100644 --- a/virt/kvm/arm/vgic.c +++ b/virt/kvm/arm/vgic.c @@ -102,9 +102,10 @@ #include "vgic.h" static void vgic_retire_disabled_irqs(struct kvm_vcpu *vcpu); -static void vgic_retire_lr(int lr_nr, int irq, struct kvm_vcpu *vcpu); +static void vgic_retire_lr(int lr_nr, struct kvm_vcpu *vcpu); static struct vgic_lr vgic_get_lr(const struct kvm_vcpu *vcpu, int lr); static void vgic_set_lr(struct kvm_vcpu *vcpu, int lr, struct vgic_lr lr_desc); +static u64 vgic_get_elrsr(struct kvm_vcpu *vcpu); static struct irq_phys_map *vgic_irq_map_search(struct kvm_vcpu *vcpu, int virt_irq); @@ -683,9 +684,11 @@ bool vgic_handle_cfg_reg(u32 *reg, struct kvm_exit_mmio *mmio, void vgic_unqueue_irqs(struct kvm_vcpu *vcpu) { struct vgic_cpu *vgic_cpu = &vcpu->arch.vgic_cpu; + u64 elrsr = vgic_get_elrsr(vcpu); + unsigned long *elrsr_ptr = u64_to_bitmask(&elrsr); int i; - for_each_set_bit(i, vgic_cpu->lr_used, vgic_cpu->nr_lr) { + for_each_clear_bit(i, elrsr_ptr, vgic_cpu->nr_lr) { struct vgic_lr lr = vgic_get_lr(vcpu, i); /* @@ -728,7 +731,7 @@ void vgic_unqueue_irqs(struct kvm_vcpu *vcpu) * Mark the LR as free for other use. */ BUG_ON(lr.state & LR_STATE_MASK); - vgic_retire_lr(i, lr.irq, vcpu); + vgic_retire_lr(i, vcpu); vgic_irq_clear_queued(vcpu, lr.irq); /* Finally update the VGIC state. */ @@ -1087,15 +1090,12 @@ static inline void vgic_enable(struct kvm_vcpu *vcpu) vgic_ops->enable(vcpu); } -static void vgic_retire_lr(int lr_nr, int irq, struct kvm_vcpu *vcpu) +static void vgic_retire_lr(int lr_nr, struct kvm_vcpu *vcpu) { - struct vgic_cpu *vgic_cpu = &vcpu->arch.vgic_cpu; struct vgic_lr vlr = vgic_get_lr(vcpu, lr_nr); vlr.state = 0; vgic_set_lr(vcpu, lr_nr, vlr); - clear_bit(lr_nr, vgic_cpu->lr_used); - vgic_cpu->vgic_irq_lr_map[irq] = LR_EMPTY; vgic_sync_lr_elrsr(vcpu, lr_nr, vlr); } @@ -1110,14 +1110,15 @@ static void vgic_retire_lr(int lr_nr, int irq, struct kvm_vcpu *vcpu) */ static void vgic_retire_disabled_irqs(struct kvm_vcpu *vcpu) { - struct vgic_cpu *vgic_cpu = &vcpu->arch.vgic_cpu; + u64 elrsr = vgic_get_elrsr(vcpu); + unsigned long *elrsr_ptr = u64_to_bitmask(&elrsr); int lr; - for_each_set_bit(lr, vgic_cpu->lr_used, vgic->nr_lr) { + for_each_clear_bit(lr, elrsr_ptr, vgic->nr_lr) { struct vgic_lr vlr = vgic_get_lr(vcpu, lr); if (!vgic_irq_is_enabled(vcpu, vlr.irq)) { - vgic_retire_lr(lr, vlr.irq, vcpu); + vgic_retire_lr(lr, vcpu); if (vgic_irq_is_queued(vcpu, vlr.irq)) vgic_irq_clear_queued(vcpu, vlr.irq); } @@ -1169,8 +1170,9 @@ static void vgic_queue_irq_to_lr(struct kvm_vcpu *vcpu, int irq, */ bool vgic_queue_irq(struct kvm_vcpu *vcpu, u8 sgi_source_id, int irq) { -
Re: [PATCH 1/2] KVM: arm/arm64: Optimize away redundant LR tracking
Hi, On 02/10/15 15:44, Pavel Fedin wrote: > Currently we use vgic_irq_lr_map in order to track which LRs hold which > IRQs, and lr_used bitmap in order to track which LRs are used or free. > > vgic_irq_lr_map is actually used only for piggy-back optimization, and > can be easily replaced by iteration over lr_used. This is good because in > future, when LPI support is introduced, number of IRQs will grow up to at > least 16384, while numbers from 1024 to 8192 are never going to be used. > This would be a huge memory waste. > > In its turn, lr_used is also completely redundant since > ae705930fca6322600690df9dc1c7d0516145a93 ("arm/arm64: KVM: Keep elrsr/aisr > in sync with software model"), because together with lr_used we also update > elrsr. This allows to easily replace lr_used with elrsr, inverting all > conditions (because in elrsr '1' means 'free'). So this looks pretty good to me, probably a better (because less intrusive) solution than my first two patches of the ITS emulation, which have a very similar scope. I will give this some testing on my boxes here to spot any regressions, but I guess I will use these two patches as the base for my next version of the ITS emulation series. Christoffer, Marc, do you consider these for 4.4 (since they are an independent cleanup) or do you want them to be part of the ITS emulation series since they make more sense in there? Cheers, Andre. > > Signed-off-by: Pavel Fedin > --- > include/kvm/arm_vgic.h | 6 > virt/kvm/arm/vgic.c| 74 > +++--- > 2 files changed, 28 insertions(+), 52 deletions(-) > > diff --git a/include/kvm/arm_vgic.h b/include/kvm/arm_vgic.h > index 4e14dac..d908028 100644 > --- a/include/kvm/arm_vgic.h > +++ b/include/kvm/arm_vgic.h > @@ -296,9 +296,6 @@ struct vgic_v3_cpu_if { > }; > > struct vgic_cpu { > - /* per IRQ to LR mapping */ > - u8 *vgic_irq_lr_map; > - > /* Pending/active/both interrupts on this VCPU */ > DECLARE_BITMAP( pending_percpu, VGIC_NR_PRIVATE_IRQS); > DECLARE_BITMAP( active_percpu, VGIC_NR_PRIVATE_IRQS); > @@ -309,9 +306,6 @@ struct vgic_cpu { > unsigned long *active_shared; > unsigned long *pend_act_shared; > > - /* Bitmap of used/free list registers */ > - DECLARE_BITMAP( lr_used, VGIC_V2_MAX_LRS); > - > /* Number of list registers on this CPU */ > int nr_lr; > > diff --git a/virt/kvm/arm/vgic.c b/virt/kvm/arm/vgic.c > index 6bd1c9b..2f4d25a 100644 > --- a/virt/kvm/arm/vgic.c > +++ b/virt/kvm/arm/vgic.c > @@ -102,9 +102,10 @@ > #include "vgic.h" > > static void vgic_retire_disabled_irqs(struct kvm_vcpu *vcpu); > -static void vgic_retire_lr(int lr_nr, int irq, struct kvm_vcpu *vcpu); > +static void vgic_retire_lr(int lr_nr, struct kvm_vcpu *vcpu); > static struct vgic_lr vgic_get_lr(const struct kvm_vcpu *vcpu, int lr); > static void vgic_set_lr(struct kvm_vcpu *vcpu, int lr, struct vgic_lr > lr_desc); > +static u64 vgic_get_elrsr(struct kvm_vcpu *vcpu); > static struct irq_phys_map *vgic_irq_map_search(struct kvm_vcpu *vcpu, > int virt_irq); > > @@ -683,9 +684,11 @@ bool vgic_handle_cfg_reg(u32 *reg, struct kvm_exit_mmio > *mmio, > void vgic_unqueue_irqs(struct kvm_vcpu *vcpu) > { > struct vgic_cpu *vgic_cpu = &vcpu->arch.vgic_cpu; > + u64 elrsr = vgic_get_elrsr(vcpu); > + unsigned long *elrsr_ptr = u64_to_bitmask(&elrsr); > int i; > > - for_each_set_bit(i, vgic_cpu->lr_used, vgic_cpu->nr_lr) { > + for_each_clear_bit(i, elrsr_ptr, vgic_cpu->nr_lr) { > struct vgic_lr lr = vgic_get_lr(vcpu, i); > > /* > @@ -728,7 +731,7 @@ void vgic_unqueue_irqs(struct kvm_vcpu *vcpu) >* Mark the LR as free for other use. >*/ > BUG_ON(lr.state & LR_STATE_MASK); > - vgic_retire_lr(i, lr.irq, vcpu); > + vgic_retire_lr(i, vcpu); > vgic_irq_clear_queued(vcpu, lr.irq); > > /* Finally update the VGIC state. */ > @@ -1087,15 +1090,12 @@ static inline void vgic_enable(struct kvm_vcpu *vcpu) > vgic_ops->enable(vcpu); > } > > -static void vgic_retire_lr(int lr_nr, int irq, struct kvm_vcpu *vcpu) > +static void vgic_retire_lr(int lr_nr, struct kvm_vcpu *vcpu) > { > - struct vgic_cpu *vgic_cpu = &vcpu->arch.vgic_cpu; > struct vgic_lr vlr = vgic_get_lr(vcpu, lr_nr); > > vlr.state = 0; > vgic_set_lr(vcpu, lr_nr, vlr); > - clear_bit(lr_nr, vgic_cpu->lr_used); > - vgic_cpu->vgic_irq_lr_map[irq] = LR_EMPTY; > vgic_sync_lr_elrsr(vcpu, lr_nr, vlr); > } > > @@ -1110,14 +1110,15 @@ static void vgic_retire_lr(int lr_nr, int irq, struct > kvm_vcpu *vcpu) > */ > static void vgic_retire_disabled_irqs(struct kvm_vcpu *vcpu) > { > - struct vgic_cpu *vgic_cpu = &vcpu->arch.vgic_cpu; > + u64 elrsr = vgic_get_elrsr(vcpu); > + unsign
Re: [PATCH 1/2] KVM: arm/arm64: Optimize away redundant LR tracking
On Mon, Oct 12, 2015 at 05:56:14PM +0100, Andre Przywara wrote: > Hi, > > On 02/10/15 15:44, Pavel Fedin wrote: > > Currently we use vgic_irq_lr_map in order to track which LRs hold which > > IRQs, and lr_used bitmap in order to track which LRs are used or free. > > > > vgic_irq_lr_map is actually used only for piggy-back optimization, and > > can be easily replaced by iteration over lr_used. This is good because in > > future, when LPI support is introduced, number of IRQs will grow up to at > > least 16384, while numbers from 1024 to 8192 are never going to be used. > > This would be a huge memory waste. > > > > In its turn, lr_used is also completely redundant since > > ae705930fca6322600690df9dc1c7d0516145a93 ("arm/arm64: KVM: Keep elrsr/aisr > > in sync with software model"), because together with lr_used we also update > > elrsr. This allows to easily replace lr_used with elrsr, inverting all > > conditions (because in elrsr '1' means 'free'). > > So this looks pretty good to me, probably a better (because less > intrusive) solution than my first two patches of the ITS emulation, > which have a very similar scope. > I will give this some testing on my boxes here to spot any regressions, > but I guess I will use these two patches as the base for my next version > of the ITS emulation series. > > Christoffer, Marc, do you consider these for 4.4 (since they are an > independent cleanup) or do you want them to be part of the ITS emulation > series since they make more sense in there? > I'll try to have a look at these tomorrow and I'll think about how to merge this after I've looked at them. Did you review these to the point where you can give your r-b tag here? Thanks, -Christoffer -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH 1/2] KVM: arm/arm64: Optimize away redundant LR tracking
On Fri, Oct 02, 2015 at 05:44:28PM +0300, Pavel Fedin wrote: > Currently we use vgic_irq_lr_map in order to track which LRs hold which > IRQs, and lr_used bitmap in order to track which LRs are used or free. > > vgic_irq_lr_map is actually used only for piggy-back optimization, and > can be easily replaced by iteration over lr_used. This is good because in > future, when LPI support is introduced, number of IRQs will grow up to at > least 16384, while numbers from 1024 to 8192 are never going to be used. > This would be a huge memory waste. > > In its turn, lr_used is also completely redundant since > ae705930fca6322600690df9dc1c7d0516145a93 ("arm/arm64: KVM: Keep elrsr/aisr > in sync with software model"), because together with lr_used we also update > elrsr. This allows to easily replace lr_used with elrsr, inverting all > conditions (because in elrsr '1' means 'free'). > > Signed-off-by: Pavel Fedin > --- > include/kvm/arm_vgic.h | 6 > virt/kvm/arm/vgic.c| 74 > +++--- > 2 files changed, 28 insertions(+), 52 deletions(-) > > diff --git a/include/kvm/arm_vgic.h b/include/kvm/arm_vgic.h > index 4e14dac..d908028 100644 > --- a/include/kvm/arm_vgic.h > +++ b/include/kvm/arm_vgic.h > @@ -296,9 +296,6 @@ struct vgic_v3_cpu_if { > }; > > struct vgic_cpu { > - /* per IRQ to LR mapping */ > - u8 *vgic_irq_lr_map; > - > /* Pending/active/both interrupts on this VCPU */ > DECLARE_BITMAP( pending_percpu, VGIC_NR_PRIVATE_IRQS); > DECLARE_BITMAP( active_percpu, VGIC_NR_PRIVATE_IRQS); > @@ -309,9 +306,6 @@ struct vgic_cpu { > unsigned long *active_shared; > unsigned long *pend_act_shared; > > - /* Bitmap of used/free list registers */ > - DECLARE_BITMAP( lr_used, VGIC_V2_MAX_LRS); > - > /* Number of list registers on this CPU */ > int nr_lr; > > diff --git a/virt/kvm/arm/vgic.c b/virt/kvm/arm/vgic.c > index 6bd1c9b..2f4d25a 100644 > --- a/virt/kvm/arm/vgic.c > +++ b/virt/kvm/arm/vgic.c > @@ -102,9 +102,10 @@ > #include "vgic.h" > > static void vgic_retire_disabled_irqs(struct kvm_vcpu *vcpu); > -static void vgic_retire_lr(int lr_nr, int irq, struct kvm_vcpu *vcpu); > +static void vgic_retire_lr(int lr_nr, struct kvm_vcpu *vcpu); > static struct vgic_lr vgic_get_lr(const struct kvm_vcpu *vcpu, int lr); > static void vgic_set_lr(struct kvm_vcpu *vcpu, int lr, struct vgic_lr > lr_desc); > +static u64 vgic_get_elrsr(struct kvm_vcpu *vcpu); > static struct irq_phys_map *vgic_irq_map_search(struct kvm_vcpu *vcpu, > int virt_irq); > > @@ -683,9 +684,11 @@ bool vgic_handle_cfg_reg(u32 *reg, struct kvm_exit_mmio > *mmio, > void vgic_unqueue_irqs(struct kvm_vcpu *vcpu) > { > struct vgic_cpu *vgic_cpu = &vcpu->arch.vgic_cpu; > + u64 elrsr = vgic_get_elrsr(vcpu); > + unsigned long *elrsr_ptr = u64_to_bitmask(&elrsr); > int i; > > - for_each_set_bit(i, vgic_cpu->lr_used, vgic_cpu->nr_lr) { > + for_each_clear_bit(i, elrsr_ptr, vgic_cpu->nr_lr) { > struct vgic_lr lr = vgic_get_lr(vcpu, i); > > /* > @@ -728,7 +731,7 @@ void vgic_unqueue_irqs(struct kvm_vcpu *vcpu) >* Mark the LR as free for other use. >*/ > BUG_ON(lr.state & LR_STATE_MASK); > - vgic_retire_lr(i, lr.irq, vcpu); > + vgic_retire_lr(i, vcpu); > vgic_irq_clear_queued(vcpu, lr.irq); > > /* Finally update the VGIC state. */ > @@ -1087,15 +1090,12 @@ static inline void vgic_enable(struct kvm_vcpu *vcpu) > vgic_ops->enable(vcpu); > } > > -static void vgic_retire_lr(int lr_nr, int irq, struct kvm_vcpu *vcpu) > +static void vgic_retire_lr(int lr_nr, struct kvm_vcpu *vcpu) > { > - struct vgic_cpu *vgic_cpu = &vcpu->arch.vgic_cpu; > struct vgic_lr vlr = vgic_get_lr(vcpu, lr_nr); > > vlr.state = 0; > vgic_set_lr(vcpu, lr_nr, vlr); > - clear_bit(lr_nr, vgic_cpu->lr_used); > - vgic_cpu->vgic_irq_lr_map[irq] = LR_EMPTY; > vgic_sync_lr_elrsr(vcpu, lr_nr, vlr); > } > > @@ -1110,14 +1110,15 @@ static void vgic_retire_lr(int lr_nr, int irq, struct > kvm_vcpu *vcpu) > */ > static void vgic_retire_disabled_irqs(struct kvm_vcpu *vcpu) > { > - struct vgic_cpu *vgic_cpu = &vcpu->arch.vgic_cpu; > + u64 elrsr = vgic_get_elrsr(vcpu); > + unsigned long *elrsr_ptr = u64_to_bitmask(&elrsr); > int lr; > > - for_each_set_bit(lr, vgic_cpu->lr_used, vgic->nr_lr) { > + for_each_clear_bit(lr, elrsr_ptr, vgic->nr_lr) { > struct vgic_lr vlr = vgic_get_lr(vcpu, lr); > > if (!vgic_irq_is_enabled(vcpu, vlr.irq)) { > - vgic_retire_lr(lr, vlr.irq, vcpu); > + vgic_retire_lr(lr, vcpu); > if (vgic_irq_is_queued(vcpu, vlr.irq)) > vgic_irq_clear
RE: [PATCH 1/2] KVM: arm/arm64: Optimize away redundant LR tracking
Hello! > -Original Message- > From: Christoffer Dall [mailto:christoffer.d...@linaro.org] > Sent: Friday, October 23, 2015 12:43 AM > To: Pavel Fedin > Cc: kvm...@lists.cs.columbia.edu; kvm@vger.kernel.org; Marc Zyngier; Andre > Przywara > Subject: Re: [PATCH 1/2] KVM: arm/arm64: Optimize away redundant LR tracking > > On Fri, Oct 02, 2015 at 05:44:28PM +0300, Pavel Fedin wrote: > > Currently we use vgic_irq_lr_map in order to track which LRs hold which > > IRQs, and lr_used bitmap in order to track which LRs are used or free. > > > > vgic_irq_lr_map is actually used only for piggy-back optimization, and > > can be easily replaced by iteration over lr_used. This is good because in > > future, when LPI support is introduced, number of IRQs will grow up to at > > least 16384, while numbers from 1024 to 8192 are never going to be used. > > This would be a huge memory waste. > > > > In its turn, lr_used is also completely redundant since > > ae705930fca6322600690df9dc1c7d0516145a93 ("arm/arm64: KVM: Keep elrsr/aisr > > in sync with software model"), because together with lr_used we also update > > elrsr. This allows to easily replace lr_used with elrsr, inverting all > > conditions (because in elrsr '1' means 'free'). > > > > Signed-off-by: Pavel Fedin > > --- > > include/kvm/arm_vgic.h | 6 > > virt/kvm/arm/vgic.c| 74 > > +++--- > > 2 files changed, 28 insertions(+), 52 deletions(-) > > > > diff --git a/include/kvm/arm_vgic.h b/include/kvm/arm_vgic.h > > index 4e14dac..d908028 100644 > > --- a/include/kvm/arm_vgic.h > > +++ b/include/kvm/arm_vgic.h > > @@ -296,9 +296,6 @@ struct vgic_v3_cpu_if { > > }; > > > > struct vgic_cpu { > > - /* per IRQ to LR mapping */ > > - u8 *vgic_irq_lr_map; > > - > > /* Pending/active/both interrupts on this VCPU */ > > DECLARE_BITMAP( pending_percpu, VGIC_NR_PRIVATE_IRQS); > > DECLARE_BITMAP( active_percpu, VGIC_NR_PRIVATE_IRQS); > > @@ -309,9 +306,6 @@ struct vgic_cpu { > > unsigned long *active_shared; > > unsigned long *pend_act_shared; > > > > - /* Bitmap of used/free list registers */ > > - DECLARE_BITMAP( lr_used, VGIC_V2_MAX_LRS); > > - > > /* Number of list registers on this CPU */ > > int nr_lr; > > > > diff --git a/virt/kvm/arm/vgic.c b/virt/kvm/arm/vgic.c > > index 6bd1c9b..2f4d25a 100644 > > --- a/virt/kvm/arm/vgic.c > > +++ b/virt/kvm/arm/vgic.c > > @@ -102,9 +102,10 @@ > > #include "vgic.h" > > > > static void vgic_retire_disabled_irqs(struct kvm_vcpu *vcpu); > > -static void vgic_retire_lr(int lr_nr, int irq, struct kvm_vcpu *vcpu); > > +static void vgic_retire_lr(int lr_nr, struct kvm_vcpu *vcpu); > > static struct vgic_lr vgic_get_lr(const struct kvm_vcpu *vcpu, int lr); > > static void vgic_set_lr(struct kvm_vcpu *vcpu, int lr, struct vgic_lr > > lr_desc); > > +static u64 vgic_get_elrsr(struct kvm_vcpu *vcpu); > > static struct irq_phys_map *vgic_irq_map_search(struct kvm_vcpu *vcpu, > > int virt_irq); > > > > @@ -683,9 +684,11 @@ bool vgic_handle_cfg_reg(u32 *reg, struct > > kvm_exit_mmio *mmio, > > void vgic_unqueue_irqs(struct kvm_vcpu *vcpu) > > { > > struct vgic_cpu *vgic_cpu = &vcpu->arch.vgic_cpu; > > + u64 elrsr = vgic_get_elrsr(vcpu); > > + unsigned long *elrsr_ptr = u64_to_bitmask(&elrsr); > > int i; > > > > - for_each_set_bit(i, vgic_cpu->lr_used, vgic_cpu->nr_lr) { > > + for_each_clear_bit(i, elrsr_ptr, vgic_cpu->nr_lr) { > > struct vgic_lr lr = vgic_get_lr(vcpu, i); > > > > /* > > @@ -728,7 +731,7 @@ void vgic_unqueue_irqs(struct kvm_vcpu *vcpu) > > * Mark the LR as free for other use. > > */ > > BUG_ON(lr.state & LR_STATE_MASK); > > - vgic_retire_lr(i, lr.irq, vcpu); > > + vgic_retire_lr(i, vcpu); > > vgic_irq_clear_queued(vcpu, lr.irq); > > > > /* Finally update the VGIC state. */ > > @@ -1087,15 +1090,12 @@ static inline void vgic_enable(struct kvm_vcpu > > *vcpu) > > vgic_ops->enable(vcpu); > > } > > > > -static void vgic_retire_lr(int lr_nr, int irq, struct kvm_vcpu *vcpu) > > +static void vgic_retire_lr(int lr_nr, struct kvm_vcpu *vcpu) > > { > > - struct vgic_cpu *v