Hi,
I know of devices that will enumerate twice, first as one device, then
after a certain setup exchange as another. But that seems to be covered
by the suggestion here, it will just be identicle to two completely different
transports.
Or identify devices by physical port instead of bus add
Hi,
It could be that the implementation decides to not handle synchroneous
commands synchroneous at all. The main difference I'm trying to make
here is between commands which do things to end points which cannot
be done while other packets are in flight (like setting configuration,
or interfac
On Thu, Dec 16, 2010 at 1:45 AM, Benjamin Herrenschmidt
wrote:
> On Mon, 2010-12-13 at 08:32 +0100, Hannes Reinecke wrote:
>> On 12/10/2010 11:14 PM, Paolo Bonzini wrote:
>> > On 11/24/2010 05:50 PM, Christoph Hellwig wrote:
>> >> Btw, it might make sense to split this series into two.
>> >>
>> >>
On Thu, Dec 16, 2010 at 04:08:16PM +0900, Isaku Yamahata wrote:
> On Wed, Dec 15, 2010 at 08:27:49AM -0700, Alex Williamson wrote:
> > On Wed, 2010-12-15 at 11:56 +0200, Michael S. Tsirkin wrote:
> > > On Tue, Dec 14, 2010 at 11:34:53AM -0700, Alex Williamson wrote:
> > > > On Tue, 2010-12-14 at 14
On 12/15/2010 08:00 PM, Luiz Capitulino wrote:
> >
> > Looks like a GUI feature to me,
>
> Really? Can't see how you can build "NMI to all CPUs" from "NMI this
> CPU". Or am I misunderstanding you?
I guess so. Avi referred to 'nmi button on many machines', I assumed he
meant a virtual mac
On Thu, Dec 16, 2010 at 04:37:41PM +0900, Yoshiaki Tamura wrote:
> 2010/11/28 Yoshiaki Tamura :
> > 2010/11/28 Michael S. Tsirkin :
> >> On Thu, Nov 25, 2010 at 03:06:50PM +0900, Yoshiaki Tamura wrote:
> >>> Record ioport event to replay it upon failover.
> >>>
> >>> Signed-off-by: Yoshiaki Tamura
Luiz Capitulino writes:
> On Wed, 15 Dec 2010 18:45:09 +0100
> Markus Armbruster wrote:
>
>> Luiz Capitulino writes:
>>
>> > On Wed, 15 Dec 2010 19:18:32 +0200
>> > Avi Kivity wrote:
[...]
>> >> I'd like to see cpu-index made optional; if not present, nmi all cpus
>> >> (that's what the nmi
2010/12/16 Michael S. Tsirkin :
> On Thu, Dec 16, 2010 at 04:37:41PM +0900, Yoshiaki Tamura wrote:
>> 2010/11/28 Yoshiaki Tamura :
>> > 2010/11/28 Michael S. Tsirkin :
>> >> On Thu, Nov 25, 2010 at 03:06:50PM +0900, Yoshiaki Tamura wrote:
>> >>> Record ioport event to replay it upon failover.
>> >>
On Thu, Dec 16, 2010 at 04:36:16PM +0900, Yoshiaki Tamura wrote:
> 2010/12/3 Yoshiaki Tamura :
> > 2010/12/2 Michael S. Tsirkin :
> >> On Wed, Dec 01, 2010 at 05:03:43PM +0900, Yoshiaki Tamura wrote:
> >>> 2010/11/28 Michael S. Tsirkin :
> >>> > On Sun, Nov 28, 2010 at 08:27:58PM +0900, Yoshiaki Ta
On Thu, Dec 16, 2010 at 06:50:04PM +0900, Yoshiaki Tamura wrote:
> 2010/12/16 Michael S. Tsirkin :
> > On Thu, Dec 16, 2010 at 04:37:41PM +0900, Yoshiaki Tamura wrote:
> >> 2010/11/28 Yoshiaki Tamura :
> >> > 2010/11/28 Michael S. Tsirkin :
> >> >> On Thu, Nov 25, 2010 at 03:06:50PM +0900, Yoshiaki
Sorry for the late review.
jes.soren...@redhat.com writes:
> From: Jes Sorensen
>
> This introduces strtosz_suffix() which allows the caller to specify a
> default suffix in case the non default of MB is wanted.
>
> strtosz() is kept as a wrapper for strtosz_suffix() which keeps it's
> current d
Alon Levy writes:
> Adding a chardev backend for spice, for usage by spice vdagent over a
> with virtio-serial device.
Please put this into PATCH 1/1, so it gets committed. Thanks.
On 12/16/2010 12:48 PM, Luiz Capitulino wrote:
Ok, I didn't know that, but I had another idea: the command could accept
either a single cpu index or a list:
{ "execute": "inject-nmi", "arguments": { "cpus": 2 } }
{ "execute": "inject-nmi", "arguments": { "cpus": [1, 2, 3, 4] } }
This ha
On Thu, 16 Dec 2010 11:03:38 +0200
Avi Kivity wrote:
> On 12/15/2010 08:00 PM, Luiz Capitulino wrote:
> > > >
> > > > Looks like a GUI feature to me,
> > >
> > > Really? Can't see how you can build "NMI to all CPUs" from "NMI this
> > > CPU". Or am I misunderstanding you?
> >
> > I guess s
From: Jes Sorensen
This patch re-factors img_create() moving the code doing the actual
work into block.c where it can be shared with QEMU. This is needed to
be able to create images from QEMU to be used for live snapshots.
Signed-off-by: Jes Sorensen
---
block.c| 144 +
From: Jes Sorensen
Signed-off-by: Jes Sorensen
---
block.c |7 +++
1 files changed, 7 insertions(+), 0 deletions(-)
diff --git a/block.c b/block.c
index 765f9f3..027dc6a 100644
--- a/block.c
+++ b/block.c
@@ -2769,6 +2769,13 @@ int bdrv_img_create(const char *filename, const char
*fmt
From: Jes Sorensen
Hi,
This set of patches re-factors img_create() and moves the core part of
it into block.c so it can be accessed from qemu as well as
qemu-img. The second patch adds basic live snapshots support to the
code, however only snapshots to external QCOW2 images is supported for
now.
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
---
Makefile.objs
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
From: Jes Sorensen
The monitor command is:
snapshot_blkdev [snapshot-file] [format]
Default format is qcow2. For now snapshots without a snapshot-file, eg
internal snapshots, are not supported.
Signed-off-by: Jes Sorensen
---
blockdev.c | 61 +++
On Thu, 16 Dec 2010 12:51:14 +0200
Avi Kivity wrote:
> On 12/16/2010 12:48 PM, Luiz Capitulino wrote:
> > Ok, I didn't know that, but I had another idea: the command could accept
> > either a single cpu index or a list:
> >
> >
> >{ "execute": "inject-nmi", "arguments": { "cpus": 2 } }
> >
>
On Wed, 15 Dec 2010 16:58:03 -0600
Adam Litke wrote:
> On Wed, 2010-12-15 at 15:39 -0200, Luiz Capitulino wrote:
> > On Wed, 15 Dec 2010 16:20:05 +0900
> > "Ken'ichi Ohmichi" wrote:
> >
> > >
> > > Hi,
> > >
> > > I tried to get the memory stats by using virDomainMemoryStats() of
> > > libvi
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
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
Am 16.12.2010 12:04, schrieb jes.soren...@redhat.com:
> From: Jes Sorensen
>
> This patch re-factors img_create() moving the code doing the actual
> work into block.c where it can be shared with QEMU. This is needed to
> be able to create images from QEMU to be used for live snapshots.
>
> Signe
On Thu, Dec 16, 2010 at 11:04 AM, wrote:
> From: Jes Sorensen
>
> Signed-off-by: Jes Sorensen
> ---
> block.c | 7 +++
> 1 files changed, 7 insertions(+), 0 deletions(-)
>
> diff --git a/block.c b/block.c
> index 765f9f3..027dc6a 100644
> --- a/block.c
> +++ b/block.c
> @@ -2769,6 +2769
Add documentation for the usb-ccid device and accompanying two card
devices, ccid-card-emulated and ccid-card-passthru.
---
docs/ccid.txt | 125 ++
docs/libcacard.txt | 483
2 files changed, 125 insertions(+), 483 deletions(-)
Implement the correct NaN propagation rules for ARM targets by
providing an appropriate pickNaN function.
Signed-off-by: Peter Maydell
---
fpu/softfloat-specialize.h | 23 +++
1 files changed, 23 insertions(+), 0 deletions(-)
diff --git a/fpu/softfloat-specialize.h b/fpu/s
Am 16.12.2010 12:04, schrieb jes.soren...@redhat.com:
> From: Jes Sorensen
>
> The monitor command is:
> snapshot_blkdev [snapshot-file] [format]
>
> Default format is qcow2. For now snapshots without a snapshot-file, eg
> internal snapshots, are not supported.
>
> Signed-off-by: Jes Sorensen
IEEE754 doesn't specify precisely what NaN should be returned as the
result of an operation on two input NaNs. The existing softfloat code
hardwires x87 propagation rules. This patchset abstracts out the code
in the various propagateFloat*NaN() functions which picks a NaN to
return, so that it can
On 12/16/10 12:45, Kevin Wolf wrote:
> Am 16.12.2010 12:04, schrieb jes.soren...@redhat.com:
>> From: Jes Sorensen
>>
>> The monitor command is:
>> snapshot_blkdev [snapshot-file] [format]
>>
>> Default format is qcow2. For now snapshots without a snapshot-file, eg
>> internal snapshots, are not
Am 16.12.2010 12:04, schrieb jes.soren...@redhat.com:
> From: Jes Sorensen
>
> Signed-off-by: Jes Sorensen
> ---
> block.c |7 +++
> 1 files changed, 7 insertions(+), 0 deletions(-)
>
> diff --git a/block.c b/block.c
> index 765f9f3..027dc6a 100644
> --- a/block.c
> +++ b/block.c
> @@
Luiz Capitulino writes:
> On Thu, 16 Dec 2010 11:03:38 +0200
> Avi Kivity wrote:
>
>> On 12/15/2010 08:00 PM, Luiz Capitulino wrote:
>> > > >
>> > > > Looks like a GUI feature to me,
>> > >
>> > > Really? Can't see how you can build "NMI to all CPUs" from "NMI this
>> > > CPU". Or am I mi
Am 10.12.2010 15:47, schrieb Stefan Hajnoczi:
> We register the vm change state handler in a PCI BAR map() function.
> This function can be called multiple times throughout the lifetime of a
> PCI IDE device. This results in duplicate vm change state handlers
> being register, none of which are ev
On 12/16/10 12:35, Kevin Wolf wrote:
> Am 16.12.2010 12:04, schrieb jes.soren...@redhat.com:
>> +
>> +backing_fmt = get_option_parameter(param, BLOCK_OPT_BACKING_FMT);
>> +if (backing_fmt && backing_fmt->value.s) {
>> +if (!bdrv_find_format(backing_fmt->value.s)) {
>> +e
On 12/16/10 12:06, Alon Levy wrote:
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/develope
On 12/16/10 12:06, Alon Levy wrote:
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
Acked-by: Gerd
Adding a chardev backend for spice, for usage by spice vdagent in
conjunction with a properly named virtio-serial device.
---
Makefile.objs |2 +-
qemu-char.c |7 ++
qemu-config.c |9 ++
qemu-options.hx | 18 -
spice-qemu-char.c | 222
IEEE754 doesn't specify precisely what NaN should be returned as
the result of an operation on two input NaNs. This is therefore
target-specific. Abstract out the code in propagateFloat*NaN()
which was implementing the x87 propagation rules, so that it
can be easily replaced on a per-target basis.
On 12/15/2010 08:04 PM, Michael S. Tsirkin wrote:
This assuming upstream developers do not care about downstreams.
To give a chance for downstream to cherry-pick changes, upstream
should use subsections instead of version ids too.
Then version ids should be deprecated altogether. Nothing again
On 12/16/2010 01:47 PM, Markus Armbruster wrote:
>
> This has the feature of injecting the nmi in just some cpus, although I'm
> not sure this is going to be desired/useful.
Use case for NMI-ing a subset of the CPUs?
Heck, use case for anything else but "NMI all"?
Exactly.
--
error compili
From: Jes Sorensen
The monitor command is:
snapshot_blkdev [snapshot-file] [format]
Default format is qcow2. For now snapshots without a snapshot-file, eg
internal snapshots, are not supported.
Signed-off-by: Jes Sorensen
---
blockdev.c | 62 +++
From: Jes Sorensen
This patch re-factors img_create() moving the code doing the actual
work into block.c where it can be shared with QEMU. This is needed to
be able to create images from QEMU to be used for live snapshots.
Signed-off-by: Jes Sorensen
---
block.c| 141 +
From: Jes Sorensen
Hi,
This set of patches re-factors img_create() and moves the core part of
it into block.c so it can be accessed from qemu as well as
qemu-img. The second patch adds basic live snapshots support to the
code, however only snapshots to external QCOW2 images is supported for
now.
On 12/16/10 12:06, Alon Levy wrote:
libcacard emulates a Common Access Card (CAC) which is a standard
for smartcards. It is used by the emulated ccid card introduced in
a following patch. Docs are available in docs/libcacard.txt
Looks good to me, although I'm not a crypto expert. Most of the
From: Jes Sorensen
Kevin suggested to have bdrv_img_create() return proper -errno values
on error.
Signed-off-by: Jes Sorensen
---
block.c | 23 ++-
1 files changed, 10 insertions(+), 13 deletions(-)
diff --git a/block.c b/block.c
index 0c14eee..fe07d0b 100644
--- a/bloc
On 12/16/10 12:06, Alon Levy wrote:
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 wo
From: Jes Sorensen
Signed-off-by: Jes Sorensen
---
block.c | 15 +++
1 files changed, 11 insertions(+), 4 deletions(-)
diff --git a/block.c b/block.c
index a48b30c..0c14eee 100644
--- a/block.c
+++ b/block.c
@@ -2764,7 +2764,7 @@ int bdrv_img_create(const char *filename, const ch
On 12/16/10 12:06, Alon Levy wrote:
Add documentation for the usb-ccid device and accompanying two card
devices, ccid-card-emulated and ccid-card-passthru.
---
docs/ccid.txt | 125 ++
docs/libcacard.txt | 483
Guess that
On Thu, 16 Dec 2010 14:50:08 +0200
Avi Kivity wrote:
> On 12/16/2010 01:47 PM, Markus Armbruster wrote:
> > >
> > > This has the feature of injecting the nmi in just some cpus, although I'm
> > > not sure this is going to be desired/useful.
> >
> > Use case for NMI-ing a subset of the CPUs?
> >
On 12/16/2010 03:09 PM, Luiz Capitulino wrote:
On Thu, 16 Dec 2010 14:50:08 +0200
Avi Kivity wrote:
> On 12/16/2010 01:47 PM, Markus Armbruster wrote:
> > >
> > > This has the feature of injecting the nmi in just some cpus, although
I'm
> > > not sure this is going to be desired/use
On Thu, 16 Dec 2010 15:11:50 +0200
Avi Kivity wrote:
> On 12/16/2010 03:09 PM, Luiz Capitulino wrote:
> > On Thu, 16 Dec 2010 14:50:08 +0200
> > Avi Kivity wrote:
> >
> > > On 12/16/2010 01:47 PM, Markus Armbruster wrote:
> > > > >
> > > > > This has the feature of injecting the nmi in ju
On 12/16/10 12:29, Alon Levy wrote:
Adding a chardev backend for spice, for usage by spice vdagent in
conjunction with a properly named virtio-serial device.
Usage example would be nice here.
+#ifdef CONFIG_SPICE
+#include "spice-qemu-char.h"
+#endif
#ifdef can be dropped.
+#ifdef CONFIG_
On Thu, Dec 16, 2010 at 12:29 PM, Kevin Wolf wrote:
> Am 10.12.2010 15:47, schrieb Stefan Hajnoczi:
>> We register the vm change state handler in a PCI BAR map() function.
>> This function can be called multiple times throughout the lifetime of a
>> PCI IDE device. This results in duplicate vm ch
From: Jes Sorensen
Signed-off-by: Jes Sorensen
---
qemu-img.c | 128 +--
1 files changed, 63 insertions(+), 65 deletions(-)
diff --git a/qemu-img.c b/qemu-img.c
index 603bdb3..d3921b6 100644
--- a/qemu-img.c
+++ b/qemu-img.c
@@ -26,6 +26
Call error_set_progname during the qemu-img initialization, so that error
messages printed with error_report() use the right prefix.
Signed-off-by: Kevin Wolf
---
qemu-img.c |3 +++
1 files changed, 3 insertions(+), 0 deletions(-)
diff --git a/qemu-img.c b/qemu-img.c
index 603bdb3..0ff179f
Am 16.12.2010 13:52, schrieb jes.soren...@redhat.com:
> From: Jes Sorensen
>
> Hi,
>
> This set of patches re-factors img_create() and moves the core part of
> it into block.c so it can be accessed from qemu as well as
> qemu-img. The second patch adds basic live snapshots support to the
> code,
Am 16.12.2010 14:31, schrieb jes.soren...@redhat.com:
> From: Jes Sorensen
>
> Signed-off-by: Jes Sorensen
Thanks, applied to the block branch.
Kevin
2010/12/16 Michael S. Tsirkin :
> On Thu, Dec 16, 2010 at 04:36:16PM +0900, Yoshiaki Tamura wrote:
>> 2010/12/3 Yoshiaki Tamura :
>> > 2010/12/2 Michael S. Tsirkin :
>> >> On Wed, Dec 01, 2010 at 05:03:43PM +0900, Yoshiaki Tamura wrote:
>> >>> 2010/11/28 Michael S. Tsirkin :
>> >>> > On Sun, Nov 28
On 12/16/10 15:13, Kevin Wolf wrote:
> Call error_set_progname during the qemu-img initialization, so that error
> messages printed with error_report() use the right prefix.
>
> Signed-off-by: Kevin Wolf
Acked-by: Jes Sorensen
> ---
> qemu-img.c |3 +++
> 1 files changed, 3 insertions(+)
It's an indirect call to qemu_malloc, which never returns an error.
Signed-off-by: Kevin Wolf
---
hw/xen_disk.c | 17 ++---
qemu-img.c|5 +
qemu-io.c |2 --
qemu-nbd.c|2 --
4 files changed, 7 insertions(+), 19 deletions(-)
diff --git a/hw/xen_disk.c b/
On Thu, Dec 16, 2010 at 11:28:46PM +0900, Yoshiaki Tamura wrote:
> 2010/12/16 Michael S. Tsirkin :
> > On Thu, Dec 16, 2010 at 04:36:16PM +0900, Yoshiaki Tamura wrote:
> >> 2010/12/3 Yoshiaki Tamura :
> >> > 2010/12/2 Michael S. Tsirkin :
> >> >> On Wed, Dec 01, 2010 at 05:03:43PM +0900, Yoshiaki T
It's an indirect call to qemu_malloc, which never returns an error.
Signed-off-by: Kevin Wolf
---
v2:
- Fixed bdrv_open failure case in xen_disk
hw/xen_disk.c | 17 ++---
qemu-img.c|5 +
qemu-io.c |2 --
qemu-nbd.c|2 --
4 files changed, 7 insertions(+
Am 16.12.2010 02:45, schrieb Benjamin Herrenschmidt:
> On Mon, 2010-12-13 at 08:32 +0100, Hannes Reinecke wrote:
>> On 12/10/2010 11:14 PM, Paolo Bonzini wrote:
>>> On 11/24/2010 05:50 PM, Christoph Hellwig wrote:
Btw, it might make sense to split this series into two.
Patches 1 to 1
Hi all!
I am a student, trying to understand QEMU, specifically TCG
translation/execution.
After spending much time on the code I still have big doubts. I think my
doubts are due to the classic idea I have of an emulator.
Actually as a student, I've never developed even a simple classic
emu
On 16 December 2010 15:20, Stefano Bonifazi wrote:
> so I thought tcg_qemu_tb_exec "function" should do the work of executing the
> translated binary in the host.
> But then I found out it is just a define in tcg.h:
>
>> #define tcg_qemu_tb_exec(tb_ptr) ((long REGPARM (*)(void
>> *))code_gen_prolo
Am 10.12.2010 16:00, schrieb Christoph Hellwig:
> This patchset adds support for the ATA TRIM and SCSI WRITE SAME with
> unmap commands, which allow reclaiming free space from a backing image.
>
> The user facing implementation is pretty complete, but not really
> efficient because the underlying
Am 10.12.2010 16:00, schrieb Christoph Hellwig:
> Support discards via the WRITE SAME command with the unmap bit set, and
> tell the initiator about the support for it via the block limit and the
> new thin provisioning EVPD pages. Also fix the comment which incorrectly
> describedthe block limits
We register the vm change state handler in a PCI BAR map() function.
This function can be called multiple times throughout the lifetime of a
PCI IDE device. This results in duplicate vm change state handlers
being register, none of which are ever unregistered.
Instead, register the vm change stat
Hi Stefano
I'll try to share what I know about TCG..
On Thu, Dec 16, 2010 at 22:20, Stefano Bonifazi
wrote:
> Actually as a student, I've never developed even a simple classic emulator
> myself,
you're not alone...trust me.. :)
>but in my idea it should follow this flow:
> 1) Fetch
2010/12/16 Michael S. Tsirkin :
> On Thu, Dec 16, 2010 at 11:28:46PM +0900, Yoshiaki Tamura wrote:
>> 2010/12/16 Michael S. Tsirkin :
>> > On Thu, Dec 16, 2010 at 04:36:16PM +0900, Yoshiaki Tamura wrote:
>> >> 2010/12/3 Yoshiaki Tamura :
>> >> > 2010/12/2 Michael S. Tsirkin :
>> >> >> On Wed, Dec 0
From: Jes Sorensen
In addition this adds missing braces to the function to be consistent
with the coding style.
Signed-off-by: Jes Sorensen
---
block/qcow2.c | 61
1 files changed, 43 insertions(+), 18 deletions(-)
diff --git a/block/
From: Jes Sorensen
Hi,
These two patches tries to clean up the qcow2 code a little. First
makes the function names consistent, ie. we shouldn't have qcow_
functions in the qcow2 code. Second tries to add proper errno return
values to qcow2_open().
Jes Sorensen (2):
block/qcow2.c: rename qcow_
From: Jes Sorensen
It doesn't really make sense for functions in qcow2.c to be named
qcow_ so convert the names to match correctly.
Signed-off-by: Jes Sorensen
---
block/qcow2-cluster.c |6 +-
block/qcow2-snapshot.c |6 +-
block/qcow2.c | 210 +---
Hello,
Is a bad idea if I run multiples "qemu" vms pointing to the same disk
img using "-snapshot" parameter? What kind of problem may I have?
Thanks,
On Thu, Dec 16, 2010 at 9:50 AM, Yoshiaki Tamura
wrote:
> 2010/12/16 Michael S. Tsirkin :
>> On Thu, Dec 16, 2010 at 04:37:41PM +0900, Yoshiaki Tamura wrote:
>>> 2010/11/28 Yoshiaki Tamura :
>>> > 2010/11/28 Michael S. Tsirkin :
>>> >> On Thu, Nov 25, 2010 at 03:06:50PM +0900, Yoshiaki Tamura wrot
On Thu, Dec 16, 2010 at 4:21 PM, Amador Pahim wrote:
> Is a bad idea if I run multiples "qemu" vms pointing to the same disk
> img using "-snapshot" parameter? What kind of problem may I have?
It should work fine. -snapshot means that QEMU creates a temporary
qcow2 file backed by the disk image
Hello Stefan,
Thank you for your answer. Just one more question: If, while my
"snapshot" vms are running, the main disk is modified by a non
"snapshot" vm? For example, installing some extra software.. this can
freeze vms or something?
Regards,
Pahim
On Thu, Dec 16, 2010 at 2:28 PM, Stefan Hajno
On Thu, Dec 16, 2010 at 04:43:20PM +0100, Kevin Wolf wrote:
> The SCSI changes seem to apply, but the rest conflicts with recent
> changes, most notably AHCI. Can you rebase on top of the block branch?
I've tried to, but with the maze of pointer indirections I'm totally
lost for now. I'll have to
On Wed, Dec 15, 2010 at 12:59 PM, Stefan Hajnoczi wrote:
> On Wed, Dec 15, 2010 at 12:14 PM, Michael S. Tsirkin wrote:
>> On Wed, Dec 15, 2010 at 11:42:12AM +, Stefan Hajnoczi wrote:
>>> Are you happy with this patchset if I remove virtio-net-pci
>>> ioeventfd=on|off so only virtio-blk-pci ha
On Thu, Dec 16, 2010 at 04:48:15PM +0100, Kevin Wolf wrote:
> > +scsi_req_set_status(r, GOOD, NO_SENSE);
> > +scsi_req_complete(&r->req);
> > +scsi_remove_request(r);
>
> Isn't this the same as scsi_command_complete()?
Yes.
>
> > +return 0;
>
> And if you break;
On Thu, Dec 16, 2010 at 02:21:16PM +0100, Gerd Hoffmann wrote:
> On 12/16/10 12:29, Alon Levy wrote:
> >Adding a chardev backend for spice, for usage by spice vdagent in
> >conjunction with a properly named virtio-serial device.
>
> Usage example would be nice here.
>
> >+#ifdef CONFIG_SPICE
> >+
On Thu, Dec 16, 2010 at 02:01:34PM +0100, Gerd Hoffmann wrote:
> On 12/16/10 12:06, Alon Levy wrote:
> >Add documentation for the usb-ccid device and accompanying two card
> >devices, ccid-card-emulated and ccid-card-passthru.
> >---
> > docs/ccid.txt | 125 ++
> > docs/libcacard
Hi,
+//#define SPICE_QEMU_CHAR_USE_IOCTL
Why is this disabled?
Does it depend on the chardev patches from Amit?
There was a long discussion that concluded we don't want IOCTL's at all,
and that there should be some other mechanism for connection state
communication between the two sides.
On Thu, Dec 16, 2010 at 4:34 PM, Amador Pahim wrote:
> Thank you for your answer. Just one more question: If, while my
> "snapshot" vms are running, the main disk is modified by a non
> "snapshot" vm? For example, installing some extra software.. this can
> freeze vms or something?
Correct, it is
On Thu, Dec 16, 2010 at 2:44 PM, Kevin Wolf wrote:
> It's an indirect call to qemu_malloc, which never returns an error.
>
> Signed-off-by: Kevin Wolf
> ---
>
> v2:
> - Fixed bdrv_open failure case in xen_disk
>
> hw/xen_disk.c | 17 ++---
> qemu-img.c | 5 +
> qemu-io.c
On Mon, Dec 13, 2010 at 4:29 PM, Kevin Wolf wrote:
> diff --git a/block/qcow2.c b/block/qcow2.c
> index 537c479..e445913 100644
> --- a/block/qcow2.c
> +++ b/block/qcow2.c
> @@ -136,6 +136,20 @@ static int qcow_read_extensions(BlockDriverState *bs,
> uint64_t start_offset,
> return 0;
> }
>
Ok, I think this might actually make everyone happy, but I've been
known to be wrong about that many times before. Juan challenged me
to find an rtl8139 migration scenario that fails when hotplug is
not involved (and not switch device creation order since that's a
usage bug). I couldn't come up w
To query whether migration is active.
Signed-off-by: Marcelo Tosatti
Index: qemu-kvm-block-copy/migration.c
===
--- qemu-kvm-block-copy.orig/migration.c
+++ qemu-kvm-block-copy/migration.c
@@ -448,3 +448,13 @@ int migrate_fd_close(v
On 7 December 2010 15:50, Peter Maydell wrote:
> The following changes since commit 2c90fe2b71df2534884bce96d90cbfcc93aeedb8:
> Kirill Batuzov (1):
> Speedup 'tb_find_slow' by using the same heuristic as during
> memory page lookup
>
> are available in the git repository at:
>
> git://git
Add support for live block copy. This is similar (and based on), block
migration, except it is performed without VM migration.
Am 16.12.2010 18:45, schrieb Stefan Hajnoczi:
On Thu, Dec 16, 2010 at 4:34 PM, Amador Pahim wrote:
Thank you for your answer. Just one more question: If, while my
"snapshot" vms are running, the main disk is modified by a non
"snapshot" vm? For example, installing some extra software.. this can
Every system should have some sort of main system bus,
so sysbus_get_default should always return a valid bus.
Without this patch, at least mipssim and malta no longer
start but raise a null pointer access exception (caused by
commit ec990eb622ad46df5ddcb1e94c418c271894d416).
Cc: Anthony Liguori
This patchset adds support for the SCSI WRITE SAME with unmap command,
which allows reclaiming free space from a backing image.
The user facing implementation is pretty complete, but not really
efficient because the underlying bdrv_discard implementation doesn't
use the aio implementation yet. Th
Add a new bdrv_discard method to free blocks in a mapping image, and a new
drive property to set the granularity for these discard. If no discard
granularity support is set discard support is disabled.
Signed-off-by: Christoph Hellwig
Index: qemu/block.c
Support discards via the WRITE SAME command with the unmap bit set, and
tell the initiator about the support for it via the block limit and the
new thin provisioning EVPD pages. Also fix the comment which incorrectly
describedthe block limits EVPD page.
Signed-off-by: Christoph Hellwig
Index: q
Add support to discard blocks in a raw image residing on an XFS filesystem
by calling the XFS_IOC_UNRESVSP64 ioctl to punch holes. Support for other
hole punching mechanisms can be added when they become available.
Signed-off-by: Christoph Hellwig
Index: qemu/block/raw-posix.c
=
Create a trivial interface to track whether the machine has been
modified since boot. Adding or removing devices will trigger this
to return true. An example usage scenario for such an interface is
the rtl8139 driver which includes a cpu_register_io_memory() value
in it's migration stream. For t
Signed-off-by: Marcelo Tosatti
Index: qemu-kvm-block-copy/migration.c
===
--- qemu-kvm-block-copy.orig/migration.c
+++ qemu-kvm-block-copy/migration.c
@@ -19,6 +19,7 @@
#include "block.h"
#include "qemu_socket.h"
#include "block-m
Add support for live block copy.
Signed-off-by: Marcelo Tosatti
Index: qemu-kvm/block-copy.c
===
--- /dev/null
+++ qemu-kvm/block-copy.c
@@ -0,0 +1,728 @@
+/*
+ * QEMU live block copy
+ *
+ * Copyright (C) 2010 Red Hat Inc.
+ *
+ *
1 - 100 of 114 matches
Mail list logo