Re: [Qemu-devel] [PATCH v2 07/10] target-mips: kvm: Add main KVM support for MIPS

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Am 16.12.2013 15:12, schrieb James Hogan:
> From: Sanjay Lal <sanjayl@xxxxxxxxxxx>
> 
> Implement the main KVM arch API for MIPS.
> 
> Signed-off-by: Sanjay Lal <sanjayl@xxxxxxxxxxx>
> Signed-off-by: James Hogan <james.hogan@xxxxxxxxxx>
> Cc: Aurelien Jarno <aurelien@xxxxxxxxxxx>
> Cc: Gleb Natapov <gleb@xxxxxxxxxx>
> Cc: Paolo Bonzini <pbonzini@xxxxxxxxxx>
> ---
> Changes in v2:
>  - Expand commit message
>  - Checkpatch cleanups.
>  - Some interrupt bug fixes from Yann Le Du <ledu@xxxxxxxxxxx>
>  - Add get/set register functionality from Yann Le Du <ledu@xxxxxxxxxxx>
>  - Use new 64 bit compatible ABI from Cavium from Sanjay Lal
>    <sanjayl@xxxxxxxxxxx>
>  - 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 <sanjayl@xxxxxxxxxxx>
> +*/
> +
> +#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, &regs);
> +
> +    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, &regs);
> +
> +    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 <sanjayl@xxxxxxxxxxx>
> +*/
> +
> +#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 majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [KVM ARM]     [KVM ia64]     [KVM ppc]     [Virtualization Tools]     [Spice Development]     [Libvirt]     [Libvirt Users]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite Questions]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux