On Wed, Oct 14, 2020 at 1:37 PM Bin Meng wrote:
>
> Hi Green,
>
> On Mon, Sep 28, 2020 at 6:12 PM Green Wan wrote:
> >
> > - Add write operation to update fuse data bit when PWE bit is on.
> > - Add array, fuse_wo, to store the 'written' status for all bits
> >of OTP to block the write oper
On Tue, Oct 13, 2020 at 08:58:59PM -0400, Raphael Norwitz wrote:
> On Tue, Oct 6, 2020 at 5:48 AM Igor Mammedov wrote:
> >
> > On Mon, 28 Sep 2020 21:17:31 +0800
> > Jiajun Chen wrote:
> >
> > > Used_memslots is equal to dev->mem->nregions now, it is true for
> > > vhost kernel, but not for vhost
On Wed, 2020-10-14 at 01:36 +, Victor Kamensky (kamensky) wrote:
> Thank you very much for looking at this. I gave a spin to
> your 3 patch series in original setup, and as expected with
> '-cpu 34Kf,tlb-entries=64' option it works great.
>
> If nobody objects, and your patches could be merged
Hi Green,
On Wed, Oct 14, 2020 at 3:02 PM Green Wan wrote:
>
> On Wed, Oct 14, 2020 at 1:37 PM Bin Meng wrote:
> >
> > Hi Green,
> >
> > On Mon, Sep 28, 2020 at 6:12 PM Green Wan wrote:
> > >
> > > - Add write operation to update fuse data bit when PWE bit is on.
> > > - Add array, fuse_wo, t
On Tue, 2020-10-13 at 19:22 -0700, Richard Henderson wrote:
> On 10/13/20 4:11 PM, Richard Henderson wrote:
> > On 10/13/20 6:25 AM, Philippe Mathieu-Daudé wrote:
> > > Yocto developers have expressed interest in running MIPS32
> > > CPU with custom number of TLB:
> > > https://lists.gnu.org/archiv
From: "Rao, Lei"
we should set ram_bulk_stage to false after ram_state_init,
otherwise the bitmap will be unused in migration_bitmap_find_dirty.
all pages in ram cache will be flushed to the ram of secondary guest
for each checkpoint.
Signed-off-by: leirao
Signed-off-by: Zhang Chen
Reviewed-by
From: Pan Nengyuan
s->connection_track_table forgot to destroy in colo_rewriter_cleanup. Fix it.
Reported-by: Euler Robot
Signed-off-by: Pan Nengyuan
Signed-off-by: Zhang Chen
Reviewed-by: Zhang Chen
Reviewed-by: Li Qiang
---
net/filter-rewriter.c | 2 ++
1 file changed, 2 insertions(+)
d
From: Zhang Chen
Hi Jason, this series include latest COLO related patches.
please check and merge it.
Thanks
Zhang Chen
Li Zhijian (2):
colo-compare: fix missing compare_seq initialization
colo-compare: check mark in mutual exclusion
Pan Nengyuan (1):
net/filter-rewriter: destroy g_hash
From: Li Zhijian
Signed-off-by: Li Zhijian
Signed-off-by: Zhang Chen
Reviewed-by: Zhang Chen
---
net/colo.c | 5 +
1 file changed, 1 insertion(+), 4 deletions(-)
diff --git a/net/colo.c b/net/colo.c
index a6c66d829a..ef00609848 100644
--- a/net/colo.c
+++ b/net/colo.c
@@ -133,14 +133,11
From: "Rao, Lei"
In COLO mode, if the startup parameters of QEMU include "no-shutdown",
QEMU will crash when the guest shutdown. The root cause is when the
guest shutdown, the state of VM will switch COLO to SHUTDOWN. When do
checkpoint again, the state will be changed to COLO. But the state
swit
From: "Rao, Lei"
The seq of tcp has been filled in fill_pkt_tcp_info, it
can be used directly here.
Signed-off-by: leirao
Signed-off-by: Zhang Chen
Reviewed-by: Li Zhijian
Reviewed-by: Zhang Chen
---
net/colo-compare.c | 6 +-
1 file changed, 1 insertion(+), 5 deletions(-)
diff --git a
From: Zhang Chen
This parameter need compare with the return of qemu_clock_get_ms(),
it is uinit64_t. So we need fix this issue here.
Reported-by: Derek Su
Signed-off-by: Zhang Chen
Reviewed-by: Li Zhijian
---
net/colo-compare.c | 12 ++--
1 file changed, 6 insertions(+), 6 deletions
From: Zhang Chen
Detect queued secondary packet to sync VM state in time.
Signed-off-by: Zhang Chen
Reviewed-by: Li Zhijian
---
net/colo-compare.c | 25 -
1 file changed, 16 insertions(+), 9 deletions(-)
diff --git a/net/colo-compare.c b/net/colo-compare.c
index 61c95
On Tue, Oct 13, 2020 at 11:30:52AM -0400, Alexander Bulekov wrote:
> On 201007 1647, Dima Stepanov wrote:
> > The virtio-blk fuzz target sets up and fuzzes the available virtio-blk
> > queues. The implementation is based on two files:
> > - tests/qtest/fuzz/virtio_scsi_fuzz.c
> > - tests/qtest/
From: Li Zhijian
Signed-off-by: Li Zhijian
Signed-off-by: Zhang Chen
Reviewed-by: Zhang Chen
---
net/colo-compare.c | 6 ++
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/net/colo-compare.c b/net/colo-compare.c
index 86980cef5e..24c366eec0 100644
--- a/net/colo-compare.c
++
after the initial split into 3 tcg variants, we proceed to also
split tcg_start_vcpu_thread.
We actually split it in 2 this time, since the icount variant
just uses the round robin function.
Signed-off-by: Claudio Fontana
---
accel/tcg/tcg-all.c | 5
accel/tcg/tcg-cpus-icount.c |
From: Zhang Chen
The virtual clock only runs during the emulation. It stops
when the virtual machine is stopped.
The host clock should be used for device models that emulate accurate
real time sources. It will continue to run when the virtual machine
is suspended. COLO need to know the host time
The purpose of this series is to split the tcg-cpus into
3 variants:
tcg_cpus_mttcg(multithreaded tcg vcpus)
tcg_cpus_rr (single threaded round robin vcpus)
tcg_cpus_icount (same as RR, but using icount)
Alex, I read the comment in tcg_start_vcpu_thread saying:
/*
* Initiali
split up the CpusAccel tcg_cpus into three TCG variants:
tcg_cpus_rr (single threaded, round robin cpus)
tcg_cpus_icount (same as rr, but with instruction counting enabled)
tcg_cpus_mttcg (multi-threaded cpus)
Signed-off-by: Claudio Fontana
---
accel/tcg/meson.build | 9 +-
accel/tcg/tc
Hi
On Wed, Oct 14, 2020 at 1:14 AM Philippe Mathieu-Daudé
wrote:
>
> Hi Marc-André,
>
> On 10/13/20 10:25 PM, marcandre.lur...@redhat.com wrote:
> > From: Marc-André Lureau
> >
> > Add new commands to add and remove SSH public keys from
> > ~/.ssh/authorized_keys.
> >
> > I took a different appr
On Wed, Oct 14, 2020 at 10:29:41AM +0300, Dima Stepanov wrote:
> On Tue, Oct 13, 2020 at 11:30:52AM -0400, Alexander Bulekov wrote:
> > On 201007 1647, Dima Stepanov wrote:
> > > The virtio-blk fuzz target sets up and fuzzes the available virtio-blk
> > > queues. The implementation is based on two
On 10/14/20 9:36 AM, Claudio Fontana wrote:
split up the CpusAccel tcg_cpus into three TCG variants:
tcg_cpus_rr (single threaded, round robin cpus)
tcg_cpus_icount (same as rr, but with instruction counting enabled)
tcg_cpus_mttcg (multi-threaded cpus)
Signed-off-by: Claudio Fontana
---
acc
On 10/14/20 9:36 AM, Claudio Fontana wrote:
The purpose of this series is to split the tcg-cpus into
3 variants:
tcg_cpus_mttcg(multithreaded tcg vcpus)
tcg_cpus_rr (single threaded round robin vcpus)
tcg_cpus_icount (same as RR, but using icount)
Good idea!
I traced this bug to hw/char/serial.c/serial_ioport_read
Bug disappears when I add qemu_log("serial_ioport_read %x %x\n",
(int)addr, ret); into the end of this function.
I suppose that there is avocado (or socket) io synchronization problem,
because running the same test without avocado works nor
From: Jason Andryuk
Currently `-vnc none` is fatal when built with `--disable-vnc`. Make
vnc_parse accept "none", so QEMU still run without using vnc.
Signed-off-by: Jason Andryuk
Message-id: 20201009014032.3507-1-jandr...@gmail.com
Signed-off-by: Gerd Hoffmann
---
ui/vnc-stubs.c | 3 +++
1
From: Colin Xu
If object-del input-linux object on-the-fly, instance finalize will
close evdev fd without resetting it. However the main thread is still
trying to lock_acquire/lock_release during ppoll, which leads to a very
high CPU utilization.
Signed-off-by: Colin Xu
Reviewed-by: Li Qiang
M
From: Yonggang Luo
The mingw pkg-config are showing following absolute path and contains : as the
separator,
-D_XOPEN_SOURCE=600 -D_POSIX_C_SOURCE=199506L
-IC:/CI-Tools/msys64/mingw64/include/ncursesw:-I/usr/include/ncursesw:
-DNCURSES_WIDECHAR -D_XOPEN_SOURCE=600 -D_POSIX_C_SOURCE=199506L -IC
From: Yonggang Luo
This is the compiling error:
../ui/curses.c: In function 'curses_refresh':
../ui/curses.c:256:5: error: 'next_maybe_keycode' may be used uninitialized in
this function [-Werror=maybe-uninitialized]
256 | curses2foo(_curses2keycode, _curseskey2keycode, chr, maybe_keycode)
Not needed and makes some compilers error out with:
qemu-edid.c:15:1: error: initializer element is not constant
Signed-off-by: Gerd Hoffmann
Reviewed-by: Daniel P. Berrangé
Message-id: 20201013091615.14166-1-kra...@redhat.com
---
qemu-edid.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(
Since commits 0979ed017f0 ("meson: rename .inc.h files to .h.inc")
and 139c1837db7 ("meson: rename included C source files to .c.inc")
'git-diff --function-context' stopped displaying C function context
correctly.
We can help git-diff by providing attributes to the .[ch].inc path
names. See:
https:
From: Samuel Thibault
./chardev/baum.c expects the default window_id value to be -1, and not 0
which could be confused with a proper window id (when numbered from 0 by
the ui backend).
This fixes getting Braille output with the curses and gtk frontends.
Fixes: f29b3431f62 ("console: move window
The following changes since commit 96292515c07e3a99f5a29540ed2f257b1ff75111:
Merge remote-tracking branch
'remotes/vivier2/tags/trivial-branch-for-5.2-pull-request' into staging
(2020-10-13 14:06:22 +0100)
are available in the Git repository at:
git://git.kraxel.org/qemu tags/u
From: Yonggang Luo
msys2/mingw lacks the POSIX-required langinfo.h.
gcc test.c -DNCURSES_WIDECHAR -I/mingw64/include/ncursesw -pipe -lncursesw
-lgnurx -ltre -lintl -liconv
test.c:4:10: fatal error: langinfo.h: No such file or directory
4 | #include
| ^~~~
compilation
From: Yonggang Luo
We remove the CONFIG_LOCALTIME_R detection option in configure, and move the
check
existence of gmtime_r from configure into C header and source directly by using
macro
`_POSIX_THREAD_SAFE_FUNCTIONS`.
Before this patch, the configure script are always assume the compiler does
From: Jan Henrik Weinstock
We need to specify SDL_WINDOW_OPENGL if we want to create an OpenGL context on
it, i.e. when using '-device virtio-gpu-pci,virgl=on'
Signed-off-by: Jan Henrik Weinstock
Message-id: b2ba98b3-2975-0d4d-1c56-f659923c7...@rwth-aachen.de
Signed-off-by: Gerd Hoffmann
---
On 13/10/20 22:36, harry harry wrote:
> Hi Paolo and Sean,
>
> Thanks much for your prompt replies and clear explanations.
>
> On Tue, Oct 13, 2020 at 2:43 AM Paolo Bonzini wrote:
>>
>> No, the logic to find the HPA with a given HVA is the same as the
>> hardware logic to translate HVA -> HPA.
On 14/10/20 00:40, harry harry wrote:
> Q1: Is there any file like ``/proc/pid/pagemap'' to record the
> mappings between GPAs and HVAs in the host OS?
No, there isn't.
> Q2: Seems that there might be extra overhead (e.g., synchronization
> between EPT tables and host regular page tables; maintai
On Tue, 2020-10-13 at 16:36 -0400, harry harry wrote:
> Hi Paolo and Sean,
>
> Thanks much for your prompt replies and clear explanations.
>
> On Tue, Oct 13, 2020 at 2:43 AM Paolo Bonzini wrote:
> > No, the logic to find the HPA with a given HVA is the same as the
> > hardware logic to translat
From: Klaus Jensen
This adds support for reporting the Deallocated or Unwritten Logical
Block error (DULBE).
Rely on the block status reported by the block layer. Here, we consider
blocks with the BDRV_BLOCK_ZERO status to deallocated. This is because a
pdiscard or write zeroes with unmap won't
On Oct 13 14:06, Keith Busch wrote:
> On Tue, Oct 13, 2020 at 09:08:46PM +0200, Klaus Jensen wrote:
> > From: Klaus Jensen
> >
> > This adds support for reporting the Deallocated or Unwritten Logical
> > Block error (DULBE). This requires tracking the allocated/deallocated
> > status of all logic
Hi Dima,
On Wednesday, 2020-10-14 at 10:39:01 +03, Dima Stepanov wrote:
> On Wed, Oct 14, 2020 at 10:29:41AM +0300, Dima Stepanov wrote:
>> On Tue, Oct 13, 2020 at 11:30:52AM -0400, Alexander Bulekov wrote:
>> > On 201007 1647, Dima Stepanov wrote:
...
>> >
>> > I sent a patch to disable QEMU's
On Tuesday, 2020-10-13 at 17:52:46 +01, Daniel P. Berrangé wrote:
> On Tue, Oct 13, 2020 at 05:50:37PM +0100, Darren Kenny wrote:
>> Hi Alex,
>>
>> This mentions the use of atexit() to perform some cleanup, but I'm not
>> seeing that being added here, should it be?
>
> The reference to atexit is s
On Tue, Oct 13, 2020 at 11:16:31AM -0400, Shashi Mallela wrote:
> Included the newly implemented SBSA generic watchdog device model into
> SBSA platform
>
> Signed-off-by: Shashi Mallela
> ---
> hw/arm/sbsa-ref.c | 50 +++
> 1 file changed, 50 insertio
On Tue, Oct 13, 2020 at 11:26:45AM +0400, Marc-André Lureau wrote:
> Hi
>
> On Mon, Oct 12, 2020 at 12:36 PM Tomáš Golembiovský
> wrote:
>
> > The command lists all disks (real and virtual) as well as disk
> > partitions. For each disk the list of dependent disks is also listed and
> > /dev path
On 14.10.20 03:03, Chenqun (kuhn) wrote:
>
>
>> -Original Message-
>> From: Max Reitz [mailto:mre...@redhat.com]
>> Sent: Tuesday, October 13, 2020 10:47 PM
>> To: Chenqun (kuhn) ; qemu-devel@nongnu.org;
>> qemu-triv...@nongnu.org
>> Cc: vsement...@virtuozzo.com; stefa...@redhat.com; f...
> -Original Message-
> From: Richard Henderson [mailto:richard.hender...@linaro.org]
> Sent: Friday, October 9, 2020 10:34 PM
> To: Jiangyifei ; qemu-devel@nongnu.org;
> qemu-ri...@nongnu.org
> Cc: Zhanghailiang ;
> sag...@eecs.berkeley.edu; kbast...@mail.uni-paderborn.de; Zhangxiaofeng
>
Claudio Fontana writes:
> The purpose of this series is to split the tcg-cpus into
> 3 variants:
>
> tcg_cpus_mttcg(multithreaded tcg vcpus)
> tcg_cpus_rr (single threaded round robin vcpus)
> tcg_cpus_icount (same as RR, but using icount)
I've no objection to the cosmetic clean-up
VS-stage translation at get_physical_address needs to translate pte
address by G-stage translation. But the G-stage translation error
can not be distinguished from VS-stage translation error in
riscv_cpu_tlb_fill. On migration, destination needs to rebuild pte,
and this G-stage translation error mu
在 2020/10/13 21:25, Philippe Mathieu-Daudé 写道:
Allow changing the number of TLB entries for
testing/tunning purpose.
Example to force a 34Kf cpu with 64 TLB:
$ qemu-system-mipsel -cpu 34Kf,tlb-entries=64 ...
This is helpful for developers of the Yocto Project [*]:
Yocto Project uses
On 10/14/20 12:14 PM, Alex Bennée wrote:
>
> Claudio Fontana writes:
>
>> The purpose of this series is to split the tcg-cpus into
>> 3 variants:
>>
>> tcg_cpus_mttcg(multithreaded tcg vcpus)
>> tcg_cpus_rr (single threaded round robin vcpus)
>> tcg_cpus_icount (same as RR, but using
> -Original Message-
> From: Richard Henderson [mailto:richard.hender...@linaro.org]
> Sent: Saturday, October 10, 2020 9:23 PM
> To: Jiangyifei ; qemu-devel@nongnu.org;
> qemu-ri...@nongnu.org
> Cc: pal...@dabbelt.com; alistair.fran...@wdc.com;
> sag...@eecs.berkeley.edu; kbast...@mail.un
The EHCI Host Controller emulation attempt to locate the device
associated with a periodic isochronous transfer description (iTD) and
when this fail the host controller is reset.
But according the EHCI spec 1.0 section 5.15.2.4 Host System
Error, the host controller is supposed to reset itself onl
On 10/13/20 10:25 PM, marcandre.lur...@redhat.com wrote:
From: Marc-André Lureau
Hi,
Add two new commands to help modify ~/.ssh/authorized_keys.
Apart from Philippe's comments, this path is configurable in
sshd.config. But I doubt we should care as ssh-copy-id doesn't care.
Although it'
On 12.10.20 19:43, Andrey Shinkevich wrote:
> Provide API for the COR-filter insertion/removal.
> Also, drop the filter child permissions for an inactive state when the
> filter node is being removed.
>
> Signed-off-by: Andrey Shinkevich
> Reviewed-by: Vladimir Sementsov-Ogievskiy
> ---
> block
On Wed, Oct 14, 2020 at 12:25:00AM +0400, marcandre.lur...@redhat.com wrote:
> From: Marc-André Lureau
>
> Hi,
>
> Add two new commands to help modify ~/.ssh/authorized_keys.
>
> Although it's possible already to modify the authorized_keys files via
> file-{read,write} or exec, the commands are
On 10/14/20 12:20 PM, Jiaxun Yang wrote:
在 2020/10/13 21:25, Philippe Mathieu-Daudé 写道:
Allow changing the number of TLB entries for
testing/tunning purpose.
Example to force a 34Kf cpu with 64 TLB:
$ qemu-system-mipsel -cpu 34Kf,tlb-entries=64 ...
This is helpful for developers of the Yoc
On 12.10.20 19:43, Andrey Shinkevich wrote:
> We are going to use the COR-filter for a block-stream job.
> To limit COR operations by the base node in the backing chain during
> stream job, pass the name of overlay base node to the copy-on-read
> driver as base node itself may change due to possibl
10.10.2020 00:55, Eric Blake wrote:
Rather than open-coding the translation from the deprecated
NbdServerAddOptions type to the preferred BlockExportOptionsNbd, it's
better to utilize QAPI_CLONE_MEMBERS. This solves a couple of issues:
first, if we do any more refactoring of the base type (which
On 10/14/20 9:37 AM, Marc-André Lureau wrote:
On Wed, Oct 14, 2020 at 1:14 AM Philippe Mathieu-Daudé
wrote:
On 10/13/20 10:25 PM, marcandre.lur...@redhat.com wrote:
diff --git a/qga/qapi-schema.json b/qga/qapi-schema.json
index cec98c7e06..50e2854b45 100644
--- a/qga/qapi-schema.json
+++ b/qga
> -Original Message-
> From: Qemu-devel
> [mailto:qemu-devel-bounces+kuhn.chenqun=huawei@nongnu.org] On
> Behalf Of Max Reitz
> Sent: Wednesday, October 14, 2020 5:36 PM
> To: Chenqun (kuhn) ; qemu-devel@nongnu.org;
> qemu-triv...@nongnu.org
> Cc: f...@euphon.net; ganqixin ;
> vsement..
On 10/14/20 12:14 PM, Alex Bennée wrote:
Claudio Fontana writes:
The purpose of this series is to split the tcg-cpus into
3 variants:
tcg_cpus_mttcg(multithreaded tcg vcpus)
tcg_cpus_rr (single threaded round robin vcpus)
tcg_cpus_icount (same as RR, but using icount)
I've no o
From: Klaus Jensen
Add the 'uuid' nvme-ns device parameter such that users who requires a
persistent namespace UUID can explicitly specify it.
If not specified, the property will autogenerate an UUID for each QEMU
invocation.
Signed-off-by: Klaus Jensen
---
hw/block/nvme-ns.h | 1 +
hw/block
From: Gollu Appalanaidu
This is to support for the Commands Supported and Effects log page. See
NVM Express Spec 1.3d, sec. 5.14.1.5 ("Commands Supported and Effects")
Signed-off-by: Gollu Appalanaidu
Signed-off-by: Klaus Jensen
---
include/block/nvme.h | 25 +--
hw/block/nvme.c
From: Klaus Jensen
This adds basic read and write for zoned namespaces.
A zoned namespace is created by setting the iocs namespace parameter to
0x2 and specifying the zns.zcap parameter (zone capacity) in number of
logical blocks per zone. If a zone size (zns.zsze) is not specified, the
namespac
From: Klaus Jensen
Allow the controller to release open resources by transitioning
implicitly and explicitly opened zones to closed. This is done using a
naive "least recently opened" strategy.
Signed-off-by: Klaus Jensen
---
hw/block/nvme-ns.h| 5
hw/block/nvme-ns.c| 5
hw
From: Klaus Jensen
Implement support for TP 4056 ("Namespace Types"). This adds the 'iocs'
(I/O Command Set) device parameter to the nvme-ns device.
Signed-off-by: Klaus Jensen
---
docs/specs/nvme.txt | 3 +
hw/block/nvme-ns.h| 11 ++-
hw/block/nvme.h | 3 +
include/block/nvme
From: Klaus Jensen
Add the Zone Append command.
Signed-off-by: Klaus Jensen
---
hw/block/nvme.h | 6 ++
include/block/nvme.h | 7 +++
hw/block/nvme.c | 46 +++
hw/block/trace-events | 1 +
4 files changed, 60 insertions(+)
diff -
From: Klaus Jensen
Updated version of my proposal.
Based-on: <20201014084324.333774-1-...@irrelevant.dk>
Changes for v3
~~
* Rebased on nvme-next with "[PATCH v2] hw/block/nvme: add dulbe
support" applied.
* "hw/block/nvme: add support for dulbe and block utilization track
From: Klaus Jensen
Add the Zone Management Send command.
Signed-off-by: Klaus Jensen
---
hw/block/nvme.h | 2 +
include/block/nvme.h | 28 +++
hw/block/nvme.c | 384 +-
hw/block/trace-events | 11 ++
4 files changed, 420 insertions(+), 5
On Oct 14 06:42, Dmitry Fomichev wrote:
> In NVMe 1.4, a namespace must report an ID descriptor of UUID type
> if it doesn't support EUI64 or NGUID. Add a new namespace property,
> "uuid", that provides the user the option to either specify the UUID
> explicitly or have a UUID generated automatical
From: Klaus Jensen
Add the Zone Management Receive command.
Signed-off-by: Klaus Jensen
---
hw/block/nvme-ns.h| 8 +++
hw/block/nvme.h | 1 +
include/block/nvme.h | 46 +
hw/block/nvme-ns.c| 12 +++-
hw/block/nvme.c | 146 +
A default value is provided for the variable 'bitmap_name' to avoid compiler
warning.
The compiler show warning:
migration/block-dirty-bitmap.c:1090:13: warning: ‘bitmap_name’
may be used uninitialized in this function [-Wmaybe-uninitialized]
g_strlcpy(s->bitmap_name, bitmap_name, sizeof(s
From: Klaus Jensen
Track number of open/active resources.
Signed-off-by: Klaus Jensen
---
docs/specs/nvme.txt | 6
hw/block/nvme-ns.h | 7 +
include/block/nvme.h | 2 ++
hw/block/nvme-ns.c | 25 +++--
hw/block/nvme.c | 66 ++
We sometime get kernel panic with some devices on Aarch64
hosts. Alex Williamson suggests it might be broken PCIe
root complex. Add trace event to record the latest I/O
access before crashing. In case, assert our accesses are
aligned.
Signed-off-by: Philippe Mathieu-Daudé
---
Cc: Alex Williamson
10.10.2020 00:55, Eric Blake wrote:
'qemu-img map' provides a way to determine which extents of an image
come from the top layer vs. inherited from a backing chain. This is
useful information worth exposing over NBD. There is a proposal to
add a QMP command block-dirty-bitmap-populate which can
Change the confuse "VFIO IOMMU check failed" error message by
the explicit "VFIO IOMMU Type1 is not supported" once.
Example on POWER:
$ qemu-system-ppc64 -drive
if=none,id=nvme0,file=nvme://0001:01:00.0/1,format=raw
qemu-system-ppc64: -drive
if=none,id=nvme0,file=nvme://0001:01:00.0/1,format
For debug purpose, trace BAR regions info.
Signed-off-by: Philippe Mathieu-Daudé
---
util/vfio-helpers.c | 8
util/trace-events | 1 +
2 files changed, 9 insertions(+)
diff --git a/util/vfio-helpers.c b/util/vfio-helpers.c
index 1d4efafcaa4..cd6287c3a98 100644
--- a/util/vfio-helpers
A bunch of boring patches that have been proven helpful
while debugging.
Philippe Mathieu-Daudé (9):
util/vfio-helpers: Improve reporting unsupported IOMMU type
util/vfio-helpers: Trace PCI I/O config accesses
util/vfio-helpers: Trace PCI BAR region info
util/vfio-helpers: Trace where BARs
The QEMU_VFIO_DEBUG definition is only modifiable at build-time.
Trace events can be enabled at run-time. As we prefer the latter,
convert qemu_vfio_dump_mappings() to use trace events instead
of fprintf().
Signed-off-by: Philippe Mathieu-Daudé
---
util/vfio-helpers.c | 19 ---
u
On 13/10/2020 12.16, Philippe Mathieu-Daudé wrote:
> To avoid the MIPS target being orphan, volunteer to keep an eye
> on it and put together pull requests.
>
> Signed-off-by: Philippe Mathieu-Daudé
> ---
> MAINTAINERS | 6 --
> 1 file changed, 4 insertions(+), 2 deletions(-)
>
> diff --git
For debugging purpose, trace where DMA regions are mapped.
Signed-off-by: Philippe Mathieu-Daudé
---
util/vfio-helpers.c | 3 ++-
util/trace-events | 5 +++--
2 files changed, 5 insertions(+), 3 deletions(-)
diff --git a/util/vfio-helpers.c b/util/vfio-helpers.c
index 278c54902e7..c24a510df82
Pass qemu_vfio_do_mapping() an Error* argument so it can propagate
any error to callers. Replace error_report() which only report
to the monitor by the more generic error_setg_errno().
Signed-off-by: Philippe Mathieu-Daudé
---
util/vfio-helpers.c | 8
1 file changed, 4 insertions(+), 4
Currently qemu_vfio_dma_map() displays errors on stderr.
When using management interface, this information is simply
lost. Pass qemu_vfio_dma_map() an Error* argument so it can
propagate the error to callers.
Signed-off-by: Philippe Mathieu-Daudé
---
include/qemu/vfio-helpers.h | 2 +-
block/nv
For debugging purpose, trace where a BAR is mapped.
Signed-off-by: Philippe Mathieu-Daudé
---
util/vfio-helpers.c | 2 ++
util/trace-events | 1 +
2 files changed, 3 insertions(+)
diff --git a/util/vfio-helpers.c b/util/vfio-helpers.c
index cd6287c3a98..278c54902e7 100644
--- a/util/vfio-help
Instead of displaying the error on stderr, use error_report()
which also report to the monitor.
Signed-off-by: Philippe Mathieu-Daudé
---
util/vfio-helpers.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/util/vfio-helpers.c b/util/vfio-helpers.c
index 2c4598d7faa..488dd
On 12.10.20 19:43, Andrey Shinkevich wrote:
> Limit COR operations by the base node in the backing chain when the
> overlay base node name is given. It will be useful for a block stream
> job when the COR-filter is applied. The overlay base node is passed as
> the base itself may change due to conc
On 12.10.20 19:43, Andrey Shinkevich wrote:
> Limit COR operations by the base node in the backing chain when the
> overlay base node name is given. It will be useful for a block stream
> job when the COR-filter is applied. The overlay base node is passed as
> the base itself may change due to conc
On 14.10.20 13:09, Max Reitz wrote:
> On 12.10.20 19:43, Andrey Shinkevich wrote:
>> We are going to use the COR-filter for a block-stream job.
>> To limit COR operations by the base node in the backing chain during
>> stream job, pass the name of overlay base node to the copy-on-read
>> driver as
Hi,
> I sent you a patch to fix up several assert()s, including that one, about a
> month ago. Did you miss it?
> https://lore.kernel.org/qemu-devel/20200920021449.830-1-pauld...@gmail.com
Seems I missed that, or deleted by accident.
Added to qemu queue now.
thanks,
Gerd
On Wed, Oct 14, 2020 at 06:42:06AM +0900, Dmitry Fomichev wrote:
> The emulation code has been changed to advertise NVM Command Set when
> "zoned" device property is not set (default) and Zoned Namespace
> Command Set otherwise.
>
> Define values and structures that are needed to support Zoned
> N
On Wed, Oct 14, 2020 at 11:41:06AM +0100, Anthony PERARD wrote:
> The EHCI Host Controller emulation attempt to locate the device
> associated with a periodic isochronous transfer description (iTD) and
> when this fail the host controller is reset.
>
> But according the EHCI spec 1.0 section 5.15.
The current implementation will extend the CPUID level to 0x14 if
Intel PT is enabled in the guest(in x86_cpu_expand_features()) and
the Intel PT will be disabled if it can't pass the capabilities
checking later(in x86_cpu_filter_features()). In this case, the
level of CPUID will be still 0x14 and
The current implementation will disable the guest Intel PT
feature if the Intel PT LIP feature is supported on the host,
but the LIP feature is comming soon(e.g. SnowRidge and later).
This patch will make the guest LIP feature configurable and
Intel PT feature can be enabled in guest when the gues
On 14.10.2020 09:51, Claudio Fontana wrote:
привет Pavel!
On 10/14/20 7:42 AM, Pavel Dovgalyuk wrote:
On 13.10.2020 22:21, Claudio Fontana wrote:
this fixes non-TCG builds broken recently by replay reverse debugging.
stub the needed functions in stub/, including errors for hmp and qmp.
This i
Save the parent object's open function pointer in the (new)
VCChardevClass struct instead before overwriting it, so we
can look it up when needed.
Signed-off-by: Gerd Hoffmann
Reviewed-by: Philippe Mathieu-Daudé
---
ui/spice-app.c | 17 +
1 file changed, 13 insertions(+), 4 dele
Signed-off-by: Gerd Hoffmann
Reviewed-by: Philippe Mathieu-Daudé
---
include/chardev/spice.h | 3 ---
chardev/spice.c | 8
2 files changed, 4 insertions(+), 7 deletions(-)
diff --git a/include/chardev/spice.h b/include/chardev/spice.h
index 99f26aedde54..1115502cdfbd 100644
---
Not needed and makes some compilers error out with:
qemu-edid.c:15:1: error: initializer element is not constant
Signed-off-by: Gerd Hoffmann
Reviewed-by: Daniel P. Berrangé
Message-id: 20201013091615.14166-1-kra...@redhat.com
---
qemu-edid.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(
Signed-off-by: Gerd Hoffmann
---
util/module.c | 2 ++
chardev/meson.build | 7 ++-
2 files changed, 8 insertions(+), 1 deletion(-)
diff --git a/util/module.c b/util/module.c
index a44ec38d9362..4a5735dfdc76 100644
--- a/util/module.c
+++ b/util/module.c
@@ -264,6 +264,8 @@ static stru
v3:
- fix spice-app
- init objects before spice
- meson cleanup
v4:
- rebase to latest master, adapt to qom changes.
- add patches to fix modular builds.
- pick up review tags.
v5:
- rebase to latest master, fixup conflicts.
Gerd Hoffmann (7):
qemu-edid: drop cast
ui/spice-app: don't us
Initialize spice before chardevs. That allows to register the spice
chardevs directly in the init function and removes the need to maintain
a linked list of chardevs just for registration.
Signed-off-by: Gerd Hoffmann
---
include/chardev/spice.h | 1 -
include/ui/qemu-spice.h | 1 -
chardev/s
1 - 100 of 358 matches
Mail list logo
