On Sun, Feb 20, 2022 at 05:18:33PM +, Mark Cave-Ayland wrote:
> On 08/02/2022 13:10, Daniel P. Berrangé wrote:
>
> > On Tue, Feb 08, 2022 at 01:06:59PM +, Mark Cave-Ayland wrote:
> > > On 08/02/2022 12:49, Daniel P. Berrangé wrote:
> > >
> > > > > I was under the impression that
'event-loop-backend' provides basic property handling for all
'AioContext' based event loops. So let's define a new 'MainLoopClass'
that inherits from it. This will permit tweaking the main loop's
properties through qapi as well as through the command line using the
'-object' keyword[1]. Only one
Peter Maydell writes:
> On Thu, 10 Feb 2022 at 11:30, Alex Bennée wrote:
>>
>> The previous numbers were a guess at best and rather arbitrary without
>> taking into account anything that might be loaded. Instead of using
>> guesses based on the state of registers implement a new function
On 21/2/22 17:49, David Edmondson wrote:
The 'last' member of QEMUClock was removed some time ago, but the
python gdb helper did not notice.
Fixes: 3c2d4c8aa6 ("timer: last, remove last bits of last")
Signed-off-by: David Edmondson
---
scripts/qemugdb/timers.py | 5 ++---
1 file changed, 2
Hi,
We have two tests (as far as I know) that use compression_type=zstd for
qcow2 but do not check whether that is actually supported. Thomas
reported this for 065, but it’s also the case for 303.
This series makes these tests be skipped when qemu-img reports this
compression type to not be
The io_uring fixed "Don't truncate addr fields to 32-bit on 32-bit":
https://git.kernel.dk/cgit/liburing/commit/?id=d84c29b19ed0b13619cff40141bb1fc3615b
This leads to build failure:
../util/fdmon-io_uring.c: In function ‘add_poll_remove_sqe’:
../util/fdmon-io_uring.c:182:36: error: passing
* Peter Xu (pet...@redhat.com) wrote:
> Add a helper to cleanup the transport listener.
>
> When do it, we should also null-ify the cleanup hook and the data, then it's
> even safe to call it multiple times.
>
> Move the socket_address_list cleanup altogether, because that's a mirror of
> the
>
> -Original Message-
> From: Peter Maydell
> Sent: Monday, February 21, 2022 23:55
> To: Stefan Hajnoczi
> Cc: Wang, Haiyue ; Fam Zheng ;
> qemu-devel@nongnu.org; open
> list:Block I/O path
> Subject: Re: [PATCH v1] aio-posix: fix build failure io_uring 2.2
>
> On Mon, 21 Feb 2022 at
Thomas Huth writes:
> QEMU will soon drop the support for Ubuntu 18.04, so let's update
> the Travis jobs that were still using this version to 20.04 instead.
>
> While we're at it, also remove an obsolete comment about Ubuntu
> Xenial being the default for our Travis jobs.
Queued to
In attempting to use the Python GDB plugins a couple of problems were
encountered.
David Edmondson (2):
scripts/qemu-gdb/mtree.py: Int128 are decimal rather than hex
scripts/qemu-gdb/timers.py: the 'last' attribute is no more
scripts/qemugdb/mtree.py | 2 +-
scripts/qemugdb/timers.py | 5
The 'last' member of QEMUClock was removed some time ago, but the
python gdb helper did not notice.
Fixes: 3c2d4c8aa6 ("timer: last, remove last bits of last")
Signed-off-by: David Edmondson
---
scripts/qemugdb/timers.py | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git
When parsing QEMU's native Int128 type, do not attempt to convert from
hexadecimal.
Fixes: 8037fa55ac ("scripts/qemugdb/mtree.py: fix up mtree dump")
Signed-off-by: David Edmondson
---
scripts/qemugdb/mtree.py | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git
> -Original Message-
> From: Stefan Hajnoczi
> Sent: Monday, February 21, 2022 22:54
> To: Wang, Haiyue
> Cc: qemu-devel@nongnu.org; Fam Zheng ; open list:Block I/O
> path
> Subject: Re: [PATCH v1] aio-posix: fix build failure io_uring 2.2
>
> On Fri, Feb 18, 2022 at 12:16:27AM +0800,
Thanks for reviewing, Juan!
On Fri, Feb 18, 2022 at 1:36 PM Juan Quintela wrote:
>
> Leonardo Bras wrote:
> > Add flags to io_writev and introduce io_flush as optional callback to
> > QIOChannelClass, allowing the implementation of zero copy writes by
> > subclasses.
> >
> > How to use them:
>
On Mon, Feb 21, 2022 at 04:08:50PM +, Dov Murik wrote:
> Add a new field 'cpu0-id' to the response of query-sev-capabilities
> QMP command. The value of the field is the hex-encoded 64-byte unique
> ID of the CPU0 (socket 0), which can be used to retrieve the signed CEK
> of the CPU from
* Peter Xu (pet...@redhat.com) wrote:
> Will be reused in postcopy fast load thread.
>
> Signed-off-by: Peter Xu
Reviewed-by: Dr. David Alan Gilbert
> ---
> migration/ram.c | 2 +-
> migration/ram.h | 1 +
> 2 files changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/migration/ram.c
Hello Juan,
Thanks for thew feedback!
On Fri, Feb 18, 2022 at 1:31 PM Juan Quintela wrote:
>
> Leonardo Bras Soares Passos wrote:
> > On Wed, Jan 19, 2022 at 3:16 PM Daniel P. Berrangé
> > wrote:
> >>
> >> On Wed, Jan 19, 2022 at 03:03:29PM -0300, Leonardo Bras Soares Passos
> >> wrote:
>
15.02.2022 20:53, Vladimir Sementsov-Ogievskiy wrote:
We don't need extra bitmap. All we need is to backup the original
bitmap when we do first merge. So, drop extra temporary bitmap and work
directly with target and backup.
Note that block_dirty_bitmap_merge() semantics changed: on failure
Add a new field 'cpu0-id' to the response of query-sev-capabilities
QMP command. The value of the field is the hex-encoded 64-byte unique
ID of the CPU0 (socket 0), which can be used to retrieve the signed CEK
of the CPU from AMD's Key Distribution Service (KDS).
Signed-off-by: Dov Murik
---
On Thu, Feb 17, 2022 at 02:48:56AM -0500, Jagannathan Raman wrote:
> @@ -221,6 +272,14 @@ static void vfu_object_finalize(Object *obj)
>
> o->device = NULL;
>
> +if (o->unplug_blocker && o->pci_dev) {
> +qdev_del_unplug_blocker(DEVICE(o->pci_dev), o->unplug_blocker);
> +
* Peter Xu (pet...@redhat.com) wrote:
> Per the title, remove the return code and simplify the callers as the errors
> will never be triggered. No functional change intended.
>
> Signed-off-by: Peter Xu
Reviewed-by: Dr. David Alan Gilbert
> ---
> migration/postcopy-ram.c | 25
* Peter Xu (pet...@redhat.com) wrote:
> We used to have quite a few places making sure -EIO happened and that's the
> only way to trigger postcopy recovery. That's based on the assumption that
> we'll only return -EIO for channel issues.
>
> It'll work in 99.99% cases but logically that won't
Introduce QMPShellConnectError (subclass of QMPShellError)
to handle disconnection in read_exec_command().
Signed-off-by: Damien Hedde
---
python/qemu/aqmp/qmp_shell.py | 15 +++
1 file changed, 11 insertions(+), 4 deletions(-)
diff --git a/python/qemu/aqmp/qmp_shell.py
Instead of handling these error in _excute_cmd(), now
raise the exception and let read_exec_command() handle it.
Introduce QMPShellParseError (subclass of QMPShellError)
to replace QMPShellError. In next commit we will introduce
another subclass.
Introduce _print_parse_error() method because
* Peter Xu (pet...@redhat.com) wrote:
> Postcopy create threads. A common manner is we init a sem and use it to sync
> with the thread. Namely, we have fault_thread_sem and listen_thread_sem and
> they're only used for this.
>
> Make it a shared infrastructure so it's easier to create yet
In order to support more user-friendly command list file,
this commit adds the support for:
+ comment lines: line staring by '#' are ignored
+ escaped enf-of-line: line with trailing ' \' are continued
on next one
For eol: we impose a space before the '\' in order not to trigger
the escape if
This option makes qmp_shell exit (with error code 1)
as soon as one of the following error occurs:
+ command parsing error
+ disconnection
+ command failure (response is an error)
_execute_cmd() method now returns None or the response
so that read_exec_command() can do the last check.
This is
Hi,
The main idea of this series is to be a bit more user-friendly when
using qmp-shell in a non-interactive way: with an input redirection
from a file containing a list of commands.
I'm working on dynamic qapi config of a qemu machine, this would
be very useful to provide and reproduce small
In that case, there is no echo anyway. So the prompt is just
garbage.
Signed-off-by: Damien Hedde
---
python/qemu/aqmp/qmp_shell.py | 2 ++
1 file changed, 2 insertions(+)
diff --git a/python/qemu/aqmp/qmp_shell.py b/python/qemu/aqmp/qmp_shell.py
index d11bf54b00..a6e0f5af42 100644
---
On Mon, 21 Feb 2022 at 15:02, Stefan Hajnoczi wrote:
> So the QEMU add_poll_remove_sqe() function would do:
>
> io_uring_prep_poll_remove(sqe, (__u64)(uintptr_t)node);
__u64 is a linux-kernel-ism -- we should use uint64_t, I think.
thanks
-- PMM
* Peter Xu (pet...@redhat.com) wrote:
> Postcopy handles huge pages in a special way that currently we can only have
> one "channel" to transfer the page.
>
> It's because when we install pages using UFFDIO_COPY, we need to have the
> whole
> huge page ready, it also means we need to have a temp
Peter Maydell writes:
> The tsc210x doesn't support anything other than 16-bit reads on the
> SPI bus, but the guest can program the SPI controller to attempt
> them anyway. If this happens, don't abort QEMU, just log this as
> a guest error.
>
> This fixes our
On Thu, Feb 17, 2022 at 02:48:55AM -0500, Jagannathan Raman wrote:
> @@ -124,6 +190,11 @@ static void vfu_object_init(Object *obj)
>
> k->nr_devs++;
>
> +if (!phase_check(PHASE_MACHINE_READY)) {
> +o->machine_done.notify = vfu_object_machine_done;
> +
QEMU will soon drop the support for Ubuntu 18.04, so let's update
the Travis jobs that were still using this version to 20.04 instead.
While we're at it, also remove an obsolete comment about Ubuntu
Xenial being the default for our Travis jobs.
Signed-off-by: Thomas Huth
---
.travis.yml | 12
On Thu, Feb 17, 2022 at 02:48:54AM -0500, Jagannathan Raman wrote:
> +struct VfuObjectClass {
> +ObjectClass parent_class;
> +
> +unsigned int nr_devs;
> +
> +/*
> + * Can be set to shutdown automatically when all server object
> + * instances are destroyed
> + */
> +
On Thu, Feb 17, 2022 at 02:48:52AM -0500, Jagannathan Raman wrote:
> Add vfio-user to x-remote machine. It is a boolean, which indicates if
> the machine supports vfio-user protocol. The machine configures the bus
> differently vfio-user and multiprocess protocols, so this property
> informs it on
On Thu, Feb 17, 2022 at 02:48:51AM -0500, Jagannathan Raman wrote:
> Allow hotplugging of PCI(e) devices to remote machine
>
> Signed-off-by: Elena Ufimtseva
> Signed-off-by: John G Johnson
> Signed-off-by: Jagannathan Raman
> ---
> hw/remote/machine.c | 10 ++
> 1 file changed, 10
On Thu, Feb 17, 2022 at 02:48:50AM -0500, Jagannathan Raman wrote:
> diff --git a/softmmu/qdev-monitor.c b/softmmu/qdev-monitor.c
> index 01f3834db5..69d9cf3f25 100644
> --- a/softmmu/qdev-monitor.c
> +++ b/softmmu/qdev-monitor.c
> @@ -945,10 +945,36 @@ void qmp_device_del(const char *id, Error
On 2/21/22 06:58, Igor Mammedov wrote:
> On Fri, 18 Feb 2022 17:12:21 +
> Joao Martins wrote:
>
>> On 2/14/22 15:31, Igor Mammedov wrote:
>>> On Mon, 14 Feb 2022 15:05:00 +
>>> Joao Martins wrote:
On 2/14/22 14:53, Igor Mammedov wrote:
> On Mon, 7 Feb 2022 20:24:20
On Thu, Feb 17, 2022 at 02:48:50AM -0500, Jagannathan Raman wrote:
> Add blocker to prevent hot-unplug of devices
>
> Signed-off-by: Elena Ufimtseva
> Signed-off-by: John G Johnson
> Signed-off-by: Jagannathan Raman
> ---
> include/hw/qdev-core.h | 35 +++
>
On 18/02/2022 16.04, Peter Maydell wrote:
Hi; is anybody else seeing 'make check-acceptance' fail on some of
the s390 tests?
(009/183) tests/avocado/boot_linux.py:BootLinuxS390X.test_s390_ccw_virtio_tcg:
INTERRUPTED: Test interrupted by SIGTERM\nRunner error occurred:
Timeout
On 2/21/22 12:27, Paolo Bonzini wrote:
> On 2/21/22 10:36, Michal Prívozník wrote:
>> Indeed. Libvirt's participating on its own since 2016, IIRC. Since we're
>> still in org acceptance phase we have some time to decide this,
>> actually. We can do the final decision after participating orgs are
On Mon, 21 Feb 2022 at 14:29, Stefan Hajnoczi wrote:
>
> Compiler optimizations can cache TLS values across coroutine yield
> points, resulting in stale values from the previous thread when a
> coroutine is re-entered by a new thread.
>
> Serge Guelton developed an __attribute__((noinline))
On 21/2/22 13:27, zhenwei pi wrote:
Since 2020, linux kernel started to export pvpanic.h. Import the
latest version from linux into QEMU.
Signed-off-by: zhenwei pi
---
include/standard-headers/linux/pvpanic.h | 9 +
scripts/update-linux-headers.sh | 3 ++-
2 files changed,
On 21/2/22 15:29, Stefan Hajnoczi wrote:
QEMU TLS macros must be used to make TLS variables safe with coroutines.
Signed-off-by: Stefan Hajnoczi
---
util/async.c | 12 +++-
1 file changed, 7 insertions(+), 5 deletions(-)
Reviewed-by: Philippe Mathieu-Daudé
On 21/2/22 15:29, Stefan Hajnoczi wrote:
qemu_mutex_iothread_locked() may be used from coroutines. Standard
__thread variables cannot be used by coroutines. Use the coroutine TLS
macros instead.
Signed-off-by: Stefan Hajnoczi
---
softmmu/cpus.c | 8
1 file changed, 4 insertions(+),
On 21/2/22 15:29, Stefan Hajnoczi wrote:
qemu_mutex_iothread_locked() may be used from coroutines. Standard
__thread variables cannot be used by coroutines. Use the coroutine TLS
macros instead.
Signed-off-by: Stefan Hajnoczi
---
softmmu/cpus.c | 8
1 file changed, 4 insertions(+),
On Fri, Feb 18, 2022 at 12:16:27AM +0800, Haiyue Wang wrote:
> The io_uring fixed "Don't truncate addr fields to 32-bit on 32-bit":
> https://git.kernel.dk/cgit/liburing/commit/?id=d84c29b19ed0b13619cff40141bb1fc3615b
>
> This leads to build failure:
> ../util/fdmon-io_uring.c: In function
Windows 11 with WSL2 enabled (Hyper-V) fails to boot with Icelake-Server
{-v5} CPU model but boots well with '-cpu host'. Apparently, it expects
5-level paging and 5-level EPT support to come in pair but QEMU's
Icelake-Server CPU model lacks the later. Introduce 'Icelake-Server-v6'
CPU model with
5-level EPT is present in Icelake Server CPUs and is supported by QEMU
('vmx-page-walk-5').
Signed-off-by: Vitaly Kuznetsov
---
scripts/kvm/vmxcap | 1 +
1 file changed, 1 insertion(+)
diff --git a/scripts/kvm/vmxcap b/scripts/kvm/vmxcap
index 6fe66d5f5753..f140040104bf 100755
---
> -Original Message-
> From: Paolo Bonzini [mailto:paolo.bonz...@gmail.com] On Behalf Of Paolo
> Bonzini
> Sent: Monday, February 21, 2022 7:31 PM
> To: Longpeng (Mike, Cloud Infrastructure Service Product Dept.)
> ; berra...@redhat.com; m...@redhat.com
> Cc: qemu-devel@nongnu.org;
qemu_mutex_iothread_locked() may be used from coroutines. Standard
__thread variables cannot be used by coroutines. Use the coroutine TLS
macros instead.
Signed-off-by: Stefan Hajnoczi
---
softmmu/cpus.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/softmmu/cpus.c
Hi Daniel,
> -Original Message-
> From: Daniel P. Berrangé [mailto:berra...@redhat.com]
> Sent: Monday, February 21, 2022 7:12 PM
> To: Longpeng (Mike, Cloud Infrastructure Service Product Dept.)
>
> Cc: pbonz...@redhat.com; m...@redhat.com; qemu-devel@nongnu.org; Gonglei
> (Arei)
>
>
> -Original Message-
> From: Paolo Bonzini [mailto:paolo.bonz...@gmail.com] On Behalf Of Paolo
> Bonzini
> Sent: Monday, February 21, 2022 7:42 PM
> To: Longpeng (Mike, Cloud Infrastructure Service Product Dept.)
> ; berra...@redhat.com; m...@redhat.com
> Cc: qemu-devel@nongnu.org;
QEMU TLS macros must be used to make TLS variables safe with coroutines.
Signed-off-by: Stefan Hajnoczi
---
util/async.c | 12 +++-
1 file changed, 7 insertions(+), 5 deletions(-)
diff --git a/util/async.c b/util/async.c
index 08d25feef5..2ea1172f3e 100644
--- a/util/async.c
+++
RCU may be used from coroutines. Standard __thread variables cannot be
used by coroutines. Use the coroutine TLS macros instead.
Signed-off-by: Stefan Hajnoczi
---
include/qemu/rcu.h | 7 ---
tests/unit/rcutorture.c| 10 +-
tests/unit/test-rcu-list.c | 4 ++--
v4:
- Dropped '[RFC]'.
- Dropped inline asm for now. -fPIC versions of the code are missing and I
hit several issues including a clang LTO bug where thread local variables are
incorrectly discarded because inline asm is not analyzed to find symbol
dependencies (Serge Guelton is aware).
-
Compiler optimizations can cache TLS values across coroutine yield
points, resulting in stale values from the previous thread when a
coroutine is re-entered by a new thread.
Serge Guelton developed an __attribute__((noinline)) wrapper and tested
it with clang and gcc. I formatted his idea
On 21/2/22 15:11, Peter Maydell wrote:
On Mon, 21 Feb 2022 at 13:55, Philippe Mathieu-Daudé
wrote:
On 21/2/22 13:27, zhenwei pi wrote:
QEMU side has already imported pvpanic.h from linux, remove bit
definitions from include/hw/misc/pvpanic.h, and use
include/standard-headers/linux/pvpanic.h
Signed-off-by: windy
---
hw/i386/fw_cfg.c | 7 ++-
1 file changed, 6 insertions(+), 1 deletion(-)
diff --git a/hw/i386/fw_cfg.c b/hw/i386/fw_cfg.c
index a283785..44f27ae 100644
--- a/hw/i386/fw_cfg.c
+++ b/hw/i386/fw_cfg.c
@@ -56,9 +56,14 @@ void fw_cfg_build_smbios(MachineState *ms,
On Mon, 21 Feb 2022 at 13:55, Philippe Mathieu-Daudé
wrote:
>
> On 21/2/22 13:27, zhenwei pi wrote:
> > QEMU side has already imported pvpanic.h from linux, remove bit
> > definitions from include/hw/misc/pvpanic.h, and use
> > include/standard-headers/linux/pvpanic.h instead.
> > Also minor
The tsc210x doesn't support anything other than 16-bit reads on the
SPI bus, but the guest can program the SPI controller to attempt
them anyway. If this happens, don't abort QEMU, just log this as
a guest error.
This fixes our machine_arm_n8x0.py:N8x0Machine.test_n800
acceptance test, which hits
repository at:
https://git.linaro.org/people/pmaydell/qemu-arm.git
tags/pull-target-arm-20220221-1
for you to fetch changes up to ca511604925eef8572e22ecbf0d3c758d7277924:
ui/cocoa: Fix the leak of qemu_console_get_label (2022-02-21 13:30:21 +
On Wednesday, 2022-02-16 at 22:04:33 -08, Yang Zhong wrote:
> From: Zeng Guang
>
> XFD(eXtended Feature Disable) allows to enable a
> feature on xsave state while preventing specific
> user threads from using the feature.
>
> Support save and restore XFD MSRs if CPUID.D.1.EAX[4]
> enumerate to
Drop the use of OUTPUT_DIR (test/qemu-iotests under the build
directory), and instead write test output files (.out.bad, .notrun, and
.casenotrun) to TEST_DIR.
With this, the same test can be run concurrently without the separate
instances interfering, because they will need separate TEST_DIRs
On 21/2/22 13:27, zhenwei pi wrote:
QEMU side has already imported pvpanic.h from linux, remove bit
definitions from include/hw/misc/pvpanic.h, and use
include/standard-headers/linux/pvpanic.h instead.
Also minor changes for PVPANIC_CRASHLOADED -> PVPANIC_CRASH_LOADED.
It seems to fail to
On Wednesday, 2022-02-16 at 22:04:32 -08, Yang Zhong wrote:
> From: Jing Liu
>
> When dynamic xfeatures (e.g. AMX) are used by the guest, the xsave
> area would be larger than 4KB. KVM_GET_XSAVE2 and KVM_SET_XSAVE
> under KVM_CAP_XSAVE2 works with a xsave buffer larger than 4KB.
> Always use the
* Daniel P. Berrangé (berra...@redhat.com) wrote:
> On Tue, Feb 15, 2022 at 10:53:58AM +0100, Gerd Hoffmann wrote:
> > Hi,
> >
> > > I don't know what behavior should be if firmware tries to program
> > > PCI64 hole beyond supported phys-bits.
> >
> > Well, you are basically f*cked.
> >
> >
On Wednesday, 2022-02-16 at 22:04:30 -08, Yang Zhong wrote:
> From: Jing Liu
>
> Intel introduces XFD faulting mechanism for extended
> XSAVE features to dynamically enable the features in
> runtime. If CPUID (EAX=0Dh, ECX=n, n>1).ECX[2] is set
> as 1, it indicates support for XFD faulting of
On Wed, 16 Feb 2022 at 17:30, Peter Maydell wrote:
>
> On Tue, 8 Feb 2022 at 18:18, Patrick Venture wrote:
> >
> > From: Shengtan Mao
> >
> > Reviewed-by: Hao Wu
> > Reviewed-by: Chris Rauer
> > Signed-off-by: Shengtan Mao
> > Signed-off-by: Patrick Venture
> > ---
>
>
>
> Applied to
Since 2020, linux kernel started to export pvpanic.h. Import the
latest version from linux into QEMU.
Signed-off-by: zhenwei pi
---
include/standard-headers/linux/pvpanic.h | 9 +
scripts/update-linux-headers.sh | 3 ++-
2 files changed, 11 insertions(+), 1 deletion(-)
create
On Wednesday, 2022-02-16 at 22:04:27 -08, Yang Zhong wrote:
> From: Jing Liu
>
> The extended state subleaves (EAX=0Dh, ECX=n, n>1).ECX[1]
> indicate whether the extended state component locates
> on the next 64-byte boundary following the preceding state
> component when the compacted format of
On 21/2/22 10:41, Peter Maydell wrote:
The AN547 application note URL has changed: update our comment
accordingly. (Rev B is still downloadable from the old URL,
but there is a new Rev C of the document now.)
Signed-off-by: Peter Maydell
---
hw/arm/mps2-tz.c | 2 +-
1 file changed, 1
On Wednesday, 2022-02-16 at 22:04:28 -08, Yang Zhong wrote:
> From: Jing Liu
>
> The AMX TILECFG register and the TMMx tile data registers are
> saved/restored via XSAVE, respectively in state component 17
> (64 bytes) and state component 18 (8192 bytes).
>
> Add AMX feature bits to
From: Philippe Mathieu-Daudé
"exec/ramblock.h" requires "qemu/rcu.h" for the definition of
rcu_head, and "exec/ramlist.h" for the definition of RAMBlockNotifier.
Add them to avoid when when refactoring include/:
include/exec/ramblock.h:26:21: error: field has incomplete type 'struct
QEMU side has already imported pvpanic.h from linux, remove bit
definitions from include/hw/misc/pvpanic.h, and use
include/standard-headers/linux/pvpanic.h instead.
Also minor changes for PVPANIC_CRASHLOADED -> PVPANIC_CRASH_LOADED.
Signed-off-by: zhenwei pi
---
hw/misc/pvpanic-isa.c | 4
From: Philippe Mathieu-Daudé
Signed-off-by: Philippe Mathieu-Daudé
Reviewed-by: Thomas Huth
Reviewed-by: Richard Henderson
Message-Id: <20220207082756.82600-7-f4...@amsat.org>
Signed-off-by: Thomas Huth
---
hw/core/ptimer.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/hw/core/ptimer.c
On Mon, 21 Feb 2022 11:19:33 +0100
Thomas Huth wrote:
> QEMU's default screen resolution recently changed to 1280x800, so the
> resolution in the screen shot header changed of course, too.
>
> Reported-by: Peter Maydell
> Fixes: de72c4b7cd ("edid: set default resolution to 1280x800 (WXGA)")
>
On 21/2/22 11:19, Thomas Huth wrote:
QEMU's default screen resolution recently changed to 1280x800, so the
resolution in the screen shot header changed of course, too.
Reported-by: Peter Maydell
Fixes: de72c4b7cd ("edid: set default resolution to 1280x800 (WXGA)")
Signed-off-by: Thomas Huth
From: Philippe Mathieu-Daudé
Add "hw/qdev-core.h" to avoid when refactoring include/:
softmmu/qtest.c:404:9: error: use of undeclared identifier 'NamedGPIOList'
NamedGPIOList *ngl;
^
Signed-off-by: Philippe Mathieu-Daudé
Reviewed-by: Thomas Huth
Reviewed-by: Richard
On 21/2/22 12:59, Thomas Huth wrote:
* Improve virtio-net failover test
* Some small fixes for the qtests
* Misc header cleanups by Philippe
Philippe
On 07/02/2022 09.27, Philippe Mathieu-Daudé via wrote:
Yet another noisy/boring refactor, but it will allow us to better
split exec/ includes as being target-agnostic/specific (the next
series).
Since v1:
- Remove unrelated "hw/s390x/virtio: missing 'cpu.h'" (thuth)
- Reword some commit
From: Philippe Mathieu-Daudé
TriCore boards certainly don't need the ARM loader API :)
Reviewed-by: Richard Henderson
Reviewed-by: Thomas Huth
Signed-off-by: Philippe Mathieu-Daudé
Message-Id: <20220214183144.27402-4-f4...@amsat.org>
Signed-off-by: Thomas Huth
---
This script has been useful for the timer API rewrite in 2013, but
it is of no use anymore today. Let's remove it now.
Message-Id: <20220124102001.35930-1-th...@redhat.com>
Signed-off-by: Thomas Huth
---
scripts/switch-timer-api | 178 ---
1 file changed, 178
From: Philippe Mathieu-Daudé
hwaddr type is defined in "exec/hwaddr.h".
Signed-off-by: Philippe Mathieu-Daudé
Message-Id: <20220209215446.58402-3-f4...@amsat.org>
Reviewed-by: Richard Henderson
Signed-off-by: Thomas Huth
---
include/hw/m68k/mcf.h | 1 +
1 file changed, 1 insertion(+)
diff
From: Philippe Mathieu-Daudé
Many files use "qemu/log.h" declarations but neglect to include
it (they inherit it via "exec/exec-all.h"). "exec/exec-all.h" is
a core component and shouldn't be used that way. Move the
"qemu/log.h" inclusion locally to each unit requiring it.
Signed-off-by:
+Thomas
On 20/2/22 18:18, Mark Cave-Ayland wrote:
On 08/02/2022 13:10, Daniel P. Berrangé wrote:
On Tue, Feb 08, 2022 at 01:06:59PM +, Mark Cave-Ayland wrote:
On 08/02/2022 12:49, Daniel P. Berrangé wrote:
I was under the impression that monitor_register_hmp_info_hrt()
does all the
From: Philippe Mathieu-Daudé
"tpm_ppi.h" only requires to include "exec/memory.h" to get
the MemoryRegion declaration.
tpm_ppi.c requires "hw/qdev-core.h" to use the DEVICE() macro,
tpm_crb.c is the only source file requiring "exec/address-spaces.h".
Signed-off-by: Philippe Mathieu-Daudé
From: Laurent Vivier
If QEMU provides the VIRTIO_NET_F_STANDBY feature but the guest doesn't
the primary device must be kept hidden
Signed-off-by: Laurent Vivier
Message-Id: <20220203141537.972317-5-lviv...@redhat.com>
Signed-off-by: Thomas Huth
---
tests/qtest/virtio-net-failover.c | 33
From: Philippe Mathieu-Daudé
Add the missing 'qemu/log.h' header and remove the
unused 'exec/exec-all.h' one.
Reviewed-by: Richard Henderson
Reviewed-by: Paolo Bonzini
Signed-off-by: Philippe Mathieu-Daudé
Message-Id: <20220207075426.81934-14-f4...@amsat.org>
Signed-off-by: Thomas Huth
---
From: Laurent Vivier
The primary device is not plugged and the migration is done only with
the standby device
Signed-off-by: Laurent Vivier
Message-Id: <20220203141537.972317-7-lviv...@redhat.com>
Signed-off-by: Thomas Huth
---
tests/qtest/virtio-net-failover.c | 142
From: Philippe Mathieu-Daudé
"qemu/timer.h" declares cpu_get_host_ticks().
Signed-off-by: Philippe Mathieu-Daudé
Reviewed-by: Thomas Huth
Reviewed-by: Richard Henderson
Message-Id: <20220207082756.82600-9-f4...@amsat.org>
Signed-off-by: Thomas Huth
---
linux-user/i386/cpu_loop.c | 1 +
From: Laurent Vivier
Check QEMU provides the VIRTIO_NET_F_STANDBY if failover is on,
and doesn't if failover is off
Signed-off-by: Laurent Vivier
Message-Id: <20220203141537.972317-4-lviv...@redhat.com>
Signed-off-by: Thomas Huth
---
tests/qtest/virtio-net-failover.c | 139
From: Philippe Mathieu-Daudé
timer_new_ns(), cpu_get_host_ticks() and NANOSECONDS_PER_SECOND are
declared in "qemu/timer.h".
Signed-off-by: Philippe Mathieu-Daudé
Reviewed-by: Thomas Huth
Reviewed-by: Richard Henderson
Reviewed-by: Alistair Francis
Message-Id:
From: Philippe Mathieu-Daudé
Signed-off-by: Philippe Mathieu-Daudé
Reviewed-by: Richard Henderson
Reviewed-by: Igor Mammedov
Message-Id: <20220207082756.82600-4-f4...@amsat.org>
Signed-off-by: Thomas Huth
---
hw/acpi/memory_hotplug.c | 1 -
1 file changed, 1 deletion(-)
diff --git
From: Laurent Vivier
This allows g_assert() to correctly report the line number of the error
in the test case.
Signed-off-by: Laurent Vivier
Message-Id: <20220203141537.972317-3-lviv...@redhat.com>
Signed-off-by: Thomas Huth
---
tests/qtest/virtio-net-failover.c | 47
From: Philippe Mathieu-Daudé
Currently "qemu/error-report.h" is implicitly included, however
if headers in include/ get refactored, we get:
hw/remote/proxy-memory-listener.c: In function ‘proxy_memory_listener_commit’:
hw/remote/proxy-memory-listener.c:183:9: error: implicit declaration of
From: Laurent Vivier
Test abort during active migration when failover is disabled from QEMU
or from guest side.
Signed-off-by: Laurent Vivier
Message-Id: <20220203141537.972317-8-lviv...@redhat.com>
Signed-off-by: Thomas Huth
---
tests/qtest/virtio-net-failover.c | 180
From: Laurent Vivier
clearly indentify parameters, hotplug and migration tests
Signed-off-by: Laurent Vivier
Message-Id: <20220203141537.972317-2-lviv...@redhat.com>
Signed-off-by: Thomas Huth
---
tests/qtest/virtio-net-failover.c | 21 +
1 file changed, 13 insertions(+),
From: Eric Auger
-netdev vhost-user,vhostforce is deprecated and vhostforce=on
should be used instead.
Signed-off-by: Eric Auger
Reviewed-by: Thomas Huth
Message-Id: <20220210145254.157790-2-eric.au...@redhat.com>
Signed-off-by: Thomas Huth
---
tests/qtest/vhost-user-test.c | 2 +-
1 file
101 - 200 of 264 matches
Mail list logo