On Sat, Jul 16, 2011 at 12:27 AM, Luiz Capitulino
wrote:
> On Fri, 15 Jul 2011 09:15:18 +0800
> Zhi Yong Wu wrote:
>
>> On Fri, Jul 15, 2011 at 4:00 AM, Michael Roth
>> wrote:
>> > This is Set 3/3 of the QAPI+QGA patchsets.
>> >
>> > These patches apply on top of qapi-backport-set2-v6, and can
On 09/06/2011 08:50 PM, malc wrote:
> Correct me if i'm wrong, previously the code worked like this:
>
> mov tmp, 0
> sub tmp, 1
> sar r, r, tmp
>
> Still UB as far as TCG is concerned but since no immediates are involved
> things worked, now, with constant folding, we are asked to sar by -1
> d
At 09/06/2011 03:45 PM, Avi Kivity Write:
> On 09/06/2011 06:06 AM, Wen Congyang wrote:
>> > Use the uio driver -
>> > http://docs.blackfin.uclinux.org/kernel/generated/uio-howto/. You
>> just
>> > mmap() the BAR from userspace and play with it.
>>
>> When I try to bind ivshmem to uio_pci_gener
Hi,
Sometime back I posted a patch for fixing x86 CPU topology (
http://lists.gnu.org/archive/html/qemu-devel/2011-08/msg02022.html).
Here is the next version of the fix which addresses all but one
comment received during that post.
- Fixed code style issues
- Ensured that the fix doesn't break T
Add --disable-debug-symbols to disable building with debug
symbols and --optflags to override the optimization flags
passed to the compiler.
---
configure | 18 +++---
1 files changed, 15 insertions(+), 3 deletions(-)
diff --git a/configure b/configure
index c3044c7..c7948b6 100755
The Buildbot has detected a new failure on builder qmp_x86_64_debian_6_0 while
building qemu.
Full details are available at:
http://buildbot.b1-systems.de/qemu/builders/qmp_x86_64_debian_6_0/builds/23
Buildbot URL: http://buildbot.b1-systems.de/qemu/
Buildslave for this Build: yuzuki
Build Rea
On 09/04/2011 12:38 PM, Michael S. Tsirkin wrote:
On Thu, Sep 01, 2011 at 11:00:56PM -0400, Stefan Berger wrote:
initstate_fd=
initstate_base64=on/off (or base64/bin if you really expect
more formats in the future)
and use qemu routines to get the fd so they can be
passed through the mo
On 09/04/2011 12:58 PM, Michael S. Tsirkin wrote:
On Thu, Sep 01, 2011 at 10:23:51PM -0400, Stefan Berger wrote:
Checks are added that test
- whether encryption is supported follwing the revision of the directory
structure (rev>= 2)
You never generate rev 1 code, right?
I did this in the pr
On 09/04/2011 03:32 PM, Michael S. Tsirkin wrote:
On Thu, Sep 01, 2011 at 09:53:40PM -0400, Stefan Berger wrote:
Generally, what all other devices do is perform validation
as the last step in migration when device state
is restored. On failure, management can decide what to do:
retry migration o
The print function is listed in the vector table as 0xbfc00808. The backedge
of the loop is improperly encoded, and so jumps to 0xb814 instead of the
desired 0xbfc00814.
Signed-off-by: Andrew Drake
---
hw/mips_malta.c |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a
Am 01.09.2011 um 16:56 schrieb Fabien Chouteau :
>
> Signed-off-by: Fabien Chouteau
Tanks, applied. But please add a patch description next time :)
Alex
Am 01.09.2011 um 10:20 schrieb Fabien Chouteau :
> While working on the emulation of the freescale p2010 (e500v2) I realized that
> there's no implementation of booke's timers features. Currently mpc8544 uses
> ppc_emb (ppc_emb_timers_init) which is close but not exactly like booke (for
> exampl
On Tue, Sep 6, 2011 at 18:19, octane indice wrote:
> En réponse à Stefan Hajnoczi :
>> You can run QEMU completely without a disk, just run:
>> $ gdb qemu
>> (gdb) r
>>
>> I wonder if it crashes that way too.
>>
> Yes:
> (gdb) r
> Starting program: /usr/src/qemu-0.15.0/i386-softmmu/qemu -vnc 127.
Dear Customer,
We would like to introduce you
the good performance jetski, jetboat with the
original SUZUKI 1400CC engine.
we truthly believe in mind that
only our customer's success make us going further.
2011 model Jet Ski:
Model name: HSJ3A
Engine : 1400cc Suzuki
To let device models distinguish between eject and load.
Signed-off-by: Markus Armbruster
---
block.c| 12 ++--
block.h|3 ++-
hw/fdc.c |2 +-
hw/ide/core.c |2 +-
hw/scsi-disk.c |2 +-
hw/sd.c|2 +-
6 files changed, 12 insertions(+),
The device model knows best when to accept the guest's eject command.
No need to detour through the block layer.
bdrv_eject() can't fail anymore. Make it void.
Signed-off-by: Markus Armbruster
---
block.c|7 +--
block.h|2 +-
hw/ide/atapi.c | 29 +-
We already track it in BlockDriverState. Just like tray open/close
state, we should track it in the device models instead, because it's
device state.
Signed-off-by: Markus Armbruster
---
hw/scsi-disk.c |4 +++-
1 files changed, 3 insertions(+), 1 deletions(-)
diff --git a/hw/scsi-disk.c b/
Luiz Capitulino writes:
> On Tue, 06 Sep 2011 17:55:12 +0200
> Jan Kiszka wrote:
>> On 2011-09-06 15:14, Luiz Capitulino wrote:
>> > This commit could have been folded with the previous one, however
>> > doing it separately will allow for easy bisect and revert if needed.
>> >
>> > Checking and
Requires new BlockDevOps member is_medium_locked(). Implement for IDE
and SCSI CD-ROMs.
Signed-off-by: Markus Armbruster
---
block.c| 16 +---
block.h|7 ++-
block_int.h|1 -
blockdev.c |2 +-
hw/ide/core.c |6 ++
hw/scsi-disk.c |
On Tue, 06 Sep 2011 22:55:17 +0530, "Aneesh Kumar K.V"
wrote:
> On Tue, 06 Sep 2011 09:31:32 -0500, Anthony Liguori
> wrote:
> > On 09/06/2011 06:22 AM, Gerd Hoffmann wrote:
> > > Hi,
> > >
> > > virtfs stopped working for me in master, the guest (fedora 15) just
> > > hangs at boot when mounti
Must set the ATAPI device signature, see ATA4 8.27.5.2 Outputs for
PACKET Command feature set devices, and ACS-2 7.36.6 Outputs for
PACKET feature set devices.
Signed-off-by: Markus Armbruster
---
hw/ide/core.c |4 +++-
1 files changed, 3 insertions(+), 1 deletions(-)
diff --git a/hw/ide/co
We already track it in BlockDriverState since commit 4be9762a. As
discussed in that commit's message, we should track it in the device
device models instead, because it's device state.
Signed-off-by: Markus Armbruster
---
hw/ide/atapi.c|6 +-
hw/ide/internal.h |1 +
2 files chan
Anthony,
The following patches have been sent to the list and look good to me. I've
also tested them.
The changes (since 344eecf6995f4a0ad1d887cec922f6806f91a3f8) are available
in the following repository:
git://repo.or.cz/qemu/qmp-unstable.git queue/qmp
Jan Kiszka (1):
Fix qjson test
Use a subsection, so that migration to older version still works,
provided the tray is closed and unlocked.
Signed-off-by: Markus Armbruster
---
hw/ide/core.c | 32
1 files changed, 32 insertions(+), 0 deletions(-)
diff --git a/hw/ide/core.c b/hw/ide/core.c
in
On Tue, Sep 06, 2011 at 11:28:09AM -0500, Anthony Liguori wrote:
> On 09/06/2011 11:09 AM, Michael S. Tsirkin wrote:
> >On Tue, Sep 06, 2011 at 10:51:26AM -0500, Anthony Liguori wrote:
> >>On 09/06/2011 10:45 AM, Jan Kiszka wrote:
> >>>On 2011-09-06 16:48, Michael S. Tsirkin wrote:
> I'm afraid
From: Jan Kiszka
"\/" is supposed to be decoded as "/", but there is no need to encode
"/" via escape. Fix the existing test and add a second one expressing
this.
Signed-off-by: Jan Kiszka
Acked-by: Michael Roth
Signed-off-by: Luiz Capitulino
---
check-qjson.c |3 ++-
1 files changed, 2
From: Paolo Bonzini
Writes go through scsi_write_complete at least twice, the first time
to get some data without having actually written anything. Because
of this, the first time scsi_write_complete is called it will call
bdrv_acct_done and account a read incorrectly. Fix this by looking
at th
The QDict unit-tests (check-qdict) will fail when ran on a different
build directory. That's, it only works when ran on the source dir.
This happens because its data file (qdict-test-data.txt) is not
copied to the build dir. Fix it.
Signed-off-by: Luiz Capitulino
---
configure |2 +-
1 file
It's convenience stuff for block device models, so block.h isn't the
ideal home either, but better than block_int.h.
Permits moving some #include "block_int.h" from device model .h into
.c.
Signed-off-by: Markus Armbruster
---
block.h | 38 ++
blo
From: Markus Armbruster
So we can more easily add device model callbacks.
Signed-off-by: Markus Armbruster
Signed-off-by: Kevin Wolf
---
block.c | 43 +--
block.h |9 ++---
block_int.h |5 ++---
hw/ide/core.c |6 +
We already track it in BlockDriverState. Just like tray open/close
state, we should track it in the device models instead, because it's
device state.
Signed-off-by: Markus Armbruster
---
hw/ide/atapi.c|4 +++-
hw/ide/internal.h |1 +
2 files changed, 4 insertions(+), 1 deletions(-)
On Tue, 06 Sep 2011 09:31:32 -0500, Anthony Liguori
wrote:
> On 09/06/2011 06:22 AM, Gerd Hoffmann wrote:
> > Hi,
> >
> > virtfs stopped working for me in master, the guest (fedora 15) just
> > hangs at boot when mounting the virtfs filesystems. Bisecting points to
> > this commit:
> >
> > rincew
Commit 4be9762a is now completely redone.
Signed-off-by: Markus Armbruster
---
block.c |1 -
block_int.h |1 -
2 files changed, 0 insertions(+), 2 deletions(-)
diff --git a/block.c b/block.c
index 6fe8add..8c86251 100644
--- a/block.c
+++ b/block.c
@@ -3062,7 +3062,6 @@ int bdrv_eje
"eject" is misleading; it means "eject" when start is clear, but
"load" when start is set. Rename to loej, because that's how MMC-5
calls it, in section 6.40.
Signed-off-by: Markus Armbruster
---
hw/ide/atapi.c |8
1 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/hw/i
ACS-2 Table B.2 explicitly prohibits ATAPI devices from implementing
WIN_RECAL, WIN_READ_EXT, WIN_READDMA_EXT, WIN_READ_NATIVE_MAX,
WIN_MULTREAD_EXT, WIN_WRITE, WIN_WRITE_ONCE, WIN_WRITE_EXT,
WIN_WRITEDMA_EXT, WIN_MULTWRITE_EXT, WIN_WRITE_VERIFY, WIN_VERIFY,
WIN_VERIFY_ONCE, WIN_VERIFY_EXT, WIN_SPE
It's a confused mess (see previous commit). No users remain.
Signed-off-by: Markus Armbruster
---
block.c |8
block.h |1 -
block_int.h |1 -
blockdev.c|5 -
hw/fdc.c |1 -
hw/ide/core.c |1 -
hw/scsi-disk.c
From: Markus Armbruster
hw/fdc.c is the only one that cares.
Signed-off-by: Markus Armbruster
Signed-off-by: Kevin Wolf
---
block.c | 20 ++--
block_int.h |1 -
hw/fdc.c| 40 ++--
3 files changed, 24 insertions(+), 37 deletio
Device models should be able to set it without an unclean include of
block_int.h.
Signed-off-by: Markus Armbruster
---
block.c |6 --
block.h |1 +
hw/ide/core.c |2 +-
hw/scsi-disk.c |2 +-
hw/virtio-blk.c |3 +--
5 files changed, 8 insertions(+), 6 de
Don't fail when tray is already open.
Signed-off-by: Markus Armbruster
---
hw/scsi-bus.c | 10 ++
hw/scsi-disk.c | 15 +++
hw/scsi.h |4
3 files changed, 25 insertions(+), 4 deletions(-)
diff --git a/hw/scsi-bus.c b/hw/scsi-bus.c
index 160eaee..79cb29d 100
change fails while the tray is locked by the guest. eject -f forces
it open and removes any media. Unfortunately, the tray closes again
instantly. Since the lock remains as it is, there is no way to insert
another medium unless the guest voluntarily unlocks.
Fix by leaving the tray open after m
No functional change.
It would be nice to have handler functions in the table, like commit
e1a064f9 did for ATAPI. Left for another day.
Signed-off-by: Markus Armbruster
---
hw/ide/core.c | 105 +++-
1 files changed, 80 insertions(+), 25 del
From: Markus Armbruster
savevm and loadvm silently ignore block devices with removable media,
such as floppies and SD cards. Rolling back a VM to a previous
checkpoint will *not* roll back writes to block devices with removable
media.
Moreover, bdrv_is_removable() is a confused mess, and wrong
Commit 4be9762a changed bdrv_is_inserted() to fail when the tray is
open. Unfortunately, there are two different kinds of users, with
conflicting needs.
1. Device models using bdrv_eject(), currently ide-cd and scsi-cd.
They expect bdrv_is_inserted() to reflect the tray status. Commit
4be9762a m
Signed-off-by: Markus Armbruster
---
hw/scsi-disk.c | 17 +
1 files changed, 13 insertions(+), 4 deletions(-)
diff --git a/hw/scsi-disk.c b/hw/scsi-disk.c
index 9724d0f..c8ad2e7 100644
--- a/hw/scsi-disk.c
+++ b/hw/scsi-disk.c
@@ -814,6 +814,18 @@ static int scsi_disk_emulate_
We already track it in BlockDriverState since commit 4be9762a. As
discussed in that commit's message, we should track it in the device
device models instead, because it's device state.
Signed-off-by: Markus Armbruster
---
hw/scsi-disk.c |2 ++
1 files changed, 2 insertions(+), 0 deletions(-
Including it in device models is unclean, including it without a
reason adds insult to injury.
Signed-off-by: Markus Armbruster
---
hw/fdc.c|1 -
hw/ide/cmd646.c |1 -
hw/ide/ich.c|1 -
hw/ide/isa.c|1 -
hw/ide/macio.c |1 -
hw/ide/microdr
BlockDriverState member removable is a confused mess. It is true when
an ide-cd, scsi-cd or floppy qdev is attached, or when the
BlockDriverState was created with -drive if={floppy,sd} or -drive
if={ide,scsi,xen,none},media=cdrom ("created removable"), except when
an ide-hd, scsi-hd, scsi-generic
While there, make the locked parameter bool.
Signed-off-by: Markus Armbruster
---
block.c |8
block.h |2 +-
block/raw-posix.c |8
block/raw.c |6 +++---
block_int.h |2 +-
hw/ide/atapi.c|2 +-
hw/scsi-disk.c|2
Signed-off-by: Markus Armbruster
---
block/nbd.c |1 +
nbd.c |1 +
nbd.h |2 --
3 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/block/nbd.c b/block/nbd.c
index 55cb2fd..70edd81 100644
--- a/block/nbd.c
+++ b/block/nbd.c
@@ -28,6 +28,7 @@
#include "qemu-c
MMC-5 6.40.2.6 specifies that START STOP UNIT succeeds when the drive
already has the requested state. cmd_start_stop_unit() fails when
asked to eject while the tray is open and locked. Fix that.
Signed-off-by: Markus Armbruster
---
hw/ide/atapi.c |2 +-
1 files changed, 1 insertions(+), 1
BlockDriverState member buffer_alignment is initially 512. The device
model may set them, with bdrv_set_buffer_alignment(). If the device
model gets detached (hot unplug), the device's alignment is left
behind. Only okay because device hot unplug automatically destroys
the BlockDriverState. But
From: Fam Zheng
Add support for reading/writing compressed extent.
Signed-off-by: Fam Zheng
Reviewed-by: Stefan Hajnoczi
Signed-off-by: Kevin Wolf
---
block/vmdk.c | 111 --
1 files changed, 100 insertions(+), 11 deletions(-)
diff --g
This patch series looks bigger than it is. All the patches are small
and hopefully easy to review.
Objectives:
* Push BlockDriverState members locked, tray_open, media_changed into
device models, where they belong. Kevin picked the patches pushing
media_changed from v2, so that part's gone
Need to ask the device, so this requires new BlockDevOps member
is_tray_open().
Signed-off-by: Markus Armbruster
---
block.c | 18 --
block.h |6 ++
hw/ide/core.c |6 ++
hw/scsi-disk.c |6 ++
qmp-commands.hx |2 ++
5 files changed,
From: Fam Zheng
Factor out read/write extent code, since there will be more things to
take care of once reading/writing compressed clusters is introduced.
Signed-off-by: Fam Zheng
Reviewed-by: Stefan Hajnoczi
Signed-off-by: Kevin Wolf
---
block/vmdk.c | 54 +
Requests depending on a failed request would end up waiting forever. This fixes
the error path to continue dependent requests even when the request has failed.
Signed-off-by: Kevin Wolf
---
block/qcow2.c |7 ---
1 files changed, 4 insertions(+), 3 deletions(-)
diff --git a/block/qcow2.c
From: Frediano Ziglio
Add some notes about Linux AIO explaining why we don't use AIO in
some situations.
Signed-off-by: Frediano Ziglio
Signed-off-by: Kevin Wolf
---
block/raw-posix.c |4
linux-aio.c |1 +
2 files changed, 5 insertions(+), 0 deletions(-)
diff --git a/block
From: Fam Zheng
Haiku provides a specially formed vmdk image, which let qemu abort. It a
combination of sparse header and flat data (i.e. with not l1/l2 table at
all). The fix is turn to descriptor when sparse header is zero in field
'capacity'.
Signed-off-by: Fam Zheng
Reviewed-by: Stefan Hajn
From: Fam Zheng
Add twoGbMaxExtentSparse support. Introduce vmdk_free_last_extent.
Signed-off-by: Fam Zheng
Reviewed-by: Stefan Hajnoczi
Signed-off-by: Kevin Wolf
---
block/vmdk.c | 133 --
1 files changed, 83 insertions(+), 50 deletio
From: Markus Armbruster
pci_piix3_xen_ide_unplug() unplugs only disks, not CD-ROMs. It peeks
into the DriveInfo's BlockDriverState to distinguish between the two.
Unclean; use DriveInfo member media_cd, like xen_config_dev_blk().
Signed-off-by: Markus Armbruster
Signed-off-by: Kevin Wolf
---
From: Markus Armbruster
Multiplexing callbacks complicates matters needlessly.
Signed-off-by: Markus Armbruster
Signed-off-by: Kevin Wolf
---
block.c | 23 +++
block.h | 12 +++-
block_int.h |3 ---
hw/ide/core.c |8 ++--
hw/sd
On Tue, 06 Sep 2011 17:55:12 +0200
Jan Kiszka wrote:
> On 2011-09-06 15:14, Luiz Capitulino wrote:
> > This commit could have been folded with the previous one, however
> > doing it separately will allow for easy bisect and revert if needed.
> >
> > Checking and testing all valid transitions was
From: Paolo Bonzini
Even though we do not use them, we should include the last three
bytes of sense data in the additional sense length.
Signed-off-by: Paolo Bonzini
Signed-off-by: Kevin Wolf
---
hw/scsi-bus.c |3 ++-
1 files changed, 2 insertions(+), 1 deletions(-)
diff --git a/hw/scsi-
From: Markus Armbruster
Block driver "raw" forwards most methods to the underlying block
driver. However, it doesn't implement method bdrv_media_changed().
Makes bdrv_media_changed() always return -ENOTSUP.
I believe -fda /dev/fd0 gives you raw over host_floppy, and disk
change detection (fdc r
From: Markus Armbruster
Signed-off-by: Markus Armbruster
Signed-off-by: Kevin Wolf
---
hw/ide/core.c |6 +++---
1 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/hw/ide/core.c b/hw/ide/core.c
index ff59a45..1806e00 100644
--- a/hw/ide/core.c
+++ b/hw/ide/core.c
@@ -2010,7 +201
From: Markus Armbruster
Signed-off-by: Markus Armbruster
Signed-off-by: Kevin Wolf
---
hw/ide/core.c | 12 ++--
1 files changed, 6 insertions(+), 6 deletions(-)
diff --git a/hw/ide/core.c b/hw/ide/core.c
index d1cbfe7..ff59a45 100644
--- a/hw/ide/core.c
+++ b/hw/ide/core.c
@@ -979,7
From: Fam Zheng
Creating streamOptimized subformat. Added subformat option
'streamOptimized', to create a image with compression enabled and each
cluster with a GrainMarker.
Signed-off-by: Fam Zheng
Reviewed-by: Stefan Hajnoczi
Signed-off-by: Kevin Wolf
---
block/vmdk.c | 18 --
On 2011-09-06 18:08, Anthony Liguori wrote:
> On 09/06/2011 11:05 AM, Jan Kiszka wrote:
>> On 2011-09-06 17:51, Anthony Liguori wrote:
>>> I'm still contemplating how we go about doing this. This series
>>> introduces a couple new concepts like QMP class hinting anonymous IDs.
>>> I'm concerned th
From: Markus Armbruster
Change (!bdrv_is_removable(bs) || bdrv_is_inserted(bs)) to just
bdrv_is_inserted(). Rationale:
The value of bdrv_is_removable(bs) matters only when
bdrv_is_inserted(bs) is false.
bdrv_is_inserted(bs) is true when bs is open (bs->drv != NULL) and
not an e
From: Paolo Bonzini
- do not return extra pages when requesting all pages (PAGE CODE = 0x3f)
- return correct sense code for PC = 3 (saved parameters not supported)
- do not return geometry pages for CD devices
Signed-off-by: Paolo Bonzini
Signed-off-by: Kevin Wolf
---
hw/scsi-disk.c | 96
On 2011-09-06 18:09, Michael S. Tsirkin wrote:
> On Tue, Sep 06, 2011 at 10:51:26AM -0500, Anthony Liguori wrote:
>> On 09/06/2011 10:45 AM, Jan Kiszka wrote:
>>> On 2011-09-06 16:48, Michael S. Tsirkin wrote:
I'm afraid that won't be enough to stop people
scripting this command - libvirt
- Original message -
> On 09/06/11 10:02, Brad wrote:
> > Only build with -g CFLAGS/LDFLAGS if using --enable-debug.
> > Add --optflags to allow overriding the default optimization
> > level added to CFLAGS.
> >
> > This is a first draft of coming up with a patch I could potentially
> > pu
On 09/06/2011 11:09 AM, Michael S. Tsirkin wrote:
On Tue, Sep 06, 2011 at 10:51:26AM -0500, Anthony Liguori wrote:
On 09/06/2011 10:45 AM, Jan Kiszka wrote:
On 2011-09-06 16:48, Michael S. Tsirkin wrote:
I'm afraid that won't be enough to stop people
scripting this command - libvirt accessed
H
From: Markus Armbruster
fdctrl_change_cb() gets called on a virtual media change via monitor.
It would be nice if host device block drivers called it on physical
media change, but they don't.
bdrv_media_changed() lets you poll for media change, but it returns
"don't know" except with block drive
From: Fam Zheng
Enable the createType 'twoGbMaxExtentFlat'. The supporting code is
already in.
Signed-off-by: Fam Zheng
Reviewed-by: Stefan Hajnoczi
Signed-off-by: Kevin Wolf
---
block/vmdk.c |4 +++-
1 files changed, 3 insertions(+), 1 deletions(-)
diff --git a/block/vmdk.c b/block/vmd
From: Markus Armbruster
Device models should be able to use it without an unclean include of
block_int.h.
Signed-off-by: Markus Armbruster
Signed-off-by: Kevin Wolf
---
block.h |2 ++
block_int.h |2 --
hw/sd.c |1 -
3 files changed, 2 insertions(+), 3 deletions(-)
diff -
The following changes since commit f69539b14bdba7a5cd22e1f4bed439b476b17286:
apb_pci: convert PCI space to memory API (2011-09-04 09:28:04 +)
are available in the git repository at:
git://repo.or.cz/qemu/kevin.git for-anthony
Fam Zheng (8):
VMDK: enable twoGbMaxExtentFlat
VMD
From: Markus Armbruster
For now, this just protects against programming errors like having the
same drive back multiple non-qdev devices, or untimely bdrv_delete().
Later commits will add other interesting uses.
While there, rename BlockDriverState member peer to dev, bdrv_attach()
to bdrv_attac
From: Fam Zheng
The vSphere 4 exported image is streamOptimized extent, which is not
quite correctly handled. Ignore rdgOffset when RGD flag bit not set.
Signed-off-by: Fam Zheng
Reviewed-by: Stefan Hajnoczi
Signed-off-by: Kevin Wolf
---
block/vmdk.c |9 +++--
1 files changed, 7 inse
On 09/06/2011 11:05 AM, Jan Kiszka wrote:
On 2011-09-06 17:51, Anthony Liguori wrote:
I'm still contemplating how we go about doing this. This series
introduces a couple new concepts like QMP class hinting anonymous IDs.
I'm concerned that we'll further complicate the need to support
backwards
On Tue, Sep 06, 2011 at 10:51:26AM -0500, Anthony Liguori wrote:
> On 09/06/2011 10:45 AM, Jan Kiszka wrote:
> >On 2011-09-06 16:48, Michael S. Tsirkin wrote:
> >>I'm afraid that won't be enough to stop people
> >>scripting this command - libvirt accessed
> >>HMP for years.
> >>
> >>On the other ha
On 2011-09-06 17:51, Anthony Liguori wrote:
> I'm still contemplating how we go about doing this. This series
> introduces a couple new concepts like QMP class hinting anonymous IDs.
> I'm concerned that we'll further complicate the need to support
> backwards compatibility.
Anonymous IDs must
From: Markus Armbruster
Drop WIN_SRST, it has the same value as WIN_DEVICE_RESET.
Drop unused WIN_RESTORE, it has the same value as WIN_RECAL.
Drop codes that are not implemented and long obsolete: WIN_READ_LONG,
WIN_READ_LONG_ONCE, WIN_WRITE_LONG, WIN_WRITE_LONG_ONCE, WIN_FORMAT
(all obsolete
On 09/06/2011 09:31 AM, Paolo Bonzini wrote:
On 08/22/2011 03:47 PM, Paolo Bonzini wrote:
On 08/22/2011 03:45 PM, Anthony Liguori wrote:
Almost: in Win32 you need to use g_io_channel_win32_new_socket. But
indeed on Windows you can only use qemu_set_fd_handler for sockets too.
I think that's
On 2011-09-06 15:14, Luiz Capitulino wrote:
> This commit could have been folded with the previous one, however
> doing it separately will allow for easy bisect and revert if needed.
>
> Checking and testing all valid transitions wasn't trivial, chances
> are this will need broader testing to beco
On 09/06/2011 10:45 AM, Jan Kiszka wrote:
On 2011-09-06 16:48, Michael S. Tsirkin wrote:
I'm afraid that won't be enough to stop people
scripting this command - libvirt accessed
HMP for years.
On the other hand, no QMP command means e.g.
libvirt users don't get any benefit from this.
What I th
From: Fam Zheng
Added flags field for compressed/streamOptimized extents, open and save
image configuration.
Signed-off-by: Fam Zheng
Reviewed-by: Stefan Hajnoczi
Signed-off-by: Kevin Wolf
---
block/vmdk.c | 16
1 files changed, 16 insertions(+), 0 deletions(-)
diff --git
On 2011-09-06 16:48, Michael S. Tsirkin wrote:
> On Fri, Aug 26, 2011 at 04:48:10PM +0200, Jan Kiszka wrote:
>> More than one year ago I posted some patches to add a monitor command
>> callend device_show. The purpose of that command is to dump the state of
>> some qdev device based on its vmstate.
From: Paolo Bonzini
Signed-off-by: Paolo Bonzini
Signed-off-by: Kevin Wolf
---
hw/scsi-disk.c | 41 +
1 files changed, 25 insertions(+), 16 deletions(-)
diff --git a/hw/scsi-disk.c b/hw/scsi-disk.c
index 3cc830f..5c652a2 100644
--- a/hw/scsi-disk.c
++
From: Markus Armbruster
We try the drive defined with -drive if=ide,index=0 (or equivalent
sugar). We use it only if (dinfo && bdrv_is_inserted(dinfo->bdrv) &&
!bdrv_is_removable(dinfo->bdrv)). This is a convoluted way to test
for "drive media can't be removed".
The only way to create such a d
From: Paolo Bonzini
Rename SERVICE_ACTION_IN to SERVICE_ACTION_IN_16 to distinguish
from the 12-byte CDB variant, and add a constant for the subcommand.
Signed-off-by: Paolo Bonzini
Signed-off-by: Kevin Wolf
---
hw/scsi-bus.c |3 ++-
hw/scsi-defs.h |8 +++-
hw/scsi-disk.c |6
Dependency list pointers filled with random garbage from the stack aren't a
good idea.
Signed-off-by: Kevin Wolf
---
block/qcow2.c |5 +++--
1 files changed, 3 insertions(+), 2 deletions(-)
diff --git a/block/qcow2.c b/block/qcow2.c
index b725d68..f26f7b6 100644
--- a/block/qcow2.c
+++ b/bl
qemu may segfault when a BH handler first deletes a BH and then (possibly
indirectly) calls a nested qemu_bh_poll(). This is because the inner instance
frees the BH and deletes it from the list that the outer one processes.
This patch deletes BHs only in the outermost qemu_bh_poll instance.
Commi
We're affected by this bug, too. Trying to find a workaround, last
friday we changed VGA model to cirrus, and the machine is working
properly without new entries in log.
--
You received this bug notification because you are a member of qemu-
devel-ml, which is subscribed to QEMU.
https://bugs.lau
On Tue, 6 Sep 2011, Richard Henderson wrote:
> On 09/04/2011 08:03 AM, malc wrote:
> > On Sun, 4 Sep 2011, Richard Henderson wrote:
> >
> >> On 09/03/2011 03:47 PM, malc wrote:
> >>> Doesn't make much sense to me, guest clearly asked for 0 and not -1,
> >>> besides -1 violates TCG's sar constrain
Sorry, I forgot to include Daniel Berrange who might have thoughts
about a nice way of running the privileged virtfs helper and how to
integrate with libvirt.
On Tue, Sep 6, 2011 at 3:48 PM, Stefan Hajnoczi wrote:
> On Mon, Sep 05, 2011 at 09:48:21PM +0530, M. Mohan Kumar wrote:
>> Qemu need to b
On 09/04/2011 08:03 AM, malc wrote:
> On Sun, 4 Sep 2011, Richard Henderson wrote:
>
>> On 09/03/2011 03:47 PM, malc wrote:
>>> Doesn't make much sense to me, guest clearly asked for 0 and not -1,
>>> besides -1 violates TCG's sar constraints and PPC obliges by emiting
>>> illegal instruction in t
On Mon, Sep 05, 2011 at 09:48:21PM +0530, M. Mohan Kumar wrote:
> Qemu need to be invoked by root user for using virtfs with passthrough
> security model (i.e to use chroot() syscall).
>
> Question is: Is running qemu by root user expected and allowed? Some of the
> virtfs features can be utilized
On Fri, Aug 26, 2011 at 04:48:10PM +0200, Jan Kiszka wrote:
> More than one year ago I posted some patches to add a monitor command
> callend device_show. The purpose of that command is to dump the state of
> some qdev device based on its vmstate.
>
> To improve the usability of that interface, th
On 09/06/2011 06:22 AM, Gerd Hoffmann wrote:
Hi,
virtfs stopped working for me in master, the guest (fedora 15) just
hangs at boot when mounting the virtfs filesystems. Bisecting points to
this commit:
rincewind kraxel ~/projects/qemu ((69e5bb6...)|BISECTING)# git bisect good
4d88a2ac8643265108
1 - 100 of 160 matches
Mail list logo
