ed-off-by: Zhuocheng Ding
Co-developed-by: Zhuocheng Ding
Signed-off-by: Zhao Liu
Tested-by: Babu Moger
Message-ID: <20240424154929.1487382-16-zhao1@intel.com>
Signed-off-by: Paolo Bonzini
---
include/hw/i386/topology.h | 17 +
hw/i386/x86-common.c | 27 ++
-off-by: Zhao Liu
Tested-by: Babu Moger
Message-ID: <20240424154929.1487382-17-zhao1@intel.com>
Signed-off-by: Paolo Bonzini
---
target/i386/cpu.h| 1 +
hw/i386/x86-common.c | 33 +
target/i386/cpu.c| 2 ++
3 files changed, 28 insertions(+), 8 del
1
modules.
Tested-by: Yongwei Ma
Signed-off-by: Zhao Liu
Tested-by: Babu Moger
Message-ID: <20240424154929.1487382-15-zhao1@intel.com>
Signed-off-by: Paolo Bonzini
---
include/hw/i386/topology.h | 6 --
target/i386/cpu.h | 1 +
hw/i386/x86-common.c | 2 +-
t
bu Moger
Message-ID: <20240424154929.1487382-19-zhao1@intel.com>
Signed-off-by: Paolo Bonzini
---
hw/i386/pc.c| 1 +
qemu-options.hx | 18 ++
2 files changed, 11 insertions(+), 8 deletions(-)
diff --git a/hw/i386/pc.c b/hw/i386/pc.c
index 6126bfdd2a7..7b638da7aaa
Signed-off-by: Zhuocheng Ding
Signed-off-by: Zhao Liu
Tested-by: Babu Moger
Message-ID: <20240424154929.1487382-14-zhao1@intel.com>
Signed-off-by: Paolo Bonzini
---
include/hw/i386/topology.h | 22 +++
hw/i386/x86.c | 9 +++-
target/i386/cpu.c
d-by: Yongwei Ma
Signed-off-by: Zhao Liu
Tested-by: Babu Moger
Message-ID: <20240424154929.1487382-7-zhao1@intel.com>
Signed-off-by: Paolo Bonzini
---
target/i386/cpu.c | 50 +--
1 file changed, 40 insertions(+), 10 deletions(-)
diff --git
CPUCacheInfo.share_level.
Signed-off-by: Zhao Liu
Tested-by: Babu Moger
Tested-by: Yongwei Ma
Acked-by: Michael S. Tsirkin
Message-ID: <20240424154929.1487382-20-zhao1@intel.com>
Signed-off-by: Paolo Bonzini
---
target/i386/cpu.h | 7 +++
target/i386/cpu.
Zhuocheng Ding
Signed-off-by: Zhuocheng Ding
Signed-off-by: Zhao Liu
Tested-by: Babu Moger
Message-ID: <20240424154929.1487382-13-zhao1@intel.com>
Signed-off-by: Paolo Bonzini
---
target/i386/cpu.h| 3 +++
hw/i386/x86-common.c | 5 +
target/i386/cpu.c| 1 +
3 files chang
uo
Reviewed-by: Xiaoyao Li
Reviewed-by: Zhao Liu
Message-ID: <20240112060042.19925-3-binbin...@linux.intel.com>
Signed-off-by: Paolo Bonzini
---
target/i386/cpu.h| 7 ++-
target/i386/helper.c | 4
2 files changed, 10 insertions(+), 1 deletion(-)
diff --git a/target/i386/cpu.
, wrap the encoding of CPUID[4].EAX[bits 31:26] into a
helper to make the code cleaner.
Tested-by: Yongwei Ma
Signed-off-by: Zhao Liu
Tested-by: Babu Moger
Message-ID: <20240424154929.1487382-21-zhao1@intel.com>
Signed-off-by: Paolo Bonzini
---
target/i386/cpu.
S. Tsirkin
Message-ID: <20240424154929.1487382-18-zhao1@intel.com>
Signed-off-by: Paolo Bonzini
---
tests/unit/test-x86-topo.c | 19 +++
1 file changed, 15 insertions(+), 4 deletions(-)
diff --git a/tests/unit/test-x86-topo.c b/tests/unit/test-x86-topo.c
index f21b8a5d
: 267e071bd6d6 (“hyperv: make overlay pages for SynIC”)
Suggested-by: Chao Gao
Signed-off-by: Dongsheng Zhang
Message-ID: <20240521200114.11588-1-dongsheng.x.zh...@intel.com>
Cc: qemu-sta...@nongnu.org
Signed-off-by: Paolo Bonzini
---
target/i386/kvm/hyperv.c | 2 +-
1 file changed, 1 inse
.
gen_RCL and gen_RCR use a conditional branch to skip the computation
if count is zero, so they can pass false unconditionally to gen_rot_overflow.
Signed-off-by: Paolo Bonzini
Reviewed-by: Richard Henderson
Message-ID: <20240522123914.608516-1-pbonz...@redhat.com>
Signed-off-by: Paolo Bonz
APM, vol.3, appendix.E.4.15 Function 8000_001Dh--Cache Topology
Information
Tested-by: Yongwei Ma
Signed-off-by: Zhao Liu
Reviewed-by: Babu Moger
Tested-by: Babu Moger
Reviewed-by: Xiaoyao Li
Message-ID: <20240424154929.1487382-8-zhao1....@intel.com>
Signed-off-by: Paolo Bo
level to be encoded
into CPUID[0x801D].EAX[bits 25:14].
Tested-by: Yongwei Ma
Signed-off-by: Zhao Liu
Tested-by: Babu Moger
Reviewed-by: Babu Moger
Message-ID: <20240424154929.1487382-22-zhao1@intel.com>
Signed-off-by: Paolo Bonzini
---
target/i386/cpu.c | 10 +-
1 file c
uot;system.flash0" memory region only appears standalone when "isa-bios" is
an alias.
Signed-off-by: Bernhard Beschow
Message-ID: <20240508175507.22270-7-shen...@gmail.com>
Signed-off-by: Paolo Bonzini
---
include/hw/i386/pc.h | 1 +
hw/i386/pc.c | 1 +
hw/i386/pc_piix.c
lippe Mathieu-Daudé
Tested-by: Babu Moger
Message-ID: <20240424154929.1487382-11-zhao1@intel.com>
Signed-off-by: Paolo Bonzini
---
target/i386/cpu.h | 13 +
target/i386/cpu.c | 14 +++---
2 files changed, 16 insertions(+), 11 deletions(-)
diff --git a/target/i386/cpu
CPUID[0x1F] subleaf
with specific topology level.
Tested-by: Yongwei Ma
Signed-off-by: Zhao Liu
Tested-by: Babu Moger
Reviewed-by: Xiaoyao Li
Message-ID: <20240424154929.1487382-12-zhao1....@intel.com>
Signed-off-by: Paolo Bonzini
---
target/i386/cpu.c | 135 +++
ology details.
This is also in preparation for the follow-up to decouple CPUID[0x1F]
subleaf with specific topology level.
Tested-by: Yongwei Ma
Signed-off-by: Zhao Liu
Tested-by: Babu Moger
Reviewed-by: Xiaoyao Li
Message-ID: <20240424154929.1487382-10-zhao1@intel.com>
Signed-of
ested-by: Robert Hoo
Tested-by: Yongwei Ma
Signed-off-by: Zhao Liu
Reviewed-by: Xiaoyao Li
Reviewed-by: Philippe Mathieu-Daudé
Tested-by: Babu Moger
Message-ID: <20240424154929.1487382-9-zhao1....@intel.com>
Signed-off-by: Paolo Bonzini
---
target/i386/cpu.c | 31 ++---
turn value
Bernhard Beschow (1):
hw/i386/pc_sysfw: Alias rather than copy isa-bios region
Binbin Wu (1):
target/i386: add control bits support for LAM
Paolo Bonzini (2):
target/i386: generate simpler code for ROL/ROR with immediate count
target/i386: clean up AAM/AAD
lt;20240424154929.1487382-6-zhao1@intel.com>
[Add compat property. - Paolo]
Signed-off-by: Paolo Bonzini
---
hw/i386/pc.c | 1 +
target/i386/cpu.c | 6 --
2 files changed, 5 insertions(+), 2 deletions(-)
diff --git a/hw/i386/pc.c b/hw/i386/pc.c
index 4a2d6f5a97f..6126bfdd2a7 100644
--- a/hw/i
y: Xiaoyao Li
Reviewed-by: Zhao Liu
Message-ID: <20240112060042.19925-2-binbin...@linux.intel.com>
Signed-off-by: Paolo Bonzini
---
target/i386/cpu.h | 2 ++
target/i386/cpu.c | 2 +-
2 files changed, 3 insertions(+), 1 deletion(-)
diff --git a/target/i386/cpu.h b/target/i386/cpu.h
i
t;regs[R_EAX] that generic load/writeback
code already does.
Signed-off-by: Paolo Bonzini
Reviewed-by: Richard Henderson
Message-ID: <20240522123912.608497-1-pbonz...@redhat.com>
Signed-off-by: Paolo Bonzini
---
target/i386/helper.h | 4 ++--
target/i386/tcg/int_helpe
On 5/22/24 21:30, Artyom Kunakovsky wrote:
fix linker error if the project was configured by the './configure
--cpu=unknown --target-list=riscv64-softmmu' command
Signed-off-by: Artyom Kunakovsky
---
meson.build | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/meson.build
On Wed, May 22, 2024 at 3:45 PM Paolo Bonzini wrote:
>
> On Wed, May 22, 2024 at 12:49 PM Akihiko Odaki
> wrote:
> > The non-standard .fa library prefix breaks the link source
> > de-duplication done by Meson so drop it.
>
> Can you show the difference in the command
On Wed, May 22, 2024 at 12:49 PM Akihiko Odaki wrote:
> The non-standard .fa library prefix breaks the link source
> de-duplication done by Meson so drop it.
Can you show the difference in the command lines?
One possibility to force de-duplication of objects is to change
"link_whole: foo" to "ob
.
gen_RCL and gen_RCR use a conditional branch to skip the computation
if count is zero, so they can pass false unconditionally to gen_rot_overflow.
Signed-off-by: Paolo Bonzini
---
target/i386/tcg/emit.c.inc | 26 ++
1 file changed, 14 insertions(+), 12 deletions(-)
di
t;regs[R_EAX] that generic load/writeback
code already does.
Signed-off-by: Paolo Bonzini
---
target/i386/helper.h | 4 ++--
target/i386/tcg/int_helper.c | 19 ---
target/i386/tcg/decode-new.c.inc | 4 ++--
target/i386/tcg/emit.c.inc | 8
4 files chang
Queued, thanks.
Paolo
Queued, thanks.
Paolo
Queued, thanks.
Paolo
Il mar 21 mag 2024, 09:10 Bernhard Beschow ha scritto:
> This is the only patch in this series which hasn't got an R-b tag yet (the
> others are already in master) and I'm not aware of any open issues.
>
I will queue it then.
Paolo
> Best regards,
> Bernhard
>
>
sions, and deprecate the property just in case someone is
using it as "pcommit=off".
Reviewed-by: Richard Henderson
Signed-off-by: Paolo Bonzini
---
docs/about/deprecated.rst | 8
target/i386/cpu.h | 2 --
target/i386/cpu.c | 2 +-
target/i386/
Reviewed-by: Zhao Liu
Signed-off-by: Paolo Bonzini
---
tests/tcg/i386/test-i386.c | 25 +
1 file changed, 25 insertions(+)
diff --git a/tests/tcg/i386/test-i386.c b/tests/tcg/i386/test-i386.c
index 864c4e620d5..ce3bf74b5a8 100644
--- a/tests/tcg/i386/test-i386.c
+++ b
ated
* fix --without-default-devices build
* fix --without-default-devices qtests on s390x and arm
--------
Paolo Bonzini (27):
target/i386: remove PCOMMIT from TCG, deprecate property
target/i386: fix operand size for DATA16 RE
mentioned explicitly in --target-list, in which case
the build will fail.
Reviewed-by: Philippe Mathieu-Daudé
Signed-off-by: Paolo Bonzini
---
configs/targets/aarch64-softmmu.mak | 1 +
configs/targets/arm-softmmu.mak | 1 +
configs/targets/i386-softmmu.mak | 1 -
confi
All other boards require libfdt if it can be used (including for example
i386/x86_64), so change the "imply" to "select" and always allow -dtb
in qemu-system-xtensa.
Reviewed-by: Philippe Mathieu-Daudé
Signed-off-by: Paolo Bonzini
---
hw/xtensa/xtfpga.c | 9 -
hw
Reject 0x66/0xf3/0xf2 in front of them.
Cc: qemu-sta...@nongnu.org
Reviewed-by: Richard Henderson
Signed-off-by: Paolo Bonzini
---
target/i386/tcg/translate.c | 6 --
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/target/i386/tcg/translate.c b/target/i386/tcg/translate.c
ARM/aarch64 are easy to fix because they already have to pass a machine
type by hand. Just guard the tests with a check that the machine actually
exists.
Signed-off-by: Paolo Bonzini
Reviewed-by: Philippe Mathieu-Daudé
Reviewed-by: Thomas Huth
Message-ID: <20240509170044.190795-14-pb
backend_register callback, whose only purpose is to avoid registering
nonfunctional backends.
Signed-off-by: Paolo Bonzini
Reviewed-by: Philippe Mathieu-Daudé
Message-ID: <20240509170044.190795-8-pbonz...@redhat.com>
Signed-off-by: Paolo Bonzini
---
include/hw/xen/xen-legacy-backend.
}
which prints 5//5 on real hardware and 5/0000/ffff
on QEMU.
Cc: qemu-sta...@nongnu.org
Reviewed-by: Zhao Liu
Reviewed-by: Richard Henderson
Signed-off-by: Paolo Bonzini
---
target/i386/tcg/translate.c | 17 +
1 file changed, 1 insertion(+), 16 deletion
d but the APIC is.
Signed-off-by: Paolo Bonzini
Reviewed-by: Zhao Liu
Message-ID: <20240509170044.190795-13-pbonz...@redhat.com>
Signed-off-by: Paolo Bonzini
---
hw/intc/ioapic-stub.c | 29 +
.gitlab-ci.d/buildtest.yml | 2 +-
hw/intc/meson.build| 2
The VMX feature bit depends on general availability of WAITPKG,
not the other way round.
Fixes: 33cc88261c3 ("target/i386: add support for
VMX_SECONDARY_EXEC_ENABLE_USER_WAIT_PAUSE", 2023-08-28)
Cc: qemu-sta...@nongnu.org
Reviewed-by: Zhao Liu
Signed-off-by: Paolo Bonzini
---
t
Acked-by: Alex Bennée
Signed-off-by: Paolo Bonzini
---
.gitlab-ci.d/buildtest.yml | 5 +++--
hw/sh4/meson.build | 2 +-
target/sh4/Kconfig | 2 ++
3 files changed, 6 insertions(+), 3 deletions(-)
diff --git a/.gitlab-ci.d/buildtest.yml b/.gitlab-ci.d/buildtest.yml
index
Introduce a new Kconfig symbol, CONFIG_DEVICE_TREE, that specifies whether
to include the common device tree code in system/device_tree.c and to
link to libfdt. For now, include it unconditionally if libfdt is
available.
Signed-off-by: Paolo Bonzini
---
meson.build | 1
Instead of mucking with css_migration_enabled(), add a property specific to
the FLIC device, similar to what is done for TYPE_S390_STATTRIB.
Signed-off-by: Paolo Bonzini
Reviewed-by: Thomas Huth
Message-ID: <20240509170044.190795-3-pbonz...@redhat.com>
Signed-off-by: Paolo Bonzini
---
i
Signed-off-by: Paolo Bonzini
Reviewed-by: Thomas Huth
Message-ID: <20240509170044.190795-5-pbonz...@redhat.com>
Signed-off-by: Paolo Bonzini
---
.gitlab-ci.d/buildtest.yml | 4 ++--
target/s390x/Kconfig | 2 ++
2 files changed, 4 insertions(+), 2 deletions(-)
diff --git a/.gitla
fw_cfg.c and vapic.c are currently included unconditionally but
depend on other components. vapic.c depends on the local APIC,
while fw_cfg.c includes a piece of AML builder code that depends
on CONFIG_ACPI.
Signed-off-by: Paolo Bonzini
Reviewed-by: Zhao Liu
Message-ID: <20240509170044.190
Avoid having to list dependencies such as libfdt twice, both on common_ss
and specific_ss. Instead, just take all the dependencies in common_ss
and allow the target-specific libqemu-*.fa library to use them.
Reviewed-by: Philippe Mathieu-Daudé
Signed-off-by: Paolo Bonzini
---
meson.build
to sigp.c for simplicity.
Signed-off-by: Paolo Bonzini
Reviewed-by: Thomas Huth
Message-ID: <20240509170044.190795-2-pbonz...@redhat.com>
Signed-off-by: Paolo Bonzini
---
hw/s390x/s390-virtio-ccw.c | 16
target/s390x/sigp.c| 17 +
2 files chang
n express
whether it needs FDT. It can then include the common code in the
build via "select DEVICE_TREE", which will also as tell meson to link
with libfdt.
This allows building non-microvm x86 emulators without having
libfdt available.
Reviewed-by: Philippe Mathieu-Daudé
Keep the basic X86MachineState definition in x86.c. Move out functions that
are only needed by other files: x86-common.c for the pc and microvm machines,
x86-cpu.c for those used by accelerator code.
Signed-off-by: Paolo Bonzini
Reviewed-by: Zhao Liu
Message-ID: <20240509170044.190795-11-pb
-by: Philippe Mathieu-Daudé
Signed-off-by: Paolo Bonzini
---
meson.build | 79 ++---
1 file changed, 38 insertions(+), 41 deletions(-)
diff --git a/meson.build b/meson.build
index ab1f44b25c2..dd4a28f8f8f 100644
--- a/meson.build
+++ b/meson.build
er,
a somewhat realistic failure case would be with -Dpkg_config_path
and a pkg-config directory that contains spaces.
Cc: qemu-sta...@nongnu.org
Reviewed-by: Thomas Huth
Signed-off-by: Paolo Bonzini
---
configure | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/configure b/configure
rything is
ready at the time the xen_backend_init() functions are called,
remove the xen_be_init() function from all the boards and
place it directly in xen_bus_init().
Signed-off-by: Paolo Bonzini
Reviewed-by: Philippe Mathieu-Daudé
Message-ID: <20240509170044.190795-7-pbonz...@redhat.com>
Sign
There is no need to override these fields of MachineClass because they are
already set to the right value in the superclass.
Signed-off-by: Paolo Bonzini
Reviewed-by: Zhao Liu
Message-ID: <20240509170044.190795-10-pbonz...@redhat.com>
Signed-off-by: Paolo Bonzini
---
include/hw/i386/x86
The CSS subsystem uses global variables, just face the truth and use
a variable also for whether the CSS vmstate is in use; remove the
indirection of fetching it from the machine type, which makes the
TCG code depend unnecessarily on the virtio-ccw machine.
Signed-off-by: Paolo Bonzini
Message
t get the memo; plus another one does
not need a machine at all and can be run with -M none.
Signed-off-by: Paolo Bonzini
Reviewed-by: Thomas Huth
Message-ID: <20240509170044.190795-6-pbonz...@redhat.com>
Signed-off-by: Paolo Bonzini
---
tests/qtest/drive_del-test.c | 7 ++-
1 file
tracking of all created RTC devices. Move the implementation
to hw/i386, so that 1) it is available even if no RTC device exist
2) the only RTC that exists is easily found in x86ms->rtc.
Signed-off-by: Paolo Bonzini
Reviewed-by: Zhao Liu
Message-ID: <20240509170044.190795-12-pbonz...@redh
Henderson
Reviewed-by: Zhao Liu
Signed-off-by: Paolo Bonzini
---
target/i386/tcg/decode-new.h | 1 +
target/i386/tcg/translate.c | 30 --
target/i386/tcg/decode-new.c.inc | 24 +---
target/i386/tcg/emit.c.inc | 5 +
4 files changed
On Fri, May 10, 2024 at 7:38 AM Thomas Huth wrote:
> I think this is wrong: By adding this to ccw_machine_2_9_class_options the
> variable now always gets set to false, even for newer machines, since the
> *class_options functions are part of the "class_init" which is always
done.
> You have to ad
ARM/aarch64 are easy to fix because they already have to pass a machine
type by hand. Just guard the tests with a check that the machine actually
exists.
Signed-off-by: Paolo Bonzini
---
tests/qtest/arm-cpu-features.c | 4
tests/qtest/migration-test.c | 6 ++
tests/qtest/numa-test.c
to sigp.c for simplicity.
Signed-off-by: Paolo Bonzini
---
hw/s390x/s390-virtio-ccw.c | 16
target/s390x/sigp.c| 17 +
2 files changed, 17 insertions(+), 16 deletions(-)
diff --git a/hw/s390x/s390-virtio-ccw.c b/hw/s390x/s390-virtio-ccw.c
index
fw_cfg.c and vapic.c are currently included unconditionally but
depend on other components. vapic.c depends on the local APIC,
while fw_cfg.c includes a piece of AML builder code that depends
on CONFIG_ACPI.
Signed-off-by: Paolo Bonzini
---
hw/i386/fw_cfg.c| 2 ++
hw/i386/meson.build | 2
d but the APIC is.
Signed-off-by: Paolo Bonzini
---
hw/intc/ioapic-stub.c | 29 +
.gitlab-ci.d/buildtest.yml | 2 +-
hw/intc/meson.build| 2 +-
target/i386/Kconfig| 1 +
4 files changed, 32 insertions(+), 2 deletions(-)
create mode 100644 hw
backend_register callback, whose only purpose is to avoid registering
nonfunctional backends.
Signed-off-by: Paolo Bonzini
---
include/hw/xen/xen-legacy-backend.h | 14 ++
include/hw/xen/xen_pvdev.h | 1 -
hw/9pfs/xen-9p-backend.c| 8 +++-
hw/display
Instead of mucking with css_migration_enabled(), add a property specific to
the FLIC device, similar to what is done for TYPE_S390_STATTRIB.
Signed-off-by: Paolo Bonzini
---
include/hw/s390x/s390_flic.h | 1 +
hw/intc/s390_flic.c | 6 +-
hw/s390x/s390-virtio-ccw.c | 1 +
3 files
The CSS subsystem uses global variables, just face the truth and use
a variable also for whether the CSS vmstate is in use; remove the
indirection of fetching it from the machine type, which makes the
TCG code depend unnecessarily on the virtio-ccw machine.
Signed-off-by: Paolo Bonzini
rything is
ready at the time the xen_backend_init() functions are called,
remove the xen_be_init() function from all the boards and
place it directly in xen_bus_init().
Signed-off-by: Paolo Bonzini
---
hw/i386/pc.c | 1 -
hw/xen/xen-bus.c | 4
hw/xen/xen-hvm-common.c
ut boards on vanilla
upstream configs is the more pressing problem, though.
Patches 6 and 7 were tested with the Avocado Xen-on-KVM tests.
Paolo
Paolo Bonzini (13):
s390x: move s390_cpu_addr2state to target/s390x/sigp.c
s390_flic: add migration-enabled property
s390: move css_migration_ena
t get the memo; plus another one does
not need a machine at all and can be run with -M none.
Signed-off-by: Paolo Bonzini
---
tests/qtest/drive_del-test.c | 7 ++-
1 file changed, 6 insertions(+), 1 deletion(-)
diff --git a/tests/qtest/drive_del-test.c b/tests/qtest/drive_del-test.c
tracking of all created RTC devices. Move the implementation
to hw/i386, so that 1) it is available even if no RTC device exist
2) the only RTC that exists is easily found in x86ms->rtc.
Signed-off-by: Paolo Bonzini
---
include/hw/rtc/mc146818rtc.h | 2 +-
hw/i386/monitor.c|
Keep the basic X86MachineState definition in x86.c. Move out functions that
are only needed by other files: x86-common.c for the pc and microvm machines,
x86-cpu.c for those used by accelerator code.
Signed-off-by: Paolo Bonzini
---
include/hw/i386/x86.h |6 +-
hw/i386/x86-common.c | 1007
There is no need to override these fields of MachineClass because they are
already set to the right value in the superclass.
Signed-off-by: Paolo Bonzini
---
include/hw/i386/x86.h | 4
hw/i386/pc.c | 3 ---
hw/i386/x86.c | 6 +++---
3 files changed, 3 insertions(+), 10
Signed-off-by: Paolo Bonzini
---
.gitlab-ci.d/buildtest.yml | 4 ++--
target/s390x/Kconfig | 2 ++
2 files changed, 4 insertions(+), 2 deletions(-)
diff --git a/.gitlab-ci.d/buildtest.yml b/.gitlab-ci.d/buildtest.yml
index 13afd0df1f0..f8502905203 100644
--- a/.gitlab-ci.d/buildtest.yml
The XSAVEOPT, XSAVEC, XGETBV1, XSAVES features make no sense if you
cannot enable XSAVE in the first place.
Signed-off-by: Paolo Bonzini
---
target/i386/cpu.c | 4
1 file changed, 4 insertions(+)
diff --git a/target/i386/cpu.c b/target/i386/cpu.c
index f2ea6899e39..6f5ff71c6ee 100644
The VMX feature bit depends on general availability of WAITPKG,
not the other way round.
Fixes: 33cc88261c3 ("target/i386: add support for
VMX_SECONDARY_EXEC_ENABLE_USER_WAIT_PAUSE", 2023-08-28)
Cc: qemu-sta...@nongnu.org
Signed-off-by: Paolo Bonzini
---
target/i386/cpu.c | 4 ++
These are trivial to add, and moving them to the new decoder fixes some
corner cases: raising #UD instead of an instruction fetch page fault for
the undefined opcodes, and incorrectly rejecting 0F 18 prefetches with
register operands (which are treated as reserved NOPs).
Signed-off-by: Paolo
);
}
which prints 5//5 on real hardware and 5//
on QEMU.
Cc: qemu-sta...@nongnu.org
Signed-off-by: Paolo Bonzini
---
target/i386/tcg/translate.c | 17 +
1 file changed, 1 insertion(+), 16 deletions(-)
diff --git a/target/i386/tcg/translate.c b/t
Signed-off-by: Paolo Bonzini
---
tests/tcg/i386/test-i386.c | 25 +
1 file changed, 25 insertions(+)
diff --git a/tests/tcg/i386/test-i386.c b/tests/tcg/i386/test-i386.c
index 864c4e620d5..ce3bf74b5a8 100644
--- a/tests/tcg/i386/test-i386.c
+++ b/tests/tcg/i386/test-i386
Reject 0x66/0xf3/0xf2 in front of them.
Cc: qemu-sta...@nongnu.org
Signed-off-by: Paolo Bonzini
---
target/i386/tcg/translate.c | 6 --
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/target/i386/tcg/translate.c b/target/i386/tcg/translate.c
index 5366dc32dd3..3da4fdf64cc
On Tue, Apr 30, 2024 at 5:39 PM Philippe Mathieu-Daudé
wrote:
> I'm still not convinced we need a migration back compat for this...
It's absolutely needed,
memory_region_init_ram(isa_bios, NULL, "isa-bios", isa_bios_size,
&error_fatal);
will register a RAM region
sions, and deprecate the property just in case someone is
using it as "pcommit=off".
Signed-off-by: Paolo Bonzini
---
docs/about/deprecated.rst | 8
target/i386/cpu.h | 2 --
target/i386/cpu.c | 2 +-
target/i386/tcg/translate.c | 12 +---
On Wed, May 8, 2024 at 2:51 PM Daniel P. Berrangé wrote:
> Designwise, a native VSOCK backend in QEMU really should implement the
> same approach defined by firecracker, so that we have interoperability
> with systemd, firecracker and cloud-hypervisor. See
>
> https://gitlab.com/qemu-project/qem
On 5/8/24 11:38, Stefano Garzarella wrote:
On Wed, May 08, 2024 at 01:13:09PM GMT, Marc-André Lureau wrote:
Hi
On Wed, May 8, 2024 at 11:50 AM Stefano Garzarella
wrote:
Hi Roman,
On Tue, May 07, 2024 at 11:20:50PM GMT, Roman Kiryanov wrote:
>Hi Stefano,
>
>On Tue, May 7, 2024 at 1:10 AM St
: Paolo Bonzini
---
docs/about/deprecated.rst | 10 --
docs/about/removed-features.rst | 8 ++
hw/block/virtio-blk.c | 166 +++-
hw/core/machine.c | 2 -
4 files changed, 19 insertions(+), 167 deletions(-)
diff --git a/docs/about
All other boards require libfdt if it can be used (including for example
i386/x86_64), so change the "imply" to "select" and always allow -dtb
in qemu-system-xtensa.
Reviewed-by: Philippe Mathieu-Daudé
Signed-off-by: Paolo Bonzini
---
hw/xtensa/xtfpga.c | 9 -
hw
mentioned explicitly in --target-list, in which case
the build will fail.
Reviewed-by: Philippe Mathieu-Daudé
Signed-off-by: Paolo Bonzini
---
configs/targets/aarch64-softmmu.mak | 1 +
configs/targets/arm-softmmu.mak | 1 +
configs/targets/i386-softmmu.mak | 1 -
confi
Introduce a new Kconfig symbol, CONFIG_DEVICE_TREE, that specifies whether
to include the common device tree code in system/device_tree.c and to
link to libfdt. For now, include it unconditionally if libfdt is
available.
Signed-off-by: Paolo Bonzini
---
meson.build | 1
-off-by: Paolo Bonzini
---
meson.build | 79 ++---
1 file changed, 38 insertions(+), 41 deletions(-)
diff --git a/meson.build b/meson.build
index 7ca0ba4987f..2e7e2b15406 100644
--- a/meson.build
+++ b/meson.build
@@ -1858,6 +1858,34 @@ if numa.found
n express
whether it needs FDT. It can then include the common code in the
build via "select DEVICE_TREE", which will also as tell meson to link
with libfdt.
This allows building non-microvm x86 emulators without having
libfdt available.
Reviewed-by: Philippe Mathieu-Daudé
Avoid having to list dependencies such as libfdt twice, both on common_ss
and specific_ss. Instead, just take all the dependencies in common_ss
and allow the target-specific libqemu-*.fa library to use them.
Signed-off-by: Paolo Bonzini
---
meson.build | 14 +++---
hw/arm
the --without-default-features
mode", 2022-02-21)
- patch 6 does not
- split patch 3 in two parts
- fix compilation errors due to missing qmp_dumpdtb/hmp_dumpdtb
- make MIPS_BOSTON depend on FDT
Paolo Bonzini (6):
meson: pick libfdt from common_ss when building target-specific fil
Ensure that it can be used even if virt.c is not included in the build, as
is the case for --without-default-devices.
Signed-off-by: Paolo Bonzini
---
include/hw/loongarch/boot.h | 10 ++
include/hw/loongarch/virt.h | 10 --
hw/loongarch/boot.c | 3 +++
hw/loongarch
Signed-off-by: Paolo Bonzini
---
.gitlab-ci.d/buildtest.yml | 5 +++--
hw/sh4/meson.build | 2 +-
target/sh4/Kconfig | 2 ++
3 files changed, 6 insertions(+), 3 deletions(-)
diff --git a/.gitlab-ci.d/buildtest.yml b/.gitlab-ci.d/buildtest.yml
index bab61945643..13afd0df1f0
On Tue, May 7, 2024 at 1:58 PM Peter Maydell wrote:
>
> On Tue, 7 May 2024 at 11:50, Paolo Bonzini wrote:
> >
> > Ensure that they go through unmodified, instead of removing one layer
> > of quoting.
>
> Do you have an example of what goes wrong that we could
> m
should have done it.
Reviewed-by: Richard Henderson
Signed-off-by: Paolo Bonzini
---
target/i386/tcg/translate.c | 8 +---
1 file changed, 5 insertions(+), 3 deletions(-)
diff --git a/target/i386/tcg/translate.c b/target/i386/tcg/translate.c
index 466fee38c0e..024da6d88eb 100644
--- a/target
comparison against zero anyway, and it avoids shifts by 64
which are undefined behavior.
Reviewed-by: Richard Henderson
Signed-off-by: Paolo Bonzini
---
target/i386/tcg/translate.c | 28
target/i386/tcg/emit.c.inc | 5 ++---
2 files changed, 22 insertions(+), 11
Send all converted opcodes to disas_insn_new() directly from the big
decoding switch statement; once more, the debugging/bisecting logic
disappears.
Reviewed-by: Richard Henderson
Signed-off-by: Paolo Bonzini
---
target/i386/helper.h| 11 -
target/i386/tcg
701 - 800 of 25505 matches
Mail list logo