Signed-off-by: Kevin Wolf
Reviewed-by: Eric Blake
---
block/block-backend.c | 3 +--
include/sysemu/block-backend.h | 1 +
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/block/block-backend.c b/block/block-backend.c
index
On Thu, Jul 13, 2017 at 06:23:14AM -0500, Eric Blake wrote:
On 07/13/2017 05:01 AM, Manos Pitsidianakis wrote:
The following functions fail if bs->drv is a filter and does not
implement them:
bdrv_probe_blocksizes
bdrv_probe_geometry
bdrv_truncate
bdrv_has_zero_init
bdrv_get_info
Instead, the
On 07/07/2017 22:30, Eric Blake wrote:
> The NBD protocol has now finalized the extension that fixes
> several shortcomings with NBD_OPT_EXPORT_NAME (namely, no error
> reporting, no way for the server to advertise block sizes to the
> client):
>
On 07/13/2017 05:03 AM, Philippe Mathieu-Daudé wrote:
> useful to avoid Travis CI jobs getting killed after 10min of inactivity.
>
> Signed-off-by: Philippe Mathieu-Daudé
> ---
> MAINTAINERS | 1 +
> scripts/aliveness.sh | 32
> 2
On 12/07/2017 17:41, Peng Hao wrote:
> Parallel device don't register be->chr_can_read function, but remote
> disconnect event is handled in chr_read.So connected parallel device
> can not detect remote disconnect event. The chardevs with chr_can_read=NULL
> has the same problem.
>
>
On 12/07/2017 07:52, Thomas Huth wrote:
> On 11.07.2017 20:55, Philippe Mathieu-Daudé wrote:
>> Signed-off-by: Philippe Mathieu-Daudé
>> ---
>> same as Thomas previous patch. this file had no entry in MAINTAINERS.
>>
>> atomic_template.h => accel/tcg/atomic_template.h | 0
>> 1
On 07/13/17 12:47, Peter Maydell wrote:
> On 12 July 2017 at 00:43, Ben Warren wrote:
>> Yes, it’s definitely a setup time problem. With the values that are checked
>> in, I can’t get it to fail on my setup, but if I wind the numbers down I see
>> the same failure as
Am 12.07.2017 um 17:47 hat Eric Blake geschrieben:
> On 07/12/2017 07:57 AM, Kevin Wolf wrote:
> > Instead of listing only monitor-owned BlockBackends in query-block, also
> > add those anonymous BlockBackends that are owned by a qdev device and as
> > such under the control of the user.
> >
> >
On 07/13/2017 05:01 AM, Manos Pitsidianakis wrote:
> The following functions fail if bs->drv is a filter and does not
> implement them:
>
> bdrv_probe_blocksizes
> bdrv_probe_geometry
> bdrv_truncate
> bdrv_has_zero_init
> bdrv_get_info
>
> Instead, the call should be passed to bs->file if it
Eric Blake writes:
> On 07/13/2017 02:08 AM, Alistair Francis wrote:
>> On Wed, Jul 12, 2017 at 6:49 PM, Markus Armbruster wrote:
>>> Alistair Francis writes:
>>>
QEMU currently has a standard method to report errors with
On 07/13/2017 12:32 PM, Nikunj A Dadhania wrote:
> Cédric Le Goater writes:
>
>> On 07/13/2017 11:10 AM, Nikunj A Dadhania wrote:
>>> Cédric Le Goater writes:
>>>
On 07/13/2017 09:55 AM, Nikunj A Dadhania wrote:
> Cédric Le Goater writes:
On 07/13/2017 05:01 AM, Manos Pitsidianakis wrote:
> This series makes implementing some of the bdrv_* callbacks easier for block
> filters 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.
>
>
On 02/15/2017 11:05 AM, Paolo Bonzini wrote:
[now several months later...]
>
>
> On 03/02/2017 16:47, Vladimir Sementsov-Ogievskiy wrote:
>> Hi all!
>>
>> We really need exporting dirty bitmaps feature as well as remote
>> get_block_status for nbd devices. So, here is minimalistic and
13.07.2017 14:03, Vladimir Sementsov-Ogievskiy wrote:
13.07.2017 04:05, Eric Blake wrote:
Now that we have adjusted the majority of the calls this function
makes to be byte-based, it is easier to read the code if it makes
passes over the image using bytes rather than sectors.
Signed-off-by:
13.07.2017 04:05, Eric Blake wrote:
Now that we have adjusted the majority of the calls this function
makes to be byte-based, it is easier to read the code if it makes
passes over the image using bytes rather than sectors.
Signed-off-by: Eric Blake
Reviewed-by: John Snow
On 13/07/2017 12:23, Daniel P. Berrange wrote:
> On Thu, Jul 13, 2017 at 03:15:49AM -0700, Alistair Francis wrote:
>> diff --git a/io/channel-watch.c b/io/channel-watch.c
>> index 8640d1c464..d80722f496 100644
>> --- a/io/channel-watch.c
>> +++ b/io/channel-watch.c
>> @@ -286,9 +286,21 @@ GSource
13.07.2017 04:05, Eric Blake wrote:
Now that we have adjusted the majority of the calls this function
makes to be byte-based, it is easier to read the code if it makes
passes over the image using bytes rather than sectors.
Signed-off-by: Eric Blake
Reviewed-by: John Snow
virtqueue_error includes queue index in the error output and is preferred
for errors that pertain to a virtqueue rather than to the device as a whole.
Signed-off-by: Ladi Prosek
---
hw/virtio/virtio-crypto.c | 56 ---
1 file
virtqueue_error includes queue index in the error output and is preferred
for errors that pertain to a virtqueue rather than to the device as a whole.
Signed-off-by: Ladi Prosek
---
hw/9pfs/virtio-9p-device.c | 37 +++--
1 file changed, 15
virtqueue_error includes queue index in the error output and is preferred
for errors that pertain to a virtqueue rather than to the device as a whole.
Signed-off-by: Ladi Prosek
---
hw/block/virtio-blk.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git
virtqueue_error includes queue index in the error output and is preferred
for errors that pertain to a virtqueue rather than to the device as a whole.
Signed-off-by: Ladi Prosek
---
hw/net/virtio-net.c | 24
1 file changed, 12 insertions(+), 12
virtqueue_error includes queue index in the error output and is preferred
for errors that pertain to a virtqueue rather than to the device as a whole.
Signed-off-by: Ladi Prosek
---
hw/scsi/virtio-scsi.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git
Output like "Virtqueue size exceeded" is not much useful in identifying the
culprit. This series beefs up virtio_error to print the virtio device name
and id, and introduces virtqueue_error which additionally includes the index
of the virtqueue where the error occured.
Patches 1 to 3 lay the
Most virtio error output pertains to a specific virtqueue so it makes
sense to include the queue index in error messages. This commit makes
all error output in virtio.c use the newly introduced virtqueue_error.
Suggested-by: Stefan Hajnoczi
Signed-off-by: Ladi Prosek
Callers of error_report may want to compose the error message out of multiple
separate format strings. To save them from always having to combine their
strings into one (a process that would likely involve memory allocation which
is good to avoid on error paths), new "nolf" variants of
Output like "Virtqueue size exceeded" is not much useful in identifying the
culprit. This commit adds virtio device name (e.g. "virtio-input") and id
if set (e.g. "mouse0") to all virtio error messages to improve debuggability.
Some virtio devices (virtio-scsi, virtio-serial) insert a bus between
On 07/13/2017 04:16 AM, Vladimir Sementsov-Ogievskiy wrote:
> 03.07.2017 18:10, Eric Blake wrote:
>> Now that we have adjusted the majority of the calls this function
>> makes to be byte-based, it is easier to read the code if it makes
>> passes over the image using bytes rather than sectors.
>>
On 07/13/2017 12:40 PM, Christian Borntraeger wrote:
> here are the remaining PCI patches also enabling AIS via KVM.
> CPU model wise we do not put AIS in a default model for zEC12 and z13
> since almost all distribution kernels do not have ais yet. This would
> break -cpu zEC12 and -cpu z13 if
On Thu, Jul 13, 2017 at 11:09:08AM +0200, John Paul Adrian Glaubitz wrote:
> Just as a heads-up: The current set of patches as they are present in
> the tgt-sh4 branch on git://github.com/rth7680/qemu.git are a HUGE
> improvement for qemu-sh4 user mode. Several packages which previously
> crashed
On 12 July 2017 at 00:43, Ben Warren wrote:
> Yes, it’s definitely a setup time problem. With the values that are checked
> in, I can’t get it to fail on my setup, but if I wind the numbers down I see
> the same failure as Peter. So now we have the ages-old problem of
here are the remaining PCI patches also enabling AIS via KVM.
CPU model wise we do not put AIS in a default model for zEC12 and z13
since almost all distribution kernels do not have ais yet. This would
break -cpu zEC12 and -cpu z13 if QEMU is updated.
patches are against previous series.
Yi Min
From: Yi Min Zhao
During migration we should transfer ais states to the target guest.
This patch introduces a subsection to kvm_s390_flic_vmstate and new
vmsd for qemu_flic. The ais states need to be migrated only when
ais is supported.
Signed-off-by: Yi Min Zhao
From: Yi Min Zhao
By initializing the CPU firstly, we are able to retrieve and use the
CPU model features when initializing other subsystem or devices.
Signed-off-by: Yi Min Zhao
Reviewed-by: Pierre Morel
From: Yi Min Zhao
zPCI instructions and facilities are available since IBM zEnterprise
EC12. To support z/PCI in QEMU we enable zpci, aen and ais facilities
starting with zEC12 GA1. And we always set zpci and aen bits in max cpu
model. Later they might be switched off
On 13 July 2017 at 08:27, Alistair Francis wrote:
> On Tue, Jul 11, 2017 at 6:35 PM, Peter Maydell
> wrote:
>> On 11 July 2017 at 15:33, Alistair Francis wrote:
>>> On Tue, Jul 11, 2017 at 1:17 PM, Peter Maydell
On 07/13/2017 02:44 AM, Michal Privoznik wrote:
In the subject: s/virtion/virtio/
> We have a function that checks if given number is power of two.
> We should prefer it instead of expanding the check on our own.
>
> Signed-off-by: Michal Privoznik
> ---
>
On 07/12/17 04:08, Dongjiu Geng wrote:
> (1) Add related APEI/HEST table structures and macros, these
> definition refer to ACPI 6.1 and UEFI 2.6 spec.
> (2) Add generic error status block and CPER memory section
> definition, user space only handle memory section errors.
>
>
On 07/13/2017 02:08 AM, Alistair Francis wrote:
> On Wed, Jul 12, 2017 at 6:49 PM, Markus Armbruster wrote:
>> Alistair Francis writes:
>>
>>> QEMU currently has a standard method to report errors with
>>> error_repot(). This ensure a sane and
Cédric Le Goater writes:
> On 07/13/2017 11:10 AM, Nikunj A Dadhania wrote:
>> Cédric Le Goater writes:
>>
>>> On 07/13/2017 09:55 AM, Nikunj A Dadhania wrote:
Cédric Le Goater writes:
>>> How about the following, we do not report
On Thu, Jul 13, 2017 at 03:15:49AM -0700, Alistair Francis wrote:
> When calling WAEventSelect() only wait on events as specified by the
> condition variable. This requires that the condition variable is set
> correctly for the specific events that we need to wait for.
Can you describe the actual
Hi Hervé,
On 06/21/2017 01:34 AM, Hervé Poussineau wrote:
Signed-off-by: Hervé Poussineau
---
hw/audio/adlib.c | 2 +-
hw/audio/fmopl.c | 18 +-
hw/audio/fmopl.h | 7 ---
3 files changed, 18 insertions(+), 9 deletions(-)
diff --git
On 06/21/2017 01:33 AM, Hervé Poussineau wrote:
Hi,
This patchset removes a global variable in adlib emulation, and allows to
have multiple adlib sound cards.
Note that indentation seem off because I have replaced some tabulations by
spaces.
Before:
qemu-system-i386 -device adlib,iobase=0x220
When calling WAEventSelect() only wait on events as specified by the
condition variable. This requires that the condition variable is set
correctly for the specific events that we need to wait for.
Signed-off-by: Alistair Francis
---
io/channel-watch.c | 18
On 07/13/2017 11:10 AM, Nikunj A Dadhania wrote:
> Cédric Le Goater writes:
>
>> On 07/13/2017 09:55 AM, Nikunj A Dadhania wrote:
>>> Cédric Le Goater writes:
>>>
>> How about the following, we do not report work until MSR_EE is disabled:
>
> With this
On 07/12/2017 09:57 PM, David Gibson wrote:
On Wed, Jul 12, 2017 at 10:48:38AM -0300, Daniel Henrique Barboza wrote:
The dreaded Libvirt hotplug-migrate-hotunplug scenario is working nicely.
Good to hear.
device_add when the machine is in RUN_STATE_PRELAUNCH (-S) still doesn't
work but it
useful to avoid Travis CI jobs getting killed after 10min of inactivity.
Signed-off-by: Philippe Mathieu-Daudé
---
MAINTAINERS | 1 +
scripts/aliveness.sh | 32
2 files changed, 33 insertions(+)
create mode 100755 scripts/aliveness.sh
Travis CI has specific time limits for each job, and will stop the build and
add an error message to the build log in the following situation[s]:
- A job produces no log output for 10 minutes
See: https://docs.travis-ci.com/user/customizing-the-build#build-timeouts
Signed-off-by: Philippe
On Thu, Jul 13, 2017 at 05:55:00PM +0800, long mike wrote:
> 2017-07-11 20:29 GMT+08:00 Daniel P. Berrange :
> > On Tue, Jul 04, 2017 at 04:57:05PM +0800, Longpeng(Mike) wrote:
> >> Adds afalg-backend cipher support: introduces some private APIs
> >> firstly, and then
Hi, still trying to reduce Travis false negatives...
Suggested yesterday on IRC:
pm215: this one seems to be that we hit the "10 minutes with no output" limit
stefanha: f4bug: We could literally have a SIGALRM that wakes up to print out
a '.' :-)
Let's try with an easy shell script.
This
Now that bdrv_truncate is passed to bs->file by default, remove the
callback from block/blkdebug.c and set is_filter to true.
Reviewed-by: Eric Blake
Reviewed-by: Stefan Hajnoczi
Signed-off-by: Manos Pitsidianakis
---
This function is not used anywhere, so remove it.
Markus Armbruster adds:
The i82078 floppy device model used to call bdrv_media_changed() to
implement its media change bit when backed by a host floppy. This
went away in 21fcf36 "fdc: simplify media change handling".
Probably broke host floppy
The following functions fail if bs->drv is a filter and does not
implement them:
bdrv_probe_blocksizes
bdrv_probe_geometry
bdrv_truncate
bdrv_has_zero_init
bdrv_get_info
Instead, the call should be passed to bs->file if it exists, to allow
filter drivers to support those methods without
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: Fam Zheng
This series makes implementing some of the bdrv_* callbacks easier for block
filters 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
2017-07-11 20:29 GMT+08:00 Daniel P. Berrange :
> On Tue, Jul 04, 2017 at 04:57:05PM +0800, Longpeng(Mike) wrote:
>> Adds afalg-backend cipher support: introduces some private APIs
>> firstly, and then intergrates them into qcrypto_cipher_afalg_driver.
>>
>> Signed-off-by:
On 11 July 2017 at 09:35, Stefan Hajnoczi wrote:
> The following changes since commit 6b06e3e49eb8c91cc286c16d6bf3181ac296f33d:
>
> Merge remote-tracking branch 'remotes/ericb/tags/pull-nbd-2017-07-10-v2'
> into staging (2017-07-10 16:12:47 +0100)
>
> are available in the
Nikunj A Dadhania writes:
> Cédric Le Goater writes:
>
>> On 07/13/2017 09:55 AM, Nikunj A Dadhania wrote:
>>> Cédric Le Goater writes:
>>>
>> How about the following, we do not report work until MSR_EE is disabled:
>
> With
On Wed, Jul 12, 2017 at 06:55:45PM -0300, Philippe Mathieu-Daudé wrote:
> Hi Daniel,
>
> On 07/12/2017 01:25 PM, Daniel P. Berrange wrote:
> > When trying to debug problems with tests it is natural to set
> > DEBUG=1 when starting the docker environment. Unfortunately
> > this has a side-effect
On Thu, Jul 13, 2017 at 10:44:53AM +0200, Peter Lieven wrote:
> Am 10.07.2017 um 15:55 schrieb Daniel P. Berrange:
> > On Mon, Jul 10, 2017 at 03:52:18PM +0200, Kevin Wolf wrote:
> > > Am 10.07.2017 um 15:44 hat Daniel P. Berrange geschrieben:
> > > > On Mon, Jul 10, 2017 at 03:34:59PM +0200,
On Thu, Jul 13, 2017 at 10:52:41AM +0200, Kevin Wolf wrote:
> Am 13.07.2017 um 10:45 hat Peter Lieven geschrieben:
> > Am 10.07.2017 um 15:30 schrieb Kevin Wolf:
> > >Am 10.07.2017 um 15:24 hat Peter Lieven geschrieben:
> > >>Am 10.07.2017 um 15:10 schrieb Kevin Wolf:
> > >>>Am 29.06.2017 um 12:57
03.07.2017 18:10, Eric Blake wrote:
Now that we have adjusted the majority of the calls this function
makes to be byte-based, it is easier to read the code if it makes
passes over the image using bytes rather than sectors.
Signed-off-by: Eric Blake
---
v4: new patch
---
Cédric Le Goater writes:
> On 07/13/2017 09:55 AM, Nikunj A Dadhania wrote:
>> Cédric Le Goater writes:
>>
> How about the following, we do not report work until MSR_EE is disabled:
With this fix, I could test the XIVE<->XICS transitions at reboot
Just as a heads-up: The current set of patches as they are present in
the tgt-sh4 branch on git://github.com/rth7680/qemu.git are a HUGE
improvement for qemu-sh4 user mode. Several packages which previously
crashed during build or use can now be used properly!
So, I hope to see this series merged
On Fri, 7 Jul 2017 15:53:37 -0500
Eric Blake wrote:
> On 06/23/2017 08:02 AM, Tomáš Golembiovský wrote:
> > Events can play an integral role when monitoring internal state of the
> > guest OS. This patch adds the core functionality for adding events to
> > QEMU Guest Agent.
Am 13.07.2017 um 10:45 hat Peter Lieven geschrieben:
> Am 10.07.2017 um 15:30 schrieb Kevin Wolf:
> >Am 10.07.2017 um 15:24 hat Peter Lieven geschrieben:
> >>Am 10.07.2017 um 15:10 schrieb Kevin Wolf:
> >>>Am 29.06.2017 um 12:57 hat Peter Lieven geschrieben:
> Signed-off-by: Peter Lieven
On Fri, 7 Jul 2017 15:55:31 -0500
Eric Blake wrote:
> On 06/23/2017 08:02 AM, Tomáš Golembiovský wrote:
> > This is just a draft, or a request for comments if you will.
> >
> > This patch sets drafts the support of sending events by QEMU Guest Agent.
> > Events can plan
On 2017年07月12日 16:13, Peter Xu wrote:
It is not wise to disgard all the IOTLB cache when cache size reaches
max, but that's what we do now. A slightly better (but still simple) way
to do this is, we just throw away the least recent used cache entry.
This patch implemented MRU list algorithm
Fam Zheng writes:
> On Wed, 07/12 17:25, Daniel P. Berrange wrote:
>> When trying to debug problems with tests it is natural to set
>> DEBUG=1 when starting the docker environment. Unfortunately
>> this has a side-effect of enabling an eth0 network interface
>> in the
Am 10.07.2017 um 15:30 schrieb Kevin Wolf:
Am 10.07.2017 um 15:24 hat Peter Lieven geschrieben:
Am 10.07.2017 um 15:10 schrieb Kevin Wolf:
Am 29.06.2017 um 12:57 hat Peter Lieven geschrieben:
Signed-off-by: Peter Lieven
---
qapi/block-core.json | 44
On Wed, 12 Jul 2017 14:57:45 +0200
Christian Borntraeger wrote:
> From: Yi Min Zhao
>
> Let's use the new inject_airq callback of flic to inject adapter
> interrupts. For kvm case, if the kernel flic doesn't support the new
> interface, the
Am 10.07.2017 um 15:55 schrieb Daniel P. Berrange:
On Mon, Jul 10, 2017 at 03:52:18PM +0200, Kevin Wolf wrote:
Am 10.07.2017 um 15:44 hat Daniel P. Berrange geschrieben:
On Mon, Jul 10, 2017 at 03:34:59PM +0200, Kevin Wolf wrote:
Am 10.07.2017 um 15:29 hat Peter Lieven geschrieben:
Am
On 07/13/2017 08:22 AM, Michael S. Tsirkin wrote:
On Wed, Jul 12, 2017 at 08:40:21PM +0800, Wei Wang wrote:
Add a new vq, cmdq, to handle requests between the device and driver.
This patch implements two commands sent from the device and handled in
the driver.
1)
On 2017年07月12日 16:13, Peter Xu wrote:
IOMMUTLBEntry.iova is returned incorrectly in one PT path (though mostly
we cannot really trigger this path, even if we do, we are mostly
disgarding this value, so it didn't break anything). Fix it by
converting the VTD_PAGE_MASK into normal definition
Am 12.07.2017 um 18:58 hat Max Reitz geschrieben:
> On 2017-07-12 16:52, Kevin Wolf wrote:
> > Am 12.07.2017 um 13:46 hat Pavel Butsykin geschrieben:
> >> This patch add shrinking of the image file for qcow2. As a result, this
> >> allows
> >> us to reduce the virtual image size and free up space
On 07/13/2017 08:16 AM, Michael S. Tsirkin wrote:
On Wed, Jul 12, 2017 at 08:40:20PM +0800, Wei Wang wrote:
This patch enables for_each_zone()/for_each_populated_zone() to be
invoked by a kernel module.
... for use by virtio balloon.
With this patch, other kernel modules can also use the
On 07/13/2017 08:33 AM, Michael S. Tsirkin wrote:
On Wed, Jul 12, 2017 at 08:40:19PM +0800, Wei Wang wrote:
This patch adds support for reporting blocks of pages on the free list
specified by the caller.
As pages can leave the free list during this call or immediately
afterwards, they are not
On 07/13/2017 09:55 AM, Nikunj A Dadhania wrote:
> Cédric Le Goater writes:
>
How about the following, we do not report work until MSR_EE is disabled:
>>>
>>> With this fix, I could test the XIVE<->XICS transitions at reboot
>>> under TCG. However, the second boot is very
On 13.07.2017 10:08, Christian Borntraeger wrote:
> On 07/12/2017 04:26 PM, Thomas Huth wrote:
>> On 12.07.2017 14:57, Christian Borntraeger wrote:
>>> From: Yi Min Zhao
>>>
>>> Let's use the new inject_airq callback of flic to inject adapter
>>> interrupts. For kvm
On 13/07/2017 04:53, peng.h...@zte.com.cn wrote:
> > I think what you're seeing is a race like this:
> >
> > VCPU 0 VCPU 1
> > [qemu] kvm_get_mp_state
> > [kvm] kvm_apic_accept_events
> > __apic_accept_irq
> >
On 07/12/2017 04:26 PM, Thomas Huth wrote:
> On 12.07.2017 14:57, Christian Borntraeger wrote:
>> From: Yi Min Zhao
>>
>> Let's use the new inject_airq callback of flic to inject adapter
>> interrupts. For kvm case, if the kernel flic doesn't support the new
>>
On 07/10/2017 08:40 PM, Stefan Hajnoczi wrote:
On Mon, Jul 10, 2017 at 01:24:23PM +0800, Xie Changlong wrote:
在 7/9/2017 5:57 PM, Wang Dong 写道:
Hi,
I am new to QEMU. But I got some problem so that I want to figure it out.
So I try to debug qemu to see what happened.
And I found trace
On 07/04/2017 03:44 AM, Wei Wang wrote:
> On 07/04/2017 03:44 AM, Michael S. Tsirkin wrote:
>> From: Wei Wang
>>
>> This patch enables the virtio-net tx queue size to be configurable
>> between 256 (the default queue size) and 1024 by the user when the
>> vhost-user backend
Cédric Le Goater writes:
>>> How about the following, we do not report work until MSR_EE is disabled:
>>
>> With this fix, I could test the XIVE<->XICS transitions at reboot
>> under TCG. However, the second boot is very slow for some reason.
>
> hmm, I am not sure this is
Cédric Le Goater writes:
> On 07/13/2017 06:38 AM, Nikunj A Dadhania wrote:
>> David Gibson writes:
>>
>>>
>>> Ok, but we definitely should be able to fix this without new
>>> variables. If we can quiesce the secondary CPUs for the first boot,
>>>
On Thu, 13 Jul 2017 09:35:16 +0200
Christian Borntraeger wrote:
> On 07/13/2017 09:27 AM, Cornelia Huck wrote:
> > On Thu, 13 Jul 2017 09:12:50 +0200
> > Christian Borntraeger wrote:
> >
> >> On 07/12/2017 05:07 PM, Cornelia Huck wrote:
>
On 07/13/2017 09:35 AM, Thomas Huth wrote:
> On 13.07.2017 09:11, Christian Borntraeger wrote:
>> Thanks for the review, all valid. Claudio has provided me the following
>> fixup.
>>
>> I plan to fold that in the base patch (retest pending).
>>
>> Christian
>>
>>
>> ---
>>
We have a function that checks if given number is power of two.
We should prefer it instead of expanding the check on our own.
Signed-off-by: Michal Privoznik
---
hw/net/virtio-net.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/hw/net/virtio-net.c
On 07/12/2017 09:56 PM, Michael S. Tsirkin wrote:
So the way I see it, there are several issues:
- internal wait - forces multiple APIs like kick/kick_sync
note how kick_sync can fail but your code never checks return code
- need to re-write the last descriptor - might not work
for
On Tue, Jul 11, 2017 at 1:17 PM, Peter Maydell wrote:
> Add the UARTs to the MPS2 board models.
>
> Unfortunately the details of the wiring of the interrupts through
> various OR gates differ between AN511 and AN385 so this can't
> be purely a data-driven difference.
>
>
On 13.07.2017 09:11, Christian Borntraeger wrote:
> Thanks for the review, all valid. Claudio has provided me the following fixup.
>
> I plan to fold that in the base patch (retest pending).
>
> Christian
>
>
> ---
> hw/s390x/s390-stattrib-kvm.c | 26 +++---
>
On 07/13/2017 09:27 AM, Cornelia Huck wrote:
> On Thu, 13 Jul 2017 09:12:50 +0200
> Christian Borntraeger wrote:
>
>> On 07/12/2017 05:07 PM, Cornelia Huck wrote:
+
+switch (mode) {
+case SIC_IRQ_MODE_ALL:
+flic->simm &=
On Tue, Jul 11, 2017 at 6:45 PM, Peter Maydell wrote:
> On 11 July 2017 at 16:40, Alistair Francis wrote:
>> On Tue, Jul 11, 2017 at 5:33 PM, Peter Maydell
>> wrote:
>>> On 11 July 2017 at 16:12, Alistair Francis
On Thu, 13 Jul 2017 09:11:34 +0200
Christian Borntraeger wrote:
> Thanks for the review, all valid. Claudio has provided me the following fixup.
>
> I plan to fold that in the base patch (retest pending).
Looks good to me, and I have no further comments beyond those by
On Thu, 13 Jul 2017 09:12:50 +0200
Christian Borntraeger wrote:
> On 07/12/2017 05:07 PM, Cornelia Huck wrote:
> >> +
> >> +switch (mode) {
> >> +case SIC_IRQ_MODE_ALL:
> >> +flic->simm &= ~AIS_MODE_MASK(isc);
> >> +flic->nimm &=
On Tue, Jul 11, 2017 at 6:35 PM, Peter Maydell wrote:
> On 11 July 2017 at 15:33, Alistair Francis wrote:
>> On Tue, Jul 11, 2017 at 1:17 PM, Peter Maydell
>> wrote:
>>> Model the ARM MPS2/MPS2+ FPGA based development
On 07/12/2017 05:07 PM, Cornelia Huck wrote:
>> +
>> +switch (mode) {
>> +case SIC_IRQ_MODE_ALL:
>> +flic->simm &= ~AIS_MODE_MASK(isc);
>> +flic->nimm &= ~AIS_MODE_MASK(isc);
>> +break;
>> +case SIC_IRQ_MODE_SINGLE:
>> +flic->simm |= AIS_MODE_MASK(isc);
Thanks for the review, all valid. Claudio has provided me the following fixup.
I plan to fold that in the base patch (retest pending).
Christian
---
hw/s390x/s390-stattrib-kvm.c | 26 +++---
hw/s390x/s390-stattrib.c | 12 +++-
On Thu, Jul 13, 2017 at 8:24 AM, Markus Armbruster wrote:
> Eduardo Habkost writes:
>
>> On Wed, Jul 12, 2017 at 11:39:59AM +0200, Markus Armbruster wrote:
>>> Alistair Francis writes:
>>>
>>> > Signed-off-by: Alistair Francis
On Wed, Jul 12, 2017 at 6:49 PM, Markus Armbruster wrote:
> Alistair Francis writes:
>
>> QEMU currently has a standard method to report errors with
>> error_repot(). This ensure a sane and standard format when printing
>> errors. This series is
On 07/13/2017 08:43 AM, Cédric Le Goater wrote:
> On 07/13/2017 06:38 AM, Nikunj A Dadhania wrote:
>> David Gibson writes:
>>
>>> On Fri, Jun 09, 2017 at 10:32:25AM +0530, Nikunj A Dadhania wrote:
David Gibson writes:
> On
401 - 500 of 509 matches
Mail list logo