Le 29/10/2018 à 14:39, Mark Cave-Ayland a écrit :
Co-developed-by: Mark Cave-Ayland
Signed-off-by: Mark Cave-Ayland
Signed-off-by: Laurent Vivier
---
hw/display/macfb.c | 56 ++
include/hw/display/macfb.h | 21 +
2 files c
Le 29/10/2018 à 14:39, Mark Cave-Ayland a écrit :
From: Laurent Vivier
This is needed by Quadra 800, this card can run on little-endian
or big-endian bus.
Signed-off-by: Laurent Vivier
Tested-by: Hervé Poussineau
Reviewed-by: Philippe Mathieu-Daudé
Reviewed-by: Hervé Poussineau
Le 29/10/2018 à 14:39, Mark Cave-Ayland a écrit :
From: Laurent Vivier
Co-developed-by: Mark Cave-Ayland
Signed-off-by: Mark Cave-Ayland
Signed-off-by: Laurent Vivier
---
hw/scsi/esp.c | 292 +-
include/hw/scsi/esp.h | 7 ++
2 fil
Le 29/10/2018 à 14:39, Mark Cave-Ayland a écrit :
From: Laurent Vivier
Co-developed-by: Mark Cave-Ayland
Signed-off-by: Mark Cave-Ayland
Signed-off-by: Laurent Vivier
---
hw/block/Makefile.objs | 1 +
hw/block/swim.c | 415
incl
Le 29/10/2018 à 14:39, Mark Cave-Ayland a écrit :
From: Laurent Vivier
Co-developed-by: Mark Cave-Ayland
Signed-off-by: Mark Cave-Ayland
Signed-off-by: Laurent Vivier
---
arch_init.c| 4 +
hw/display/Makefile.objs | 1 +
hw/display/macfb.c | 419
Le 29/10/2018 à 14:39, Mark Cave-Ayland a écrit :
From: Laurent Vivier
Co-developed-by: Mark Cave-Ayland
Signed-off-by: Mark Cave-Ayland
Signed-off-by: Laurent Vivier
---
hw/Makefile.objs| 1 +
hw/nubus/Makefile.objs | 4 +
hw/nubus/mac-nubus-bridge.c
Le 29/10/2018 à 14:39, Mark Cave-Ayland a écrit :
From: Laurent Vivier
Co-developed-by: Mark Cave-Ayland
Signed-off-by: Mark Cave-Ayland
Signed-off-by: Laurent Vivier
---
hw/misc/mac_via.c | 190 ++
include/hw/misc/mac_via.h | 7 ++
2
Le 29/10/2018 à 14:39, Mark Cave-Ayland a écrit :
From: Laurent Vivier
Co-developed-by: Mark Cave-Ayland
Signed-off-by: Mark Cave-Ayland
Signed-off-by: Laurent Vivier
---
hw/misc/Makefile.objs | 1 +
hw/misc/mac_via.c | 666 ++
in
Le 29/10/2018 à 14:39, Mark Cave-Ayland a écrit :
From: Laurent Vivier
This is broken as the linux driver seems broken too...
Co-developed-by: Mark Cave-Ayland
Signed-off-by: Mark Cave-Ayland
Signed-off-by: Laurent Vivier
---
hw/audio/Makefile.objs | 1 +
hw/audio/asc.c | 446 +
Le 29/10/2018 à 14:39, Mark Cave-Ayland a écrit :
From: Laurent Vivier
On Sparc and PowerMac, the bit 0 of the address
selects the register type (control or data) and
bit 1 selects the channel (B or A).
On m68k Macintosh, the bit 0 selects the channel and
bit 1 the register type.
This patch i
On Mon, Oct 29, 2018 at 11:32:07PM +0100, Philippe Mathieu-Daudé wrote:
> From: Guenter Roeck
>
> Signed-off-by: Guenter Roeck
> Reviewed-by: Philippe Mathieu-Daudé
> [PMD: Intented tabs -> space]
> Signed-off-by: Philippe Mathieu-Daudé
> ---
> hw/arm/allwinner-a10.c | 2 ++
> 1 file changed,
On Mon, Oct 29, 2018 at 11:32:06PM +0100, Philippe Mathieu-Daudé wrote:
> From: Guenter Roeck
>
> Signed-off-by: Guenter Roeck
> Signed-off-by: Philippe Mathieu-Daudé
> ---
> Gerd: You already review this code here:
> https://lists.gnu.org/archive/html/qemu-arm/2017-01/msg00345.html
> Can we us
+-- On Mon, 29 Oct 2018, Paolo Bonzini wrote --+
| On 29/10/2018 18:56, Paolo Bonzini wrote:
| > On 26/10/2018 22:55, Peter Maydell wrote:
| >>> +assert(len <= LSI_MAX_MSGIN_LEN);
| >>> pci_dma_write(PCI_DEVICE(s), s->dnad, s->msg, len);
| >>> /* Linux drivers rely on the last byte be
From: Prasad J Pandit
While writing a message in 'lsi_do_msgin', message length value
in 'msg_len' could be invalid. Add check to avoid OOB access issue.
Signed-off-by: Prasad J Pandit
---
hw/scsi/lsi53c895a.c | 19 +--
1 file changed, 17 insertions(+), 2 deletions(-)
Update v
On Mon, Oct 29, 2018 at 08:58:16PM +0800, Fei Li wrote:
> In our current code, when multifd is used during migration, if there
> is an error before the destination receives all new channels, the
> source keeps running, however the destination does not exit but keeps
> waiting until the source is ki
On Mon, Oct 29, 2018 at 04:57:27PM +0400, Marc-André Lureau wrote:
> Hi,
>
> Here is a small series of fixes for the monitor, mostly related to
> threading issues.
Hi, Marc-André, Markus,
I'd be glad to know how you think about these monitor series,
considering that I just noticed it's exactly s
On Mon, Oct 29, 2018 at 04:57:32PM +0400, Marc-André Lureau wrote:
> Add a monitor_destroyed global to check if monitor_cleanup() has been
> already called. In this case, don't insert the new monitor in the
> list, but free it instead.
Pure question: how to trigger the condition when doing monitor
There is no need to make another reference.
Signed-off-by: Li Qiang
---
hw/block/nvme.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/hw/block/nvme.c b/hw/block/nvme.c
index a406c23..f63e344 100644
--- a/hw/block/nvme.c
+++ b/hw/block/nvme.c
@@ -1235,7 +1235,7 @@ st
As this function can fail.
Signed-off-by: Li Qiang
---
hw/block/nvme.c | 5 -
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/hw/block/nvme.c b/hw/block/nvme.c
index 72c9644..a406c23 100644
--- a/hw/block/nvme.c
+++ b/hw/block/nvme.c
@@ -1250,7 +1250,10 @@ static void nvme_reali
Including sanity check and code cleanup.
Li Qiang (4):
nvme: use TYPE_NVME instead of constant string
nvme: ensure the num_queues is not zero
nvme: check msix_init_exclusive_bar return value
nvme: use pci_dev directly in nvme_realize
hw/block/nvme.c | 17 -
1 file changed
When it is zero, it causes segv. Backtrack:
Thread 5 "qemu-system-x86" received signal SIGSEGV, Segmentation fault.
[Switching to Thread 0x7fffc6c17700 (LWP 51808)]
0x55acbd90 in nvme_start_ctrl (n=0x624c8100) at hw/block/nvme.c:820
warning: Source file is more recent than executable.
8
Signed-off-by: Li Qiang
---
hw/block/nvme.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/hw/block/nvme.c b/hw/block/nvme.c
index fc7dacb..676cc48 100644
--- a/hw/block/nvme.c
+++ b/hw/block/nvme.c
@@ -1379,7 +1379,7 @@ static void nvme_instance_init(Object *obj)
}
stati
At 2018-10-30 08:08:55, "Philippe Mathieu-Daudé" wrote:
>On 28/10/18 13:40, Li Qiang wrote:
>> Signed-off-by: Li Qiang
>> ---
>> tests/fw_cfg-test.c | 13 -
>> 1 file changed, 12 insertions(+), 1 deletion(-)
>>
>> diff --git a/tests/fw_cfg-test.c b/tests/fw_cfg-test.c
>>
On 2018/10/30 3:59, Philippe Mathieu-Daudé wrote:
Shannon Zhao's email at Huawei is bouncing: remove it.
X-Failed-Recipients: zhaoshengl...@huawei.com
** Address not found **
Your message wasn't delivered to zhaoshengl...@huawei.com because the
address couldn't be found, or is
This is useful to write qtest about fw_cfg file entry.
Signed-off-by: Li Qiang
---
tests/libqos/fw_cfg.c | 33 +
tests/libqos/fw_cfg.h | 2 ++
2 files changed, 35 insertions(+)
diff --git a/tests/libqos/fw_cfg.c b/tests/libqos/fw_cfg.c
index d0889d1e22..45338655
This patchset is the test case for the following patch:
https://lists.gnu.org/archive/html/qemu-devel/2018-10/msg05254.html
v2:
Change qfw_cfg_get_file per Philippe's review:
https://lists.gnu.org/archive/html/qemu-devel/2018-10/msg06340.html
Li Qiang (2):
tests: fw_cfg: add a function to get
Signed-off-by: Li Qiang
---
tests/fw_cfg-test.c | 10 ++
1 file changed, 10 insertions(+)
diff --git a/tests/fw_cfg-test.c b/tests/fw_cfg-test.c
index 1c5103fe1c..aeabd17ec0 100644
--- a/tests/fw_cfg-test.c
+++ b/tests/fw_cfg-test.c
@@ -99,6 +99,15 @@ static void test_fw_cfg_boot_menu(vo
The original code just follow the TCP state diagram,
but in this case, we can skip the TCPS_TIME_WAIT state to simplify
the implementation.
Signed-off-by: Zhang Chen
---
net/filter-rewriter.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/net/filter-rewriter.c b/net/filter
On Mon, Oct 29, 2018 at 04:57:29PM +0400, Marc-André Lureau wrote:
> Chardev backends may not handle safely IO events from concurrent
> threads. Better to wake up the chardev from the monitor IO thread if
> it's being used as the chardev context.
>
> Unify code paths by using a BH in all cases.
>
On Mon, Oct 29, 2018 at 04:57:28PM +0400, Marc-André Lureau wrote:
> The function were not named with "mon_iothread", or following the AIO
> vs GMainContext distinction. Inline them instead.
>
> Signed-off-by: Marc-André Lureau
Reviewed-by: Peter Xu
--
Peter Xu
Quoting Tomáš Golembiovský (2018-10-23 06:23:09)
> Note that PCI controller reporting on Windows was and still is broken.
> Unfortunately I don't know how to fix it at the momemnt. See commit message
> and
> code comment. If anyone has environment where the original code works let me
> know. CCing
Hi, everyone.
When I have installed the QEMU3.0.0 in the Linux on non X86 CPUS, I want to
use user space emulator to run Wine. I can get the source code for Wine and the
RPM file for Wine from the WineHQ web page.
I used to compile Wine in the X86-based Linux system and used it to run most
W
On Tue, Oct 30, 2018 at 10:02 AM Jason Wang wrote:
>
> On 2018/10/29 下午7:01, Peter Maydell wrote:
> > On 19 October 2018 at 04:22, Jason Wang wrote:
> >> From: Zhang Chen
> >>
> >> We add almost full TCP state machine in filter-rewriter, except
> >> TCPS_LISTEN and some simplify in VM active cl
On 2018/10/29 下午7:01, Peter Maydell wrote:
On 19 October 2018 at 04:22, Jason Wang wrote:
From: Zhang Chen
We add almost full TCP state machine in filter-rewriter, except
TCPS_LISTEN and some simplify in VM active close FIN states.
The reason for this simplify job is because guest kernel wi
Hello Philippe,
Thanks for your review,
I will send v2 later.
Thanks.
Li Qiang
在 2018/10/30 7:46, Philippe Mathieu-Daudé 写道:
> Hi Li,
>
> On 28/10/18 13:40, Li Qiang wrote:
>> This is useful to write qtest abount fw_cfg file entry.
>>
>> Signed-off-by: Li Qiang
>> ---
>> tests/libqos/fw_
On Mon, Oct 29, 2018 at 10:56:57AM +0100, Igor Mammedov wrote:
> On Fri, 26 Oct 2018 12:13:21 -0300
> Eduardo Habkost wrote:
>
> > On Mon, Oct 22, 2018 at 03:33:30PM +0100, Igor Mammedov wrote:
> > > On Wed, 12 Sep 2018 16:55:23 +0400
> > > Marc-André Lureau wrote:
> > >
> > > > Improve a bit
Philippe Mathieu-Daudé 于2018年10月30日周二 上午7:50写道:
> On 25/10/18 3:45, Li Qiang wrote:
> > Hello Laszlo and Philippe ,
> >
> > Thanks for your review,
> >
> >
> >
> > Philippe Mathieu-Daudé mailto:phi...@redhat.com>> 于
> > 2018年10月25日周四 上午6:56写道:
> >
> > Hi,
> >
> > On 24/10/18 13:35, Laszlo
在 2018/10/30 8:08, Philippe Mathieu-Daudé 写道:
> On 28/10/18 13:40, Li Qiang wrote:
>> Signed-off-by: Li Qiang
>> ---
>> tests/fw_cfg-test.c | 13 -
>> 1 file changed, 12 insertions(+), 1 deletion(-)
>>
>> diff --git a/tests/fw_cfg-test.c b/tests/fw_cfg-test.c
>> index 1c5103fe1c..3
Glad you think it's worth it.
Thanks,
Sebastien
From: Philippe Mathieu-Daudé [phi...@redhat.com]
Sent: Monday, October 29, 2018 2:02 PM
To: Samuel Ortiz; qemu-devel@nongnu.org
Cc: Michael S. Tsirkin; Jing Liu; Boeuf, Sebastien; Paolo Bonzini; Igor Mammedov
Yes makes sense. Let's change the naming.
Thanks,
Sebastien
From: Philippe Mathieu-Daudé [phi...@redhat.com]
Sent: Monday, October 29, 2018 12:46 PM
To: Samuel Ortiz; qemu-devel@nongnu.org
Cc: Eduardo Habkost; Michael S. Tsirkin; Jing Liu; Boeuf, Sebastien;
Fix the extraneous extra blank lines in the test output when running with V=1.
Before:
TEST: tests/bios-tables-test... (pid=25678)
/i386/acpi/piix4:
Looking for expected file 'tests/acpi-test-data/pc/DSDT'
Using expected file 'tests/acpi-test-data/pc/DSDT'
Looking for expe
On 28/10/18 13:40, Li Qiang wrote:
Signed-off-by: Li Qiang
---
tests/fw_cfg-test.c | 13 -
1 file changed, 12 insertions(+), 1 deletion(-)
diff --git a/tests/fw_cfg-test.c b/tests/fw_cfg-test.c
index 1c5103fe1c..37765f15f8 100644
--- a/tests/fw_cfg-test.c
+++ b/tests/fw_cfg-test.
On Mon, Oct 29, 2018 at 4:24 PM Philippe Mathieu-Daudé
wrote:
>
> Signed-off-by: Philippe Mathieu-Daudé
Reviewed-by: Alistair Francis
Alistair
> ---
> MAINTAINERS| 1 +
> hw/arm/xilinx_zynq.c | 18 ++
> hw/dma/pl330.c | 2 +-
> include/hw/dma/pl330.h |
On 25/10/18 3:45, Li Qiang wrote:
Hello Laszlo and Philippe ,
Thanks for your review,
Philippe Mathieu-Daudé mailto:phi...@redhat.com>> 于
2018年10月25日周四 上午6:56写道:
Hi,
On 24/10/18 13:35, Laszlo Ersek wrote:
> On 10/24/18 09:11, Li Qiang wrote:
>> This can avoid setting a n
On Mon, Oct 29, 2018 at 4:21 PM Philippe Mathieu-Daudé
wrote:
>
> From: Guenter Roeck
>
> QEMU already supports pl330. Instantiate it for smdkc210.
>
> Signed-off-by: Guenter Roeck
> Reviewed-by: Philippe Mathieu-Daudé
> [PMD: Use pl330_init from "hw/dma/pl330.h"]
> Signed-off-by: Philippe Math
On Mon, Oct 29, 2018 at 4:22 PM Philippe Mathieu-Daudé
wrote:
>
> Signed-off-by: Philippe Mathieu-Daudé
Reviewed-by: Alistair Francis
Alistair
> ---
> hw/arm/xilinx_zynq.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/hw/arm/xilinx_zynq.c b/hw/arm/xilinx_zynq.c
> i
Hi Li,
On 28/10/18 13:40, Li Qiang wrote:
This is useful to write qtest abount fw_cfg file entry.
Signed-off-by: Li Qiang
---
tests/libqos/fw_cfg.c | 30 ++
tests/libqos/fw_cfg.h | 2 ++
2 files changed, 32 insertions(+)
diff --git a/tests/libqos/fw_cfg.c b/te
Hi Marc, Laurent.
On Mon, Oct 29, 2018 at 2:43 PM Mark Cave-Ayland
wrote:
>
> From: Laurent Vivier
>
> On Sparc and PowerMac, the bit 0 of the address
> selects the register type (control or data) and
> bit 1 selects the channel (B or A).
>
> On m68k Macintosh, the bit 0 selects the channel and
Signed-off-by: Philippe Mathieu-Daudé
---
MAINTAINERS| 1 +
hw/arm/xilinx_zynq.c | 18 ++
hw/dma/pl330.c | 2 +-
include/hw/dma/pl330.h | 41 +
4 files changed, 45 insertions(+), 17 deletions(-)
create mode 100644 in
Signed-off-by: Philippe Mathieu-Daudé
---
hw/arm/xilinx_zynq.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/hw/arm/xilinx_zynq.c b/hw/arm/xilinx_zynq.c
index f1496d2927..57497b0c4d 100644
--- a/hw/arm/xilinx_zynq.c
+++ b/hw/arm/xilinx_zynq.c
@@ -294,7 +294,7 @@ static void
From: Guenter Roeck
QEMU already supports pl330. Instantiate it for smdkc210.
Signed-off-by: Guenter Roeck
Reviewed-by: Philippe Mathieu-Daudé
[PMD: Use pl330_init from "hw/dma/pl330.h"]
Signed-off-by: Philippe Mathieu-Daudé
---
hw/arm/exynos4_boards.c | 12
1 file changed, 12 i
Hi, this series is another intent to salvage previous work from Guenter Roeck:
https://lists.gnu.org/archive/html/qemu-devel/2018-10/msg06302.html
Instead of duplicate the pl330_init() from the xilinx_zynq, I factored it out.
Regards,
Phil.
Guenter Roeck (1):
arm: exynos4: Add dma support for
From: Guenter Roeck
Signed-off-by: Guenter Roeck
Reviewed-by: Philippe Mathieu-Daudé
[PMD: Intented tabs -> space]
Signed-off-by: Philippe Mathieu-Daudé
---
hw/arm/allwinner-a10.c | 7 +++
1 file changed, 7 insertions(+)
diff --git a/hw/arm/allwinner-a10.c b/hw/arm/allwinner-a10.c
index
From: Guenter Roeck
Signed-off-by: Guenter Roeck
Signed-off-by: Philippe Mathieu-Daudé
---
Gerd: You already review this code here:
https://lists.gnu.org/archive/html/qemu-arm/2017-01/msg00345.html
Can we use your R-b again?
hw/usb/hcd-ehci-sysbus.c | 17 +
hw/usb/hcd-ehci.h
From: Guenter Roeck
Signed-off-by: Guenter Roeck
Reviewed-by: Philippe Mathieu-Daudé
[PMD: Intented tabs -> space]
Signed-off-by: Philippe Mathieu-Daudé
---
hw/arm/allwinner-a10.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/hw/arm/allwinner-a10.c b/hw/arm/allwinner-a10.c
index 0a8ac
Hi, this series intend to salvage previous work from Guenter Roeck [1].
A similar series from Icenowy Zheng was later reviewed by Gerd Hoffmann:
https://lists.gnu.org/archive/html/qemu-arm/2017-01/msg00345.html [2].
I reviewed patches 1 and 3, Gerd already reviewed patch 2 ([2]), if he
agrees to
On 29/10/18 21:24, Guenter Roeck wrote:
On Mon, Oct 29, 2018 at 04:03:40PM +0100, Philippe Mathieu-Daudé wrote:
On 29/10/18 15:09, Guenter Roeck wrote:
On 10/29/18 6:24 AM, Peter Maydell wrote:
On 27 October 2018 at 12:04, Guenter Roeck wrote:
On 10/26/18 3:12 AM, Peter Maydell wrote:
Hi Gu
On Mon, Oct 29, 2018 at 16:34:49 +, Alex Bennée wrote:
>
> Emilio G. Cota writes:
>
> > We don't pass a pointer to qemu_global_mutex anymore.
> >
> > Reviewed-by: Richard Henderson
> > Signed-off-by: Emilio G. Cota
>
> As discussed on IRC I don't fundamentally object to this being in
> cp
From: Vladimir Sementsov-Ogievskiy
This patch aims to bring the following behavior:
1. We don't load bitmaps, when started in inactive mode. It's the case
of incoming migration. In this case we wait for bitmaps migration
through migration channel (if 'dirty-bitmaps' capability is enabled) or
for
From: Vladimir Sementsov-Ogievskiy
Before previous patch, iotest 169 was actually broken for the case
test_persistent__not_migbitmap__offline_shared, while formally
passing.
After migration log of vm_b had message:
qemu-system-x86_64: Could not reopen qcow2 layer: Bitmap already
exists:
In prior commits that made merge transactionable, we removed the
assertion that merge cannot operate on disabled bitmaps. In addition,
we want to make sure that we are prohibiting merges to "locked" bitmaps.
Use the new user_locked function to check.
Reported-by: Eric Blake
Signed-off-by: John S
Whether it's "locked" or "frozen", it's in use and should
not be allowed for the purposes of this operation.
Signed-off-by: John Snow
Reviewed-by: Vladimir Sementsov-Ogievskiy
Message-id: 20181002230218.13949-7-js...@redhat.com
Signed-off-by: John Snow
---
nbd/server.c | 4 ++--
1 file changed
Similarly to merge, it's OK to allow clear operations on disabled
bitmaps, as this condition only means that they are not recording
new writes. We are free to clear it if the user requests it.
Signed-off-by: John Snow
Reviewed-by: Eric Blake
Reviewed-by: Vladimir Sementsov-Ogievskiy
Message-id:
We're not being consistent about this. If it's in use by an operation,
the user should not be able to change the behavior of that bitmap.
Signed-off-by: John Snow
Reviewed-by: Eric Blake
Reviewed-by: Vladimir Sementsov-Ogievskiy
Message-id: 20181002230218.13949-5-js...@redhat.com
Signed-off-by:
Instead of both frozen and qmp_locked checks, wrap it into one check.
frozen implies the bitmap is split in two (for backup), and shouldn't
be modified. qmp_locked implies it's being used by another operation,
like being exported over NBD. In both cases it means we shouldn't allow
the user to modif
From: Vladimir Sementsov-Ogievskiy
This variable doesn't work as it should, because it is actually cleared
in qcow2_co_invalidate_cache() by memset(). Drop it, as the following
patch will introduce new behavior.
Signed-off-by: Vladimir Sementsov-Ogievskiy
Reviewed-by: John Snow
Signed-off-by:
From: Eric Blake
We need an accurate count of the number of bits set in a bitmap
after a merge. In particular, since the merge operation short-circuits
a merge from an empty source, if you have bitmaps A, B, and C where
B started empty, then merge C into B, and B into A, an inaccurate
count meant
If the bitmap is frozen, we shouldn't touch it.
Signed-off-by: John Snow
Reviewed-by: Vladimir Sementsov-Ogievskiy
Message-id: 20181002230218.13949-6-js...@redhat.com
Signed-off-by: John Snow
---
blockdev.c | 12 ++--
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/blockd
From: Vladimir Sementsov-Ogievskiy
Signed-off-by: Vladimir Sementsov-Ogievskiy
[Maintainer edit -- touched up error message. --js]
Reviewed-by: John Snow
Signed-off-by: John Snow
---
block/qcow2.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/block/qcow2.c b/block/
From: Vladimir Sementsov-Ogievskiy
New action is like clean action: do the whole thing in .prepare and
undo in .abort. This behavior for bitmap-changing actions is needed
because backup job actions use bitmap in .prepare.
Signed-off-by: Vladimir Sementsov-Ogievskiy
Reviewed-by: John Snow
Signe
From: Vladimir Sementsov-Ogievskiy
Test that we can resume source vm after [failed] migration, and bitmaps
are ok.
Signed-off-by: Vladimir Sementsov-Ogievskiy
Signed-off-by: John Snow
---
tests/qemu-iotests/169 | 60 +-
tests/qemu-iotests/169.ou
From: Vladimir Sementsov-Ogievskiy
Rename block-dirty-bitmap-clear transaction handlers to reuse them for
x-block-dirty-bitmap-merge transaction in the following patch.
Signed-off-by: Vladimir Sementsov-Ogievskiy
Reviewed-by: John Snow
Signed-off-by: John Snow
---
blockdev.c | 8
1
From: Vladimir Sementsov-Ogievskiy
Add backup parameter to bdrv_merge_dirty_bitmap() to be used then with
bdrv_restore_dirty_bitmap() if it needed to restore the bitmap after
merge operation.
This is needed to implement bitmap merge transaction action in further
commit.
Signed-off-by: Vladimir
From: Vladimir Sementsov-Ogievskiy
Signed-off-by: Vladimir Sementsov-Ogievskiy
Reviewed-by: John Snow
Signed-off-by: John Snow
---
tests/qemu-iotests/169 | 1 -
1 file changed, 1 deletion(-)
diff --git a/tests/qemu-iotests/169 b/tests/qemu-iotests/169
index f243db9955..df408f8367 100755
---
The following changes since commit 62b658db1df7c6fa574caae038144f24bf6ca495:
Merge remote-tracking branch 'remotes/stefanha/tags/block-pull-request' into
staging (2018-10-29 17:03:27 +)
are available in the Git repository at:
https://github.com/jnsnow/qemu.git tags/bitmaps-pull-request
From: Vladimir Sementsov-Ogievskiy
Use more generic names to reuse the function for bitmap merge in the
following commit.
Signed-off-by: Vladimir Sementsov-Ogievskiy
Reviewed-by: John Snow
Signed-off-by: John Snow
---
block/dirty-bitmap.c | 4 ++--
blockdev.c| 2 +-
incl
It is only an oversight that we don't allow incremental backup with
blockdev-backup. Add the bitmap argument which enables this.
Signed-off-by: John Snow
Message-id: 20180830211605.13683-2-js...@redhat.com
Signed-off-by: John Snow
---
blockdev.c | 18 +-
qapi/block-cor
From: Vladimir Sementsov-Ogievskiy
Move checks from qmp_x_block_dirty_bitmap_merge() to
bdrv_merge_dirty_bitmap(), to share them with dirty bitmap merge
transaction action in future commit.
Note: for now, only qmp_x_block_dirty_bitmap_merge() calls
bdrv_merge_dirty_bitmap().
Signed-off-by: Vlad
On 29/10/18 18:01, Samuel Ortiz wrote:
All PC machine type derivatives will use the same ACPI table build
methods. But with that change in place, any new x86 machine type will be
able to re-use the acpi-build API and customize part of it by defining
its own ACPI table build methods.
Signed-off-b
Hi Samuel,
On 29/10/18 18:01, Samuel Ortiz wrote:
It is going to be used by the PC machine type as the MADT table builder
method and thus needs to be exported outside of acpi-build.c
Also, now that the generic build_madt() API is exported, we have to
rename the ARM static one in order to avoid
On 29.10.18 16:42, Christian Borntraeger wrote:
> I discussed the following maintainer updates at the kvm forum with
> Conny. This is the first set of changes.
> Can the affected persons please Ack the relevant patches?
>
> PS: Alex, do you still want to be listed?
No, I've been meaning to rem
Hi Samuel,
On 29/10/18 18:01, Samuel Ortiz wrote:
In order to decouple ACPI APIs from specific machine types, we are
creating an ACPI builder interface that each ACPI platform can choose to
implement.
This way, a new machine type can re-use the high level ACPI APIs and
define some custom table b
On 29/10/18 18:01, Samuel Ortiz wrote:
From: Sebastien Boeuf
Instead of using the machine type specific method find_i440fx() to
retrieve the PCI bus, this commit aims to rely on the fact that the
PCI bus is known by the structure AcpiPciHpState.
When the structure is initialized through acpi_p
Le 29/10/2018 à 15:53, Aleksandar Markovic a écrit :
>> From: Stefan Markovic
>> Subject: [PATCH v2 0/6] target/mips: Add support for prctl() PR_GET_FP_MODE
>> and PR_SET_FP_MODE
>>
>> From: Stefan Markovic
>>
>> This series includes support for prctl() PR_GET_FP_MODE and PR_SET_FP_MODE.
>> Thi
On Mon, Oct 29, 2018 at 04:03:40PM +0100, Philippe Mathieu-Daudé wrote:
> Hi Guenter,
>
> On 29/10/18 15:09, Guenter Roeck wrote:
> >On 10/29/18 6:24 AM, Peter Maydell wrote:
> >>On 27 October 2018 at 12:04, Guenter Roeck wrote:
> >>>On 10/26/18 3:12 AM, Peter Maydell wrote:
> Hi Guenter; the
On Mon, Oct 29, 2018 at 7:46 PM Stefan Hajnoczi wrote:
> @@ -39,8 +47,12 @@ static void microbit_init(MachineState *machine)
> &error_fatal);
> object_property_set_bool(soc, true, "realized", &error_fatal);
>
> -armv7m_load_kernel(ARM_CPU(first_cpu), machine-
On 10/26/2018 10:22 AM, Daniel P. Berrangé wrote:
>
> On Fri, Oct 26, 2018 at 04:03:51PM +0200, Markus Armbruster wrote:
>> This is from my (imperfect) notes, corrections welcome.
>>
>> Motivation: QEMU contains stuff of dubious value, which gets in the way
>> in various (sometimes painful and
From: Li Qiang
Call it in device unrealize function.
Signed-off-by: Li Qiang
Message-id: 1539946236-18028-2-git-send-email-liq...@gmail.com
Signed-off-by: Gerd Hoffmann
---
hw/usb/ccid-card-emulated.c | 7 +++
1 file changed, 7 insertions(+)
diff --git a/hw/usb/ccid-card-emulated.c b/hw/
The following changes since commit b312532fd03413d0e6ae6767ec793a3e30f487b8:
Merge remote-tracking branch 'remotes/bonzini/tags/for-upstream' into staging
(2018-10-19 19:01:07 +0100)
are available in the git repository at:
git://git.kraxel.org/qemu tags/usb-20181029-pull-reques
From: Li Qiang
Signed-off-by: Li Qiang
Message-id: 1539946236-18028-3-git-send-email-liq...@gmail.com
Signed-off-by: Gerd Hoffmann
---
hw/usb/ccid-card-emulated.c | 20 ++--
1 file changed, 14 insertions(+), 6 deletions(-)
diff --git a/hw/usb/ccid-card-emulated.c b/hw/usb/ccid
From: Li Qiang
This can avoid setting OCHIState.num_ports to a negative num.
Signed-off-by: Li Qiang
Message-id: 1540263618-18344-1-git-send-email-liq...@gmail.com
Signed-off-by: Gerd Hoffmann
---
hw/usb/hcd-ohci.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/hw/u
On Sat, 27 Oct 2018 01:42:02 PDT (-0700), alistai...@gmail.com wrote:
On Fri, Oct 26, 2018 at 7:14 PM Dayeol Lee wrote:
Hi,
I submitted the patch, but just found this has been already fixed by
Michael Clark
and pushed to riscv/riscv-qemu https://github.com/riscv/riscv-qemu/pull/166
but not in
Shannon Zhao's email at Huawei is bouncing: remove it.
X-Failed-Recipients: zhaoshengl...@huawei.com
** Address not found **
Your message wasn't delivered to zhaoshengl...@huawei.com because the
address couldn't be found, or is unable to receive mail.
Note that the section still cont
Hi Samuel,
On 29/10/18 18:01, Samuel Ortiz wrote:
ACPI tables are platform and machine type and even architecture
agnostic, and as such we want to provide an internal ACPI API that
only depends on platform agnostic information.
For the x86 architecture, in order to build ACPI tables independent
On 29/10/18 18:01, Samuel Ortiz wrote:
From: Sebastien Boeuf
The ACPI hotplug support for PCI devices APIs are not x86 or even
machine type specific. In order for future machine types to be able to
re-use that code, we export it through the architecture agnostic
hw/acpi folder.
Signed-off-by:
Microbit programs are typically in Intel HEX (.hex) format. The generic
loader supports .hex files but it doesn't work as expected:
$ qemu-system-arm -M microbit -device loader,file=microbit.hex
Guest image must be specified (using -kernel)
This error comes from armv7m_load_kernel(). Don't
On 29/10/18 18:01, Samuel Ortiz wrote:
From: Yang Zhong
The SRAT ACPI table is not x86 specific and will be needed for the
Hardware-reduced ACPI implementation. So we should export it through the
architecture independent hw/acpi folder.
Also, now that the generic build_srat() API is exported,
On 29/10/18 18:01, Samuel Ortiz wrote:
This property is currently defined under i386/pc while it only describes
a region size that's eventually fetched from the AML ACPI code.
We can make it more generic and shareable across machine types by moving
it to memory-device.h instead.
It makes sens
On 29/10/18 18:01, Samuel Ortiz wrote:
CPU and memory ACPI hotplug are not necessarily handled through SCI
events. For example, with Hardware-reduced ACPI, the GED device will
manage ACPI hotplug entirely.
As a consequence, we make the CPU and memory specific events AML
generation optional. The c
On 29/10/18 18:01, Samuel Ortiz wrote:
From: Yang Zhong
The ACPI MCFG getter is not x86 specific and could be called from
anywhere within generic ACPI API, so let's export it.
Signed-off-by: Yang Zhong
Reviewed-by: Philippe Mathieu-Daudé
Tested-by: Philippe Mathieu-Daudé
---
hw/acpi/a
1 - 100 of 391 matches
Mail list logo