[Qemu-devel] [PATCH] qemu-iotests: move blank lines of output in case 059

2013-10-11 Thread Fam Zheng
Move the blank line to above the test step banner, so it looks clearer in blocks. Signed-off-by: Fam Zheng --- tests/qemu-iotests/059 | 8 tests/qemu-iotests/059.out | 8 2 files changed, 8 insertions(+), 8 deletions(-) diff --git a/tests/qemu-iotests/059 b/tests/qemu-iote

Re: [Qemu-devel] Qemu Boot failure with /dev/ram0

2013-10-11 Thread Philipp Hahn
Hello, On Monday 16 September 2013 07:40:08 Saptarshi Sen wrote: >I want to use a ram disk as a block device for my kernel.( i am > measuring file io on ram ) > >I am trying to boot a kernel from using /dev/ram0. However I am getting > "No root Device found" > >qemu-system-x86_64 -m

Re: [Qemu-devel] [PATCH] block/iscsi: introduce bdrv_co_{readv, writev, flush_to_disk}

2013-10-11 Thread Peter Lieven
On 08.10.2013 14:39, Kevin Wolf wrote: Am 08.10.2013 um 14:35 hat Paolo Bonzini geschrieben: Il 08/10/2013 14:33, Kevin Wolf ha scritto: this converts read, write and flush functions from aio to coroutines. I'm not sure it's already the time for this... Cancellation sucks in QEMU, and this is

[Qemu-devel] [PATCH v2 1/2] qcow2: Undo leaked allocations in co_writev

2013-10-11 Thread Max Reitz
If the write request spans more than one L2 table, qcow2_alloc_cluster_offset cannot handle the required allocations atomically. This results in leaks if it allocated new clusters in any but the last L2 table touched and an error occurs in qcow2_co_writev before having established the L2 link. Thes

[Qemu-devel] [PATCH v2 0/2] qcow2: Undo leaked allocations in co_writev

2013-10-11 Thread Max Reitz
If a write request on a qcow2 image spans more than one L2 table, qcow2_alloc_cluster_offset cannot allocate the required clusters in a single operation. This results in leaks, if a subsequent (atomic) allocation in that function fails, because qcow2_co_writev does not undo unused cluster allocatio

[Qemu-devel] [PATCH v2 2/2] qemu-iotests: Extend test 026

2013-10-11 Thread Max Reitz
Extend test case 026 by an aio_write fail test, which should not result in any leaked clusters. Signed-off-by: Max Reitz --- tests/qemu-iotests/026 | 31 +++ tests/qemu-iotests/026.out | 8 tests/qemu-iotests/026.out.nocache | 8

Re: [Qemu-devel] [PATCH v2 0/2] vmdk: convert error reporting

2013-10-11 Thread Fam Zheng
On Thu, 10/10 20:38, Fam Zheng wrote: > The first patch converts "fprintf(stderr,..." to error_setg with errp, and > fixes style of error message texts. > > The second patch checks the compatibility of zeroed_grain flag and flat type > and reports error if both are true. > > v2: [01] More convers

[Qemu-devel] [PATCH v3 0/2] vmdk: convert error reporting

2013-10-11 Thread Fam Zheng
The first patch converts "fprintf(stderr,..." to error_setg with errp, and fixes style of error message texts. The second patch checks the compatibility of zeroed_grain flag and flat type and reports error if both are true. v3: Update qemu-iotests. v2: [01] More conversion of error messages, als

[Qemu-devel] [PATCH v3 1/2] vmdk: convert error code to use errp

2013-10-11 Thread Fam Zheng
Convert "fprintf(stderr,..." and standardize error messages: Remove a few local_error's and use errp. Remove "VMDK:" or "Vmdk:" prefixes in error message and fix to upper case. Signed-off-by: Fam Zheng --- block/vmdk.c | 116 - tests/qe

[Qemu-devel] [PATCH v3 2/2] vmdk: refuse enabling zeroed grain with flat images

2013-10-11 Thread Fam Zheng
This is a header flag and we needs sparse for the header. Signed-off-by: Fam Zheng --- block/vmdk.c | 4 tests/qemu-iotests/059 | 4 tests/qemu-iotests/059.out | 4 3 files changed, 12 insertions(+) diff --git a/block/vmdk.c b/block/vmdk.c index 4f8ae77..90340eb

[Qemu-devel] [PATCH] vmdk: Only read cid from image file when opening

2013-10-11 Thread Fam Zheng
Previously cid of parent is parsed from image file for every IO request. We already have L1/L2 cache and don't have assumption that parent image can be updated behind us, so remove this to get more efficiency. The parent CID is checked them when opening backing file. Signed-off-by: Fam Zheng ---

Re: [Qemu-devel] [PATCH] Use qemu-project.org domain name

2013-10-11 Thread Stefan Hajnoczi
On Thu, Oct 10, 2013 at 07:42:30PM +0200, Stefan Weil wrote: > Am 10.10.2013 11:39, schrieb Stefan Hajnoczi: > > qemu.org is held by a third-party and no core community contributor has > > access to the DNS configuration. This leaves the website exposed to > > outages due to DNS issues or IP addre

[Qemu-devel] [PATCH] block/iscsi: remove .bdrv_has_zero_init

2013-10-11 Thread Peter Lieven
since commit 3ac21627 the default value changed to 0. Signed-off-by: Peter Lieven --- block/iscsi.c |7 --- 1 file changed, 7 deletions(-) diff --git a/block/iscsi.c b/block/iscsi.c index a2a961e..a660a1c 100644 --- a/block/iscsi.c +++ b/block/iscsi.c @@ -1447,11 +1447,6 @@ static int i

Re: [Qemu-devel] [PATCH V5 1/6] block/qemu-img: Refine and export infinite loop checking in collect_image_info_list()

2013-10-11 Thread Xu Wang
于 2013/8/3 6:09, Eric Blake 写道: On 08/02/2013 03:02 AM, Xu Wang wrote: From: Xu Wang If there is a loop exists in the backing file chain, many problems could be caused by it, such as no response and segment fault during system boot. Hence stopping backing file loop appear is very necessary. Th

[Qemu-devel] [PATCH v2 1/2] qapi: Add optional field 'compressed' to ImageInfo

2013-10-11 Thread Fam Zheng
Signed-off-by: Fam Zheng --- qapi-schema.json | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/qapi-schema.json b/qapi-schema.json index a1a81a4..b187fdb 100644 --- a/qapi-schema.json +++ b/qapi-schema.json @@ -256,6 +256,8 @@ # # @encrypted: #optional true if the image is

[Qemu-devel] [PATCH v2 0/2] vmdk: Implement bdrv_get_specific_info

2013-10-11 Thread Fam Zheng
This is based on "[PATCH v3 0/2] vmdk: convert error reporting". The new information looks like: image: /tmp/foo.vmdk file format: vmdk virtual size: 100G (107374182400 bytes) disk size: 4.0K Format specific information: cid: 0 create_type: twoGbMaxExtentFlat

[Qemu-devel] [PATCH v2 2/2] vmdk: implment bdrv_get_specific_info

2013-10-11 Thread Fam Zheng
Implement .bdrv_get_specific_info to return the extent information. Signed-off-by: Fam Zheng --- block/vmdk.c | 57 +- qapi-schema.json | 24 ++- tests/qemu-iotests/059 | 2 +- tests/qemu-iotests/059.out |

Re: [Qemu-devel] [PATCH v6 2/5] hpet: enable to entitle more irq pins for hpet

2013-10-11 Thread Paolo Bonzini
Il 11/10/2013 04:59, liu ping fan ha scritto: > On Thu, Oct 10, 2013 at 7:41 PM, Michael S. Tsirkin wrote: >> On Thu, Oct 10, 2013 at 11:46:42AM +0200, Paolo Bonzini wrote: >>> Il 10/10/2013 11:41, Michael S. Tsirkin ha scritto: >> Are you sure? This is not done for any other compat property.

[Qemu-devel] [PATCH] block/iscsi: updated copyright

2013-10-11 Thread Peter Lieven
added myself to reflect recent work on the iscsi block driver. Signed-off-by: Peter Lieven --- block/iscsi.c |1 + 1 file changed, 1 insertion(+) diff --git a/block/iscsi.c b/block/iscsi.c index a660a1c..ee8cdd3 100644 --- a/block/iscsi.c +++ b/block/iscsi.c @@ -2,6 +2,7 @@ * QEMU Block d

Re: [Qemu-devel] PPC: Timer issues with git master

2013-10-11 Thread Paolo Bonzini
Il 11/10/2013 00:27, Mark Cave-Ayland ha scritto: > > I've just upgraded my git repository from around the 1.6 release > timeframe to git master and I'm seeing some strange timing issues > running through my OpenBIOS test suite which weren't there before my git > update. > > The two most noticeab

Re: [Qemu-devel] [PATCH 2/2] aio: add aio_context_acquire() and aio_context_release()

2013-10-11 Thread Wenchao Xia
Reviewed-by: Wenchao Xia

Re: [Qemu-devel] [PATCH 1/2] rfifolock: add recursive FIFO lock

2013-10-11 Thread Wenchao Xia
Logic looks fine to me, only a minor comments. > QemuMutex does not guarantee fairness and cannot be acquired > recursively: > > Fairness means each locker gets a turn and the scheduler cannot cause > starvation. > > Recursive locking is useful for composition, it allows a sequence of > locking op

Re: [Qemu-devel] kvm binary is deprecated

2013-10-11 Thread Stefan Hajnoczi
On Thu, Oct 10, 2013 at 02:00:39PM +0300, Alexander Binun wrote: > Stefan , great thanks! We are setting up the scene for experiments... > > Unfortunately, we ran into yet one trouble. The configuration: Ubuntu 13.04, > internal KVM, Qemu 1.4.0. VMs are created using virt-manager. > > When we tr

[Qemu-devel] [PATCH] configure: create fsdev/ directory

2013-10-11 Thread Michael Tokarev
In some cases when building with parallelism (make -jN), build fails because the directory where output files are supposed to be does not exist. In particular, when make decides to build virtfs-proxy-helper.1 before other files in fsdev/, build will fail with the following error: perl -Ww -- BUIL

Re: [Qemu-devel] [PATCH 2/5] block/raw-posix: Employ error parameter

2013-10-11 Thread Kevin Wolf
Am 10.10.2013 um 15:44 hat Max Reitz geschrieben: > Make use of the error parameter in the opening and creating functions in > block/raw-posix.c. > > Signed-off-by: Max Reitz > --- > block/raw-posix.c | 60 > --- > 1 file changed, 48 insertion

Re: [Qemu-devel] [PATCH 1/2] qcow2: Undo leaked allocations in co_writev

2013-10-11 Thread Stefan Hajnoczi
On Thu, Oct 10, 2013 at 02:26:25PM +0200, Kevin Wolf wrote: > Am 10.10.2013 um 10:52 hat Max Reitz geschrieben: > > If the write request spans more than one L2 table, > > qcow2_alloc_cluster_offset cannot handle the required allocations > > atomically. This results in leaks if it allocated new clus

Re: [Qemu-devel] [PATCH v6 2/5] hpet: enable to entitle more irq pins for hpet

2013-10-11 Thread liu ping fan
On Fri, Oct 11, 2013 at 4:38 PM, Paolo Bonzini wrote: > Il 11/10/2013 04:59, liu ping fan ha scritto: >> On Thu, Oct 10, 2013 at 7:41 PM, Michael S. Tsirkin wrote: >>> On Thu, Oct 10, 2013 at 11:46:42AM +0200, Paolo Bonzini wrote: Il 10/10/2013 11:41, Michael S. Tsirkin ha scritto: >>> A

Re: [Qemu-devel] [PATCH 2/4] vring: factor common code for error exits

2013-10-11 Thread Paolo Bonzini
Il 10/10/2013 22:53, Richard Henderson ha scritto: > On 10/10/2013 08:07 AM, Paolo Bonzini wrote: >> return head; >> + >> +out: >> +assert(ret < 0); >> +if (ret == -EFAULT) { >> +vring->broken = true; >> +} >> +return ret; > > If this is only the error path, can we cal

[Qemu-devel] [PATCH v2 2/5] block/raw-posix: Employ error parameter

2013-10-11 Thread Max Reitz
Make use of the error parameter in the opening and creating functions in block/raw-posix.c. Signed-off-by: Max Reitz --- v2: - fixed call to raw_open_common() from the FreeBSD cdrom_open() --- block/raw-posix.c | 69 --- 1 file changed, 55 ins

Re: [Qemu-devel] [PATCH 0/5] block: Employ error parameter

2013-10-11 Thread Kevin Wolf
Am 10.10.2013 um 15:43 hat Max Reitz geschrieben: > The series "block: Error parameter for opening/creating images" > introduced exactly that. However, right now, this parameter is in fact > only used by qcow2. This series adds support to some additional block > drivers for which it was basically t

Re: [Qemu-devel] [PATCH] qemu-iotests: move blank lines of output in case 059

2013-10-11 Thread Kevin Wolf
Am 11.10.2013 um 08:59 hat Fam Zheng geschrieben: > Move the blank line to above the test step banner, so it looks clearer > in blocks. > > Signed-off-by: Fam Zheng Thanks, applied to the block branch. Kevin

Re: [Qemu-devel] QEMU/CPC project: an experience report

2013-10-11 Thread Stefan Hajnoczi
On Thu, Oct 10, 2013 at 11:14:22PM +0100, Gabriel Kerneis wrote: > We were so excited about our QEMU/CPC GSoC project that we have decided > to write a 12-page article to share our experience: > >QEMU/CPC: static analysis and CPS conversion >for safe, portable, and efficien

[Qemu-devel] [PATCH v3 2/5] block/raw-posix: Employ error parameter

2013-10-11 Thread Max Reitz
Make use of the error parameter in the opening and creating functions in block/raw-posix.c. Signed-off-by: Max Reitz --- v3: - fixed possible errno trashing by error_setg_errno by storing its value before calling that function v2: - fixed call to raw_open_common() from the FreeBSD cdrom_ope

Re: [Qemu-devel] [PATCH V2 0/2] build: trivial patches for test*

2013-10-11 Thread Wenchao Xia
于 2013/9/26 8:42, Wenchao Xia 写道: > v2: > Address Michael's comments: > 01: better commit message to tip what it is doing. > > Wenchao Xia (2): > 1 tests: build the helper program by default > 2 build: add command check-clean > > Makefile |1 - > tests/Makefile | 16 +++

[Qemu-devel] [PATCH] hw/gpio/max7310.c : add output register property and update the outputports level when necessary

2013-10-11 Thread Zhou Yuan
From: Zhouy To: qemu-devel@nongnu.org Date: Fri, 11 Oct 2013 15:54:47 -0400 Subject: [PATCH 1/1] add output register property and update the outputports level when necessary Signed-off-by: Zhouy --- qemu-master/hw/gpio/max7310.c | 33 ++--- 1 files changed, 22 ins

Re: [Qemu-devel] [PATCH] block/iscsi: remove .bdrv_has_zero_init

2013-10-11 Thread Kevin Wolf
Am 11.10.2013 um 09:59 hat Peter Lieven geschrieben: > since commit 3ac21627 the default value changed to 0. > > Signed-off-by: Peter Lieven Reviewed-by: Kevin Wolf

Re: [Qemu-devel] [PATCH v3 2/5] block/raw-posix: Employ error parameter

2013-10-11 Thread Kevin Wolf
Am 11.10.2013 um 11:37 hat Max Reitz geschrieben: > Make use of the error parameter in the opening and creating functions in > block/raw-posix.c. > > Signed-off-by: Max Reitz > --- > v3: > - fixed possible errno trashing by error_setg_errno by storing its >value before calling that function

[Qemu-devel] Networking in QEMU

2013-10-11 Thread Ayaz Akram
I have to emulate netwotking drivers for a particular mips board in qemu. I have gone through MALTA mips machine and its emulation in qemu, but I have not been able to find any good documentation to find out what should be starting point? Any help in this regard would be highly appreciated...

Re: [Qemu-devel] [PATCH v3 0/2] vmdk: convert error reporting

2013-10-11 Thread Kevin Wolf
Am 11.10.2013 um 09:43 hat Fam Zheng geschrieben: > The first patch converts "fprintf(stderr,..." to error_setg with errp, and > fixes style of error message texts. > > The second patch checks the compatibility of zeroed_grain flag and flat type > and reports error if both are true. > > v3: Updat

[Qemu-devel] [PATCH] map 64-bit PCI BARs at location provided by emulator

2013-10-11 Thread Igor Mammedov
Currently 64-bit PCI BARs are unconditionally mapped by BIOS right over 4G + RamSizeOver4G location, which doesn't allow to reserve extra space before 64-bit PCI window. For memory hotplug an extra RAM space might be reserved after present 64-bit RAM end and BIOS should map 64-bit PCI BARs after it

Re: [Qemu-devel] Networking in QEMU

2013-10-11 Thread Antony Pavlov
On Fri, 11 Oct 2013 03:01:48 -0700 Ayaz Akram wrote: > I have to emulate netwotking drivers for a particular mips board in qemu. I > have gone through MALTA mips machine and its emulation in qemu, but I have > not been able to find any good documentation to find out what should be > starting poin

[Qemu-devel] [PATCH V4 3/5] qemu-iotests: add 058 internal snapshot export with qemu-nbd case

2013-10-11 Thread Wenchao Xia
Signed-off-by: Wenchao Xia --- tests/qemu-iotests/058 | 102 tests/qemu-iotests/058.out | 32 ++ tests/qemu-iotests/check |1 + tests/qemu-iotests/group |1 + 4 files changed, 136 insertions(+), 0 deletions(-) create mod

[Qemu-devel] [PATCH V4 0/5] export internal snapshot by qemu-nbd

2013-10-11 Thread Wenchao Xia
This series allow user to read internal snapshot's contents without qemu-img convert. V2: Address Stefan's comments: 02: add 'fall through' comments in the case statement. 03: add doc about the difference of internal snapshot and backing chain snapshot, which is used in previous '--snapshot'

[Qemu-devel] [PATCH V4 1/5] snapshot: distinguish id and name in load_tmp

2013-10-11 Thread Wenchao Xia
Since later this function will be used so improve it. The only caller of it now is qemu-img, and it is not impacted by introduce function bdrv_snapshot_load_tmp_by_id_or_name() that call bdrv_snapshot_load_tmp() twice to keep old search logic. bdrv_snapshot_load_tmp_by_id_or_name() return int to le

[Qemu-devel] [PATCH V4 2/5] qemu-nbd: support internal snapshot export

2013-10-11 Thread Wenchao Xia
Now it is possible to directly export an internal snapshot, which can be used to probe the snapshot's contents without qemu-img convert. Signed-off-by: Wenchao Xia --- block/snapshot.c | 18 + include/block/snapshot.h |8 +++ qemu-nbd.c | 47

[Qemu-devel] [PATCH V4 4/5] qemu-img: add -l for snapshot in convert

2013-10-11 Thread Wenchao Xia
Now qemu-img convert have similar options as qemu-nbd for internal snapshot. Signed-off-by: Wenchao Xia --- qemu-img-cmds.hx |2 +- qemu-img.c | 45 - qemu-img.texi| 12 3 files changed, 45 insertions(+), 14 deletions(-)

[Qemu-devel] [PATCH V4 5/5] qemu-iotests: add test for snapshot in qemu-img convert

2013-10-11 Thread Wenchao Xia
Signed-off-by: Wenchao Xia --- tests/qemu-iotests/058 | 19 ++- tests/qemu-iotests/058.out | 12 2 files changed, 30 insertions(+), 1 deletions(-) diff --git a/tests/qemu-iotests/058 b/tests/qemu-iotests/058 index 5b821cf..d4987ae 100755 --- a/tests/qemu-iote

[Qemu-devel] [PATCH] PPC: fix PCI configuration space MemoryRegions for grackle/uninorth

2013-10-11 Thread Mark Cave-Ayland
OpenBIOS prior to SVN r1225 had a horrible bug when accessing PCI configuration space for PPC Mac architectures - instead of writing the PCI configuration data value to the data register address, it would instead write it to the data register address plus the PCI configuration address. For this re

Re: [Qemu-devel] [PATCH 2/5] block/raw-posix: Employ error parameter

2013-10-11 Thread Markus Armbruster
Kevin Wolf writes: > Am 10.10.2013 um 15:44 hat Max Reitz geschrieben: >> Make use of the error parameter in the opening and creating functions in >> block/raw-posix.c. >> >> Signed-off-by: Max Reitz >> --- >> block/raw-posix.c | 60 >> --- >>

[Qemu-devel] [PATCH -V5] target-ppc: Fix page table lookup with kvm enabled

2013-10-11 Thread Aneesh Kumar K.V
From: "Aneesh Kumar K.V" With kvm enabled, we store the hash page table information in the hypervisor. Use ioctl to read the htab contents. Without this we get the below error when trying to read the guest address (gdb) x/10 do_fork 0xc0098660 : Cannot access memory at address 0xc00

Re: [Qemu-devel] [RfC PATCH] e820: pass high memory too.

2013-10-11 Thread Igor Mammedov
On Thu, 10 Oct 2013 14:54:29 +0200 Gerd Hoffmann wrote: > We have a fw_cfg entry to pass e820 entries from qemu to the firmware. > Today it's used to pass reservations only. This patch makes qemu pass > entries for RAM too. > > This allows to pass RAM sizes larger than 1TB to the firmware and i

Re: [Qemu-devel] [PATCH] vmdk: Fix vmdk_parse_extents

2013-10-11 Thread Kevin Wolf
Am 11.10.2013 um 08:04 hat Fam Zheng geschrieben: > An extra 'p++' after while loop when *p == '\n' will move p to unknown > data position, risking parsing junk data or memory access violation. > > Cc: qemu-sta...@nongnu.org > Signed-off-by: Fam Zheng > --- > block/vmdk.c | 7 +-- > 1 file c

Re: [Qemu-devel] [PATCH v6 2/5] hpet: enable to entitle more irq pins for hpet

2013-10-11 Thread Paolo Bonzini
Il 11/10/2013 11:18, liu ping fan ha scritto: > On Fri, Oct 11, 2013 at 4:38 PM, Paolo Bonzini wrote: >> Il 11/10/2013 04:59, liu ping fan ha scritto: >>> On Thu, Oct 10, 2013 at 7:41 PM, Michael S. Tsirkin wrote: On Thu, Oct 10, 2013 at 11:46:42AM +0200, Paolo Bonzini wrote: > Il 10/10/

Re: [Qemu-devel] [PATCH] vmdk: Fix vmdk_parse_extents

2013-10-11 Thread Fam Zheng
On Fri, 10/11 13:23, Kevin Wolf wrote: > Am 11.10.2013 um 08:04 hat Fam Zheng geschrieben: > > An extra 'p++' after while loop when *p == '\n' will move p to unknown > > data position, risking parsing junk data or memory access violation. > > > > Cc: qemu-sta...@nongnu.org > > Signed-off-by: Fam Z

Re: [Qemu-devel] [PATCH v2 0/2] vmdk: Implement bdrv_get_specific_info

2013-10-11 Thread Kevin Wolf
Am 11.10.2013 um 10:31 hat Fam Zheng geschrieben: > This is based on "[PATCH v3 0/2] vmdk: convert error reporting". I have that series in the block branch now, but it still doesn't apply. Can you rebase? Kevin

Re: [Qemu-devel] QEMU/CPC project: an experience report

2013-10-11 Thread Gabriel Kerneis
Hi Pierre, On Fri, Oct 11, 2013 at 01:34:02PM +0200, Pierre Tardy wrote: > Do I summarized too much if I say that this is about automatically > translating thread based code to event driven code? This is exactly what CPC is about. If you want more details on the translation itself, you might be

[Qemu-devel] [PATCH v2] vmdk: Fix vmdk_parse_extents

2013-10-11 Thread Fam Zheng
An extra 'p++' after while loop when *p == '\n' will move p to unknown data position, risking parsing junk data or memory access violation. Cc: qemu-sta...@nongnu.org Signed-off-by: Fam Zheng --- block/vmdk.c | 7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/block/vmdk.c

Re: [Qemu-devel] [PATCH v2 2/2] vmdk: implment bdrv_get_specific_info

2013-10-11 Thread Eric Blake
On 10/11/2013 02:31 AM, Fam Zheng wrote: > Implement .bdrv_get_specific_info to return the extent information. > > Signed-off-by: Fam Zheng > --- > + > +*spec_info->vmdk = (ImageInfoSpecificVmdk) { > +.create_type = g_strdup(s->create_type), > +.cid = s->cid, > +}; > + >

Re: [Qemu-devel] [PATCH v2] vmdk: Fix vmdk_parse_extents

2013-10-11 Thread Kevin Wolf
Am 11.10.2013 um 13:48 hat Fam Zheng geschrieben: > An extra 'p++' after while loop when *p == '\n' will move p to unknown > data position, risking parsing junk data or memory access violation. > > Cc: qemu-sta...@nongnu.org > Signed-off-by: Fam Zheng Thanks, applied to the block branch. Kevin

[Qemu-devel] [PATCH 0/2] qcow2/iotests: Add test for inactive L2 overlap

2013-10-11 Thread Max Reitz
With the ability to adjust the metadata overlap checks performed through the command line, we can now add a test for checks not performed by the default "cached" mode, i.e. those regarding inactive L2 tables. In order to be able to use qemu-io for this test, however, it must first be extended to a

[Qemu-devel] [PATCH 1/2] qemu-io: Let "open" pass options to block driver

2013-10-11 Thread Max Reitz
Add an option to the open command to specify runtime options for the block driver used. Signed-off-by: Max Reitz --- qemu-io.c | 39 +++ 1 file changed, 31 insertions(+), 8 deletions(-) diff --git a/qemu-io.c b/qemu-io.c index f4b8efc..3b3340a 100644 --- a/qe

[Qemu-devel] [PATCH 2/2] qemu-iotests: Add test for inactive L2 overlap

2013-10-11 Thread Max Reitz
Extend 060 by a test which creates a corrupted image with an active L2 entry pointing to an inactive L2 table and writes to the corresponding guest offset. Also, use overlap-check=all for all tests in 060. Signed-off-by: Max Reitz --- tests/qemu-iotests/060 | 47

Re: [Qemu-devel] [PATCH v2 2/2] vmdk: implment bdrv_get_specific_info

2013-10-11 Thread Fam Zheng
On Fri, 10/11 05:52, Eric Blake wrote: > On 10/11/2013 02:31 AM, Fam Zheng wrote: > > Implement .bdrv_get_specific_info to return the extent information. > > > > Signed-off-by: Fam Zheng > > --- > > > + > > +*spec_info->vmdk = (ImageInfoSpecificVmdk) { > > +.create_type = g_strdup(s-

[Qemu-devel] [PATCH v3 0/2] vmdk: Implement bdrv_get_specific_info

2013-10-11 Thread Fam Zheng
The new information looks like: image: /tmp/foo.vmdk file format: vmdk virtual size: 100G (107374182400 bytes) disk size: 4.0K Format specific information: cid: 0 create_type: twoGbMaxExtentFlat parent cid: 0 extents: [0]:

[Qemu-devel] [PATCH v3 2/2] vmdk: Implment bdrv_get_specific_info

2013-10-11 Thread Fam Zheng
Implement .bdrv_get_specific_info to return the extent information. Signed-off-by: Fam Zheng --- block/vmdk.c | 57 +- qapi-schema.json | 24 ++- tests/qemu-iotests/059 | 2 +- tests/qemu-iotests/059.out |

Re: [Qemu-devel] [PATCH v3 0/2] vmdk: Implement bdrv_get_specific_info

2013-10-11 Thread Eric Blake
On 10/11/2013 06:08 AM, Fam Zheng wrote: > The new information looks like: > > image: /tmp/foo.vmdk > file format: vmdk > virtual size: 100G (107374182400 bytes) > disk size: 4.0K > Format specific information: > cid: 0 > create_type: twoGbMaxExtentFlat >

Re: [Qemu-devel] [PATCH 0/2] qcow2/iotests: Add test for inactive L2 overlap

2013-10-11 Thread Kevin Wolf
Am 11.10.2013 um 14:02 hat Max Reitz geschrieben: > With the ability to adjust the metadata overlap checks performed through > the command line, we can now add a test for checks not performed by the > default "cached" mode, i.e. those regarding inactive L2 tables. > > In order to be able to use qe

[Qemu-devel] [PATCH v3 1/2] qapi: Add optional field 'compressed' to ImageInfo

2013-10-11 Thread Fam Zheng
Signed-off-by: Fam Zheng --- qapi-schema.json | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/qapi-schema.json b/qapi-schema.json index 60f3fd1..add97e2 100644 --- a/qapi-schema.json +++ b/qapi-schema.json @@ -256,6 +256,8 @@ # # @encrypted: #optional true if the image is

Re: [Qemu-devel] [RfC PATCH] e820: pass high memory too.

2013-10-11 Thread Gerd Hoffmann
On Fr, 2013-10-11 at 13:20 +0200, Igor Mammedov wrote: > On Thu, 10 Oct 2013 14:54:29 +0200 > Gerd Hoffmann wrote: > > > We have a fw_cfg entry to pass e820 entries from qemu to the firmware. > > Today it's used to pass reservations only. This patch makes qemu pass > > entries for RAM too. > >

Re: [Qemu-devel] [PATCH v2 2/2] vmdk: implment bdrv_get_specific_info

2013-10-11 Thread Max Reitz
On 2013-10-11 14:07, Fam Zheng wrote: On Fri, 10/11 05:52, Eric Blake wrote: On 10/11/2013 02:31 AM, Fam Zheng wrote: Implement .bdrv_get_specific_info to return the extent information. Signed-off-by: Fam Zheng --- + +*spec_info->vmdk = (ImageInfoSpecificVmdk) { +.create_type = g_

[Qemu-devel] [PATCH] block/raw-win32: Always use -errno in hdev_open

2013-10-11 Thread Max Reitz
On one occasion, hdev_open() returned -1 in case of an unknown error instead of a proper -errno value. Adjust this to match the behavior of raw_open() (in raw-win32), which is to return -EINVAL in this case. Also, change the call to error_setg*() to match the one in raw_open() as well. Signed-off-

[Qemu-devel] [PATCH] target-ppc: Fix htab_mask calculation

2013-10-11 Thread Aneesh Kumar K.V
From: "Aneesh Kumar K.V" Correctly update the htab_mask using the return value of KVM_PPC_ALLOCATE_HTAB ioctl. Also we don't update sdr1 on GET_SREGS for HV. So don't update htab_mask if sdr1 is found to be zero. Fix the pte index calculation to be same as that found in the kernel Signed-off-by:

Re: [Qemu-devel] [PATCH] map 64-bit PCI BARs at location provided by emulator

2013-10-11 Thread Gerd Hoffmann
On Fr, 2013-10-11 at 12:19 +0200, Igor Mammedov wrote: > Currently 64-bit PCI BARs are unconditionally mapped by BIOS right > over 4G + RamSizeOver4G location, which doesn't allow to reserve > extra space before 64-bit PCI window. For memory hotplug an extra > RAM space might be reserved after pres

Re: [Qemu-devel] [PATCH] block/raw-win32: Always use -errno in hdev_open

2013-10-11 Thread Eric Blake
On 10/11/2013 06:30 AM, Max Reitz wrote: > On one occasion, hdev_open() returned -1 in case of an unknown error > instead of a proper -errno value. Adjust this to match the behavior of > raw_open() (in raw-win32), which is to return -EINVAL in this case. > Also, change the call to error_setg*() to

Re: [Qemu-devel] [PATCH] target-ppc: Fix htab_mask calculation

2013-10-11 Thread Aneesh Kumar K.V
"Aneesh Kumar K.V" writes: > From: "Aneesh Kumar K.V" > > Correctly update the htab_mask using the return value of > KVM_PPC_ALLOCATE_HTAB ioctl. Also we don't update sdr1 > on GET_SREGS for HV. So don't update htab_mask if sdr1 > is found to be zero. Fix the pte index calculation to be > same a

[Qemu-devel] [PATCH v2] Use qemu-project.org domain name

2013-10-11 Thread Stefan Hajnoczi
qemu.org is held by a third-party and no core community contributor has access to the DNS configuration. This leaves the website exposed to outages due to DNS issues or IP address changes. For example, if the web server IP address needs to change we cannot guarantee qemu.org will point to it! Th

[Qemu-devel] [v2 00/13] Stage 2 VSX Support

2013-10-11 Thread Tom Musta
This patch series continues adding support for the PowerPC Vector Scalar Extension (VSX). Patches are relative to the Stage 1 delivery (see http://lists.nongnu.org/archive/html/qemu-ppc/2013-09/msg00231.html). This series adds the following: a) all remaining load and store instructions defi

[Qemu-devel] [v2 01/13] Abandon GEN_VSX_* macros

2013-10-11 Thread Tom Musta
This patch eliminates the GEN_VSX_LXVNX/GEN_VSX_STXVNX macros which did not provide significant value above the general GEN_HANDLER_E macro. Signed-off-by: Tom Musta --- target-ppc/translate.c | 12 ++-- 1 files changed, 2 insertions(+), 10 deletions(-) diff --git a/target-ppc/transl

[Qemu-devel] [v2 02/13] Add lxsdx

2013-10-11 Thread Tom Musta
This patch adds the Load VSX Scalar Doubleowrd Indexed (lxsdx) instruction. The lower 8 bytes of the target register are undefined; this implementation leaves those bytes unaltered. Signed-off-by: Tom Musta --- target-ppc/translate.c | 16 1 files changed, 16 insertions(+),

[Qemu-devel] [v2 03/13] Add lxvdsx

2013-10-11 Thread Tom Musta
This patch adds the Load VSX Vector Doubleword & Splat Indexed (lxvdsx) instruction. Signed-off-by: Tom Musta --- target-ppc/translate.c | 16 1 files changed, 16 insertions(+), 0 deletions(-) diff --git a/target-ppc/translate.c b/target-ppc/translate.c index 6ee0d80..e5021c

[Qemu-devel] [v2 04/13] Add lxvw4x

2013-10-11 Thread Tom Musta
This patch adds the Load VSX Vector Word*4 Indexed (lxvw4x) instruction. V2: changed to use deposit_i64 per Richard Henderson's review. Signed-off-by: Tom Musta --- target-ppc/translate.c | 29 + 1 files changed, 29 insertions(+), 0 deletions(-) diff --git a/targ

[Qemu-devel] [v2 05/13] Add stxsdx

2013-10-11 Thread Tom Musta
This patch adds the Store VSX Scalar Doubleword Indexed (stxsdx) instruction. Signed-off-by: Tom Musta --- target-ppc/translate.c | 15 +++ 1 files changed, 15 insertions(+), 0 deletions(-) diff --git a/target-ppc/translate.c b/target-ppc/translate.c index 064cfa1..46b8380 100644

[Qemu-devel] [v2 06/13] Add stxvw4x

2013-10-11 Thread Tom Musta
This patch adds the Store VSX Vector Word*4 Indexed (stxvw4x) instruction. Signed-off-by: Tom Musta --- target-ppc/translate.c | 28 1 files changed, 28 insertions(+), 0 deletions(-) diff --git a/target-ppc/translate.c b/target-ppc/translate.c index 46b8380..8d53

[Qemu-devel] [v2 07/13] Add VSX Scalar Move Instructions

2013-10-11 Thread Tom Musta
This patch adds the VSX scalar move instructions: - xsabsdp (Scalar Absolute Value Double-Precision) - xsnabspd (Scalar Negative Absolute Value Double-Precision) - xsnegdp (Scalar Negate Double-Precision) - xscpsgndp (Scalar Copy Sign Double-Precision) A common generator macro (VSX_SCALA

[Qemu-devel] [v2 08/13] Add VSX Vector Move Instructions

2013-10-11 Thread Tom Musta
This patch adds the vector move instructions: - xvabsdp - Vector Absolute Value Double-Precision - xvnabsdp - Vector Negative Absolute Value Double-Precision - xvnegdp - Vector Negate Double-Precision - xvcpsgndp - Vector Copy Sign Double-Precision - xvabssp - Vector Absolute Value Sing

[Qemu-devel] [v2 09/13] Add Power7 VSX Logical Instructions

2013-10-11 Thread Tom Musta
This patch adds the VSX logical instructions that are defined by the Version 2.06 Power ISA (aka Power7): - xxland - xxlandc - xxlor - xxlxor - xxlnor Signed-off-by: Tom Musta --- target-ppc/translate.c | 29 + 1 files changed, 29 insertions(+), 0 d

[Qemu-devel] [v2 10/13] Add xxmrgh/xxmrgl

2013-10-11 Thread Tom Musta
This patch adds the VSX Merge High Word and VSX Merge Low Word instructions. V2: Now implemented using deposit (per Richard Henderson's comment) Signed-off-by: Tom Musta --- target-ppc/translate.c | 41 + 1 files changed, 41 insertions(+), 0 deletions(

[Qemu-devel] [v2 11/13] Add xxsel

2013-10-11 Thread Tom Musta
This patch adds the VSX Select (xxsel) instruction. The xxsel instruction has four VSR operands. Thus the xC instruction decoder is added. The xxsel instruction is massively overloaded in the opcode table since only bits 26 and 27 are opcode bits. This overloading is done in matrix fashion wit

[Qemu-devel] [PATCH] linux-user: pass correct parameter to do_shmctl()

2013-10-11 Thread Petar Jovanovic
From: Petar Jovanovic Fix shmctl issue by passing correct parameter buf to do_shmctl(). Signed-off-by: Petar Jovanovic --- linux-user/syscall.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/linux-user/syscall.c b/linux-user/syscall.c index 4a14a43..1f6492e 100644 --- a

[Qemu-devel] [v2 12/13] Add xxspltw

2013-10-11 Thread Tom Musta
This patch adds the VSX Splat Word (xxsplatw) instruction. This is the first instruction to use the UIM immediate field and consequently a decoder is also added. V2: reworked implementation per Richard Henderson's comments. Signed-off-by: Tom Musta --- target-ppc/translate.c | 31 ++

[Qemu-devel] [v2 13/13] Add xxsldwi

2013-10-11 Thread Tom Musta
This patch adds the VSX Shift Left Double by Word Immediate (xxsldwi) instruction. Signed-off-by: Tom Musta --- target-ppc/translate.c | 62 1 files changed, 62 insertions(+), 0 deletions(-) diff --git a/target-ppc/translate.c b/target-ppc/tra

[Qemu-devel] [Bug 1100843] Re: Live Migration Causes Performance Issues

2013-10-11 Thread Chris J Arges
I have verified this on my local machine using virt-manager's save memory, savevm/loadvm via the qemu monitor , and migrate via qemu monitor. ** Tags removed: verification-needed ** Tags added: verification-done -- You received this bug notification because you are a member of qemu- devel-ml, wh

Re: [Qemu-devel] [PATCH 1/2] rfifolock: add recursive FIFO lock

2013-10-11 Thread Stefan Hajnoczi
On Fri, Oct 11, 2013 at 04:55:31PM +0800, Wenchao Xia wrote: > > +/* Recursive FIFO lock > > + * > > + * This lock provides more features than a plain mutex: > > + * > > + * 1. Fairness - enforces FIFO order. > > + * 2. Nesting - can be taken recursively. > > + * 3. Contention callback - optional,

Re: [Qemu-devel] [PATCH] Use qemu-project.org domain name

2013-10-11 Thread Anthony Liguori
On Fri, Oct 11, 2013 at 12:55 AM, Stefan Hajnoczi wrote: > On Thu, Oct 10, 2013 at 07:42:30PM +0200, Stefan Weil wrote: >> Am 10.10.2013 11:39, schrieb Stefan Hajnoczi: >> > qemu.org is held by a third-party and no core community contributor has >> > access to the DNS configuration. This leaves t

Re: [Qemu-devel] [PATCH -V5] target-ppc: Fix page table lookup with kvm enabled

2013-10-11 Thread Alexander Graf
On 11.10.2013, at 13:13, Aneesh Kumar K.V wrote: > From: "Aneesh Kumar K.V" > > With kvm enabled, we store the hash page table information in the hypervisor. > Use ioctl to read the htab contents. Without this we get the below error when > trying to read the guest address > > (gdb) x/10 do_f

Re: [Qemu-devel] [RFC] SPAPR-PCI Hotplug Support in Qemu

2013-10-11 Thread Anthony Liguori
On Thu, Oct 10, 2013 at 3:25 PM, Mike Day wrote: > > [RFC] SPAPR-PCI Hotplug Support in Qemu > > Background: > ppc64 has a unique bus structure for PCI slots: each slot is connected > to its PHB by a pci switch. This is true in some IBM hardware as well as > paravirtual hardware (PAPR). > > SLOF f

Re: [Qemu-devel] [PATCH] map 64-bit PCI BARs at location provided by emulator

2013-10-11 Thread Igor Mammedov
On Fri, 11 Oct 2013 14:35:50 +0200 Gerd Hoffmann wrote: > On Fr, 2013-10-11 at 12:19 +0200, Igor Mammedov wrote: > > Currently 64-bit PCI BARs are unconditionally mapped by BIOS right > > over 4G + RamSizeOver4G location, which doesn't allow to reserve > > extra space before 64-bit PCI window. Fo

Re: [Qemu-devel] [RfC PATCH] e820: pass high memory too.

2013-10-11 Thread Igor Mammedov
On Fri, 11 Oct 2013 14:27:12 +0200 Gerd Hoffmann wrote: > On Fr, 2013-10-11 at 13:20 +0200, Igor Mammedov wrote: > > On Thu, 10 Oct 2013 14:54:29 +0200 > > Gerd Hoffmann wrote: > > > > > We have a fw_cfg entry to pass e820 entries from qemu to the firmware. > > > Today it's used to pass reserva

Re: [Qemu-devel] [PATCH] block/iscsi: remove .bdrv_has_zero_init

2013-10-11 Thread Paolo Bonzini
Il 11/10/2013 11:41, Kevin Wolf ha scritto: > Am 11.10.2013 um 09:59 hat Peter Lieven geschrieben: >> since commit 3ac21627 the default value changed to 0. >> >> Signed-off-by: Peter Lieven > > Reviewed-by: Kevin Wolf Applied to scsi-next branch, thanks. Paolo

Re: [Qemu-devel] [PATCH] block/iscsi: updated copyright

2013-10-11 Thread Paolo Bonzini
Il 11/10/2013 10:41, Peter Lieven ha scritto: > added myself to reflect recent work on the iscsi block driver. > > Signed-off-by: Peter Lieven > --- > block/iscsi.c |1 + > 1 file changed, 1 insertion(+) > > diff --git a/block/iscsi.c b/block/iscsi.c > index a660a1c..ee8cdd3 100644 > --- a/

  1   2   3   >