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.busy)
On 13/08/2015 07:46, Liang Li wrote:
Some cleanup operations take long time during the pause and copy stage,
especially with the KVM patch 3ea3b7fa9af067, do these operations after
the completion of live migration can help to reduce VM downtime.
Ony the first patch changes the behavior,
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 that use a
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 m...@redhat.com
To: Marcel Apfelbaum mar...@redhat.com
Cc: qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] [PATCH] virtio/vhost: drop unnecessary
From: Jean-Christophe Dubois j...@tribudubois.net
Signed-off-by: Jean-Christophe Dubois j...@tribudubois.net
Reviewed-by: Peter Crosthwaite crosthwaite.pe...@gmail.com
Message-id:
01e1d9026220992405819f25640ebd5bb843fc93.1437080501.git@tribudubois.net
Signed-off-by: Peter Maydell
From: Edgar E. Iglesias edgar.igles...@xilinx.com
Prepare for adding the Hypervisor timer, no functional change.
Signed-off-by: Edgar E. Iglesias edgar.igles...@xilinx.com
Reviewed-by: Peter Maydell peter.mayd...@linaro.org
Message-id: 1436791864-4582-5-git-send-email-edgar.igles...@gmail.com
-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:
* i.MX code
From: Dr. David Alan Gilbert dgilb...@redhat.com
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
From: Dr. David Alan Gilbert dgilb...@redhat.com
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
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 p.fe...@samsung.com
---
target-arm/kvm.c | 10 +-
1 file changed, 1 insertion(+), 9 deletions(-)
From: Edgar E. Iglesias edgar.igles...@xilinx.com
Adds control for trapping selected timer and counter accesses to EL2.
Reviewed-by: Peter Maydell peter.mayd...@linaro.org
Signed-off-by: Edgar E. Iglesias edgar.igles...@xilinx.com
Message-id:
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
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 peter.mayd...@linaro.org
Message-id: 1437047249-2357-5-git-send-email-peter.mayd...@linaro.org
Reviewed-by: Edgar E. Iglesias
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/detach
From: Jean-Christophe Dubois j...@tribudubois.net
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 j...@tribudubois.net
Reviewed-by: Peter Crosthwaite
From: Jean-Christophe Dubois j...@tribudubois.net
Signed-off-by: Jean-Christophe Dubois j...@tribudubois.net
Reviewed-by: Peter Crosthwaite peter.crosthwa...@xilinx.com
Message-id:
948927cab0c85da9a753c5f6d5501323d5604c8e.1437080501.git@tribudubois.net
Signed-off-by: Peter Maydell
From: Pavel Fedin p.fe...@samsung.com
Extracted duplicated initialization code from SW-emulated and KVM GIC
implementations and put into gic_init_irqs_and_mmio()
Signed-off-by: Pavel Fedin p.fe...@samsung.com
Message-id:
From: Jean-Christophe Dubois j...@tribudubois.net
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
From: Dr. David Alan Gilbert dgilb...@redhat.com
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 dgilb...@redhat.com
Reviewed-by: Juan Quintela quint...@redhat.com
Reviewed-by: Amit
From: Edgar E. Iglesias edgar.igles...@xilinx.com
Signed-off-by: Edgar E. Iglesias edgar.igles...@xilinx.com
Reviewed-by: Peter Maydell peter.mayd...@linaro.org
Message-id: 1436791864-4582-6-git-send-email-edgar.igles...@gmail.com
Signed-off-by: Peter Maydell peter.mayd...@linaro.org
---
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
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
From: Jean-Christophe Dubois j...@tribudubois.net
Signed-off-by: Jean-Christophe Dubois j...@tribudubois.net
Reviewed-by: Peter Crosthwaite crosthwaite.pe...@gmail.com
Message-id:
23ab872b7cd30b1399384fb26a2ebb75e9761d7b.1437080501.git@tribudubois.net
Signed-off-by: Peter Maydell
From: Jean-Christophe Dubois j...@tribudubois.net
Signed-off-by: Jean-Christophe Dubois j...@tribudubois.net
Reviewed-by: Peter Crosthwaite peter.crosthwa...@xilinx.com
Message-id:
a51ef50fa222a614169056d5389a6d3ed6a63b04.1437080501.git@tribudubois.net
Signed-off-by: Peter Maydell
From: Edgar E. Iglesias edgar.igles...@xilinx.com
Reviewed-by: Peter Maydell peter.mayd...@linaro.org
Signed-off-by: Edgar E. Iglesias edgar.igles...@xilinx.com
Message-id: 1436791864-4582-8-git-send-email-edgar.igles...@gmail.com
Signed-off-by: Peter Maydell peter.mayd...@linaro.org
---
From: Jean-Christophe Dubois j...@tribudubois.net
Signed-off-by: Jean-Christophe Dubois j...@tribudubois.net
Reviewed-by: Peter Crosthwaite crosthwaite.pe...@gmail.com
Message-id:
ff0b6720b1c55204e663f07be47c0203f6871084.1437080501.git@tribudubois.net
Signed-off-by: Peter Maydell
From: Dr. David Alan Gilbert dgilb...@redhat.com
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
On 12 August 2015 at 14:36, Paolo Bonzini pbonz...@redhat.com 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
On 13 August 2015 at 10:37, Paolo Bonzini pbonz...@redhat.com 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
From: Dr. David Alan Gilbert dgilb...@redhat.com
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 dgilb...@redhat.com
---
include/migration/qemu-file.h | 10 +-
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.
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
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
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)
}
}
-
On Thu, Aug 13, 2015 at 11:18 AM, Michael S. Tsirkin m...@redhat.com 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
From: Jean-Christophe Dubois j...@tribudubois.net
Signed-off-by: Jean-Christophe Dubois j...@tribudubois.net
Reviewed-by: Peter Crosthwaite crosthwaite.pe...@gmail.com
Message-id:
cc7d1589e774e87c346b75a6c25e07957f436ced.1437080501.git@tribudubois.net
Signed-off-by: Peter Maydell
From: Jean-Christophe Dubois j...@tribudubois.net
Signed-off-by: Jean-Christophe Dubois j...@tribudubois.net
Reviewed-by: Peter Crosthwaite crosthwaite.pe...@gmail.com
Message-id:
d8d70683c6a48ac318c1635595619cfb0eb31681.1437080501.git@tribudubois.net
Signed-off-by: Peter Maydell
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
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 peter.mayd...@linaro.org
Reviewed-by: Edgar E. Iglesias edgar.igles...@xilinx.com
Message-id:
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 edgar.igles...@xilinx.com
Cheers,
Edgar
(1) implement the flush the TLB only for a specified MMU index
functionality that we talked about
From: Edgar E. Iglesias edgar.igles...@xilinx.com
Replace magic constants with macros from
hw/arm/virt.h and hw/intc/arm_gic_common.h.
Reviewed-by: Peter Maydell peter.mayd...@linaro.org
Signed-off-by: Edgar E. Iglesias edgar.igles...@xilinx.com
Message-id:
From: Pavel Fedin p.fe...@samsung.com
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 p.fe...@samsung.com
Reviewed-by: Peter Maydell peter.mayd...@linaro.org
Message-id:
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 m...@redhat.com 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
From: Edgar E. Iglesias edgar.igles...@gmail.com
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 edgar.igles...@xilinx.com
Reviewed-by: Peter Maydell
From: Shlomo Pongratz shlomo.pongr...@huawei.com
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
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 code in
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 'dash',
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
From: Pavel Fedin p.fe...@samsung.com
Just specifying ops = NULL in some cases can be more convenient than having
two functions.
Signed-off-by: Pavel Fedin p.fe...@samsung.com
Acked-by: Paolo Bonzini pbonz...@redhat.com
Reviewed-by: Peter Maydell peter.mayd...@linaro.org
Message-id:
From: Jean-Christophe Dubois j...@tribudubois.net
Signed-off-by: Jean-Christophe Dubois j...@tribudubois.net
Reviewed-by: Peter Crosthwaite peter.crosthwa...@xilinx.com
Message-id:
06829257e845d693be05c7d491134313c1615d1a.1437080501.git@tribudubois.net
Signed-off-by: Peter Maydell
From: Edgar E. Iglesias edgar.igles...@xilinx.com
Adds support for the virtual timer offset controlled by EL2.
Reviewed-by: Peter Maydell peter.mayd...@linaro.org
Signed-off-by: Edgar E. Iglesias edgar.igles...@xilinx.com
Message-id: 1436791864-4582-2-git-send-email-edgar.igles...@gmail.com
From: Jean-Christophe Dubois j...@tribudubois.net
Signed-off-by: Jean-Christophe Dubois j...@tribudubois.net
Reviewed-by: Peter Crosthwaite peter.crosthwa...@xilinx.com
Message-id:
e32fba56b9dae3cc7c83726550514b2d0c890ae0.1437080501.git@tribudubois.net
Signed-off-by: Peter Maydell
From: Dr. David Alan Gilbert dgilb...@redhat.com
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 dgilb...@redhat.com
---
migration/migration.c | 75
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
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 ehabk...@redhat.com
---
hw/i386/pc_piix.c| 4 +---
hw/i386/pc_q35.c | 4 +---
include/hw/xen/xen.h | 4 ++--
xen-hvm.c
From: Jean-Christophe Dubois j...@tribudubois.net
Signed-off-by: Jean-Christophe Dubois j...@tribudubois.net
Reviewed-by: Peter Crosthwaite peter.crosthwa...@xilinx.com
Message-id:
b1d6f990229b2608bbaba24f4ff359571c0b07da.1437080501.git@tribudubois.net
Signed-off-by: Peter Maydell
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
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
From: Eduardo Habkost ehabk...@redhat.com
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
From: Eduardo Habkost ehabk...@redhat.com
Signed-off-by: Eduardo Habkost ehabk...@redhat.com
Reviewed-by: Michael S. Tsirkin m...@redhat.com
Signed-off-by: Michael S. Tsirkin m...@redhat.com
---
include/hw/i386/pc.h | 3 +--
hw/i386/pc.c | 7 +++
hw/i386/pc_piix.c| 3 +--
From: Laurent Vivier lviv...@redhat.com
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 mdr...@linux.vnet.ibm.com
Add pci_allow_0_addr in MachineClass to conditionally
allow addr 0 for pseries, as this
From: Eduardo Habkost ehabk...@redhat.com
Remove arguments that can be found in PCMachineState.
Signed-off-by: Eduardo Habkost ehabk...@redhat.com
Reviewed-by: Michael S. Tsirkin m...@redhat.com
Signed-off-by: Michael S. Tsirkin m...@redhat.com
---
include/hw/i386/pc.h | 2 --
hw/i386/pc.c
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 m...@redhat.com
---
MAINTAINERS | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/MAINTAINERS b/MAINTAINERS
index 978b717..a059d5d
On 13/08/2015 12:51, Dr. David Alan Gilbert (git) wrote:
From: Dr. David Alan Gilbert dgilb...@redhat.com
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
On 12/08/2015 18:40, Paolo Bonzini wrote:
Signed-off-by: Paolo Bonzini pbonz...@redhat.com
---
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
From: Eduardo Habkost ehabk...@redhat.com
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
From: Victor Kaplansky vict...@redhat.com
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
Hello!
Looking at the code in vl.c I see a hack for SDL introduced in
commit 59a36a2f6728081050afc6ec97d0018467999f79
Author: Stefan Weil w...@mail.berlios.de
Date: Thu Jun 18 20:11:03 2009 +0200
Win32: Fix compilation with SDL.
Just a hint which may have to do with this.
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
From: Eduardo Habkost ehabk...@redhat.com
They will be used inside hw/xen/xen.h, which doesn't include
hw/i386/pc.h.
Signed-off-by: Eduardo Habkost ehabk...@redhat.com
Reviewed-by: Michael S. Tsirkin m...@redhat.com
Signed-off-by: Michael S. Tsirkin m...@redhat.com
---
include/hw/i386/pc.h|
From: Eduardo Habkost ehabk...@redhat.com
The function is not used by PC code anymore and can be removed.
Signed-off-by: Eduardo Habkost ehabk...@redhat.com
Reviewed-by: Michael S. Tsirkin m...@redhat.com
Signed-off-by: Michael S. Tsirkin m...@redhat.com
---
target-i386/cpu.h | 3 ---
From: Eduardo Habkost ehabk...@redhat.com
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 ehabk...@redhat.com
Reviewed-by: Michael S. Tsirkin m...@redhat.com
From: Eduardo Habkost ehabk...@redhat.com
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
From: Wei Huang w...@redhat.com
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 so...@cmu.edu
From: Jason Wang jasow...@redhat.com
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 jasow...@redhat.com
Reviewed-by: Michael S. Tsirkin m...@redhat.com
On 12/08/2015 18:41, Paolo Bonzini wrote:
From: KONRAD Frederic fred.kon...@greensocs.com
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
+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 we want to
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
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 fred.kon...@greensocs.com
Message-Id: 1439220437-23957-9-git-send-email-fred.kon...@greensocs.com
[Keep
On 2015/8/13 18:51, Dr. David Alan Gilbert (git) wrote:
From: Dr. David Alan Gilbert dgilb...@redhat.com
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 dgilb...@redhat.com
---
From: Eduardo Habkost ehabk...@redhat.com
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 ehabk...@redhat.com
Reviewed-by: Michael S. Tsirkin m...@redhat.com
Signed-off-by: Michael
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.
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
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);
opn =
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 calling
fork().
On 13/08/2015 15:12, Paolo Bonzini wrote:
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
Am 03.07.2015 um 09:28 hat Bo Tu geschrieben:
Bo Tu (5):
qemu-iotests: qemu machine type support
qemu-iotests: disable default qemu devices for cross-platform
compatibility
qemu-iotests: s390x: fix test 041 and 055
qemu-iotests: s390x: fix test 049, reject negative sizes in
From: Daniel P. Berrange berra...@redhat.com
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
On 2015/8/13 18:51, Dr. David Alan Gilbert (git) wrote:
From: Dr. David Alan Gilbert dgilb...@redhat.com
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
From: Eduardo Habkost ehabk...@redhat.com
Remove arguments that can be found in PCMachineState.
Signed-off-by: Eduardo Habkost ehabk...@redhat.com
Reviewed-by: Michael S. Tsirkin m...@redhat.com
Signed-off-by: Michael S. Tsirkin m...@redhat.com
---
include/hw/i386/pc.h | 5 +
hw/i386/pc.c
From: Wei Huang w...@redhat.com
This patch extracts out the procedure of buidling x86 SMBIOS tables
into a dedicated function.
Acked-by: Gabriel Somlo so...@cmu.edu
Tested-by: Gabriel Somlo so...@cmu.edu
Reviewed-by: Laszlo Ersek ler...@redhat.com
Tested-by: Leif Lindholm
On 2015/8/13 18:51, Dr. David Alan Gilbert (git) wrote:
From: Dr. David Alan Gilbert dgilb...@redhat.com
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
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 TranslationBlock
On 12/08/2015 18:40, Paolo Bonzini wrote:
From: KONRAD Frederic fred.kon...@greensocs.com
This removes tcg_halt_cond global variable.
We need one QemuCond per virtual cpu for multithread TCG.
Signed-off-by: KONRAD Frederic fred.kon...@greensocs.com
Message-Id:
From: Eduardo Habkost ehabk...@redhat.com
Remove arguments that can be found in PCMachineState.
Signed-off-by: Eduardo Habkost ehabk...@redhat.com
Reviewed-by: Michael S. Tsirkin m...@redhat.com
Signed-off-by: Michael S. Tsirkin m...@redhat.com
---
include/hw/i386/pc.h | 2 --
hw/i386/pc.c
From: Eduardo Habkost ehabk...@redhat.com
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 ehabk...@redhat.com
Reviewed-by: Michael S. Tsirkin m...@redhat.com
Signed-off-by: Michael S. Tsirkin
From: Victor Kaplansky vict...@redhat.com
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
From: Eduardo Habkost ehabk...@redhat.com
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 ehabk...@redhat.com
Reviewed-by: Michael S. Tsirkin m...@redhat.com
Signed-off-by:
From: Eduardo Habkost ehabk...@redhat.com
No errors should happen when registering the properties, but we
shouldn't silently ignore them if they happen.
Signed-off-by: Eduardo Habkost ehabk...@redhat.com
Reviewed-by: Michael S. Tsirkin m...@redhat.com
Signed-off-by: Michael S. Tsirkin
1 - 100 of 170 matches
Mail list logo