bug in kvm lapic 2.6.30

rtable

1. 
   
2. /* 2.6.30
   
3. * x86/kvm/lapic.c
   
4. */
   
5. 
   
6. static void apic_send_ipi(struct kvm_lapic *apic)
   
7. {
   
8.         u32 icr_low = apic_get_reg(apic, APIC_ICR);
   
9.         u32 icr_high = apic_get_reg(apic, APIC_ICR2);
   
10. 
    
11.         unsigned int dest = GET_APIC_DEST_FIELD(icr_high);
    
12. 
    
13.         unsigned int short_hand                = icr_low & APIC_SHORT_MASK;
    
14.         unsigned int trig_mode                = icr_low & APIC_INT_LEVELTRIG;
    
15.         unsigned int level                        = icr_low & APIC_INT_ASSERT;
    
16.         unsigned int dest_mode                = icr_low & APIC_DEST_MASK;
    
17.         unsigned int delivery_mode        = icr_low & APIC_MODE_MASK;
    
18.         unsigned int vector                        = icr_low & APIC_VECTOR_MASK;
    
19. 
    
20.         struct kvm_vcpu *target;
    
21.         struct kvm_vcpu *vcpu;
    
22.         unsigned long lpr_map = 0;
    
23.         int i;
    
24. 
    
25.         apic_debug("icr_high 0x%x, icr_low 0x%x, "
    
26.                    "short_hand 0x%x, dest 0x%x, trig_mode 0x%x, level 0x%x, "
    
27.                    "dest_mode 0x%x, delivery_mode 0x%x, vector 0x%x\n",
    
28.                    icr_high, icr_low, short_hand, dest,
    
29.                    trig_mode, level, dest_mode, delivery_mode, vector);
    
30. 
    
31.         for (i = 0; i < KVM_MAX_VCPUS; i++) {
    
32.                 vcpu = apic->vcpu->kvm->vcpus[i];
    
33.                 if (!vcpu)
    
34.                         continue;
    
35. 
    
36.                 if (vcpu->arch.apic &&
    
37.                     apic_match_dest(vcpu, apic, short_hand, dest, dest_mode)) {
    
38.                         if (delivery_mode == APIC_DM_LOWEST)
    
39.                                 set_bit(vcpu->vcpu_id, &lpr_map);
    
40.                         else
    
41.                                 __apic_accept_irq(vcpu->arch.apic, delivery_mode,
    
42.                                                                   vector, level, trig_mode);
    
43.                 }
    
44.         }
    
45. 
    
46.         if (delivery_mode == APIC_DM_LOWEST) {
    
47. -                target = kvm_get_lowest_prio_vcpu(vcpu->kvm, vector, lpr_map);
    
48. +                target = kvm_get_lowest_prio_vcpu(apic->vcpu->kvm, vector, lpr_map);
    
49.                 if (target != NULL)
    
50.                         __apic_accept_irq(target->arch.apic, delivery_mode,
    
51.                                                           vector, level, trig_mode);
    
52.         }
    
53. }
    

复制代码

rtable

not a bug

1. 
   
2. static u32 apic_get_tmcct(struct kvm_lapic *apic)
   
3. {
   
4.         ktime_t remaining;
   
5.         s64 ns;
   
6.         u32 tmcct; //cct cycle count
   
7. 
   
8.         ASSERT(apic != NULL);
   
9. 
   
10.         /* if initial count is 0, current count should also be 0 */
    
11.         if (apic_get_reg(apic, APIC_TMICT) == 0)
    
12.                 return 0;
    
13. 
    
14.         remaining = hrtimer_expires_remaining(&apic->timer.dev);
    
15.         if (ktime_to_ns(remaining) < 0)
    
16. -                remaining = ktime_set(0, 0);
    
17. +                return 0;
    
18. 
    
19.         ns = mod_64(ktime_to_ns(remaining), apic->timer.period);
    
20. 
    
21.         tmcct = div64_u64(ns, (APIC_BUS_CYCLE_NS * apic->timer.divide_count));
    
22.         return tmcct;
    
23. }
    

复制代码

rtable

and patch it

1. 
   
2. static u32 apic_get_tmcct(struct kvm_lapic *apic)
   
3. {
   
4.         ktime_t remaining;
   
5.         s64 ns;
   
6.         u32 tmcct; //cct cycle count
   
7. 
   
8.         ASSERT(apic != NULL);
   
9. 
   
10.         /* if initial count is 0, current count should also be 0 */
    
11.         if (apic_get_reg(apic, APIC_TMICT) == 0)
    
12.                 return 0;
    
13. 
    
14.         remaining = hrtimer_expires_remaining(&apic->timer.dev);
    
15. -        if (ktime_to_ns(remaining) < 0)
    
16. -                remaining = ktime_set(0, 0);
    
17. -
    
18. -        ns = mod_64(ktime_to_ns(remaining), apic->timer.period);
    
19. +        ns = ktime_to_ns(remaining);
    
20. +        if (ns < 0)
    
21. +                return 0;
    
22. +        ns = mod_64(ns, apic->timer.period);
    
23. 
    
24.         tmcct = div64_u64(ns, (APIC_BUS_CYCLE_NS * apic->timer.divide_count));
    
25.         return tmcct;
    
26. }
    

复制代码

[ 本帖最后由 rtable 于 2009-7-2 22:03 编辑 ]

rtable

patch

1. 
   
2. unsigned long segment_base(u16 selector)
   
3. {
   
4.         struct descriptor_table gdt;
   
5.         struct desc_struct *d;
   
6.         unsigned long table_base;
   
7.         unsigned long v;
   
8. 
   
9.         if (selector == 0)
   
10.                 return 0;
    
11. 
    
12. -        asm("sgdt %0" : "=m"(gdt));
    
13. -        table_base = gdt.base;
    
14. -
    
15.         if (selector & 4) { /* from ldt */
    
16.                 u16 ldt_selector;
    
17. 
    
18.                 asm("sldt %0" : "=g"(ldt_selector));
    
19.                 table_base = segment_base(ldt_selector);
    
20.         }
    
21. +        else {
    
22. +                asm("sgdt %0" : "=m"(gdt));
    
23. +                table_base = gdt.base;
    
24. +        }
    
25.         d = (struct desc_struct *)(table_base + (selector & ~7));
    
26.         v = d->base0 | ((unsigned long)d->base1 << 16) |
    
27.                 ((unsigned long)d->base2 << 24);
    
28. #ifdef CONFIG_X86_64
    
29.         if (d->s == 0 && (d->type == 2 || d->type == 9 || d->type == 11))
    
30.                 v |= ((unsigned long)((struct ldttss_desc64 *)d)->base3) << 32;
    
31. #endif
    
32.         return v;
    
33. }
    
34. EXPORT_SYMBOL_GPL(segment_base);
    
35. 
    
36. u64 kvm_get_apic_base(struct kvm_vcpu *vcpu)
    
37. {
    
38. -        if (irqchip_in_kernel(vcpu->kvm))
    
39. -                return vcpu->arch.apic_base;
    
40. -        else
    
41.                 return vcpu->arch.apic_base;
    
42. }
    
43. EXPORT_SYMBOL_GPL(kvm_get_apic_base);
    
44. 
    

复制代码

[ 本帖最后由 rtable 于 2009-7-2 20:39 编辑 ]

rtable

patch

1. 
   
2. /*
   
3. * Load the pae pdptrs.  Return true is they are all valid.
   
4. */
   
5. int load_pdptrs(struct kvm_vcpu *vcpu, unsigned long cr3)
   
6. {
   
7.         gfn_t pdpt_gfn = cr3 >> PAGE_SHIFT;
   
8.         unsigned offset = ((cr3 & (PAGE_SIZE-1)) >> 5) << 2;
   
9.         int i;
   
10.         int ret;
    
11.         u64 pdpte[ARRAY_SIZE(vcpu->arch.pdptrs)];
    
12. 
    
13.         ret = kvm_read_guest_page(vcpu->kvm, pdpt_gfn,
    
14.                                                                 pdpte, offset * sizeof(u64), sizeof(pdpte));
    
15.         if (ret < 0) {
    
16.                 ret = 0;
    
17.                 goto out;
    
18.         }
    
19. -        for (i = 0; i < ARRAY_SIZE(pdpte); ++i) {
    
20. -                //since logic err, del the for loop
    
21. -                if ((pdpte[i] & 1) && (pdpte[i] & 0xfffffff0000001e6ull)) {
    
22. -                        ret = 0;
    
23. -                        goto out;
    
24. -                }
    
25. -        }
    
26.         ret = 1;
    
27. 
    
28.         memcpy(vcpu->arch.pdptrs, pdpte, sizeof(vcpu->arch.pdptrs));
    
29. out:
    
30.         return ret;
    
31. }
    
32. EXPORT_SYMBOL_GPL(load_pdptrs);
    

复制代码

Godbach

LZ描述一下BUG

rtable

concern

1. 
   
2. void kvm_inject_page_fault(struct kvm_vcpu *vcpu, unsigned long addr, u32 error_code)
   
3. {
   
4.         ++vcpu->stat.pf_guest;
   
5. 
   
6.         if (vcpu->arch.exception.pending) {
   
7.                 if (vcpu->arch.exception.nr == PF_VECTOR) {
   
8.                         printk(KERN_DEBUG "kvm: inject_page_fault: double fault 0x%lx\n", addr);
   
9.                         vcpu->arch.exception.nr = DF_VECTOR;
   
10.                         vcpu->arch.exception.error_code = 0;
    
11.                 }
    
12.                 else if (vcpu->arch.exception.nr == DF_VECTOR) {
    
13.                         /* triple fault -> shutdown */
    
14.                         set_bit(KVM_REQ_TRIPLE_FAULT, &vcpu->requests);
    
15.                 }
    
16. +                /* else
    
17. +                        current fault lost,
    
18. +                        is deferred until pending processed?
    
19. +                */
    
20.                 return;
    
21.         }
    
22.         vcpu->arch.cr2 = addr;
    
23.         kvm_queue_exception_e(vcpu, PF_VECTOR, error_code);
    
24. }
    
25. 
    

复制代码

fido_zy

就是，拿文字描述下BUG，再贴下PATCH

rtable

patch

1. 
   
2. u32 kvm_ioapic_get_delivery_bitmask(struct kvm_ioapic *ioapic, u8 dest,
   
3.                                                                     u8 dest_mode)
   
4. {
   
5.         u32 mask = 0;
   
6.         int i;
   
7.         struct kvm *kvm = ioapic->kvm;
   
8.         struct kvm_vcpu *vcpu;
   
9. 
   
10.         ioapic_debug("dest %d dest_mode %d\n", dest, dest_mode);
    
11. 
    
12.         if (dest_mode == 0) {        /* Physical mode. */
    
13.                 if (dest == 0xFF) {        /* Broadcast. */
    
14.                         for (i = 0; i < KVM_MAX_VCPUS; ++i)
    
15.                                 if (kvm->vcpus[i] && kvm->vcpus[i]->arch.apic)
    
16.                                         mask |= 1 << i;
    
17.                         return mask;
    
18.                 }
    
19.                 for (i = 0; i < KVM_MAX_VCPUS; ++i) {
    
20.                         vcpu = kvm->vcpus[i];
    
21. -                        if (!vcpu)
    
22. -                                continue;
    
23. -                        if (kvm_apic_match_physical_addr(vcpu->arch.apic, dest)) {
    
24. -                                if (vcpu->arch.apic)
    
25. -                                        mask = 1 << i;
    
26. -                                break;
    
27. -                        }
    
28. +                        if (vcpu && vcpu->arch.apic &&
    
29. +                                kvm_apic_match_physical_addr(vcpu->arch.apic, dest)) {
    
30. +                                mask = 1 << i;
    
31. +                                break;
    
32. +                        }
    
33.                 }
    
34.         }
    
35.         else if (dest != 0)        /* Logical mode, MDA non-zero. */
    
36.                 for (i = 0; i < KVM_MAX_VCPUS; ++i) {
    
37.                         vcpu = kvm->vcpus[i];
    
38. -                        if (!vcpu)
    
39. -                                continue;
    
40. -                        if (vcpu->arch.apic &&
    
41. +                        if (vcpu && vcpu->arch.apic &&
    
42.                             kvm_apic_match_logical_addr(vcpu->arch.apic, dest))
    
43.                                 mask |= 1 << vcpu->vcpu_id;
    
44.                 }
    
45. 
    
46.         ioapic_debug("mask %x\n", mask);
    
47.         return mask;
    
48. }
    

复制代码

fido_zy

原帖由 rtable 于 2009-7-13 09:43 发表
patch
u32 kvm_ioapic_get_delivery_bitmask(struct kvm_ioapic *ioapic, u8 dest,
                                                                    u8 dest_mode)
{
        u32 mask = 0;
        int i;
        struct kvm *kvm = ioapic->kvm;
        struct kvm_vcpu *vcpu;

...

这么改，不遍历剩下的VCPU 就break了，这不是会有问题吗？