[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 07/10] target-mips: kvm: Add main KVM support
From: |
Aurelien Jarno |
Subject: |
Re: [Qemu-devel] [PATCH v2 07/10] target-mips: kvm: Add main KVM support for MIPS |
Date: |
Sat, 21 Dec 2013 16:35:13 +0100 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Mon, Dec 16, 2013 at 02:12:42PM +0000, James Hogan wrote:
> From: Sanjay Lal <address@hidden>
>
> Implement the main KVM arch API for MIPS.
>
> Signed-off-by: Sanjay Lal <address@hidden>
> Signed-off-by: James Hogan <address@hidden>
> Cc: Aurelien Jarno <address@hidden>
> Cc: Gleb Natapov <address@hidden>
> Cc: Paolo Bonzini <address@hidden>
> ---
> Changes in v2:
> - Expand commit message
> - Checkpatch cleanups.
> - Some interrupt bug fixes from Yann Le Du <address@hidden>
> - Add get/set register functionality from Yann Le Du <address@hidden>
> - Use new 64 bit compatible ABI from Cavium from Sanjay Lal
> <address@hidden>
> - Add dummy kvm_arch_init_irq_routing()
> The common KVM code insists on calling kvm_arch_init_irq_routing() as
> soon as it sees kernel header support for it (regardless of whether
> QEMU supports it). Provide a dummy function to satisfy this.
> - Remove request_interrupt_window code (Peter Maydell)
> ---
> target-mips/kvm.c | 463
> +++++++++++++++++++++++++++++++++++++++++++++++++
> target-mips/kvm_mips.h | 28 +++
> 2 files changed, 491 insertions(+)
> create mode 100644 target-mips/kvm.c
> create mode 100644 target-mips/kvm_mips.h
>
> diff --git a/target-mips/kvm.c b/target-mips/kvm.c
> new file mode 100644
> index 0000000..951959b
> --- /dev/null
> +++ b/target-mips/kvm.c
> @@ -0,0 +1,463 @@
> +/*
> + * This file is subject to the terms and conditions of the GNU General Public
> + * License. See the file "COPYING" in the main directory of this archive
> + * for more details.
> + *
> + * KVM/MIPS: MIPS specific KVM APIs
> + *
> + * Copyright (C) 2012-2013 Imagination Technologies Ltd.
> + * Authors: Sanjay Lal <address@hidden>
> +*/
> +
> +#include <sys/types.h>
> +#include <sys/ioctl.h>
> +#include <sys/mman.h>
> +
> +#include <linux/kvm.h>
> +
> +#include "qemu-common.h"
> +#include "qemu/timer.h"
> +#include "sysemu/sysemu.h"
> +#include "sysemu/kvm.h"
> +#include "cpu.h"
> +#include "sysemu/cpus.h"
> +#include "kvm_mips.h"
> +
> +#define DEBUG_KVM 0
> +
> +#define dprintf(fmt, ...) \
> + do { if (DEBUG_KVM) { fprintf(stderr, fmt, ## __VA_ARGS__); } } while (0)
> +
> +const KVMCapabilityInfo kvm_arch_required_capabilities[] = {
> + KVM_CAP_LAST_INFO
> +};
> +
> +unsigned long kvm_arch_vcpu_id(CPUState *cpu)
> +{
> + return cpu->cpu_index;
> +}
> +
> +int kvm_arch_init(KVMState *s)
> +{
> + dprintf("%s\n", __func__);
> + return 0;
> +}
> +
> +int kvm_arch_init_vcpu(CPUState *env)
> +{
> + int ret = 0;
> + dprintf("%s\n", __func__);
> + return ret;
> +}
> +
> +void kvm_arch_reset_vcpu(CPUState *env)
> +{
> + dprintf("%s\n", __func__);
> +}
> +
> +int kvm_arch_put_registers(CPUState *cs, int level)
> +{
> + MIPSCPU *cpu = MIPS_CPU(cs);
> + CPUMIPSState *env = &cpu->env;
> + struct kvm_regs regs;
> + int ret;
> + int i;
> +
> + /* Set the registers based on QEMU's view of things */
> + for (i = 0; i < 32; i++) {
> + regs.gpr[i] = env->active_tc.gpr[i];
> + }
> +
> + regs.hi = env->active_tc.HI[0];
> + regs.lo = env->active_tc.LO[0];
> + regs.pc = env->active_tc.PC;
> +
> + ret = kvm_vcpu_ioctl(cs, KVM_SET_REGS, ®s);
> +
> + if (ret < 0) {
> + return ret;
> + }
> +
> + ret = kvm_mips_te_put_cp0_registers(cs, KVM_PUT_FULL_STATE);
> + if (ret < 0) {
> + return ret;
> + }
> +
> + return ret;
> +}
> +
> +int kvm_arch_get_registers(CPUState *cs)
> +{
> + MIPSCPU *cpu = MIPS_CPU(cs);
> + CPUMIPSState *env = &cpu->env;
> + int ret = 0;
> + struct kvm_regs regs;
> + int i;
> +
> + /* Get the current register set as KVM seems it */
> + ret = kvm_vcpu_ioctl(cs, KVM_GET_REGS, ®s);
> +
> + if (ret < 0) {
> + return ret;
> + }
> +
> + for (i = 0; i < 32; i++) {
> + env->active_tc.gpr[i] = regs.gpr[i];
> + }
> +
> + env->active_tc.HI[0] = regs.hi;
> + env->active_tc.LO[0] = regs.lo;
> + env->active_tc.PC = regs.pc;
> +
> + kvm_mips_te_get_cp0_registers(cs);
> +
> + return ret;
> +}
> +
> +int kvm_arch_insert_sw_breakpoint(CPUState *env, struct kvm_sw_breakpoint
> *bp)
> +{
> + dprintf("%s\n", __func__);
> + return 0;
> +}
> +
> +int kvm_arch_remove_sw_breakpoint(CPUState *env, struct kvm_sw_breakpoint
> *bp)
> +{
> + dprintf("%s\n", __func__);
> + return 0;
> +}
> +
> +static inline int cpu_mips_io_interrupts_pending(CPUArchState *env)
> +{
> + dprintf("%s: %#x\n", __func__, env->CP0_Cause & (1 << (2 + CP0Ca_IP)));
> + return env->CP0_Cause & (0x1 << (2 + CP0Ca_IP));
> +}
> +
> +
> +void kvm_arch_pre_run(CPUState *cs, struct kvm_run *run)
> +{
> + MIPSCPU *cpu = MIPS_CPU(cs);
> + CPUMIPSState *env = &cpu->env;
> + int r;
> + struct kvm_mips_interrupt intr;
> +
> + if ((cs->interrupt_request & CPU_INTERRUPT_HARD) &&
> + (cpu_mips_io_interrupts_pending(env))) {
> + intr.cpu = -1;
> + intr.irq = 2;
> + r = kvm_vcpu_ioctl(cs, KVM_INTERRUPT, &intr);
> + if (r < 0) {
> + printf("cpu %d fail inject %x\n", cs->cpu_index, intr.irq);
> + }
> + }
> +}
> +
> +void kvm_arch_post_run(CPUState *env, struct kvm_run *run)
> +{
> + dprintf("%s\n", __func__);
> +}
> +
> +int kvm_arch_process_async_events(CPUState *cs)
> +{
> + return cs->halted;
> +}
> +
> +int kvm_arch_handle_exit(CPUState *env, struct kvm_run *run)
> +{
> + int ret;
> +
> + dprintf("%s\n", __func__);
> + switch (run->exit_reason) {
> + default:
> + fprintf(stderr, "KVM: unknown exit reason %d\n", run->exit_reason);
> + ret = -1;
> + break;
> + }
> +
> + return ret;
> +}
> +
> +bool kvm_arch_stop_on_emulation_error(CPUState *env)
> +{
> + dprintf("%s\n", __func__);
> + return true;
> +}
> +
> +int kvm_arch_on_sigbus_vcpu(CPUState *env, int code, void *addr)
> +{
> + dprintf("%s\n", __func__);
> + return 1;
> +}
> +
> +int kvm_arch_on_sigbus(int code, void *addr)
> +{
> + dprintf("%s\n", __func__);
> + return 1;
> +}
> +
> +void kvm_arch_init_irq_routing(KVMState *s)
> +{
> +}
> +
> +int kvm_mips_set_interrupt(CPUMIPSState *env, int irq, int level)
> +{
> + CPUState *cs = ENV_GET_CPU(env);
> + struct kvm_mips_interrupt intr;
> +
> + if (!kvm_enabled()) {
> + return 0;
> + }
> +
> + intr.cpu = -1;
> +
> + if (level) {
> + intr.irq = irq;
> + } else {
> + intr.irq = -irq;
> + }
> +
> + kvm_vcpu_ioctl(cs, KVM_INTERRUPT, &intr);
> +
> + return 0;
> +}
> +
> +int kvm_mips_set_ipi_interrupt(CPUArchState *env, int irq, int level)
> +{
> + CPUState *cs = current_cpu;
> + CPUState *dest_cs = ENV_GET_CPU(env);
> + struct kvm_mips_interrupt intr;
> +
> + if (!kvm_enabled()) {
> + return 0;
> + }
> +
> + intr.cpu = dest_cs->cpu_index;
> +
> + if (level) {
> + intr.irq = irq;
> + } else {
> + intr.irq = -irq;
> + }
> +
> + dprintf("%s: CPU %d, IRQ: %d\n", __func__, intr.cpu, intr.irq);
> +
> + kvm_vcpu_ioctl(cs, KVM_INTERRUPT, &intr);
> +
> + return 0;
> +}
> +
> +#define KVM_REG_MIPS_CP0_INDEX (0x10000 + (8 * 0) + 0)
> +#define KVM_REG_MIPS_CP0_ENTRYLO0 (0x10000 + (8 * 2) + 0)
> +#define KVM_REG_MIPS_CP0_ENTRYLO1 (0x10000 + (8 * 3) + 0)
> +#define KVM_REG_MIPS_CP0_CONTEXT (0x10000 + (8 * 4) + 0)
> +#define KVM_REG_MIPS_CP0_USERLOCAL (0x10000 + (8 * 4) + 2)
> +#define KVM_REG_MIPS_CP0_PAGEMASK (0x10000 + (8 * 5) + 0)
> +#define KVM_REG_MIPS_CP0_PAGEGRAIN (0x10000 + (8 * 5) + 1)
> +#define KVM_REG_MIPS_CP0_WIRED (0x10000 + (8 * 6) + 0)
> +#define KVM_REG_MIPS_CP0_HWRENA (0x10000 + (8 * 7) + 0)
> +#define KVM_REG_MIPS_CP0_BADVADDR (0x10000 + (8 * 8) + 0)
> +#define KVM_REG_MIPS_CP0_COUNT (0x10000 + (8 * 9) + 0)
> +#define KVM_REG_MIPS_CP0_ENTRYHI (0x10000 + (8 * 10) + 0)
> +#define KVM_REG_MIPS_CP0_COMPARE (0x10000 + (8 * 11) + 0)
> +#define KVM_REG_MIPS_CP0_STATUS (0x10000 + (8 * 12) + 0)
> +#define KVM_REG_MIPS_CP0_INTCTL (0x10000 + (8 * 12) + 1)
> +#define KVM_REG_MIPS_CP0_CAUSE (0x10000 + (8 * 13) + 0)
> +#define KVM_REG_MIPS_CP0_EPC (0x10000 + (8 * 14) + 0)
> +#define KVM_REG_MIPS_CP0_PRID (0x10000 + (8 * 15) + 0)
> +#define KVM_REG_MIPS_CP0_EBASE (0x10000 + (8 * 15) + 1)
> +#define KVM_REG_MIPS_CP0_CONFIG (0x10000 + (8 * 16) + 0)
> +#define KVM_REG_MIPS_CP0_CONFIG1 (0x10000 + (8 * 16) + 1)
> +#define KVM_REG_MIPS_CP0_CONFIG2 (0x10000 + (8 * 16) + 2)
> +#define KVM_REG_MIPS_CP0_CONFIG3 (0x10000 + (8 * 16) + 3)
> +#define KVM_REG_MIPS_CP0_CONFIG4 (0x10000 + (8 * 16) + 4)
> +#define KVM_REG_MIPS_CP0_CONFIG5 (0x10000 + (8 * 16) + 5)
> +#define KVM_REG_MIPS_CP0_CONFIG7 (0x10000 + (8 * 16) + 7)
> +#define KVM_REG_MIPS_CP0_XCONTEXT (0x10000 + (8 * 20) + 0)
> +#define KVM_REG_MIPS_CP0_ERROREPC (0x10000 + (8 * 30) + 0)
> +
> +inline int kvm_mips_put_one_reg(CPUState *cs, int reg_id, int32 *addr)
> +{
> + __u64 val64 = (__u64)*addr;
> + struct kvm_one_reg cp0reg = {
> + .id = reg_id,
> + .addr = (__u64)((target_ulong)&val64)
> + };
> +
> + return kvm_vcpu_ioctl(cs, KVM_SET_ONE_REG, &cp0reg);
> +}
> +
> +inline int kvm_mips_put_one_ulreg(CPUState *cs, int reg_id, target_ulong
> *addr)
> +{
> + __u64 val64 = (__u64)*addr;
> + struct kvm_one_reg cp0reg = {
> + .id = reg_id,
> + .addr = (__u64)((target_ulong)&val64)
> + };
> +
> + return kvm_vcpu_ioctl(cs, KVM_SET_ONE_REG, &cp0reg);
> +}
> +
> +inline int kvm_mips_get_one_reg(CPUState *cs, int reg_id, int32 *addr)
> +{
> + int ret;
> + __u64 val64 = 0;
> + struct kvm_one_reg cp0reg = {
> + .id = reg_id,
> + .addr = (__u64)((target_ulong)&val64)
> + };
> +
> + ret = kvm_vcpu_ioctl(cs, KVM_GET_ONE_REG, &cp0reg);
> + if (ret < 0) {
> + return ret;
> + }
> +
> + *addr = (int32)val64;
> + return ret;
> +}
> +
> +inline int kvm_mips_get_one_ulreg(CPUState *cs, int reg_id, target_ulong
> *addr)
> +{
> + int ret;
> + __u64 val64 = 0;
> + struct kvm_one_reg cp0reg = {
> + .id = reg_id,
> + .addr = (__u64)((target_ulong)&val64)
> + };
> +
> + ret = kvm_vcpu_ioctl(cs, KVM_GET_ONE_REG, &cp0reg);
> + if (ret < 0) {
> + return ret;
> + }
> +
> + *addr = (target_ulong)val64;
> + return ret;
> +}
> +
> +int kvm_mips_te_put_cp0_registers(CPUState *cs, int level)
> +{
> + MIPSCPU *cpu = MIPS_CPU(cs);
> + CPUMIPSState *env = &cpu->env;
> + int ret;
> +
> + (void)level;
> +
> + ret = kvm_mips_put_one_reg(cs, KVM_REG_MIPS_CP0_INDEX, &env->CP0_Index);
> + if (ret < 0) {
> + return ret;
> + }
> + ret |= kvm_mips_put_one_ulreg(cs, KVM_REG_MIPS_CP0_CONTEXT,
> + &env->CP0_Context);
> + if (ret < 0) {
> + return ret;
> + }
> + ret |= kvm_mips_put_one_reg(cs, KVM_REG_MIPS_CP0_PAGEMASK,
> + &env->CP0_PageMask);
> + if (ret < 0) {
> + return ret;
> + }
> + ret |= kvm_mips_put_one_reg(cs, KVM_REG_MIPS_CP0_WIRED, &env->CP0_Wired);
> + if (ret < 0) {
> + return ret;
> + }
> + ret |= kvm_mips_put_one_ulreg(cs, KVM_REG_MIPS_CP0_BADVADDR,
> + &env->CP0_BadVAddr);
> + if (ret < 0) {
> + return ret;
> + }
> + ret |= kvm_mips_put_one_reg(cs, KVM_REG_MIPS_CP0_COUNT, &env->CP0_Count);
> + if (ret < 0) {
> + return ret;
> + }
> + ret |= kvm_mips_put_one_ulreg(cs, KVM_REG_MIPS_CP0_ENTRYHI,
> + &env->CP0_EntryHi);
> + if (ret < 0) {
> + return ret;
> + }
> + ret |= kvm_mips_put_one_reg(cs, KVM_REG_MIPS_CP0_COMPARE,
> + &env->CP0_Compare);
> + if (ret < 0) {
> + return ret;
> + }
> + ret |= kvm_mips_put_one_reg(cs, KVM_REG_MIPS_CP0_STATUS,
> &env->CP0_Status);
> + if (ret < 0) {
> + return ret;
> + }
> + ret |= kvm_mips_put_one_reg(cs, KVM_REG_MIPS_CP0_CAUSE, &env->CP0_Cause);
> + if (ret < 0) {
> + return ret;
> + }
> + ret |= kvm_mips_put_one_ulreg(cs, KVM_REG_MIPS_CP0_EPC, &env->CP0_EPC);
> + if (ret < 0) {
> + return ret;
> + }
> + ret |= kvm_mips_put_one_ulreg(cs, KVM_REG_MIPS_CP0_ERROREPC,
> + &env->CP0_ErrorEPC);
> + if (ret < 0) {
> + return ret;
> + }
> +
> + return ret;
> +}
> +
> +int kvm_mips_te_get_cp0_registers(CPUState *cs)
> +{
> + MIPSCPU *cpu = MIPS_CPU(cs);
> + CPUMIPSState *env = &cpu->env;
> + int ret;
> +
> + ret = kvm_mips_get_one_reg(cs, KVM_REG_MIPS_CP0_INDEX, &env->CP0_Index);
> + if (ret < 0) {
> + return ret;
> + }
> + ret |= kvm_mips_get_one_ulreg(cs, KVM_REG_MIPS_CP0_CONTEXT,
> + &env->CP0_Context);
> + if (ret < 0) {
> + return ret;
> + }
> + ret |= kvm_mips_get_one_reg(cs, KVM_REG_MIPS_CP0_PAGEMASK,
> + &env->CP0_PageMask);
> + if (ret < 0) {
> + return ret;
> + }
> + ret |= kvm_mips_get_one_reg(cs, KVM_REG_MIPS_CP0_WIRED, &env->CP0_Wired);
> + if (ret < 0) {
> + return ret;
> + }
> + ret |= kvm_mips_get_one_ulreg(cs, KVM_REG_MIPS_CP0_BADVADDR,
> + &env->CP0_BadVAddr);
> + if (ret < 0) {
> + return ret;
> + }
> + ret |= kvm_mips_get_one_reg(cs, KVM_REG_MIPS_CP0_COUNT, &env->CP0_Count);
> + if (ret < 0) {
> + return ret;
> + }
> + ret |= kvm_mips_get_one_ulreg(cs, KVM_REG_MIPS_CP0_ENTRYHI,
> + &env->CP0_EntryHi);
> + if (ret < 0) {
> + return ret;
> + }
> + ret |= kvm_mips_get_one_reg(cs, KVM_REG_MIPS_CP0_COMPARE,
> + &env->CP0_Compare);
> + if (ret < 0) {
> + return ret;
> + }
> + ret |= kvm_mips_get_one_reg(cs, KVM_REG_MIPS_CP0_STATUS,
> &env->CP0_Status);
> + if (ret < 0) {
> + return ret;
> + }
> + ret |= kvm_mips_get_one_reg(cs, KVM_REG_MIPS_CP0_CAUSE, &env->CP0_Cause);
> + if (ret < 0) {
> + return ret;
> + }
> + ret |= kvm_mips_get_one_ulreg(cs, KVM_REG_MIPS_CP0_EPC, &env->CP0_EPC);
> + if (ret < 0) {
> + return ret;
> + }
> + ret |= kvm_mips_get_one_ulreg(cs, KVM_REG_MIPS_CP0_ERROREPC,
> + &env->CP0_ErrorEPC);
> + if (ret < 0) {
> + return ret;
> + }
> +
> + return ret;
> +}
> diff --git a/target-mips/kvm_mips.h b/target-mips/kvm_mips.h
> new file mode 100644
> index 0000000..0da4edf
> --- /dev/null
> +++ b/target-mips/kvm_mips.h
> @@ -0,0 +1,28 @@
> +/*
> + * This file is subject to the terms and conditions of the GNU General Public
> + * License. See the file "COPYING" in the main directory of this archive
> + * for more details.
> + *
> + * KVM/MIPS: MIPS specific KVM APIs
> + *
> + * Copyright (C) 2012-2013 Imagination Technologies Ltd.
> + * Authors: Sanjay Lal <address@hidden>
> +*/
> +
> +#ifndef __KVM_MIPS_H__
> +#define __KVM_MIPS_H__
> +
> +
> +int kvm_mips_set_interrupt(CPUMIPSState *env, int irq, int level);
> +
> +int kvm_mips_set_ipi_interrupt(CPUArchState *env, int irq, int level);
> +
> +inline int kvm_mips_put_one_reg(CPUState *cs, int reg_id, int32 *addr);
> +inline int kvm_mips_put_one_ulreg(CPUState *cs, int reg_id, target_ulong
> *addr);
> +inline int kvm_mips_get_one_reg(CPUState *cs, int reg_id, int32 *addr);
> +inline int kvm_mips_get_one_ulreg(CPUState *cs, int reg_id, target_ulong
> *addr);
> +
> +int kvm_mips_te_put_cp0_registers(CPUState *cs, int level);
> +int kvm_mips_te_get_cp0_registers(CPUState *cs);
> +
> +#endif /* __KVM_MIPS_H__ */
This doesn't look bad, but I don't have a good enough knowledge of KVM
to review this patch.
--
Aurelien Jarno GPG: 1024D/F1BCDB73
address@hidden http://www.aurel32.net
- Re: [Qemu-devel] [PATCH v2 10/10] target-mips: Enable KVM support in build system, (continued)
- [Qemu-devel] [PATCH v2 02/10] hw/mips: Add API to convert KVM guest KSEG0 <-> GPA, James Hogan, 2013/12/16
- [Qemu-devel] [PATCH v2 06/10] target-mips: Set target page size to 16K in KVM mode, James Hogan, 2013/12/16
- [Qemu-devel] [PATCH v2 09/10] hw/mips: malta: Add KVM support, James Hogan, 2013/12/16
- [Qemu-devel] [PATCH v2 03/10] target-mips: get_physical_address: Add defines for segment bases, James Hogan, 2013/12/16
- [Qemu-devel] [PATCH v2 07/10] target-mips: kvm: Add main KVM support for MIPS, James Hogan, 2013/12/16
- Re: [Qemu-devel] [PATCH v2 07/10] target-mips: kvm: Add main KVM support for MIPS,
Aurelien Jarno <=
- [Qemu-devel] [PATCH v2 01/10] hw/mips/cputimer: Don't start periodic timer in KVM mode, James Hogan, 2013/12/16
- [Qemu-devel] [PATCH v2 04/10] target-mips: get_physical_address: Add KVM awareness, James Hogan, 2013/12/16