X-Git-Url: http://v3vee.org/palacios/gitweb/gitweb.cgi?a=blobdiff_plain;f=palacios%2Fsrc%2Fdevices%2Fapic.c;h=3d7f7925f01cef1188016a1e21b1406e0c8a0a5e;hb=b187d8db6f451cd5cea5452b284c3282dd5a4b67;hp=234c2fce2f60ee8b995fd5ec9b9f005bca6035d7;hpb=30434aee50c56ff4a23400191562c5c9ffaff830;p=palacios.releases.git diff --git a/palacios/src/devices/apic.c b/palacios/src/devices/apic.c index 234c2fc..3d7f792 100644 --- a/palacios/src/devices/apic.c +++ b/palacios/src/devices/apic.c @@ -33,14 +33,14 @@ #endif #ifdef CONFIG_DEBUG_APIC -static char *shorthand_str[] = { +static char * shorthand_str[] = { "(no shorthand)", "(self)", "(all)", "(all-but-me)", }; -static char *deliverymode_str[] = { +static char * deliverymode_str[] = { "(fixed)", "(lowest priority)", "(SMI)", @@ -135,10 +135,6 @@ typedef enum { APIC_TMR_INT, APIC_THERM_INT, APIC_PERF_INT, #define EXT_INT_LOC_VEC_TBL_OFFSET2 0x520 // 0x500 - 0x530 #define EXT_INT_LOC_VEC_TBL_OFFSET3 0x530 // 0x500 - 0x530 - - - - struct apic_msr { union { uint64_t value; @@ -211,6 +207,8 @@ struct apic_state { struct guest_info * core; + struct v3_timer * timer; + uint32_t eoi; v3_lock_t lock; @@ -560,7 +558,7 @@ static int should_deliver_ipi(struct guest_info * dst_core, return 1; } - return should_deliver_cluster_ipi(dst_core, dst_apic, mda); + return should_deliver_flat_ipi(dst_core, dst_apic, mda); } else if (dst_apic->dst_fmt.model == 0x0) { if (mda == 0xff) { @@ -568,7 +566,7 @@ static int should_deliver_ipi(struct guest_info * dst_core, return 1; } - return should_deliver_flat_ipi(dst_core, dst_apic, mda); + return should_deliver_cluster_ipi(dst_core, dst_apic, mda); } else { PrintError("apic %u core %u: invalid destination format register value 0x%x for logical mode delivery.\n", dst_apic->lapic_id.val, dst_core->cpu_id, dst_apic->dst_fmt.model); @@ -709,7 +707,6 @@ static int route_ipi(struct apic_dev_state * apic_dev, icr->dst, icr->val); - switch (icr->dst_shorthand) { case 0: // no shorthand @@ -1069,8 +1066,8 @@ static int apic_write(struct guest_info * core, addr_t guest_addr, void * src, u PrintDebug("apic %u: core %u: at %p and priv_data is at %p\n", apic->lapic_id.val, core->cpu_id, apic, priv_data); - PrintDebug("Write to address space (%p) (val=%x)\n", - (void *)guest_addr, *(uint32_t *)src); + PrintDebug("apic %u: core %u: write to address space (%p) (val=%x)\n", + apic->lapic_id.val, core->cpu_id, (void *)guest_addr, *(uint32_t *)src); if (msr->apic_enable == 0) { PrintError("apic %u: core %u: Write to APIC address space with disabled APIC, apic msr=0x%llx\n", @@ -1293,9 +1290,8 @@ static int apic_get_intr_number(struct guest_info * core, void * private_data) { } -int v3_apic_send_ipi(struct v3_vm_info * vm, struct vm_device * dev, - struct v3_gen_ipi * ipi) { - struct apic_dev_state * apic_dev = (struct apic_dev_state *)(dev->private_data); +int v3_apic_send_ipi(struct v3_vm_info * vm, struct v3_gen_ipi * ipi, void * dev_data) { + struct apic_dev_state * apic_dev = (struct apic_dev_state *)dev_data; struct int_cmd_reg tmp_icr; // zero out all the fields @@ -1314,9 +1310,8 @@ int v3_apic_send_ipi(struct v3_vm_info * vm, struct vm_device * dev, } -int v3_apic_raise_intr(struct v3_vm_info * vm, struct vm_device * dev, - uint32_t irq, uint32_t dst) { - struct apic_dev_state * apic_dev = (struct apic_dev_state *)(dev->private_data); +int v3_apic_raise_intr(struct v3_vm_info * vm, uint32_t irq, uint32_t dst, void * dev_data) { + struct apic_dev_state * apic_dev = (struct apic_dev_state *)(dev_data); struct apic_state * apic = &(apic_dev->apics[dst]); PrintDebug("apic %u core ?: raising interrupt IRQ %u (dst = %u).\n", apic->lapic_id.val, irq, dst); @@ -1460,30 +1455,42 @@ static struct intr_ctrl_ops intr_ops = { }; -static struct vm_timer_ops timer_ops = { +static struct v3_timer_ops timer_ops = { .update_timer = apic_update_time, }; -static int apic_free(struct vm_device * dev) { +static int apic_free(struct apic_dev_state * apic_dev) { + int i = 0; + struct v3_vm_info * vm = NULL; - /* TODO: This should crosscall to force an unhook on each CPU */ + for (i = 0; i < apic_dev->num_apics; i++) { + struct apic_state * apic = &(apic_dev->apics[i]); + struct guest_info * core = apic->core; + + vm = core->vm_info; + + // unregister intr controller + + if (apic->timer) { + v3_remove_timer(core, apic->timer); + } - // struct apic_state * apic = (struct apic_state *)dev->private_data; + // unhook memory - v3_unhook_msr(dev->vm, BASE_ADDR_MSR); + } + + v3_unhook_msr(vm, BASE_ADDR_MSR); + V3_Free(apic_dev); return 0; } static struct v3_device_ops dev_ops = { - .free = apic_free, - .reset = NULL, - .start = NULL, - .stop = NULL, + .free = (int (*)(void *))apic_free, }; @@ -1502,10 +1509,11 @@ static int apic_init(struct v3_vm_info * vm, v3_cfg_tree_t * cfg) { apic_dev->num_apics = vm->num_cores; - struct vm_device * dev = v3_allocate_device(dev_id, &dev_ops, apic_dev); + struct vm_device * dev = v3_add_device(vm, dev_id, &dev_ops, apic_dev); - if (v3_attach_device(vm, dev) == -1) { + if (dev == NULL) { PrintError("apic: Could not attach device %s\n", dev_id); + V3_Free(apic_dev); return -1; } @@ -1520,7 +1528,13 @@ static int apic_init(struct v3_vm_info * vm, v3_cfg_tree_t * cfg) { v3_register_intr_controller(core, &intr_ops, apic_dev); - v3_add_timer(core, &timer_ops, apic_dev); + apic->timer = v3_add_timer(core, &timer_ops, apic_dev); + + if (apic->timer == NULL) { + PrintError("APIC: Failed to attach timer to core %d\n", i); + v3_remove_device(dev); + return -1; + } v3_hook_full_mem(vm, core->cpu_id, apic->base_addr, apic->base_addr + PAGE_SIZE_4KB, apic_read, apic_write, apic_dev);