On 23 May 2018 at 04:53, Shannon Zhao <zhaoshengl...@huawei.com> wrote: > It forgot to increase clroffset during the loop. So it only clear the > first 4 bytes. > > Fixes: 367b9f527becdd20ddf116e17a3c0c2bbc486920 > Cc: qemu-sta...@nongnu.org > Signed-off-by: Shannon Zhao <zhaoshengl...@huawei.com> > Reviewed-by: Eric Auger <eric.au...@redhat.com> > --- > Changes in V3: Add reviewed-by tag > --- > hw/intc/arm_gicv3_kvm.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/hw/intc/arm_gicv3_kvm.c b/hw/intc/arm_gicv3_kvm.c > index ec37177..3536795 100644 > --- a/hw/intc/arm_gicv3_kvm.c > +++ b/hw/intc/arm_gicv3_kvm.c > @@ -243,6 +243,7 @@ static void kvm_dist_putbmp(GICv3State *s, uint32_t > offset, > if (clroffset != 0) { > reg = 0; > kvm_gicd_access(s, clroffset, ®, true); > + clroffset += 4; > } > reg = *gic_bmp_ptr32(bmp, irq); > kvm_gicd_access(s, offset, ®, true); > --
Applied this patch to target-arm.next. PS: if you could remember to always provide a cover letter for multi-patch series that would be helpful. thanks -- PMM