[Qemu-devel] [PATCH 26/27] Implement PAPR VPA functions for pSeries shared processor partitions

2011-03-23 Thread David Gibson
Shared-processor partitions are those where a CPU is time-sliced between partitions, rather than being permanently dedicated to a single partition. qemu emulated partitions, since they are just scheduled with the qemu user process, behave mostly like shared processor partitions. In order to bette

[Qemu-devel] [PATCH 17/27] Implement assorted pSeries hcalls and RTAS methods

2011-03-23 Thread David Gibson
This patch adds several small utility hypercalls and RTAS methods to the pSeries platform emulation. Specifically: * 'display-character' rtas call This just prints a character to the console, it's occasionally used for early debug of the OS. The support includes a hack to make this RTAS call re

[Qemu-devel] [PATCH 25/27] Add a PAPR TCE-bypass mechanism for the pSeries machine

2011-03-23 Thread David Gibson
From: Ben Herrenschmidt Usually, PAPR virtual IO devices use a virtual IOMMU mechanism, TCEs, to mediate all DMA transfers. While this is necessary for some sorts of operation, it can be complex to program and slow for others. This patch implements a mechanism for bypassing TCE translation, tre

[Qemu-devel] [PATCH 15/27] Virtual hash page table handling on pSeries machine

2011-03-23 Thread David Gibson
On pSeries logical partitions, excepting the old POWER4-style full system partitions, the guest does not have direct access to the hardware page table. Instead, the pagetable exists in hypervisor memory, and the guest must manipulate it with hypercalls. However, our current pSeries emulation more

Re: [Qemu-devel] [PATCH] simpletrace: Thread-safe tracing

2011-03-23 Thread Stefan Hajnoczi
On Tue, Mar 22, 2011 at 11:52 PM, Andreas Färber wrote: > Am 28.02.2011 um 10:38 schrieb Stefan Hajnoczi: > >> Trace events outside the global mutex cannot be used with the simple >> trace backend since it is not thread-safe.  There is no check to prevent >> them being enabled so people sometimes

Re: [Qemu-devel] [PATCH] simpletrace: Thread-safe tracing

2011-03-23 Thread Stefan Hajnoczi
On Wed, Mar 23, 2011 at 7:39 AM, Stefan Hajnoczi wrote: > On Tue, Mar 22, 2011 at 11:52 PM, Andreas Färber > wrote: >> Am 28.02.2011 um 10:38 schrieb Stefan Hajnoczi: >> >>> Trace events outside the global mutex cannot be used with the simple >>> trace backend since it is not thread-safe.  There

Re: [Qemu-devel] [PATCH] simpletrace: Thread-safe tracing

2011-03-23 Thread Stefan Hajnoczi
On Wed, Mar 23, 2011 at 7:58 AM, Stefan Hajnoczi wrote: > On Wed, Mar 23, 2011 at 7:39 AM, Stefan Hajnoczi wrote: >> On Tue, Mar 22, 2011 at 11:52 PM, Andreas Färber >> wrote: >>> Am 28.02.2011 um 10:38 schrieb Stefan Hajnoczi: >>> Trace events outside the global mutex cannot be used with

Re: [Qemu-devel] [PATCH 01/11] Add hard build dependency on glib

2011-03-23 Thread Stefan Hajnoczi
On Wed, Mar 23, 2011 at 12:16 AM, Anthony Liguori wrote: > GLib is an extremely common library that has a portable thread implementation > along with tons of other goodies. > > GLib and GObject have a fantastic amount of infrastructure we can leverage in > QEMU including an object oriented program

Re: [Qemu-devel] [PATCH] simpletrace: Thread-safe tracing

2011-03-23 Thread Alexander Graf
On 23.03.2011, at 08:59, Stefan Hajnoczi wrote: > On Wed, Mar 23, 2011 at 7:58 AM, Stefan Hajnoczi wrote: >> On Wed, Mar 23, 2011 at 7:39 AM, Stefan Hajnoczi wrote: >>> On Tue, Mar 22, 2011 at 11:52 PM, Andreas Färber >>> wrote: Am 28.02.2011 um 10:38 schrieb Stefan Hajnoczi: >

Re: [Qemu-devel] [PATCH 01/11] Add hard build dependency on glib

2011-03-23 Thread Roy Tam
2011/3/23 Stefan Hajnoczi : > On Wed, Mar 23, 2011 at 12:16 AM, Anthony Liguori wrote: >> GLib is an extremely common library that has a portable thread implementation >> along with tons of other goodies. >> >> GLib and GObject have a fantastic amount of infrastructure we can leverage in >> QEMU i

Re: [Qemu-devel] [PATCH 11/11] test-vmstate: add test case to verify we don't change VMState

2011-03-23 Thread Stefan Hajnoczi
On Wed, Mar 23, 2011 at 12:16 AM, Anthony Liguori wrote: > +static QObject *read_current_schema(void) > +{ > +    char buffer[65536]; > +    int fd; > +    int ret; > +    size_t offset = 0; > +    ssize_t len; > + > +    ret = system("i386-softmmu/qemu -vmstate-dump > /tmp/schema.json"); Please

Re: [Qemu-devel] [PATCH 01/11] Add hard build dependency on glib

2011-03-23 Thread Stefan Hajnoczi
On Wed, Mar 23, 2011 at 8:19 AM, Roy Tam wrote: > 2011/3/23 Stefan Hajnoczi : >> On Wed, Mar 23, 2011 at 12:16 AM, Anthony Liguori >> wrote: >>> GLib is an extremely common library that has a portable thread >>> implementation >>> along with tons of other goodies. >>> >>> GLib and GObject have

[Qemu-devel] Re: [PATCH 27/27] Add SLOF-based partition firmware for pSeries machine, allowing more boot options

2011-03-23 Thread Benjamin Herrenschmidt
On Wed, 2011-03-23 at 16:30 +1100, David Gibson wrote: > +- SLOF (Slimline Open Firmware) is a free IEEE 1275 Open Firmware > + implementation for certain IBM POWER hardware. The image currently > + in qemu is built from git tag qemu-slof-20110323. > + For those who can't wai

[Qemu-devel] Re: [PATCH v2 1/2] hw/arm_sysctl.c: Add the Versatile Express system registers

2011-03-23 Thread Paolo Bonzini
On 03/22/2011 09:32 PM, Peter Maydell wrote: > Just to make things more complicated, this has been "deprecated"O:-) It has? Your examples below still use it... The case in which the "subsection needed" function returns true should be rare, so the version number should rarely need to be bump

Re: [Qemu-devel] [PATCH 01/11] Add hard build dependency on glib

2011-03-23 Thread Roy Tam
2011/3/23 Stefan Hajnoczi : > On Wed, Mar 23, 2011 at 8:19 AM, Roy Tam wrote: >> 2011/3/23 Stefan Hajnoczi : >>> On Wed, Mar 23, 2011 at 12:16 AM, Anthony Liguori >>> wrote: GLib is an extremely common library that has a portable thread implementation along with tons of other goo

Re: Supsend/resume regression in c995b4 WAS: Re: [Qemu-devel] [PATCH] Fix migration uint8 arrys handled

2011-03-23 Thread Avi Kivity
On 03/22/2011 03:26 PM, Anthony Liguori wrote: Here's how I propose we tackle this. This patch adds a -dump-savevm option that takes a version. It spits out all of the fields we save for a particular version (well, not really, but it should). We also can add type information. The idea is

[Qemu-devel] Re: [PATCH 01/11] Add hard build dependency on glib

2011-03-23 Thread Paolo Bonzini
On 03/23/2011 09:58 AM, Roy Tam wrote: I think I have to change my words. Glib works in win32, but adding Glib to QEMU will bloat the binary size. It adds more dependency on building and the result binary. I wonder if it is a must to add it. That's very far from my definition of "breaking". Pa

Re: [Qemu-devel] [PATCH 01/11] Add hard build dependency on glib

2011-03-23 Thread Stefan Hajnoczi
On Wed, Mar 23, 2011 at 8:58 AM, Roy Tam wrote: > I think I have to change my words. Glib works in win32, but adding > Glib to QEMU will bloat the binary size. It adds more dependency on > building and the result binary. I wonder if it is a must to add it. If we stick to re-implementing cross-pla

[Qemu-devel] Re: [PATCH v2 1/2] hw/arm_sysctl.c: Add the Versatile Express system registers

2011-03-23 Thread Juan Quintela
Peter Maydell wrote: > On 22 March 2011 19:53, Juan Quintela wrote: >> Peter Maydell wrote: >>> Migration from the old version to the new version can be supported >>> if it is OK for the new fields to remain in their default state >>> [XXX is this right? are they zeroed, or do they get the value

[Qemu-devel] Re: [PATCH 04/11] sb16: fix migration quirk

2011-03-23 Thread Juan Quintela
Anthony Liguori wrote: > We seem to migrate the same field twice. It's been this way since Fabrice > committed the original file. Since semantically, we basically ignore the > first > value, make this an unused entry. > > Signed-off-by: Anthony Liguori > --- > hw/sb16.c |3 ++- > 1 files

[Qemu-devel] Re: [PATCH 05/11] vga-isa: fix migration by breaking it

2011-03-23 Thread Juan Quintela
Anthony Liguori wrote: > This is pretty sad. We use the same section name for vga-isa as we do for > vga-pci even though we use separate formats. This breaks the live migration > protocol because we may misinterpret the vga-isa as a vga-pci device. > > vga-isa should use it's own wrapper just li

[Qemu-devel] Re: [PATCH] virtio-serial: don't crash on invalid input

2011-03-23 Thread Michael S. Tsirkin
On Tue, Mar 22, 2011 at 10:25:06PM +0530, Amit Shah wrote: > On (Tue) 22 Mar 2011 [18:32:50], Michael S. Tsirkin wrote: > > Fix crash on invalid input in virtio-serial. > > Discovered by code review, untested. > > > > Signed-off-by: Michael S. Tsirkin > > --- > > hw/virtio-serial-bus.c |3 ++

[Qemu-devel] Re: [PATCH 07/11] eeprom93xx: Use the new hack macro to avoid duplicate field names

2011-03-23 Thread Juan Quintela
Anthony Liguori wrote: > I don't fully understand this hack business but we need field to be unique > so.. > > Signed-off-by: Anthony Liguori > --- > hw/eeprom93xx.c |2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/hw/eeprom93xx.c b/hw/eeprom93xx.c > index cfa695d.

[Qemu-devel] Re: [PATCH 27/27] Add SLOF-based partition firmware for pSeries machine, allowing more boot options

2011-03-23 Thread Benjamin Herrenschmidt
in qemu is built from git tag qemu-slof-20110323. > > + > > For those who can't wait to check it out ... > > David forgot to mention the actual git URL ;-) > > It's on github (https://github.com/dgibson/SLOF) And I just realized it wasn't actually pushed out either :-) I've just fixed that. Cheers, Ben.

Re: [Qemu-devel] [PATCH 11/11] test-vmstate: add test case to verify we don't change VMState

2011-03-23 Thread Peter Maydell
On 23 March 2011 00:16, Anthony Liguori wrote: > +    if (old_version != new_version) { > +        g_error("Version %d of device `%s' is available in QEMU, but schema > still reports %d, please update schema.\n", > +                new_version, device, old_version); > +    } Might be nice for th

Re: [Qemu-devel] [PATCH V11 03/15] xen: Support new libxc calls from xen unstable.

2011-03-23 Thread Alexander Graf
On 01.03.2011, at 19:35, anthony.per...@citrix.com wrote: > From: Anthony PERARD > > This patch updates the libxenctrl calls in Qemu to use the new interface, > otherwise Qemu wouldn't be able to build against new versions of the > library. > > We check libxenctrl version in configure, from Xe

Re: [Qemu-devel] [PATCH V11 02/15] xen: Make Xen build once.

2011-03-23 Thread Alexander Graf
On 01.03.2011, at 19:35, anthony.per...@citrix.com wrote: > From: Anthony PERARD > > xen_domainbuild is now build in libhw. And xen_machine_pv is build only > for i386 targets. > > Signed-off-by: Anthony PERARD > --- > Makefile.objs|3 +++ > Makefile.target |2 +- > hw/xen_

[Qemu-devel] [Bug 740895] [NEW] qemu freeze when loading msdos with EMM386.EXE NOEMS HIGHSCAN

2011-03-23 Thread Guillaume Robin
Public bug reported: Qemu version used : 0.11.2 and 0.14.0 Guest : Ms-Dos 6.2 Host : Ubuntu 10.04 with 2.6.32-29-generic SMP i686 Starting Qemu with command : qemu -hda dos.img -cpu 486 -m 16 When I start msDos under Qemu with the option (in CONFIG.SYS) DEVICE=C:\DOS\EMM386.EXE NOEMS HIGHSCAN th

Re: Supsend/resume regression in c995b4 WAS: Re: [Qemu-devel] [PATCH] Fix migration uint8 arrys handled

2011-03-23 Thread Yoshiaki Tamura
2011/3/23 Avi Kivity : > On 03/22/2011 03:26 PM, Anthony Liguori wrote: >> >> Here's how I propose we tackle this.  This patch adds a -dump-savevm >> option that takes a version.  It spits out all of the fields we save for a >> particular version (well, not really, but it should).  We also can add

Re: [Qemu-devel] [PATCH v2 0/3] Allow ipv6 for migration

2011-03-23 Thread Amit Shah
On (Fri) 18 Mar 2011 [14:22:47], Juan Quintela wrote: > Hi > > > v2: > - create socket_set_reuseaddr() to have it different for > Windows and everything else (Peter Maydell) > - use strerror() instead of perror (Peter Maydell) > - fprintf(): one of them removed, the other one ... > it is comp

Re: [Qemu-devel] [PATCH 1/3] create socket_set_reuseaddr()

2011-03-23 Thread Amit Shah
On (Fri) 18 Mar 2011 [14:22:48], Juan Quintela wrote: > Windows is different than unix, SO_REUSEADDR is the default value > there. Create one function to do it and change all callers. > > Signed-off-by: Juan Quintela > --- > gdbstub.c |6 ++ > migration-tcp.c|4 +--- >

Re: [Qemu-devel] [PATCH 2/3] Use getaddrinfo for migration

2011-03-23 Thread Amit Shah
On (Fri) 18 Mar 2011 [14:22:49], Juan Quintela wrote: > This allows us to use ipv4/ipv6 for migration addresses. > Once there, it also uses /etc/services names (it came free). > > Signed-off-by: Juan Quintela > --- > migration-tcp.c | 51 +++ > net.c | 106 >

Re: [Qemu-devel] [PATCH V11 06/15] xen: Add the Xen platform pci device

2011-03-23 Thread Alexander Graf
On 01.03.2011, at 19:35, anthony.per...@citrix.com wrote: > From: Steven Smith > > Introduce a new emulated PCI device, specific to fully virtualized Xen > guests. The device is necessary for PV on HVM drivers to work. > > Signed-off-by: Steven Smith > Signed-off-by: Anthony PERARD > Signed

Re: Supsend/resume regression in c995b4 WAS: Re: [Qemu-devel] [PATCH] Fix migration uint8 arrys handled

2011-03-23 Thread Anthony Liguori
On 03/23/2011 04:10 AM, Avi Kivity wrote: On 03/22/2011 03:26 PM, Anthony Liguori wrote: Here's how I propose we tackle this. This patch adds a -dump-savevm option that takes a version. It spits out all of the fields we save for a particular version (well, not really, but it should). We al

Re: [Qemu-devel] Re: [PATCH 04/11] sb16: fix migration quirk

2011-03-23 Thread Anthony Liguori
On 03/23/2011 04:51 AM, Juan Quintela wrote: Anthony Liguori wrote: We seem to migrate the same field twice. It's been this way since Fabrice committed the original file. Since semantically, we basically ignore the first value, make this an unused entry. Signed-off-by: Anthony Liguori ---

[Qemu-devel] Re: [PULL (resend, rebase) 1/5] virtio-serial: Use a struct to pass config information from proxy

2011-03-23 Thread Juan Quintela
Amit Shah wrote: > Instead of using a single variable to pass to the virtio_serial_init > function, use a struct so that expanding the number of variables to be > passed on later is easier. > > Signed-off-by: Amit Shah Reviewed-by: Juan Quintela

[Qemu-devel] Re: [PULL (resend, rebase) 2/5] virtio-serial: Disallow generic ports at id 0

2011-03-23 Thread Juan Quintela
Amit Shah wrote: > On (Thu) 10 Mar 2011 [11:39:16], Amit Shah wrote: >> Port 0 is reserved for virtconsole devices for backward compatibility >> with the old -virtioconsole (from qemu 0.12) device type. >> >> libvirt prior to commit 8e28c5d40200b4c5d483bd585d237b9d870372e5 used >> port 0 for gene

[Qemu-devel] Re: [PULL (resend, rebase) 3/5] virtio-serial: Enable ioeventfd

2011-03-23 Thread Juan Quintela
Amit Shah wrote: > Enable ioeventfd for virtio-serial devices by default. Commit > 25db9ebe15125deb32958c6df74996f745edf1f9 lists the benefits of using > ioeventfd. > > Copying a file from guest to host over a virtio-serial channel didn't > show much difference in time or io_exit rate. > > Signed

Re: [Qemu-devel] [PATCH 02/11] vmstate: register all VMStateDescriptions

2011-03-23 Thread Peter Maydell
On 23 March 2011 00:16, Anthony Liguori wrote: > This is a purely mechanical change. > +/* Register the VMState Description to support VMState introspection */ > +static void init_vmstate_description_0(void) > +{ > +    register_vmstate_description(&vmstate_ac97); > +} > + > +vmstate_init(init_vm

Re: [Qemu-devel] Re: [PATCH 07/11] eeprom93xx: Use the new hack macro to avoid duplicate field names

2011-03-23 Thread Anthony Liguori
On 03/23/2011 04:58 AM, Juan Quintela wrote: Anthony Liguori wrote: I don't fully understand this hack business but we need field to be unique so.. Signed-off-by: Anthony Liguori --- hw/eeprom93xx.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/hw/eeprom93xx.c b/

[Qemu-devel] Re: [PULL (resend, rebase) 4/5] virtio-serial-bus: Simplify handle_output() function

2011-03-23 Thread Juan Quintela
Amit Shah wrote: > There's no code change, just re-arrangement to simplify the function > after recent modifications. > > Reported-by: Juan Quintela > Signed-off-by: Amit Shah Reviewed-by: Juan Quintela

Re: [Qemu-devel] [PATCH 02/11] vmstate: register all VMStateDescriptions

2011-03-23 Thread Anthony Liguori
On 03/23/2011 07:30 AM, Peter Maydell wrote: On 23 March 2011 00:16, Anthony Liguori wrote: This is a purely mechanical change. +/* Register the VMState Description to support VMState introspection */ +static void init_vmstate_description_0(void) +{ +register_vmstate_description(&vmstate_ac

Re: [Qemu-devel] OVMF, SeaBIOS & non-CSM based legacy boot

2011-03-23 Thread Gleb Natapov
On Tue, Mar 22, 2011 at 02:53:16PM -0700, Jordan Justen wrote: > 2011/3/22 Gleb Natapov : > > On Tue, Mar 22, 2011 at 12:28:51PM -0700, Jordan Justen wrote: > >> Can this cover a full path like this? > >> /pci@i0cf8/ide@1,1/drive@1/disk@0 => partition0 => /path/abc.efi > >> > > Open Firmware have s

Re: [Qemu-devel] [PATCH V11 09/15] xen: Introduce the Xen mapcache

2011-03-23 Thread Alexander Graf
On 01.03.2011, at 19:35, anthony.per...@citrix.com wrote: > From: Jun Nakajima > > On IA32 host or IA32 PAE host, at present, generally, we can't create > an HVM guest with more than 2G memory, because generally it's almost > impossible for Qemu to find a large enough and consecutive virtual >

Re: [Qemu-devel] [PATCH 11/11] test-vmstate: add test case to verify we don't change VMState

2011-03-23 Thread Anthony Liguori
On 03/23/2011 05:22 AM, Peter Maydell wrote: On 23 March 2011 00:16, Anthony Liguori wrote: +if (old_version != new_version) { +g_error("Version %d of device `%s' is available in QEMU, but schema still reports %d, please update schema.\n", +new_version, device, old_

Re: [Qemu-devel] [PATCH V11 13/15] xen: Initialize event channels and io rings

2011-03-23 Thread Alexander Graf
On 01.03.2011, at 19:35, anthony.per...@citrix.com wrote: > From: Arun Sharma > > Open and bind event channels; map ioreq and buffered ioreq rings. > > Signed-off-by: Arun Sharma > Signed-off-by: Anthony PERARD > Signed-off-by: Stefano Stabellini > Acked-by: Alexander Graf > --- > hw/xen_c

Re: [Qemu-devel] [PATCH V11 00/15] Xen device model support

2011-03-23 Thread Alexander Graf
On 01.03.2011, at 19:35, anthony.per...@citrix.com wrote: > From: Anthony PERARD > > Hi all, > > Here is the few change since the V10: > > - Add braces for blocks with single statement in the clean-up patch; > - the patch that builds Xen only for x86 have been removed, instead, >xen_dom

[Qemu-devel] Re: [PATCH 27/27] Add SLOF-based partition firmware for pSeries machine, allowing more boot options

2011-03-23 Thread David Gibson
> + implementation for certain IBM POWER hardware. The image currently > > > + in qemu is built from git tag qemu-slof-20110323. > > > + > > > > For those who can't wait to check it out ... > > > > David forgot to mention the actual git URL ;-)

Re: [Qemu-devel] [PATCH v22 06/11] libcacard: initial commit

2011-03-23 Thread Alon Levy
On Tue, Mar 22, 2011 at 03:25:11PM +, Stefan Hajnoczi wrote: > On Mon, Mar 21, 2011 at 10:07 PM, Alon Levy wrote: > > +# check for libcacard for smartcard support > > +if test "$smartcard" != "no" ; then > > +    smartcard="yes" > > +    smartcard_cflags="" > > +    # TODO - what's the minimal

Re: [Qemu-devel] [PATCH v22 06/11] libcacard: initial commit

2011-03-23 Thread Alon Levy
On Wed, Mar 23, 2011 at 02:54:04PM +0200, Alon Levy wrote: > On Tue, Mar 22, 2011 at 03:25:11PM +, Stefan Hajnoczi wrote: > > On Mon, Mar 21, 2011 at 10:07 PM, Alon Levy wrote: > > > +# check for libcacard for smartcard support > > > +if test "$smartcard" != "no" ; then > > > +    smartcard="y

[Qemu-devel] [PATCH v23 01/11] trace: move trace objects from Makefile to Makefile.objs

2011-03-23 Thread Alon Levy
--- Makefile | 32 Makefile.objs | 32 2 files changed, 32 insertions(+), 32 deletions(-) diff --git a/Makefile b/Makefile index 89e88b4..209e14d 100644 --- a/Makefile +++ b/Makefile @@ -112,38 +112,6 @@ ui/vnc.o: QEMU_CFL

[Qemu-devel] [PATCH v23 05/11] ccid: add passthru card device

2011-03-23 Thread Alon Levy
The passthru ccid card is a device sitting on the usb-ccid bus and using a chardevice to communicate with a remote device using the VSCard protocol defined in libcacard/vscard_common.h Usage docs available in following patch in docs/ccid.txt Signed-off-by: Alon Levy --- Changes from v20->v21:

[Qemu-devel] [PATCH v23 00/11] usb-ccid

2011-03-23 Thread Alon Levy
This patchset adds three new devices, usb-ccid, ccid-card-passthru and ccid-card-emulated, providing a CCID bus, a simple passthru protocol implementing card requiring a client, and a standalone emulated card. It also introduces a new directory libcaccard with CAC card emulation, CAC is a type of

[Qemu-devel] [PATCH v23 02/11] qemu-thread.h: include inttypes.h

2011-03-23 Thread Alon Levy
qemu-thread.h relies on uint64_t being defined, but doesn't include inttypes.h explicitly. This makes it easier to use it from vscclient (part of libcacard). --- qemu-thread.h |2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/qemu-thread.h b/qemu-thread.h index edc7ab6..0a7

[Qemu-devel] [PATCH v23 08/11] libcacard: add passthru

2011-03-23 Thread Alon Levy
From: Robert Relyea In this mode libcacard doesn't emulate a card, but just passes apdu's straight to the underlying card. Not to be confused with ccid-card-passthru, which doesn't use libcacard at all. So with this functionality in libcacard you can talk directly to the host accessible card, fo

[Qemu-devel] [PATCH v23 11/11] ccid: add docs

2011-03-23 Thread Alon Levy
Add documentation for the usb-ccid device and accompanying two card devices, ccid-card-emulated and ccid-card-passthru. Signed-off-by: Alon Levy --- docs/ccid.txt | 135 + 1 files changed, 135 insertions(+), 0 deletions(-) create mode 100

[Qemu-devel] [PATCH v23 04/11] introduce libcacard/vscard_common.h

2011-03-23 Thread Alon Levy
--- Signed-off-by: Alon Levy v20->v21 changes: (Jes Sorenson review) * license set to 2+ * long comment fixes, remove empty line at eof. * add reference to COPYING v19->v20 changes: * checkpatch.pl v15->v16 changes: Protocol change: * VSCMsgInit capabilities and magic * removed ReaderRe

[Qemu-devel] [PATCH v23 03/11] usb-ccid: add CCID bus

2011-03-23 Thread Alon Levy
A CCID device is a smart card reader. It is a USB device, defined at [1]. This patch introduces the usb-ccid device that is a ccid bus. Next patches will introduce two card types to use it, a passthru card and an emulated card. [1] http://www.usb.org/developers/devclass_docs/DWG_Smart-Card_CCID_R

[Qemu-devel] Re: [PATCH 1/7] virtio-serial: Use a struct to pass config information from proxy

2011-03-23 Thread Juan Quintela
Amit Shah wrote: > Instead of using a single variable to pass to the virtio_serial_init > function, use a struct so that expanding the number of variables to be > passed on later is easier. > > Signed-off-by: Amit Shah Reviewed-by: Juan Quintela

[Qemu-devel] [PATCH v23 07/11] libcacard: add vscclient

2011-03-23 Thread Alon Levy
From: Robert Relyea client to talk to ccid-card-passthru and use smartcard on client to perform actual operations. --- libcacard/Makefile|7 +- libcacard/vscclient.c | 730 + 2 files changed, 736 insertions(+), 1 deletions(-) create mode

[Qemu-devel] [PATCH v23 10/11] ccid: add ccid-card-emulated device

2011-03-23 Thread Alon Levy
This devices uses libcacard (internal) to emulate a smartcard conforming to the CAC standard. It attaches to the usb-ccid bus. Usage instructions (example command lines) are in the following patch in docs/ccid.txt. It uses libcacard which uses nss, so it can work with both hw cards and certificates

[Qemu-devel] [PATCH v23 09/11] libcacard: add docs

2011-03-23 Thread Alon Levy
From: Robert Relyea --- docs/libcacard.txt | 483 1 files changed, 483 insertions(+), 0 deletions(-) create mode 100644 docs/libcacard.txt diff --git a/docs/libcacard.txt b/docs/libcacard.txt new file mode 100644 index 000..5dee6fa ---

[Qemu-devel] Re: [PATCH 5/7] virtio-serial: Don't clear ->have_data() pointer after unplug

2011-03-23 Thread Amit Shah
On (Wed) 23 Mar 2011 [14:33:25], Juan Quintela wrote: > Amit Shah wrote: > > After a port unplug operation, the port->info->have_data() pointer was > > set to NULL. The problem is, the ->info struct is shared by all ports, > > effectively disabling writes to other ports. > > > > Reported-by: juzh

[Qemu-devel] Re: [PATCH] vl.c: Fix compilation failure if CONFIG_SDL isn't defined

2011-03-23 Thread Jes Sorensen
On 03/23/11 14:40, Peter Maydell wrote: > Fix a compilation failure if CONFIG_SDL isn't defined (gcc complained > that the label 'invalid_display' wasn't used). > > Signed-off-by: Peter Maydell Acked-by: Jes Sorensen

[Qemu-devel] Re: [PATCH 5/7] virtio-serial: Don't clear ->have_data() pointer after unplug

2011-03-23 Thread Juan Quintela
Amit Shah wrote: > After a port unplug operation, the port->info->have_data() pointer was > set to NULL. The problem is, the ->info struct is shared by all ports, > effectively disabling writes to other ports. > > Reported-by: juzhang > Signed-off-by: Amit Shah > --- > hw/virtio-console.c |

[Qemu-devel] Re: [PATCH 7/7] char: Prevent multiple devices opening same chardev

2011-03-23 Thread Juan Quintela
Amit Shah wrote: > Prevent: > > -chardev socket,path=/tmp/foo,server,nowait,id=c0 \ > -device virtserialport,chardev=c0,id=vs0 \ > -device virtserialport,chardev=c0,id=vs1 > > Reported-by: Mike Cao > Signed-off-by: Amit Shah > @@ -197,6 +197,10 @@ void qemu_chr_add_handlers(CharDriverState *s,

[Qemu-devel] Re: [PULL #7 0/7] virtio-serial fixes, enhancements

2011-03-23 Thread Juan Quintela
Amit Shah wrote: > Hello, > > This series fixes a few bugs reported against virtio-serial. Please > apply. > > The following changes since commit e0efb993b817564ef84e462ac1fe35f89b57ad7b: > > Fix conversions from pointer to int and vice versa (2011-03-20 21:39:23 > +) > > are available in

[Qemu-devel] Re: [PULL #7 0/7] virtio-serial fixes, enhancements

2011-03-23 Thread Amit Shah
On (Wed) 23 Mar 2011 [14:31:09], Juan Quintela wrote: > Reviewed-by: Juan Quintela Thanks! Anthony, ping again for pulling this. Amit

[Qemu-devel] Re: [PATCH 07/11] eeprom93xx: Use the new hack macro to avoid duplicate field names

2011-03-23 Thread Juan Quintela
Anthony Liguori wrote: > On 03/23/2011 04:58 AM, Juan Quintela wrote: >> Anthony Liguori wrote: >>> I don't fully understand this hack business but we need field to be unique >>> so.. >>> >>> Signed-off-by: Anthony Liguori >>> --- >>> hw/eeprom93xx.c |2 +- >>> 1 files changed, 1 insertio

[Qemu-devel] Re: [PATCH 11/11] test-vmstate: add test case to verify we don't change VMState

2011-03-23 Thread Juan Quintela
Peter Maydell wrote: > On 23 March 2011 00:16, Anthony Liguori wrote: >> +    if (old_version != new_version) { >> +        g_error("Version %d of device `%s' is available in QEMU, but schema >> still reports %d, please update schema.\n", >> +                new_version, device, old_version); >>

[Qemu-devel] Re: [0/27] Implement emulation of pSeries logical partitions (v4)

2011-03-23 Thread Alexander Graf
On 23.03.2011, at 06:30, David Gibson wrote: > This patch series adds a "pseries" machine to qemu, allowing it to > emulate IBM pSeries logical partitions. More specifically it > implements the interface defined by the "PowerPC Architecture Platform > Requirements" document (PAPR, or sPAPR for s

[Qemu-devel] Re: [PATCH 11/11] test-vmstate: add test case to verify we don't change VMState

2011-03-23 Thread Peter Maydell
On 23 March 2011 14:19, Juan Quintela wrote: > Peter Maydell wrote: > ARM people are sending lots of vmstate changes, I guess/hope that > somebody is trying to get it working. > > /me looks at Peter O:-), hint, hint, ... Well, the main thing I care about currently (or did back before Christmas w

[Qemu-devel] Re: [0/27] Implement emulation of pSeries logical partitions (v4)

2011-03-23 Thread Alexander Graf
On 23.03.2011, at 06:30, David Gibson wrote: > This patch series adds a "pseries" machine to qemu, allowing it to > emulate IBM pSeries logical partitions. More specifically it > implements the interface defined by the "PowerPC Architecture Platform > Requirements" document (PAPR, or sPAPR for s

Re: [Qemu-devel] Re: [PATCH 11/11] test-vmstate: add test case to verify we don't change VMState

2011-03-23 Thread Anthony Liguori
On 03/23/2011 09:17 AM, Juan Quintela wrote: Anthony Liguori wrote: On 03/23/2011 05:22 AM, Peter Maydell wrote: On 23 March 2011 00:16, Anthony Liguori wrote: +if (old_version != new_version) { +g_error("Version %d of device `%s' is available in QEMU, but schema still reports

[Qemu-devel] Re: [PATCH 07/11] eeprom93xx: Use the new hack macro to avoid duplicate field names

2011-03-23 Thread Anthony Liguori
On 03/23/2011 09:14 AM, Juan Quintela wrote: Anthony Liguori wrote: On 03/23/2011 04:58 AM, Juan Quintela wrote: Anthony Liguori wrote: I don't fully understand this hack business but we need field to be unique so.. Signed-off-by: Anthony Liguori --- hw/eeprom93xx.c |2 +- 1 files

Re: [Qemu-devel] Re: [0/27] Implement emulation of pSeries logical partitions (v4)

2011-03-23 Thread Alexander Graf
On 23.03.2011, at 15:45, Alexander Graf wrote: > > On 23.03.2011, at 06:30, David Gibson wrote: > >> This patch series adds a "pseries" machine to qemu, allowing it to >> emulate IBM pSeries logical partitions. More specifically it >> implements the interface defined by the "PowerPC Architectu

[Qemu-devel] Re: [0/27] Implement emulation of pSeries logical partitions (v4)

2011-03-23 Thread Alexander Graf
On 23.03.2011, at 06:30, David Gibson wrote: > This patch series adds a "pseries" machine to qemu, allowing it to > emulate IBM pSeries logical partitions. More specifically it > implements the interface defined by the "PowerPC Architecture Platform > Requirements" document (PAPR, or sPAPR for s

Re: [Qemu-devel] Re: [PATCH 11/11] test-vmstate: add test case to verify we don't change VMState

2011-03-23 Thread Peter Maydell
On 23 March 2011 14:52, Anthony Liguori wrote: > I think we ought to merge VMStateDescription into DeviceInfo.  For > compatibility, we probably need a vmstate_alias name since the device names > don't always map 1-1 with the qdev names.  But this should eliminate the > problem of reusing VMStateD

[Qemu-devel] Re: [PATCH 11/11] test-vmstate: add test case to verify we don't change VMState

2011-03-23 Thread Juan Quintela
Anthony Liguori wrote: > On 03/23/2011 05:22 AM, Peter Maydell wrote: >> On 23 March 2011 00:16, Anthony Liguori wrote: >>> +if (old_version != new_version) { >>> +g_error("Version %d of device `%s' is available in QEMU, but >>> schema still reports %d, please update schema.\n", >>>

Re: [Qemu-devel] Re: [PATCH 11/11] test-vmstate: add test case to verify we don't change VMState

2011-03-23 Thread Jan Kiszka
On 2011-03-23 16:00, Peter Maydell wrote: > On 23 March 2011 14:52, Anthony Liguori wrote: >> I think we ought to merge VMStateDescription into DeviceInfo. For >> compatibility, we probably need a vmstate_alias name since the device names >> don't always map 1-1 with the qdev names. But this sho

[Qemu-devel] Re: [PATCH 11/11] test-vmstate: add test case to verify we don't change VMState

2011-03-23 Thread Juan Quintela
Peter Maydell wrote: > On 23 March 2011 14:19, Juan Quintela wrote: >> Peter Maydell wrote: >> ARM people are sending lots of vmstate changes, I guess/hope that >> somebody is trying to get it working. >> >> /me looks at Peter O:-), hint, hint, ... > > Well, the main thing I care about currently

Re: [Qemu-devel] Re: [0/27] Implement emulation of pSeries logical partitions (v4)

2011-03-23 Thread Alexander Graf
On 23.03.2011, at 15:55, Alexander Graf wrote: > > On 23.03.2011, at 15:45, Alexander Graf wrote: > >> >> On 23.03.2011, at 06:30, David Gibson wrote: >> >>> This patch series adds a "pseries" machine to qemu, allowing it to >>> emulate IBM pSeries logical partitions. More specifically it >>

[Qemu-devel] [PATCH] vl.c: Fix compilation failure if CONFIG_SDL isn't defined

2011-03-23 Thread Peter Maydell
Fix a compilation failure if CONFIG_SDL isn't defined (gcc complained that the label 'invalid_display' wasn't used). Signed-off-by: Peter Maydell --- vl.c | 13 +++-- 1 files changed, 7 insertions(+), 6 deletions(-) diff --git a/vl.c b/vl.c index dbb927d..c387f2b 100644 --- a/vl.c +++

[Qemu-devel] Re: [PATCH 11/11] test-vmstate: add test case to verify we don't change VMState

2011-03-23 Thread Peter Maydell
On 23 March 2011 15:13, Juan Quintela wrote: > Peter Maydell wrote: >> There are prebuilt images on Aurelien's website for >> ARM and others, which is the simplest thing: >> http://www.aurel32.net/info/debian_arm_qemu.php > > That images don't migrate for me at all.  Guest got hung after > migrat

[Qemu-devel] Re: [PATCH 11/11] test-vmstate: add test case to verify we don't change VMState

2011-03-23 Thread Juan Quintela
Anthony Liguori wrote: > On 03/23/2011 09:17 AM, Juan Quintela wrote: >> Anthony Liguori wrote: >> We need to fix the ordering problem. > > Dunno what you mean by ordering. vmstate: static const VMStateDescription vmstate_cpu = { .name = "cpu", .version_id = CPU_SAVE_VERSION, .minim

Re: [Qemu-devel] [PATCH v23 00/11] usb-ccid

2011-03-23 Thread Hans de Goede
Ack Series Acked-by: Hans de Goede On 03/23/2011 02:19 PM, Alon Levy wrote: This patchset adds three new devices, usb-ccid, ccid-card-passthru and ccid-card-emulated, providing a CCID bus, a simple passthru protocol implementing card requiring a client, and a standalone emulated card. It also

[Qemu-devel] Re: [PATCH] virtio-serial: don't crash on invalid input

2011-03-23 Thread Amit Shah
On (Tue) 22 Mar 2011 [18:32:50], Michael S. Tsirkin wrote: > Fix crash on invalid input in virtio-serial. > Discovered by code review, untested. > > Signed-off-by: Michael S. Tsirkin Acked-by: Amit Shah Amit

[Qemu-devel] Re: [PATCH] virtio-serial: don't crash on invalid input

2011-03-23 Thread Amit Shah
On (Wed) 23 Mar 2011 [11:56:57], Michael S. Tsirkin wrote: > On Tue, Mar 22, 2011 at 10:25:06PM +0530, Amit Shah wrote: > > On (Tue) 22 Mar 2011 [18:32:50], Michael S. Tsirkin wrote: > > > Fix crash on invalid input in virtio-serial. > > > Discovered by code review, untested. > > > @@ -654,6 +654,

[Qemu-devel] Re: [PATCH 11/11] test-vmstate: add test case to verify we don't change VMState

2011-03-23 Thread Anthony Liguori
On 03/23/2011 10:26 AM, Juan Quintela wrote: Anthony Liguori wrote: On 03/23/2011 09:17 AM, Juan Quintela wrote: Anthony Liguori wrote: We need to fix the ordering problem. Dunno what you mean by ordering. vmstate: static const VMStateDescription vmstate_cpu = { .name = "cpu", .

[Qemu-devel] Re: [PATCH] virtio-serial: don't crash on invalid input

2011-03-23 Thread Michael S. Tsirkin
On Wed, Mar 23, 2011 at 09:39:46PM +0530, Amit Shah wrote: > On (Wed) 23 Mar 2011 [11:56:57], Michael S. Tsirkin wrote: > > On Tue, Mar 22, 2011 at 10:25:06PM +0530, Amit Shah wrote: > > > On (Tue) 22 Mar 2011 [18:32:50], Michael S. Tsirkin wrote: > > > > Fix crash on invalid input in virtio-serial

Re: [Qemu-devel] Re: [PATCH 11/11] test-vmstate: add test case to verify we don't change VMState

2011-03-23 Thread Anthony Liguori
On 03/23/2011 10:00 AM, Peter Maydell wrote: On 23 March 2011 14:52, Anthony Liguori wrote: I think we ought to merge VMStateDescription into DeviceInfo. For compatibility, we probably need a vmstate_alias name since the device names don't always map 1-1 with the qdev names. But this should e

Re: [Qemu-devel] Re: [PATCH 11/11] test-vmstate: add test case to verify we don't change VMState

2011-03-23 Thread Jan Kiszka
On 2011-03-23 17:27, Anthony Liguori wrote: > On 03/23/2011 10:00 AM, Peter Maydell wrote: >> On 23 March 2011 14:52, Anthony Liguori wrote: >>> I think we ought to merge VMStateDescription into DeviceInfo. For >>> compatibility, we probably need a vmstate_alias name since the device names >>> do

Re: [Qemu-devel] Re: [PATCH 11/11] test-vmstate: add test case to verify we don't change VMState

2011-03-23 Thread Peter Maydell
On 23 March 2011 16:27, Anthony Liguori wrote: > Migration uses the VMStateDescription name as a section identifier.  The > section identifiers MUST be unique for a given device.  Otherwise, if both > devices are present, migration fails miserably. So how does it work if you have two devices of t

[Qemu-devel] Re: [PATCH 11/11] test-vmstate: add test case to verify we don't change VMState

2011-03-23 Thread Juan Quintela
Anthony Liguori wrote: > On 03/23/2011 10:26 AM, Juan Quintela wrote: >> Anthony Liguori wrote: >>> On 03/23/2011 09:17 AM, Juan Quintela wrote: Anthony Liguori wrote: >> Once told that, I think that doing a big schema is just wrong, we should >> do an schema for device (or at least for a

[Qemu-devel] EMBEDED QEMU

2011-03-23 Thread Sosthene Grosset-Janin
Hello Team, I am trying to develop a small but ambitious project and would very much welcome any suggestion or assistance: my goal is to create an application that is completely corss-platform and yet contains all the tools that a full scale OS has in order to render content-rich and internet-ena

[Qemu-devel] [Bug 741115] [NEW] Add support of coprocessor cp15, cp14 registers exposion in the embedded gdb server

2011-03-23 Thread Anton Kochkov
Public bug reported: Please add support of exposion of ARM coprocesor registers/logic at the embedded gdb server, for example of cp15, cp14, etc registers. Related project http://jtagarmgdbsrvr.sourceforge.net/index.html Also filled bug in the GDB http://sourceware.org/bugzilla/show_bug.cgi?id

Re: [Qemu-devel] Re: [PATCH 11/11] test-vmstate: add test case to verify we don't change VMState

2011-03-23 Thread Anthony Liguori
On 03/23/2011 11:45 AM, Peter Maydell wrote: On 23 March 2011 16:27, Anthony Liguori wrote: Migration uses the VMStateDescription name as a section identifier. The section identifiers MUST be unique for a given device. Otherwise, if both devices are present, migration fails miserably. So how

[Qemu-devel] [PATCH 2/3] block: Do not cache device size for removable media

2011-03-23 Thread Stefan Hajnoczi
The block layer caches the device size to avoid doing lseek(fd, 0, SEEK_END) every time this value is needed. For removable media the device size becomes stale if a new medium is inserted. This patch simply prevents device size caching for removable media. A smarter solution is to update the cac

[Qemu-devel] [PATCH 3/3] raw-posix: Re-open host CD-ROM after media change

2011-03-23 Thread Stefan Hajnoczi
Piggy-back on the guest CD-ROM polling to poll on the host. Open and close the host CD-ROM file descriptor to ensure we read the new size and not a stale size. Two things are going on here: 1. If hald/udisks is not already polling CD-ROMs on the host then re-opening the CD-ROM causes the host

[Qemu-devel] [PATCH 1/3] trace: Trace bdrv_set_locked()

2011-03-23 Thread Stefan Hajnoczi
It can be handy to know when the guest locks/unlocks the CD-ROM tray. This trace event makes that possible. Signed-off-by: Stefan Hajnoczi --- block.c |2 ++ trace-events |1 + 2 files changed, 3 insertions(+), 0 deletions(-) diff --git a/block.c b/block.c index b476479..8f224b4 10

  1   2   >