Hi
- Original Message -
> On Fri, Aug 12, 2016 at 11:54:54AM -0400, Marc-André Lureau wrote:
> > Hi
> >
> > - Original Message -
> > > On Fri, Aug 12, 2016 at 03:20:56AM -0400, Marc-André Lureau wrote:
> > > > Hi
> > > >
> > > > - Original Message -
> > > > > sent a follo
On 08/01/2016 08:32 AM, Peter Maydell wrote:
> On 1 August 2016 at 14:26, Andrea Bolognani wrote:
>> On Mon, 2016-08-01 at 15:08 +0200, Andrew Jones wrote:
I'm not sure a warning is enough: if I start a guest and
explicitly ask for a PMU, I expect it to be there, or for
the guest
This patch adds a pmu=[on/off] option to enable/disable host vPMU
support in guest VM. There are several reasons to justify this option.
First, host-backed vPMU can be problematic for cross-migration between
different SoC as perf counters are architecture-dependent. It is more
flexible to have an o
Alright! I'll test that new patch and submit it.
Thanks,
- Andrew
On Fri, Aug 12, 2016 at 4:01 AM, Peter Maydell wrote:
> On 10 August 2016 at 04:35, Andrew Dutcher wrote:
>> Hello!
>>
>> I ran into an issue where qemu (specifically, the unicorn engine)
>> would hang forever in the middle of th
Hi Michael,
On 2016/8/10 4:04, Michael Roth wrote:
> Quoting Michael Roth (2016-08-08 16:03:31)
>> > Hi everyone,
>> >
>> > The following new patches are queued for QEMU stable v2.6.1:
>> >
>> > https://github.com/mdroth/qemu/commits/stable-2.6-staging
>> >
>> > The release is planned for 201
On 8/13/2016 2:55 AM, Alex Williamson wrote:
> On Fri, 12 Aug 2016 23:27:01 +0530
> Kirti Wankhede wrote:
>
>> On 8/12/2016 12:13 AM, Alex Williamson wrote:
>>
>>>
>>> TBH, I don't see how providing a default implementation of
>>> validate_map_request() is useful. How many mediated devices are
On 8/13/2016 2:46 AM, Alex Williamson wrote:
> On Sat, 13 Aug 2016 00:14:39 +0530
> Kirti Wankhede wrote:
>
>> On 8/10/2016 12:30 AM, Alex Williamson wrote:
>>> On Thu, 4 Aug 2016 00:33:51 +0530
>>> Kirti Wankhede wrote:
>>>
>>> This is used later by mdev_device_start() and mdev_device_stop()
From: Cao jin
emu_regs is a pointer, ARRAY_SIZE doesn't return what we expect.
Since the remaining message is enough for debugging, so just remove it.
Also tweaked the message a little.
Signed-off-by: Cao jin
Signed-off-by: Stefano Stabellini
Acked-by: Stefano Stabellini
---
hw/xen/xen_pt_co
From: Paul Durrant
VMs created on older versions on Xen will not have been provisioned with
pages to support creation of non-default ioreq servers. In this case
the ioreq server API is not supported and QEMU's only option is to fall
back to using the default ioreq server pages as it did prior to
gt; 'remotes/amit-migration/tags/migration-for-2.7-7' into staging (2016-08-11
> 17:53:35 +0100)
>
> are available in the git repository at:
>
>
> git://xenbits.xen.org/people/sstabellini/qemu-dm.git tags/xen-20160812-tag
>
> for you to fetch changes up to f
From: Paul Durrant
VMs created on older versions on Xen will not have been provisioned with
pages to support creation of non-default ioreq servers. In this case
the ioreq server API is not supported and QEMU's only option is to fall
back to using the default ioreq server pages as it did prior to
From: Cao jin
emu_regs is a pointer, ARRAY_SIZE doesn't return what we expect.
Since the remaining message is enough for debugging, so just remove it.
Also tweaked the message a little.
Signed-off-by: Cao jin
---
hw/xen/xen_pt_config_init.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletio
llini/qemu-dm.git tags/xen-20160812-tag
for you to fetch changes up to f7ca8a4fdcb478fdb0503c2d39b0b85160c913d9:
xen: handle inbound migration of VMs without ioreq server pages (2016-08-12
15:16:15 -0700)
Xen
They never touch the buffer, it should be const, this helps when
drivers want to pass const pointers.
This is purely signature changes, there are no actual code changes.
Signed-off-by: Benjamin Herrenschmidt
---
include/net/checksum.h | 10 +-
net/checksum.c | 6 +++---
2 files
On Fri, 12 Aug 2016, Paul Durrant wrote:
> > -Original Message-
> > From: Stefano Stabellini [mailto:sstabell...@kernel.org]
> > Sent: 11 August 2016 21:07
> > To: Paul Durrant
> > Cc: Anthony Perard; xen-de...@lists.xenproject.org; qemu-
> > de...@nongnu.org; Stefano Stabellini
> > Subject
Updates the help messages to remove misleading information about SDL
being the normal display used.
Signed-off-by: Colin Lord
---
v3: Try to make help more general to cover all cases nicely.
qemu-options.hx | 38 +-
1 file changed, 21 insertions(+), 17 deletio
On Fri, Aug 12, 2016 at 04:32:56PM +0100, Stefan Hajnoczi wrote:
> virtqueue_descard()
discard
> moves vq->last_avail_idx back so the element can be
> popped again. It's necessary to decrement vq->inuse to avoid "leaking"
> the element count.
>
> Signed-off-by: Stefan Hajnoczi
With the correc
On Fri, Aug 12, 2016 at 04:32:55PM +0100, Stefan Hajnoczi wrote:
> The vq->inuse field is not migrated. Many devices don't hold
> VirtQueueElements across migration so it doesn't matter that vq->inuse
> starts at 0 on the destination QEMU.
>
> At least virtio-serial, virtio-blk, and virtio-balloo
On Fri, 12 Aug 2016 23:27:01 +0530
Kirti Wankhede wrote:
> On 8/12/2016 12:13 AM, Alex Williamson wrote:
>
> >
> > TBH, I don't see how providing a default implementation of
> > validate_map_request() is useful. How many mediated devices are going
> > to want to identity map resources from the
On Sat, 13 Aug 2016 00:14:39 +0530
Kirti Wankhede wrote:
> On 8/10/2016 12:30 AM, Alex Williamson wrote:
> > On Thu, 4 Aug 2016 00:33:51 +0530
> > Kirti Wankhede wrote:
> >
> > This is used later by mdev_device_start() and mdev_device_stop() to get
> > the parent_device so it can call the start
On Fri, Aug 12, 2016 at 11:54:54AM -0400, Marc-André Lureau wrote:
> Hi
>
> - Original Message -
> > On Fri, Aug 12, 2016 at 03:20:56AM -0400, Marc-André Lureau wrote:
> > > Hi
> > >
> > > - Original Message -
> > > > sent a follow-up response to GET_FEATURES), I am now wondering
On Fri, Aug 12, 2016 at 11:08 PM, Valentine Sinitsyn <
valentine.sinit...@gmail.com> wrote:
> On 11.08.2016 00:42, David Kiarie wrote:
>
>> Introduce AMD IOMMU interrupt remapping and hook it onto
>
>
>> +static inline int amdvi_ir_pass(MSIMessage *src, MSIMessage *dst,
>> uint8_t bit,
>> +
On Fri, Aug 12, 2016 at 11:14 PM, Valentine Sinitsyn <
valentine.sinit...@gmail.com> wrote:
> On 11.08.2016 00:42, David Kiarie wrote:
>
>> Report IOAPIC via IVRS which effectively allows linux AMD-Vi
>> driver to enable interrupt remapping
>>
>> Signed-off-by: David Kiarie
>> ---
>> hw/i386/acp
On 11.08.2016 00:42, David Kiarie wrote:
Report IOAPIC via IVRS which effectively allows linux AMD-Vi
driver to enable interrupt remapping
Signed-off-by: David Kiarie
---
hw/i386/acpi-build.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/hw/i386/acpi-build.c b/hw/i386/acpi-build.c
inde
On 11.08.2016 00:42, David Kiarie wrote:
Introduce AMD IOMMU interrupt remapping and hook it onto
the existing interrupt remapping infrastructure
Signed-off-by: David Kiarie
---
hw/i386/amd_iommu.c | 226 +++-
hw/i386/amd_iommu.h | 2 +
2 files
On Fri, Aug 12, 2016 at 10:10 PM, Valentine Sinitsyn <
valentine.sinit...@gmail.com> wrote:
> Hi David,
>
> On 02.08.2016 13:39, David Kiarie wrote:
>
>> Add AMD IOMMU emulaton to Qemu in addition to Intel IOMMU.
>> The IOMMU does basic translation, error checking and has a
>> minimal IOTLB implem
Hi,
Your series failed automatic build test. Please find the testing commands and
their output below. If you have docker installed, you can probably reproduce it
locally.
Message-id: 1471015978-1123-1-git-send-email-stefa...@redhat.com
Subject: [Qemu-devel] [PATCH for-2.7 0/4] virtio-balloon: fix
On 13.08.2016 00:40, David Kiarie wrote:
On Fri, Aug 12, 2016 at 10:10 PM, Valentine Sinitsyn
mailto:valentine.sinit...@gmail.com>> wrote:
Hi David,
On 02.08.2016 13:39, David Kiarie wrote:
Add AMD IOMMU emulaton to Qemu in addition to Intel IOMMU.
The IOMMU does basi
On Wed, Aug 10, 2016 at 09:08:01PM +0200, Cédric Le Goater wrote:
> From: Greg Kurz
>
> Considering that features are converted to global properties and
> global properties are automatically applied to every new instance
> of created CPU (at object_new() time), there is no point in
> parsing cpu_
On 11.08.2016 00:42, David Kiarie wrote:
Introduce macros and trace events for use in AMD IOMMU
interrupt remapping
Signed-off-by: David Kiarie
---
hw/i386/amd_iommu.h | 72
hw/i386/trace-events | 7 +
2 files changed, 79 insertions(+)
Hi David,
On 02.08.2016 13:39, David Kiarie wrote:
Add AMD IOMMU emulaton to Qemu in addition to Intel IOMMU.
The IOMMU does basic translation, error checking and has a
minimal IOTLB implementation. This IOMMU bypassed the need
for target aborts by responding with IOMMU_NONE access rights
and ex
On 8/10/2016 12:30 AM, Alex Williamson wrote:
> On Thu, 4 Aug 2016 00:33:51 +0530
> Kirti Wankhede wrote:
>
> This is used later by mdev_device_start() and mdev_device_stop() to get
> the parent_device so it can call the start and stop ops callbacks
> respectively. That seems to imply that all
Manipulate data and store 8bytes instead of 4bytes.
Signed-off-by: Nikunj A Dadhania
---
target-ppc/translate/vsx-impl.inc.c | 27 +--
1 file changed, 13 insertions(+), 14 deletions(-)
diff --git a/target-ppc/translate/vsx-impl.inc.c
b/target-ppc/translate/vsx-impl.inc.
stxvb16x: Store VSX Vector Byte*16
stxvh8x: Store VSX Vector Halfword*8
Signed-off-by: Nikunj A Dadhania
---
target-ppc/translate/vsx-impl.inc.c | 55 +
target-ppc/translate/vsx-ops.inc.c | 2 ++
2 files changed, 57 insertions(+)
diff --git a/target-ppc/tr
xxspltib: VSX Vector Splat Immediate Byte
Copy the immediate byte in each byte of target VSR
Signed-off-by: Nikunj A Dadhania
---
target-ppc/translate.c | 2 ++
target-ppc/translate/vsx-impl.inc.c | 20
target-ppc/translate/vsx-ops.inc.c | 5 +
3 files c
lxvb16x: Load VSX Vector Byte*16
lxvh8x: Load VSX Vector Halfword*8
Signed-off-by: Nikunj A Dadhania
---
target-ppc/helper.h | 1 +
target-ppc/mem_helper.c | 6
target-ppc/translate/vsx-impl.inc.c | 57 +
target-ppc/translat
Use macro for ld64 as well, this changes the function signature from
gen_qemu_ld64 => gen_qemu_ld64_i64. Replace this at all the call sites.
Signed-off-by: Nikunj A Dadhania
---
target-ppc/translate.c | 39 +++---
target-ppc/translate/fp-impl.inc.c | 42
lxsibzx - Load VSX Scalar as Integer Byte & Zero Indexed
lxsihzx - Load VSX Scalar as Integer Halfword & Zero Indexed
Signed-off-by: Nikunj A Dadhania
---
target-ppc/translate.c | 2 ++
target-ppc/translate/vsx-impl.inc.c | 2 ++
target-ppc/translate/vsx-ops.inc.c | 2 ++
3 files c
stxsibx - Store VSX Scalar as Integer Byte Indexed
stxsihx - Store VSX Scalar as Integer Halfword Indexed
Signed-off-by: Nikunj A Dadhania
---
target-ppc/translate.c | 2 ++
target-ppc/translate/vsx-impl.inc.c | 3 +++
target-ppc/translate/vsx-ops.inc.c | 2 ++
3 files changed, 7 i
Use macro for ld64 as well, this changes the function signature from
gen_qemu_st64 => gen_qemu_st64_i64. Replace this at all the call sites.
Signed-off-by: Nikunj A Dadhania
---
target-ppc/translate.c | 37 ++--
target-ppc/translate/fp-impl.inc.c | 42 ++
From: Ravi Bangoria
darn: Deliver A Random Number
Currently return invalid random number for all the case. This needs
proper algorithm to provide cryptographically suitable random data.
Reading from /dev/random can block and that is not an expected behaviour
while the cpu instruction is getting
Use tcg_gen_qemu_ld in the load with reservation instructions.
Signed-off-by: Nikunj A Dadhania
---
target-ppc/translate.c | 22 +++---
1 file changed, 11 insertions(+), 11 deletions(-)
diff --git a/target-ppc/translate.c b/target-ppc/translate.c
index 21092d0..87857f7 100644
--
Being a 16byte operation, qemu_ld/st still does not support this. Move
this out so other store operation can use qemu_ld/st in the following
patch. Also, convert it to two MO_Q operations for stqcx.
Signed-off-by: Nikunj A Dadhania
---
target-ppc/translate.c | 69
Use tcg_gen_qemu_st store conditional instructions.
Signed-off-by: Nikunj A Dadhania
---
target-ppc/translate.c | 58 +-
1 file changed, 24 insertions(+), 34 deletions(-)
diff --git a/target-ppc/translate.c b/target-ppc/translate.c
index f3d2c4e..
Make byte-swap routines use the common GEN_QEMU_LOAD macro
Signed-off-by: Nikunj A Dadhania
---
target-ppc/translate.c | 32 ++--
1 file changed, 10 insertions(+), 22 deletions(-)
diff --git a/target-ppc/translate.c b/target-ppc/translate.c
index bd16681..21092d0 100
Implement macro to consolidate store operations using newer
tcg_gen_qemu_ld functions.
Signed-off-by: Nikunj A Dadhania
---
target-ppc/translate.c | 58 +-
1 file changed, 20 insertions(+), 38 deletions(-)
diff --git a/target-ppc/translate.c b/tar
Load 8byte at a time and manipulate.
Signed-off-by: Nikunj A Dadhania
---
target-ppc/helper.h | 1 +
target-ppc/mem_helper.c | 5 +
target-ppc/translate/vsx-impl.inc.c | 34 --
3 files changed, 26 insertions(+), 14 deletions(-)
d
1) Consolidate Load/Store operations using tcg_gen_qemu_ld/st functions
2) This series contains 10 new instructions for POWER9 ISA3.0
Use newer qemu load/store tcg helpers and optimize stxvw4x and lxvw4x.
Patches:
01-09: Cleanup load/store operations in ppc translator
10: xxspltib: VSX
Implement macro to consolidate store operations using newer
tcg_gen_qemu_st function.
Signed-off-by: Nikunj A Dadhania
---
target-ppc/translate.c | 35 ---
1 file changed, 16 insertions(+), 19 deletions(-)
diff --git a/target-ppc/translate.c b/target-ppc/translat
Make byte-swap routines use the common GEN_QEMU_LOAD macro
Signed-off-by: Nikunj A Dadhania
---
target-ppc/translate.c | 27 ++-
1 file changed, 10 insertions(+), 17 deletions(-)
diff --git a/target-ppc/translate.c b/target-ppc/translate.c
index 42f403a..a33e0ca 100644
-
On Fri, Aug 12, 2016 at 03:14:32PM -0300, Eduardo Habkost wrote:
> The kvm_pv_unhalt feature doesn't work if kernel_irqchip is
> disabled, so we need to report it as unsupported.
>
> Signed-off-by: Eduardo Habkost
> ---
> target-i386/kvm.c | 7 +++
> 1 file changed, 7 insertions(+)
Reviewed
On 8/12/2016 12:13 AM, Alex Williamson wrote:
>
> TBH, I don't see how providing a default implementation of
> validate_map_request() is useful. How many mediated devices are going
> to want to identity map resources from the parent? Even if they do, it
> seems we can only support a single me
Sorry, forgot the for-2.7 tag in the Subject. This is a bug fix I
would like to get into QEMU 2.7.
On Fri, Aug 12, 2016 at 03:14:32PM -0300, Eduardo Habkost wrote:
> The kvm_pv_unhalt feature doesn't work if kernel_irqchip is
> disabled, so we need to report it as unsupported.
>
> Signed-off-by:
The kvm_pv_unhalt feature doesn't work if kernel_irqchip is
disabled, so we need to report it as unsupported.
Signed-off-by: Eduardo Habkost
---
target-i386/kvm.c | 7 +++
1 file changed, 7 insertions(+)
diff --git a/target-i386/kvm.c b/target-i386/kvm.c
index 0b2016a..d1a25c5 100644
--- a/
* Matthew Garrett (mj...@coreos.com) wrote:
> Trusted Boot is based around having a trusted store of measurement data and
> a secure communications channel between that store and an attestation
> target. In actual hardware, that's a TPM. Since the TPM can only be accessed
> via the host system, thi
Using QemuOpts will prevent qemu from crashing if the input options have
not been validated (which is the case when they are specified on the
command line or in a json: filename) and some have the wrong type.
Signed-off-by: Max Reitz
---
block/nbd.c | 74 +
Store the configuration file's filename so it can later be used in
bdrv_refresh_filename() without having to directly access the options
QDict which may contain a value of a non-string type.
Signed-off-by: Max Reitz
---
block/blkdebug.c | 17 -
1 file changed, 12 insertions(+), 5
The SSH and NBD block drivers currently directly extract their runtime
options from the options QDict they receive. This is bad practice and
can lead to segmentation faults (which, however, will always be a NULL
pointer dereference, so it should not be exploitable beyond a DoS).
This series fixes
Signed-off-by: Max Reitz
---
tests/qemu-iotests/162 | 96 ++
tests/qemu-iotests/162.out | 17
tests/qemu-iotests/group | 1 +
3 files changed, 114 insertions(+)
create mode 100755 tests/qemu-iotests/162
create mode 100644 tests/qemu-io
Using QemuOpts will prevent qemu from crashing if the input options have
not been validated (which is the case when they are specified on the
command line or in a json: filename) and some have the wrong type.
Signed-off-by: Max Reitz
---
block/ssh.c | 77 +
Store the runtime option values in the BDRVNBDState so they can later be
used in nbd_refresh_filename() without having to directly access the
options QDict which may contain values of non-string types.
Signed-off-by: Max Reitz
---
block/nbd.c | 105 +++
Signed-off-by: Marc-André Lureau
---
tests/ide-test.c | 2 ++
tests/ipmi-bt-test.c | 5 -
tests/ipmi-kcs-test.c | 8 ++--
3 files changed, 12 insertions(+), 3 deletions(-)
diff --git a/tests/ide-test.c b/tests/ide-test.c
index 1e51af2..f6e10ad 100644
--- a/tests/ide-test.c
+++ b/te
On 12 August 2016 at 16:52, Marc-André Lureau
wrote:
> Add some missing flags description.
>
> Signed-off-by: Marc-André Lureau
> ---
> configure | 6 ++
> 1 file changed, 6 insertions(+)
>
> diff --git a/configure b/configure
> index 8d84919..352cbdd 100755
> --- a/configure
> +++ b/configu
On 12 August 2016 at 17:48, Marc-André Lureau wrote:
> Hi
>
> - Original Message -
>> On 12 August 2016 at 16:52, Marc-André Lureau
>> wrote:
>> > Add some missing flags description.
>> >
>> > Signed-off-by: Marc-André Lureau
>> > ---
>> > configure | 6 ++
>> > 1 file changed, 6 in
On 12/08/16 12:08 pm, "Fam Zheng" wrote:
>On Wed, 08/10 18:30, Michael S. Tsirkin wrote:
>> From: Prerna Saxena
>>
>> The set_mem_table command currently does not seek a reply. Hence, there is
>> no easy way for a remote application to notify to QEMU when it finished
>> setting up memory,
On Fri, Aug 12, 2016 at 2:48 PM, Li, Liang Z wrote:
>> >> > BTW. Is it possible to bypass the shared block in the
>> >> 'ram_find_and_save_block'?
>> >> > I mean no to check if a page is dirty for a shared block, it may
>> >> > make things
>> >> faster.
>> >>
>> >> Nice spotted. That would make t
Hi
- Original Message -
> On 12 August 2016 at 16:52, Marc-André Lureau
> wrote:
> > Add some missing flags description.
> >
> > Signed-off-by: Marc-André Lureau
> > ---
> > configure | 6 ++
> > 1 file changed, 6 insertions(+)
> >
> > diff --git a/configure b/configure
> > index 8d
On Thu, Aug 11, 2016 at 06:22:20PM +0200, Kevin Wolf wrote:
> A while ago we were debugging a hang where coroutines were waiting for a mutex
> to be unlocked, but we couldn't find out who held the lock. This series adds
> some information to Coroutine and CoMutex that both allows to add a few
> ass
On Mon, Aug 08, 2016 at 05:11:21PM +0200, Laurent Vivier wrote:
> Documentation is docs/tracing.txt instead of docs/trace-events.txt.
>
> find . -name trace-events -exec \
> sed -i "s?See docs/trace-events.txt for syntax documentation.?See
> docs/tracing.txt for syntax documentation.?" \
>
Add some missing flags description.
Signed-off-by: Marc-André Lureau
---
configure | 6 ++
1 file changed, 6 insertions(+)
diff --git a/configure b/configure
index 8d84919..352cbdd 100755
--- a/configure
+++ b/configure
@@ -1324,9 +1324,11 @@ disabled with --disable-FEATURE, default is enab
Richard Henderson writes:
> On 08/12/2016 05:52 AM, Nikunj A Dadhania wrote:
>> Richard Henderson writes:
>>
>>> On 08/10/2016 08:00 PM, Nikunj A Dadhania wrote:
+#define GEN_QEMU_LOAD_64(ldop, op) \
+static void glue(gen_qemu_, glue(ldop, _i64))(DisasC
On Fri, Aug 12, 2016 at 01:01:16PM +0100, Peter Maydell wrote:
> On 12 August 2016 at 08:20, Marc-André Lureau wrote:
> > Hi
> >
> > - Original Message -
> >> sent a follow-up response to GET_FEATURES), I am now wondering if this
> >> patch
> >> may break existing vhost applications too ?
On Fri, Aug 12, 2016 at 03:20:56AM -0400, Marc-André Lureau wrote:
> Hi
>
> - Original Message -
> > sent a follow-up response to GET_FEATURES), I am now wondering if this patch
> > may break existing vhost applications too ? If so, reverting it possibly
> > better.
> > What confuses me is
Looks like Serge's fix has been included here:
http://git.qemu.org/?p=qemu.git;a=commitdiff;h=efc8243d00ab4cf4fa05a9b
... so let's close this bug now.
** Changed in: qemu
Status: New => Fix Released
--
You received this bug notification because you are a member of qemu-
devel-ml, which is
Peter's fix had been included here:
http://git.qemu.org/?p=qemu.git;a=commitdiff;h=967c0da73a7b0da186baba6
... so I think we can close this bug ticket now.
** Changed in: qemu
Status: New => Fix Released
--
You received this bug notification because you are a member of qemu-
devel-ml, whi
Hi Dou,
I'm catching up on mail after vacation so I reviewed the wrong
version of this patch first... Darn. I'll try again with this
version. Please still check my comments in that old version though,
as I'm going faster through this one and may miss things that are
here too (although not all com
The statistics virtqueue is not migrated properly because virtio-balloon
does not include s->stats_vq_elem in the migration stream.
After migration the statistics virtqueue hangs because the host never
completes the last element (s->stats_vq_elem is NULL on the destination
QEMU). Therefore the gu
On Mon, Aug 08, 2016 at 02:07:17PM -0400, Colin Lord wrote:
> This commit moves the initialization of the QemuOptsList qemu_iscsi_opts
> struct out of block/iscsi.c in order to allow the iscsi module to be
> dynamically loaded.
>
> Signed-off-by: Colin Lord
> Reviewed-by: Fam Zheng
> ---
> bloc
An explicit if/else is clearer than arithmetic assuming #true is 1,
while the compiler should be able to generate just as optimal code.
Signed-off-by: Lluís Vilanova
---
stubs/trace-control.c | 17 +++--
trace/control-target.c | 31 ++-
2 files change
virtqueue_descard() moves vq->last_avail_idx back so the element can be
popped again. It's necessary to decrement vq->inuse to avoid "leaking"
the element count.
Signed-off-by: Stefan Hajnoczi
---
hw/virtio/virtio.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/hw/virtio/virtio.c b/hw/vir
As far as I can see, this should have been fixed in upstream QEMU by this
commit here:
http://git.qemu.org/?p=qemu.git;a=commitdiff;h=8cb6bfb54e91b1a31a
... so closing this issue now.
** Changed in: qemu
Status: New => Fix Released
--
You received this bug notification because you are a
On Fri, Aug 12, 2016 at 10:34:44AM +0200, Paolo Bonzini wrote:
>
>
> On 11/08/2016 15:29, Peter Xu wrote:
> > +
> > +static void vtd_eim_prop_set(Object *o, bool value, Error **errp)
> > +{
> > +IntelIOMMUState *s = INTEL_IOMMU_DEVICE(o);
> > +s->eim_supported = value;
> > +}
> > +
> > +s
According to Stefan, this problem has been fixed by this commit:
http://git.qemu.org/?p=qemu.git;a=commitdiff;h=cc4662f9642995c78
... so let's close this bug ticket now.
** Changed in: qemu
Status: Confirmed => Fix Released
--
You received this bug notification because you are a member of
On 10 August 2016 at 04:35, Andrew Dutcher wrote:
> Hello!
>
> I ran into an issue where qemu (specifically, the unicorn engine)
> would hang forever in the middle of the emulated square root
> instruction under certain circumstances. I eventually tracked the
> issue down to the square root of an
Hi
- Original Message -
> On Fri, Aug 12, 2016 at 03:20:56AM -0400, Marc-André Lureau wrote:
> > Hi
> >
> > - Original Message -
> > > sent a follow-up response to GET_FEATURES), I am now wondering if this
> > > patch
> > > may break existing vhost applications too ? If so, revert
Previously all test cases in a category, such as check-qtest-y, are
executed in a single long gtester command. This patch separates each
test into its own make target to allow better parallism.
Signed-off-by: Fam Zheng
---
This saves 50% of the time "make check takes" compared to on master
(I us
Thanks for this bug report. The problem here is that QEMU is not
correctly handling the obsolete "unnormal" 80-bit floating point format.
The Intel architecture reference says that this should be handled by
raising the invalid-input exception and returning the default NaN.
See also the discussion
On 08/11/2016 12:47 PM, Peter Maydell wrote:
> On 2 August 2016 at 18:15, Cédric Le Goater wrote:
>> On the AST2500, I am still having a little issue under uboot which
>> sets the vbar doing :
>>
>> mcr p15, 0, r0, c12, c0, 0 /* Set VBAR */
>>
>> and this is trapped as an
On Mon, Aug 08, 2016 at 10:28:02AM +0800, Dou Liyang wrote:
> This document describes how to use cpu hotplug in QEMU.
>
> Signed-off-by: Dou Liyang
> ---
> docs/cpu-hotplug.txt | 110
> +++
> 1 file changed, 110 insertions(+)
> create mode 100644
Removes the event state array used for early initialization. Since only
events with the "vcpu" property need a late initialization fixup,
threats their initialization specially.
Assumes that the user won't touch the state of "vcpu" events between
early and late initialization (e.g., through QMP).
Since there is no udev file in upstream QEMU, I guess this bug was meant
for the qemu Ubuntu package instead?
** Project changed: qemu => qemu (Ubuntu)
--
You received this bug notification because you are a member of qemu-
devel-ml, which is subscribed to QEMU.
https://bugs.launchpad.net/bugs/8
On Fri, 12 Aug 2016 09:35:12 +0100
Stefan Hajnoczi wrote:
> On Fri, Aug 12, 2016 at 02:54:02PM +0800, Xiao Guangrong wrote:
> > This patchset is against commit c597dc90fbcd6 (virtio-net: allow increasing
> > rx queue siz) on pci branch of Michael's git tree and can be found at:
> > https://
On 08.08.2016 20:07, Colin Lord wrote:
> One more minor revision from v6, no big changes.
>
> v7:
> - Add ifdef around qemu_iscsi_opts in vl.c (first patch)
>
> v6:
> - Fix bug so that users can specify a modularized driver on the cli
> without qemu exiting
> - Remove extra lines from Makefile
Removes the need for 'trace_events_dstate_init' and does a little cleanup in how
state values are modified (to avoid implicit conversions from bool).
Changes in v2
=
* Fix late-init state value [Daniel P. Berrange].
Changes in v3
=
* Avoid implicit conversions from bool
virtqueue_discard() requires a VirtQueueElement but virtio-balloon does
not migrate its in-use element. Introduce a new function that is
similar to virtqueue_discard() but doesn't require a VirtQueueElement.
This will allow virtio-balloon to access element again after migration
with the usual pro
Hi
- Original Message -
> sent a follow-up response to GET_FEATURES), I am now wondering if this patch
> may break existing vhost applications too ? If so, reverting it possibly
> better.
> What confuses me is why it doesn’t fail all the time, but only about 20% to
> 30% time as Fam report
From: Laurent Vivier
Documentation is docs/tracing.txt instead of docs/trace-events.txt.
find . -name trace-events -exec \
sed -i "s?See docs/trace-events.txt for syntax documentation.?See
docs/tracing.txt for syntax documentation.?" \
{} \;
Signed-off-by: Laurent Vivier
Message-id:
On 11/08/2016 11:36, Lv Zheng wrote:
>
> -error_setg(errp, "'-acpitable' requires one of 'data' or 'file'");
> +val = qemu_opt_get((QemuOpts *)opts, "fadt");
> +if (val) {
> +unsigned long rev;
Don't use qemu_opt_get. Add the field to AcpiTableOptions in
qapi-schema.json,
The vq->inuse field is not migrated. Many devices don't hold
VirtQueueElements across migration so it doesn't matter that vq->inuse
starts at 0 on the destination QEMU.
At least virtio-serial, virtio-blk, and virtio-balloon migrate while
holding VirtQueueElements. For these devices we need to re
G 3 writes:
> On Aug 12, 2016, at 9:19 AM, Alex Bennée wrote:
>
>> On 11 August 2016 at 17:43, G 3 wrote:
>>>
>>> On Aug 11, 2016, at 11:24 AM, qemu-devel-requ...@nongnu.org wrote:
>>>
>>>
>>> Performance
>>>
>>> ===
>>>
>>>
>>> You can't do full work-load testing on this tree due to th
1 - 100 of 164 matches
Mail list logo
