From: Frank Chang
Signed-off-by: Frank Chang
Reviewed-by: Alistair Francis
Message-id: 20220118014522.13613-18-frank.ch...@sifive.com
Signed-off-by: Alistair Francis
---
target/riscv/cpu.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/target/riscv/cpu.c b/target/riscv/cpu.c
index ef2693
On Fri, Jan 21, 2022 at 3:50 PM Alistair Francis
wrote:
>
> From: Wilfred Mallawa
>
> This patch removes the left-over/unused `ibex_plic.h` file. Previously
> used by opentitan, which now follows the RISC-V standard and uses the
> SiFivePlicState.
>
> Fixes: 434e7e021 ("hw/intc: Remove the Ibex P
From: LIU Zhiwei
As pc will be written by the xepc in exception return, just ignore
pc in translation.
Signed-off-by: LIU Zhiwei
Reviewed-by: Richard Henderson
Reviewed-by: Alistair Francis
Message-id: 20220120122050.41546-3-zhiwei_...@c-sky.com
Signed-off-by: Alistair Francis
---
target/ri
From: LIU Zhiwei
Replace the array of pm_mask/pm_base with scalar variables.
Remove the cached array value in DisasContext.
Signed-off-by: LIU Zhiwei
Reviewed-by: Richard Henderson
Reviewed-by: Alistair Francis
Message-id: 20220120122050.41546-13-zhiwei_...@c-sky.com
Signed-off-by: Alistair F
From: Anup Patel
Now that RISC-V Spike machine can use BIN BIOS images, we remove
the macros used for ELF BIOS image names.
Signed-off-by: Anup Patel
Reviewed-by: Alistair Francis
Reviewed-by: Bin Meng
Signed-off-by: Alistair Francis
---
include/hw/riscv/boot.h | 2 --
hw/riscv/spike.c
From: LIU Zhiwei
When pc is written, it is sign-extended to fill the widest supported XLEN.
Signed-off-by: LIU Zhiwei
Reviewed-by: Richard Henderson
Reviewed-by: Alistair Francis
Message-id: 20220120122050.41546-5-zhiwei_...@c-sky.com
Signed-off-by: Alistair Francis
---
target/riscv/transla
From: LIU Zhiwei
Current xlen has been used in helper functions and many other places.
The computation of current xlen is not so trivial, so that we should
recompute it as little as possible.
Fortunately, xlen only changes in very seldom cases, such as exception,
misa write, mstatus write, cpu r
From: Frank Chang
Vector narrowing conversion instructions are provided to and from all
supported integer EEWs for Zve32f extension.
Signed-off-by: Frank Chang
Reviewed-by: Alistair Francis
Message-id: 20220118014522.13613-17-frank.ch...@sifive.com
Signed-off-by: Alistair Francis
---
target/
From: Frank Chang
Vector widening conversion instructions are provided to and from all
supported integer EEWs for Zve32f extension.
Signed-off-by: Frank Chang
Reviewed-by: Alistair Francis
Message-id: 20220118014522.13613-16-frank.ch...@sifive.com
Signed-off-by: Alistair Francis
---
target/r
From: LIU Zhiwei
The mask comes from the pointer masking extension, or the max value
corresponding to XLEN bits.
Signed-off-by: LIU Zhiwei
Acked-by: Alistair Francis
Reviewed-by: Richard Henderson
Message-id: 20220120122050.41546-20-zhiwei_...@c-sky.com
Signed-off-by: Alistair Francis
---
t
From: LIU Zhiwei
The read from PC for translation is in cpu_get_tb_cpu_state, before translation.
Signed-off-by: LIU Zhiwei
Reviewed-by: Richard Henderson
Reviewed-by: Alistair Francis
Message-id: 20220120122050.41546-7-zhiwei_...@c-sky.com
Signed-off-by: Alistair Francis
---
target/riscv/c
From: LIU Zhiwei
Signed-off-by: LIU Zhiwei
Reviewed-by: Richard Henderson
Reviewed-by: Alistair Francis
Message-id: 20220120122050.41546-23-zhiwei_...@c-sky.com
Signed-off-by: Alistair Francis
---
target/riscv/cpu_bits.h | 3 +++
target/riscv/csr.c | 28 ++--
2
From: Anup Patel
Currently, we have to use OpenSBI firmware ELF as bios for the spike
machine because the HTIF console requires ELF for parsing "fromhost"
and "tohost" symbols.
The latest OpenSBI can now optionally pick-up HTIF register address
from HTIF DT node so using this feature spike machi
From: LIU Zhiwei
Signed-off-by: LIU Zhiwei
Acked-by: Richard Henderson
Reviewed-by: Alistair Francis
Message-id: 20220120122050.41546-18-zhiwei_...@c-sky.com
Signed-off-by: Alistair Francis
---
target/riscv/cpu.h | 1 -
1 file changed, 1 deletion(-)
diff --git a/target/riscv/cpu.h b/target/
From: LIU Zhiwei
Signed-off-by: LIU Zhiwei
Reviewed-by: Richard Henderson
Reviewed-by: Alistair Francis
Message-id: 20220120122050.41546-2-zhiwei_...@c-sky.com
Signed-off-by: Alistair Francis
---
target/riscv/csr.c | 19 +++
target/riscv/pmp.c | 12
2 files chang
From: LIU Zhiwei
Signed-off-by: LIU Zhiwei
Reviewed-by: Richard Henderson
Reviewed-by: Alistair Francis
Message-id: 20220120122050.41546-17-zhiwei_...@c-sky.com
Signed-off-by: Alistair Francis
---
target/riscv/cpu.h | 5 +
target/riscv/vector_helper.c | 7 +--
2 files chang
From: LIU Zhiwei
Signed-off-by: LIU Zhiwei
Reviewed-by: Alistair Francis
Message-id: 20220120122050.41546-4-zhiwei_...@c-sky.com
Signed-off-by: Alistair Francis
---
target/riscv/translate.c| 4 +---
target/riscv/insn_trans/trans_rvi.c.inc | 4 +---
2 files changed, 2 insertion
From: Klaus Jensen
This fixes CVE-2021-3929 "locally" by denying DMA to the iomem of the
device itself. This still allows DMA to MMIO regions of other devices
(e.g. doing P2P DMA to the controller memory buffer of another NVMe
device).
Fixes: CVE-2021-3929
Reported-by: Qiuhao Li
Reviewed-by: Ke
From: LIU Zhiwei
In some cases, we must restore the guest PC to the address of the start of
the TB, such as when the instruction counter hits zero. So extend pc register
according to current xlen for these cases.
Signed-off-by: LIU Zhiwei
Reviewed-by: Richard Henderson
Reviewed-by: Alistair Fr
From: LIU Zhiwei
Signed-off-by: LIU Zhiwei
Reviewed-by: Richard Henderson
Reviewed-by: Alistair Francis
Message-id: 20220120122050.41546-9-zhiwei_...@c-sky.com
Signed-off-by: Alistair Francis
---
target/riscv/cpu.c | 8 ++---
target/riscv/gdbstub.c | 71 +++--
On Fri, Jan 21, 2022 at 04:07:24PM +0800, Hyman Huang wrote:
> Hi, Peter, i'm working on simplifying the algorithm.
> Current throttle logic is like the following:
>
> 1. If error value(|quota - current|) less than 25MB/s, we assert throttle
> already done and do nothing.
>
> 2. Start to throttle
Hello Vitaly
On 1/21/22 01:02, Vitaly Cheptsov wrote:
Hello,
PowerPC e500mc defines MSR bit 35 differently from most other PowerPC variants.
In particular, for e500mc this is GS (Guest Supervisor) bit[1], while for
others it is NOEXEC GUARD bit[2].
QEMU ignores this architectural difference w
From: LIU Zhiwei
Only check the range that has passed the address translation.
Signed-off-by: LIU Zhiwei
Reviewed-by: Richard Henderson
Reviewed-by: Alistair Francis
Message-id: 20220120122050.41546-19-zhiwei_...@c-sky.com
Signed-off-by: Alistair Francis
---
target/riscv/vector_helper.c | 4
On Tue, Jan 18, 2022 at 01:37:20PM +0100, Paolo Bonzini wrote:
> On 1/11/22 03:22, Yang Zhong wrote:
> > Thanks Kevin, I will update this in next version.
>
> Also:
>
> The extended state subleaves (EAX=0Dh, ECX=n, n>1).ECX[1]
> indicate whether the extended state component locates
>
From: LIU Zhiwei
Signed-off-by: LIU Zhiwei
Reviewed-by: Alistair Francis
Message-id: 20220120122050.41546-24-zhiwei_...@c-sky.com
Signed-off-by: Alistair Francis
---
target/riscv/csr.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/target/riscv/csr.c b/target/risc
On Tue, Jan 18, 2022 at 01:39:59PM +0100, Paolo Bonzini wrote:
> On 1/10/22 09:23, Tian, Kevin wrote:
> >>
> >>AMX XTILECFG and XTILEDATA are managed by XSAVE feature
> >>set. State component 17 is used for 64-byte TILECFG register
> >>(XTILECFG state) and component 18 is used for 8192 bytes
> >>of
Hanna Reitz writes:
> On 20.01.22 17:00, Markus Armbruster wrote:
>> Kevin Wolf writes:
>>
>>> Am 19.01.2022 um 14:44 hat Hanna Reitz geschrieben:
On 19.01.22 13:58, Markus Armbruster wrote:
> Hanna Reitz writes:
>
>> We want to add a --daemonize argument to QSD's command line.
On Tue, Jan 18, 2022 at 01:52:51PM +0100, Paolo Bonzini wrote:
> On 1/7/22 10:31, Yang Zhong wrote:
> >-uint32_t need_align;
> >+uint32_t need_align, support_xfd;
>
> These can be replaced by a single field "uint32_t ecx".
>
> You can add also macros like
>
> #define ESA_FEATURE_ALIGN64_
On 21.01.22 07:10, Markus Armbruster wrote:
Hanna Reitz writes:
On 20.01.22 17:00, Markus Armbruster wrote:
Kevin Wolf writes:
Am 19.01.2022 um 14:44 hat Hanna Reitz geschrieben:
On 19.01.22 13:58, Markus Armbruster wrote:
Hanna Reitz writes:
We want to add a --daemonize argument to Q
From: LIU Zhiwei
We need not specially process vtype when XLEN changes.
Signed-off-by: LIU Zhiwei
Reviewed-by: Richard Henderson
Reviewed-by: Alistair Francis
Message-id: 20220120122050.41546-16-zhiwei_...@c-sky.com
Signed-off-by: Alistair Francis
---
target/riscv/cpu.h | 1 +
ta
On Tue, Jan 18, 2022 at 02:06:55PM +0100, Paolo Bonzini wrote:
> Sorry, hit send on the wrong window. This is the only patch that
> will require a bit more work.
>
> On 1/18/22 13:52, Paolo Bonzini wrote:
> >>@@ -124,6 +150,8 @@ void x86_cpus_init(X86MachineState *x86ms,
> >>int default_cpu_versi
From: LIU Zhiwei
Write mask is representing the bits we care about.
Signed-off-by: LIU Zhiwei
Reviewed-by: Richard Henderson
Reviewed-by: Alistair Francis
Message-id: 20220120122050.41546-11-zhiwei_...@c-sky.com
Signed-off-by: Alistair Francis
---
target/riscv/op_helper.c|
On Fri, Jan 21, 2022 at 2:02 AM Anup Patel wrote:
>
> Hi Alistair,
>
> On Wed, Jan 19, 2022 at 8:56 PM Anup Patel wrote:
> >
> > From: Anup Patel
> >
> > The advanced interrupt architecture (AIA) extends the per-HART local
> > interrupt support. Along with this, it also adds IMSIC (MSI contrllor
Again, I won't ask for a good algorithm as the 1st version, but then I think
it's nicer we have the simplest algorithm merged first, which should be very
easy to verify.
Hi, Peter, i'm working on simplifying the algorithm.
Current throttle logic is like the following:
1. If error value(|qu
On 20/01/2022 16.13, Peter Maydell wrote:
On Fri, 30 Jul 2021 at 16:12, Peter Maydell wrote:
"make check-acceptance" takes way way too long. I just did a run
on an arm-and-aarch64-targets-only debug build and it took over
half an hour, and this despite it skipping or cancelling 26 out
of 58 te
在 2022/1/21 上午8:35, Alistair Francis 写道:
On Wed, Jan 19, 2022 at 11:09 PM Weiwei Li wrote:
Signed-off-by: Weiwei Li
Signed-off-by: Junqiang Wang
Reviewed-by: Alistair Francis
Alistair
Thanks for your comments.
Regards,
Weiwei Li
---
target/riscv/cpu.c | 13 +
1 file
This adds a new API vu_notify_config_change() to support
sending VHOST_USER_SLAVE_CONFIG_CHANGE_MSG message to notify
that the configuration space has changed.
Signed-off-by: Xie Yongji
---
subprojects/libvhost-user/libvhost-user.c | 20
subprojects/libvhost-user/libvhost-us
To support block resize, this updates the capacity field
in configuration space and use vu_notify_config_change()
to notify the vhost-user master on the block resize callback.
Signed-off-by: Xie Yongji
---
block/export/vhost-user-blk-server.c | 16
1 file changed, 16 insertions(
Book-E architecture does not set the error code in 31:27 bits
of SRR1, but instead uses these bits for custom fields such
as GS (Guest Supervisor).
Wrongly setting these fields will result in QEMU crashes
when attempting to execute not executable code due to the attempts
to use Guest Supervisor mo
Hi Cédric,
> This looks correct and even fixing an issue that Mario reported
> on the TCG e6500 CPU with a kernel + KVM compiled in :
>
>
> https://lore.kernel.org/all/R5JFVM$911e343ff81933b99d53fd0992d88...@locati.it/
>
> KVM has some issues also with the e6500 but that's another problem
> I
On Tue, Jan 18, 2022 at 9:31 PM Atish Patra wrote:
>
> On Tue, Jan 18, 2022 at 8:48 AM Christoph Muellner
> wrote:
> >
> > The RISC-V base cache management operation ISA extension has been
> > ratified [1]. This patch adds support for the defined instructions
> > and CSRs.
> >
> > [1] https://wi
Missing review: Patch #2 (new)
Refresh lcitool to latest and remove libxml2 (mjt).
v4:
- Fixed typos and cleaned checkpatch.pl (thuth)
- Corrected libvirt-ci submodule URL
v3:
- fix 'refresh' helper to update cirrus-ci files (danpb)
v2:
- rebased (testing/next)
- include mjt's libxml2 removal p
lcitool doesn't support MSYS2 targets, so manually remove
this now unnecessary library.
Reviewed-by: Daniel P. Berrangé
Signed-off-by: Philippe Mathieu-Daudé
---
.cirrus.yml | 1 -
.gitlab-ci.d/windows.yml | 2 --
2 files changed, 3 deletions(-)
diff --git a/.cirrus.yml b/.cirrus.
On 1/21/22 06:50, Alistair Francis wrote:
> From: Wilfred Mallawa
>
> This patch removes the left-over/unused `ibex_plic.h` file. Previously
> used by opentitan, which now follows the RISC-V standard and uses the
> SiFivePlicState.
>
> Fixes: 434e7e021 ("hw/intc: Remove the Ibex PLIC")
> Signed-
Refresh lcitool submodule and the generated files by running:
$ make lcitool-refresh
Reviewed-by: Daniel P. Berrangé
Signed-off-by: Philippe Mathieu-Daudé
---
.gitlab-ci.d/cirrus/freebsd-12.vars | 2 +-
.gitlab-ci.d/cirrus/freebsd-13.vars | 2 +-
tests/docker/dockerfiles/
On 20/01/2022 14:48, Kevin Wolf wrote:
Am 20.01.2022 um 14:22 hat Paolo Bonzini geschrieben:
On 1/19/22 19:34, Kevin Wolf wrote:
So if we go back to a bdrv_invalidate_cache() that does all the graph
manipulations (and asserts that we're in the main loop) and then have a
much smaller bdrv_co_
On Debian we also need libibumad to enable RDMA:
$ ../configure --enable-rdma
ERROR: OpenFabrics librdmacm/libibverbs/libibumad not present.
Your options:
(1) Fast: Install infiniband packages (devel) from your distro.
(2) Cleanest: Install libraries from www.
Hanna Reitz writes:
> On 21.01.22 07:10, Markus Armbruster wrote:
>> Hanna Reitz writes:
>>
>>> On 20.01.22 17:00, Markus Armbruster wrote:
Kevin Wolf writes:
> Am 19.01.2022 um 14:44 hat Hanna Reitz geschrieben:
>> On 19.01.22 13:58, Markus Armbruster wrote:
>>> Hanna Rei
On 1/20/22 12:30, Philippe Mathieu-Daudé wrote:
> From: Michael Tokarev
>
> For a long time, we assumed that libxml2 is neecessary for parallels
> block format support (block/parallels*). However, this format actually
> does not use libxml [*]. Since this is the only user of libxml2 in
> while qe
From: Michael Tokarev
For a long time, we assumed that libxml2 is necessary for parallels
block format support (block/parallels*). However, this format actually
does not use libxml [*]. Since this is the only user of libxml2 in
whole QEMU tree, we can drop all libxml2 checks and dependencies too.
lcitool is used by build test / automation, we want maintainers
to get notified if the submodule is updated.
Reviewed-by: Daniel P. Berrangé
Signed-off-by: Philippe Mathieu-Daudé
---
MAINTAINERS | 1 +
1 file changed, 1 insertion(+)
diff --git a/MAINTAINERS b/MAINTAINERS
index e4b3a4bcdf4..6d5
Correct the libvirt-ci repository URL to avoid this warning when
cloning / refreshing the submodule:
warning: redirecting to https://gitlab.com/libvirt/libvirt-ci.git/
Fixes: 4ebb040f1fd ("tests: integrate lcitool for generating build env
manifests")
Signed-off-by: Philippe Mathieu-Daudé
---
The script only include the local qemu.yml for Dockerfiles.
Since we want to keep the Cirrus-CI generated files in sync,
also use the --data-dir option in generate_cirrus().
Fixes: c45a540f4bd (".gitlab-ci.d/cirrus: auto-generate variables with lcitool")
Reported-by: Daniel P. Berrangé
Reviewed-b
The previous commit removed all uses of libxml2.
Update lcitool's qemu.yml and refresh the generated files by running:
$ make lcitool-refresh
Reviewed-by: Daniel P. Berrangé
Signed-off-by: Philippe Mathieu-Daudé
---
.gitlab-ci.d/cirrus/freebsd-12.vars | 2 +-
.gitlab-ci.d/cirrus/f
20.01.2022 19:11, Hanna Reitz wrote:
On 22.12.21 18:40, Vladimir Sementsov-Ogievskiy wrote:
Introduce a new driver, that works in pair with copy-before-write to
improve fleecing.
Without fleecing driver, old fleecing scheme looks as follows:
[guest]
|
|root
v
[copy-before-write] -
On 1/21/22 11:36, Philippe Mathieu-Daudé wrote:
> Correct the libvirt-ci repository URL to avoid this warning when
> cloning / refreshing the submodule:
>
> warning: redirecting to https://gitlab.com/libvirt/libvirt-ci.git/
>
> Fixes: 4ebb040f1fd ("tests: integrate lcitool for generating build
On Fri, Jan 21, 2022 at 11:36:17AM +0100, Philippe Mathieu-Daudé wrote:
> Correct the libvirt-ci repository URL to avoid this warning when
> cloning / refreshing the submodule:
>
> warning: redirecting to https://gitlab.com/libvirt/libvirt-ci.git/
>
> Fixes: 4ebb040f1fd ("tests: integrate lcito
Thomas Huth writes:
> On 20/01/2022 16.13, Peter Maydell wrote:
>> On Fri, 30 Jul 2021 at 16:12, Peter Maydell wrote:
>>>
>>> "make check-acceptance" takes way way too long. I just did a run
>>> on an arm-and-aarch64-targets-only debug build and it took over
>>> half an hour, and this despite it
On 21.01.22 11:27, Markus Armbruster wrote:
Hanna Reitz writes:
On 21.01.22 07:10, Markus Armbruster wrote:
Hanna Reitz writes:
On 20.01.22 17:00, Markus Armbruster wrote:
Kevin Wolf writes:
Am 19.01.2022 um 14:44 hat Hanna Reitz geschrieben:
On 19.01.22 13:58, Markus Armbruster wrote
On 21/01/2022 11.36, Philippe Mathieu-Daudé wrote:
lcitool is used by build test / automation, we want maintainers
to get notified if the submodule is updated.
Reviewed-by: Daniel P. Berrangé
Signed-off-by: Philippe Mathieu-Daudé
---
MAINTAINERS | 1 +
1 file changed, 1 insertion(+)
Revie
On Fri, 21 Jan 2022 at 10:50, Markus Armbruster wrote:
> No objection, but it's no replacement for looking into why these tests
> are so slow.
>
> The #1 reason for things being slow is not giving a damn :)
See previous messages in the thread -- the test starts a
full-fat guest OS including UEFI
On 21/01/2022 11.36, Philippe Mathieu-Daudé wrote:
Refresh lcitool submodule and the generated files by running:
$ make lcitool-refresh
Reviewed-by: Daniel P. Berrangé
Signed-off-by: Philippe Mathieu-Daudé
---
.gitlab-ci.d/cirrus/freebsd-12.vars | 2 +-
.gitlab-ci.d/cirrus/fre
On 21/01/2022 11.36, Philippe Mathieu-Daudé wrote:
Correct the libvirt-ci repository URL to avoid this warning when
cloning / refreshing the submodule:
warning: redirecting to https://gitlab.com/libvirt/libvirt-ci.git/
Fixes: 4ebb040f1fd ("tests: integrate lcitool for generating build env
m
On Fri, Jan 21, 2022 at 12:40:48PM +0100, Thomas Huth wrote:
> On 21/01/2022 11.36, Philippe Mathieu-Daudé wrote:
> > Refresh lcitool submodule and the generated files by running:
> >
> >$ make lcitool-refresh
> >
> > Reviewed-by: Daniel P. Berrangé
> > Signed-off-by: Philippe Mathieu-Daudé
On 21/01/2022 11.36, Philippe Mathieu-Daudé wrote:
The previous commit removed all uses of libxml2.
Update lcitool's qemu.yml and refresh the generated files by running:
$ make lcitool-refresh
Could you merge patch 4/8 and 7/8 so that there is only one refreshment
patch in your series? ..
On 21/01/2022 11.36, Philippe Mathieu-Daudé wrote:
lcitool doesn't support MSYS2 targets, so manually remove
this now unnecessary library.
Reviewed-by: Daniel P. Berrangé
Signed-off-by: Philippe Mathieu-Daudé
---
.cirrus.yml | 1 -
.gitlab-ci.d/windows.yml | 2 --
2 files chan
On 21/01/2022 11.36, Philippe Mathieu-Daudé wrote:
From: Michael Tokarev
For a long time, we assumed that libxml2 is necessary for parallels
block format support (block/parallels*). However, this format actually
does not use libxml [*]. Since this is the only user of libxml2 in
whole QEMU tree,
Hi Roman,
чт, 20 янв. 2022 г. в 10:14, Roman Bolshakov :
> On Thu, Jan 13, 2022 at 08:22:13PM +0300, Vladislav Yaroshchuk wrote:
> > vmnet.framework dependency is added with 'vmnet' option
> > to enable or disable it. Default value is 'auto'.
> >
> > vmnet features to be used are available since
The retry_isa test is not doing what it was intended for: The
test_retry_flush() function ignores the machine parameter completely
and thus this test does not get run with the "isapc" machine.
Moreover, in the course of time, the test_retry_flush() has been
changed to depend on PCI-related function
Hello Jesse,
( For some reason, your email never reached my mailbox. In the future,
could please also Cc: qemu-devel@nongnu.org. )
On 1/18/22 20:40, Jesse Millwood wrote:
Hello,
I'm trying to get the e500 to set the initial tlb entry to mirror what the
platform expects. I have made some cha
чт, 20 янв. 2022 г. в 11:32, Roman Bolshakov :
> On Thu, Jan 13, 2022 at 08:22:14PM +0300, Vladislav Yaroshchuk wrote:
> > Create separate netdevs for each vmnet operating mode:
> > - vmnet-host
> > - vmnet-shared
> > - vmnet-bridged
> >
>
> Sure I'm late to the party but what if we add only one b
Peter Maydell writes:
> On Fri, 21 Jan 2022 at 10:50, Markus Armbruster wrote:
>> No objection, but it's no replacement for looking into why these tests
>> are so slow.
>>
>> The #1 reason for things being slow is not giving a damn :)
>
> See previous messages in the thread -- the test starts
Peter Maydell writes:
> In commit d5093d961585f02 we added a VMStateDescription to
> the TYPE_ARMV7M object, to handle migration of its Clocks.
> However a cut-and-paste error meant we used the wrong struct
> name in the VMSTATE_CLOCK() macro arguments. The result was
> that attempting a 'savev
On 21/01/2022 13.23, Alex Bennée wrote:
Peter Maydell writes:
On Fri, 21 Jan 2022 at 10:50, Markus Armbruster wrote:
No objection, but it's no replacement for looking into why these tests
are so slow.
The #1 reason for things being slow is not giving a damn :)
See previous messages in th
On 19/01/2022 11:31, Paolo Bonzini wrote:
diff --git a/blockjob.c b/blockjob.c
index cf1f49f6c2..468ba735c5 100644
--- a/blockjob.c
+++ b/blockjob.c
@@ -155,14 +155,16 @@ static void child_job_set_aio_ctx(BdrvChild *c,
AioContext *ctx,
bdrv_set_aio_context_ignore(sibling->bs, ctx,
On 1/21/22 12:47, Thomas Huth wrote:
> On 21/01/2022 11.36, Philippe Mathieu-Daudé wrote:
>> The previous commit removed all uses of libxml2.
>>
>> Update lcitool's qemu.yml and refresh the generated files by running:
>>
>> $ make lcitool-refresh
>
> Could you merge patch 4/8 and 7/8 so that th
On 2022/1/19 22:08, Markus Armbruster wrote:
> MkfsSion writes:
>
>> When using JSON syntax for -device, -set option can not find device
>> specified in JSON by id field. The following commandline is an example:
>>
>> $ qemu-system-x86_64 -device '{"id":"foo"}' -set device.foo.bar=1
>> qemu-syste
From: Michael Tokarev
For a long time, we assumed that libxml2 is necessary for parallels
block format support (block/parallels*). However, this format actually
does not use libxml [*]. Since this is the only user of libxml2 in
whole QEMU tree, we can drop all libxml2 checks and dependencies too.
The script only include the local qemu.yml for Dockerfiles.
Since we want to keep the Cirrus-CI generated files in sync,
also use the --data-dir option in generate_cirrus().
Fixes: c45a540f4bd (".gitlab-ci.d/cirrus: auto-generate variables with lcitool")
Reported-by: Daniel P. Berrangé
Reviewed-b
Correct the libvirt-ci repository URL to avoid this warning when
cloning / refreshing the submodule:
warning: redirecting to https://gitlab.com/libvirt/libvirt-ci.git/
Fixes: 4ebb040f1fd ("tests: integrate lcitool for generating build env
manifests")
Reviewed-by: Daniel P. Berrangé
Reviewed-b
Series fully reviewed.
Refresh lcitool to latest and remove libxml2 (mjt).
v5:
- update scripts/meson-buildoptions.sh (thuth)
- squash libvirt-ci updates (thuth)
v4:
- Fixed typos and cleaned checkpatch.pl (thuth)
- Corrected libvirt-ci submodule URL
v3:
- fix 'refresh' helper to update cirrus-
lcitool is used by build test / automation, we want maintainers
to get notified if the submodule is updated.
Reviewed-by: Daniel P. Berrangé
Reviewed-by: Thomas Huth
Signed-off-by: Philippe Mathieu-Daudé
---
MAINTAINERS | 1 +
1 file changed, 1 insertion(+)
diff --git a/MAINTAINERS b/MAINTAIN
lcitool doesn't support MSYS2 targets, so manually remove
this now unnecessary library.
Reviewed-by: Daniel P. Berrangé
Reviewed-by: Thomas Huth
Signed-off-by: Philippe Mathieu-Daudé
---
.cirrus.yml | 1 -
.gitlab-ci.d/windows.yml | 2 --
2 files changed, 3 deletions(-)
diff --gi
The previous commit removed all uses of libxml2.
Refresh lcitool submodule, update qemu.yml and refresh the generated
files by running:
$ make lcitool-refresh
Note: This refreshment also removes libudev dependency on Fedora
and CentOS due to libvirt-ci commit 18bfaee ("mappings: Improve
mappin
g (2022-01-20
> 16:13:17 +)
>
> are available in the Git repository at:
>
> g...@github.com:alistair23/qemu.git tags/pull-riscv-to-apply-20220121-1
>
> for you to fetch changes up to f297245f6a780f496fb171af6fcd21ff3e6783c3:
>
> target/riscv
On Debian we also need libibumad to enable RDMA:
$ ../configure --enable-rdma
ERROR: OpenFabrics librdmacm/libibverbs/libibumad not present.
Your options:
(1) Fast: Install infiniband packages (devel) from your distro.
(2) Cleanest: Install libraries from www.
Am 28.10.21 um 21:37 schrieb Markus Armbruster:
Markus Armbruster writes:
8<
diff --git a/qapi/ui.json b/qapi/ui.json
index 5292617b44..39ca0b5ead 100644
--- a/qapi/ui.json
+++ b/qapi/ui.json
@@ -69,8 +69,10 @@
'base': { 'protocol': 'DisplayProtocol',
'password':
On Fri, Jan 21, 2022 at 9:19 PM Vladislav Yaroshchuk
wrote:
>
>
> чт, 20 янв. 2022 г. в 11:32, Roman Bolshakov :
>>
>> On Thu, Jan 13, 2022 at 08:22:14PM +0300, Vladislav Yaroshchuk wrote:
>> > Create separate netdevs for each vmnet operating mode:
>> > - vmnet-host
>> > - vmnet-shared
>> > - vmne
On Fri, Jan 21, 2022 at 4:47 AM Daniel P. Berrangé
wrote:
> On Fri, Jan 21, 2022 at 12:40:48PM +0100, Thomas Huth wrote:
> > On 21/01/2022 11.36, Philippe Mathieu-Daudé wrote:
> > > Refresh lcitool submodule and the generated files by running:
> > >
> > >$ make lcitool-refresh
> > >
> > > Rev
Hanna Reitz writes:
> On 21.01.22 11:27, Markus Armbruster wrote:
>> Hanna Reitz writes:
>>> The problem I face is that currently there is no ergonomic way to wait
>>> until the QSD is up and running (besides looping until the PID file
>>> exists), and I don’t think a utility program that doesn’
On Fri, Jan 21, 2022 at 07:30:55AM -0700, Warner Losh wrote:
> On Fri, Jan 21, 2022 at 4:47 AM Daniel P. Berrangé
> wrote:
>
> > On Fri, Jan 21, 2022 at 12:40:48PM +0100, Thomas Huth wrote:
> > > On 21/01/2022 11.36, Philippe Mathieu-Daudé wrote:
> > > > Refresh lcitool submodule and the generate
MkfsSion writes:
> On 2022/1/19 22:08, Markus Armbruster wrote:
[...]
>> I don't think we should try to make -set work when using JSON arguments.
>>
> Thanks for your detailed review.
You're welcome!
> The following is my opinion towards implementing -set option for JSON
> arguments.
> Havi
We don't have a virtio-gpio implementation in QEMU and only
support a vhost-user backend. The QEMU side of the code is minimal so
it should be enough to instantiate the device and pass some vhost-user
messages over the control socket. To do this we hook into the existing
vhost-user-test code and ju
On Fri, Jan 21, 2022 at 12:23:23PM +, Alex Bennée wrote:
>
> Peter Maydell writes:
>
> > On Fri, 21 Jan 2022 at 10:50, Markus Armbruster wrote:
> >> No objection, but it's no replacement for looking into why these tests
> >> are so slow.
> >>
> >> The #1 reason for things being slow is not
On 19/01/2022 11:31, Paolo Bonzini wrote:
diff --git a/job.c b/job.c
index f16a4ef542..8a5b710d9b 100644
--- a/job.c
+++ b/job.c
@@ -566,7 +566,7 @@ void job_enter_cond_locked(Job *job, bool(*fn)(Job
*job))
job->busy = true;
real_job_unlock();
job_unlock();
- aio_co_en
If an iommu page has wrong permissions, an error message is displayed,
but the access is allowed, which is odd. This patch fixes it.
Signed-off-by: Frederic Barrat
---
hw/pci-host/pnv_phb3.c | 11 ++-
hw/pci-host/pnv_phb4.c | 11 ++-
2 files changed, 12 insertions(+), 10 deletion
On 19/01/2022 11:44, Paolo Bonzini wrote:
On 1/5/22 15:01, Emanuele Giuseppe Esposito wrote:
These functions assume that the job lock is held by the
caller, to avoid TOC/TOU conditions. Therefore, their
name must end with _locked.
Introduce also additional helpers that define _locked
functio
The current firmware descriptor schema for flash requires that both the
executable to NVRAM template paths be provided. This is fine for the
most common usage of EDK2 builds in virtualization where the separate
_CODE and _VARS files are provided.
With confidential computing technology like AMD SEV
This is my last respin on this series which is fully reviewed.
Refresh lcitool to latest and remove libxml2 (mjt).
v6:
- update libvirt-ci submodule to remove lttng-ust from FreeBSD runners
v5:
- update scripts/meson-buildoptions.sh (thuth)
- squash libvirt-ci updates (thuth)
v4:
- Fixed typos
lcitool is used by build test / automation, we want maintainers
to get notified if the submodule is updated.
Reviewed-by: Daniel P. Berrangé
Reviewed-by: Thomas Huth
Signed-off-by: Philippe Mathieu-Daudé
---
MAINTAINERS | 1 +
1 file changed, 1 insertion(+)
diff --git a/MAINTAINERS b/MAINTAIN
1 - 100 of 255 matches
Mail list logo
