From: Peter Maydell
Honour the commandline -semihosting-config userspace=on option,
instead of never permitting userspace semihosting calls in system
emulation mode, by passing the correct value to the is_userspace
argument of semihosting_enabled(), instead of manually checking and
always forbidd
On 8/26/22 16:20, Emanuele Giuseppe Esposito wrote:
In order to make it thread safe, implement a "fake rwlock",
where we allow reads under BQL *or* job_mutex held, but
writes only under BQL *and* job_mutex.
The only write we have is in child_job_set_aio_ctx, which always
happens under drain (so
From: Peter Maydell
The riscv target incorrectly enabled semihosting always, whether the
user asked for it or not. Call semihosting_enabled() passing the
correct value to the is_userspace argument, which fixes this and also
handles the userspace=on argument. Because we do this at translate
time
On 9/14/22 13:44, BALATON Zoltan wrote:
On Wed, 14 Sep 2022, Cédric Le Goater wrote:
On 9/13/22 21:52, BALATON Zoltan wrote:
The do_init parameter of ppc4xx_sdram_init() is used to map memory
regions that is normally done by the firmware by programming the SDRAM
controller. This is needed when
On 8/26/22 16:21, Emanuele Giuseppe Esposito wrote:
These public functions are not used anywhere, thus can be dropped.
Also, since this is the final job API that doesn't use AioContext
lock and replaces it with job_lock, adjust all remaining function
documentation to clearly specify if the job lo
On 8/26/22 16:21, Emanuele Giuseppe Esposito wrote:
These public functions are not used anywhere, thus can be dropped.
Signed-off-by: Emanuele Giuseppe Esposito
Reviewed-by: Stefan Hajnoczi
Reviewed-by: Kevin Wolf
Reviewed-by: Vladimir Sementsov-Ogievskiy
--
Best regards,
Vladimir
On 9/14/2022 2:57 PM, Eugenio Perez Martin wrote:
On Wed, Sep 14, 2022 at 1:33 PM Si-Wei Liu wrote:
On 9/14/2022 3:20 AM, Jason Wang wrote:
On Fri, Sep 9, 2022 at 4:02 PM Eugenio Perez Martin wrote:
On Fri, Sep 9, 2022 at 8:40 AM Jason Wang wrote:
On Fri, Sep 9, 2022 at 2:38 PM Jason
We already limit the scope of the cross system build to reduce the
cross build times. With the recent addition of more targets we are
also running into timeout issues for some of the cross user builds.
I've selected a few of those linux-user targets which are less likely
to be in common use as dis
From: Thomas Huth
The assets that this test tries to download have been removed from the
server. Update to a newer version to get it working again.
Signed-off-by: Thomas Huth
Reviewed-by: Alistair Francis
Message-Id: <20220829080940.110831-1-th...@redhat.com>
Signed-off-by: Alex Bennée
---
t
Hi,
This is the current state of the testing tree. It was slightly delayed
from the last posting as I was waiting for the libvirt-ci update to
get merged. Changes since last post:
- added some more avocado timeouts
- moved cflags hack in the --disable-pie patch into configure
- usual addres
The SDK tests take a lot longer to run and hence need a longer
timeout. As they run well over the 60 second maximum for CI lets also
disable them for CI as well.
I suspect they also suffer from the inability to detect the login
prompt due to no newlines being processed.
Signed-off-by: Alex Bennée
Update to the latest stable Debian. While we are at it flatten into a
single dockerfile. We really don't need the rest of the stuff from
the QEMU base image just to compile test images.
Signed-off-by: Alex Bennée
Reviewed-by: Thomas Huth
Message-Id: <20220826172128.353798-5-alex.ben...@linaro.or
We don't want to rely on the soon to be reduced default time. These
tests are still slow for something we want to run in CI though.
Signed-off-by: Alex Bennée
---
tests/avocado/boot_linux.py | 2 ++
1 file changed, 2 insertions(+)
diff --git a/tests/avocado/boot_linux.py b/tests/avocado/boot_li
We don't want to rely on the soon to be reduced default time. These
tests are still slow for something we want to run in CI though.
Signed-off-by: Alex Bennée
---
tests/avocado/boot_linux.py | 1 +
1 file changed, 1 insertion(+)
diff --git a/tests/avocado/boot_linux.py b/tests/avocado/boot_linu
Update to the latest stable Debian. While we are at it flatten into a
single dockerfile. We really don't need the rest of the stuff from
the QEMU base image just to compile test images.
Signed-off-by: Alex Bennée
Reviewed-by: Thomas Huth
Message-Id: <20220826172128.353798-6-alex.ben...@linaro.or
We should be aiming to keep our tests under 2 minutes so lets reduce
the default timeout to that. Tests that we know take longer should
explicitly set a longer timeout.
Signed-off-by: Alex Bennée
Message-Id: <20220826172128.353798-3-alex.ben...@linaro.org>
---
tests/avocado/avocado_qemu/__init__
We missed removing this dependency when we flattened the build.
Fixes 9e19fd7d4a (tests/docker: update debian-amd64 with lcitool)
Signed-off-by: Alex Bennée
Reviewed-by: Thomas Huth
Message-Id: <20220826172128.353798-12-alex.ben...@linaro.org>
---
tests/docker/Makefile.include | 1 -
1 file ch
Update to the latest stable Debian. While we are at it flatten into a
single dockerfile. We really don't need the rest of the stuff from
the QEMU base image just to compile test images.
Signed-off-by: Alex Bennée
Reviewed-by: Thomas Huth
Message-Id: <20220826172128.353798-4-alex.ben...@linaro.or
Update to the latest stable Debian. While we are at it flatten into a
single dockerfile. We really don't need the rest of the stuff from
the QEMU base image just to compile test images.
Signed-off-by: Alex Bennée
Reviewed-by: Thomas Huth
Message-Id: <20220826172128.353798-8-alex.ben...@linaro.or
We don't want to rely on the soon to be reduced default time. These
tests are still slow for something we want to run in CI though.
Signed-off-by: Alex Bennée
---
tests/avocado/boot_linux.py | 2 ++
1 file changed, 2 insertions(+)
diff --git a/tests/avocado/boot_linux.py b/tests/avocado/boot_li
Update to the latest stable Debian. While we are at it flatten into a
single dockerfile. We really don't need the rest of the stuff from
the QEMU base image just to compile test images.
Signed-off-by: Alex Bennée
Reviewed-by: Thomas Huth
Message-Id: <20220826172128.353798-9-alex.ben...@linaro.or
Flatten into a single dockerfile. We really don't need the rest of the
stuff from the QEMU base image just to compile test images.
Signed-off-by: Alex Bennée
Reviewed-by: Thomas Huth
Message-Id: <20220826172128.353798-10-alex.ben...@linaro.org>
---
.gitlab-ci.d/container-cross.yml
We missed removing this dependency when we flattened the build.
Fixes: 39ce923732 (gitlab: enable a very minimal build with the tricore
container)
Signed-off-by: Alex Bennée
Fixes: 39ce923732 ("gitlab: enable a very minimal build with the tricore
container")
Message-Id: <20220826172128.353798-
From: Thomas Huth
It's still based on Fedora 30 - which is not supported anymore by QEMU
since years. Seems like nobody is using (and refreshing) this, and it's
easier to test this via a container anyway, so let's remove this now.
Signed-off-by: Thomas Huth
Message-Id: <20220822175317.190551-1-
It's becoming harder to maintain a cross-compiler to test this host
architecture as the old stable Debian 10 ("Buster") moved into LTS
which supports fewer architectures. For now:
- mark it's deprecation in the docs
- downgrade the containers to build TCG tests only
- drop the cross builds f
We want to migrate from docker.py to building our images directly with
docker/podman. Before we get there we need to make sure we don't
re-introduce our layered builds so bug out if we see FROM qemu/ in a
Dockerfile.
Signed-off-by: Alex Bennée
Acked-by: Thomas Huth
Message-Id: <20220826172128.35
From: Richard Henderson
The project has reached the magic size at which we see
/usr/aarch64-linux-gnu/lib/libc.a(init-first.o): in function
`__libc_init_first':
(.text+0x10): relocation truncated to fit: R_AARCH64_LD64_GOTPAGE_LO15 against \
symbol `__environ' defined in .bss section in
/usr/a
Update to the latest stable Debian. While we are at it flatten into a
single dockerfile as we do not need anything from the base image to
build the toolchain. This is used to build both the nios and
microblaze toolchains.
Signed-off-by: Alex Bennée
Reviewed-by: Thomas Huth
Message-Id: <202208261
Now lcitool has support for building a x86_64 cross image we can use
it for this.
Signed-off-by: Alex Bennée
Acked-by: Thomas Huth
Message-Id: <20220826172128.353798-20-alex.ben...@linaro.org>
---
.gitlab-ci.d/container-cross.yml | 1 -
tests/docker/Makefile.include
Update to the latest stable Debian. While we are at it flatten into a
single dockerfile as we do not some of the extraneous packages from
the base image to build the toolchain.
Signed-off-by: Alex Bennée
Message-Id: <20220826172128.353798-22-alex.ben...@linaro.org>
---
v2
- use debian11 consis
Update to the latest stable Debian. While we are at it flatten into a
single dockerfile. We really don't need the rest of the stuff from the
QEMU base image just to compile test images. In this case it is a
binary distribution of the toolchain anyway.
Signed-off-by: Alex Bennée
Reviewed-by: Thoma
The custom runner is now using 22.04 so we can drop our hacks to deal
with broken libssh and glusterfs. The provisioning scripts will be
updated in a separate commit.
Signed-off-by: Alex Bennée
Reviewed-by: Thomas Huth
Message-Id: <20220826172128.353798-15-alex.ben...@linaro.org>
---
v2
- fix
We need this to be able to cleanly build the x86 cross images. There
are a few minor updates triggered by lcitool-refresh including adding
"libslirp" to the freebsd vars and opensuse-leap which will help when
we finally drop the slirp submodule from QEMU.
Signed-off-by: Alex Bennée
Reviewed-by: T
We no longer use these in any of our images. Clean-up the remaining
comments and documentation that reference them and remove from the
build.
Signed-off-by: Alex Bennée
Reviewed-by: Thomas Huth
Message-Id: <20220826172128.353798-25-alex.ben...@linaro.org>
---
docs/devel/testing.rst
Now that MxTxAttrs encodes a CPU we should use that to figure it out.
This solves edge cases like accessing via gdbstub or qtest.
Signed-off-by: Alex Bennée
Resolves: https://gitlab.com/qemu-project/qemu/-/issues/124
---
hw/intc/arm_gic.c | 39 ++-
1 file chan
Update to the latest stable Debian. While we are at it flatten into a
single dockerfile. We really don't need the rest of the stuff from
the QEMU base image just to compile test images.
Signed-off-by: Alex Bennée
Reviewed-by: Thomas Huth
Message-Id: <20220826172128.353798-7-alex.ben...@linaro.or
We currently have hacks across the hw/ to reference current_cpu to
work out what the current accessing CPU is. This breaks in some cases
including using gdbstub to access HW state. As we have MemTxAttrs to
describe details about the access lets extend it to mention if this is
a CPU access and which
Flatten into a single dockerfile and update to match the rest of the
test cross compile dockerfiles.
Signed-off-by: Alex Bennée
Message-Id: <20220826172128.353798-17-alex.ben...@linaro.org>
---
v2
- minor reword of commit msg
---
.gitlab-ci.d/container-cross.yml | 1 -
This removes the hacks to deal with empty current_cpu.
Signed-off-by: Alex Bennée
---
hw/timer/arm_mptimer.c | 25 ++---
1 file changed, 14 insertions(+), 11 deletions(-)
diff --git a/hw/timer/arm_mptimer.c b/hw/timer/arm_mptimer.c
index cdfca3000b..a7fe6ddc0d 100644
--- a/h
This is working around current limitation of Meson's handling of
--disable-pie.
Signed-off-by: Alex Bennée
Cc: Paolo Bonzini
---
configure | 6 ++
1 file changed, 6 insertions(+)
diff --git a/configure b/configure
index 575dde1c1f..6c169b23b5 100755
--- a/configure
+++ b/configure
@@ -1394
Hi,
This ostensibly fixes a bug with gdbstub when accessing the GIC state
(via system registers). It also proposes a pattern for removing
current_cpu/qtest_enabled() hacks in hw/ by passing the cpu index via
MemTxAttrs. For the ARM timer code we also assert that those accesses
do come from a CPU (
Update to the latest stable Debian. While we are at it flatten into a
single dockerfile. We also need to ensure we install clang as it is
used for those builds as well.
It would be nice to port this to lcitool but for now this will do.
Signed-off-by: Alex Bennée
Reviewed-by: Thomas Huth
Message
On Mon, Sep 12, 2022 at 3:56 PM Richard Henderson
wrote:
> > +static void gen_SSE4a_I(DisasContext *s, CPUX86State *env, X86DecodedInsn
> > *decode)
> > +{
> > +TCGv_i32 length = tcg_const_i32(decode->immediate & 255);
> > +TCGv_i32 index = tcg_const_i32(decode->immediate >> 8);
> > +
>
The point of qtest is to simulate how running code might interact with
the system. However because it's not a real system we have places in
the code which especially handle check qtest_enabled() before
referencing current_cpu. Now we can encode these details in the
MemTxAttrs lets do that so we can
Le 29/08/2022 à 07:17, Richard Henderson a écrit :
This is slightly more complicated than cas,
because tas is allowed on data registers.
Signed-off-by: Richard Henderson
---
target/m68k/translate.c | 40 ++--
1 file changed, 30 insertions(+), 10 deletions(
On Tue, Sep 13, 2022 at 11:31 AM Richard Henderson
wrote:
> > +void glue(helper_vpmaskmovq_st, SUFFIX)(CPUX86State *env,
> > +Reg *v, Reg *s, target_ulong a0)
> > +{
> > +int i;
> > +
> > +for (i = 0; i < (1 << SHIFT); i++) {
> > +if (v->Q(i)
On Wed, Sep 14, 2022 at 6:48 PM Howard Spoelstra wrote:
>
>
> On Tue, Sep 13, 2022 at 6:29 PM Mark Cave-Ayland <
> mark.cave-ayl...@ilande.co.uk> wrote:
>
>> On 13/09/2022 15:28, Richard Henderson wrote:
>>
>> > The second was described by Mark in the lobby of KVM Forum.
>> > The first was found
On Tue, Sep 13, 2022 at 6:29 PM Mark Cave-Ayland <
mark.cave-ayl...@ilande.co.uk> wrote:
> On 13/09/2022 15:28, Richard Henderson wrote:
>
> > The second was described by Mark in the lobby of KVM Forum.
> > The first was found by inspection of other uses of gen_helper_set_sr.
> >
> > r~
> >
> > Ri
On Mon, 12 Sep 2022 13:51:30 -0700
Nicolin Chen wrote:
> On Mon, Sep 12, 2022 at 02:38:52PM +0200, Cornelia Huck wrote:
> > External email: Use caution opening links or attachments
> >
> >
> > On Fri, Sep 09 2022, Nicolin Chen wrote:
> >
> > > Its caller vfio_connect_container() assigns a d
On Wed, 14 Sep 2022, Cédric Le Goater wrote:
On 9/14/22 13:44, BALATON Zoltan wrote:
On Wed, 14 Sep 2022, Cédric Le Goater wrote:
On 9/13/22 21:52, BALATON Zoltan wrote:
The do_init parameter of ppc4xx_sdram_init() is used to map memory
regions that is normally done by the firmware by programm
On 9/14/22 13:55, Maksim Davydov wrote:
This script run QEMU to obtain compat_props of machines and default
values of different types and produce appropriate table. This table
can be used to compare machine types to choose the most suitable
machine. Also table in json or csv format should be used
On Wed, 14 Sep 2022 12:10:29 -0600
Alex Williamson wrote:
> On Mon, 12 Sep 2022 13:51:30 -0700
> Nicolin Chen wrote:
>
> > On Mon, Sep 12, 2022 at 02:38:52PM +0200, Cornelia Huck wrote:
> > > External email: Use caution opening links or attachments
> > >
> > >
> > > On Fri, Sep 09 2022, Nic
On Wed, 14 Sep 2022, BALATON Zoltan wrote:
On Wed, 14 Sep 2022, Cédric Le Goater wrote:
On 9/14/22 13:44, BALATON Zoltan wrote:
On Wed, 14 Sep 2022, Cédric Le Goater wrote:
On 9/13/22 21:52, BALATON Zoltan wrote:
The do_init parameter of ppc4xx_sdram_init() is used to map memory
regions that
PowerPC64 processors handle direct branches better than indirect
ones, resulting in less stalled cycles and branch misses.
However, PPC's tb_target_set_jmp_target() was only using direct
branches for 16-bit jumps, while PowerPC64's unconditional branch
instructions are able to handle displacements
* Laurent Vivier (lviv...@redhat.com) wrote:
> To be consistent with socket_uri(), add 'tcp:' prefix for inet type in
> socket_parse(), by default socket_parse() use tcp when no prefix is
> provided (format is host:port).
>
> In socket_uri(), use 'vsock:' prefix for vsock type rather than 'tcp:'
>
* Laurent Vivier (lviv...@redhat.com) wrote:
> Rename SocketAddress_to_str() to socket_uri() and move it to
> util/qemu-sockets.c close to socket_parse().
>
> socket_uri() generates a string from a SocketAddress while
> socket_parse() generates a SocketAddress from a string.
>
> Signed-off-by: La
Hi Alex,
On Wed, Sep 14, 2022 at 12:10:29PM -0600, Alex Williamson wrote:
> > > > Its caller vfio_connect_container() assigns a default value
> > > > to info->iova_pgsizes, even if vfio_get_iommu_info() fails.
> > > > This would result in a "Segmentation fault" error, when the
> > > > VFIO_IOMMU_
On Wed, 14 Sep 2022 12:02:56 -0700
Nicolin Chen wrote:
> Hi Alex,
>
> On Wed, Sep 14, 2022 at 12:10:29PM -0600, Alex Williamson wrote:
>
> > > > > Its caller vfio_connect_container() assigns a default value
> > > > > to info->iova_pgsizes, even if vfio_get_iommu_info() fails.
> > > > > This wou
On Wed, 14 Sep 2022 13:53:39 -0600
Alex Williamson wrote:
> On Wed, 14 Sep 2022 12:02:56 -0700
> Nicolin Chen wrote:
>
> > Hi Alex,
> >
> > On Wed, Sep 14, 2022 at 12:10:29PM -0600, Alex Williamson wrote:
> >
> > > > > > Its caller vfio_connect_container() assigns a default value
> > > > >
On Wed, Sep 14, 2022 at 02:03:25PM -0600, Alex Williamson wrote:
> > > > +container->pgsizes = 4096;
> > >
> > > This might be a separate question/issue: I wonder if we should use
> > > "sysconf(_SC_PAGE_SIZE)" here instead of 4096.
> > >
> > > With a kernel using a larger page size, e.g.
On 14/09/2022 16.59, Alex Bennée wrote:
We don't want to rely on the soon to be reduced default time. These
tests are still slow for something we want to run in CI though.
Signed-off-by: Alex Bennée
---
tests/avocado/boot_linux.py | 2 ++
1 file changed, 2 insertions(+)
diff --git a/tests/a
On Tue, Sep 13, 2022 at 12:14 PM Richard Henderson
wrote:
> > +static void gen_VMOVLPx(DisasContext *s, CPUX86State *env, X86DecodedInsn
> > *decode)
> > +{
> > +int vec_len = sse_vec_len(s, decode);
> > +
> > +tcg_gen_ld_i64(s->tmp1_i64, cpu_env, decode->op[2].offset +
> > offsetof(XMMR
On Tue, Sep 13, 2022 at 10:17 AM Richard Henderson
wrote:
>
> On 9/12/22 00:04, Paolo Bonzini wrote:
> > +while (vec_len > 8) {
> > +vec_len -= 8;
> > +tcg_gen_shli_tl(s->T0, s->T0, 8);
> > +tcg_gen_ld8u_tl(t, cpu_env, offsetof(CPUX86State,
> > xmm_t0.ZMM_B(vec_len - 1
On 9/12/22 09:55, Richard Henderson wrote:
* Compute...
Why bother passing NULL, and fixing it up at each use...
+static void gen_op_mov_reg_v(DisasContext *s, MemOp ot, int reg, TCGv
t0)
+{
+ gen_op_deposit_reg_v(s, ot, reg, NULL, t0);
}
... when you can just as easily pass in the
Hi ,Vladimir
sometimes ,post-copy mode is not the best choice. For instance, Supposing
migrate process will take ten minutes,but network may be interruptted In this
process .
If it does happenthe , memory data of VM will be splitted into two parts, and
will not be rollback.This is a bad situatio
On Wed, Sep 14, 2022 at 2:21 PM Xuan Zhuo wrote:
>
> On Wed, 14 Sep 2022 11:13:29 +0800, Jason Wang wrote:
> >
> > 在 2022/9/12 11:10, Kangjie Xu 写道:
> > > Update vhost_net_virtqueue_reset() for vhost-user scenario.
> > >
> > > In order to reuse some functions, we process the idx for
> > > vhost-u
On Wed, Sep 14, 2022 at 7:33 PM Si-Wei Liu wrote:
>
>
>
> On 9/14/2022 3:20 AM, Jason Wang wrote:
> > On Fri, Sep 9, 2022 at 4:02 PM Eugenio Perez Martin
> > wrote:
> >> On Fri, Sep 9, 2022 at 8:40 AM Jason Wang wrote:
> >>> On Fri, Sep 9, 2022 at 2:38 PM Jason Wang wrote:
> On Wed, Sep 7
On Wed, Sep 14, 2022 at 11:44 PM Si-Wei Liu wrote:
>
>
>
> On 9/14/2022 2:57 PM, Eugenio Perez Martin wrote:
> > On Wed, Sep 14, 2022 at 1:33 PM Si-Wei Liu wrote:
> >>
> >>
> >> On 9/14/2022 3:20 AM, Jason Wang wrote:
> >>> On Fri, Sep 9, 2022 at 4:02 PM Eugenio Perez Martin
> >>> wrote:
>
On 14/09/2022 11.57, Christian Borntraeger wrote:
Fix the opcode for Load and Zero Rightmost Byte (32).
Cc: qemu-sta...@nongnu.org
Reported-by: Nathan Chancellor
Tested-by: Nathan Chancellor
Signed-off-by: Christian Borntraeger
---
target/s390x/tcg/insn-data.def | 2 +-
1 file changed, 1 i
On 9/14/22 23:59, Paolo Bonzini wrote:
On Tue, Sep 13, 2022 at 10:17 AM Richard Henderson
wrote:
On 9/12/22 00:04, Paolo Bonzini wrote:
+while (vec_len > 8) {
+vec_len -= 8;
+tcg_gen_shli_tl(s->T0, s->T0, 8);
+tcg_gen_ld8u_tl(t, cpu_env, offsetof(CPUX86State, xmm_t
101 - 170 of 170 matches
Mail list logo
