On Tue, Jun 27, 2017 at 9:46 PM Eric Blake wrote:
> On 06/11/2017 02:48 AM, Stefan Fritsch wrote:
> > Sending a break on a serial console can be useful for debugging the
> > guest. But not all chardev backends support sending breaks (only telnet
> > and mux do). The chardev-send-break command all
* Suraj Jitindar Singh (sjitindarsi...@gmail.com) wrote:
> The info registers command in the qemu monitor is used to dump register
> values.
>
> Currently this command uses the monitor cpu (which can be set by the
> user) as the cpu for whose registers will be dumped. Sometimes it is
> useful to s
On 14 June 2017 at 17:49, Paolo Bonzini wrote:
> This patch and patch 2 are
> useful, but I think I'd rather get full coverage, either with Shippable
> or by keeping on doing manual builds, than to rush things and switch to
> CI when it's not ready.
I agree with this, so is there a path forwards
On Thu, 29 Jun 2017 16:10:20 +0300
Roman Kagan wrote:
> On Thu, Jun 29, 2017 at 01:53:29PM +0200, Igor Mammedov wrote:
> > On Thu, 29 Jun 2017 12:53:27 +0300
> > Roman Kagan wrote:
> >
> > > On Wed, Jun 28, 2017 at 04:47:43PM +0200, Igor Mammedov wrote:
> > > > On Wed, 21 Jun 2017 19:24:08
* Marc-André Lureau (marcandre.lur...@gmail.com) wrote:
> On Tue, Jun 27, 2017 at 9:46 PM Eric Blake wrote:
>
> > On 06/11/2017 02:48 AM, Stefan Fritsch wrote:
> > > Sending a break on a serial console can be useful for debugging the
> > > guest. But not all chardev backends support sending break
On 06/29/2017 05:46 AM, Thomas Huth wrote:>> I figured out MAINTAINERS
was unsync, so added patches 1-4, they are not really
tci-related.
Since they are not related to TCI at all, please submit these as
separate series.
Ok.
Patches 5,6 are trivial fixes to let the codebase quite sane if the
* Eric Blake (ebl...@redhat.com) wrote:
> On 06/11/2017 02:48 AM, Stefan Fritsch wrote:
> > Sending a break on a serial console can be useful for debugging the
> > guest. But not all chardev backends support sending breaks (only telnet
> > and mux do). The chardev-send-break command allows to send
As requested by Thomas Huth, I separated these patches from my previous series
"removal of tci (tcg interpreter)".
TCG/KVM/Xen/TCI related files were moved in commits a9ded601..244f1441, update
MAINTAINERS to match the new paths.
Also add an entry for the "Unimplemented" device and voluntary myse
moved in 92229a57 to accel/
Signed-off-by: Philippe Mathieu-Daudé
---
MAINTAINERS | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/MAINTAINERS b/MAINTAINERS
index 06006fc7df..86a08c5aac 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -273,8 +273,8 @@ Overall
M: Paolo Bonzini
moved in a9ded601..244f1441 to accel/
Signed-off-by: Philippe Mathieu-Daudé
---
MAINTAINERS | 6 +-
1 file changed, 1 insertion(+), 5 deletions(-)
diff --git a/MAINTAINERS b/MAINTAINERS
index 839f7ca063..06006fc7df 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -84,14 +84,10 @@ M: Paolo Bonz
moved in 56e2cd24..28b99f47 to hw/xen/ and hw/i386/xen/
Signed-off-by: Philippe Mathieu-Daudé
Acked-by: Anthony PERARD
---
MAINTAINERS | 1 -
1 file changed, 1 deletion(-)
diff --git a/MAINTAINERS b/MAINTAINERS
index 86a08c5aac..530293044b 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -323,7 +
Also voluntary myself as reviewer
Signed-off-by: Philippe Mathieu-Daudé
---
MAINTAINERS | 7 +++
1 file changed, 7 insertions(+)
diff --git a/MAINTAINERS b/MAINTAINERS
index 9bad523060..890c25f0a1 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -1155,6 +1155,13 @@ F: docs/specs/vmgenid.txt
F
moved in 244f1441 to tcg/
Signed-off-by: Philippe Mathieu-Daudé
---
MAINTAINERS | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/MAINTAINERS b/MAINTAINERS
index 530293044b..9bad523060 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -1654,7 +1654,7 @@ TCI target
M: Stefan Weil
[meta-comment]
On 06/29/2017 01:44 AM, Thomas Huth wrote:
...
>> On 22.06.2017 13:26, Laurent Vivier wrote:
>>> CPU_POWERPC_POWER9_DD1 is 0x004E0100, so this
As a reminder, 14 levels of > quoting is a bit much. It's not only
acceptable, but encouraged, to trim replies
On Wed, 21 Jun 2017 19:24:15 +0300
Roman Kagan wrote:
> Make Hyper-V SynIC a device which is attached as a child to X86CPU. For
> now it only makes SynIC visibile in the qom hierarchy, and maintains its
> internal fields in sync with the respecitve msrs of the parent cpu (the
> fields will be us
On 29/06/2017 10:46, Thomas Huth wrote:
>> Patches 7,8 are the removal, marked RFC... let's debate!
> ... but NACK for a direct removal. Common sense is to make obsolete
> features as deprecated first and then wait for 2 public releases before
> the final removal, so that users still have a chanc
This patch enables qmp interfaces for the fsdev
devices. This provides two interfaces one
for querying info of all the fsdev devices. The second one
to set the IO limits for the required fsdev device.
Signed-off-by: Pradeep Jagadeesh
Reviewed-by: Greg Kurz
Reviewed-by: Eric Blake
---
qapi/blo
This patch creates a throttle initialization function to maximize the
code reusability. The same code is also used by fsdev.
Signed-off-by: Pradeep Jagadeesh
---
hmp.c | 21 +++--
1 file changed, 15 insertions(+), 6 deletions(-)
diff --git a/hmp.c b/hmp.c
index 8c72c58..220d301
This patch factor out the duplicate throttle code that was present in
block and fsdev devices.
Signed-off-by: Pradeep Jagadeesh
Reviewed-by: Alberto Garcia
---
blockdev.c | 44 +--
fsdev/qemu-fsdev-throttle.c | 44 ++--
On 06/29/2017 03:28 AM, Vladimir Sementsov-Ogievskiy wrote:
> Hi!
>
> Can we apply my "[PATCH v22 00/30] qcow2: persistent dirty bitmaps"
> first? It was already near to the victory a week ago, but I had to
> rebase it on new Paolo's patches.
Sure, I can rebase on top of yours (I've already rebas
These patches provide the qmp interface, to query the io throttle
status of the all fsdev devices that are present in a vm.
also, it provides an interface to set the io throttle parameters of a
fsdev to a required value. some of the patches also remove the duplicate
code that was present in block
This patch introduces qmp interfaces for the fsdev
devices. This provides two interfaces one
for querying info of all the fsdev devices. The second one
to set the IO limits for the required fsdev device.
Signed-off-by: Pradeep Jagadeesh
---
Makefile| 4 +++
fsdev/qemu-fsdev
This patch introduces hmp interfaces for the fsdev
devices.
Signed-off-by: Pradeep Jagadeesh
---
hmp-commands-info.hx | 18 ++
hmp-commands.hx | 19 +++
hmp.c| 66
hmp.h| 4
4
On 29 June 2017 at 10:28, KONRAD Frederic wrote:
> This fixes an odd bug when a ROM is present somewhere and an alias @0x
> is pointing to the ROM. The "if (rom)" test fails and we don't get a valid
> reset
> state. QEMU later crashes with an exception because the ARMv7-M starts with
> t
This patch move out the throttle code to util/throttle.c to maximize
the reusability of the code.The same code is also used by fsdev.
Signed-off-by: Pradeep Jagadeesh
---
blockdev.c | 53 +++-
include/qemu/throttle-options.h | 4 +++
util/thr
inlined create_unimplemented_device() calls sysbus_mmio_map_overlap().
Signed-off-by: Philippe Mathieu-Daudé
Reviewed-by: Eric Blake
---
This is a patch from my previous series "various easy cleanups".
v3:
- Add Eric Blake R-b
v2:
- Address review feedback from Eric Blake
v1:
- Add missing i
On Wed, 21 Jun 2017 19:24:01 +0300
Roman Kagan wrote:
> This series applies miscellaneous fixes and enhancements to Hyper-V
> emulation code in QEMU, and lays out the ground for VMBus devices.
Series is a big random mix of cleanup/refactoring/new features
which is hard to review due to mixed con
On 28 June 2017 at 20:37, Programmingkid wrote:
> The [NSEvent modifierFlags] method returns an NSEventModifierFlags type value
> in Mac OS 10.10. It use to be of type NSUInteger. Replacing
> NSEventModifierFlags with NSUInteger allows for the cooca.m file to be
> compiled on older versions of
On 27 June 2017 at 16:35, Alexander Graf wrote:
> When running with KVM enabled, you can choose between emulating the
> gic in kernel or user space. If the kernel supports in-kernel virtualization
> of the interrupt controller, it will default to that. If not, if will
> default to user space emula
Am 29.06.2017 um 17:03 schrieb Philippe Mathieu-Daudé:
> moved in 244f1441 to tcg/
>
> Signed-off-by: Philippe Mathieu-Daudé
> ---
> MAINTAINERS | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 530293044b..9bad523060 100644
> --- a/MAINTAI
On 06/29/2017 08:27 AM, Paolo Bonzini wrote:
> This will be used in the next patch, which will call bdrv_qed_do_open
> with a CoMutex taken. bdrv_qed_init_state provides a nice place to
> initialize it.
>
> Signed-off-by: Paolo Bonzini
> ---
> RFC->v2: new
>
> block/qed.c | 15
On Thu, Jun 29, 2017 at 03:07:14PM +0100, Mark Cave-Ayland wrote:
> As part of some ongoing sun4u work, I need to be able to wire the fw_cfg
> IO interface to a separate IO space by instantiating the qdev device instead
> of calling fw_cfg_init_io(). This patchset brings FW_CFG_IO in line with
> FW
On Thu, 29 Jun 2017 15:45:04 +0200
Paolo Bonzini wrote:
> On 29/06/2017 13:14, Igor Mammedov wrote:
> > link's check callback is supposed to verify/permit setting it,
> > however currently nothing restricts it from misusing it
> > and modifying target object from within.
> > Make sure that readon
Peter Maydell writes:
> On 19 June 2017 at 11:46, Alex Bennée wrote:
>> This is a precursor to record/playback support. Instead of passing the
>> socket fd we now pass helper functions for reading/writing and
>> responding. This will allow us to do the rest of the record/playback
>> cleanly out
On 29.06.2017 17:06, Paolo Bonzini wrote:
>
>
> On 29/06/2017 10:46, Thomas Huth wrote:
>>> Patches 7,8 are the removal, marked RFC... let's debate!
>> ... but NACK for a direct removal. Common sense is to make obsolete
>> features as deprecated first and then wait for 2 public releases before
>>
On 29/06/2017 17:46, Thomas Huth wrote:
>> TCI is enabled only with a specific configure argument if your machine
>> is not supported by TCG. This would break _build_ configurations, not
>> user configurations. It's a remote possibility that users are building
>> their own QEMU, with TCI enable
On 29.06.2017 17:03, Philippe Mathieu-Daudé wrote:
> moved in a9ded601..244f1441 to accel/
>
> Signed-off-by: Philippe Mathieu-Daudé
> ---
> MAINTAINERS | 6 +-
> 1 file changed, 1 insertion(+), 5 deletions(-)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 839f7ca063..06006fc7df 100644
On 29.06.2017 17:03, Philippe Mathieu-Daudé wrote:
> moved in 92229a57 to accel/
>
> Signed-off-by: Philippe Mathieu-Daudé
> ---
> MAINTAINERS | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 06006fc7df..86a08c5aac 100644
> --- a/MAINTAI
On 29.06.2017 17:47, Paolo Bonzini wrote:
>
>
> On 29/06/2017 17:46, Thomas Huth wrote:
>>> TCI is enabled only with a specific configure argument if your machine
>>> is not supported by TCG. This would break _build_ configurations, not
>>> user configurations. It's a remote possibility that us
On 06/29/2017 09:05 AM, Stefan Hajnoczi wrote:
> On Wed, Jun 28, 2017 at 01:46:15AM +0300, Manos Pitsidianakis wrote:
>> The following functions fail if bs->drv does not implement them:
>>
>> bdrv_probe_blocksizes
>> bdrv_probe_geometry
>> bdrv_truncate
>> bdrv_has_zero_init
>> bdrv_get_info
>> bdr
On 29 June 2017 at 14:43, Greg Kurz wrote:
> The following changes since commit 464588675455afda2899e20a0b120e4075de50c7:
>
> Merge remote-tracking branch 'remotes/sstabellini/tags/xen-20170627-tag'
> into staging (2017-06-29 11:45:01 +0100)
>
> are available in the git repository at:
>
> htt
On Thu, Jun 29, 2017 at 05:46:26PM +0200, Thomas Huth wrote:
> On 29.06.2017 17:06, Paolo Bonzini wrote:
> >
> >
> > On 29/06/2017 10:46, Thomas Huth wrote:
> >>> Patches 7,8 are the removal, marked RFC... let's debate!
> >> ... but NACK for a direct removal. Common sense is to make obsolete
> >>
On 06/29/2017 08:39 AM, Javier Martinez Canillas wrote:
Hello Stefan,
Thanks a lot for the summary, it's very informative. I've a question below.
On 06/27/2017 06:12 PM, Stefan Berger wrote:
QEMU TPM Device
===
= Guest-side Hardware Interface =
The QEMU TPM emulation implements
On 06/13/2017 10:54 AM, Peter Maydell wrote:
> Add a new script to automate the process of running the Coverity
> Scan build tools and uploading the resulting tarball to the
> website. This is primarily intended to be driven from Travis,
> but it can be run locally (if you are a maintainer of the
>
On 29 June 2017 at 17:07, Daniel P. Berrange wrote:
> The flipside is that even if we delete it, if someone does suddenly
> care, the code is still there in git & easy to undelete again. Given
> that we believe there are zero users, it is known broken in many
> ways, and TCG provides a working alt
On 29 June 2017 at 17:12, Eric Blake wrote:
> Thankfully, I didn't see any bashisms, so you appeared to have stuck to
> portable /bin/sh code.
My /bin/sh is dash, which catches at least some of the more
obvious bashisms.
thanks
-- PMM
On 29.06.2017 18:07, Daniel P. Berrange wrote:
> On Thu, Jun 29, 2017 at 05:46:26PM +0200, Thomas Huth wrote:
>> On 29.06.2017 17:06, Paolo Bonzini wrote:
>>>
>>>
>>> On 29/06/2017 10:46, Thomas Huth wrote:
> Patches 7,8 are the removal, marked RFC... let's debate!
... but NACK for a direc
On 06/29/2017 12:47 PM, Paolo Bonzini wrote:
On 29/06/2017 17:46, Thomas Huth wrote:
TCI is enabled only with a specific configure argument if your machine
is not supported by TCG. This would break _build_ configurations, not
user configurations. It's a remote possibility that users are buildi
The content of the backends/trace-events file was entirely
removed in
commit 6b10e573d15ef82dbc5c5b3726028e6642e134f6
Author: Marc-André Lureau
Date: Mon May 29 12:39:42 2017 +0400
char: move char devices to chardev/
Leaving the empty file around, causes tracetool to generate
an emp
On 06/29/2017 12:52 AM, Pranith Kumar wrote:
We can use a branch to register instruction for exit_tb for offsets
greater than 128MB.
CC: Richard Henderson
CC: Alex Bennée
Signed-off-by: Pranith Kumar
---
tcg/aarch64/tcg-target.inc.c | 15 +--
1 file changed, 13 insertions(+), 2 de
On 06/29/2017 11:20 AM, Daniel P. Berrange wrote:
> The content of the backends/trace-events file was entirely
> removed in
>
> commit 6b10e573d15ef82dbc5c5b3726028e6642e134f6
> Author: Marc-André Lureau
> Date: Mon May 29 12:39:42 2017 +0400
>
> char: move char devices to chardev/
>
On 06/28/2017 08:02 PM, Philippe Mathieu-Daudé wrote:
> remove some copy/paste leftover, code seems sane.
>
>
> Signed-off-by: Philippe Mathieu-Daudé
>
> Signed-off-by: Philippe Mathieu-Daudé
Double-sob seems odd.
> ---
> tcg/tci.c | 1 -
> 1 file changed, 1 deletion(-)
>
Otherwise,
Revi
From: Suraj Jitindar Singh
The info registers command in the qemu monitor is used to dump register
values.
Currently this command uses the monitor cpu (which can be set by the
user) as the cpu for whose registers will be dumped. Sometimes it is
useful to see the registers for all cpus and curren
/dagrh/qemu.git tags/pull-hmp-20170629
for you to fetch changes up to bd1d5ad9f9a1347d6f4338f294253617c565c89a:
Add chardev-send-break monitor command (2017-06-29 17:14:11 +0100)
From: Stefan Fritsch
Sending a break on a serial console can be useful for debugging the
guest. But not all chardev backends support sending breaks (only telnet
and mux do). The chardev-send-break command allows to send a break even
if using other backends.
Signed-off-by: Stefan Fritsch
Acked-b
On Thu, Jun 29, 2017 at 5:25 AM, Paolo Bonzini wrote:
>
>
> On 28/06/2017 01:57, Alistair Francis wrote:
>> Signed-off-by: Alistair Francis
>> Acked-by: Edgar E. Iglesias
>> ---
>>
>> util/oslib-win32.c | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/util/oslib-win32
On 06/29/2017 01:20 PM, Daniel P. Berrange wrote:
The content of the backends/trace-events file was entirely
removed in
commit 6b10e573d15ef82dbc5c5b3726028e6642e134f6
Author: Marc-André Lureau
Date: Mon May 29 12:39:42 2017 +0400
char: move char devices to chardev/
Leaving th
On Thu, Jun 29, 2017 at 5:34 AM, Paolo Bonzini wrote:
>
>
> On 28/06/2017 01:57, Alistair Francis wrote:
>> +/* We only found one and we are waiting on more then one. Let's try
>> + * again.
>> */
>> -if (timeout == 0 && nhandles > 1) {
>> +if (nhandles >
On 06/29/2017 12:52 AM, Pranith Kumar wrote:
We use ADRP+ADD to compute the target address for goto_tb. This patch
introduces the NOP instruction which is used to align the above
instruction pair so that we can use one atomic instruction to patch
the destination offsets.
CC: Richard Henderson
C
On Thu, Jun 29, 2017 at 6:32 AM, Paolo Bonzini wrote:
> On 28/06/2017 01:57, Alistair Francis wrote:
>> There is no way nhandles can be zero in this section so that part of the
>> if statement will always be false. Let's just remove it to make the code
>> easier to read.
>>
>> Signed-off-by: Alist
On 06/29/2017 12:52 AM, Pranith Kumar wrote:
This patch enables the indirect jump path using an LDR (literal)
instruction. It will be interesting to test and see which performs
better among the two paths.
CC: Richard Henderson
CC: Alex Bennée
Signed-off-by: Pranith Kumar
---
tcg/aarch64/tcg-ta
On 06/29/2017 01:29 PM, Eric Blake wrote:
On 06/28/2017 08:02 PM, Philippe Mathieu-Daudé wrote:
Signed-off-by: Philippe Mathieu-Daudé
Signed-off-by: Philippe Mathieu-Daudé
Double-sob seems odd.
eh you never know... I need to improve my mailing skills :|
---
tcg/tci.c | 1 -
1 file c
On 06/29/2017 05:14 PM, Peter Maydell wrote:
On 29 June 2017 at 10:28, KONRAD Frederic wrote:
This fixes an odd bug when a ROM is present somewhere and an alias @0x
is pointing to the ROM. The "if (rom)" test fails and we don't get a valid reset
state. QEMU later crashes with an excep
Am 29.06.2017 um 17:47 schrieb Paolo Bonzini:
>
>
> On 29/06/2017 17:46, Thomas Huth wrote:
>>> TCI is enabled only with a specific configure argument if your machine
>>> is not supported by TCG. This would break _build_ configurations, not
>>> user configurations. It's a remote possibility tha
On 29 June 2017 at 17:41, KONRAD Frederic wrote:
> On 06/29/2017 05:14 PM, Peter Maydell wrote:
>> This is awkward, because in the "we have a ROM but it's not been
>> copied into memory yet" case, the only thing we have is the
>> rom->addr, which is the address which the user's ROM blob said
>> it
Am 29.06.2017 um 18:29 schrieb Eric Blake:
> On 06/28/2017 08:02 PM, Philippe Mathieu-Daudé wrote:
>> remove some copy/paste leftover, code seems sane.
>>
>
>>
>> Signed-off-by: Philippe Mathieu-Daudé
>>
>> Signed-off-by: Philippe Mathieu-Daudé
>
> Double-sob seems odd.
Maybe that double SOB c
On 29 June 2017 at 17:45, Stefan Weil wrote:
> TCI was never meant to be used for production, so I think
> my approach was acceptable.
I think perhaps this is at the root of some of the difference
in perspective here -- I tend to the view that given that
QEMU is a production codebase, we should n
On 06/29/2017 10:07 AM, Javier Martinez Canillas wrote:
Hello Stefan,
On 06/28/2017 10:57 PM, Stefan Berger wrote:
On 06/28/2017 12:44 PM, Laszlo Ersek wrote:
On 06/28/17 17:22, Peter Jones wrote:
On Tue, Jun 27, 2017 at 12:12:50PM -0400, Stefan Berger wrote:
[snip]
To support measurements
On Thu, 29 Jun 2017 17:00:51 +0100
Peter Maydell wrote:
> On 29 June 2017 at 14:43, Greg Kurz wrote:
> > The following changes since commit 464588675455afda2899e20a0b120e4075de50c7:
> >
> > Merge remote-tracking branch 'remotes/sstabellini/tags/xen-20170627-tag'
> > into staging (2017-06-29 1
On 06/29/2017 01:52 PM, Stefan Weil wrote:
Am 29.06.2017 um 18:29 schrieb Eric Blake:
On 06/28/2017 08:02 PM, Philippe Mathieu-Daudé wrote:
remove some copy/paste leftover, code seems sane.
Signed-off-by: Philippe Mathieu-Daudé
Signed-off-by: Philippe Mathieu-Daudé
Double-sob seems od
On Thu, Jun 29, 2017 at 03:18:05PM +0100, Daniel P. Berrange wrote:
> On Thu, Jun 29, 2017 at 03:39:58PM +0200, Paolo Bonzini wrote:
> > On 28/06/2017 11:05, Markus Armbruster wrote:
> > > If foo() additionally returned an indication of success, you could write
> > >
> > > if (!foo(arg, errp
On 06/28/2017 01:42 PM, Laurent Vivier wrote:
use DisasCompare with FPU conditions in fscc and fbcc.
Signed-off-by: Laurent Vivier
---
target/m68k/translate.c | 210 ++--
1 file changed, 131 insertions(+), 79 deletions(-)
Reviewed-by: Richard Hend
Signed-off-by: Alistair Francis
Acked-by: Edgar E. Iglesias
---
Changes since RFC:
- Include more bitmasks for the select call
util/aio-win32.c | 13 ++---
1 file changed, 10 insertions(+), 3 deletions(-)
diff --git a/util/aio-win32.c b/util/aio-win32.c
index bca496a47a..d6d5e02f00 10
Changes since RFC:
- Remove controversial timeout increate patch (util/oslib-win32:
Recursivly pass the timeout) from the series
- Small adjustments based on comments
- Change from RFC to patch series
Original cover letter:
At Xilinx we have started to run our fork of QEMU on Windows and ar
The original ready < nhandles - 1 can be re-written as ready + 1 <
nhandles which is the same confition that we are checking on the first
itteration of the for loop. This means we can remove the if statement
and let the for loop check the code.
This also has the side effect of removing an invalid
There is no way nhandles can be zero in this section so that part of the
if statement will always be false. Let's just remove it to make the code
easier to read.
Signed-off-by: Alistair Francis
Acked-by: Edgar E. Iglesias
Reviewed-by: Philippe Mathieu-Daudé
Reviewed-by: Fam Zheng
---
util/os
On 06/29/2017 02:16 PM, Alistair Francis wrote:
Signed-off-by: Alistair Francis
Acked-by: Edgar E. Iglesias
Reviewed-by: Philippe Mathieu-Daudé
---
Changes since RFC:
- Include more bitmasks for the select call
util/aio-win32.c | 13 ++---
1 file changed, 10 insertions(+), 3
On Thu, Jun 29, 2017 at 04:39:00PM +0200, Igor Mammedov wrote:
> On Thu, 29 Jun 2017 16:10:20 +0300
> Roman Kagan wrote:
>
> > On Thu, Jun 29, 2017 at 01:53:29PM +0200, Igor Mammedov wrote:
> > > On Thu, 29 Jun 2017 12:53:27 +0300
> > > Roman Kagan wrote:
> > >
> > > > On Wed, Jun 28, 2017 at
On Thu, Jun 29, 2017 at 02:09:39PM -0300, Eduardo Habkost wrote:
> On Thu, Jun 29, 2017 at 03:18:05PM +0100, Daniel P. Berrange wrote:
> > On Thu, Jun 29, 2017 at 03:39:58PM +0200, Paolo Bonzini wrote:
> > > On 28/06/2017 11:05, Markus Armbruster wrote:
> > > > If foo() additionally returned an ind
On Thu, Jun 29, 2017 at 06:38:50PM +0100, Daniel P. Berrange wrote:
> On Thu, Jun 29, 2017 at 02:09:39PM -0300, Eduardo Habkost wrote:
> > On Thu, Jun 29, 2017 at 03:18:05PM +0100, Daniel P. Berrange wrote:
> > > On Thu, Jun 29, 2017 at 03:39:58PM +0200, Paolo Bonzini wrote:
> > > > On 28/06/2017 1
On Thu, Jun 29, 2017 at 05:05:46PM +0200, Igor Mammedov wrote:
> On Wed, 21 Jun 2017 19:24:15 +0300
> Roman Kagan wrote:
> > +static void synic_realize(DeviceState *dev, Error **errp)
> > +{
> > +Object *obj = OBJECT(dev);
> > +SynICState *synic = SYNIC(dev);
> > +
> > +synic->cpu = X8
On Thu, Jun 29, 2017 at 05:20:30PM +0200, Igor Mammedov wrote:
> On Wed, 21 Jun 2017 19:24:01 +0300
> Roman Kagan wrote:
>
> > This series applies miscellaneous fixes and enhancements to Hyper-V
> > emulation code in QEMU, and lays out the ground for VMBus devices.
>
> Series is a big random mix
This patch adds a description of the current TPM support in QEMU
to the specs.
Several public specs are referenced via their landing page on the
trustedcomputinggroup.org website.
Signed-off-by: Stefan Berger
---
docs/specs/tpm.txt | 98 ++
1
On Thu, Jun 29, 2017 at 02:47:34PM -0300, Eduardo Habkost wrote:
> On Thu, Jun 29, 2017 at 06:38:50PM +0100, Daniel P. Berrange wrote:
> > On Thu, Jun 29, 2017 at 02:09:39PM -0300, Eduardo Habkost wrote:
> > > On Thu, Jun 29, 2017 at 03:18:05PM +0100, Daniel P. Berrange wrote:
> > > > On Thu, Jun 2
On 06/29/2017 01:00 PM, Stefan Berger wrote:
> This patch adds a description of the current TPM support in QEMU
> to the specs.
>
> Several public specs are referenced via their landing page on the
> trustedcomputinggroup.org website.
>
> Signed-off-by: Stefan Berger
> ---
> docs/specs/tpm.txt
Am 29.06.2017 um 03:02 schrieb Philippe Mathieu-Daudé:
> There have been some comments on the ML about the usefulness of tci.
>
> https://lists.nongnu.org/archive/html/qemu-devel/2017-06/msg04551.html
>
> Peter Maydell> I'd prefer we just got rid of it.
>
> https://lists.nongnu.org/archive/htm
On 06/29/2017 02:07 PM, Eric Blake wrote:
On 06/29/2017 01:00 PM, Stefan Berger wrote:
This patch adds a description of the current TPM support in QEMU
to the specs.
Several public specs are referenced via their landing page on the
trustedcomputinggroup.org website.
Signed-off-by: Stefan Berge
I got this "Disassembler disagrees with translator over instruction
decoding" message asking to get reported here.
What happens here is coreboot incorrectly emits a Pentium2 instruction
while I'm running qemu with a Pentium cpu. I didn't know what to expect
but got this error message, then qem
Quoting Daniel P. Berrange (2017-06-29 03:33:19)
> On Wed, Jun 28, 2017 at 07:24:55PM -0500, Michael Roth wrote:
> > Hi everyone. Hoping to get some feedback on this approach, or some
> > alternatives proposed below, to the following issue:
> >
> > Currently libvirt immediately attempts to rebind
On 29 June 2017 at 19:20, Philippe Mathieu-Daudé wrote:
> I got this "Disassembler disagrees with translator over instruction
> decoding" message asking to get reported here.
>
> What happens here is coreboot incorrectly emits a Pentium2 instruction while
> I'm running qemu with a Pentium cpu. I d
This series makes implementing some of the bdrv_* callbacks easier for block
drivers by passing requests to bs->file if bs->drv doesn't implement it instead
of failing, and adding default bdrv_co_get_block_status() implementations.
This is based against Kevin Wolf's block branch, commit
0dd2ec6bc4
Now that passing the call to bs->file is the default for some bdrv_*
callbacks, remove the duplicate implementations in block/raw-format.c
Signed-off-by: Manos Pitsidianakis
---
block/raw-format.c | 32 +---
1 file changed, 1 insertion(+), 31 deletions(-)
diff --git
The following functions fail if bs->drv does not implement them:
bdrv_probe_blocksizes
bdrv_probe_geometry
bdrv_truncate
bdrv_has_zero_init
bdrv_get_info
bdrv_media_changed
bdrv_eject
bdrv_lock_medium
bdrv_co_ioctl
Instead, the call should be passed to bs->file if it exists, to allow
filter drive
On 06/21/2017 10:34 AM, Vladimir Sementsov-Ogievskiy wrote:
> Separate case when client sent NBD_OPT_ABORT from other errors.
Commit messages are best written in imperative tense (you can supply an
implicit "apply this patch in order to" prefix in front of the message,
and it should still generall
bdrv_co_get_block_status_from_file() and
bdrv_co_get_block_status_from_backing() set *file to bs->file and
bs->backing respectively, so that bdrv_co_get_block_status() can recurse
to them. Future block drivers won't have to duplicate code to implement
this.
Reviewed-by: Stefan Hajnoczi
Signed-off
On 06/28/2017 08:24 PM, Michael Roth wrote:
> Hi everyone. Hoping to get some feedback on this approach, or some
> alternatives proposed below, to the following issue:
>
> Currently libvirt immediately attempts to rebind a managed device back to the
> host driver when it receives a DEVICE_DELETED
On 06/21/2017 08:30 AM, Max Reitz wrote:
> A user may specify a relative path for accessing qemu, qemu-img, etc.
> through environment variables ($QEMU_PROG and friends) or a symlink.
>
> If a test decides to change its working directory, relative paths will
> cease to work, however. Work around t
* Dr. David Alan Gilbert (git) (dgilb...@redhat.com) wrote:
> From: "Dr. David Alan Gilbert"
>
> Hi,
> This is a RFC/WIP series that enables postcopy migration
> with shared memory to a vhost-user process.
> It's based off current-head + Juan's load_cleanup series, and
> Alexey's bitmap series
Signed-off-by: Laurent Vivier
---
fpu/softfloat-specialize.h | 10 ++
fpu/softfloat.c| 38 ++
include/fpu/softfloat.h| 8 +++-
3 files changed, 43 insertions(+), 13 deletions(-)
diff --git a/fpu/softfloat-specialize.h b/fpu/softflo
On 06/14/2017 08:51 AM, Halil Pasic wrote:
[apologies for the delayed response, and also adding Markus]
>>
>> One reason I choose error_report_err is to be consistent about hint
>> reporting (the other one is that was what Connie suggested). I do
>> not understand why do we omit hints if QMP, bu
201 - 300 of 412 matches
Mail list logo
