I enabled MSI interrupts, and now for 2 nights in a row I gamed 2 hours
straight and shut down the Windows VM without a freeze. Never in my 7
months of living with this bug have I gotten no freeze twice in a row. I
think the MSI interrupts have fixed it for me, and no, I did not remove
my HDMI soun
On 25/06/16 4:43 am, "Michael S. Tsirkin" wrote:
>On Fri, Jun 24, 2016 at 05:39:31PM +, Prerna Saxena wrote:
>>
>>
>> On 24/06/16 9:15 pm, "Felipe Franciosi" wrote:
>>
>> >We talked to MST on IRC a while back and he brainstormed the idea of doing
>> >this per-message.
>> >(I even re
Hi Alex,
We should never depend on the guest driver to behave in a certain way,
but we need to prioritize what that actually means. vfio in the kernel
has a responsibility first and foremost to the host kernel. User owned
devices cannot be allowed to exploit or interfere with the host
regardle
On Fri, Jun 24, 2016 at 05:39:31PM +, Prerna Saxena wrote:
>
>
> On 24/06/16 9:15 pm, "Felipe Franciosi" wrote:
>
> >We talked to MST on IRC a while back and he brainstormed the idea of doing
> >this per-message.
> >(I even recall proposing to call this feature REPLY_ALL and he suggested
On Fri, Jun 24, 2016 at 07:55:33AM +0200, Gerd Hoffmann wrote:
> On Do, 2016-06-23 at 19:38 +0300, Michael S. Tsirkin wrote:
> > On Thu, Jun 23, 2016 at 10:40:03AM +0200, Gerd Hoffmann wrote:
> > > Hi,
> > >
> > > > > Well the crash of guest phys bits > host phys bits, should be easy to
> > > >
On Thu, Jun 23, 2016 at 11:57:45AM +0200, Peter Lieven wrote:
> Am 21.06.2016 um 15:18 schrieb Dr. David Alan Gilbert:
> > * Peter Lieven (p...@kamp.de) wrote:
> > > Hi,
> > >
> > > while upgrading from Qemu 2.2.0 to Qemu 2.5.1.1 I noticed that the RSS
> > > memory usage has heavily increased.
>
On Fri, Jun 24, 2016 at 01:17:08AM -0700, Prerna Saxena wrote:
> From: Prerna Saxena
>
> Signed-off-by: Prerna Saxena
> ---
> docs/specs/vhost-user.txt | 36 +++
> hw/virtio/vhost-user.c| 153
> +-
> 2 files changed, 186 insertions(+), 3
On Fri, Jun 24, 2016 at 01:17:07AM -0700, Prerna Saxena wrote:
> From: Prerna Saxena
>
> The current vhost-user protocol requires the client to send responses to only
> few commands. For the remaining commands, it is impossible for QEMU to know
> the status of the requested operation -- ie, did
On Fri, Jun 24, 2016 at 03:25:30PM +0200, Marc-André Lureau wrote:
> On Thu, Jun 23, 2016 at 7:13 PM, Michael S. Tsirkin wrote:
> >> > If it's ok and we can recover, then why should we print errors?
> >>
> >> To me, the current disconnect handling is not handled cleanly. There
> >> is not much/not
On Fri, Jun 24, 2016 at 03:22:19PM +0200, Marc-André Lureau wrote:
> Hi
>
> On Thu, Jun 23, 2016 at 6:31 AM, Michael S. Tsirkin wrote:
> >
> > OK so if it's ok for read or write to fail, then I think
> > the callback should just return success.
> > This will make sure backends such as vhost net i
On Fri, Jun 24, 2016 at 02:08:52PM +0200, Marc-André Lureau wrote:
> On Thu, Jun 23, 2016 at 7:01 PM, Michael S. Tsirkin wrote:
> >> > Maybe what you want is a need_unlink feature.
> >> > Set it for unix sockets only, that would make some sense.
> >>
> >> Oh perhaps what you mean is that if the fd
On Fri, Jun 24, 2016 at 08:00:20AM +0200, Igor Mammedov wrote:
> On Fri, 24 Jun 2016 08:53:25 +0300
> "Michael S. Tsirkin" wrote:
>
> > On Thu, Jun 23, 2016 at 03:47:36PM +0200, Igor Mammedov wrote:
> > > On Thu, 23 Jun 2016 16:08:38 +0300
> > > Marcel Apfelbaum wrote:
> > >
> > > > On 06/16/20
On 06/23/2016 08:36 AM, Kevin Wolf wrote:
> As cache.writeback is a BlockBackend property and as such more related
> to the guest device than the BlockDriverState, we already removed it
> from the blockdev-add interface. This patch adds the new way to set it,
> as a qdev property of the correspondi
On 06/23/2016 08:36 AM, Kevin Wolf wrote:
> In order to remove the necessity to use BlockBackend names in the
> external API, we want to allow node-names everywhere. This is an example
> conversion that accepts a node-name without lifting the restriction that
> we're operating at a root node. Chang
On 06/23/2016 08:36 AM, Kevin Wolf wrote:
> This is an example conversion of a QMP command that operates on the
> BlockBackend level to accept both the device model name (which is
> supposed to become the primary interface) and the BlockBackend name.
>
> Naming suggestions for the new QMP field ar
Software should see timer counter wrap around only after IRQ being triggered.
Change returned counter value to "1" for the expired timer and avoid returning
wrapped around counter value in periodic mode for the timer that has bottom-half
handler setup, assuming it is IRQ handler.
This fixes regres
On 06/23/2016 08:36 AM, Kevin Wolf wrote:
> This finds the BlockBackend attached to the device model identified by
> its qdev ID.
>
> Signed-off-by: Kevin Wolf
> ---
> include/sysemu/block-backend.h | 1 +
> qdev-monitor.c | 18 ++
> 2 files changed, 19 insertion
Call gen_pause for all "or rx,rx,rx" encodings other nop. This
provides a reasonable implementation for yield, and a better
approximation for mdoio, mdoom, and miso. The choice to pause for all
encodings !=0 leverages the PowerISA admonition that the reserved
encodings might change program prior
On 24/06/16 9:15 pm, "Felipe Franciosi" wrote:
>We talked to MST on IRC a while back and he brainstormed the idea of doing
>this per-message.
>(I even recall proposing to call this feature REPLY_ALL and he suggested
>REPLY_ANY due to that.)
>
>I agree with doing it per message, as the protoco
On 24.06.2016 21:37, Mark Cave-Ayland wrote:
> On 24/06/16 19:19, Dmitry Osipenko wrote:
>
>> On 24.06.2016 19:02, Peter Maydell wrote:
>>> On 24 June 2016 at 16:58, Mark Cave-Ayland
>>> wrote:
On 06/06/16 15:47, Peter Maydell wrote:
> From: Dmitry Osipenko
>
> ptimer_get_c
On Sat, 25 Jun 2016 00:04:27 +0530
Kirti Wankhede wrote:
> Thanks Alex.
>
>
> On 6/22/2016 4:18 AM, Alex Williamson wrote:
> > On Mon, 20 Jun 2016 22:01:47 +0530
> > Kirti Wankhede wrote:
> >
> >> +
> >> +static int get_mdev_region_info(struct mdev_device *mdev,
> >> +
On Fri, 24 Jun 2016 23:24:58 +0530
Kirti Wankhede wrote:
> Alex,
>
> Thanks for taking closer look. I'll incorporate all the nits you suggested.
>
> On 6/22/2016 3:00 AM, Alex Williamson wrote:
> > On Mon, 20 Jun 2016 22:01:46 +0530
> > Kirti Wankhede wrote:
> >
> ...
> >> +
> >> +config MDE
Eliminate redundant and incorrect booke206_page_size_to_tlb function
from ppce500_spin.c in preference to previously existing but newly
exported definition from e500.c
Signed-off-by: Aaron Larson
---
hw/ppc/e500.c | 2 +-
hw/ppc/e500.h | 2 ++
hw/ppc/ppce500_spin.c | 7 +--
Thanks Alex.
On 6/22/2016 4:18 AM, Alex Williamson wrote:
> On Mon, 20 Jun 2016 22:01:47 +0530
> Kirti Wankhede wrote:
>
>> +
>> +static int get_mdev_region_info(struct mdev_device *mdev,
>> +struct pci_region_info *vfio_region_info,
>> +i
On 24.06.2016 19:02, Peter Maydell wrote:
> On 24 June 2016 at 16:58, Mark Cave-Ayland
> wrote:
>> On 06/06/16 15:47, Peter Maydell wrote:
>>
>>> From: Dmitry Osipenko
>>>
>>> ptimer_get_count() might be called while QEMU timer already been expired.
>>> In that case ptimer would return counter =
On 24/06/16 19:19, Dmitry Osipenko wrote:
On 24.06.2016 19:02, Peter Maydell wrote:
On 24 June 2016 at 16:58, Mark Cave-Ayland
wrote:
On 06/06/16 15:47, Peter Maydell wrote:
From: Dmitry Osipenko
ptimer_get_count() might be called while QEMU timer already been expired.
In that case ptimer
Artyom has located the regression and posted a patch here:
https://lists.gnu.org/archive/html/qemu-devel/2016-06/msg07226.html.
--
You received this bug notification because you are a member of qemu-
devel-ml, which is subscribed to QEMU.
https://bugs.launchpad.net/bugs/1588328
Title:
Qemu 2.6
Alex,
Thanks for taking closer look. I'll incorporate all the nits you suggested.
On 6/22/2016 3:00 AM, Alex Williamson wrote:
> On Mon, 20 Jun 2016 22:01:46 +0530
> Kirti Wankhede wrote:
>
...
>> +
>> +config MDEV
>> +tristate "Mediated device driver framework"
>> +depends on VFIO
>> +
Am 24.06.2016 um 19:35 hat Eric Blake geschrieben:
> On 06/23/2016 08:36 AM, Kevin Wolf wrote:
> > If a node name instead of a BlockBackend name is specified as the driver
> > for a guest device, an anonymous BlockBackend is created now.
> >
> > Signed-off-by: Kevin Wolf
> > ---
> > hw/core/qdev
On 06/23/2016 08:36 AM, Kevin Wolf wrote:
> Signed-off-by: Kevin Wolf
> ---
> qdev-monitor.c | 16
> 1 file changed, 12 insertions(+), 4 deletions(-)
Reviewed-by: Eric Blake
--
Eric Blake eblake redhat com+1-919-301-3266
Libvirt virtualization library http://libvirt.org
Hi Peter,
This request just contains Artyom's fix for the Solaris 9 regression introduced
by the previous register window changes. Please pull.
ATB,
Mark.
The following changes since commit a01aef5d2f96c334d048f43f0d3573a1152b37ca:
Merge remote-tracking branch 'remotes/mst/tags/for_upstrea
On 06/23/2016 08:36 AM, Kevin Wolf wrote:
> This finds a BlockBackend given the device model that is attached to it.
>
> Signed-off-by: Kevin Wolf
> ---
> block/block-backend.c | 19 +++
> include/sysemu/block-backend.h | 1 +
> 2 files changed, 20 insertions(+)
Revie
On 06/23/2016 08:36 AM, Kevin Wolf wrote:
> If a node name instead of a BlockBackend name is specified as the driver
> for a guest device, an anonymous BlockBackend is created now.
>
> Signed-off-by: Kevin Wolf
> ---
> hw/core/qdev-properties-system.c | 18 +-
> 1 file changed, 1
On 24 June 2016 at 18:22, Andrew Jones wrote:
> On Fri, Jun 24, 2016 at 05:41:55PM +0100, Peter Maydell wrote:
>> KVM AArch32 is 4 CPUs per cluster:
>> http://lxr.free-electrons.com/source/arch/arm/kvm/coproc.c#L109
>
> Hmm... yes, it should use coproc.c, but here's what I get when I
> test
>
> qe
On Fri, Jun 24, 2016 at 05:41:55PM +0100, Peter Maydell wrote:
> On 24 June 2016 at 17:15, Andrew Jones wrote:
> > On Fri, Jun 24, 2016 at 06:03:21PM +0200, Andrew Jones wrote:
> >> So we can either
> >> a) play it safe and always use clusters of 4 for ARM guests, and
> >>KVM will get "fixed"
From: Artyom Tarasenko
Signed-off-by: Artyom Tarasenko
Reviewed-by: Richard Henderson
Signed-off-by: Mark Cave-Ayland
---
target-sparc/translate.c |5 -
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/target-sparc/translate.c b/target-sparc/translate.c
index afd46b8..0f4f
On 06/15/2016 10:14 AM, Laurent Vivier wrote:
> Script from LKML.
Link to a particular message?
>
> Signed-off-by: Laurent Vivier
> ---
> scripts/coccinelle/typecast.cocci | 7 +++
> 1 file changed, 7 insertions(+)
> create mode 100644 scripts/coccinelle/typecast.cocci
>
> diff --git a/s
On 03/06/16 23:40, Alex Bennée wrote:
> In preparation for multi-threaded TCG we remove tcg_exec_all and move
> all the CPU cycling into the main thread function. When MTTCG is enabled
> we shall use a separate thread function which only handles one vCPU.
>
> Signed-off-by: Alex Bennée
Reviewed-b
On 24/06/16 17:01, Artyom Tarasenko wrote:
On Fri, Jun 24, 2016 at 5:51 PM, Mark Cave-Ayland
wrote:
On 24/06/16 13:34, Artyom Tarasenko wrote:
Signed-off-by: Artyom Tarasenko
---
target-sparc/translate.c | 5 -
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/target-sparc/t
On 24 June 2016 at 17:09, Laurent Desnogues wrote:
> On Fri, Jun 24, 2016 at 5:49 PM, Peter Maydell
> wrote:
>> SYS_HEAPINFO is one of the few semihosting calls which has to write
>> values back into a parameter block in memory. When we added
>> support for 64-bit semihosting we updated the cod
On 06/24/2016 10:05 AM, Igor Mammedov wrote:
> custom apic-id setter/getter doesn't do any property specific
> checks anymorer, so clean it up and use more compact static
s/anymorer/anymore/
> property DEFINE_PROP_UINT32 instead.
>
> Signed-off-by: Igor Mammedov
> ---
> target-i386/cpu.c | 45
Signed-off-by: Igor Mammedov
---
target-i386/cpu.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/target-i386/cpu.c b/target-i386/cpu.c
index 04c0b79..2fa445d 100644
--- a/target-i386/cpu.c
+++ b/target-i386/cpu.c
@@ -2765,6 +2765,7 @@ static void x86_cpu_apic_create(X86CPU *cpu, Error **err
ACPI subsystem already has all logic in place the only
thing left to eject CPU is destroy it and ammend
present CPUs counter in CMOS, do so.
Signed-off-by: Igor Mammedov
---
hw/i386/pc.c | 11 +--
1 file changed, 5 insertions(+), 6 deletions(-)
diff --git a/hw/i386/pc.c b/hw/i386/pc.c
i
object_property_add_child() silently fails with error that it can't
create duplicate propery 'apic' as we already have 'apic' property
registered for AIPC ID. As result generic device_realize puts
apic as into unattached container.
As it's programming error, abort on it and fix property name for
a
first remove VCPU from exec loop and only then remove lapic.
Signed-off-by: Chen Fan
Signed-off-by: Gu Zheng
Signed-off-by: Zhu Guihua
Signed-off-by: Igor Mammedov
---
target-i386/cpu.c | 15 +++
1 file changed, 15 insertions(+)
diff --git a/target-i386/cpu.c b/target-i386/cpu.c
On 24 June 2016 at 17:15, Andrew Jones wrote:
> On Fri, Jun 24, 2016 at 06:03:21PM +0200, Andrew Jones wrote:
>> So we can either
>> a) play it safe and always use clusters of 4 for ARM guests, and
>>KVM will get "fixed" when we start managing the guest's MPIDR
>>from userspace, or
>> b) u
callbacks will do necessary cleanups before APIC device is deleted
Signed-off-by: Chen Fan
Signed-off-by: Gu Zheng
Signed-off-by: Zhu Guihua
Signed-off-by: Igor Mammedov
---
hw/i386/kvm/apic.c | 5 +
hw/intc/apic.c | 10 ++
hw/intc/apic_common.c
these properties will be used by as address where to plug
CPU with help -device/device_add commands.
Signed-off-by: Igor Mammedov
---
v2:
- rename socket/core/thread properties to socket-id/core-id/thread-id
- add mismatch checks for apic_id and socket-id/core-id/thread-id
in case both ar
local_apics[] is sized to contain all APIC ID supported in xAPIC mode,
so use APIC ID as index in it instead of constantly increasing counter idx.
Fixes error "apic initialization failed" when a CPU hotplugged and
unplugged more times than there are free slots in local_apics[].
Signed-off-by: Igo
consolidate possible_cpus array management in pc_cpu_plug()
for smp_cpus, coldplugged with -device and hotplugged with
device_add.
Signed-off-by: Igor Mammedov
---
hw/i386/pc.c | 25 +
1 file changed, 9 insertions(+), 16 deletions(-)
diff --git a/hw/i386/pc.c b/hw/i386/p
Signed-off-by: Igor Mammedov
---
target-i386/cpu.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/target-i386/cpu.c b/target-i386/cpu.c
index 1ec40a0..ebf4140 100644
--- a/target-i386/cpu.c
+++ b/target-i386/cpu.c
@@ -3239,6 +3239,7 @@ static void x86_cpu_common_class_init(ObjectClass *oc,
Machine code knows about all possible APIC IDs so use that
instead of hack which does O(n^2) complexity duplicate
checks, interating over global CPUs list.
As result duplicate check is done only once with O(log n) complexity.
Signed-off-by: Igor Mammedov
---
hw/i386/pc.c | 44 ++
currently present CPUs counter in CMOS only contains
smp_cpus (i.e. initial CPUs specified with -smp X) and
doesn't account for CPUs created with -device.
If VM is started with additional CPUs added with
-device, it will hang in BIOS waiting for condition
smp_cpus == counted_cpus
forever as cou
custom apic-id setter/getter doesn't do any property specific
checks anymorer, so clean it up and use more compact static
property DEFINE_PROP_UINT32 instead.
Signed-off-by: Igor Mammedov
---
target-i386/cpu.c | 45 ++---
1 file changed, 6 insertions(+), 3
On 24/06/16 16:58, Richard Henderson wrote:
On 06/24/2016 05:34 AM, Artyom Tarasenko wrote:
Signed-off-by: Artyom Tarasenko
---
target-sparc/translate.c | 5 -
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/target-sparc/translate.c b/target-sparc/translate.c
index 5111cf0..0
On Fri, Jun 24, 2016 at 5:49 PM, Peter Maydell wrote:
> SYS_HEAPINFO is one of the few semihosting calls which has to write
> values back into a parameter block in memory. When we added
> support for 64-bit semihosting we updated the code which reads from
> the parameter block to read 64-bit word
it's reverse of apicid_from_topo_ids() and will be used in follow up
patches to fill in data structures for query-hotpluggable-cpus and
for user friendly error reporting
Signed-off-by: Igor Mammedov
---
include/hw/i386/topology.h | 15 +++
1 file changed, 15 insertions(+)
diff --git
On Fri, Jun 24, 2016 at 5:51 PM, Mark Cave-Ayland
wrote:
> On 24/06/16 13:34, Artyom Tarasenko wrote:
>
>> Signed-off-by: Artyom Tarasenko
>> ---
>> target-sparc/translate.c | 5 -
>> 1 file changed, 4 insertions(+), 1 deletion(-)
>>
>> diff --git a/target-sparc/translate.c b/target-sparc/tr
Public bug reported:
it happened to me during a build of yocto/bitbake related cross tools.
the auto-configuration part titled "SDL probe" for qemu-2.2.0 i found
the configuration step failing for the compile_prog routine. actually
those test compile went fine but only the test linking failed.
th
it returns a list of present/possible to hotplug CPU
objects with a list of properties to use with
device_add.
in PC case returned list would looks like:
-> { "execute": "query-hotpluggable-cpus" }
<- {"return": [
{
"type": "qemu64-x86_64-cpu", "vcpus-count": 1,
"props": {"cor
Pranith Kumar writes:
> Signed-off-by: Pranith Kumar
> ---
> target-arm/translate-a64.c | 18 +-
> 1 file changed, 13 insertions(+), 5 deletions(-)
>
> diff --git a/target-arm/translate-a64.c b/target-arm/translate-a64.c
> index ce8141a..fa24bf2 100644
> --- a/target-arm/transl
MAX_APICS is only used by child 'apic' class and not
by its parent TYPE_APIC_COMMON or any other derived
class.
Move check into end user 'apic' class so it won't
get in the way of other APIC implementations
if they support more then MAX_APICS.
Signed-off-by: Igor Mammedov
---
hw/intc/apic.c
On Thu, Jun 23, 2016 at 12:15:59PM +0100, Peter Maydell wrote:
> On 21 June 2016 at 19:58, Andrew Jones wrote:
> > Signed-off-by: Andrew Jones
>
> I think this commit message could be improved...it's both
> very short and a bit off the mark.
>
> > ---
> > hw/arm/virt.c | 10 ++
> > 1 f
We talked to MST on IRC a while back and he brainstormed the idea of doing this
per-message.
(I even recall proposing to call this feature REPLY_ALL and he suggested
REPLY_ANY due to that.)
I agree with doing it per message, as the protocol itself should be flexible in
that sense.
(Even if qemu
On Fri, Jun 24, 2016 at 06:03:21PM +0200, Andrew Jones wrote:
> On Thu, Jun 23, 2016 at 12:15:59PM +0100, Peter Maydell wrote:
> > On 21 June 2016 at 19:58, Andrew Jones wrote:
> > > Signed-off-by: Andrew Jones
> >
> > I think this commit message could be improved...it's both
> > very short and
redo
9886e834 target-i386: Require APIC ID to be explicitly set before CPU realize
in another way that doesn't use int64_t to detect
if apic-id property was set.
Use the fact that 0x is the broadcast
value that a CPU can't have and set default
uint32_t apic_id to it instead of using int64
On 24 June 2016 at 16:58, Mark Cave-Ayland
wrote:
> On 06/06/16 15:47, Peter Maydell wrote:
>
>> From: Dmitry Osipenko
>>
>> ptimer_get_count() might be called while QEMU timer already been expired.
>> In that case ptimer would return counter = 0, which might be undesirable
>> in case of polled t
On 06/24/2016 05:34 AM, Artyom Tarasenko wrote:
Signed-off-by: Artyom Tarasenko
---
target-sparc/translate.c | 5 -
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/target-sparc/translate.c b/target-sparc/translate.c
index 5111cf0..065326c 100644
--- a/target-sparc/translate.c
+
2016-06-24 1:26 GMT+08:00 Peter Maydell :
> On 17 June 2016 at 03:37, Tsung-Han Lin wrote:
> > Hi, I made some changes to TRY TO fix the ARM semihosting issue in
> > SYS_HEAPINFO handling.
> > This problem has been bothering me for quite a while.
> >
> > A new global variable 'main_ram_base' is a
Changelog:
since v1:
* s/pc_find_cpu/pc_find_cpu_slot/ + add comment to it
* add more sanity checks for socket-id/core-id/thread-id and 'apic'
properties
* include device_del cpu patches and related fixes to x86 CPU/apic
Series enabling usage of -device/device_add for adding CP
CPU added with device_add help won't have APIC ID set,
so set it according to socket/core/thread ids provided
with device_add command.
Signed-off-by: Igor Mammedov
---
v2:
- add validity checks for socket-id/core-id/thread-id values
---
hw/i386/pc.c | 44
On 06/06/16 15:47, Peter Maydell wrote:
From: Dmitry Osipenko
ptimer_get_count() might be called while QEMU timer already been expired.
In that case ptimer would return counter = 0, which might be undesirable
in case of polled timer. Do counter wrap around for periodic timer to keep
it distrib
On Fri, Jun 24, 2016 at 5:49 PM, Peter Maydell wrote:
> The fields in the TaskState heap_base, heap_limit and stack_base
> are all guest addresses (representing the locations of the heap
> and stack for the guest binary), so they should be abi_ulong
> rather than uint32_t. (This only in practice a
it will be reused in the next patch at pre_plug time
Signed-off-by: Igor Mammedov
---
v2:
- rename pc_find_cpu() into pc_find_cpu_slot() and add comment to it
Eduardo Habkost
---
hw/i386/pc.c | 29 ++---
1 file changed, 22 insertions(+), 7 deletions(-)
diff --git
On 24.06.2016 18:31, Eric Blake wrote:
On 06/24/2016 09:06 AM, Denis V. Lunev wrote:
From: Evgeny Yakovlev
Some guests (win2008 server for example) do a lot of unnecessary
flushing when underlying media has not changed. This adds additional
overhead on host when calling fsync/fdatasync.
This
SYS_HEAPINFO is one of the few semihosting calls which has to write
values back into a parameter block in memory. When we added
support for 64-bit semihosting we updated the code which reads from
the parameter block to read 64-bit words but forgot to change the
code that writes back into the block
From: Peter Crosthwaite
QOMify registers as a child of TYPE_DEVICE. This allows registers to
define GPIOs.
Define an init helper that will do QOM initialisation.
Signed-off-by: Peter Crosthwaite
Signed-off-by: Alistair Francis
Reviewed-by: KONRAD Frederic
Reviewed-by: Peter Maydell
---
V7:
On 24/06/16 13:34, Artyom Tarasenko wrote:
Signed-off-by: Artyom Tarasenko
---
target-sparc/translate.c | 5 -
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/target-sparc/translate.c b/target-sparc/translate.c
index 5111cf0..065326c 100644
--- a/target-sparc/translate.c
+++ b
On 06/24/2016 05:34 AM, Artyom Tarasenko wrote:
Signed-off-by: Artyom Tarasenko
---
target-sparc/translate.c | 5 -
1 file changed, 4 insertions(+), 1 deletion(-)
Reviewed-by: Richard Henderson
r~
These patches fix bugs in the ARM semihosting SYS_HEAPINFO
syscall for 64-bit guests:
* the fields in linux-user's TaskState should be abi_ulong,
not uint32_t, since they're guest addresses
* the SYS_HEAPINFO implementation needs to write its return
data struct using fields of the right wid
From: Peter Crosthwaite
Define some macros that can be used for defining registers and fields.
The REG32 macro will define A_FOO, for the byte address of a register
as well as R_FOO for the uint32_t[] register number (A_FOO / 4).
The FIELD macro will define FOO_BAR_MASK, FOO_BAR_SHIFT and
FOO_B
On 24.06.2016 18:39, Eric Blake wrote:
On 06/23/2016 03:15 AM, Pavel Butsykin wrote:
@@ -3309,6 +3315,7 @@ void qmp_drive_backup(const char *device, const
char *target,
bool has_mode, enum NewImageMode mode,
bool has_speed, int64_t speed,
On 23 June 2016 at 18:26, Peter Maydell wrote:
> I suggest you start by sending a patch which just fixes
> the 64-bit case to write 64-bit values, since that's the
> easy bit.
I ran into this bug myself this afternoon, so I wrote
some patches which fix this part and will send them out
in a minute
This API provides some encapsulation of registers and factors out some
common functionality to common code. Bits of device state (usually MMIO
registers) often have all sorts of access restrictions and semantics
associated with them. This API allows you to define what those
restrictions are on a bi
On 03/06/16 23:40, Alex Bennée wrote:
> This adds calls to the assert_memory_lock for all public APIs which are
> documented as holding the mmap_lock for user-mode.
>
> Signed-off-by: Alex Bennée
> ---
> linux-user/elfload.c | 4
> translate-all.c | 20
> 2 files c
The fields in the TaskState heap_base, heap_limit and stack_base
are all guest addresses (representing the locations of the heap
and stack for the guest binary), so they should be abi_ulong
rather than uint32_t. (This only in practice affects ARM AArch64
since all the other semihosting implementati
From: Peter Crosthwaite
Signed-off-by: Peter Crosthwaite
Signed-off-by: Alistair Francis
Reviewed-by: Peter Maydell
---
V7:
- Remove unused property add
V4:
- Small corrections to the device model logic
hw/arm/xilinx_zynq.c | 6 ++
1 file changed, 6 insertions(+)
diff --git a/hw/arm/x
Add memory io handlers that glue the register API to the memory API.
Just translation functions at this stage. Although it does allow for
devices to be created without all-in-one mmio r/w handlers.
This patch also adds the RegisterInfoArray struct, which allows all of
the individual RegisterInfo s
This patch series is based on Peter C's original register API. His
original cover letter is below.
Future work: Allow support for memory attributes.
V8:
- Remove GPIO infrastructure from patch set
V7:
- Rebase
- Fixup the DecCFG model
- Fixes and corrections to the register API model and func
Add a minimal model for the devcfg device which is part of Zynq.
This model supports DMA capabilities and interrupt generation.
Signed-off-by: Peter Crosthwaite
Signed-off-by: Alistair Francis
---
V7:
- Corrections to the internal device
V5:
- Corrections to the device model logic
default-co
From: Peter Crosthwaite
Add a helper that will scan a static RegisterAccessInfo Array
and populate a container MemoryRegion with registers as defined.
Signed-off-by: Peter Crosthwaite
Signed-off-by: Alistair Francis
---
The reason that I'm not using GArray is because the array needs to store
t
On 05/31/2016 03:15 AM, Denis V. Lunev wrote:
> From: Pavel Butsykin
>
> For bdrv_pwrite_compressed() it looks like most of the code creating coroutine
> is duplicated in blk_prw(). So we can just add a
> flag(BDRV_REQ_WRITE_COMPRESSED)
> and use the blk_prw() as a generic one.
>
> Signed-off-b
Add a macro that creates a 64bit value which has length number of ones
shifted across by the value of shift.
Signed-off-by: Alistair Francis
Reviewed-by: Alex Bennée
Reviewed-by: Peter Maydell
---
V7:
- Fixup commit typo
- Use the method from deposit64()
V5:
- Re-write to a 64-bit mask inste
On 06/23/2016 03:15 AM, Pavel Butsykin wrote:
>>> @@ -3309,6 +3315,7 @@ void qmp_drive_backup(const char *device, const
>>> char *target,
>>> bool has_mode, enum NewImageMode mode,
>>> bool has_speed, int64_t speed,
>>> bool h
On 03/06/16 23:40, Alex Bennée wrote:
> diff --git a/translate-all.c b/translate-all.c
> index e3f44d9..8b162ff 100644
> --- a/translate-all.c
> +++ b/translate-all.c
(snip)
> @@ -347,8 +349,10 @@ bool cpu_restore_state(CPUState *cpu, uintptr_t retaddr)
> cpu_restore_state_from_tb(cpu, tb,
On 06/21/2016 03:21 AM, Kevin Wolf wrote:
> No code changes, just moved from one file to another.
>
> Signed-off-by: Kevin Wolf
> ---
> block.c | 110 ---
> block/Makefile.objs | 3 +-
> block/commit.c | 111
> ++
v2:
* Rebased onto latest v4 Stefan's series:
[PATCH v4 0/7] virtio-blk: multiqueue support
Hello, all.
This is RFC because mostly this patch is a quick attempt to get true
multithreaded multiqueue support for a block device with native AIO.
The goal is to squeeze everything possible on loc
On 06/24/2016 09:06 AM, Denis V. Lunev wrote:
> From: Evgeny Yakovlev
>
> Some guests (win2008 server for example) do a lot of unnecessary
> flushing when underlying media has not changed. This adds additional
> overhead on host when calling fsync/fdatasync.
>
> This change introduces a dirty fl
On 03/06/16 23:40, Alex Bennée wrote:
> diff --git a/exec.c b/exec.c
> index a3a93ae..b225282 100644
> --- a/exec.c
> +++ b/exec.c
> @@ -25,6 +25,7 @@
> #include "qemu/cutils.h"
> #include "cpu.h"
> #include "exec/exec-all.h"
> +#include "qom/cpu.h"
> #include "tcg.h"
> #include "hw/qdev-core.
On 06/24/2016 09:06 AM, Denis V. Lunev wrote:
> Changes from v2:
> - Better comments
> - Rebased on latest master
Remember, 'git send-email -v3' will automatically put v3 in all your
subject lines, making it easier to see that this is an updated series.
Rebasing on Kevin's block branch may be mor
1 - 100 of 313 matches
Mail list logo
