On Thu, 2013-11-28 at 15:41 +0100, Igor Mammedov wrote:
On Wed, 9 Oct 2013 17:43:08 +0800
Chen Fan chen.fan.f...@cn.fujitsu.com wrote:
Via implementing ACPI standard methods _EJ0 in bios, after Guest OS hot
remove
one vCPU, it is able to send a signal to QEMU, then QEMU could notify
On Thu, Dec 19, 2013 at 12:25:57PM +0530, Aneesh Kumar K.V wrote:
Alexander Graf ag...@suse.de writes:
This breaks if you run a 64-bit guest on a 32-bit host trying to
access memory beyond 4GB. In that case htab_base is hwaddr (64bit)
while uint8_t is only 32bit wide.
Wow!! didn't know
On 19 December 2013 05:49, Christoffer Dall christoffer.d...@linaro.org wrote:
On Thu, Nov 28, 2013 at 05:43:54PM +, Peter Maydell wrote:
The other possibility is that it's a correct implementation
of 11MPCore GIC semantics -- the documentation of the
11MPCore definitely says that level
On Wed, Dec 18, 2013 at 01:53:56PM +0200, Alexander Binun wrote:
We are trying to monitor the traffic (network packets etc) between VMs in
KVM. We succeeded to get the address of the system call table (see
http://syprog.blogspot.co.il/2011/10/hijack-linux-system-calls-part-iii.html)
and
On 19 December 2013 06:01, Peter Crosthwaite
peter.crosthwa...@xilinx.com wrote:
On Wed, Dec 18, 2013 at 1:12 AM, Peter Maydell peter.mayd...@linaro.org
wrote:
+*key = ENCODE_AA64_CP_REG(r2-cp, r-crn, crm,
+ r-opc0, opc1, opc2);
On Wed, Dec 18, 2013 at 06:05:39PM +0100, Markus Armbruster wrote:
Andreas Färber afaer...@suse.de writes:
I had a brief look at this series. Dropping redundant fields certainly
sounds good and after the lengthy QOM'ifications of IDE devices we seem
to no longer break any devices, but
On Wed, 2013-12-18 at 19:20 +0200, Michael S. Tsirkin wrote:
On Wed, Dec 18, 2013 at 06:49:24PM +0200, Marcel Apfelbaum wrote:
On Wed, 2013-12-18 at 18:33 +0200, Michael S. Tsirkin wrote:
On Wed, Dec 18, 2013 at 06:27:12PM +0200, Marcel Apfelbaum wrote:
On Wed, 2013-12-18 at 17:22 +0200,
On Thu, 2013-12-19 at 11:37 +0200, Marcel Apfelbaum wrote:
On Wed, 2013-12-18 at 19:20 +0200, Michael S. Tsirkin wrote:
On Wed, Dec 18, 2013 at 06:49:24PM +0200, Marcel Apfelbaum wrote:
On Wed, 2013-12-18 at 18:33 +0200, Michael S. Tsirkin wrote:
On Wed, Dec 18, 2013 at 06:27:12PM +0200,
On 12/17/2013 05:26 PM, Juan Quintela wrote:
Performance is important in this function, and we want to optimize even further.
Signed-off-by: Juan Quintela quint...@redhat.com
---
kvm-all.c | 7 +--
1 file changed, 5 insertions(+), 2 deletions(-)
diff --git a/kvm-all.c b/kvm-all.c
index
On 12/17/2013 05:26 PM, Juan Quintela wrote:
Signed-off-by: Juan Quintela quint...@redhat.com
---
kvm-all.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/kvm-all.c b/kvm-all.c
index 308dfba..cb62ba4 100644
--- a/kvm-all.c
+++ b/kvm-all.c
@@ -382,7 +382,8 @@
On 17 December 2013 15:26, Juan Quintela quint...@redhat.com wrote:
@@ -399,8 +401,9 @@ static int
kvm_get_dirty_pages_log_range(MemoryRegionSection *section,
page_number = (i * HOST_LONG_BITS + j) * hpratio;
addr1 = page_number * TARGET_PAGE_SIZE;
On 12/17/2013 05:26 PM, Juan Quintela wrote:
We want to have all the functions that handle directly the dirty
bitmap near. We will change it later.
Signed-off-by: Juan Quintela quint...@redhat.com
---
include/exec/ram_addr.h | 31 +++
kvm-all.c | 27
On 12/17/2013 05:26 PM, Juan Quintela wrote:
If bitmaps are aligned properly, use bitmap operations. If they are
not, just use old bit at a time code.
Signed-off-by: Juan Quintela quint...@redhat.com
---
include/exec/ram_addr.h | 54 -
1 file
On Thu, Dec 19, 2013 at 11:49:21AM +0200, Marcel Apfelbaum wrote:
Basically something like the below (warning: completely untested,
sorry).
I tested and it works fine for both windows and linux guests!
Actually it doesn't work :(
I was testing a working hack, not the master branch.
I
On 12/17/2013 05:26 PM, Juan Quintela wrote:
This function is the only bit where we care about speed.
Signed-off-by: Juan Quintela quint...@redhat.com
---
arch_init.c | 34 --
1 file changed, 20 insertions(+), 14 deletions(-)
diff --git a/arch_init.c
On 12/17/2013 05:26 PM, Juan Quintela wrote:
We use the old code if the bitmaps are not aligned
Signed-off-by: Juan Quintela quint...@redhat.com
---
arch_init.c | 38 +-
1 file changed, 29 insertions(+), 9 deletions(-)
diff --git a/arch_init.c
On 12/18/2013 02:48 PM, Corey Bryant wrote:
This fixes a bug where we weren't exiting if seccomp_init() failed.
Signed-off-by: Corey Bryant cor...@linux.vnet.ibm.com
---
qemu-seccomp.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/qemu-seccomp.c b/qemu-seccomp.c
index
On 12/16/2013 08:23 AM, Andreas Färber wrote:
Am 04.12.2013 06:51, schrieb Peter Crosthwaite:
On Wed, Dec 4, 2013 at 1:42 AM, Paolo Bonzini pbonz...@redhat.com wrote:
From: Hervé Poussineau hpous...@reactos.org
We should not modify the type hash table while it is being iterated on.
Assert
On 19 December 2013 07:27, Fedorov Sergey s.fedo...@samsung.com wrote:
Yes, this banking scheme makes state changing events quite heavy. But
maintaining the active copies allows to keep translation table walking code
untouched. I think there is a trade-off between state changing and
Hi,
So that would look like a SeaBIOS feature freeze on Dec 6th and target
release date of Dec 20th. Works for me.
/me got sick, so with the xmas/newyear holidays being just around the
corner that implies I'll be offline for the rest of the year.
Kevin will most likely tag v1.7.4
On Thu, Dec 19, 2013 at 9:38 PM, Peter Maydell peter.mayd...@linaro.org wrote:
On 19 December 2013 07:27, Fedorov Sergey s.fedo...@samsung.com wrote:
Yes, this banking scheme makes state changing events quite heavy. But
maintaining the active copies allows to keep translation table walking code
On Mon, Dec 09, 2013 at 01:24:36PM +0800, Fam Zheng wrote:
VMFS extent line in description file should be with 4 fields:
RW size VMFS file-name.vmdk
Check the number explicitly and report error if offset is appended as
FLAT, which should be invalid format.
Reported-by: Paolo Bonzini
Hi Mark,
this patch breaks booting with OBP:
Probing /iommu@0,1000/sbus@0,10001000 at 5,0 espdma esp sd st
SUNW,bpp ledma le
Probing /iommu@0,1000/sbus@0,10001000 at 4,0 SUNW,CS4231 power-management
Probing /iommu@0,1000/sbus@0,10001000 at 1,0 Nothing there
Probing
On Thu, Dec 19, 2013 at 5:10 PM, Edgar E. Iglesias
edgar.igles...@gmail.com wrote:
On Thu, Dec 19, 2013 at 04:37:22PM +1000, Peter Crosthwaite wrote:
On Thu, Dec 19, 2013 at 3:51 PM, edgar.igles...@gmail.com wrote:
From: Edgar E. Iglesias edgar.igles...@xilinx.com
Signed-off-by: Edgar E.
On Tue, Dec 17, 2013 at 08:00:00PM +0800, Fam Zheng wrote:
@@ -1511,48 +1521,55 @@ static int vmdk_create_extent(const char *filename,
int64_t filesize,
header.check_bytes[3] = 0xa;
/* write all the data */
-ret = qemu_write_full(fd, magic, sizeof(magic));
-if (ret !=
On 12/10/2013 07:16 PM, Alexey Kardashevskiy wrote:
On 12/03/2013 02:30 PM, Alexey Kardashevskiy wrote:
Hi!
This is some cleanup. Please, comment. Thanks!
Ping, anyone?
Ping?
Changes:
v5:
* cleanup
* removed cpustate::kvm_cpu_id
* split into 2 patches - new PPC API and the
Peter Maydell peter.mayd...@linaro.org wrote:
On 17 December 2013 15:26, Juan Quintela quint...@redhat.com wrote:
@@ -399,8 +401,9 @@ static int
kvm_get_dirty_pages_log_range(MemoryRegionSection *section,
page_number = (i * HOST_LONG_BITS + j) * hpratio;
On 12/19/2013 04:44 PM, Peter Crosthwaite wrote:
On Thu, Dec 19, 2013 at 9:38 PM, Peter Maydell peter.mayd...@linaro.org wrote:
On 19 December 2013 07:27, Fedorov Sergey s.fedo...@samsung.com wrote:
Yes, this banking scheme makes state changing events quite heavy. But
maintaining the active
On Thu, Dec 19, 2013 at 7:03 PM, Peter Maydell peter.mayd...@linaro.org wrote:
On 19 December 2013 05:49, Christoffer Dall christoffer.d...@linaro.org
wrote:
On Thu, Nov 28, 2013 at 05:43:54PM +, Peter Maydell wrote:
The other possibility is that it's a correct implementation
of 11MPCore
On 19 December 2013 13:49, Peter Crosthwaite
peter.crosthwa...@xilinx.com wrote:
On Thu, Dec 19, 2013 at 7:03 PM, Peter Maydell peter.mayd...@linaro.org
wrote:
My initial thought would be either to have if statements at the
relevant points (which is how we've handled 11mpcore
differences so
On Thu, Dec 19, 2013 at 11:39 PM, Fedorov Sergey s.fedo...@samsung.com wrote:
On 12/19/2013 04:44 PM, Peter Crosthwaite wrote:
On Thu, Dec 19, 2013 at 9:38 PM, Peter Maydell peter.mayd...@linaro.org
wrote:
On 19 December 2013 07:27, Fedorov Sergey s.fedo...@samsung.com wrote:
Yes, this
On 19 December 2013 14:01, Peter Crosthwaite
peter.crosthwa...@xilinx.com wrote:
On Thu, Dec 19, 2013 at 11:39 PM, Fedorov Sergey s.fedo...@samsung.com
wrote:
Banked system control registers have two copies, one Secure and one
Non-secure.
Ok fair enough. I will wager though that sooner or
On Fri, Dec 13, 2013 at 05:22:08PM +0100, Igor Mammedov wrote:
.. so it could be used for adding CPU hotplug to Q35 machine
Signed-off-by: Igor Mammedov imamm...@redhat.com
Sounds good but please call this file cpu_hotplug.c/.h or cpu-hotplug.c/h
and make all prefixes acpi_cpu_hotplug and
On Fri, Dec 13, 2013 at 05:22:06PM +0100, Igor Mammedov wrote:
Hardcoded GPE0 mask isn't really needed. Since GPE0_STS initialized
with all bits cleared and only QEMU itself can set bits there (i.e.
guest can only clear bits in it). So guest can't triger SCI
by setting _STS _EN bits and there
On Fri, Dec 13, 2013 at 05:22:10PM +0100, Igor Mammedov wrote:
it fixes IRQ storm since guest isn't able to lower SCI IRQ
after it has been handled when it clears GPE event.
Signed-off-by: Igor Mammedov imamm...@redhat.com
Applied, thanks.
---
hw/acpi/ich9.c | 1 +
1 file changed, 1
On Fri, Dec 13, 2013 at 05:22:07PM +0100, Igor Mammedov wrote:
... and rename it into acpi_update_sci() since it changes
SCI on only on PM registers status.
Signed-off-by: Igor Mammedov imamm...@redhat.com
Applied, thanks.
---
---
hw/acpi/core.c | 18 ++
On Fri, Dec 13, 2013 at 05:22:11PM +0100, Igor Mammedov wrote:
.. including readonly 'cpu-hotplug-io-base' property
Signed-off-by: Igor Mammedov imamm...@redhat.com
Is this documented in the Q35 spec? Could not find it there.
If this is our PV code like PIIX, then we need to document this
Il 18/12/2013 19:03, Andreas Färber ha scritto:
@@ -61,7 +69,6 @@ static void ioapic_common_realize(DeviceState *dev,
Error **errp)
{
IOAPICCommonState *s = IOAPIC_COMMON(dev);
IOAPICCommonClass *info;
-static int ioapic_no;
if (ioapic_no = MAX_IOAPICS) {
On Thu, Dec 19, 2013 at 11:59 PM, Peter Maydell
peter.mayd...@linaro.org wrote:
On 19 December 2013 13:49, Peter Crosthwaite
peter.crosthwa...@xilinx.com wrote:
On Thu, Dec 19, 2013 at 7:03 PM, Peter Maydell peter.mayd...@linaro.org
wrote:
My initial thought would be either to have if
On 19 December 2013 14:26, Peter Crosthwaite
peter.crosthwa...@xilinx.com wrote:
On Thu, Dec 19, 2013 at 11:59 PM, Peter Maydell
peter.mayd...@linaro.org wrote:
On 19 December 2013 13:49, Peter Crosthwaite
peter.crosthwa...@xilinx.com wrote:
On Thu, Dec 19, 2013 at 7:03 PM, Peter Maydell
Il 18/12/2013 18:21, Peter Lieven ha scritto:
Not completely I think, but please correct me if I am wrong.
If cache=writeback is set we issue just a write. In libnfs or libiscsi case
that guarantees that the request has been successfully executed
on the target / server. This is enough to be
On Mon, Nov 25, 2013 at 09:43:48AM -0700, Eric Blake wrote:
On 11/25/2013 09:32 AM, Paolo Bonzini wrote:
Yes please. Firing up a calculator to figure out how much is 1G is not
friendly, neither is firing it up to figure out what did management do
with QMP. It should be a text based
From: Markus Armbruster arm...@redhat.com
Once upon a time, the error message was:
qemu: -device nonexistant: Device nonexistant not found. Try -device
'?' for a list.
But progress marches on, and conversion to QError (commit 0204276)
changed it into:
Invalid parameter 'driver'
On Thu, 19 Dec 2013 16:14:03 +0200
Michael S. Tsirkin m...@redhat.com wrote:
On Fri, Dec 13, 2013 at 05:22:08PM +0100, Igor Mammedov wrote:
.. so it could be used for adding CPU hotplug to Q35 machine
Signed-off-by: Igor Mammedov imamm...@redhat.com
Sounds good but please call this
Michael S. Tsirkin m...@redhat.com writes:
On Mon, Nov 25, 2013 at 09:43:48AM -0700, Eric Blake wrote:
On 11/25/2013 09:32 AM, Paolo Bonzini wrote:
Yes please. Firing up a calculator to figure out how much is 1G is not
friendly, neither is firing it up to figure out what did management do
On Thu, 19 Dec 2013 16:18:59 +0200
Michael S. Tsirkin m...@redhat.com wrote:
On Fri, Dec 13, 2013 at 05:22:11PM +0100, Igor Mammedov wrote:
.. including readonly 'cpu-hotplug-io-base' property
Signed-off-by: Igor Mammedov imamm...@redhat.com
Is this documented in the Q35 spec? Could
Commit 75884afd5c6c42e523b08565e289dbe319e17ad9 (virtio-blk: Convert to
QOM realize) dropped a duplicate error_report() call. Now we no longer
get the following error message twice:
QEMU_PROG: -drive if=virtio: Device initialization failed.
Update qemu-iotests 051.
Signed-off-by: Stefan
On Thu, Dec 19, 2013 at 04:14:56PM +0100, Markus Armbruster wrote:
Michael S. Tsirkin m...@redhat.com writes:
On Mon, Nov 25, 2013 at 09:43:48AM -0700, Eric Blake wrote:
On 11/25/2013 09:32 AM, Paolo Bonzini wrote:
Yes please. Firing up a calculator to figure out how much is 1G is not
On Thu, Dec 19, 2013 at 04:17:08PM +0100, Igor Mammedov wrote:
On Thu, 19 Dec 2013 16:18:59 +0200
Michael S. Tsirkin m...@redhat.com wrote:
On Fri, Dec 13, 2013 at 05:22:11PM +0100, Igor Mammedov wrote:
.. including readonly 'cpu-hotplug-io-base' property
Signed-off-by: Igor
Il 19/12/2013 16:25, Stefan Hajnoczi ha scritto:
On Mon, Dec 9, 2013 at 9:48 PM, Paolo Bonzini pbonz...@redhat.com wrote:
From: Andreas Färber afaer...@suse.de
Signed-off-by: Andreas Färber afaer...@suse.de
Signed-off-by: Paolo Bonzini pbonz...@redhat.com
---
hw/block/virtio-blk.c
On Mon, Dec 9, 2013 at 9:48 PM, Paolo Bonzini pbonz...@redhat.com wrote:
From: Andreas Färber afaer...@suse.de
Signed-off-by: Andreas Färber afaer...@suse.de
Signed-off-by: Paolo Bonzini pbonz...@redhat.com
---
hw/block/virtio-blk.c | 10 ++
hw/char/virtio-serial-bus.c
Il 19/12/2013 15:40, Michael S. Tsirkin ha scritto:
On Mon, Nov 25, 2013 at 09:43:48AM -0700, Eric Blake wrote:
On 11/25/2013 09:32 AM, Paolo Bonzini wrote:
Yes please. Firing up a calculator to figure out how much is 1G is not
friendly, neither is firing it up to figure out what did
On 12/18/2013 4:37 PM, Alexander Graf wrote:
Am 18.12.2013 um 23:11 schrieb Scott Wood scottw...@freescale.com:
On Wed, 2013-12-18 at 23:09 +0100, Alexander Graf wrote:
On 18.12.2013, at 23:02, Scott Wood scottw...@freescale.com wrote:
On Mon, 2013-12-09 at 09:46 -0600, Tom Musta wrote:
On Thu, Dec 19, 2013 at 04:31:41PM +0100, Paolo Bonzini wrote:
Il 19/12/2013 15:40, Michael S. Tsirkin ha scritto:
On Mon, Nov 25, 2013 at 09:43:48AM -0700, Eric Blake wrote:
On 11/25/2013 09:32 AM, Paolo Bonzini wrote:
Yes please. Firing up a calculator to figure out how much is 1G is
On 19.12.2013, at 07:55, Aneesh Kumar K.V aneesh.ku...@linux.vnet.ibm.com
wrote:
Alexander Graf ag...@suse.de writes:
On 07.11.2013, at 15:31, Aneesh Kumar K.V aneesh.ku...@linux.vnet.ibm.com
wrote:
From: Aneesh Kumar K.V aneesh.ku...@linux.vnet.ibm.com
With kvm enabled, we store
On Wed, Dec 18, 2013 at 07:31:48PM +0100, Andreas Färber wrote:
Am 18.12.2013 17:34, schrieb Stefan Hajnoczi:
Use g_assert_cmpstr() instead of combining g_assert() and strcmp(3).
I vaguely remember that some such handy function was introduced only
after the minimum GLib version we require.
On Thu, Dec 19, 2013 at 03:39:58PM +, bharat.bhus...@freescale.com wrote:
-Original Message-
From: Michael S. Tsirkin [mailto:m...@redhat.com]
Sent: Thursday, December 19, 2013 3:55 AM
To: Alexander Graf
Cc: Bhushan Bharat-R65777; Wood Scott-B07421; QEMU Developers;
Il 19/12/2013 16:40, Michael S. Tsirkin ha scritto:
On Thu, Dec 19, 2013 at 04:31:41PM +0100, Paolo Bonzini wrote:
Il 19/12/2013 15:40, Michael S. Tsirkin ha scritto:
On Mon, Nov 25, 2013 at 09:43:48AM -0700, Eric Blake wrote:
On 11/25/2013 09:32 AM, Paolo Bonzini wrote:
Yes please. Firing
On 12/19/2013 07:30 AM, arm...@redhat.com wrote:
From: Markus Armbruster arm...@redhat.com
We've since abandoned our quest for rich error objects, fortunately
before it turned all error messages into mush. Time to undo the
damage to this one. Make it:
qemu: -device nonexistant:
On Wednesday, December 18, 2013 11:48:11 AM Corey Bryant wrote:
This fixes a bug where we weren't exiting if seccomp_init() failed.
Signed-off-by: Corey Bryant cor...@linux.vnet.ibm.com
---
qemu-seccomp.c | 1 +
1 file changed, 1 insertion(+)
Acked-by: Paul Moore pmo...@redhat.com
diff
On Thu, Nov 21, 2013 at 03:38:24AM +0100, Igor Mammedov wrote:
'etc/reserved-memory-end' will allow QEMU to tell BIOS where PCI
BARs mapping could safely start in high memory.
Allowing BIOS to start mapping 64-bit PCI BARs at address where it
wouldn't conflict with other mappings QEMU might
From: Aneesh Kumar K.V aneesh.ku...@linux.vnet.ibm.com
Targets like ppc64 support different typed of KVM, one which use
hypervisor mode and the other which doesn't. Add a new machine
property kvm-type that helps in selecting the respective ones
We also add a new QEMUMachine callback get_vm_type
On Wed, Dec 18, 2013 at 06:55:24PM +0200, Marcel Apfelbaum wrote:
On Wed, 2013-12-18 at 11:34 -0500, Paolo Bonzini wrote:
Or put an array of (bdf, offset, size, value) tuples somewhere in low
memory,
fill it at startup, and reproduce it blindly at S3 resume time. This is
similar
to
On 19.12.2013, at 16:38, bharat.bhus...@freescale.com wrote:
-Original Message-
From: Alexander Graf [mailto:ag...@suse.de]
Sent: Thursday, December 19, 2013 3:18 AM
To: Bhushan Bharat-R65777
Cc: Wood Scott-B07421; QEMU Developers; qemu-ppc; Bhushan Bharat-R65777
Subject: Re:
On 19.12.2013, at 16:39, bharat.bhus...@freescale.com wrote:
-Original Message-
From: Michael S. Tsirkin [mailto:m...@redhat.com]
Sent: Thursday, December 19, 2013 3:55 AM
To: Alexander Graf
Cc: Bhushan Bharat-R65777; Wood Scott-B07421; QEMU Developers; qemu-ppc;
Bhushan
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
Am 16.12.2013 17:51, schrieb Eric Blake:
On 07/15/2013 04:25 PM, Igor Mammedov wrote:
features check, enforce, hv_relaxed and hv_vapic are treated as
boolean set to 'on' when passed from command line, so it's not
neccessary to handle each of them
Am 16.12.2013 09:21, schrieb Marcel Apfelbaum:
On Sun, 2013-12-15 at 21:37 +0100, Andreas Färber wrote:
Am 28.11.2013 17:26, schrieb arm...@redhat.com:
Markus Armbruster (10):
qdev: Replace no_user by cannot_instantiate_with_device_add_yet
sysbus: Set
On 12/17/2013 07:12 AM, Peter Maydell wrote:
+TCGMemOp memop = MO_TE + size;
Watch the double space (multiple occurrences).
+if (is_vector) {
+if (opc == 3) {
+unallocated_encoding(s);
+return;
+}
In the integer LDP description I see
if
-Original Message-
From: Alexander Graf [mailto:ag...@suse.de]
Sent: Thursday, December 19, 2013 3:18 AM
To: Bhushan Bharat-R65777
Cc: Wood Scott-B07421; QEMU Developers; qemu-ppc; Bhushan Bharat-R65777
Subject: Re: [PATCH 1/2] ppc-e500: some pci related cleanup
On 28.11.2013,
-Original Message-
From: Michael S. Tsirkin [mailto:m...@redhat.com]
Sent: Thursday, December 19, 2013 9:20 PM
To: Bhushan Bharat-R65777
Cc: Alexander Graf; Wood Scott-B07421; QEMU Developers; qemu-ppc
Subject: Re: [PATCH 2/2] ppc-e500: implement PCI INTx routing
On Thu, Dec 19,
Public bug reported:
Git version f46e720a82ccdf1a521cf459448f3f96ed895d43 (and possibly
b91f93243bbe36dc436a64a662a9bbfb2362534a) fail to build under Linux
using gcc 4.8.1:
block/iscsi.c: In function ‘iscsi_co_readv’:
block/iscsi.c:363:40: error: too many arguments to function
-Original Message-
From: Michael S. Tsirkin [mailto:m...@redhat.com]
Sent: Thursday, December 19, 2013 3:55 AM
To: Alexander Graf
Cc: Bhushan Bharat-R65777; Wood Scott-B07421; QEMU Developers; qemu-ppc;
Bhushan
Bharat-R65777
Subject: Re: [PATCH 2/2] ppc-e500: implement PCI INTx
On Thu, 2013-12-19 at 09:35 -0600, Tom Musta wrote:
It turns out that all of the instructions added in this series are *not*
implemented
in e500mc. So the patch, as is, is correct for e500mc as well as P7/P8.
That may be true for e500mc (for some reason it didn't look that way
when I checked
On 19 December 2013 16:58, Richard Henderson r...@twiddle.net wrote:
On 12/17/2013 07:12 AM, Peter Maydell wrote:
+TCGMemOp memop = MO_TE + size;
Watch the double space (multiple occurrences).
+if (is_vector) {
+if (opc == 3) {
+unallocated_encoding(s);
+
On 12/17/2013 07:12 AM, Peter Maydell wrote:
+if (size == 3 opc == 2) {
+/* PRFM - prefetch */
+return;
+}
+if (opc == 3 size 1) {
+unallocated_encoding(s);
+return;
+}
+is_store = (opc == 0);
+
On Thu, 2013-12-19 at 11:06 -0500, Kevin O'Connor wrote:
On Wed, Dec 18, 2013 at 06:55:24PM +0200, Marcel Apfelbaum wrote:
On Wed, 2013-12-18 at 11:34 -0500, Paolo Bonzini wrote:
Or put an array of (bdf, offset, size, value) tuples somewhere in low
memory,
fill it at startup, and
On Thu, Dec 19, 2013 at 08:03:15PM +0200, Marcel Apfelbaum wrote:
On Thu, 2013-12-19 at 11:06 -0500, Kevin O'Connor wrote:
On Wed, Dec 18, 2013 at 06:55:24PM +0200, Marcel Apfelbaum wrote:
On Wed, 2013-12-18 at 11:34 -0500, Paolo Bonzini wrote:
Or put an array of (bdf, offset, size,
On Fri, Dec 06, 2013 at 05:54:27PM +0100, Paolo Bonzini wrote:
Post-order is the only sensible direction for the reset signals.
For example, suppose pre-order is used and the parent has some data
structures that cache children state (for example a list of active
requests). When the reset
On Thu, Dec 19, 2013 at 05:28:17PM +0100, Alexander Graf wrote:
On 19.12.2013, at 16:39, bharat.bhus...@freescale.com wrote:
-Original Message-
From: Michael S. Tsirkin [mailto:m...@redhat.com]
Sent: Thursday, December 19, 2013 3:55 AM
To: Alexander Graf
Cc: Bhushan
From: John V. Baboval john.babo...@citrix.com
This is an optimization to allow the UI to show/hide the current cursor without
setting it to a new bitmap.
In-guest screen sharing applications which show/hide the cursor when capturing
the screen every frame (GoToMeeting, WebEx, etc.) cause high
From: John V. Baboval john.babo...@citrix.com
The pointer is a place for the UI to hang a structure off of. In the multi-head
case, the UI can use this pointer to distinguish which QemuConsole a dpy_* call
was made for.
Signed-off-by: John V. Baboval john.babo...@citrix.com
---
From: John V. Baboval john.babo...@citrix.com
This is mostly just RFC at this point, since I'm not quite ready to submit the
associated multi-head video adapter, drivers and UI that consume this stuff.
(I still have to remove some Xen/XenClient specific stuff from it).
These are the
From: John V. Baboval john.babo...@citrix.com
Hook to notify a display adapter of interesting UI changes such as
monitor hotplug, orientation changes, etc...
Signed-off-by: John V. Baboval john.babo...@citrix.com
---
include/ui/console.h |9 +
ui/console.c |7 +++
2
From: John V. Baboval john.babo...@citrix.com
To be called by graphics adapter to have the ui re-initialize monitor state.
Signed-off-by: John V. Baboval john.babo...@citrix.com
---
include/ui/console.h |3 +++
ui/console.c |8
2 files changed, 11 insertions(+)
diff
From: John V. Baboval john.babo...@citrix.com
Add a hook in GraphicHwOps to pass monitor EDID for a QemuConsole
from the ui to the hw.
Signed-off-by: John V. Baboval john.babo...@citrix.com
---
include/ui/console.h |2 ++
ui/console.c | 10 ++
2 files changed, 12
From: John V. Baboval john.babo...@citrix.com
Add a hook in GraphicHwOps for the ui to inform the hw of display orientation
changes
Signed-off-by: John V. Baboval john.babo...@citrix.com
---
include/ui/console.h |2 ++
ui/console.c | 10 ++
2 files changed, 12
From: John V. Baboval john.babo...@citrix.com
Hook for adding another QemuConsole to the active display adapter.
Signed-off-by: John V. Baboval john.babo...@citrix.com
---
include/ui/console.h |2 ++
ui/console.c |8
2 files changed, 10 insertions(+)
diff --git
On Fri, Dec 06, 2013 at 05:54:23PM +0100, Paolo Bonzini wrote:
PCI is handling resetting of its devices before the bus is reset,
but this is only necessary because qdev is broken and usually does
pre-order reset. Post-order is a much better definition. Drop
the unnecessary flexibility that
On 12/17/2013 07:12 AM, Peter Maydell wrote:
+tcg_gen_mul_i64(tcg_tmp, tcg_op1, tcg_op2);
+if (is_sub) {
+tcg_gen_sub_i64(cpu_reg(s, rd), cpu_reg(s, ra), tcg_tmp);
+} else {
+tcg_gen_add_i64(cpu_reg(s, rd), cpu_reg(s, ra), tcg_tmp);
+}
Perhaps worth noticing
On 19/12/13 12:59, Artyom Tarasenko wrote:
Hi Mark,
this patch breaks booting with OBP:
Probing /iommu@0,1000/sbus@0,10001000 at 5,0 espdma esp sd st
SUNW,bpp ledma le
Probing /iommu@0,1000/sbus@0,10001000 at 4,0 SUNW,CS4231 power-management
Probing /iommu@0,1000/sbus@0,10001000
It should be possible to use a format as a driver for a file which in
turn requires another file, i.e., nesting file formats.
Allowing nested file formats results in e.g. qcow2 BlockDriverStates
never being directly passed to bdrv_open_common() from bdrv_file_open(),
but instead being handed
On Tue, Nov 26, 2013 at 01:38:30AM -0500, Xu Wang wrote:
If there is a loop in the backing file chain, it could cause problems
such as no response or a segfault during system boot. Hence detecting a
backing file loop is necessary. This patch extracts the loop check from
Use qemu_config_parse_qdict() to parse the command-line options in
addition to the config file.
Signed-off-by: Max Reitz mre...@redhat.com
Reviewed-by: Kevin Wolf kw...@redhat.com
---
block/blkdebug.c | 15 ---
1 file changed, 12 insertions(+), 3 deletions(-)
diff --git
Allow specifying a reference to an existing block device (by name) for
bdrv_file_open() instead of a filename and/or options.
Signed-off-by: Max Reitz mre...@redhat.com
Reviewed-by: Kevin Wolf kw...@redhat.com
---
block.c | 25 ++---
block/blkdebug.c | 2
This function basically parses command-line options given as a QDict
replacing a config file.
For instance, the QDict {section.opt1: 42, section.opt2: 23}
corresponds to the config file:
[section]
opt1 = 42
opt2 = 23
It is possible to specify multiple sections and also multiple sections
of the
Move the check whether there actually is a config file into the
read_config() function.
Signed-off-by: Max Reitz mre...@redhat.com
Reviewed-by: Kevin Wolf kw...@redhat.com
---
block/blkdebug.c | 36 +++-
1 file changed, 19 insertions(+), 17 deletions(-)
diff
Specifying the image filename through the file option is a legacy
option and should not be supported by blockdev-add (in that case, giving
a string for file references an existing block device).
Signed-off-by: Max Reitz mre...@redhat.com
Reviewed-by: Kevin Wolf kw...@redhat.com
---
blockdev.c |
Introduce the image option as an alternative to specifying the image
through the filename.
Signed-off-by: Max Reitz mre...@redhat.com
---
block/blkdebug.c | 12 +++-
1 file changed, 3 insertions(+), 9 deletions(-)
diff --git a/block/blkdebug.c b/block/blkdebug.c
index c73a6cf..3a46ecd
Currently, the configuration of blkdebug and blkverify is done through
the filename alone. There is now way of manually choosing blkdebug or
blkverify as a driver and using a normal image filename.
In the case of blkdebug, the filename starts with the protocol prefix,
follows up with the name of
If the filename is not prefixed by blkdebug: in
blkdebug_parse_filename(), the blkdebug driver was not selected through
that protocol prefix, but by an explicit command line option
(file.driver=blkdebug or something similar). Contrary to the current
reaction, this is not a problem at all; we just
1 - 100 of 236 matches
Mail list logo