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
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 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 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 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
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 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
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 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 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 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 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, 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
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_
* 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
* 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:'
>
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
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
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 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, 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 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 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 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 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)
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(
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
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);
> > +
>
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
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 (
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
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
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 -
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
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
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
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
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
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
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
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
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 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
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
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
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
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-
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-
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
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
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-8-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-4-alex.ben...@linaro.or
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
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__
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 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-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
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
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
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
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
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
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 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 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
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 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
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
This separates guest file descriptors from host file descriptors,
and utilizes shared infrastructure for integration with gdbstub.
Acked-by: Laurent Vivier
Signed-off-by: Richard Henderson
---
target/m68k/m68k-semi.c | 281 +++-
1 file changed, 49 insertions(
From: Peter Maydell
Honour the commandline -semihosting-config userspace=on option,
instead of always permitting userspace semihosting calls in system
emulation mode, by passing the correct value to the is_userspace
argument of semihosting_enabled().
Note that this is a behaviour change: if the
The following changes since commit 79dfa177ae348bb5ab5f97c0915359b13d6186e2:
Merge tag 'pull-qapi-2022-09-07' of git://repo.or.cz/qemu/armbru into staging
(2022-09-07 13:13:30 -0400)
are available in the Git repository at:
https://gitlab.com/rth7680/qemu.git tags/pull-semi-202
The semihosting abi used by nios2 uses the gdb remote
protocol filesys errnos.
Signed-off-by: Richard Henderson
---
target/nios2/nios2-semi.c | 33 +++--
1 file changed, 31 insertions(+), 2 deletions(-)
diff --git a/target/nios2/nios2-semi.c b/target/nios2/nios2-semi
On 8/26/22 16:21, Emanuele Giuseppe Esposito wrote:
Some callbacks implementation use bdrv_* APIs that assume the
AioContext lock is held. Make sure this invariant is documented.
Signed-off-by: Emanuele Giuseppe Esposito
Reviewed-by: Vladimir Sementsov-Ogievskiy
--
Best regards,
Vladimir
From: "Lucas Mateus Castro (alqotel)"
The alpine docker image only comes with busybox, which doesn't have the
'-e' option on its readlink, so change it to 'realpath' to avoid that
problem.
Suggested-by: Daniel P. Berrangé
Signed-off-by: Lucas Mateus Castro (alqotel)
---
tests/docker/run | 2 +
On 8/26/22 16:20, Emanuele Giuseppe Esposito wrote:
iostatus is the only field (together with .job) that needs
protection using the job mutex.
It is set in the main loop (GLOBAL_STATE functions) but read
in I/O code (block_job_error_action).
In order to protect it, change block_job_iostatus_set
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 Wang wrote:
> On Wed, Sep 7
On 8/26/22 16:20, Emanuele Giuseppe Esposito wrote:
Now that the API offers also _locked() functions, take advantage
of it and give also the caller control to take the lock and call
_locked functions.
This makes sense especially when we have for loops, because it
makes no sense to have:
for(job
From: Peter Maydell
Honour the commandline -semihosting-config userspace=on option,
instead of always permitting userspace semihosting calls in system
emulation mode, by passing the correct value to the is_userspace
argument of semihosting_enabled().
Note that this is a behaviour change: if the
From: "Lucas Mateus Castro (alqotel)"
XEN hypervisor is only available in ARM and x86, but the yaml only
checked if the architecture is different from s390x, changed it to
a more accurate test.
Tested this change on a Ubuntu 20.04 ppc64le.
Signed-off-by: Lucas Mateus Castro (alqotel)
Reviewed-b
Hello,
Based on several discussions I've had in the past 2 days and time spent
looking at the migration stream code I came up with the following
proposal for changes to the stream format. Let me recap what we have
right now:
...()
Where is put only when the current page we are writing
to belo
From: "Lucas Mateus Castro (alqotel)"
ninja-build is missing from the RHEL environment, so a system prepared
with that script would still fail to compile QEMU.
Tested on a Fedora 36
Signed-off-by: Lucas Mateus Castro (alqotel)
---
scripts/ci/setup/build-environment.yml | 1 +
1 file changed, 1
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
From: Peter Maydell
Honour the commandline -semihosting-config userspace=on option,
instead of always permitting userspace semihosting calls in system
emulation mode, by passing the correct value to the is_userspace
argument of semihosting_enabled().
Note that this is a behaviour change: if the
From: Peter Maydell
Currently our semihosting implementations generally prohibit use of
semihosting calls in system emulation from the guest userspace. This
is a very long standing behaviour justified originally "to provide
some semblance of security" (since code with access to the
semihosting A
From: "Lucas Mateus Castro (alqotel)"
This patch series aim to make easier to set up a compilation and CI
environment on PPC64 and PPC64LE machines.
v2:
This patch series are only patches 2-4 of v1 and an alternative to patch 1
suggested by Daniel.
Lucas Mateus Castro (alqotel) (4):
scripts/c
The semihosting abi used by m68k uses the gdb remote
protocol filesys errnos.
Acked-by: Laurent Vivier
Signed-off-by: Richard Henderson
---
target/m68k/m68k-semi.c | 33 +++--
1 file changed, 31 insertions(+), 2 deletions(-)
diff --git a/target/m68k/m68k-semi.c b/ta
This separates guest file descriptors from host file descriptors,
and utilizes shared infrastructure for integration with gdbstub.
Signed-off-by: Richard Henderson
---
target/nios2/nios2-semi.c | 296 +++---
1 file changed, 50 insertions(+), 246 deletions(-)
diff
On 9/14/22 11:50, Clément Chigot wrote:
When requested, the alignment for VLD4.32 is 8 and not 16.
See ARM documentation about VLD4 encoding:
ebytes = 1 << UInt(size);
if size == '10' then
alignment = if a == '0' then 1 else 8;
else
alignment = if a == '0' then 1
From: Peter Maydell
Currently armv7m_load_kernel() takes the size of the block of memory
where it should load the initial guest image, but assumes that it
should always load it at address 0. This happens to be true of all
our M-profile boards at the moment, but it isn't guaranteed to always
be s
On 8/26/22 16:20, Emanuele Giuseppe Esposito wrote:
Once job lock is used and aiocontext is removed, mirror has
to perform job operations under the same critical section,
Note: at this stage, job_{lock/unlock} and job lock guard macros
are*nop*.
Signed-off-by: Emanuele Giuseppe Esposito
Review
From: Peter Maydell
Our feature test functions that check the PMU version are named
isar_feature_{aa32,aa64,any}_pmu_8_{1,4}. This doesn't match the
current Arm ARM official feature names, which are FEAT_PMUv3p1 and
FEAT_PMUv3p4. Rename these functions to _pmuv3p1 and _pmuv3p4.
This commit was
Like commit 034d00d48581 ("e1000: set RX descriptor status in
a separate operation"), there is also same issue in e1000e, which
would cause lost packets or stop sending packets to VM with DPDK.
Do similar fix in e1000e.
Resolves: https://gitlab.com/qemu-project/qemu/-/issues/402
Signed-off-by: Di
From: "Lucas Mateus Castro (alqotel)"
Changed build-environment.yml to only install spice-server on x86_64 and
aarch64 as this package is only available on those architectures.
Signed-off-by: Lucas Mateus Castro (alqotel)
---
scripts/ci/setup/build-environment.yml | 12 +++-
1 file cha
If translation is disabled, the default memory type is Device,
which requires alignment checking. Document, but defer, the
more general case of per-page alignment checking.
Reported-by: Idan Horowitz
Resolves: https://gitlab.com/qemu-project/qemu/-/issues/1204
Signed-off-by: Richard Henderson
-
From: Peter Maydell
In pmccntr_op_finish() and pmevcntr_op_finish() we calculate the next
point at which we will get an overflow and need to fire the PMU
interrupt or set the overflow flag. We do this by calculating the
number of nanoseconds to the overflow event and then adding it to
qemu_clock
From: Peter Maydell
Arm system emulation targets always have TARGET_BIG_ENDIAN clear, so
there is no need to have handling in armv7m_load_kernel() for the
case when it is defined. Remove the unnecessary code.
Side notes:
* our M-profile implementation is always little-endian (that is, it
ma
From: Peter Maydell
The architecture requires that if PMCR.LC is set (for a 64-bit cycle
counter) then PMCR.D (which enables the clock divider so the counter
ticks every 64 cycles rather than every cycle) should be ignored. We
were always honouring PMCR.D; fix the bug so we correctly ignore it
i
From: Peter Maydell
Update the ID registers for TCG's '-cpu max' to report a FEAT_PMUv3p5
compliant PMU.
Signed-off-by: Peter Maydell
Reviewed-by: Richard Henderson
Message-Id: <20220822132358.3524971-11-peter.mayd...@linaro.org>
Signed-off-by: Richard Henderson
---
docs/system/arm/emulation
1 - 100 of 170 matches
Mail list logo