[Qemu-devel] [PATCH] tcg: pack TCGTemp to reduce size by 8 bytes

2015-03-20 Thread Emilio G. Cota
This brings down the size of the struct from 56 to 48 bytes. Signed-off-by: Emilio G. Cota --- tcg/tcg.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tcg/tcg.h b/tcg/tcg.h index add7f75..3276924 100644 --- a/tcg/tcg.h +++ b/tcg/tcg.h @@ -429,8 +429,8 @@ typedef struct TCGT

[Qemu-devel] [PATCH] translate-all: use bitmap helpers for PageDesc's bitmap

2015-03-20 Thread Emilio G. Cota
Note that this test if (b & ((1 << len) - 1)) can be simplified to if (b & 1) , since we know that iff the first bit of a tb is set, all other bits from that tb are set too. Signed-off-by: Emilio G. Cota --- translate-all.c | 39 +-- 1 file cha

[Qemu-devel] [PATCH] translate-all: remove superfluous #ifdef FOO || 1

2015-03-20 Thread Emilio G. Cota
It always evaluates to true. Signed-off-by: Emilio G. Cota --- translate-all.c | 4 1 file changed, 4 deletions(-) diff --git a/translate-all.c b/translate-all.c index 9f47ce7..11763c6 100644 --- a/translate-all.c +++ b/translate-all.c @@ -1334,8 +1334,6 @@ static inline void tb_alloc_page

Re: [Qemu-devel] GSoC Proposal: ARM Virtualization Extensions

2015-03-20 Thread Edgar E. Iglesias
On Sat, Mar 21, 2015 at 06:16:54AM +0300, Sergey Fedorov wrote: > On 20.03.2015 20:44, Peter Maydell wrote: > > On 20 March 2015 at 17:25, Merten Sach wrote: > >> I'm interested in participating in this years edition of Google Summer of > >> Code. > >> Initially I wanted to propose the implementa

Re: [Qemu-devel] GSoC Proposal: ARM Virtualization Extensions

2015-03-20 Thread Sergey Fedorov
On 20.03.2015 20:44, Peter Maydell wrote: > On 20 March 2015 at 17:25, Merten Sach wrote: >> I'm interested in participating in this years edition of Google Summer of >> Code. >> Initially I wanted to propose the implementation of ARMv7 >> virtualization extensions. I know this is not supported a

Re: [Qemu-devel] [PATCH] checkrom: Fix typo in error message

2015-03-20 Thread Kevin O'Connor
On Thu, Mar 19, 2015 at 05:23:19PM +0100, Andreas Färber wrote: > increate -> increase > > Signed-off-by: Andreas Färber Thanks - I added it to my queue. -Kevin

Re: [Qemu-devel] [PATCH 1/6 v7] target-tilegx: Firstly add TILE-Gx with minimized features

2015-03-20 Thread Chen Gang
On 3/21/15 07:30, Peter Maydell wrote: > On 20 March 2015 at 22:52, Chen Gang wrote: >> On 3/21/15 01:48, Peter Maydell wrote: >>> On 20 March 2015 at 15:25, Chen Gang wrote: 10 files changed, 3523 insertions(+) >>> >>> If you have any patch which has a diffstat like this then >>> you *must

Re: [Qemu-devel] [PATCH 1/6 v7] target-tilegx: Firstly add TILE-Gx with minimized features

2015-03-20 Thread Chen Gang
On 3/21/15 01:45, Richard Henderson wrote: > On 03/20/2015 08:25 AM, Chen Gang wrote: >> +/* >> + * The related functional description for bfextu in isa document: >> + * >> + * uint64_t mask = 0; >> + * mask = (-1ULL) ^ ((-1ULL << ((BFEnd - BFStart) & 63)) << 1); >> + * uint64_t rot_src = (((uint6

Re: [Qemu-devel] [PATCH 1/6 v7] target-tilegx: Firstly add TILE-Gx with minimized features

2015-03-20 Thread Peter Maydell
On 20 March 2015 at 22:52, Chen Gang wrote: > On 3/21/15 01:48, Peter Maydell wrote: >> On 20 March 2015 at 15:25, Chen Gang wrote: >>> 10 files changed, 3523 insertions(+) >> >> If you have any patch which has a diffstat like this then >> you *must* split it up into separate patches. Even >> 10

Re: [Qemu-devel] [PATCH 1/6 v6] target-tilegx: Firstly add TILE-Gx with minimized features

2015-03-20 Thread Chen Gang
On 3/21/15 01:03, Richard Henderson wrote: > On 03/18/2015 07:04 PM, Chen Gang wrote: >> For me, I am not quite sure about it, the related functional description >> is: >> >> rf[Dest] = signExtend32 ((int32_t) rf[SrcA] * (int32_t) rf[SrcB]); >> >> Do you mean it is equal to: >> >> rf[Dest] = si

Re: [Qemu-devel] [PATCH 1/6 v7] target-tilegx: Firstly add TILE-Gx with minimized features

2015-03-20 Thread Chen Gang
On 3/21/15 01:48, Peter Maydell wrote: > On 20 March 2015 at 15:25, Chen Gang wrote: >> It is the configure and build system support for TILE-Gx (tilegx will be >> used in configure and real sub-directory name). >> >> At present, it is linux-user only, and can finish the first system call >> (unam

Re: [Qemu-devel] Mac OS X no longer boots from cd

2015-03-20 Thread John Snow
On 03/20/2015 06:25 PM, Programmingkid wrote: On Mar 20, 2015, at 4:35 PM, John Snow wrote: If you feel like the CD rom boot failure is "new", I'd be very interested to know how recent that failure is. The macio RFC might help, but if you have the cycles to do a bisect upstream, I would a

Re: [Qemu-devel] Mac OS X no longer boots from cd

2015-03-20 Thread Programmingkid
On Mar 20, 2015, at 4:35 PM, John Snow wrote: > > > On 03/20/2015 04:08 PM, Programmingkid wrote: >> >> On Mar 20, 2015, at 4:03 AM, Mark Cave-Ayland wrote: >> >>> On 20/03/15 02:38, Programmingkid wrote: >>> On Mar 19, 2015, at 11:29 AM, Alexander Graf wrote: > > >

Re: [Qemu-devel] PCI passthrough of 40G ethernet interface (Openstack/KVM)

2015-03-20 Thread jacob jacob
On Thu, Mar 19, 2015 at 10:18 AM, Stefan Assmann wrote: > On 19.03.2015 15:04, jacob jacob wrote: >> Hi Stefan, >> have you been able to get PCI passthrough working without any issues >> after the upgrade? > > My XL710 fails to transfer regular TCP traffic (netperf). If that works > for you then y

[Qemu-devel] GSoC Proposal: ARM Virtualization Extensions

2015-03-20 Thread Merten Sach
Hi, I'm interested in participating in this years edition of Google Summer of Code. I the last years I did a lot of work on hypervisors. Currently I'm working with a hypervisor on ARM for my master degree. On previous projects using x86, the ability to do a quick debugging run using QEMU with S

Re: [Qemu-devel] [PATCH 0/6 v7] tilegx: Firstly add tilegx feature for linux-user

2015-03-20 Thread Chen Gang
Sorry, I sent send the mail by my another mail address (it is incorrect). I should send them again by my xili_gchen_5...@hotmail.com. Thanks. On 3/20/15 23:18, Chen Gang wrote: > After load elf64 binary, qemu tilegx can finish executing the first > system call (uname) successfully in _dl_discov

[Qemu-devel] [PATCH 0/6 v7] tilegx: Firstly add tilegx feature for linux-user

2015-03-20 Thread Chen Gang
After load elf64 binary, qemu tilegx can finish executing the first system call (uname) successfully in _dl_discover_osversion(), and return to __libc_start_main(). Chen Gang (6): target-tilegx: Firstly add TILE-Gx with minimized features linux-user: tilegx: Firstly add architecture related f

Re: [Qemu-devel] Mac OS X no longer boots from cd

2015-03-20 Thread John Snow
On 03/20/2015 04:08 PM, Programmingkid wrote: On Mar 20, 2015, at 4:03 AM, Mark Cave-Ayland wrote: On 20/03/15 02:38, Programmingkid wrote: On Mar 19, 2015, at 11:29 AM, Alexander Graf wrote: On 19.03.15 15:50, Programmingkid wrote: On Mar 19, 2015, at 3:18 AM, Alexander Graf wrote:

Re: [Qemu-devel] Mac OS X no longer boots from cd

2015-03-20 Thread Programmingkid
On Mar 20, 2015, at 4:03 AM, Mark Cave-Ayland wrote: > On 20/03/15 02:38, Programmingkid wrote: > >> On Mar 19, 2015, at 11:29 AM, Alexander Graf wrote: >> >>> >>> >>> On 19.03.15 15:50, Programmingkid wrote: On Mar 19, 2015, at 3:18 AM, Alexander Graf wrote: > > >>

Re: [Qemu-devel] [PATCH v4 00/20] block: transactionless incremental backup series

2015-03-20 Thread John Snow
On 03/20/2015 03:52 PM, Max Reitz wrote: On 2015-03-20 at 15:16, John Snow wrote: I've run out of cheeky jokes for my cover letters. This patchset enables the in-memory part of the incremental backup feature, without transactional support. Support for transactions was separated into a separa

Re: [Qemu-devel] [PATCH v4 00/20] block: transactionless incremental backup series

2015-03-20 Thread Max Reitz
On 2015-03-20 at 15:16, John Snow wrote: I've run out of cheeky jokes for my cover letters. This patchset enables the in-memory part of the incremental backup feature, without transactional support. Support for transactions was separated into a separate series which is also now available on-lis

Re: [Qemu-devel] [PATCH v4 19/20] iotests: add simple incremental backup case

2015-03-20 Thread Max Reitz
On 2015-03-20 at 15:17, John Snow wrote: Signed-off-by: John Snow --- tests/qemu-iotests/124 | 153 + tests/qemu-iotests/124.out | 4 +- 2 files changed, 155 insertions(+), 2 deletions(-) Reviewed-by: Max Reitz

Re: [Qemu-devel] [PATCH v4 20/20] iotests: add incremental backup failure recovery test

2015-03-20 Thread Max Reitz
On 2015-03-20 at 15:17, John Snow wrote: Test the failure case for incremental backups. Signed-off-by: John Snow --- blockdev.c | 1 - tests/qemu-iotests/124 | 55 ++ tests/qemu-iotests/124.out | 4 ++-- 3 files changed, 57

Re: [Qemu-devel] [PATCH v4 17/20] iotests: add invalid input incremental backup tests

2015-03-20 Thread Max Reitz
On 2015-03-20 at 15:17, John Snow wrote: Signed-off-by: John Snow --- tests/qemu-iotests/124 | 104 + tests/qemu-iotests/124.out | 5 +++ tests/qemu-iotests/group | 1 + 3 files changed, 110 insertions(+) create mode 100644 tests/qemu-

Re: [Qemu-devel] [PATCH v4 16/20] hbitmap: truncate tests

2015-03-20 Thread Max Reitz
On 2015-03-20 at 15:16, John Snow wrote: The general approach is to set bits close to the boundaries of where we are truncating and ensure that everything appears to have gone OK. We test growing and shrinking by different amounts: - Less than the granularity - Less than the granularity, but acr

Re: [Qemu-devel] [PATCH v4 04/20] qmp: Add block-dirty-bitmap-add and block-dirty-bitmap-remove

2015-03-20 Thread Max Reitz
On 2015-03-20 at 15:16, John Snow wrote: The new command pair is added to manage a user created dirty bitmap. The dirty bitmap's name is mandatory and must be unique for the same device, but different devices can have bitmaps with the same names. The granularity is an optional field. If it is no

Re: [Qemu-devel] [PATCH 3/3] block: add 'node-name' field to BLOCK_IMAGE_CORRUPTED

2015-03-20 Thread Max Reitz
On 2015-03-20 at 10:33, Alberto Garcia wrote: Since this event can occur in nodes that cannot have a device name associated, include also a field with the node name. Signed-off-by: Alberto Garcia --- block/qcow2.c | 8 ++-- docs/qmp/qmp-events.txt | 16 +--- qapi/

[Qemu-devel] [PATCH v4 20/20] iotests: add incremental backup failure recovery test

2015-03-20 Thread John Snow
Test the failure case for incremental backups. Signed-off-by: John Snow --- blockdev.c | 1 - tests/qemu-iotests/124 | 55 ++ tests/qemu-iotests/124.out | 4 ++-- 3 files changed, 57 insertions(+), 3 deletions(-) diff --git a/blo

[Qemu-devel] [PATCH v4 14/20] block: Ensure consistent bitmap function prototypes

2015-03-20 Thread John Snow
We often don't need the BlockDriverState for functions that operate on bitmaps. Remove it. Signed-off-by: John Snow Reviewed-by: Max Reitz --- block.c | 13 ++--- block/backup.c| 2 +- block/mirror.c| 26 ++ blockdev.c|

[Qemu-devel] [PATCH v4 16/20] hbitmap: truncate tests

2015-03-20 Thread John Snow
The general approach is to set bits close to the boundaries of where we are truncating and ensure that everything appears to have gone OK. We test growing and shrinking by different amounts: - Less than the granularity - Less than the granularity, but across a boundary - Less than sizeof(unsigned

Re: [Qemu-devel] [PATCH 2/3] block: use bdrv_get_device_or_node_name() in error messages

2015-03-20 Thread Max Reitz
On 2015-03-20 at 10:33, Alberto Garcia wrote: There are several error messages that identify a BlockDriverState by its device name. However those errors can be produced in nodes that don't have a device name associated. In those cases we should use bdrv_get_device_or_node_name() to fall back to

[Qemu-devel] [PATCH v4 11/20] qmp: add block-dirty-bitmap-clear

2015-03-20 Thread John Snow
Add bdrv_clear_dirty_bitmap and a matching QMP command, qmp_block_dirty_bitmap_clear that enables a user to reset the bitmap attached to a drive. This allows us to reset a bitmap in the event of a full drive backup. Signed-off-by: John Snow Reviewed-by: Max Reitz --- block.c | 8

[Qemu-devel] [PATCH v4 13/20] block: add BdrvDirtyBitmap documentation

2015-03-20 Thread John Snow
Signed-off-by: John Snow Reviewed-by: Max Reitz --- block.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/block.c b/block.c index 005b535..c2e1b27 100644 --- a/block.c +++ b/block.c @@ -60,11 +60,11 @@ * or enabled. A frozen bitmap can only abdicate() or re

[Qemu-devel] [PATCH v4 12/20] qmp: Add dirty bitmap status field in query-block

2015-03-20 Thread John Snow
Add the "frozen" status booleans, to inform clients when a bitmap is occupied doing a task. Signed-off-by: Fam Zheng Signed-off-by: John Snow Reviewed-by: Max Reitz --- block.c | 1 + qapi/block-core.json | 5 - 2 files changed, 5 insertions(+), 1 deletion(-) diff --git a/blo

[Qemu-devel] [PATCH v4 02/20] qapi: Add optional field "name" to block dirty bitmap

2015-03-20 Thread John Snow
From: Fam Zheng This field will be set for user created dirty bitmap. Also pass in an error pointer to bdrv_create_dirty_bitmap, so when a name is already taken on this BDS, it can report an error message. This is not global check, two BDSes can have dirty bitmap with a common name. Implemented

[Qemu-devel] [PATCH v4 17/20] iotests: add invalid input incremental backup tests

2015-03-20 Thread John Snow
Signed-off-by: John Snow --- tests/qemu-iotests/124 | 104 + tests/qemu-iotests/124.out | 5 +++ tests/qemu-iotests/group | 1 + 3 files changed, 110 insertions(+) create mode 100644 tests/qemu-iotests/124 create mode 100644 tests/qemu-iotes

[Qemu-devel] [PATCH v4 10/20] qmp: Add support of "dirty-bitmap" sync mode for drive-backup

2015-03-20 Thread John Snow
For "dirty-bitmap" sync mode, the block job will iterate through the given dirty bitmap to decide if a sector needs backup (backup all the dirty clusters and skip clean ones), just as allocation conditions of "top" sync mode. Signed-off-by: Fam Zheng Signed-off-by: John Snow Reviewed-by: Max Rei

[Qemu-devel] [PATCH v4 19/20] iotests: add simple incremental backup case

2015-03-20 Thread John Snow
Signed-off-by: John Snow --- tests/qemu-iotests/124 | 153 + tests/qemu-iotests/124.out | 4 +- 2 files changed, 155 insertions(+), 2 deletions(-) diff --git a/tests/qemu-iotests/124 b/tests/qemu-iotests/124 index 85675ec..ce2cda7 100644 --- a/te

[Qemu-devel] [PATCH v4 07/20] hbitmap: add hbitmap_merge

2015-03-20 Thread John Snow
We add a bitmap merge operation to assist in error cases where we wish to combine two bitmaps together. This is algorithmically O(bits) provided HBITMAP_LEVELS remains constant. For a full bitmap on a 64bit machine: sum(bits/64^k, k, 0, HBITMAP_LEVELS) ~= 1.01587 * bits We may be able to improve

[Qemu-devel] [PATCH v4 15/20] block: Resize bitmaps on bdrv_truncate

2015-03-20 Thread John Snow
Signed-off-by: John Snow Reviewed-by: Max Reitz --- block.c| 18 + include/qemu/hbitmap.h | 10 ++ util/hbitmap.c | 52 ++ 3 files changed, 80 insertions(+) diff --git a/block.c b/block.c index 81d17

[Qemu-devel] [PATCH v4 09/20] block: Add bitmap successors

2015-03-20 Thread John Snow
A bitmap successor is an anonymous BdrvDirtyBitmap that is intended to be created just prior to a sensitive operation (e.g. Incremental Backup) that can either succeed or fail, but during the course of which we still want a bitmap tracking writes. On creating a successor, we "freeze" the parent bi

[Qemu-devel] [PATCH v4 18/20] iotests: add QMP event waiting queue

2015-03-20 Thread John Snow
A filter is added to allow callers to request very specific events to be pulled from the event queue, while leaving undesired events still in the stream. This allows to poll for completion data for multiple asynchronous events in any arbitrary order. Signed-off-by: John Snow Reviewed-by: Max Rei

[Qemu-devel] [PATCH v4 01/20] docs: incremental backup documentation

2015-03-20 Thread John Snow
Reviewed-by: Max Reitz Signed-off-by: John Snow --- docs/bitmaps.md | 311 1 file changed, 311 insertions(+) create mode 100644 docs/bitmaps.md diff --git a/docs/bitmaps.md b/docs/bitmaps.md new file mode 100644 index 000..ad8c33b --

[Qemu-devel] [PATCH v4 05/20] block: Introduce bdrv_dirty_bitmap_granularity()

2015-03-20 Thread John Snow
This returns the granularity (in bytes) of dirty bitmap, which matches the QMP interface and the existing query interface. Signed-off-by: John Snow Reviewed-by: Max Reitz Reviewed-by: Eric Blake Reviewed-by: Stefan Hajnoczi --- block.c | 8 ++-- include/block/block.h | 1 +

[Qemu-devel] [PATCH v4 06/20] hbitmap: cache array lengths

2015-03-20 Thread John Snow
As a convenience: between incremental backups, bitmap migrations and bitmap persistence we seem to need to recalculate these a lot. Because the lengths are a little bit-twiddly, let's just solidly cache them and be done with it. Reviewed-by: Max Reitz Signed-off-by: John Snow --- util/hbitmap.

[Qemu-devel] [PATCH v4 00/20] block: transactionless incremental backup series

2015-03-20 Thread John Snow
I've run out of cheeky jokes for my cover letters. This patchset enables the in-memory part of the incremental backup feature, without transactional support. Support for transactions was separated into a separate series which is also now available on-list. Getting this portion of the series commi

[Qemu-devel] [PATCH v4 04/20] qmp: Add block-dirty-bitmap-add and block-dirty-bitmap-remove

2015-03-20 Thread John Snow
The new command pair is added to manage a user created dirty bitmap. The dirty bitmap's name is mandatory and must be unique for the same device, but different devices can have bitmaps with the same names. The granularity is an optional field. If it is not specified, we will choose a default granu

[Qemu-devel] [PATCH v4 08/20] block: Add bitmap disabled status

2015-03-20 Thread John Snow
Add a status indicating the enabled/disabled state of the bitmap. A bitmap is by default enabled, but you can lock the bitmap into a read-only state by setting disabled = true. A previous version of this patch added a QMP interface for changing the state of the bitmap, but it has since been remove

[Qemu-devel] [PATCH v4 03/20] qmp: Ensure consistent granularity type

2015-03-20 Thread John Snow
We treat this field with a variety of different types everywhere in the code. Now it's just uint32_t. Reviewed-by: Eric Blake Reviewed-by: Max Reitz Reviewed-by: Stefan Hajnoczi Signed-off-by: John Snow --- block.c | 11 ++- block/mirror.c| 4 ++-- inclu

Re: [Qemu-devel] [PATCH 1/3] block: add bdrv_get_device_or_node_name()

2015-03-20 Thread Max Reitz
On 2015-03-20 at 10:33, Alberto Garcia wrote: This function gets the device name associated with a BlockDriverState, or its node name if the device name is empty. Signed-off-by: Alberto Garcia --- block.c | 9 + block/quorum.c| 5 + include/block/block.h |

[Qemu-devel] [PATCH v2] hw/arm/virt: Fix memory leak reported by Coverity

2015-03-20 Thread Stefan Weil
As the conditional statement had to be split anyway, we can also add a better error report message. Signed-off-by: Stefan Weil --- v2: Fix commit message. hw/arm/virt.c |9 - 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/hw/arm/virt.c b/hw/arm/virt.c index b652b07..7

[Qemu-devel] [PATCH v2] hw/arm/vexpress: Fix memory leak reported by Coverity

2015-03-20 Thread Stefan Weil
As the conditional statement had to be split anyway, we can also add a better error report message. Signed-off-by: Stefan Weil --- v2: Fix commit message. Thank you to Eric Blake for his hint. hw/arm/vexpress.c | 11 +-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/h

Re: [Qemu-devel] [PATCH] hw/arm/vexpress: Fix memory leak reported by Coverity

2015-03-20 Thread Stefan Weil
Am 15.03.2015 um 03:33 schrieb Eric Blake: On 03/14/2015 04:59 AM, Stefan Weil wrote: As the conditional statement had to be splitted anyway, we can also s/splitted/split/ (one of those weird English words that has the same spelling for both present and past tenses) Thanks. This is fixed in v

Re: [Qemu-devel] [PATCH] hw/arm/vexpress: Fix memory leak reported by Coverity

2015-03-20 Thread Stefan Weil
Am 14.03.2015 um 12:50 schrieb Stefan Weil: As the conditional statement had to be splitted anyway, we can also add a better error report message. Signed-off-by: Stefan Weil --- hw/arm/virt.c | 11 +-- 1 file changed, 9 insertions(+), 2 deletions(-) Superseded by new patch v2.

Re: [Qemu-devel] [PATCH v5 13/45] ram_debug_dump_bitmap: Dump a migration bitmap as text

2015-03-20 Thread Dr. David Alan Gilbert
* David Gibson (da...@gibson.dropbear.id.au) wrote: > On Wed, Feb 25, 2015 at 04:51:36PM +, Dr. David Alan Gilbert (git) wrote: > > From: "Dr. David Alan Gilbert" > > > > Misses out lines that are all the expected value so the output > > can be quite compact depending on the circumstance. > >

Re: [Qemu-devel] [PATCH v2 5/5] fw_cfg: insert fw_cfg file blobs via qemu cmdline

2015-03-20 Thread Gabriel L. Somlo
On Fri, Mar 20, 2015 at 02:01:25PM -0400, Gabriel L. Somlo wrote: > On Thu, Mar 19, 2015 at 06:38:53PM +0100, Laszlo Ersek wrote: > > H... that's messy, again. fw_cfg is built into the qemu binary only > > if you have CONFIG_SOFTMMU. I guess something like this should work: > > > > #ifdef CONF

Re: [Qemu-devel] [PATCH v2 4/5] fw_cfg: prohibit insertion of duplicate fw_cfg file names

2015-03-20 Thread Laszlo Ersek
On 03/20/15 15:34, Gabriel L. Somlo wrote: > On Fri, Mar 20, 2015 at 07:51:06AM +0100, Laszlo Ersek wrote: >> Here's an idea I had this morning. >> >> This series gives equal rank to fw_cfg file names that originate >> internally and those that come from the user, via the command line. >> >> That m

Re: [Qemu-devel] [PATCH v5 12/45] Return path: Source handling of return path

2015-03-20 Thread Dr. David Alan Gilbert
* David Gibson (da...@gibson.dropbear.id.au) wrote: > On Wed, Feb 25, 2015 at 04:51:35PM +, Dr. David Alan Gilbert (git) wrote: > > From: "Dr. David Alan Gilbert" > > > > Open a return path, and handle messages that are received upon it. > > > > Signed-off-by: Dr. David Alan Gilbert > > ---

Re: [Qemu-devel] [PATCH for-2.3] linux-user: fix emulation of splice syscall

2015-03-20 Thread Andreas Färber
Am 10.03.2015 um 17:27 schrieb Peter Maydell: > On 10 March 2015 at 16:23, Andreas Schwab wrote: >> Peter Maydell writes: >> >>> What's changed? >> >> Only the title. > > Please don't drop all the reviewed-by: you got on the first > version, then! i.e., Reviewed-by: Peter Maydell Ping for -r

Re: [Qemu-devel] [PATCH V6 for-2.3 00/26] hw/pc: implement multiple primary busses for pc machines

2015-03-20 Thread Peter Maydell
On 20 March 2015 at 17:56, Paolo Bonzini wrote: > I understand, but you have to draw a line somewhere. The easiest place > to draw the line is 2.3.0-rc0. I definitely agree -- looking at the diffstat and the scope of the patch, this is definitely not suitable for committing to master after hardf

Re: [Qemu-devel] [PATCH v2 5/5] fw_cfg: insert fw_cfg file blobs via qemu cmdline

2015-03-20 Thread Gabriel L. Somlo
On Thu, Mar 19, 2015 at 06:38:53PM +0100, Laszlo Ersek wrote: > On 03/19/15 01:18, Gabriel L. Somlo wrote: > > Allow user supplied files to be inserted into the fw_cfg > > device before starting the guest. Since fw_cfg_add_file() > > already disallows duplicate fw_cfg file names, qemu will > > exit

Re: [Qemu-devel] [PATCH V6 for-2.3 00/26] hw/pc: implement multiple primary busses for pc machines

2015-03-20 Thread Paolo Bonzini
On 20/03/2015 09:37, Marcel Apfelbaum wrote: >> I think this should not be committed to 2.3 after -rc0. > While I agree that looking at the above the series do not > fit for 2.3, I still think the risk is not big because: > - The biggest amount of modifications are in pci_bus/pci_host > and *is

Re: [Qemu-devel] [PATCH 1/6 v7] target-tilegx: Firstly add TILE-Gx with minimized features

2015-03-20 Thread Peter Maydell
On 20 March 2015 at 15:25, Chen Gang wrote: > It is the configure and build system support for TILE-Gx (tilegx will be > used in configure and real sub-directory name). > > At present, it is linux-user only, and can finish the first system call > (uname) execution in __libc_start_main(). > > Signe

Re: [Qemu-devel] [PATCH 1/6 v7] target-tilegx: Firstly add TILE-Gx with minimized features

2015-03-20 Thread Richard Henderson
On 03/20/2015 08:25 AM, Chen Gang wrote: > +/* > + * The related functional description for bfextu in isa document: > + * > + * uint64_t mask = 0; > + * mask = (-1ULL) ^ ((-1ULL << ((BFEnd - BFStart) & 63)) << 1); > + * uint64_t rot_src = (((uint64_t) rf[SrcA]) >> BFStart) > + *

Re: [Qemu-devel] GSoC Proposal: ARM Virtualization Extensions

2015-03-20 Thread Peter Maydell
On 20 March 2015 at 17:25, Merten Sach wrote: > I'm interested in participating in this years edition of Google Summer of > Code. > Initially I wanted to propose the implementation of ARMv7 > virtualization extensions. I know this is not supported at the moment. > Also based on the documentation

Re: [Qemu-devel] [PATCH for-2.3 0/4] ahci: fix big endian PIO failures

2015-03-20 Thread Andreas Färber
Am 20.03.2015 um 01:24 schrieb John Snow: > Two issues were unearthed from ahci-test on ppc64: > > (1) The ahci_populate_sglist function had endian issues, > which is only likely to impact PIO transfers for buffers > greater than one sector, and > > (2) multiple-sector PIO which I attempt

[Qemu-devel] [PATCH] acpi: Add missing GCC_FMT_ATTR to local function

2015-03-20 Thread Stefan Weil
This fixes these gcc warnings (not enabled in default build): hw/acpi/aml-build.c:83:5: warning: function might be possible candidate for ‘gnu_printf’ format attribute [-Wsuggest-attribute=format] hw/acpi/aml-build.c:88:5: warning: function might be possible candidate for ‘gnu_printf’ format at

Re: [Qemu-devel] [PATCH 1/6 v6] target-tilegx: Firstly add TILE-Gx with minimized features

2015-03-20 Thread Richard Henderson
On 03/18/2015 07:04 PM, Chen Gang wrote: > For me, I am not quite sure about it, the related functional description > is: > > rf[Dest] = signExtend32 ((int32_t) rf[SrcA] * (int32_t) rf[SrcB]); > > Do you mean it is equal to: > > rf[Dest] = signExtend32 (rf[SrcA] * rf[SrcB]); Yes, of course.

Re: [Qemu-devel] [for-2.3 PATCH v2] gitignore: Ignore more .pod files.

2015-03-20 Thread Stefan Weil
Am 20.03.2015 um 17:30 schrieb Eric Blake: kvm_stat.{1,pod} started showing up as untracked files in my directory, and I nearly accidentally merged them into a commit with my usual habit of 'git add .'. Rather than spelling out each such file, just ignore the entire pattern. Signed-off-by: Eric

[Qemu-devel] [PATCH] Probe via #define check for FreeBSD, NetBSD and DragonFly BSD

2015-03-20 Thread Wei Liu
Signed-off-by: Wei Liu --- configure | 6 ++ 1 file changed, 6 insertions(+) diff --git a/configure b/configure index 589798e..ceacd81 100755 --- a/configure +++ b/configure @@ -441,6 +441,12 @@ elif check_define _WIN32 ; then targetos='MINGW32' elif check_define __OpenBSD__ ; then ta

[Qemu-devel] [for-2.3 PATCH v2] gitignore: Ignore more .pod files.

2015-03-20 Thread Eric Blake
kvm_stat.{1,pod} started showing up as untracked files in my directory, and I nearly accidentally merged them into a commit with my usual habit of 'git add .'. Rather than spelling out each such file, just ignore the entire pattern. Signed-off-by: Eric Blake CC: qemu-triv...@nongnu.org --- v2:

Re: [Qemu-devel] [for-2.3 PATCH] gitignore: Ignore more .pod files.

2015-03-20 Thread Eric Blake
On 03/20/2015 10:26 AM, Eric Blake wrote: > kvm_stat.{1,pod} started showing up as untracked files in my > directory, and I nearly accidentally merged them into a commit > with my usual habit of 'git add .'. Rather than naming every > such generated doc file, just exclude the pattern. > > Signed-

[Qemu-devel] [for-2.3 PATCH] gitignore: Ignore more .pod files.

2015-03-20 Thread Eric Blake
kvm_stat.{1,pod} started showing up as untracked files in my directory, and I nearly accidentally merged them into a commit with my usual habit of 'git add .'. Rather than naming every such generated doc file, just exclude the pattern. Signed-off-by: Eric Blake CC: qemu-triv...@nongnu.org --- .

[Qemu-devel] [PATCH 6/6 v7] linux-user/syscall.c: conditionalize syscalls which are not defined in tilegx

2015-03-20 Thread Chen Gang
For tilegx, several syscall macros are not supported, so switch them to avoid building break. Signed-off-by: Chen Gang --- linux-user/syscall.c | 50 +- 1 file changed, 49 insertions(+), 1 deletion(-) diff --git a/linux-user/syscall.c b/linux-user

[Qemu-devel] [PATCH 5/6 v7] linux-user: Support tilegx architecture in linux-user

2015-03-20 Thread Chen Gang
Add main working flow feature, system call processing feature, and elf64 tilegx binary loading feature, based on Linux kernel tilegx 64-bit implementation. Signed-off-by: Chen Gang --- include/elf.h| 2 ++ linux-user/elfload.c | 23 ++ linux-user/main.c| 86 +

[Qemu-devel] [PATCH 4/6 v7] linux-user: Support tilegx architecture in syscall

2015-03-20 Thread Chen Gang
Add tilegx architecture in "syscall_defs.h", all related features (ioctrl, and stat) are based on Linux kernel tilegx 64-bit implementation. Signed-off-by: Chen Gang --- linux-user/syscall_defs.h | 38 ++ 1 file changed, 34 insertions(+), 4 deletions(-) diff

[Qemu-devel] [PATCH 3/6 v7] linux-user: tilegx: Add target features support within qemu

2015-03-20 Thread Chen Gang
They are for target features within qemu which independent from outside. Signed-off-by: Chen Gang --- linux-user/tilegx/target_cpu.h | 35 +++ linux-user/tilegx/target_signal.h | 28 ++ linux-user/tilegx/target_structs.h | 48 +

[Qemu-devel] [PATCH 2/6 v7] linux-user: tilegx: Firstly add architecture related features

2015-03-20 Thread Chen Gang
They are based on Linux kernel tilegx architecture for 64 bit binary, also based on tilegx ABI reference document. Signed-off-by: Chen Gang --- linux-user/tilegx/syscall.h| 80 linux-user/tilegx/syscall_nr.h | 278 linux-user/tilegx/term

[Qemu-devel] [PATCH 1/6 v7] target-tilegx: Firstly add TILE-Gx with minimized features

2015-03-20 Thread Chen Gang
It is the configure and build system support for TILE-Gx (tilegx will be used in configure and real sub-directory name). At present, it is linux-user only, and can finish the first system call (uname) execution in __libc_start_main(). Signed-off-by: Chen Gang --- configure

[Qemu-devel] [PATCH 0/6 v7] tilegx: Firstly add tilegx feature for linux-user

2015-03-20 Thread Chen Gang
After load elf64 binary, qemu tilegx can finish executing the first system call (uname) successfully in _dl_discover_osversion(), and return to __libc_start_main(). Chen Gang (6): target-tilegx: Firstly add TILE-Gx with minimized features linux-user: tilegx: Firstly add architecture related f

Re: [Qemu-devel] [PATCH] target-tricore: fix DVINIT_HU/BU calculating overflow before result

2015-03-20 Thread Bastian Koppelmann
On 03/20/2015 02:34 PM, Bastian Koppelmann wrote: dvinit_hu/bu for ISA v1.3 calculate the higher part of the result, that is needed for the overflow bits, after calculating the overflow bits. Signed-off-by: Bastian Koppelmann --- target-tricore/translate.c | 29 +

Re: [Qemu-devel] [PATCH] e1000: work around win 8.0 boot hang

2015-03-20 Thread Wei Huang
On 02/24/2015 05:46 AM, Stefan Hajnoczi wrote: > On Tue, Feb 24, 2015 at 11:35 AM, Stefan Hajnoczi wrote: >> On Thu, Feb 19, 2015 at 08:24:19PM +0100, Radim Krčmář wrote: >>> Window 8.0 driver has a particular behavior for a small time frame after >>> it enables rx interrupts: the interrupt han

Re: [Qemu-devel] [PATCH v3 for-2.3 0/2] numa: Fix default VCPUs to node mapping

2015-03-20 Thread Igor Mammedov
On Fri, 20 Mar 2015 11:52:46 -0300 Eduardo Habkost wrote: > On Fri, Mar 20, 2015 at 11:24:14AM +0100, Igor Mammedov wrote: > > On Thu, 19 Mar 2015 17:09:20 + > > Igor Mammedov wrote: > > > > > Changes since v2: > > > - fix spelling errors > > > - split out PC hunks itno a separate patch >

Re: [Qemu-devel] [PATCH v3 for-2.3 0/2] numa: Fix default VCPUs to node mapping

2015-03-20 Thread Eduardo Habkost
On Fri, Mar 20, 2015 at 11:24:14AM +0100, Igor Mammedov wrote: > On Thu, 19 Mar 2015 17:09:20 + > Igor Mammedov wrote: > > > Changes since v2: > > - fix spelling errors > > - split out PC hunks itno a separate patch > > > > Igor Mammedov (2): > > numa: introduce machine callback for VCPU

[Qemu-devel] [PATCH 1/3] block: add bdrv_get_device_or_node_name()

2015-03-20 Thread Alberto Garcia
This function gets the device name associated with a BlockDriverState, or its node name if the device name is empty. Signed-off-by: Alberto Garcia --- block.c | 9 + block/quorum.c| 5 + include/block/block.h | 1 + 3 files changed, 11 insertions(+), 4 deletions

[Qemu-devel] [PATCH 3/3] block: add 'node-name' field to BLOCK_IMAGE_CORRUPTED

2015-03-20 Thread Alberto Garcia
Since this event can occur in nodes that cannot have a device name associated, include also a field with the node name. Signed-off-by: Alberto Garcia --- block/qcow2.c | 8 ++-- docs/qmp/qmp-events.txt | 16 +--- qapi/block-core.json| 13 - 3 files chan

Re: [Qemu-devel] [PATCH v2 4/5] fw_cfg: prohibit insertion of duplicate fw_cfg file names

2015-03-20 Thread Gabriel L. Somlo
On Fri, Mar 20, 2015 at 07:51:06AM +0100, Laszlo Ersek wrote: > Here's an idea I had this morning. > > This series gives equal rank to fw_cfg file names that originate > internally and those that come from the user, via the command line. > > That means that whenever qemu developers want to introd

[Qemu-devel] [PATCH 2/3] block: use bdrv_get_device_or_node_name() in error messages

2015-03-20 Thread Alberto Garcia
There are several error messages that identify a BlockDriverState by its device name. However those errors can be produced in nodes that don't have a device name associated. In those cases we should use bdrv_get_device_or_node_name() to fall back to the node name and produce a more meaningful mess

[Qemu-devel] [PATCH v3 0/3] Add bdrv_get_device_or_node_name()

2015-03-20 Thread Alberto Garcia
One more try. v3: - The node-name field in BLOCK_IMAGE_CORRUPTED is now Since: 2.4 - Remove the QERR_ macros instead of updating them. The text message is adapted to each case where applicable, and 'device' is renamed to 'node' only where it makes sense. v2: - bdrv_get_device_or_node_name() i

Re: [Qemu-devel] [PATCH RFC for-2.3 1/1] block: New command line option --no-format-probing

2015-03-20 Thread Eric Blake
On 03/20/2015 08:19 AM, Markus Armbruster wrote: >> If (a working version of) this makes it in 2.3, libvirt WILL use it in >> the next release. It will take me less than 5 minutes to write up the >> libvirt patch, as long as the new option is advertised via >> query-command-line-options (which me

Re: [Qemu-devel] [PATCH RFC for-2.3 0/1] block: New command line option --no-format-probing

2015-03-20 Thread Markus Armbruster
Eric Blake writes: > On 03/20/2015 07:05 AM, Markus Armbruster wrote: >> First of all, my apologies for being so late with this. I realized >> part way through the current development cycle that I couldn't do both >> the error work and my half of the block probing work we discussed back >> in No

Re: [Qemu-devel] [PATCH RFC for-2.3 1/1] block: New command line option --no-format-probing

2015-03-20 Thread Markus Armbruster
Eric Blake writes: > On 03/20/2015 07:49 AM, Max Reitz wrote: > > Disabling probing entirely is a (hamfisted) way to ensure you always > specify the format. > > >>> >>> I'd appreciate opinions on whether this is wanted for 2.3. If it is, >>> I'll post a version that actually works. >

[Qemu-devel] [RFC PATCH] qemu: enforce no format probing when possible

2015-03-20 Thread Eric Blake
qemu 2.3 added an option, with this documentation: | Probing is convenient, but probing untrusted raw images is insecure | (CVE-2008-2004). To avoid it, users should always specify raw format | explicitly. This isn't trivial, and even sophisticated users have | gotten it wrong (libvirt CVE-2010-

[Qemu-devel] [PATCH] mips_malta: use compat props to avoid loading efi-pcnet.rom

2015-03-20 Thread Leon Alrae
Currently qemu-system-mips aborts if it fails to find efi-pcnet.rom file which does not make sense. NIC on Malta board should not require x86 firmware. Reported-by: Maciej W. Rozycki Suggested-by: Gerd Hoffmann Signed-off-by: Leon Alrae --- Hi, This fixes a problem reported some time ago: http

[Qemu-devel] [PATCH v2 5/6] virtio-input: emulated devices

2015-03-20 Thread Gerd Hoffmann
This patch adds the virtio-input-hid base class and virtio-{keyboard,mouse,tablet} subclasses building on the base class. They are hooked up to the qemu input core and deliver input events to the guest like all other hid devices (ps/2 kbd, usb tablet, ...). Using them is as simple as adding "-devi

[Qemu-devel] [PATCH v2 3/6] virtio-input: add linux/input.h

2015-03-20 Thread Gerd Hoffmann
Linux input layer (evdev) header file. Signed-off-by: Gerd Hoffmann --- include/standard-headers/linux/input.h | 1198 scripts/update-linux-headers.sh|4 +- 2 files changed, 1201 insertions(+), 1 deletion(-) create mode 100644 include/standard-header

[Qemu-devel] [PATCH v2 2/6] virtio-input: add virtio_input.h

2015-03-20 Thread Gerd Hoffmann
Using scripts/update-linux-headers.sh with linux-input kernel branch. Signed-off-by: Gerd Hoffmann --- include/standard-headers/linux/virtio_ids.h | 1 + include/standard-headers/linux/virtio_input.h | 75 +++ 2 files changed, 76 insertions(+) create mode 100644 inclu

[Qemu-devel] [PATCH v2 6/6] virtio-input: evdev passthrough

2015-03-20 Thread Gerd Hoffmann
This allows to assign host input devices to the guest: qemu -device virto-input-host-pci,evdev=/dev/input/event The guest gets exclusive access to the input device, so be careful with assigning the keyboard if you have only one connected to your machine. Signed-off-by: Gerd Hoffmann --- hw/inp

[Qemu-devel] [PATCH v2 1/6] pci: add PCI_CLASS_INPUT_*

2015-03-20 Thread Gerd Hoffmann
Signed-off-by: Gerd Hoffmann --- include/hw/pci/pci_ids.h | 7 +++ 1 file changed, 7 insertions(+) diff --git a/include/hw/pci/pci_ids.h b/include/hw/pci/pci_ids.h index d7be386..dfccefc 100644 --- a/include/hw/pci/pci_ids.h +++ b/include/hw/pci/pci_ids.h @@ -46,6 +46,13 @@ #define PCI_CLAS

[Qemu-devel] [PATCH v2 4/6] virtio-input: core code & base class

2015-03-20 Thread Gerd Hoffmann
This patch adds virtio-input support to qemu. It brings a abstract base class providing core support, other classes can build on it to actually implement input devices. virtio-input basically sends linux input layer events (evdev) over virtio. Signed-off-by: Gerd Hoffmann --- hw/input/Makefile

  1   2   >