Am 16.12.2013 15:12, schrieb James Hogan:
> From: Sanjay Lal <sanj...@kymasys.com>
>
> Implement the main KVM arch API for MIPS.
>
> Signed-off-by: Sanjay Lal <sanj...@kymasys.com>
> Signed-off-by: James Hogan <james.ho...@imgtec.com>
> Cc: Aurelien Jarno <aurel...@aurel32.net>
> Cc: Gleb Natapov <g...@redhat.com>
> Cc: Paolo Bonzini <pbonz...@redhat.com>
> ---
> Changes in v2:
> - Expand commit message
> - Checkpatch cleanups.
> - Some interrupt bug fixes from Yann Le Du <l...@kymasys.com>
> - Add get/set register functionality from Yann Le Du <l...@kymasys.com>
> - Use new 64 bit compatible ABI from Cavium from Sanjay Lal
> <sanj...@kymasys.com>
> - 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 <sanj...@kymasys.com>
> +*/
> +
> +#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, ...) \
dprintf is the name of a stdio.h function, so DPRINTF may be a better name.
> + do { if (DEBUG_KVM) { fprintf(stderr, fmt, ## __VA_ARGS__); } } while (0)
This looks really modern, thanks. :)
> +
> +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)
Please use "env" only for CPUMIPSState, use "cpu" or "cs" here. The
usual convention is "cs" for CPUState in target-*/ so that "cpu" can be
used for MIPSCPU.
> +{
> + int ret = 0;
> + dprintf("%s\n", __func__);
> + return ret;
> +}
> +
> +void kvm_arch_reset_vcpu(CPUState *env)
Dito.
> +{
> + 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)
Dito.
> +{
> + dprintf("%s\n", __func__);
> + return 0;
> +}
> +
> +int kvm_arch_remove_sw_breakpoint(CPUState *env, struct kvm_sw_breakpoint
> *bp)
Dito.
> +{
> + dprintf("%s\n", __func__);
> + return 0;
> +}
> +
> +static inline int cpu_mips_io_interrupts_pending(CPUArchState *env)
Please don't use CPUArchState in MIPS-specific code, use CPUMIPSState.
Although in this trivial case MIPSCPU would be more future-proof.
> +{
> + 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))) {
Parentheses around cpu_mips_io_interrupts_pending() seem unnecessary
here FWIW.
> + 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);
Should this really be a printf() rather than error_report() or trace point?
> + }
> + }
> +}
> +
> +void kvm_arch_post_run(CPUState *env, struct kvm_run *run)
"cs"
> +{
> + 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)
"cs"
> +{
> + int ret;
> +
> + dprintf("%s\n", __func__);
> + switch (run->exit_reason) {
> + default:
> + fprintf(stderr, "KVM: unknown exit reason %d\n", run->exit_reason);
error_report() and in that case without trailing \n.
> + ret = -1;
> + break;
> + }
> +
> + return ret;
> +}
> +
> +bool kvm_arch_stop_on_emulation_error(CPUState *env)
"cs"
> +{
> + dprintf("%s\n", __func__);
> + return true;
> +}
> +
> +int kvm_arch_on_sigbus_vcpu(CPUState *env, int code, void *addr)
"cs"
> +{
> + 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);
CPU(mips_env_get_cpu(env)) please - ENV_GET_CPU() is for generic code
only and supposed to go away.
Any chance a MIPSCPU *cpu (or CPUState *cs) argument can be used instead?
> + 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);
Similar question here.
> + 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 <sanj...@kymasys.com>
> +*/
> +
> +#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__ */
Regards,
Andreas
--
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg
--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
