On 01/15/10 21:07, Jan Kiszka wrote:
The missing '@' broke 'udp::port@:port' parsing.
if (sscanf(p, %64[^:]:%32[^@,]%n, host, port,pos) 2) {
host[0] = 0;
-if (sscanf(p, :%32[^,]%n, port,pos) 1) {
+if (sscanf(p, :%32[^@,]%n, port,pos) 1) {
Stefan Weil w...@mail.berlios.de writes:
The command line options -device, -nodefaults, -readconfig,
-writeconfig had entries for command line help, but
documentation for texi and derived formats (man, html, info)
was missing.
This also required moving @end table to the end of
Nathan Baum nat...@parenthephobia.org.uk writes:
On Fri, 2010-01-15 at 19:14 +0100, Markus Armbruster wrote:
Nathan Baum nat...@parenthephobia.org.uk writes:
Returns a QObject with information about a USB device.
Signed-off-by: Nathan Baum nat...@parenthephobia.org.uk
---
Gerd Hoffmann wrote:
On 01/15/10 21:07, Jan Kiszka wrote:
The missing '@' broke 'udp::port@:port' parsing.
if (sscanf(p, %64[^:]:%32[^@,]%n, host, port,pos) 2) {
host[0] = 0;
-if (sscanf(p, :%32[^,]%n, port,pos) 1) {
+if (sscanf(p,
Michael S. Tsirkin m...@redhat.com writes:
On Sun, Jan 17, 2010 at 04:48:13PM +0200, Naphtali Sprei wrote:
Instead of using the field 'readonly' of the BlockDriverState struct for
passing the request,
pass the request in the flags parameter to the function.
Signed-off-by: Naphtali Sprei
On 01/18/10 11:15, Markus Armbruster wrote:
Nathan Baumnat...@parenthephobia.org.uk writes:
+static QObject *usb_bus_dev_info(Monitor *mon, DeviceState *qdev)
+{
+USBDevice *dev = DO_UPCAST(USBDevice, qdev, qdev);
+USBBus *bus = usb_bus_from_device(dev);
+return
However, because there are both device properties and bus properties
(really: device properties common to all devices on this bus), their
names can clash. Device properties take precedence (see
qdev_prop_find()). Hmm, qdev_printf() prints even overridden bus
properties, not sure that's
On 01/18/10 11:21, Jan Kiszka wrote:
Gerd Hoffmann wrote:
On 01/15/10 21:07, Jan Kiszka wrote:
The missing '@' broke 'udp::port@:port' parsing.
if (sscanf(p, %64[^:]:%32[^@,]%n, host, port,pos) 2) {
host[0] = 0;
-if (sscanf(p, :%32[^,]%n, port,pos)
On Mon, Jan 18, 2010 at 11:34:59AM +0100, Markus Armbruster wrote:
Michael S. Tsirkin m...@redhat.com writes:
On Sun, Jan 17, 2010 at 04:48:13PM +0200, Naphtali Sprei wrote:
Instead of using the field 'readonly' of the BlockDriverState struct for
passing the request,
pass the request
Alexander, so I assume the following patchset should be enough for you
to implement u3 support, simply by creating your own rwhandler, and
using pci_data_read/write directly there. I have pushed it to a
temporary branch in my tree:
git://git.kernel.org/pub/scm/linux/kernel/git/mst/qemu.git
add helper that can swap values of 4, 2, 1 bytes
Signed-off-by: Michael S. Tsirkin m...@redhat.com
---
bswap.h |6 ++
1 files changed, 6 insertions(+), 0 deletions(-)
diff --git a/bswap.h b/bswap.h
index 4558704..aace9b7 100644
--- a/bswap.h
+++ b/bswap.h
@@ -214,4 +214,10 @@ static
Some users prefer a single callback with length passed as parameter to
using b/w/l callbacks. It would maybe be cleaner to just pass length to
existing callbacks but that's a lot of churn. So for now add a wrapper.
For convenience use uint64_t for address so a single callback can be
used for
Save a ton of code by switching pcihost to use rwhandler.
Signed-off-by: Michael S. Tsirkin m...@redhat.com
---
hw/pci_host.c | 172 +++
hw/pci_host.h |4 +
hw/pci_host_template.h | 109 --
3 files
Gerd Hoffmann wrote:
On 01/18/10 11:21, Jan Kiszka wrote:
Gerd Hoffmann wrote:
On 01/15/10 21:07, Jan Kiszka wrote:
The missing '@' broke 'udp::port@:port' parsing.
if (sscanf(p, %64[^:]:%32[^@,]%n, host, port,pos) 2) {
host[0] = 0;
-if (sscanf(p,
While you are at it, can you also zap this debug leftover?
Looks like there are three of them in qemu_chr_parse_compat - kill
them all?
Yes.
thanks,
Gerd
Signed-off-by: Jan Kiszkajan.kis...@siemens.com
Acked-by: Gerd Hoffmann kra...@redhat.com
-fprintf(stderr, udp
Michael S. Tsirkin wrote:
On Sun, Jan 17, 2010 at 04:48:13PM +0200, Naphtali Sprei wrote:
Instead of using the field 'readonly' of the BlockDriverState struct for
passing the request,
pass the request in the flags parameter to the function.
Signed-off-by: Naphtali Sprei nsp...@redhat.com
On 01/17/2010 05:06 PM, Loïc Minier wrote:
On Sun, Jan 17, 2010, Stefan Weil wrote:
On systems were sdl-config isn't installed, ./configure triggers this
warning:
./configure: 957: sdl-config: not found
which version did you test?
Git master has no sdl-config call at configure:957.
But I get
Michael S. Tsirkin wrote:
On Sun, Jan 17, 2010 at 04:48:15PM +0200, Naphtali Sprei wrote:
Signed-off-by: Naphtali Sprei nsp...@redhat.com
---
block.c |6 --
1 files changed, 0 insertions(+), 6 deletions(-)
diff --git a/block.c b/block.c
index 8def3c4..f90e983 100644
--- a/block.c
In qcow2 (and partly also in general block code), error handling doesn't quite
do what it should. In some places, errors are silently ignored. In other
places, the error code in not passed to the called but rather turned into
false, NULL or -EIO. Besides reporting the right error to the user,
Return the appropriate error value instead of always using EIO. Don't free the
L1 table on errors, we still need it.
Signed-off-by: Kevin Wolf kw...@redhat.com
---
block/qcow2-cluster.c |8
1 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/block/qcow2-cluster.c
Don't assume success but pass the bdrv_pwrite return value on.
Signed-off-by: Kevin Wolf kw...@redhat.com
---
block/qcow2.c |5 +++--
1 files changed, 3 insertions(+), 2 deletions(-)
diff --git a/block/qcow2.c b/block/qcow2.c
index 6622eba..e06f4dd 100644
--- a/block/qcow2.c
+++
Returning 0/-errno allows it to distingush different errors classes. The
cluster offset of newly allocated clusters is now returned in the QCowL2Meta
struct.
Signed-off-by: Kevin Wolf kw...@redhat.com
---
block/qcow2-cluster.c | 28 ++--
block/qcow2.c | 36
Switching to 0/-errno allows it to distinguish different error cases.
Signed-off-by: Kevin Wolf kw...@redhat.com
---
block/qcow2-cluster.c | 30 ++
1 files changed, 18 insertions(+), 12 deletions(-)
diff --git a/block/qcow2-cluster.c b/block/qcow2-cluster.c
index
Return the appropriate error code instead of -EIO.
Signed-off-by: Kevin Wolf kw...@redhat.com
---
block/qcow2-refcount.c |9 ++---
1 files changed, 6 insertions(+), 3 deletions(-)
diff --git a/block/qcow2-refcount.c b/block/qcow2-refcount.c
index 3a2d44a..6f449c6 100644
---
Don't assume -EIO but return the real error.
Signed-off-by: Kevin Wolf kw...@redhat.com
---
block.c | 34 ++
1 files changed, 18 insertions(+), 16 deletions(-)
diff --git a/block.c b/block.c
index 115e591..a4c4953 100644
--- a/block.c
+++ b/block.c
@@ -717,6
If update_refcount fails, try to undo any changes made so far to avoid
inconsistencies in the image file.
Signed-off-by: Kevin Wolf kw...@redhat.com
---
block/qcow2-refcount.c | 32 +---
1 files changed, 25 insertions(+), 7 deletions(-)
diff --git
There's absolutely no problem with updating the refcounts of 0 clusters.
At least snapshot code is doing this and would fail once the result of
update_refcount isn't ignored any more.
Signed-off-by: Kevin Wolf kw...@redhat.com
---
block/qcow2-refcount.c |4 +++-
1 files changed, 3
Now that qcow2_alloc_clusters can return error codes, we must handle them in
the callers of qcow2_alloc_clusters.
Signed-off-by: Kevin Wolf kw...@redhat.com
---
block/qcow2-cluster.c | 18 --
block/qcow2-refcount.c |6 ++
block/qcow2-snapshot.c | 11 ++-
3
update_refcount can return errors that need to be handled by the callers.
Signed-off-by: Kevin Wolf kw...@redhat.com
---
block/qcow2-refcount.c | 31 +++
1 files changed, 23 insertions(+), 8 deletions(-)
diff --git a/block/qcow2-refcount.c b/block/qcow2-refcount.c
Gerd Hoffmann kra...@redhat.com writes:
However, because there are both device properties and bus properties
(really: device properties common to all devices on this bus), their
names can clash. Device properties take precedence (see
qdev_prop_find()). Hmm, qdev_printf() prints even
Gerd Hoffmann kra...@redhat.com writes:
On 01/18/10 11:15, Markus Armbruster wrote:
Nathan Baumnat...@parenthephobia.org.uk writes:
+static QObject *usb_bus_dev_info(Monitor *mon, DeviceState *qdev)
+{
+USBDevice *dev = DO_UPCAST(USBDevice, qdev, qdev);
+USBBus *bus =
Michael S. Tsirkin m...@redhat.com writes:
On Mon, Jan 18, 2010 at 11:34:59AM +0100, Markus Armbruster wrote:
Michael S. Tsirkin m...@redhat.com writes:
On Sun, Jan 17, 2010 at 04:48:13PM +0200, Naphtali Sprei wrote:
Instead of using the field 'readonly' of the BlockDriverState struct for
On 01/18/10 13:34, Markus Armbruster wrote:
However, because there are both device properties and bus properties
(really: device properties common to all devices on this bus), their
names can clash. Device properties take precedence (see
qdev_prop_find()). Hmm, qdev_printf() prints even
On 01/04/2010 09:49 PM, Michael S. Tsirkin wrote:
This adds notifiers for phys memory changes: a set of callbacks that
vhost can register and update kernel accordingly. Down the road, kvm
code can be switched to use these as well, instead of calling kvm code
directly from exec.c as is done now.
Hi
Sorry about this post, tried removing myself from this list about a
week ago, yet I keep getting messages.
Please remove from list
Thanks in advance,
Panarchy
On Mon, Jan 18, 2010 at 03:02:59PM +0200, Avi Kivity wrote:
On 01/04/2010 09:49 PM, Michael S. Tsirkin wrote:
This adds notifiers for phys memory changes: a set of callbacks that
vhost can register and update kernel accordingly. Down the road, kvm
code can be switched to use these as well,
On 01/18/2010 03:52 PM, Michael S. Tsirkin wrote:
On Mon, Jan 18, 2010 at 03:02:59PM +0200, Avi Kivity wrote:
On 01/04/2010 09:49 PM, Michael S. Tsirkin wrote:
This adds notifiers for phys memory changes: a set of callbacks that
vhost can register and update kernel accordingly. Down
On Fri, 15 Jan 2010 13:54:29 -0600
Adam Litke a...@us.ibm.com wrote:
This version improves support for multiple monitors and has been ported up to
HEAD as of 01/14.
Overall review on the Monitor related changes seems ok, but I'm not sure
how I should enable it.
On Fri, 15 Jan 2010 17:25:25 +0100
Markus Armbruster arm...@redhat.com wrote:
Signed-off-by: Markus Armbruster arm...@redhat.com
---
qerror.c |4
qerror.h |3 +++
2 files changed, 7 insertions(+), 0 deletions(-)
diff --git a/qerror.c b/qerror.c
index 5f8fc5d..e7b8ca7
On Mon, 18 Jan 2010 12:23:28 -0200
Luiz Capitulino lcapitul...@redhat.com wrote:
On Fri, 15 Jan 2010 17:25:25 +0100
Markus Armbruster arm...@redhat.com wrote:
Signed-off-by: Markus Armbruster arm...@redhat.com
---
qerror.c |4
qerror.h |3 +++
2 files changed, 7
On Mon, Jan 18, 2010 at 03:58:51PM +0200, Avi Kivity wrote:
On 01/18/2010 03:52 PM, Michael S. Tsirkin wrote:
On Mon, Jan 18, 2010 at 03:02:59PM +0200, Avi Kivity wrote:
On 01/04/2010 09:49 PM, Michael S. Tsirkin wrote:
This adds notifiers for phys memory changes: a set of
On 01/18/2010 04:44 PM, Michael S. Tsirkin wrote:
The point is that clients can be registered at any point.
A client that registered when memory is present needs to
be notified about it.
It looks very expensive.
Shouldn't be hard to optimize ...
It's O(memory size),
Luiz Capitulino lcapitul...@redhat.com writes:
On Fri, 15 Jan 2010 17:25:25 +0100
Markus Armbruster arm...@redhat.com wrote:
Signed-off-by: Markus Armbruster arm...@redhat.com
---
qerror.c |4
qerror.h |3 +++
2 files changed, 7 insertions(+), 0 deletions(-)
diff --git
On Mon, Jan 18, 2010 at 04:52:10PM +0200, Avi Kivity wrote:
On 01/18/2010 04:44 PM, Michael S. Tsirkin wrote:
The point is that clients can be registered at any point.
A client that registered when memory is present needs to
be notified about it.
It looks very expensive.
On Mon, Jan 18, 2010 at 04:52:10PM +0200, Avi Kivity wrote:
On 01/18/2010 04:44 PM, Michael S. Tsirkin wrote:
The point is that clients can be registered at any point.
A client that registered when memory is present needs to
be notified about it.
It looks very expensive.
On 01/18/2010 05:45 PM, Michael S. Tsirkin wrote:
cpu_register_physical_memory_offset already is O(memory size) btw.
Right, but we'd like to replace it with a range API.
Maybe we mandate clients be registered at init-time?
This might be tricky - vhost currently only
On Mon, Jan 18, 2010 at 06:04:40PM +0200, Avi Kivity wrote:
On 01/18/2010 05:45 PM, Michael S. Tsirkin wrote:
cpu_register_physical_memory_offset already is O(memory size) btw.
Right, but we'd like to replace it with a range API.
So, when we do the implementation of notifiers can
Btw, what is the following hack for in do_unassigned_access?
saved_env = env;
env = cpu_single_env;
//...
env = saved_env;
I wonder whether I modify the correct env here:
env = saved_env;
+/* flush neverland mappings created during no-fault mode,
+ so the sequential
On Mon, 18 Jan 2010 15:52:13 +0100
Markus Armbruster arm...@redhat.com wrote:
Luiz Capitulino lcapitul...@redhat.com writes:
On Fri, 15 Jan 2010 17:25:25 +0100
Markus Armbruster arm...@redhat.com wrote:
Signed-off-by: Markus Armbruster arm...@redhat.com
---
qerror.c |4
This is a rework of the prior version which adds definitions
for contemporary processors selected via -cpu model, as an
alternative to the existing use of -cpu qemu64 augmented
with a series of feature flags.
The primary motivation was determination of a least common
denominator within a given
Luiz Capitulino lcapitul...@redhat.com writes:
This commit adds the pci_bridge key to the PCI device QDict,
it also adds support for printing it in the user protocol.
IMPORTANT: This code is being added separately because I could
NOT test it properly. According to Michael Tsirkin, it depends
Luiz Capitulino lcapitul...@redhat.com writes:
The returned QObject is a QList of all buses. Each bus is
represented by a QDict, which has a key with a QList of all
PCI devices attached to it. Each device is represented by
a QDict.
IMPORTANT: support for printing PCI bridge information and
Luiz Capitulino lcapitul...@redhat.com writes:
Hi,
This is a new version of the pci_info() conversion.
changelog
-
V0 - V1
- Coding style fixes
- Make 'BAR' and 'IRQ' keys lowercase
- Add 'irq' key to the documentation
Thanks.
PATCH 3/5 regresses info pci, 4/5 and 5/5
On Mon, Jan 18, 2010 at 4:17 PM, Artyom Tarasenko
atar4q...@googlemail.com wrote:
Btw, what is the following hack for in do_unassigned_access?
saved_env = env;
env = cpu_single_env;
//...
env = saved_env;
env is a host CPU register, see for example target-sparc/exec.h. Code
which
Andreas Färber andreas.faer...@web.de writes:
Hello,
Am 27.10.2009 um 18:38 schrieb malc:
On Tue, 27 Oct 2009, Markus Armbruster wrote:
Mark McLoughlin mar...@redhat.com writes:
On Thu, 2009-10-01 at 09:42 -0500, Justin M. Forbes wrote:
Author: Justin M. Forbes jfor...@redhat.com
Date:
On Mon, Jan 18, 2010 at 10:56 AM, Michael S. Tsirkin m...@redhat.com wrote:
Alexander, so I assume the following patchset should be enough for you
to implement u3 support, simply by creating your own rwhandler, and
using pci_data_read/write directly there. I have pushed it to a
temporary
On Mon, 2010-01-18 at 12:12 -0200, Luiz Capitulino wrote:
On Fri, 15 Jan 2010 13:54:29 -0600
Adam Litke a...@us.ibm.com wrote:
This version improves support for multiple monitors and has been ported up
to
HEAD as of 01/14.
Overall review on the Monitor related changes seems ok, but
On Mon, Jan 18, 2010 at 05:50:35PM +, Blue Swirl wrote:
On Mon, Jan 18, 2010 at 10:56 AM, Michael S. Tsirkin m...@redhat.com wrote:
Alexander, so I assume the following patchset should be enough for you
to implement u3 support, simply by creating your own rwhandler, and
using
Markus Armbruster schrieb:
Stefan Weil w...@mail.berlios.de writes:
The command line options -device, -nodefaults, -readconfig,
-writeconfig had entries for command line help, but
documentation for texi and derived formats (man, html, info)
was missing.
This also required moving @end table
On Mon, Jan 18, 2010 at 07:53:25PM +0200, Michael S. Tsirkin wrote:
On Mon, Jan 18, 2010 at 05:50:35PM +, Blue Swirl wrote:
On Mon, Jan 18, 2010 at 10:56 AM, Michael S. Tsirkin m...@redhat.com
wrote:
Alexander, so I assume the following patchset should be enough for you
to
On Mon, Jan 18, 2010 at 7:41 PM, Michael S. Tsirkin m...@redhat.com wrote:
On Mon, Jan 18, 2010 at 07:53:25PM +0200, Michael S. Tsirkin wrote:
On Mon, Jan 18, 2010 at 05:50:35PM +, Blue Swirl wrote:
On Mon, Jan 18, 2010 at 10:56 AM, Michael S. Tsirkin m...@redhat.com
wrote:
From: Igor V. Kovalenko igor.v.kovale...@gmail.com
sparc64 timer has tick counter which can be set and read,
and tick compare value used as deadline to fire timer interrupt.
The timer is not used as periodic timer, instead deadline
is set each time new timer interrupt is needed.
v1 - v2:
- new
Please send in any agenda items you are interested in covering in
tomorrow's call.
thanks,
-chris
63 matches
Mail list logo