• Fix irq route entries exceeding KVM_MAX_IRQ_ROUTES · bdf02631
    马文霜 提交于
    Last month, we experienced several guests crash(6cores-8cores), qemu logs
    display the following messages:
    
    qemu-system-x86_64: /build/qemu-2.1.2/kvm-all.c:976:
    kvm_irqchip_commit_routes: Assertion `ret == 0' failed.
    
    After analysis and verification, we can confirm it's irq-balance
    daemon(in guest) leads to the assertion failure. Start a 8 core guest with
    two disks, execute the following scripts will reproduce the BUG quickly:
    
    irq_affinity.sh
    ========================================================================
    
    vda_irq_num=25
    vdb_irq_num=27
    while [ 1 ]
    do
        for irq in {1,2,4,8,10,20,40,80}
            do
                echo $irq > /proc/irq/$vda_irq_num/smp_affinity
                echo $irq > /proc/irq/$vdb_irq_num/smp_affinity
                dd if=/dev/vda of=/dev/zero bs=4K count=100 iflag=direct
                dd if=/dev/vdb of=/dev/zero bs=4K count=100 iflag=direct
            done
    done
    ========================================================================
    
    QEMU setup static irq route entries in kvm_pc_setup_irq_routing(), PIC and
    IOAPIC share the first 15 GSI numbers, take up 23 GSI numbers, but take up
    38 irq route entries. When change irq smp_affinity in guest, a dynamic route
    entry may be setup, the current logic is: if allocate GSI number succeeds,
    a new route entry can be added. The available dynamic GSI numbers is
    1021(KVM_MAX_IRQ_ROUTES-23), but available irq route entries is only
    986(KVM_MAX_IRQ_ROUTES-38), GSI numbers greater than route entries.
    irq-balance's behavior will eventually leads to total irq route entries
    exceed KVM_MAX_IRQ_ROUTES, ioctl(KVM_SET_GSI_ROUTING) fail and
    kvm_irqchip_commit_routes() trigger assertion failure.
    
    This patch fix the BUG.
    Signed-off-by: NWenshuang Ma <kevinnma@tencent.com>
    Cc: qemu-stable@nongnu.org
    Signed-off-by: NPaolo Bonzini <pbonzini@redhat.com>
    bdf02631
kvm-all.c 58.9 KB