From: Guo Hongyu <guohongyu24@mails.ucas.ac.cn>

Refer to the link below for a description of the vldi instructions:

https://jia.je/unofficial-loongarch-intrinsics-guide/lsx/misc/#synopsis_88

Fixed errors in vldi instruction implementation.

Signed-off-by: Guo Hongyu <guohongyu24@mails.ucas.ac.cn>

Tested-by: Xianglai Li <lixianglai@loongson.cn>

Signed-off-by: Xianglai Li <lixianglai@loongson.cn>

Reviewed-by: Bibo Mao <maobibo@loongson.cn>

Signed-off-by: Bibo Mao <maobibo@loongson.cn>

---

target/loongarch/tcg/insn_trans/trans_vec.c.inc | 2 +-

1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/target/loongarch/tcg/insn_trans/trans_vec.c.inc b/target/loongarch/tcg/insn_trans/trans_vec.c.inc

index XXXXXXX..XXXXXXX 100644

--- a/target/loongarch/tcg/insn_trans/trans_vec.c.inc

+++ b/target/loongarch/tcg/insn_trans/trans_vec.c.inc

@@ -XXX,XX +XXX,XX @@ static uint64_t vldi_get_value(DisasContext *ctx, uint32_t imm)

break;

case 1:

/* data: {2{16'0, imm[7:0], 8'0}} */

- data = (t << 24) | (t << 8);

+ data = (t << 40) | (t << 8);

break;

case 2:

/* data: {2{8'0, imm[7:0], 16'0}} */

--

2.43.5

Hardcoded 32 bytes is used for vbsrl emulation check, there is

problem when options lsx=on,lasx=off is used for vbsrl.v instruction

in TCG mode. It injects LASX exception rather LSX exception.

Here actual operand size is used.

Cc: qemu-stable@nongnu.org

Fixes: df97f338076 ("target/loongarch: Implement xvreplve xvinsve0 xvpickve")

Signed-off-by: Bibo Mao <maobibo@loongson.cn>

Reviewed-by: Richard Henderson <richard.henderson@linaro.org>

Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>

---

target/loongarch/tcg/insn_trans/trans_vec.c.inc | 2 +-

1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/target/loongarch/tcg/insn_trans/trans_vec.c.inc b/target/loongarch/tcg/insn_trans/trans_vec.c.inc

index XXXXXXX..XXXXXXX 100644

--- a/target/loongarch/tcg/insn_trans/trans_vec.c.inc

+++ b/target/loongarch/tcg/insn_trans/trans_vec.c.inc

@@ -XXX,XX +XXX,XX @@ static bool do_vbsrl_v(DisasContext *ctx, arg_vv_i *a, uint32_t oprsz)

{

int i, ofs;

- if (!check_vec(ctx, 32)) {

+ if (!check_vec(ctx, oprsz)) {

return true;

}

--

2.43.5

The same with ACPI table, fdt table is created on machine done

notification. Some objects like CPU objects can be created with cold-plug

method with command such as -smp x, -device la464-loongarch-cpu, so all

objects finish to create when machine is done.

Signed-off-by: Bibo Mao <maobibo@loongson.cn>

Reviewed-by: Bibo Mao <maobibo@loongson.cn>

---

hw/loongarch/virt.c | 103 ++++++++++++++++++++++++--------------------

1 file changed, 57 insertions(+), 46 deletions(-)

diff --git a/hw/loongarch/virt.c b/hw/loongarch/virt.c

index XXXXXXX..XXXXXXX 100644

--- a/hw/loongarch/virt.c

+++ b/hw/loongarch/virt.c

@@ -XXX,XX +XXX,XX @@ static void virt_build_smbios(LoongArchVirtMachineState *lvms)

}

}

+static void virt_fdt_setup(LoongArchVirtMachineState *lvms)

+{

+ MachineState *machine = MACHINE(lvms);

+ uint32_t cpuintc_phandle, eiointc_phandle, pch_pic_phandle, pch_msi_phandle;

+ int i;

+

+ create_fdt(lvms);

+ fdt_add_cpu_nodes(lvms);

+ fdt_add_memory_nodes(machine);

+ fdt_add_fw_cfg_node(lvms);

+ fdt_add_flash_node(lvms);

+

+ /* Add cpu interrupt-controller */

+ fdt_add_cpuic_node(lvms, &cpuintc_phandle);

+ /* Add Extend I/O Interrupt Controller node */

+ fdt_add_eiointc_node(lvms, &cpuintc_phandle, &eiointc_phandle);

+ /* Add PCH PIC node */

+ fdt_add_pch_pic_node(lvms, &eiointc_phandle, &pch_pic_phandle);

+ /* Add PCH MSI node */

+ fdt_add_pch_msi_node(lvms, &eiointc_phandle, &pch_msi_phandle);

+ /* Add pcie node */

+ fdt_add_pcie_node(lvms, &pch_pic_phandle, &pch_msi_phandle);

+

+ /*

+ * Create uart fdt node in reverse order so that they appear

+ * in the finished device tree lowest address first

+ */

+ for (i = VIRT_UART_COUNT; i-- > 0;) {

+ hwaddr base = VIRT_UART_BASE + i * VIRT_UART_SIZE;

+ int irq = VIRT_UART_IRQ + i - VIRT_GSI_BASE;

+ fdt_add_uart_node(lvms, &pch_pic_phandle, base, irq, i == 0);

+ }

+

+ fdt_add_rtc_node(lvms, &pch_pic_phandle);

+ fdt_add_ged_reset(lvms);

+ platform_bus_add_all_fdt_nodes(machine->fdt, "/platic",

+ VIRT_PLATFORM_BUS_BASEADDRESS,

+ VIRT_PLATFORM_BUS_SIZE,

+ VIRT_PLATFORM_BUS_IRQ);

+

+ /*

+ * Since lowmem region starts from 0 and Linux kernel legacy start address

+ * at 2 MiB, FDT base address is located at 1 MiB to avoid NULL pointer

+ * access. FDT size limit with 1 MiB.

+ * Put the FDT into the memory map as a ROM image: this will ensure

+ * the FDT is copied again upon reset, even if addr points into RAM.

+ */

+ qemu_fdt_dumpdtb(machine->fdt, lvms->fdt_size);

+ rom_add_blob_fixed_as("fdt", machine->fdt, lvms->fdt_size, FDT_BASE,

+ &address_space_memory);

+ qemu_register_reset_nosnapshotload(qemu_fdt_randomize_seeds,

+ rom_ptr_for_as(&address_space_memory, FDT_BASE, lvms->fdt_size));

+}

+

static void virt_done(Notifier *notifier, void *data)

{

LoongArchVirtMachineState *lvms = container_of(notifier,

LoongArchVirtMachineState, machine_done);

virt_build_smbios(lvms);

loongarch_acpi_setup(lvms);

+ virt_fdt_setup(lvms);

}

static void virt_powerdown_req(Notifier *notifier, void *opaque)

@@ -XXX,XX +XXX,XX @@ static DeviceState *create_platform_bus(DeviceState *pch_pic)

}

static void virt_devices_init(DeviceState *pch_pic,

- LoongArchVirtMachineState *lvms,

- uint32_t *pch_pic_phandle,

- uint32_t *pch_msi_phandle)

+ LoongArchVirtMachineState *lvms)

{

MachineClass *mc = MACHINE_GET_CLASS(lvms);

DeviceState *gpex_dev;

@@ -XXX,XX +XXX,XX @@ static void virt_devices_init(DeviceState *pch_pic,

gpex_set_irq_num(GPEX_HOST(gpex_dev), i, 16 + i);

}

- /* Add pcie node */

- fdt_add_pcie_node(lvms, pch_pic_phandle, pch_msi_phandle);

-

/*

* Create uart fdt node in reverse order so that they appear

* in the finished device tree lowest address first

@@ -XXX,XX +XXX,XX @@ static void virt_devices_init(DeviceState *pch_pic,

serial_mm_init(get_system_memory(), base, 0,

qdev_get_gpio_in(pch_pic, irq),

115200, serial_hd(i), DEVICE_LITTLE_ENDIAN);

- fdt_add_uart_node(lvms, pch_pic_phandle, base, irq, i == 0);

}

/* Network init */

@@ -XXX,XX +XXX,XX @@ static void virt_devices_init(DeviceState *pch_pic,

sysbus_create_simple("ls7a_rtc", VIRT_RTC_REG_BASE,

qdev_get_gpio_in(pch_pic,

VIRT_RTC_IRQ - VIRT_GSI_BASE));

- fdt_add_rtc_node(lvms, pch_pic_phandle);

- fdt_add_ged_reset(lvms);

/* acpi ged */

lvms->acpi_ged = create_acpi_ged(pch_pic, lvms);

@@ -XXX,XX +XXX,XX @@ static void virt_irq_init(LoongArchVirtMachineState *lvms)

CPULoongArchState *env;

CPUState *cpu_state;

int cpu, pin, i, start, num;

- uint32_t cpuintc_phandle, eiointc_phandle, pch_pic_phandle, pch_msi_phandle;

/*

* Extended IRQ model.

@@ -XXX,XX +XXX,XX @@ static void virt_irq_init(LoongArchVirtMachineState *lvms)

memory_region_add_subregion(&lvms->system_iocsr, MAIL_SEND_ADDR,

sysbus_mmio_get_region(SYS_BUS_DEVICE(ipi), 1));

- /* Add cpu interrupt-controller */

- fdt_add_cpuic_node(lvms, &cpuintc_phandle);

-

for (cpu = 0; cpu < ms->smp.cpus; cpu++) {

cpu_state = qemu_get_cpu(cpu);

cpudev = DEVICE(cpu_state);

@@ -XXX,XX +XXX,XX @@ static void virt_irq_init(LoongArchVirtMachineState *lvms)

}

}

- /* Add Extend I/O Interrupt Controller node */

- fdt_add_eiointc_node(lvms, &cpuintc_phandle, &eiointc_phandle);

-

pch_pic = qdev_new(TYPE_LOONGARCH_PIC);

num = VIRT_PCH_PIC_IRQ_NUM;

qdev_prop_set_uint32(pch_pic, "pch_pic_irq_num", num);

@@ -XXX,XX +XXX,XX @@ static void virt_irq_init(LoongArchVirtMachineState *lvms)

qdev_connect_gpio_out(DEVICE(d), i, qdev_get_gpio_in(extioi, i));

}

- /* Add PCH PIC node */

- fdt_add_pch_pic_node(lvms, &eiointc_phandle, &pch_pic_phandle);

-

pch_msi = qdev_new(TYPE_LOONGARCH_PCH_MSI);

start = num;

num = EXTIOI_IRQS - start;

@@ -XXX,XX +XXX,XX @@ static void virt_irq_init(LoongArchVirtMachineState *lvms)

qdev_get_gpio_in(extioi, i + start));

}

- /* Add PCH MSI node */

- fdt_add_pch_msi_node(lvms, &eiointc_phandle, &pch_msi_phandle);

-

- virt_devices_init(pch_pic, lvms, &pch_pic_phandle, &pch_msi_phandle);

+ virt_devices_init(pch_pic, lvms);

}

static void virt_firmware_init(LoongArchVirtMachineState *lvms)

@@ -XXX,XX +XXX,XX @@ static void virt_init(MachineState *machine)

cpu_model = LOONGARCH_CPU_TYPE_NAME("la464");

}

- create_fdt(lvms);

-

/* Create IOCSR space */

memory_region_init_io(&lvms->system_iocsr, OBJECT(machine), NULL,

machine, "iocsr", UINT64_MAX);

@@ -XXX,XX +XXX,XX @@ static void virt_init(MachineState *machine)

lacpu = LOONGARCH_CPU(cpu);

lacpu->phy_id = machine->possible_cpus->cpus[i].arch_id;

}

- fdt_add_cpu_nodes(lvms);

- fdt_add_memory_nodes(machine);

fw_cfg_add_memory(machine);

/* Node0 memory */

@@ -XXX,XX +XXX,XX @@ static void virt_init(MachineState *machine)

memmap_table,

sizeof(struct memmap_entry) * (memmap_entries));

}

- fdt_add_fw_cfg_node(lvms);

- fdt_add_flash_node(lvms);

/* Initialize the IO interrupt subsystem */

virt_irq_init(lvms);

- platform_bus_add_all_fdt_nodes(machine->fdt, "/platic",

- VIRT_PLATFORM_BUS_BASEADDRESS,

- VIRT_PLATFORM_BUS_SIZE,

- VIRT_PLATFORM_BUS_IRQ);

lvms->machine_done.notify = virt_done;

qemu_add_machine_init_done_notifier(&lvms->machine_done);

/* connect powerdown request */

lvms->powerdown_notifier.notify = virt_powerdown_req;

qemu_register_powerdown_notifier(&lvms->powerdown_notifier);

- /*

- * Since lowmem region starts from 0 and Linux kernel legacy start address

- * at 2 MiB, FDT base address is located at 1 MiB to avoid NULL pointer

- * access. FDT size limit with 1 MiB.

- * Put the FDT into the memory map as a ROM image: this will ensure

- * the FDT is copied again upon reset, even if addr points into RAM.

- */

- qemu_fdt_dumpdtb(machine->fdt, lvms->fdt_size);

- rom_add_blob_fixed_as("fdt", machine->fdt, lvms->fdt_size, FDT_BASE,

- &address_space_memory);

- qemu_register_reset_nosnapshotload(qemu_fdt_randomize_seeds,

- rom_ptr_for_as(&address_space_memory, FDT_BASE, lvms->fdt_size));

-

lvms->bootinfo.ram_size = ram_size;

loongarch_load_kernel(machine, &lvms->bootinfo);

}

--

2.43.5

Like LBT feature, add type OnOffAuto for LSX feature setting. Also

add LSX feature detection with new VM ioctl command, fallback to old

method if it is not supported.

Signed-off-by: Bibo Mao <maobibo@loongson.cn>

Reviewed-by: Bibo Mao <maobibo@loongson.cn>

---

target/loongarch/cpu.c | 38 +++++++++++++++------------

target/loongarch/cpu.h | 2 ++

target/loongarch/kvm/kvm.c | 54 ++++++++++++++++++++++++++++++++++++++

3 files changed, 77 insertions(+), 17 deletions(-)

diff --git a/target/loongarch/cpu.c b/target/loongarch/cpu.c

index XXXXXXX..XXXXXXX 100644

--- a/target/loongarch/cpu.c

+++ b/target/loongarch/cpu.c

@@ -XXX,XX +XXX,XX @@ static void loongarch_la464_initfn(Object *obj)

{

LoongArchCPU *cpu = LOONGARCH_CPU(obj);

CPULoongArchState *env = &cpu->env;

+ uint32_t data = 0;

int i;

for (i = 0; i < 21; i++) {

@@ -XXX,XX +XXX,XX @@ static void loongarch_la464_initfn(Object *obj)

cpu->dtb_compatible = "loongarch,Loongson-3A5000";

env->cpucfg[0] = 0x14c010; /* PRID */

- uint32_t data = 0;

data = FIELD_DP32(data, CPUCFG1, ARCH, 2);

data = FIELD_DP32(data, CPUCFG1, PGMMU, 1);

data = FIELD_DP32(data, CPUCFG1, IOCSR, 1);

@@ -XXX,XX +XXX,XX @@ static void loongarch_la132_initfn(Object *obj)

{

LoongArchCPU *cpu = LOONGARCH_CPU(obj);

CPULoongArchState *env = &cpu->env;

-

+ uint32_t data = 0;

int i;

for (i = 0; i < 21; i++) {

@@ -XXX,XX +XXX,XX @@ static void loongarch_la132_initfn(Object *obj)

cpu->dtb_compatible = "loongarch,Loongson-1C103";

env->cpucfg[0] = 0x148042; /* PRID */

- uint32_t data = 0;

data = FIELD_DP32(data, CPUCFG1, ARCH, 1); /* LA32 */

data = FIELD_DP32(data, CPUCFG1, PGMMU, 1);

data = FIELD_DP32(data, CPUCFG1, IOCSR, 1);

@@ -XXX,XX +XXX,XX @@ static void loongarch_cpu_realizefn(DeviceState *dev, Error **errp)

static bool loongarch_get_lsx(Object *obj, Error **errp)

{

- LoongArchCPU *cpu = LOONGARCH_CPU(obj);

- bool ret;

-

- if (FIELD_EX32(cpu->env.cpucfg[2], CPUCFG2, LSX)) {

- ret = true;

- } else {

- ret = false;

- }

- return ret;

+ return LOONGARCH_CPU(obj)->lsx != ON_OFF_AUTO_OFF;

}

static void loongarch_set_lsx(Object *obj, bool value, Error **errp)

{

LoongArchCPU *cpu = LOONGARCH_CPU(obj);

+ uint32_t val;

- if (value) {

- cpu->env.cpucfg[2] = FIELD_DP32(cpu->env.cpucfg[2], CPUCFG2, LSX, 1);

- } else {

- cpu->env.cpucfg[2] = FIELD_DP32(cpu->env.cpucfg[2], CPUCFG2, LSX, 0);

- cpu->env.cpucfg[2] = FIELD_DP32(cpu->env.cpucfg[2], CPUCFG2, LASX, 0);

+ cpu->lsx = value ? ON_OFF_AUTO_ON : ON_OFF_AUTO_OFF;

+ if (kvm_enabled()) {

+ /* kvm feature detection in function kvm_arch_init_vcpu */

+ return;

}

+

+ /* LSX feature detection in TCG mode */

+ val = cpu->env.cpucfg[2];

+ if (cpu->lsx == ON_OFF_AUTO_ON) {

+ if (FIELD_EX32(val, CPUCFG2, LSX) == 0) {

+ error_setg(errp, "Failed to enable LSX in TCG mode");

+ return;

+ }

+ }

+

+ cpu->env.cpucfg[2] = FIELD_DP32(val, CPUCFG2, LSX, value);

}

static bool loongarch_get_lasx(Object *obj, Error **errp)

@@ -XXX,XX +XXX,XX @@ void loongarch_cpu_post_init(Object *obj)

{

LoongArchCPU *cpu = LOONGARCH_CPU(obj);

+ cpu->lsx = ON_OFF_AUTO_AUTO;

object_property_add_bool(obj, "lsx", loongarch_get_lsx,

loongarch_set_lsx);

object_property_add_bool(obj, "lasx", loongarch_get_lasx,

@@ -XXX,XX +XXX,XX @@ void loongarch_cpu_post_init(Object *obj)

} else {

cpu->lbt = ON_OFF_AUTO_OFF;

+ cpu->pmu = ON_OFF_AUTO_OFF;

}

}

diff --git a/target/loongarch/cpu.h b/target/loongarch/cpu.h

index XXXXXXX..XXXXXXX 100644

--- a/target/loongarch/cpu.h

+++ b/target/loongarch/cpu.h

@@ -XXX,XX +XXX,XX @@ typedef struct LoongArchTLB LoongArchTLB;

#endif

enum loongarch_features {

+ LOONGARCH_FEATURE_LSX,

LOONGARCH_FEATURE_LBT, /* loongson binary translation extension */

LOONGARCH_FEATURE_PMU,

};

@@ -XXX,XX +XXX,XX @@ struct ArchCPU {

uint32_t phy_id;

OnOffAuto lbt;

OnOffAuto pmu;

+ OnOffAuto lsx;

/* 'compatible' string for this CPU for Linux device trees */

const char *dtb_compatible;

diff --git a/target/loongarch/kvm/kvm.c b/target/loongarch/kvm/kvm.c

index XXXXXXX..XXXXXXX 100644

--- a/target/loongarch/kvm/kvm.c

+++ b/target/loongarch/kvm/kvm.c

@@ -XXX,XX +XXX,XX @@ static bool kvm_feature_supported(CPUState *cs, enum loongarch_features feature)

{

int ret;

struct kvm_device_attr attr;

+ uint64_t val;

switch (feature) {

+ case LOONGARCH_FEATURE_LSX:

+ attr.group = KVM_LOONGARCH_VM_FEAT_CTRL;

+ attr.attr = KVM_LOONGARCH_VM_FEAT_LSX;

+ ret = kvm_vm_ioctl(kvm_state, KVM_HAS_DEVICE_ATTR, &attr);

+ if (ret == 0) {

+ return true;

+ }

+

+ /* Fallback to old kernel detect interface */

+ val = 0;

+ attr.group = KVM_LOONGARCH_VCPU_CPUCFG;

+ /* Cpucfg2 */

+ attr.attr = 2;

+ attr.addr = (uint64_t)&val;

+ ret = kvm_vcpu_ioctl(cs, KVM_HAS_DEVICE_ATTR, &attr);

+ if (!ret) {

+ ret = kvm_vcpu_ioctl(cs, KVM_GET_DEVICE_ATTR, &attr);

+ if (ret) {

+ return false;

+ }

+

+ ret = FIELD_EX32((uint32_t)val, CPUCFG2, LSX);

+ return (ret != 0);

+ }

+ return false;

+

case LOONGARCH_FEATURE_LBT:

/*

* Return all if all the LBT features are supported such as:

@@ -XXX,XX +XXX,XX @@ static bool kvm_feature_supported(CPUState *cs, enum loongarch_features feature)

return false;

}

+static int kvm_cpu_check_lsx(CPUState *cs, Error **errp)

+{

+ CPULoongArchState *env = cpu_env(cs);

+ LoongArchCPU *cpu = LOONGARCH_CPU(cs);

+ bool kvm_supported;

+

+ kvm_supported = kvm_feature_supported(cs, LOONGARCH_FEATURE_LSX);

+ env->cpucfg[2] = FIELD_DP32(env->cpucfg[2], CPUCFG2, LSX, 0);

+ if (cpu->lsx == ON_OFF_AUTO_ON) {

+ if (kvm_supported) {

+ env->cpucfg[2] = FIELD_DP32(env->cpucfg[2], CPUCFG2, LSX, 1);

+ } else {

+ error_setg(errp, "'lsx' feature not supported by KVM on this host");

+ return -ENOTSUP;

+ }

+ } else if ((cpu->lsx == ON_OFF_AUTO_AUTO) && kvm_supported) {

+ env->cpucfg[2] = FIELD_DP32(env->cpucfg[2], CPUCFG2, LSX, 1);

+ }

+

+ return 0;

+}

+

static int kvm_cpu_check_lbt(CPUState *cs, Error **errp)

{

CPULoongArchState *env = cpu_env(cs);

@@ -XXX,XX +XXX,XX @@ int kvm_arch_init_vcpu(CPUState *cs)

brk_insn = val;

}

+ ret = kvm_cpu_check_lsx(cs, &local_err);

+ if (ret < 0) {

+ error_report_err(local_err);

+ }

+

ret = kvm_cpu_check_lbt(cs, &local_err);

if (ret < 0) {

error_report_err(local_err);

--

2.43.5

Like LSX feature, add type OnOffAuto for LASX feature setting.

Signed-off-by: Bibo Mao <maobibo@loongson.cn>

Reviewed-by: Bibo Mao <maobibo@loongson.cn>

---

target/loongarch/cpu.c | 50 +++++++++++++++++++++++------------

target/loongarch/cpu.h | 2 ++

target/loongarch/kvm/kvm.c | 53 ++++++++++++++++++++++++++++++++++++++

3 files changed, 89 insertions(+), 16 deletions(-)

diff --git a/target/loongarch/cpu.c b/target/loongarch/cpu.c

index XXXXXXX..XXXXXXX 100644

--- a/target/loongarch/cpu.c

+++ b/target/loongarch/cpu.c

@@ -XXX,XX +XXX,XX @@ static void loongarch_set_lsx(Object *obj, bool value, Error **errp)

uint32_t val;

cpu->lsx = value ? ON_OFF_AUTO_ON : ON_OFF_AUTO_OFF;

+ if (cpu->lsx == ON_OFF_AUTO_OFF) {

+ cpu->lasx = ON_OFF_AUTO_OFF;

+ if (cpu->lasx == ON_OFF_AUTO_ON) {

+ error_setg(errp, "Failed to disable LSX since LASX is enabled");

+ return;

+ }

+ }

+

if (kvm_enabled()) {

/* kvm feature detection in function kvm_arch_init_vcpu */

return;

@@ -XXX,XX +XXX,XX @@ static void loongarch_set_lsx(Object *obj, bool value, Error **errp)

error_setg(errp, "Failed to enable LSX in TCG mode");

return;

}

+ } else {

+ cpu->env.cpucfg[2] = FIELD_DP32(val, CPUCFG2, LASX, 0);

+ val = cpu->env.cpucfg[2];

}

cpu->env.cpucfg[2] = FIELD_DP32(val, CPUCFG2, LSX, value);

@@ -XXX,XX +XXX,XX @@ static void loongarch_set_lsx(Object *obj, bool value, Error **errp)

static bool loongarch_get_lasx(Object *obj, Error **errp)

{

- LoongArchCPU *cpu = LOONGARCH_CPU(obj);

- bool ret;

-

- if (FIELD_EX32(cpu->env.cpucfg[2], CPUCFG2, LASX)) {

- ret = true;

- } else {

- ret = false;

- }

- return ret;

+ return LOONGARCH_CPU(obj)->lasx != ON_OFF_AUTO_OFF;

}

static void loongarch_set_lasx(Object *obj, bool value, Error **errp)

{

LoongArchCPU *cpu = LOONGARCH_CPU(obj);

+ uint32_t val;

- if (value) {

-    if (!FIELD_EX32(cpu->env.cpucfg[2], CPUCFG2, LSX)) {

- cpu->env.cpucfg[2] = FIELD_DP32(cpu->env.cpucfg[2], CPUCFG2, LSX, 1);

-    }

- cpu->env.cpucfg[2] = FIELD_DP32(cpu->env.cpucfg[2], CPUCFG2, LASX, 1);

- } else {

- cpu->env.cpucfg[2] = FIELD_DP32(cpu->env.cpucfg[2], CPUCFG2, LASX, 0);

+ cpu->lasx = value ? ON_OFF_AUTO_ON : ON_OFF_AUTO_OFF;

+ if ((cpu->lsx == ON_OFF_AUTO_OFF) && (cpu->lasx == ON_OFF_AUTO_ON)) {

+ error_setg(errp, "Failed to enable LASX since lSX is disabled");

+ return;

+ }

+

+ if (kvm_enabled()) {

+ /* kvm feature detection in function kvm_arch_init_vcpu */

+ return;

}

+

+ /* LASX feature detection in TCG mode */

+ val = cpu->env.cpucfg[2];

+ if (cpu->lasx == ON_OFF_AUTO_ON) {

+ if (FIELD_EX32(val, CPUCFG2, LASX) == 0) {

+ error_setg(errp, "Failed to enable LASX in TCG mode");

+ return;

+ }

+ }

+

+ cpu->env.cpucfg[2] = FIELD_DP32(val, CPUCFG2, LASX, value);

}

static bool loongarch_get_lbt(Object *obj, Error **errp)

@@ -XXX,XX +XXX,XX @@ void loongarch_cpu_post_init(Object *obj)

LoongArchCPU *cpu = LOONGARCH_CPU(obj);

cpu->lsx = ON_OFF_AUTO_AUTO;

+ cpu->lasx = ON_OFF_AUTO_AUTO;

object_property_add_bool(obj, "lsx", loongarch_get_lsx,

loongarch_set_lsx);

object_property_add_bool(obj, "lasx", loongarch_get_lasx,

diff --git a/target/loongarch/cpu.h b/target/loongarch/cpu.h

index XXXXXXX..XXXXXXX 100644

--- a/target/loongarch/cpu.h

+++ b/target/loongarch/cpu.h

@@ -XXX,XX +XXX,XX @@ typedef struct LoongArchTLB LoongArchTLB;

enum loongarch_features {

LOONGARCH_FEATURE_LSX,

+ LOONGARCH_FEATURE_LASX,

LOONGARCH_FEATURE_LBT, /* loongson binary translation extension */

LOONGARCH_FEATURE_PMU,

};

@@ -XXX,XX +XXX,XX @@ struct ArchCPU {

OnOffAuto lbt;

OnOffAuto pmu;

OnOffAuto lsx;

+ OnOffAuto lasx;

/* 'compatible' string for this CPU for Linux device trees */

const char *dtb_compatible;

diff --git a/target/loongarch/kvm/kvm.c b/target/loongarch/kvm/kvm.c

index XXXXXXX..XXXXXXX 100644

--- a/target/loongarch/kvm/kvm.c

+++ b/target/loongarch/kvm/kvm.c

@@ -XXX,XX +XXX,XX @@ static bool kvm_feature_supported(CPUState *cs, enum loongarch_features feature)

}

return false;

+ case LOONGARCH_FEATURE_LASX:

+ attr.group = KVM_LOONGARCH_VM_FEAT_CTRL;

+ attr.attr = KVM_LOONGARCH_VM_FEAT_LASX;

+ ret = kvm_vm_ioctl(kvm_state, KVM_HAS_DEVICE_ATTR, &attr);

+ if (ret == 0) {

+ return true;

+ }

+

+ /* Fallback to old kernel detect interface */

+ val = 0;

+ attr.group = KVM_LOONGARCH_VCPU_CPUCFG;

+ /* Cpucfg2 */

+ attr.attr = 2;

+ attr.addr = (uint64_t)&val;

+ ret = kvm_vcpu_ioctl(cs, KVM_HAS_DEVICE_ATTR, &attr);

+ if (!ret) {

+ ret = kvm_vcpu_ioctl(cs, KVM_GET_DEVICE_ATTR, &attr);

+ if (ret) {

+ return false;

+ }

+

+ ret = FIELD_EX32((uint32_t)val, CPUCFG2, LASX);

+ return (ret != 0);

+ }

+ return false;

+

case LOONGARCH_FEATURE_LBT:

/*

* Return all if all the LBT features are supported such as:

@@ -XXX,XX +XXX,XX @@ static int kvm_cpu_check_lsx(CPUState *cs, Error **errp)

return 0;

}

+static int kvm_cpu_check_lasx(CPUState *cs, Error **errp)

+{

+ CPULoongArchState *env = cpu_env(cs);

+ LoongArchCPU *cpu = LOONGARCH_CPU(cs);

+ bool kvm_supported;

+

+ kvm_supported = kvm_feature_supported(cs, LOONGARCH_FEATURE_LASX);

+ env->cpucfg[2] = FIELD_DP32(env->cpucfg[2], CPUCFG2, LASX, 0);

+ if (cpu->lasx == ON_OFF_AUTO_ON) {

+ if (kvm_supported) {

+ env->cpucfg[2] = FIELD_DP32(env->cpucfg[2], CPUCFG2, LASX, 1);

+ } else {

+ error_setg(errp, "'lasx' feature not supported by KVM on host");

+ return -ENOTSUP;

+ }

+ } else if ((cpu->lasx == ON_OFF_AUTO_AUTO) && kvm_supported) {

+ env->cpucfg[2] = FIELD_DP32(env->cpucfg[2], CPUCFG2, LASX, 1);

+ }

+

+ return 0;

+}

+

static int kvm_cpu_check_lbt(CPUState *cs, Error **errp)

{

CPULoongArchState *env = cpu_env(cs);

@@ -XXX,XX +XXX,XX @@ int kvm_arch_init_vcpu(CPUState *cs)

error_report_err(local_err);

}

+ ret = kvm_cpu_check_lasx(cs, &local_err);

+ if (ret < 0) {

+ error_report_err(local_err);

+ }

+

ret = kvm_cpu_check_lbt(cs, &local_err);

if (ret < 0) {

error_report_err(local_err);

--

2.43.5