On Tue, 28 Apr 2015 10:16:04 +0200
"Michael S. Tsirkin" wrote:
> On Tue, Apr 28, 2015 at 10:04:15AM +0200, Cornelia Huck wrote:
> > On Tue, 28 Apr 2015 09:14:07 +0200
> > "Michael S. Tsirkin" wrote:
> >
> > > On Tue, Apr 28, 2015 at 02:13:20PM +0800, Jason Wang wrote:
> > > >
> > > >
> > > >
On Tue, Apr 28, 2015 at 01:21:14PM +0300, Marcel Apfelbaum wrote:
> On 04/28/2015 11:43 AM, Michael S. Tsirkin wrote:
> >On Tue, Apr 28, 2015 at 11:39:42AM +0300, Marcel Apfelbaum wrote:
> >>On 04/27/2015 05:45 PM, Michael S. Tsirkin wrote:
> >>>On Mon, Apr 27, 2015 at 04:01:16PM +0300, Marcel Apfe
The error reporting in pci_nic_init() is quite erratic: Some errors
are printed directly with error_report(), and some are passed back
to the caller pci_nic_init_nofail() via an Error pointer.
Since pci_nic_init() is only used by pci_nic_init_nofail(), the
functions can be simply merged to clean up
On 2015/4/24 16:35, Dr. David Alan Gilbert wrote:
* Wen Congyang (we...@cn.fujitsu.com) wrote:
On 04/22/2015 07:18 PM, Dr. David Alan Gilbert wrote:
* zhanghailiang (zhang.zhanghaili...@huawei.com) wrote:
Hi,
ping ...
I will get to look at this again; but not until after next week.
The ma
On Tue, Apr 28, 2015 at 12:40:07PM +0200, Cornelia Huck wrote:
> On Tue, 28 Apr 2015 10:16:04 +0200
> "Michael S. Tsirkin" wrote:
>
> > On Tue, Apr 28, 2015 at 10:04:15AM +0200, Cornelia Huck wrote:
> > > On Tue, 28 Apr 2015 09:14:07 +0200
> > > "Michael S. Tsirkin" wrote:
> > >
> > > > On Tue,
On Tue, 28 Apr 2015 10:40:09 +0200
"Michael S. Tsirkin" wrote:
> On Tue, Apr 28, 2015 at 09:26:47AM +0200, Thomas Huth wrote:
> > On Mon, 27 Apr 2015 10:55:41 -0700
> > Peter Crosthwaite wrote:
> >
> > > On Mon, Apr 27, 2015 at 10:51 AM, Thomas Huth wrote:
> > > > The error reporting in pci_ni
On Tue, Apr 28, 2015 at 10:46:59AM +0300, Denis V. Lunev wrote:
> Plain image expansion spends a lot of time to update image file size.
> This seriously affects the performance. The following simple test
> qemu_img create -f parallels -o cluster_size=64k ./1.hds 64G
> qemu_io -n -c "write -P 0x
Eric Blake writes:
> Demonstrate that the qapi generator doesn't deal well with
> expressions that aren't up to par. Later patches will improve
> the expected results as the generator is made stricter. Only
> a few of the the added tests actually behaves sanely at
behave
> rejecting obvious pr
On Tue, Apr 28, 2015 at 10:46:42AM +0300, Denis V. Lunev wrote:
> Support write on Parallels images. The code is almost the same as one
> in the previous patch implemented scatter-gather IO for read.
>
> Signed-off-by: Denis V. Lunev
> CC: Roman Kagan
> CC: Kevin Wolf
> CC: Stefan Hajnoczi
> -
On Tue, Apr 28, 2015 at 10:46:58AM +0300, Denis V. Lunev wrote:
> This is preparational commit for tweaks in Parallels image expansion.
> The idea is that enlarge via truncate by one data block is slow. It
> would be much better to use fallocate via bdrv_write_zeroes and
> expand by some significan
David Gibson writes:
> On Wed, Apr 22, 2015 at 04:05:34PM +0530, Nikunj A Dadhania wrote:
>> Each hardware instance has a platform unique location code. The OF
>> device tree that describes a part of a hardware entity must include
>> the “ibm,loc-code” property with a value that represents the l
David Gibson writes:
> On Wed, Apr 22, 2015 at 04:05:33PM +0530, Nikunj A Dadhania wrote:
>> All the PCI enumeration and device node creation was off-loaded to
>> SLOF. With PCI hotplug support, code needed to be added to add device
>> node. This creates multiple copy of the code one in SLOF and
On 27 April 2015 at 19:02, Eduardo Habkost wrote:
> The following changes since commit e1a5476354d396773e4c555f126d752d4ae58fa9:
>
> Open 2.4 development tree (2015-04-25 22:05:07 +0100)
>
> are available in the git repository at:
>
> git://github.com/ehabkost/qemu.git tags/numa-pull-request
>
Juan Quintela wrote:
> Hi
>
> Please, send any topic that you are interested in covering.
As there are no topics for Today call, call got cancelled.
Thanks, Juan.
>
>
> Call details:
>
> By popular demand, a google calendar public entry with it
>
>
> https://www.google.com/calendar/embed?s
Eric Blake writes:
> Now that we know every expression is valid with regards to
> its keys, we can add further tests that those keys refer to
> valid types. With this patch, all uses of a type (the 'data':
> of command, type, union, alternate, and event; the 'returns':
> of command; the 'base':
On Tue, 28 Apr 2015 12:23:25 +0530
Nikunj A Dadhania wrote:
> Machines types can have different requirement for default ram
> size. Introduce a member in the machine class and set the current
> default_ram_size to 128MB.
>
> For QEMUMachine types override the value during the registration of
> t
On Tue, 28 Apr 2015 12:23:26 +0530
Nikunj A Dadhania wrote:
> Signed-off-by: Nikunj A Dadhania
> Reviewed-by: Igor Mammedov
> ---
> hw/ppc/spapr.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/hw/ppc/spapr.c b/hw/ppc/spapr.c
> index 61ddc79..6616650 100644
> --- a/hw/ppc/spapr.c
>
On Tue, 28 Apr 2015 12:55:40 +0200
"Michael S. Tsirkin" wrote:
> On Tue, Apr 28, 2015 at 12:40:07PM +0200, Cornelia Huck wrote:
> > On Tue, 28 Apr 2015 10:16:04 +0200
> > "Michael S. Tsirkin" wrote:
> >
> > > On Tue, Apr 28, 2015 at 10:04:15AM +0200, Cornelia Huck wrote:
> > > > On Tue, 28 Apr
On 28/04/2015 12:50, Thomas Huth wrote:
> The error reporting in pci_nic_init() is quite erratic: Some errors
> are printed directly with error_report(), and some are passed back
> to the caller pci_nic_init_nofail() via an Error pointer.
> Since pci_nic_init() is only used by pci_nic_init_nofail
Eric Blake writes:
> Previous commits demonstrated that the generator overlooked various
> bad naming situations:
> - types, commands, and events need a valid name
> - enum members must be valid names, when combined with prefix
> - union and alternate branches cannot be marked optional
>
> The se
On 28/04/2015 10:11, arei.gong...@huawei.com wrote:
> From: Gonglei
>
> Coveristy reports that variable prom_buf/params_buf going
> out of scope leaks the storage it points to.
>
> Cc: Aurelien Jarno
> Cc: Leon Alrae
> Signed-off-by: Gonglei
> ---
> hw/mips/mips_fulong2e.c | 1 +
> hw/mips/m
On 28/04/2015 11:18, Eric Auger wrote:
> Check callback now takes as third argument an Object **. In
> object_set_link_property, we pass the property child as argument.
> We also assign the *child before the check call so that enhanced
> check can be performed in the callback. In case the check f
From: Shannon Zhao
So far virtio-net-device can't expose host features to guest while
using virtio-mmio because it doesn't set DEFINE_VIRTIO_NET_FEATURES on
backend or transport. So the performance is low.
The host features belong to the backend while virtio-net-pci,
virtio-net-s390 and virtio-n
From: Shannon Zhao
The reason to do this is that the virtio-net-device can't expose host
features to guest while using virtio-mmio. So the performance is low.
The virtio-*-pci, virtio-*-s390, and virtio-*-ccw already have the
ability to forward property accesses to the backend child, by calling
From: Shannon Zhao
So far virtio-scsi-device can't expose host features to guest while
using virtio-mmio because it doesn't set DEFINE_VIRTIO_SCSI_FEATURES on
backend or transport.
The host features belong to the backends while virtio-scsi-pci,
virtio-scsi-s390 and virtio-scsi-ccw set the DEFINE
Am 07.04.2015 um 15:53 schrieb Denis V. Lunev:
> Replace call of cpu_is_bsp(s->cpu) which really returns
> !!(s->apicbase & MSR_IA32_APICBASE_BSP)
> with directly collected value. Due to this the tracepoint
> trace_cpu_get_apic_base((uint64_t)s->apicbase);
> will not be hit anymore in apic_re
Eric Blake writes:
> Referring to "type" as both a meta-type (built-in, enum, union,
> alternte, or struct) and a specific type (the name that the
> schema uses for declaring structs) is confusing. The confusion
> is only made worse by the fact that the generator mostly already
> refers to struc
Am 28.04.2015 um 11:11 schrieb arei.gong...@huawei.com:
> From: Gonglei
>
> Signed-off-by: Gonglei
> ---
> hw/virtio/vhost-user.c | 22 +++---
> 1 file changed, 11 insertions(+), 11 deletions(-)
>
> diff --git a/hw/virtio/vhost-user.c b/hw/virtio/vhost-user.c
> index aefe0bb..e
Eric Blake writes:
> Referring to "type" as both a meta-type (built-in, enum, union,
> alternte, or struct) and a specific type (the name that the
> schema uses for declaring structs) is confusing. Now that the
> generator accepts 'struct' as a synonym for 'type', update all
> documentation to u
Eric Blake writes:
> Referring to "type" as both a meta-type (built-in, enum, union,
> alternte, or struct) and a specific type (the name that the
> schema uses for declaring structs) is confusing. Finish up the
> conversion to using "struct" in qapi schema by coverting all
> users, then remove
Eric Blake writes:
> In the testsuite, UserDefTwo and UserDefNested were identical
> structs other than the member names. Reduce code duplication by
> having just one type, and choose names that also favor reuse.
> This will also make it easier for a later patch to get rid of
> inline nested typ
CP0.PageGrain.ELPA enables support for large physical addresses. This field
is encoded as follows:
0: Large physical address support is disabled.
1: Large physical address support is enabled.
If this bit is a 1, the following changes occur to coprocessor 0 registers:
- The PFNX field of the EntryL
Hi,
This patchset adds large physical address support in MIPS, specifically:
* eXtended Physical Addressing (XPA)
* Large Physical Addressing (LPA)
XPA and LPA are enabled in MIPS32R5-generic and MIPS64R6-generic cores
respectively.
The series applies on top of the Config5.FRE patches.
Regards,
Extend EntryLo0, EntryLo1, LLAddr and TagLo from 32 to 64 bits in MIPS32.
Signed-off-by: Leon Alrae
---
target-mips/cpu.h | 14 +++---
target-mips/machine.c | 20 ++--
target-mips/op_helper.c | 8
target-mips/translate.c | 5 +++--
4 files changed, 24 i
Since PFNX is now supported the bits 31:30 have to be cleared.
Signed-off-by: Leon Alrae
---
target-mips/translate.c | 12 ++--
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/target-mips/translate.c b/target-mips/translate.c
index f95b655..a41fc98 100644
--- a/target-mips/
Enable XPA in MIPS32R5-generic and LPA in MIPS64R6-generic.
Signed-off-by: Leon Alrae
---
target-mips/translate_init.c | 15 +--
1 file changed, 9 insertions(+), 6 deletions(-)
diff --git a/target-mips/translate_init.c b/target-mips/translate_init.c
index af6fb7a..8674118 100644
---
Update tlb->PFN to contain PFN concatenated with PFNX. PFNX is 0 if large
physical address is not supported.
Signed-off-by: Leon Alrae
---
target-mips/op_helper.c | 32 ++--
1 file changed, 26 insertions(+), 6 deletions(-)
diff --git a/target-mips/op_helper.c b/targe
Implement MTHC0 and MFHC0 instructions. In MIPS32 they allow to access
upper word of extended to 64-bits CP0 registers.
In MIPS64, when CP0 destination register specified is the EntryLo0 or
EntryLo1, bits 1:0 of the GPR appear at bits 31:30 of EntryLo0 or
EntryLo1. This is to compensate for RI and
Signed-off-by: Leon Alrae
---
target-mips/translate_init.c | 9 -
1 file changed, 9 deletions(-)
diff --git a/target-mips/translate_init.c b/target-mips/translate_init.c
index 8e088c9..af6fb7a 100644
--- a/target-mips/translate_init.c
+++ b/target-mips/translate_init.c
@@ -553,9 +553,6 @
On Tue, Apr 28, 2015 at 01:39:51PM +0200, Cornelia Huck wrote:
> On Tue, 28 Apr 2015 12:55:40 +0200
> "Michael S. Tsirkin" wrote:
>
> > On Tue, Apr 28, 2015 at 12:40:07PM +0200, Cornelia Huck wrote:
> > > On Tue, 28 Apr 2015 10:16:04 +0200
> > > "Michael S. Tsirkin" wrote:
> > >
> > > > On Tue,
On 27 April 2015 at 19:45, Eduardo Habkost wrote:
> Changes in v2:
> * Rebase to 0d81cdddaa40a1988b24657aeac19959cfad0fde
> * Rewrote commit message of 'target-i386: Make "level" and "xlevel"
> properties
>static', to mention it fixes a bug (missing check for realized device)
>
> The follow
On Tue, Apr 28, 2015 at 07:51:11PM +0800, shannon.z...@linaro.org wrote:
> From: Shannon Zhao
>
> The reason to do this is that the virtio-net-device can't expose host
> features to guest while using virtio-mmio. So the performance is low.
>
> The virtio-*-pci, virtio-*-s390, and virtio-*-ccw al
On 28 April 2015 at 13:48, Michael S. Tsirkin wrote:
> On Tue, Apr 28, 2015 at 07:51:11PM +0800, shannon.z...@linaro.org wrote:
>> From: Shannon Zhao
>> Here we move the host features to backends, involving
>> DEFINE_VIRTIO_NET_FEATURES, DEFINE_VIRTIO_SCSI_FEATURES. So the
>> virtio-mmio devices
Am 27.04.2015 um 19:07 hat John Snow geschrieben:
>
>
> On 04/27/2015 10:23 AM, Fam Zheng wrote:
> >Coverity spotted this.
> >
> >The field is 32 bits, but if it's possible to overflow in 32 bit
> >left shift.
> >
> >Signed-off-by: Fam Zheng
> >---
> > block/vmdk.c | 2 +-
> > 1 file changed, 1
On 2015/4/28 17:54, Igor Mammedov wrote:
> On Tue, 28 Apr 2015 17:06:00 +0800
> Shannon Zhao wrote:
>
>> On 2015/4/28 16:47, Michael S. Tsirkin wrote:
>>> On Tue, Apr 28, 2015 at 10:42:25AM +0200, Igor Mammedov wrote:
On Wed, 15 Apr 2015 21:25:08 +0800
Shannon Zhao wrote:
> Fr
Eric Blake writes:
> A future patch will be using a 'name':{dictionary} entry in the
> QAPI schema to specify a default value for an optional argument;
> but existing use of inline nested structs conflicts with that goal.
>
> More precisely, a definition in the QAPI schema associates a name
> wit
On Tue, Apr 28, 2015 at 01:52:40PM +0100, Peter Maydell wrote:
> On 28 April 2015 at 13:48, Michael S. Tsirkin wrote:
> > On Tue, Apr 28, 2015 at 07:51:11PM +0800, shannon.z...@linaro.org wrote:
> >> From: Shannon Zhao
> >> Here we move the host features to backends, involving
> >> DEFINE_VIRTIO_
On 28 April 2015 at 12:51, wrote:
> From: Shannon Zhao
>
> The reason to do this is that the virtio-net-device can't expose host
> features to guest while using virtio-mmio. So the performance is low.
>
> The virtio-*-pci, virtio-*-s390, and virtio-*-ccw already have the
> ability to forward pro
On Tue, Apr 28, 2015 at 02:06:35PM +0100, Peter Maydell wrote:
> On 28 April 2015 at 12:51, wrote:
> > From: Shannon Zhao
> >
> > The reason to do this is that the virtio-net-device can't expose host
> > features to guest while using virtio-mmio. So the performance is low.
> >
> > The virtio-*-p
On 28 April 2015 at 14:13, Michael S. Tsirkin wrote:
> The patches look correct to me too, but I want s390
> cleaned up so it does not include COMMON_FEATURES
> in 100 places, and I prefer merging it all together.
It seems a bit harsh to ask Shannon to do s390 cleanup when
he doesn't have any acc
On Tue, 28 Apr 2015 15:05:47 +0200
"Michael S. Tsirkin" wrote:
> On Tue, Apr 28, 2015 at 01:52:40PM +0100, Peter Maydell wrote:
> > On 28 April 2015 at 13:48, Michael S. Tsirkin wrote:
> > > On Tue, Apr 28, 2015 at 07:51:11PM +0800, shannon.z...@linaro.org wrote:
> > >> From: Shannon Zhao
> > >
Eric Blake writes:
> The handling of \ inside QAPI strings was less than ideal, and
> really only worked JSON's \/, \\, \", and our extension of \'
> (an obvious extension, when you realize we use '' instead of ""
> for strings). For other things, like '\n', it resulted in a
> literal 'n' instea
On Tue, 28 Apr 2015 14:16:40 +0100
Peter Maydell wrote:
> On 28 April 2015 at 14:13, Michael S. Tsirkin wrote:
> > The patches look correct to me too, but I want s390
> > cleaned up so it does not include COMMON_FEATURES
> > in 100 places, and I prefer merging it all together.
>
> It seems a bi
The dirty bitmap functions are called from the block I/O processing
code. Make them visible to block_int.h users so they can be used
outside block.c.
Signed-off-by: Stefan Hajnoczi
---
block.c | 12
include/block/block_int.h | 4
2 files changed, 8 insertion
Move the code to install coroutine and aio emulation function pointers
in a BlockDriver to its own function.
Signed-off-by: Stefan Hajnoczi
---
block.c | 7 ++-
include/block/block_int.h | 8
2 files changed, 14 insertions(+), 1 deletion(-)
diff --git a/block.c b/
The bdrv_states list is a static variable in block.c.
bdrv_drain_all() and bdrv_flush_all() use this variable to iterate over
all drives.
The next patch will move bdrv_drain_all() and bdrv_flush_all() out of
block.c so it's necessary to switch to the public bdrv_next() interface.
Reviewed-by: Al
Note: This series is based on my block-next branch:
https://github.com/stefanha/qemu.git block-next
v4:
* Extracted two patches that change block.c code, now making the final
block/io.c patch purely code movement [Eric]
* Added missing block/io.c file (oops!) [Eric]
This series splits block.
On Tue, 28 Apr 2015 14:47:11 +0200
"Michael S. Tsirkin" wrote:
> On Tue, Apr 28, 2015 at 01:39:51PM +0200, Cornelia Huck wrote:
> > On Tue, 28 Apr 2015 12:55:40 +0200
> > "Michael S. Tsirkin" wrote:
> >
> > > On Tue, Apr 28, 2015 at 12:40:07PM +0200, Cornelia Huck wrote:
> > > > On Tue, 28 Apr
On 28/04/15 13:41, Leon Alrae wrote:
> Update tlb->PFN to contain PFN concatenated with PFNX. PFNX is 0 if large
> physical address is not supported.
>
> Signed-off-by: Leon Alrae
> ---
> target-mips/op_helper.c | 32 ++--
> 1 file changed, 26 insertions(+), 6 delet
Eric Blake writes:
> Our type inheritance for both 'struct' and for flat 'union' merges
> key/value pairs from the base class with those from the type in
> question. Although the C code currently boxes things so that there
> is a distinction between which member is referred to, the QMP wire
> fo
On 28/04/15 14:35, James Hogan wrote:
>
>
> On 28/04/15 13:41, Leon Alrae wrote:
>> Update tlb->PFN to contain PFN concatenated with PFNX. PFNX is 0 if large
>> physical address is not supported.
>>
>> Signed-off-by: Leon Alrae
>> ---
>> target-mips/op_helper.c | 32 ++--
Am 28.04.2015 um 15:27 hat Stefan Hajnoczi geschrieben:
> Note: This series is based on my block-next branch:
> https://github.com/stefanha/qemu.git block-next
>
> v4:
> * Extracted two patches that change block.c code, now making the final
>block/io.c patch purely code movement [Eric]
> * A
Eric Blake writes:
> We want to eventually allow qapi defaults, by making:
> 'data':{'*flag':'bool'}
> as shorthand for something like:
> 'data':{'flag':{'type':'bool', 'optional':true}}
> so that the default can be specified:
> 'data':{'flag':{'type':'bool', 'optional':true, 'default':true}}
On Tue, Apr 28, 2015 at 03:20:54PM +0200, Cornelia Huck wrote:
> On Tue, 28 Apr 2015 15:05:47 +0200
> "Michael S. Tsirkin" wrote:
>
> > On Tue, Apr 28, 2015 at 01:52:40PM +0100, Peter Maydell wrote:
> > > On 28 April 2015 at 13:48, Michael S. Tsirkin wrote:
> > > > On Tue, Apr 28, 2015 at 07:51:
On 04/28/15 04:13, Paolo Bonzini wrote:
>
>
> On 28/04/2015 00:46, Don Slutz wrote:
>> This adds one new inject command:
>>
>> inject-vmport-action
>>
>> And three guest info commands:
>>
>> vmport-guestinfo-set
>> vmport-guestinfo-get
>> query-vmport-guestinfo
>>
>> More details in qmp-commands.
On 27/04/2015 16:55, Michael S. Tsirkin wrote:
On Mon, Apr 27, 2015 at 02:19:47PM +0300, Gal Hammer wrote:
Signed-off-by: Gal Hammer
---
docs/specs/vmgenid.txt | 35 +++
1 file changed, 35 insertions(+)
create mode 100644 docs/specs/vmgenid.txt
diff --git a
Hello!
Please register the ACPI Vendor ID "QEMU" to the qemu emulator project.
Thanks!
--
MST
On Tue, Apr 28, 2015 at 03:24:19PM +0200, Cornelia Huck wrote:
> On Tue, 28 Apr 2015 14:16:40 +0100
> Peter Maydell wrote:
>
> > On 28 April 2015 at 14:13, Michael S. Tsirkin wrote:
> > > The patches look correct to me too, but I want s390
> > > cleaned up so it does not include COMMON_FEATURES
On 27/04/2015 17:56, Eric Blake wrote:
On 04/27/2015 05:19 AM, Gal Hammer wrote:
Signed-off-by: Gal Hammer
---
docs/specs/vmgenid.txt | 35 +++
1 file changed, 35 insertions(+)
create mode 100644 docs/specs/vmgenid.txt
diff --git a/docs/specs/vmgenid.txt b/
From: "Emilio G. Cota"
Since commit
b7b5233a "bsd-user/mmap.c: Don't try to override g_malloc/g_free"
the exception we make here for usermode has been unnecessary.
Get rid of it.
Signed-off-by: Emilio G. Cota
Message-Id: <1428610053-26148-1-git-send-email-c...@braap.org>
Signed-off-by: Paol
From: Fam Zheng
The function is a nop for user mode, so just remove them.
Signed-off-by: Fam Zheng
Message-Id: <1426496617-10702-3-git-send-email-f...@redhat.com>
Signed-off-by: Paolo Bonzini
---
bsd-user/main.c | 1 -
linux-user/main.c | 1 -
2 files changed, 2 deletions(-)
diff --git a/b
ioport.c is already able to split a 16-bit access into two 8-bit
accesses to consecutive ports. Tested with Epic Pinball.
Signed-off-by: Paolo Bonzini
---
hw/audio/sb16.c | 7 ---
1 file changed, 7 deletions(-)
diff --git a/hw/audio/sb16.c b/hw/audio/sb16.c
index 444eb9e..b052de5 100644
--
Remove 16-bit reads/writes, since ioport.c is able to synthesize them.
Remove the two MIDI registers (0x300 and 0x301) from gus_portio_list1,
and add the second MIDI register (0x301) to gus_portio_list2.
Tested with Second Reality.
Signed-off-by: Paolo Bonzini
---
hw/audio/gus.c | 20 +-
From: Fam Zheng
There could be a race condition when two processes call
address_space_map concurrently and both want to use the bounce buffer.
Add an in_use flag in BounceBuffer to sync it.
Signed-off-by: Fam Zheng
Message-Id: <1426496617-10702-2-git-send-email-f...@redhat.com>
Signed-off-by:
The following changes since commit e1a5476354d396773e4c555f126d752d4ae58fa9:
Open 2.4 development tree (2015-04-25 22:05:07 +0100)
are available in the git repository at:
git://github.com/bonzini/qemu.git tags/for-upstream
for you to fetch changes up to 20ee4c8245633f0430455b6f802f130e13ec6
ioport.c has not been using an alias since commit b40acf9 (ioport:
Switch dispatching to memory core layer, 2013-06-24). Remove the
obsolete comment.
Signed-off-by: Paolo Bonzini
---
ioport.c | 4
1 file changed, 4 deletions(-)
diff --git a/ioport.c b/ioport.c
index 783a3ae..eb954e3 10064
When an I/O port is more than 1 byte long, ioport.c is currently
creating "short" regions, for example 0x1ce-0x1ce for the 16-bit
Bochs index port. When I/O ports are memory mapped, and thus
accessed via a subpage_ops memory region, subpage_accepts gets
confused because it finds a hole at 0x1cf an
From: Fam Zheng
The caller's workflow is like
if (!address_space_map()) {
...
cpu_register_map_client();
}
If bounce buffer became available after address_space_map() but before
cpu_register_map_client(), the caller could miss it and has to wait for the
next bounce buffe
This is better and prepares for the next patch. When we copy
libs_softmmu's value into LIBS with a := assignment, we cannot
anymore modify libs_softmmu in the Makefiles.
Signed-off-by: Paolo Bonzini
---
hw/display/Makefile.objs | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a
From: Fam Zheng
So that accesses from multiple threads are safe.
Signed-off-by: Fam Zheng
Message-Id: <1426496617-10702-4-git-send-email-f...@redhat.com>
[Remove #if from cpu_exec_init_all. - Paolo]
Signed-off-by: Paolo Bonzini
---
exec.c | 22 +-
1 file changed, 13 insert
On Tue, Apr 28, 2015 at 03:33:37PM +0200, Cornelia Huck wrote:
> On Tue, 28 Apr 2015 14:47:11 +0200
> "Michael S. Tsirkin" wrote:
>
> > On Tue, Apr 28, 2015 at 01:39:51PM +0200, Cornelia Huck wrote:
> > > On Tue, 28 Apr 2015 12:55:40 +0200
> > > "Michael S. Tsirkin" wrote:
> > >
> > > > On Tue,
From: Fam Zheng
If DMA's owning thread cancels the IO while the bounce buffer's owning thread
is notifying the "cpu client list", a use-after-free happens:
continue_after_map_failure dma_aio_cancel
--
ai
From: Wei Liu
I discovered a problem when trying to build QEMU statically with gcc.
libm is an element of LIBS while libpixman-1 is an element in
libs_softmmu. Libpixman references functions in libm, so the original
ordering makes linking fail.
This fix is to reorder $libs_softmmu and $LIBS to m
This is a simple MemoryRegion wrapper for qemu_ram_resize.
Signed-off-by: Paolo Bonzini
---
include/exec/memory.h | 12
memory.c | 7 +++
2 files changed, 19 insertions(+)
diff --git a/include/exec/memory.h b/include/exec/memory.h
index 06ffa1d..a2ea587 100644
---
Right now, ioport.c assumes that the entire range specified with
MemoryRegionPortio includes a region with size == 1. This however
is not true for the VBE DISPI ports, which are 16-bit only. The
next patch will make these regions' length equal to two, which can
cause the assertions to trigger. R
From: Mark Cave-Ayland
These were being incorrectly declared as MISC_SIZE (1 byte) rather than
4 bytes and 2 bytes respectively. As a result accesses clamped to the
real register size would unexpectedly fail.
Signed-off-by: Mark Cave-Ayland
CC: Paolo Bonzini
Message-Id: <1427987370-15897-1-git
From: Peter Crosthwaite
address_space_translate_internal will clamp the *plen length argument
based on the size of the memory region being queried. The iommu walker
logic in addresss_space_translate was ignoring this by discarding the
post fn call value of *plen. Fix by just always using *plen as
IDE PIO data must be written, for example, at 0x1f0. You cannot
do word or dword writes to 0x1f1..0x1f3 to access the data register.
Adjust the ide_portio_list accordingly.
Cc: John Snow
Signed-off-by: Paolo Bonzini
---
hw/ide/core.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
d
From: Fam Zheng
This adds "--enable-tcmalloc" and "--disable-tcmalloc" to allow linking
to libtcmalloc from gperftools.
tcmalloc is a malloc implementation that works well with threads and is
fast, so it is good for performance.
It is disabled by default, because the MALLOC_PERTURB_ flag we use
From: "Emilio G. Cota"
Here we have an open-coded byte-based bitmap implementation.
Get rid of it since there's a ulong-based implementation to be
used by all code.
Signed-off-by: Emilio G. Cota
Signed-off-by: Paolo Bonzini
---
translate-all.c | 42 +-
ram_addr_t is an internal interface, everyone should go through
MemoryRegion. Clean it up by making rom_add_blob return a
MemoryRegion* and using the new qemu_ram_resize infrastructure.
Reviewed-by: Michael S. Tsirkin
Signed-off-by: Paolo Bonzini
---
hw/core/loader.c | 8
hw/i386
From: Nadav Amit
Due to old Seabios bug, QEMU reenable LINT0 after reset. This bug is long gone
and therefore this hack is no longer needed. Since it violates the
specifications, it is removed.
Signed-off-by: Nadav Amit
Message-Id: <1428881529-29459-2-git-send-email-na...@cs.technion.ac.il>
Si
From: Bogdan Purcareata
This fixes ioctl behavior on powerpc e6500 platforms with 64bit kernel and 32bit
userspace. The current type cast has no effect there and the value passed to the
kernel is still 0. Probably an issue related to the compiler, since I'm assuming
the same configuration works o
From: Alberto Garcia
Signed-off-by: Alberto Garcia
Message-id: 1426522925-1-1-git-send-email-be...@igalia.com
Signed-off-by: Stefan Hajnoczi
Signed-off-by: Kevin Wolf
---
MAINTAINERS | 6 ++
1 file changed, 6 insertions(+)
diff --git a/MAINTAINERS b/MAINTAINERS
index 7aab80b..7c5e71e
From: Stefan Hajnoczi
The binary search in sdp_uuid_match() only works when the number of
elements to search is a power of two.
lo = record->uuid;
hi = record->uuids;
while (hi >>= 1)
if (lo[hi] <= val)
lo += hi;
return *lo == val;
I noticed that the record->uuids calcu
The following changes since commit 84cbd63f87c1d246f51ec8eee5367a5588f367fd:
Merge remote-tracking branch 'remotes/ehabkost/tags/x86-pull-request' into
staging (2015-04-28 12:22:20 +0100)
are available in the git repository at:
git://repo.or.cz/qemu/kevin.git tags/for-upstream
for you to
From: Stefan Hajnoczi
It is not clear from the code how a 0 parameter should be handled by the
hardware. Keep the same behavior as ffs(0) - 1 == -1.
Cc: Andrzej Zaborowski
Signed-off-by: Stefan Hajnoczi
Message-id: 1427124571-28598-3-git-send-email-stefa...@redhat.com
Signed-off-by: Kevin Wol
From: Paolo Bonzini
Rewrite the loop using level &= level - 1 to clear the least significant
bit after each iteration. This simplifies the loop and makes it easy to
replace ffs(3) with ctz32().
Cc: Peter Maydell
Signed-off-by: Paolo Bonzini
Signed-off-by: Stefan Hajnoczi
Message-id: 14271245
From: Stefan Hajnoczi
The lack of ffs(3) in the MinGW headers is a hint that we shouldn't rely
on it. MinGW 4.9.2 does not make it available for linking when QEMU's
./configure --enable-debug is used (release builds are fine though).
Now that all QEMU code has been switched to ctz32() there is
From: Paolo Bonzini
By using thread-local storage, aio_poll can stop using global data during
g_poll_ns. This will make it possible to drop callbacks from rfifolock.
[Moved npfd = 0 assignment to end of walking_handlers region as
suggested by Paolo. This resolves the assert(npfd == 0) assertio
101 - 200 of 373 matches
Mail list logo
