On 20/07/2021 03.12, Eric Blake wrote:
On Mon, Jul 19, 2021 at 10:06:01AM +0200, Thomas Huth wrote:
Hi,
iotest 206 fails for me with:
--- 206.out
+++ 206.out.bad
@@ -99,55 +99,19 @@
{"execute": "blockdev-create", "arguments": {"job-id": "job0", "options":
{"driver": "qcow2",
On Tue, Jul 20 2021, Pierre Morel wrote:
> On 7/19/21 5:50 PM, Cornelia Huck wrote:
>> On Fri, Jul 16 2021, Daniel P. Berrangé wrote:
>>> Is the book/drawer thing architecture specific, or is it machine
>>> type / CPU specific. ie do /all/ the s390x machine types / CPUS
>>> QEMU support the
On Mon, Jul 12, 2021 at 03:10:30PM +0200, Paolo Bonzini wrote:
The LuringState typedef is defined twice, in include/block/raw-aio.h and
block/io_uring.c. Move it in include/block/aio.h, which is included
everywhere the typedef is needed, since include/block/aio.h already has
to define the
On Mon, 19 Jul 2021 at 23:20, Kenneth Adam Miller
wrote:
>
> Hello,
>
> I get the following error:
>
> -c ../accel/tcg/cputlb.c
> ../qemu/accel/tcg/cputlb.c: In function 'tlb_flush_page_by_mmuidx':
> ../qemu/accel/tcg/cputlb.c:602:23: error: comparison is always true due to
> limited range of
On Mon, Jul 19, 2021 at 08:12:58PM -0500, Eric Blake wrote:
> On Mon, Jul 19, 2021 at 10:06:01AM +0200, Thomas Huth wrote:
> > Hi,
> >
> > iotest 206 fails for me with:
> >
>
> > --- 206.out
> > +++ 206.out.bad
> > @@ -99,55 +99,19 @@
> >
> > {"execute": "blockdev-create", "arguments":
On Tue, 20 Jul 2021 at 10:47, Alex Bennée wrote:
>
>
> Peter Maydell writes:
>
> > On Mon, 19 Jul 2021 at 20:52, Alex Bennée wrote:
> >>
> >> We inadvertently added a symbol clash causing the build not to include
> >> the testboard needed for check-tcg.
> >>
> >> Fixes: f4063f9c31 ("meson:
On Sat, 17 Jul 2021 at 21:40, Richard Henderson
wrote:
>
> On 7/17/21 3:06 AM, Peter Maydell wrote:
> > On Fri, 16 Jul 2021 at 23:12, Richard Henderson
> > wrote:
> >>
> >> On 7/13/21 6:37 AM, Peter Maydell wrote:
> >>> Implement the MVE VMLAS insn, which multiplies a vector by a vector
> >>>
* Peter Maydell (peter.mayd...@linaro.org) wrote:
> On Mon, 19 Jul 2021 at 13:43, Peter Maydell wrote:
> >
> > Random crash in migration-tests:
> >
> > MALLOC_PERTURB_=${MALLOC_PERTURB_:-$(( ${RANDOM:-0} % 255 + 1))}
> > QTEST_QEMU_IMG=./qemu-img G_TEST_DBUS_DAEMON=/mnt/nvmedisk/linaro/qem
> >
On 7/19/21 5:52 PM, Daniel P. Berrangé wrote:
On Mon, Jul 19, 2021 at 05:43:29PM +0200, Cornelia Huck wrote:
(restored cc:s)
On Fri, Jul 16 2021, Pierre Morel wrote:
On 7/16/21 11:14 AM, Daniel P. Berrangé wrote:
I increasingly worry that we're making a mistake by going down the
route
On 7/20/21 10:20 AM, Cornelia Huck wrote:
On Tue, Jul 20 2021, Pierre Morel wrote:
On 7/19/21 5:50 PM, Cornelia Huck wrote:
On Fri, Jul 16 2021, Daniel P. Berrangé wrote:
Is the book/drawer thing architecture specific, or is it machine
type / CPU specific. ie do /all/ the s390x machine
On Wed, Jul 14, 2021 at 9:47 PM Cleber Rosa wrote:
> Since 49afbca3b, the use of an optional virgl renderer is not
> available anymore, and since b36eb8860f, the way to choose a GL based
> rendered is to use the "virtio-vga-gl" device.
>
> Signed-off-by: Cleber Rosa
>
Reviewed-by: Marc-André
Richard Henderson writes:
> On 7/19/21 11:48 AM, Alex Bennée wrote:
>> diff --git a/tests/docker/dockerfiles/debian-tricore-cross.docker
>> b/tests/docker/dockerfiles/debian-tricore-cross.docker
>> index 985925134c..4ea9112289 100644
>> ---
Peter Maydell writes:
> On Tue, 20 Jul 2021 at 10:47, Alex Bennée wrote:
>>
>>
>> Peter Maydell writes:
>>
>> > On Mon, 19 Jul 2021 at 20:52, Alex Bennée wrote:
>> >>
>> >> We inadvertently added a symbol clash causing the build not to include
>> >> the testboard needed for check-tcg.
>> >>
On 7/20/21 9:37 AM, Pierre Morel wrote:
On 7/19/21 5:52 PM, Daniel P. Berrangé wrote:
On Mon, Jul 19, 2021 at 05:43:29PM +0200, Cornelia Huck wrote:
(restored cc:s)
On Fri, Jul 16 2021, Pierre Morel wrote:
On 7/16/21 11:14 AM, Daniel P. Berrangé wrote:
I increasingly worry that we're
On Mon, 19 Jul 2021 at 18:27, Max Reitz wrote:
>
> The following changes since commit 7457b407edd6e8555e4b46488aab2f13959fccf8:
>
> Merge remote-tracking branch
> 'remotes/thuth-gitlab/tags/pull-request-2021-07-19' into staging (2021-07-19
> 11:34:08 +0100)
>
> are available in the Git
Peter Maydell writes:
> On Mon, 19 Jul 2021 at 20:52, Alex Bennée wrote:
>>
>> We inadvertently added a symbol clash causing the build not to include
>> the testboard needed for check-tcg.
>>
>> Fixes: f4063f9c31 ("meson: Introduce target-specific Kconfig")
>> Signed-off-by: Alex Bennée
>>
On Mon, Jul 19, 2021 at 12:35:53PM +0200, Stefano Garzarella wrote:
> On Tue, Jul 13, 2021 at 03:58:04PM +0100, Stefan Hajnoczi wrote:
> > On Wed, Jul 07, 2021 at 05:00:19PM +0200, Stefano Garzarella wrote:
> > > @@ -371,7 +375,7 @@ static int laio_do_submit(int fd, struct qemu_laiocb
> > >
Am 14.07.2021 um 11:29 hat Stefan Hajnoczi geschrieben:
> v3:
> * Rebased
> v2:
> * Clarify that object-memory-memfd enables share=on by default [Marc-André]
> * Rebased
>
> vhost-user requires -object memory-backend-*,share=on option so that QEMU uses
> mmap(MAP_SHARED) on guest RAM that is
On 7/19/21 5:50 PM, Cornelia Huck wrote:
On Fri, Jul 16 2021, Daniel P. Berrangé wrote:
On Fri, Jul 16, 2021 at 12:44:49PM +0200, Cornelia Huck wrote:
On Fri, Jul 16 2021, Daniel P. Berrangé wrote:
On Fri, Jul 16, 2021 at 11:10:04AM +0200, Cornelia Huck wrote:
On Thu, Jul 15 2021,
Hi all,
I've been working as one of QEMU's Google Summer of Code interns to
implement a vhost-user-scsi daemon in Rust. That work's now reached a
point where it's usable to attach read-only raw disk images to a VM,
and I'd like to get some feedback from the QEMU community, especially
anyone with
* Peter Maydell (peter.mayd...@linaro.org) wrote:
> On Mon, 19 Jul 2021 at 13:43, Peter Maydell wrote:
> >
> > Random crash in migration-tests:
> >
> > MALLOC_PERTURB_=${MALLOC_PERTURB_:-$(( ${RANDOM:-0} % 255 + 1))}
> > QTEST_QEMU_IMG=./qemu-img G_TEST_DBUS_DAEMON=/mnt/nvmedisk/linaro/qem
> >
On Tue, Jul 20 2021, Pierre Morel wrote:
> On 7/20/21 10:20 AM, Cornelia Huck wrote:
>> On Tue, Jul 20 2021, Pierre Morel wrote:
>>
>>> On 7/19/21 5:50 PM, Cornelia Huck wrote:
On Fri, Jul 16 2021, Daniel P. Berrangé wrote:
> Is the book/drawer thing architecture specific, or is it
On Tue, 20 Jul 2021 at 08:35, ishii.shuuic...@fujitsu.com
wrote:
>
> Hi, peter
> Thank you for your comment.
>
> > Hi; it looks like something with your outgoing email setup still disagrees
> > with QEMU's mailing list server :-( As far as I can tell these emails
> > didn't
> > make it to the
Alex Bennée writes:
> Peter Maydell writes:
>
>> On Tue, 20 Jul 2021 at 10:47, Alex Bennée wrote:
>>>
>>>
>>> Peter Maydell writes:
>>>
>>> > On Mon, 19 Jul 2021 at 20:52, Alex Bennée wrote:
>>> >>
>>> >> We inadvertently added a symbol clash causing the build not to include
>>> >> the
On Tue, Jul 20, 2021 at 10:46:31AM +0200, Pierre Morel wrote:
>
>
> On 7/20/21 10:20 AM, Cornelia Huck wrote:
> > On Tue, Jul 20 2021, Pierre Morel wrote:
> >
> > > On 7/19/21 5:50 PM, Cornelia Huck wrote:
> > > > On Fri, Jul 16 2021, Daniel P. Berrangé wrote:
> > > > > Is the book/drawer
Hi, peter
Thank you for your comment.
> Hi; it looks like something with your outgoing email setup still disagrees
> with QEMU's mailing list server :-( As far as I can tell these emails didn't
> make it to the list, so only people on the direct-cc list will have
> seen them :-(
As you said, it
"Dr. David Alan Gilbert (git)" wrote:
> From: "Dr. David Alan Gilbert"
>
> OpenBSD doesn't like :0 as an address, switch to using 127.0.0.1
> in baddest; it's really testing the :0 port number that isn't allowed
> on anything.
>
> (The test doesn't currently run anyway because of the userfault
>
Hi Ishii,
On 7/20/21 9:34 AM, ishii.shuuic...@fujitsu.com wrote:
> Hi, peter
> Thank you for your comment.
>
>> Hi; it looks like something with your outgoing email setup still disagrees
>> with QEMU's mailing list server :-( As far as I can tell these emails didn't
>> make it to the list, so
On 7/20/21 12:02 AM, Joe Komlodi wrote:
> If we have a field that's wider than 32-bits, we need a data type wide enough
> to
> be able to create the bitfield used to deposit the value.
>
> Signed-off-by: Joe Komlodi
> ---
> include/hw/registerfields.h | 2 +-
> 1 file changed, 1 insertion(+),
From: Alessandro Di Federico
Signed-off-by: Alessandro Di Federico
Reviewed-by: Richard Henderson
---
MAINTAINERS | 8
1 file changed, 8 insertions(+)
diff --git a/MAINTAINERS b/MAINTAINERS
index 4256ad1adb..6c825a9eb4 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -194,6 +194,8 @@
From: Niccolò Izzo
These helpers will be employed by the idef-parser generated code.
Signed-off-by: Alessandro Di Federico
Signed-off-by: Niccolò Izzo
---
target/hexagon/genptr.h | 15 +-
target/hexagon/macros.h | 9
target/hexagon/genptr.c | 113
From: Paolo Montesel
Signed-off-by: Alessandro Di Federico
Signed-off-by: Paolo Montesel
Acked-by: Richard Henderson
---
target/hexagon/macros.h | 2 +-
target/hexagon/genptr.c | 24 +---
2 files changed, 14 insertions(+), 12 deletions(-)
diff --git
From: Paolo Montesel
Signed-off-by: Alessandro Di Federico
Signed-off-by: Paolo Montesel
Reviewed-by: Richard Henderson
Reviewed-by: Taylor Simpson
---
target/hexagon/translate.h | 1 +
target/hexagon/translate.c | 3 ++-
2 files changed, 3 insertions(+), 1 deletion(-)
diff --git
From: Alessandro Di Federico
Extend gen_tcg_funcs.py in order to emit calls to the functions emitted
by the idef-parser, if available.
Signed-off-by: Alessandro Di Federico
Reviewed-by: Taylor Simpson
---
target/hexagon/gen_helper_funcs.py | 5 +-
target/hexagon/gen_helper_protos.py | 5
From: Paolo Montesel
Signed-off-by: Alessandro Di Federico
Signed-off-by: Paolo Montesel
---
target/hexagon/idef-parser/idef-parser.h | 258 +++
target/hexagon/idef-parser/idef-parser.lex| 642 ++
target/hexagon/meson.build| 4 +
invtsc_mig_blocker has static storage duration. When a CPU with
certain features is initialized, and invtsc_mig_blocker is still null,
we add a migration blocker and store it in invtsc_mig_blocker.
The object is freed when migrate_add_blocker() fails, leaving
invtsc_mig_blocker dangling. It is
Most callers check the return value. Some check whether it set an
error. Functionally equivalent, but the former tends to be easier on
the eyes, so do that everywhere.
Prior art: commit c6ecec43b2 "qemu-option: Check return value instead
of @err where convenient".
Signed-off-by: Markus
We continue after -smp help:
$ qemu-system-x86_64 -smp help -display none -monitor stdio
smp-opts options:
cores=
cpus=
dies=
maxcpus=
sockets=
threads=
QEMU 6.0.50 monitor - type 'help' for more information
(qemu)
Other options, such as
I doubt the fixes are 6.1 material at this late stage. If you
disagree, let me know.
Cc: Alex Williamson
Cc: Aravinda Prasad
Cc: Cornelia Huck
Cc: Daniel P. Berrangé
Cc: David Gibson
Cc: Dr. David Alan Gilbert
Cc: Eduardo Habkost
Cc: Elena Ufimtseva
Cc: Ganesh Goudar
Cc: Gerd Hoffmann
This is a resend of v2, now that RamDiscardManager is upstream. I retested
dumping with virtio-mem on shmem and it works as expected now.
---
Minor fixes and cleanups, followed by an optimization for virtio-mem
regarding guest dumps and tpm.
virtio-mem logically plugs/unplugs memory within a
Ping. Could this patch be reviewed? The other code fix patch in this
series has been reviewed so if these tests can be reviewed then both
this series:
https://lore.kernel.org/qemu-devel/20210705210434.45824-1-...@linux.ibm.com/
and the dependent series:
qemu_img_verbose() has a drawback of not going through generic
qemu_img_pipe_and_status(). qemu_img_verbose() is not very popular, so
update the only two users to qemu_img_log() and drop qemu_img_verbose()
at all.
Signed-off-by: Vladimir Sementsov-Ogievskiy
---
tests/qemu-iotests/044| 5
The test checks different options. It of course fails if set
IMGOPTS='compression_type=zstd'. So, let's be explicit in what
compression type we want and independent of IMGOPTS. Test both existing
compression types.
Signed-off-by: Vladimir Sementsov-Ogievskiy
---
tests/qemu-iotests/065 | 16
Adding support of IMGOPTS (like in bash tests) allows user to pass a
lot of different options. Still, some may require additional logic.
Now we want compression_type option, so add some smart logic around it:
ignore compression_type=zstd in IMGOPTS, if test want qcow2 in
compatibility mode. As
We are going to add filtering in _qcow2_dump_header and want all tests
use it.
The patch is generated by commands:
cd tests/qemu-iotests
sed -ie 's/$PYTHON qcow2.py "$TEST_IMG" dump-header\($\|
\)/_qcow2_dump_header\1/' ??? tests/*
(the difficulty is to avoid converting dump-header-exts)
We inadvertently added a symbol clash causing the build not to include
the testboard needed for check-tcg.
Fixes: f4063f9c31 ("meson: Introduce target-specific Kconfig")
Reviewed-by: Philippe Mathieu-Daudé
Reviewed-by: Peter Maydell
Reviewed-by: Richard Henderson
Signed-off-by: Alex Bennée
On 7/20/21 3:17 AM, Richard Henderson wrote:
> We are certain of a page crossing here, entering the
> PALcode image, so the call to use_goto_tb that should
> have been here will never succeed.
>
> We are shortly going to add an assert to tcg_gen_goto_tb
> that would trigger for this case.
>
>
On 7/20/21 3:17 AM, Richard Henderson wrote:
> The access internal to tb_cflags() is atomic.
> Avoid re-reading it as such for the multiple uses.
>
> Reviewed-by: Alex Bennée
> Signed-off-by: Richard Henderson
> ---
> accel/tcg/translator.c | 9 -
> 1 file changed, 4 insertions(+), 5
Hi
On Tue, Jul 20, 2021 at 4:12 PM Jag Raman wrote:
>
>
> > On Jul 19, 2021, at 4:24 PM, John Levon wrote:
> >
> > On Mon, Jul 19, 2021 at 04:00:03PM -0400, Jagannathan Raman wrote:
> >
> >> add the libvfio-user library as a submodule. build it as part of QEMU
> >>
> >> diff --git
From: Alessandro Di Federico
This commit is necessary in order to use container built by the current
run of the CI. If we don't do this, we use official containers which are
not affected by the additional dependencies we're introducing.
This should be considered as a temporary solution in order
The Error ** argument must be NULL, _abort, _fatal, or a
pointer to a variable containing NULL. Passing an argument of the
latter kind twice without clearing it in between is wrong: if the
first call sets an error, it no longer points to NULL for the second
call.
pci_proxy_dev_realize() is wrong
We might not start at the beginning of the memory region. We could also
calculate via the difference in the host address; however,
memory_region_set_dirty() also relies on memory_region_get_ram_addr()
internally, so let's just use that.
Acked-by: Stefan Berger
Fixes: ffab1be70692 ("tpm: clear
On Wed, 30 Jun 2021 15:07:18 -0400
Eric DeVolder wrote:
> Provide the definitions needed to support tracing in ACPI ERST.
trace points should be introduced in patches that use them for the first time,
as it stands now series breaks bisection.
>
> Signed-off-by: Eric DeVolder
> ---
>
On 7/20/21 9:03 AM, David Hildenbrand wrote:
Let's factor out adding a MemoryRegionSection to the list, to be reused in
RamDiscardManager context next.
Cc: Marc-André Lureau
Cc: Paolo Bonzini
Cc: "Michael S. Tsirkin"
Cc: Eduardo Habkost
Cc: Alex Williamson
Cc: Dr. David Alan Gilbert
Cc:
First, this permission never protected node from being changed, as
generic child-replacing functions don't check it.
Second, it's a strange thing: it presents a permission of parent node
to change its child. But generally, children are replaced by different
mechanisms, like jobs or qmp commands,
On Wed, Jul 14, 2021 at 01:23:06PM +0200, David Hildenbrand wrote:
> Add a mutext to protect the SIGBUS case, as we cannot mess concurrently
typo s/mutext/mutex/
> with the sigbus handler and we have to manage the global variable
> sigbus_memset_context. The MADV_POPULATE_WRITE path can run
>
The only "feature" of this "Formatting ..." line is that we have to
update it every time we add new option. Let's drop it.
Signed-off-by: Vladimir Sementsov-Ogievskiy
---
tests/qemu-iotests/255.out| 4
tests/qemu-iotests/274.out| 29 -
Rather than base of the shared Debian 10 container which would require
us to bring in even more dependencies just bring in what is needed for
building tricore-softmmu in GitLab. We don't even remove the container
from the DOCKER_PARTIAL_IMAGES lest we cause more confusion.
Signed-off-by: Alex
Mahmoud Mandour writes:
> Hello,
>
> This series introduce some minor improvements/bug fixes in the cache
> plugins and multicore cache modelling.
Queued patches 1,2 and 6 for-6.1/fixes-for-rc1 as they are bug fixes, thanks.
--
Alex Bennée
Gaelan Steele writes:
> Hi all,
>
> I've been working as one of QEMU's Google Summer of Code interns to
> implement a vhost-user-scsi daemon in Rust. That work's now reached a
> point where it's usable to attach read-only raw disk images to a VM,
> and I'd like to get some feedback from the
vhost_user_get_config() can fail without setting an error. Unclean.
Its caller vhost_dev_get_config() compensates by substituting a
generic error then. Goes back to commit 50de51387f "vhost:
Distinguish errors in vhost_dev_get_config()".
Clean up by moving the generic error from
Stillborn in commit 0ebf007dda "hw/i386: Introduce the microvm machine
type".
Cc: Sergio Lopez
Signed-off-by: Markus Armbruster
---
hw/i386/microvm.c | 5 -
1 file changed, 5 deletions(-)
diff --git a/hw/i386/microvm.c b/hw/i386/microvm.c
index aba0c83219..f257ec5a0b 100644
---
Macro ERRP_GUARD() is only needed when we want to dereference @errp or
pass it to error_prepend() or error_append_hint(). Delete superfluous
ones.
Signed-off-by: Markus Armbruster
---
hw/remote/mpqemu-link.c | 3 ---
qga/commands-posix-ssh.c | 17 -
qga/commands-win32.c |
On Wed, 30 Jun 2021 15:07:17 -0400
Eric DeVolder wrote:
> This code is called from the machine code (if ACPI supported)
> to generate the ACPI ERST table.
should be along lines:
This builds ACPI ERST table /spec ref/ to inform OSMP
how to communicate with ... device.
>
> Signed-off-by: Eric
Alex Bennée writes:
> Markus Armbruster writes:
>
>> Cc: QOM maintainers for additional eyes.
>>
>> Alex Bennée writes:
>>
>>> While we are at it add a brief preamble that explains some of the
>>> common concepts in QEMU's device emulation which will hopefully lead
>>> to less confusing about
On 7/20/21 9:03 AM, David Hildenbrand wrote:
Let's make sure to not merge when different memory regions are involved.
Unlikely, but theoretically possible.
Cc: Marc-André Lureau
Cc: Paolo Bonzini
Cc: "Michael S. Tsirkin"
Cc: Eduardo Habkost
Cc: Alex Williamson
Cc: Dr. David Alan Gilbert
On Tue, 20 Jul 2021 at 13:10, Arkadiy wrote:
>
> From: NDNF
>
> This adds hmp 'info tlb' command support for the mips platform.
> 1k pages are not supported.
>
> Signed-off-by: NDNF
Hi; for QEMU contributions we request that the signed-off-by
line is your real name, not a pseudonym; we follow
On 19/07/2021 23.41, Dongwon Kim wrote:
With "detach-all=on" for display, all VCs are detached from the beginning.
This is useful when there are multiple displays assigned to a guest OS.
Can you elaborate? (i.e. why is it useful? Do you just want to avoid having
multiple things opened at
On Tue, Jul 20, 2021 at 02:53:53PM +0200, Markus Armbruster wrote:
> We did this with scripts/coccinelle/use-error_fatal.cocci before, in
> commit 50beeb68094 and 007b06578ab. This commit cleans up rarer
> variations that don't seem worth matching with Coccinelle.
>
> Cc: Thomas Huth
> Cc:
Forget to mention in subject: for-6.2
20.07.2021 17:22, Vladimir Sementsov-Ogievskiy wrote:
First, this permission never protected node from being changed, as
generic child-replacing functions don't check it.
Second, it's a strange thing: it presents a permission of parent node
to change its
We'll use it in tests instead of explicit qcow2.py. Then we are going
to add some filtering in _qcow2_dump_header.
Signed-off-by: Vladimir Sementsov-Ogievskiy
Reviewed-by: Max Reitz
---
tests/qemu-iotests/common.rc | 10 ++
1 file changed, 10 insertions(+)
diff --git
We want iotests pass with both the default zlib compression and with
IMGOPTS='compression_type=zstd'.
Actually the only test that is interested in real compression type in
test output is 287 (test for qcow2 compression type) and it's in bash.
So for now we can safely filter out compression type
On 7/20/21 11:55 AM, Gaelan Steele wrote:
> Hi all,
>
> I've been working as one of QEMU's Google Summer of Code interns to
> implement a vhost-user-scsi daemon in Rust. That work's now reached a
> point where it's usable to attach read-only raw disk images to a VM,
> and I'd like to get some
The test-case "Corrupted size field in compressed cluster descriptor"
heavily depends on zlib compression type. So, make it explicit. This
way test passes with IMGOPTS='compression_type=zstd'.
Signed-off-by: Vladimir Sementsov-Ogievskiy
Reviewed-by: Max Reitz
---
tests/qemu-iotests/214 | 2 +-
Don't touch other incompatible bits, like compression-type. This makes
the test pass with IMGOPTS='compression_type=zstd'.
Signed-off-by: Vladimir Sementsov-Ogievskiy
Reviewed-by: Max Reitz
---
tests/qemu-iotests/060 | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git
Hi,
This fixes check-tcg for tricore and adds a test to the gitlab matrix
so we don't miss it breaking again.
v3
- added new patch for configure.sh
- changed symbol for board
The following need review:
- gitlab: enable a very minimal build with the tricore container
-
From: Alessandro Di Federico
Introduce infrastructure necessary to produce a file suitable for being
parsed by the idef-parser.
Signed-off-by: Alessandro Di Federico
---
target/hexagon/gen_idef_parser_funcs.py | 114 ++
target/hexagon/idef-parser/macros.inc | 153
Commit b673eab4e2 "multifd: Make multifd_load_setup() get an Error
parameter" changed migration_incoming_setup() to take an Error **
argument, and adjusted the callers accordingly. It neglected to
change adjust multifd_load_setup(): it still exit()s on error. Clean
that up.
The error now gets
When migrate_add_blocker(blocker, ) is followed by
error_propagate(errp, err), we can often just as well do
migrate_add_blocker(..., errp). This is the case in
vfio_migration_probe().
Prior art: commit 386f6c07d2 "error: Avoid error_propagate() after
migrate_add_blocker()".
Cc: Kirti Wankhede
vhost_user_backend_init() can fail without setting an error. Unclean.
Its caller vhost_dev_init() compensates by substituting a generic
error then. Goes back to commit 28770ff935 "vhost: Distinguish errors
in vhost_backend_init()".
Clean up by moving the generic error from vhost_dev_init() to
Let's make sure to not merge when different memory regions are involved.
Unlikely, but theoretically possible.
Cc: Marc-André Lureau
Cc: Paolo Bonzini
Cc: "Michael S. Tsirkin"
Cc: Eduardo Habkost
Cc: Alex Williamson
Cc: Dr. David Alan Gilbert
Cc: Igor Mammedov
Cc: Claudio Fontana
Cc:
On 7/20/21 12:46 AM, Klaus Jensen wrote:
> From: Klaus Jensen
>
> The new PMR test unearthed a long-standing issue with MMIO reads on
> big-endian hosts.
>
> Fix this by unconditionally storing all controller registers in little
> endian.
>
> Cc: Gollu Appalanaidu
> Reported-by: Peter Maydell
On Tue, Jul 20, 2021 at 04:20:13PM +0400, Marc-André Lureau wrote:
> > >> + libvfiouser = static_library('vfiouser',
> > >> + build_by_default: false,
> > >> + sources: vfiouser_files,
> > >> +
On Tue, 20 Jul 2021 at 13:58, Philippe Mathieu-Daudé wrote:
>
> On 7/20/21 12:46 AM, Klaus Jensen wrote:
> > From: Klaus Jensen
> >
> > The new PMR test unearthed a long-standing issue with MMIO reads on
> > big-endian hosts.
> >
> > Fix this by unconditionally storing all controller registers
On Mon, 19 Jul 2021 at 23:47, Klaus Jensen wrote:
>
> From: Klaus Jensen
>
> The new PMR test unearthed a long-standing issue with MMIO reads on
> big-endian hosts.
>
> Fix this by unconditionally storing all controller registers in little
> endian.
>
> Cc: Gollu Appalanaidu
> Reported-by:
> #1 adds support for MADV_POPULATE_WRITE, #2 cleans up the code to avoid
> global variables and prepare for concurrency and #3 makes os_mem_prealloc()
> safe to be called from multiple threads concurrently.
>
> Details regarding MADV_POPULATE_WRITE can be found in introducing upstream
> Linux
From: Matheus Ferst
In commit 8f0a4b6a9b, we started to require L=0 for ppc32 to match what
The Programming Environments Manual say:
"For 32-bit implementations, the L field must be cleared, otherwise
the instruction form is invalid."
The stricter behavior, however, broke AROS boot on
Hello,
On Tue, 20 Jul 2021, luigi burdo wrote:
i was able to build on my quad qemu, a ram bank was die and make me the issue
with gcc..
this is what happening if i run pegasos 2 with --enable-kvm.
OK. Can you also show the full command so we know what options you used?
via_superio_cfg:
If image doesn't have any compressed cluster we can easily switch to
zlib compression, which may allow to downgrade the image.
That's mostly needed to support IMGOPTS='compression_type=zstd' in some
iotests which do qcow2 downgrade.
While being here also fix checkpatch complain against '#' in
The test prints qcow2 header fields which depends on chosen compression
type. So, let's be explicit in what compression type we want and
independent of IMGOPTS. Test both existing compression types.
Signed-off-by: Vladimir Sementsov-Ogievskiy
Reviewed-by: Max Reitz
---
tests/qemu-iotests/303
On 7/20/21 3:17 AM, Richard Henderson wrote:
> New hook to return true when an architectural breakpoint is
> to be recognized and false when it should be suppressed.
>
> First use must wait until other pieces are in place.
>
> Signed-off-by: Richard Henderson
> ---
>
> On Jul 19, 2021, at 4:24 PM, John Levon wrote:
>
> On Mon, Jul 19, 2021 at 04:00:03PM -0400, Jagannathan Raman wrote:
>
>> add the libvfio-user library as a submodule. build it as part of QEMU
>>
>> diff --git a/meson.build b/meson.build
>> index 6e4d2d8..f2f9f86 100644
>> ---
On 7/20/21 11:19 AM, Daniel P. Berrangé wrote:
On Tue, Jul 20, 2021 at 10:46:31AM +0200, Pierre Morel wrote:
On 7/20/21 10:20 AM, Cornelia Huck wrote:
On Tue, Jul 20 2021, Pierre Morel wrote:
On 7/19/21 5:50 PM, Cornelia Huck wrote:
On Fri, Jul 16 2021, Daniel P. Berrangé wrote:
Is
From: Paolo Montesel
Make certain helper functions non-static, making them available outside
genptr.c. These functions are required by code generated by the
idef-parser.
This commit also makes some op_helper.c non-static in order to avoid
having them marked as unused when using the idef-parser
From: Niccolò Izzo
Signed-off-by: Alessandro Di Federico
Signed-off-by: Niccolò Izzo
---
tests/tcg/hexagon/Makefile.target | 30 +-
tests/tcg/hexagon/crt.S| 26
tests/tcg/hexagon/test_abs.S | 20 ++
tests/tcg/hexagon/test_bitcnt.S| 42
From: Alessandro Di Federico
This patchset introduces the idef-parser for target/hexagon.
It's the sixth iteration of the patchset and includes fixes suggested in
previous iterations.
`idef-parser` is a build-time tool built using flex and bison. Its aim
is to generate a large part of the tiny
There is nothing to delete after migrate_add_blocker() failed. Trying
anyway is safe, but useless. Don't.
Cc: Sunil Muthuswamy
Cc: Kamil Rytarowski
Cc: Reinoud Zandijk
Signed-off-by: Markus Armbruster
---
target/i386/nvmm/nvmm-all.c | 1 -
target/i386/whpx/whpx-all.c | 1 -
2 files
Fixes: 2500fb423adb17995485de0b4d507cf2f09e3a7f
Cc: Aravinda Prasad
Cc: Ganesh Goudar
Cc: David Gibson
Signed-off-by: Markus Armbruster
---
hw/ppc/spapr_events.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/hw/ppc/spapr_events.c b/hw/ppc/spapr_events.c
index
The Error ** argument must be NULL, _abort, _fatal, or a
pointer to a variable containing NULL. Passing an argument of the
latter kind twice without clearing it in between is wrong: if the
first call sets an error, it no longer points to NULL for the second
call.
machine_parse_property_opt() is
We did this with scripts/coccinelle/use-error_fatal.cocci before, in
commit 50beeb68094 and 007b06578ab. This commit cleans up rarer
variations that don't seem worth matching with Coccinelle.
Cc: Thomas Huth
Cc: Cornelia Huck
Cc: Peter Xu
Cc: Juan Quintela
Cc: Stefan Hajnoczi
Cc: Paolo
1 - 100 of 385 matches
Mail list logo
