On 12/08/2015 22:37, Emilio G. Cota wrote:
> > page_find is reading the radix tree outside all locks, so it has to
> > use the RCU primitives. It does not need RCU critical sections
> > because the PageDescs are never removed, so there is never a need
> > to wait for the end of code sections tha
On 08/13/2015 08:32 AM, Ouyang, Changchun wrote:
-Original Message-
Date: Wed, 12 Aug 2015 14:15:54 +0300
From: "Michael S. Tsirkin"
To: Marcel Apfelbaum
Cc: qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] [PATCH] virtio/vhost: drop unnecessary
VHOST_SET_VRING call
Message-ID
This fixes a regression introduced by commit dcfb3beb ("mirror: Do zero
write on target if sectors not allocated"), which was reported to cause
aborts with the message "Co-routine re-entered recursively".
The cause for this bug is the following code in mirror_iteration_done():
if (s->common.b
On Wed, Aug 12, 2015 at 02:25:41PM +0800, Ouyang Changchun wrote:
> Based on patch by Nikolay Nikolaev:
> Vhost-user will implement the multi queue support in a similar way
> to what vhost already has - a separate thread for each queue.
> To enable the multi queue functionality - a new command line
On Wed, Aug 12, 2015 at 02:25:42PM +0800, Ouyang Changchun wrote:
> This patch is based on top of "vhost-user: protocol updates" series
> proposed earlier by Michael S. Tsirkin.
>
> Use new message VHOST_USER_SET_VRING_FLAG to enable and disable an
> actual virt queue, which is similar to attach/d
On 13/08/2015 10:41, Kevin Wolf wrote:
> This fixes a regression introduced by commit dcfb3beb ("mirror: Do zero
> write on target if sectors not allocated"), which was reported to cause
> aborts with the message "Co-routine re-entered recursively".
>
> The cause for this bug is the following co
On 12 August 2015 at 14:36, Paolo Bonzini wrote:
> The following changes since commit cb48f67ad8c7b33c617d4f8144a27706e69fd688:
>
> bsd-user: Fix operand to cpu_x86_exec (2015-07-30 12:38:49 +0100)
>
> are available in the git repository at:
>
> git://github.com/bonzini/qemu.git tags/for-upstr
On 13/08/2015 11:28, Peter Maydell wrote:
> config-host.mak is out-of-date, running configure
> ../../configure: 2789: local: -I/usr/include/glib-2.0: bad variable name
>
> line 2789 is
> local probe_cflags=$($pkg_config --cflags $1)
>
> 'local' isn't part of POSIX shell. It is supported by 'da
On 13 August 2015 at 10:37, Paolo Bonzini wrote:
>
>
> On 13/08/2015 11:28, Peter Maydell wrote:
>> config-host.mak is out-of-date, running configure
>> ../../configure: 2789: local: -I/usr/include/glib-2.0: bad variable name
>>
>> line 2789 is
>> local probe_cflags=$($pkg_config --cflags $1)
>>
>
On Thu, Aug 13, 2015 at 11:18 AM, Michael S. Tsirkin wrote:
> On Wed, Aug 12, 2015 at 02:25:41PM +0800, Ouyang Changchun wrote:
>> Based on patch by Nikolay Nikolaev:
>> Vhost-user will implement the multi queue support in a similar way
>> to what vhost already has - a separate thread for each que
On CPUs with EL3, there are two physical timers, one for Secure and one
for Non-secure. Implement this extra timer and the AArch64 registers
which access it.
Signed-off-by: Peter Maydell
Reviewed-by: Edgar E. Iglesias
Message-id: 1437047249-2357-2-git-send-email-peter.mayd...@linaro.org
---
tar
From: Jean-Christophe Dubois
Signed-off-by: Jean-Christophe Dubois
Reviewed-by: Peter Crosthwaite
Message-id:
cc7d1589e774e87c346b75a6c25e07957f436ced.1437080501.git@tribudubois.net
Signed-off-by: Peter Maydell
---
hw/timer/imx_gpt.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletio
From: Jean-Christophe Dubois
Signed-off-by: Jean-Christophe Dubois
Reviewed-by: Peter Crosthwaite
Message-id:
d8d70683c6a48ac318c1635595619cfb0eb31681.1437080501.git@tribudubois.net
Signed-off-by: Peter Maydell
---
hw/timer/imx_epit.c | 12 ++--
1 file changed, 6 insertions(+), 6
It's easy to accidentally define two cpregs which both try
to reset the same underlying state field (for instance a
clash between an AArch64 EL3 definition and an AArch32
banked register definition). if the two definitions disagree
about the reset value then the result is dependent on which
one hap
From: Jean-Christophe Dubois
Signed-off-by: Jean-Christophe Dubois
Reviewed-by: Peter Crosthwaite
Message-id:
06829257e845d693be05c7d491134313c1615d1a.1437080501.git@tribudubois.net
Signed-off-by: Peter Maydell
---
hw/arm/kzm.c | 3 ++-
hw/intc/imx_avic.c | 40 +++-
From: "Edgar E. Iglesias"
Adds support for the virtual timer offset controlled by EL2.
Reviewed-by: Peter Maydell
Signed-off-by: Edgar E. Iglesias
Message-id: 1436791864-4582-2-git-send-email-edgar.igles...@gmail.com
Signed-off-by: Peter Maydell
---
target-arm/cpu.h| 1 +
target-arm/hel
If EL3 is AArch32, then the secure physical timer is accessed via
banking of the registers used for the non-secure physical timer.
Implement this banking.
Note that the access controls for the AArch32 banked registers
remain the same as the physical-timer checks; they are not the
same as the contr
From: Pavel Fedin
Just specifying ops = NULL in some cases can be more convenient than having
two functions.
Signed-off-by: Pavel Fedin
Acked-by: Paolo Bonzini
Reviewed-by: Peter Maydell
Message-id:
78a379ab1b6b30ab497db7971ad336dad1dbee76.1438758065.git.p.fe...@samsung.com
Signed-off-by: Pe
From: Jean-Christophe Dubois
Signed-off-by: Jean-Christophe Dubois
Reviewed-by: Peter Crosthwaite
Message-id:
a51ef50fa222a614169056d5389a6d3ed6a63b04.1437080501.git@tribudubois.net
Signed-off-by: Peter Maydell
---
hw/char/imx_serial.c | 82 +-
in
From: "Edgar E. Iglesias"
Reviewed-by: Peter Maydell
Signed-off-by: Edgar E. Iglesias
Message-id: 1436791864-4582-8-git-send-email-edgar.igles...@gmail.com
Signed-off-by: Peter Maydell
---
hw/arm/virt.c | 4
1 file changed, 4 insertions(+)
diff --git a/hw/arm/virt.c b/hw/arm/virt.c
inde
From: Jean-Christophe Dubois
Signed-off-by: Jean-Christophe Dubois
Reviewed-by: Peter Crosthwaite
Message-id:
23ab872b7cd30b1399384fb26a2ebb75e9761d7b.1437080501.git@tribudubois.net
Signed-off-by: Peter Maydell
---
hw/char/imx_serial.c | 34 +-
1 file chan
From: Jean-Christophe Dubois
Signed-off-by: Jean-Christophe Dubois
Reviewed-by: Peter Crosthwaite
Message-id:
01e1d9026220992405819f25640ebd5bb843fc93.1437080501.git@tribudubois.net
Signed-off-by: Peter Maydell
---
hw/intc/imx_avic.c | 16 +++-
1 file changed, 7 insertions(+)
From: Jean-Christophe Dubois
Move constructor to DeviceClass methods
* imx_serial_init
* imx_serial_realize
imx32_serial_properties is renamed to imx_serial_properties.
Signed-off-by: Jean-Christophe Dubois
Reviewed-by: Peter Crosthwaite
Message-id:
6854bd75e2b5af312e04e760587e249dbaff807f
From: Jean-Christophe Dubois
Signed-off-by: Jean-Christophe Dubois
Reviewed-by: Peter Crosthwaite
Message-id:
948927cab0c85da9a753c5f6d5501323d5604c8e.1437080501.git@tribudubois.net
Signed-off-by: Peter Maydell
---
hw/timer/imx_epit.c | 52 ++---
include/h
From: Pavel Fedin
Extracted duplicated initialization code from SW-emulated and KVM GIC
implementations and put into gic_init_irqs_and_mmio()
Signed-off-by: Pavel Fedin
Message-id:
8ea5b2781ef39cb5989420987fc73c70e377687d.1438758065.git.p.fe...@samsung.com
Reviewed-by: Peter Maydell
Signed-of
From: "Dr. David Alan Gilbert"
This series is a set of small cleanups, some of which are from
my postcopy series.
Dave
Dr. David Alan Gilbert (5):
migration/ram.c: Use RAMBlock rather than MemoryRegion
Split out end of migration code from migration_thread
Init page sizes in qtest
migrat
From: "Edgar E. Iglesias"
Prepare for adding the Hypervisor timer, no functional change.
Signed-off-by: Edgar E. Iglesias
Reviewed-by: Peter Maydell
Message-id: 1436791864-4582-5-git-send-email-edgar.igles...@gmail.com
Signed-off-by: Peter Maydell
---
target-arm/helper.c | 99 +++
-arm.git
tags/pull-target-arm-20150813
for you to fetch changes up to f7a6785e12d834d05200b0595070db453344b25d:
i.MX: Fix UART driver to work with unitialized "chardev" device (2015-08-13
11:26:22 +0100)
target-arm queue:
*
From: Jean-Christophe Dubois
Signed-off-by: Jean-Christophe Dubois
Reviewed-by: Peter Crosthwaite
Message-id:
e32fba56b9dae3cc7c83726550514b2d0c890ae0.1437080501.git@tribudubois.net
Signed-off-by: Peter Maydell
---
hw/timer/imx_gpt.c | 79 ++---
inclu
From: "Dr. David Alan Gilbert"
The code that gets run at the end of the migration process
is getting large, and I'm about to add more for postcopy.
Split it into a separate function.
Signed-off-by: Dr. David Alan Gilbert
---
migration/migration.c | 75 --
On Fri, Aug 07, 2015 at 01:33:24PM +0100, Peter Maydell wrote:
> This series does three things:
Hi,
Looks good to me!
Reviewed-by: Edgar E. Iglesias
Cheers,
Edgar
>
> (1) implement the "flush the TLB only for a specified MMU index"
> functionality that we talked about when we added all the
From: "Edgar E. Iglesias"
Replace magic constants with macros from
hw/arm/virt.h and hw/intc/arm_gic_common.h.
Reviewed-by: Peter Maydell
Signed-off-by: Edgar E. Iglesias
Message-id: 1436791864-4582-7-git-send-email-edgar.igles...@gmail.com
Signed-off-by: Peter Maydell
---
hw/arm/virt.c | 9
From: Pavel Fedin
This small inline returns correct GIC class name depending on whether we
use KVM acceleration or not. Avoids duplicating the condition everywhere.
Signed-off-by: Pavel Fedin
Reviewed-by: Peter Maydell
Message-id:
4f26901be9b844b563673ce3ad08eeedbb7a7132.1438758065.git.p.fe..
From: "Dr. David Alan Gilbert"
This time convert the external functions:
qemu_get_buffer, qemu_peek_buffer
qemu_put_buffer and qemu_put_buffer_async
Signed-off-by: Dr. David Alan Gilbert
---
include/migration/qemu-file.h | 10 +-
migration/qemu-file.c | 22 +++--
From: "Dr. David Alan Gilbert"
RAM migration mainly works on RAMBlocks but in a few places
uses data from MemoryRegions to access the same information that's
already held in RAMBlocks; clean it up just to avoid the
MemoryRegion use.
Signed-off-by: Dr. David Alan Gilbert
---
migration/ram.c | 2
From: Jean-Christophe Dubois
Signed-off-by: Jean-Christophe Dubois
Reviewed-by: Peter Crosthwaite
Message-id:
ff0b6720b1c55204e663f07be47c0203f6871084.1437080501.git@tribudubois.net
Signed-off-by: Peter Maydell
---
hw/misc/imx_ccm.c | 11 +--
1 file changed, 5 insertions(+), 6 de
From: "Dr. David Alan Gilbert"
One of my patches used a loop that was based on host page size;
it dies in qtest since qtest hadn't bothered init'ing it.
Signed-off-by: Dr. David Alan Gilbert
Reviewed-by: Juan Quintela
Reviewed-by: Amit Shah
---
qtest.c | 1 +
1 file changed, 1 insertion(+)
From: Jean-Christophe Dubois
The "chardev" property initialization might have failed (for example because
there are not enough chardevs provided by QEMU).
The serial device emulator needs to be able to work with an uninitialized
(NULL) chardev device pointer.
This patch adds some missing tests
From: "Dr. David Alan Gilbert"
This is a start on using size_t more in qemu-file and friends;
it fixes up QEMUFilePutBufferFunc and QEMUFileGetBufferFunc
to take size_t lengths and return ssize_t return values (like read(2))
and fixes up all the different implementations of them.
Note that I've
This series introduces support for GICv3 by KVM. Software emulation is
currently not supported.
This patchset applies on top of:
http://lists.nongnu.org/archive/html/qemu-devel/2015-08/msg00518.html
v8 => v9
- Removed all limitations on CPU and IRQ number from the base class
- Added back missing
This allows to use different GIC types from v2. There are no kernels which
could advertise KVM_CAP_DEVICE_CTRL without the actual ability to create
GIC with it.
Signed-off-by: Pavel Fedin
---
target-arm/kvm.c | 10 +-
1 file changed, 1 insertion(+), 9 deletions(-)
diff --git a/target-ar
From: "Edgar E. Iglesias"
Adds control for trapping selected timer and counter accesses to EL2.
Reviewed-by: Peter Maydell
Signed-off-by: Edgar E. Iglesias
Message-id: 1436791864-4582-3-git-send-email-edgar.igles...@gmail.com
Signed-off-by: Peter Maydell
---
target-arm/cpu.h| 1 +
targe
On Thu, Aug 13, 2015 at 12:24:16PM +0200, Maxime Leroy wrote:
> On Thu, Aug 13, 2015 at 11:18 AM, Michael S. Tsirkin wrote:
> > On Wed, Aug 12, 2015 at 02:25:41PM +0800, Ouyang Changchun wrote:
> >> Based on patch by Nikolay Nikolaev:
> >> Vhost-user will implement the multi queue support in a sim
From: "Edgar E. Iglesias"
Rename gt_cnt_reset to gt_timer_reset as the function really
resets the timers and not the counters. Move the registration
from counter regs to timer regs.
Signed-off-by: Edgar E. Iglesias
Reviewed-by: Peter Maydell
Message-id: 1436791864-4582-4-git-send-email-edgar.i
Add gic_version to VirtMachineState, set it to value of the option
and pass it around where necessary. Instantiate devices and fdt
nodes according to the choice.
max_cpus for virt machine increased to 126 (calculated from redistributor
space available in the memory map). GICv2 compatibility check
From: Jean-Christophe Dubois
Signed-off-by: Jean-Christophe Dubois
Reviewed-by: Peter Crosthwaite
Message-id:
b1d6f990229b2608bbaba24f4ff359571c0b07da.1437080501.git@tribudubois.net
Signed-off-by: Peter Maydell
---
hw/arm/kzm.c | 2 +-
hw/misc/imx_ccm.c | 70 ++-
On Wed, Jul 29, 2015 at 02:22:47PM +0200, Cornelia Huck wrote:
> Commit ef546f1275f6563e8934dd5e338d29d9f9909ca6 ("virtio: add
> feature checking helpers") introduced a helper __virtio_has_feature.
> We don't want to use reserved identifiers, though, so let's
> rename __virtio_has_feature to virtio
On Fri, Aug 07, 2015 at 04:55:56PM -0300, Eduardo Habkost wrote:
> Remove arguments that can be found in PCMachineState.
>
> Signed-off-by: Eduardo Habkost
> ---
> hw/i386/pc_piix.c| 4 +---
> hw/i386/pc_q35.c | 4 +---
> include/hw/xen/xen.h | 4 ++--
> xen-hvm.c| 25
From: "Edgar E. Iglesias"
Signed-off-by: Edgar E. Iglesias
Reviewed-by: Peter Maydell
Message-id: 1436791864-4582-6-git-send-email-edgar.igles...@gmail.com
Signed-off-by: Peter Maydell
---
target-arm/cpu-qom.h | 1 +
target-arm/cpu.c | 2 ++
target-arm/cpu.h | 3 ++-
target-arm/hel
This is the initial version of KVM-accelerated GICv3 support.
State load and save are not yet supported, live migration is
not possible.
In order to get correct class name in a simpler way, gicv3_class_name()
function is implemented, similar to gic_class_name().
Signed-off-by: Pavel Fedin
---
h
Max replied that ignoring the UTF-8 error for 130 patch reported by
checkpatch.pl is fine.
Could you please apply the series if there are no further objections.
thanks
On 07/03/2015 03:28 PM, Bo Tu wrote:
v11.
1. Add Reviewed-by of Sascha
2. Refine code change in common.config in order to be
Wire up the secure timer interrupt. Since we've defined
that the plain old physical timer is the NS timer, we can
drop the now-out-of-date comment about QEMU not having TZ.
Use a data-driven loop to wire up the timer interrupts, since
we now have four of them and the code is the same for each.
Si
Hello!
I am now finishing v9 and (i hope) i fixed everything except this one. Sorry,
too many mails, i occasionally skipped this one earlier.
> Any particular reason for having two separate VIRT_ITS_*
> entries? The spec mandates that the two 64K pages of ITS
> have to be consecutive, so it wo
Since we now support both the hypervisor and the secure physical timer, wire
their interrupt lines up in the a15mpcore wrapper object.
Signed-off-by: Peter Maydell
Message-id: 1437047249-2357-5-git-send-email-peter.mayd...@linaro.org
Reviewed-by: Edgar E. Iglesias
---
hw/cpu/a15mpcore.c | 21 ++
These functions are useful also for vGICv3 implementation. Make them accessible
from within other modules.
Actually kvm_dist_get() and kvm_dist_put() could also be made reusable, but
they would require two extra parameters (s->dev_fd and s->num_cpu) as well as
lots of typecasts of 's' to DeviceSta
From: Shlomo Pongratz
This class is to be used by both software and KVM implementations of GICv3
Currently it is mostly a placeholder, but in future it is supposed to hold
qemu's representation of GICv3 state, which is necessary for migration.
The interface of this class is fully compatible wit
On 10/08/2015 17:27, fred.kon...@greensocs.com wrote:
> +while (!cpu->exit_request) {
> qemu_clock_enable(QEMU_CLOCK_VIRTUAL,
>(cpu->singlestep_enabled & SSTEP_NOTIMER) == 0);
>
> @@ -1507,7 +1480,7 @@ static void tcg_exec_all(void)
> }
> }
When debugging some patches on Windows, I discovered that nothing printed
to stderr ever appears on the console. Eventually I discovered that if I
build with --disable-sdl, then stderr appears just fine.
Looking at the code in vl.c I see a hack for SDL introduced in
commit 59a36a2f6728081050afc
Hello!
> Looking at the code in vl.c I see a hack for SDL introduced in
>
> commit 59a36a2f6728081050afc6ec97d0018467999f79
> Author: Stefan Weil
> Date: Thu Jun 18 20:11:03 2009 +0200
>
> Win32: Fix compilation with SDL.
Just a hint which may have to do with this. Looks like SDL
The following changes since commit 5c79ae3615d5dafdf1bb09b7a356a3a005714e3d:
Update version for v2.4.0 release (2015-08-11 15:30:34 +0100)
are available in the git repository at:
git://git.kernel.org/pub/scm/virt/kvm/mst/qemu.git tags/for_upstream
for you to fetch changes up to d31e5ae7f2c1
From: Eduardo Habkost
The existing i440fx initialization code sets a PCI config register that
isn't documented anywhere in the Intel 440FX datasheet. Register 0x57 is
DRAMC (DRAM Control) and has nothing to do with the RAM size.
This was implemented in commit ec5f92ce6ac8ec09056be77e03c941be1886
From: Victor Kaplansky
The old rules.mak loads dependency .d files using include directive
with file glob pattern "*.d". This breaks the build when build tree has
left-over *.d files from another build.
This patch fixes this by
- loading precise list of .d files made from *.o and *.mo.
- spe
From: Eduardo Habkost
Now we can use compat_props to keep CPUID feature compatibility, using
the boolean QOM properties for CPUID feature flags.
This simplifies the compatibility code, and reduces duplication between
pc_piix.c and pc_q35.c.
Signed-off-by: Eduardo Habkost
Reviewed-by: Michael S
From: Eduardo Habkost
Remove arguments that can be found in PCMachineState.
Signed-off-by: Eduardo Habkost
Reviewed-by: Michael S. Tsirkin
Signed-off-by: Michael S. Tsirkin
---
include/hw/i386/pc.h | 2 --
hw/i386/pc.c | 16 +++-
hw/i386/pc_piix.c| 5 +
hw/i386/
From: Eduardo Habkost
Make the code use the same variable name everywhere. "pcms" is already
being used in existing code and it's shorter.
Signed-off-by: Eduardo Habkost
Reviewed-by: Michael S. Tsirkin
Signed-off-by: Michael S. Tsirkin
---
hw/i386/pc.c | 4 ++--
hw/i386/pc_piix.c | 20
From: Victor Kaplansky
In rules like "bar/%.o: %.c" there is a difference between $(*D) and
$(@D). $(*D) expands to '.', while $(@D) expands to 'bar'. It is
cleaner to generate *.d in the same directory where appropriate *.o
resides. This allows precise including of dependency info from .d files
From: Eduardo Habkost
The function is not used by PC code anymore and can be removed.
Signed-off-by: Eduardo Habkost
Reviewed-by: Michael S. Tsirkin
Signed-off-by: Michael S. Tsirkin
---
target-i386/cpu.h | 3 ---
target-i386/cpu.c | 26 --
2 files changed, 29 deleti
From: Eduardo Habkost
They will be used inside hw/xen/xen.h, which doesn't include
hw/i386/pc.h.
Signed-off-by: Eduardo Habkost
Reviewed-by: Michael S. Tsirkin
Signed-off-by: Michael S. Tsirkin
---
include/hw/i386/pc.h| 4 +---
include/qemu/typedefs.h | 2 ++
2 files changed, 3 insertion
On 2015/8/13 18:51, Dr. David Alan Gilbert (git) wrote:
From: "Dr. David Alan Gilbert"
The code that gets run at the end of the migration process
is getting large, and I'm about to add more for postcopy.
Split it into a separate function.
Signed-off-by: Dr. David Alan Gilbert
---
Reviewed-b
From: "Daniel P. Berrange"
When building QEMU with Mingw64 toolchain I see a warning
CCx86_64-softmmu/hw/i386/acpi-build.o
hw/i386/acpi-build.c: In function 'acpi_build':
hw/i386/acpi-build.c:1138:9: warning: 'pm.cpu_hp_io_base' may be used
uninitialized in this function [-Wmaybe-unini
From: Eduardo Habkost
No errors should happen when registering the properties, but we
shouldn't silently ignore them if they happen.
Signed-off-by: Eduardo Habkost
Reviewed-by: Michael S. Tsirkin
Signed-off-by: Michael S. Tsirkin
---
hw/i386/pc.c | 16
1 file changed, 8 inse
From: Eduardo Habkost
pc_memory_init() already expects a PCMachineState object, there's no
point in upcasting it to MachineState before calling the function.
Signed-off-by: Eduardo Habkost
Reviewed-by: Michael S. Tsirkin
Signed-off-by: Michael S. Tsirkin
---
include/hw/i386/pc.h | 2 +-
hw/i
From: Eduardo Habkost
The only PC machines that didn't call pc_default_machine_options() were
isaps and xenfv. Both were already overwriting max_cpus, and only isapc
was not overwriting hot_add_cpu.
After making isapc set hot_add_cpu to NULL, we can move the
pc_default_machine_options() code the
From: Eduardo Habkost
All TYPE_PC_MACHINE subclasses call pc_common_machine_options().
TYPE_PC_MACHINE can simply initialize the common options on class_init
directly.
Signed-off-by: Eduardo Habkost
Reviewed-by: Michael S. Tsirkin
Signed-off-by: Michael S. Tsirkin
---
include/hw/i386/pc.h |
From: Eduardo Habkost
pc_cmos_init() already expects a PCMachineState object, there's no point
in upcasting it to MachineState before calling the function.
While doing it, reorder the arguments so PCMachineState is the first
function argument.
Signed-off-by: Eduardo Habkost
Reviewed-by: Michae
From: Eduardo Habkost
Signed-off-by: Eduardo Habkost
Reviewed-by: Michael S. Tsirkin
Signed-off-by: Michael S. Tsirkin
---
include/hw/i386/pc.h | 3 +--
hw/i386/pc.c | 7 +++
hw/i386/pc_piix.c| 3 +--
hw/i386/pc_q35.c | 3 +--
4 files changed, 6 insertions(+), 10 deletions
From: Eduardo Habkost
Remove arguments that can be found in PCMachineState.
Signed-off-by: Eduardo Habkost
Reviewed-by: Michael S. Tsirkin
Signed-off-by: Michael S. Tsirkin
---
include/hw/i386/pc.h | 5 +
hw/i386/pc.c | 28
hw/i386/pc_piix.c| 6
On 2015/8/13 18:51, Dr. David Alan Gilbert (git) wrote:
From: "Dr. David Alan Gilbert"
This time convert the external functions:
qemu_get_buffer, qemu_peek_buffer
qemu_put_buffer and qemu_put_buffer_async
Signed-off-by: Dr. David Alan Gilbert
---
Reviewed-by: zhanghailiang
includ
From: Eduardo Habkost
This will make the info readily available for the other initialization
functions, and will allow us to simplify their argument list.
Signed-off-by: Eduardo Habkost
Reviewed-by: Michael S. Tsirkin
Signed-off-by: Michael S. Tsirkin
---
include/hw/i386/pc.h | 1 +
hw/i386
From: Laurent Vivier
Some kernels program a 0 address for io regions. PCI 3.0 spec
section 6.2.5.1 doesn't seem to disallow this.
based on patch by Michael Roth
Add pci_allow_0_addr in MachineClass to conditionally
allow addr 0 for pseries, as this can break other architectures.
This patch al
From: Eduardo Habkost
Remove arguments that can be found in PCMachineState.
Signed-off-by: Eduardo Habkost
Reviewed-by: Michael S. Tsirkin
Signed-off-by: Michael S. Tsirkin
---
include/hw/i386/pc.h | 2 --
hw/i386/pc.c | 18 +-
hw/i386/pc_piix.c| 1 -
hw/i386/pc
From: Wei Huang
Current smbios builds type 19 table from e820, which is x86 specific.
This patch removes smbios' dependency on e820 by passing an array
of memory area to smbios_get_tables().
Acked-by: Gabriel Somlo
Tested-by: Gabriel Somlo
Reviewed-by: Laszlo Ersek
Tested-by: Leif Lindholm
S
On 12/08/2015 18:40, Paolo Bonzini wrote:
From: KONRAD Frederic
spinlock is only used in two cases:
* cpu-exec.c: to protect TranslationBlock
* mem_helper.c: for lock helper in target-i386 (which seems broken).
It's a pthread_mutex_t in user-mode so better using QemuMutex directly in thi
From: Wei Huang
To share smbios among different architectures, this patch moves SMBIOS
code (smbios.c and smbios.h) from x86 specific folders into new
hw/smbios directories. As a result, CONFIG_SMBIOS=y is defined in
x86 default config files.
Acked-by: Gabriel Somlo
Tested-by: Gabriel Somlo
Re
From: Jason Wang
After commit 40bad8f3deba15e2074ff34cfe923c12916b1cc5("virtio-net: fix
used len for tx"), async_tx.len was no longer used afterwards. So
remove useless codes with it.
Signed-off-by: Jason Wang
Reviewed-by: Michael S. Tsirkin
Signed-off-by: Michael S. Tsirkin
---
include/hw/v
From: Wei Huang
This patch extracts out the procedure of buidling x86 SMBIOS tables
into a dedicated function.
Acked-by: Gabriel Somlo
Tested-by: Gabriel Somlo
Reviewed-by: Laszlo Ersek
Tested-by: Leif Lindholm
Signed-off-by: Wei Huang
Reviewed-by: Michael S. Tsirkin
Signed-off-by: Michael
On 2015/8/13 18:51, Dr. David Alan Gilbert (git) wrote:
From: "Dr. David Alan Gilbert"
This is a start on using size_t more in qemu-file and friends;
it fixes up QEMUFilePutBufferFunc and QEMUFileGetBufferFunc
to take size_t lengths and return ssize_t return values (like read(2))
and fixes up a
Now that smbios has its own directory, list its
maintainers. Same people as ACPI so just reuse that
entry.
Signed-off-by: Michael S. Tsirkin
---
MAINTAINERS | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/MAINTAINERS b/MAINTAINERS
index 978b717..a059d5d 100644
--- a/MAINTA
On 13/08/2015 12:51, Dr. David Alan Gilbert (git) wrote:
> From: "Dr. David Alan Gilbert"
>
> RAM migration mainly works on RAMBlocks but in a few places
> uses data from MemoryRegions to access the same information that's
> already held in RAMBlocks; clean it up just to avoid the
> MemoryRegio
On 12/08/2015 18:41, Paolo Bonzini wrote:
softmmu requires more functions to be thread-safe, because translation
blocks can be invalidated from e.g. notdirty callbacks. Probably the
same holds for user-mode emulation, it's just that no one has ever
tried to produce a coherent locking there.
Thi
When a device is hotplugged, attach() sets "configured" to
false, waiting an action from the OS to configure it and then
to call ibm,configure-connector. On ibm,configure-connector,
the hypervisor sets "configured" to true.
In case of coldplugged device, attach() sets "configured" to
false, but fi
On 12/08/2015 18:41, Paolo Bonzini wrote:
From: KONRAD Frederic
This protects TBContext with tb_lock to make tb_* thread safe.
We can still have issue with tb_flush in case of multithread TCG:
another CPU can be executing code during a flush.
This can be fixed later by making all other TCG th
On 13/08/2015 14:51, Frederic Konrad wrote:
>> diff --git a/include/qom/cpu.h b/include/qom/cpu.h
>> index 77bbff2..56b1f4d 100644
>> --- a/include/qom/cpu.h
>> +++ b/include/qom/cpu.h
>> @@ -285,7 +285,10 @@ struct CPUState {
>> void *env_ptr; /* CPUArchState */
>> struct Translati
> > +tb_lock();
> > tb = tb_gen_code(cpu, orig_tb->pc, orig_tb->cs_base, orig_tb->flags,
> >max_cycles | CF_NOCACHE);
>
> tb_gen_code() calls tb_alloc() which calls tb_flush() we end in a double
> tb_lock here.
> But that's probably not really important here as w
On 13/08/2015 15:01, Paolo Bonzini wrote:
+tb_lock();
tb = tb_gen_code(cpu, orig_tb->pc, orig_tb->cs_base, orig_tb->flags,
max_cycles | CF_NOCACHE);
tb_gen_code() calls tb_alloc() which calls tb_flush() we end in a double
tb_lock here.
But that's probably not r
On 12/08/2015 18:40, Paolo Bonzini wrote:
From: KONRAD Frederic
This removes tcg_halt_cond global variable.
We need one QemuCond per virtual cpu for multithread TCG.
Signed-off-by: KONRAD Frederic
Message-Id: <1439220437-23957-9-git-send-email-fred.kon...@greensocs.com>
[Keep tcg_halt_cond fo
On 13/08/2015 15:05, Frederic Konrad wrote:
>>
>>
>> This removes tcg_halt_cond global variable.
>> We need one QemuCond per virtual cpu for multithread TCG.
>>
>> Signed-off-by: KONRAD Frederic
>> Message-Id: <1439220437-23957-9-git-send-email-fred.kon...@greensocs.com>
>> [Keep tcg_halt_cond f
On 10/07/2015 10:57, Pavel Dovgalyuk wrote:
> @@ -2364,14 +2363,12 @@ static void gen_st_cond (DisasContext *ctx, uint32_t
> opc, int rt,
> #if defined(TARGET_MIPS64)
> case OPC_SCD:
> case R6_OPC_SCD:
> -save_cpu_state(ctx, 1);
> op_st_scd(t1, t0, rt, ctx);
>
On 13/08/2015 14:17, Frederic Konrad wrote:
>> diff --git a/linux-user/main.c b/linux-user/main.c
>> index fdee981..fd06ce9 100644
>> --- a/linux-user/main.c
>> +++ b/linux-user/main.c
>> @@ -107,7 +107,7 @@ static int pending_cpus;
>> /* Make sure everything is in a consistent state for callin
On 12/08/2015 18:40, Paolo Bonzini wrote:
Signed-off-by: Paolo Bonzini
---
cpu-exec.c | 6 ++
1 file changed, 6 insertions(+)
What about the icount part in CPUState and the tb_start/end ?
Can't this be removed as well?
Fred
diff --git a/cpu-exec.c b/cpu-exec.c
index 599e64d..bde5fd1 1
1 - 100 of 171 matches
Mail list logo
