On Mon, Feb 29, 2016 at 10:12:10AM +0530, Bharata B Rao wrote:
> > > diff --git a/hw/ppc/spapr_rtas.c b/hw/ppc/spapr_rtas.c
> > > index b7c5ebd..cc0369e 100644
> > > --- a/hw/ppc/spapr_rtas.c
> > > +++ b/hw/ppc/spapr_rtas.c
> > > @@ -34,6 +34,7 @@
> > >
> > > #include "hw/ppc/spapr.h"
> > > #in
On Mon, Feb 29, 2016 at 04:15:25PM +0100, Igor Mammedov wrote:
> On Mon, 29 Feb 2016 18:25:25 +0530
> Bharata B Rao wrote:
>
> > On Mon, Feb 29, 2016 at 11:03:16AM +0100, Igor Mammedov wrote:
> > > On Mon, 29 Feb 2016 11:20:19 +0530
> > > Bharata B Rao wrote:
> > >
> > > > On Fri, Feb 26, 201
We have two places needing this, and a third one will come shortly.
So factor things out into a helper function to reduce code duplication.
Signed-off-by: Gerd Hoffmann
Reviewed-by: Daniel P. Berrange
Reviewed-by: Markus Armbruster
---
include/ui/console.h | 2 ++
ui/console.c | 23 ++
With all fixups being in place now, we can promote input-send-event
to stable abi by removing the x- prefix.
Signed-off-by: Gerd Hoffmann
Reviewed-by: Markus Armbruster
Reviewed-by: Eric Blake
---
qapi-schema.json | 12 +++-
qmp-commands.hx | 14 +++---
ui/input.c | 6 +
Use display device qdev id and head number instead of console index to
specify the QemuConsole. This makes things consistent with input
devices (for input routing) and vnc server configuration, which both use
display and head too.
Signed-off-by: Gerd Hoffmann
Reviewed-by: Daniel P. Berrange
Rev
All lowercase, use-dash instead of CamelCase.
Signed-off-by: Gerd Hoffmann
Reviewed-by: Daniel P. Berrange
Reviewed-by: Markus Armbruster
Reviewed-by: Eric Blake
---
hw/input/hid.c | 4 ++--
hw/input/ps2.c | 4 ++--
hw/input/virtio-input-hid.c | 4 ++--
monitor.c
/pull-usb-20160229-1' into
staging (2016-02-29 12:24:26 +)
are available in the git repository at:
git://git.kraxel.org/qemu tags/pull-input-20160301-1
for you to fetch changes up to 6575ccddf4e7c2484bc14b10d5e89f57506c3953:
qapi: promote input-send-event to stable (2016-03-01
Lowercase them.
Signed-off-by: Gerd Hoffmann
Reviewed-by: Daniel P. Berrange
Reviewed-by: Markus Armbruster
Reviewed-by: Eric Blake
---
qapi-schema.json | 5 +
qmp-commands.hx | 4 ++--
scripts/qapi.py | 1 -
3 files changed, 3 insertions(+), 7 deletions(-)
diff --git a/qapi-schema.jso
git shortlog rel-1.9.0..rel-1.9.1
=
Cole Robinson (1):
biostables: Support SMBIOS 2.6+ UUID format
Kevin O'Connor (7):
xhci: Check for device disconnects during USB2 reset polling
xhci: Wait for port enable even for USB3 devices
sdcard: Only
-1' into
staging (2016-02-29 12:24:26 +)
are available in the git repository at:
git://git.kraxel.org/qemu tags/pull-seabios-20160301-1
for you to fetch changes up to fee5b753ff0eb8b25685227804a60dbc4a2ce6ea:
seabios: update to 1.9.1 stable release (2016-03-01 0
On 02/29/2016 05:38 PM, Michael S. Tsirkin wrote:
On Sun, Feb 14, 2016 at 04:51:02PM +0800, Xiao Guangrong wrote:
The dsm memory is used to save the input parameters and store
the dsm result which is filled by QEMU.
The address of dsm memory is decided by bios and patched into
int32 object na
On 02/29/2016 05:38 PM, Michael S. Tsirkin wrote:
+/* Build NAME(, 0x) where 0x is encoded as a dword,
+ * and return the offset to 0x for runtime patching.
+ *
+ * Warning: runtime patching is best avoided. Only use this as
+ * a replacement for DataTableRegion (fo
On Tue, Mar 01, 2016 at 04:53:23PM +0800, Xiao Guangrong wrote:
>
>
> On 02/29/2016 05:38 PM, Michael S. Tsirkin wrote:
>
> >+/* Build NAME(, 0x) where 0x is encoded as a dword,
> >+ * and return the offset to 0x for runtime patching.
> >+ *
> >+ * Warning: runtime pa
At the moment presence of vfio-pci devices on a bus affect the way
the guest view table is allocated. If there is no vfio-pci on a PHB
and the host kernel supports KVM acceleration of H_PUT_TCE, a table
is allocated in KVM. However, if there is vfio-pci and we do yet not
KVM acceleration for these,
Each Partitionable Endpoint (IOMMU group) has an address range on a PCI bus
where devices are allowed to do DMA. These ranges are called DMA windows.
By default, there is a single DMA window, 1 or 2GB big, mapped at zero
on a PCI bus.
PAPR defines a DDW RTAS API which allows pseries guests
queryi
Currently TCE tables are created once at start and their sizes never
change. We are going to change that by introducing a Dynamic DMA windows
support where DMA configuration may change during the guest execution.
This changes spapr_tce_new_table() to create an empty zero-size IOMMU
memory region (
We are going to have multiple DMA windows soon so let's start preparing.
This adds a new helper to create a DMA window and makes use of it in
sPAPRPHBState::realize().
Signed-off-by: Alexey Kardashevskiy
---
hw/ppc/spapr_pci.c | 40 +++-
1 file changed, 27 in
The existing memory listener is called on RAM or PCI address space
which implies potentially different page size.
This uses new memory_region_iommu_get_page_sizes() for IOMMU regions
or falls back to qemu_real_host_page_size if RAM.
Signed-off-by: Alexey Kardashevskiy
---
Changes:
* uses the sma
Since a788f227 "memory: Allow replay of IOMMU mapping notifications"
when new VFIO listener is added, all existing IOMMU mappings are
replayed. However there is a problem that the base address of
an IOMMU memory region (IOMMU MR) is ignored which is not a problem
for the existing user (which is pse
LoPAPR dictates that during system reset all DMA windows must be removed
and the default DMA32 window must be created so does the patch.
At the moment there is just one window supported so no change in
behaviour is expected.
Signed-off-by: Alexey Kardashevskiy
---
hw/ppc/spapr_iommu.c | 2 +-
The page size is an attribute of an IOMMU, not a container as a container
may contain more just one IOMMU.
This moves iova_pgsizes from VFIOContainer to VFIOGuestIOMMU.
The following patch will use this.
This removes iova_pgsizes from Type1 IOMMU as it is not used there anyway
and when it will ge
This will be later used by the "ibm,reset-pe-dma-window" RTAS handler
which resets the DMA configuration to the defaults.
Signed-off-by: Alexey Kardashevskiy
---
hw/ppc/spapr_pci.c | 11 ---
include/hw/pci-host/spapr.h | 2 ++
2 files changed, 10 insertions(+), 3 deletions(-)
At the moment VFIOContainer uses one memory listener which listens on
PCI address space for both Type1 and sPAPR IOMMUs. Soon we will need
another listener to listen on RAM; this will do DMA memory
pre-registration for sPAPR guests which basically pins all guest
pages in the host physical RAM.
Thi
This allows dynamic allocation for migrating arrays.
Already existing VMSTATE_VARRAY_UINT32 requires an array to be
pre-allocated, however there are cases when the size is not known in
advance and there is no real need to enforce it.
This defines another variant of VMSTATE_VARRAY_UINT32 with WMS_
This adds a vfio_votify() callback to inform an IOMMU (and then its owner)
that VFIO started using the IOMMU. This is used by the pseries machine to
enable/disable in-kernel acceleration of TCE hypercalls.
Signed-off-by: Alexey Kardashevskiy
---
hw/ppc/spapr_iommu.c | 9 +
hw/ppc/spap
sPAPRTCETable has a need_vfio flag which is passed to
kvmppc_create_spapr_tce() and controls whether to create a guest view
table in KVM as this depends on the host kernel ability to accelerate
H_PUT_TCE for VFIO devices. We would set this flag at the moment
when sPAPRTCETable is created in spapr_t
We are going to have multiple DMA windows at different offsets on
a PCI bus. For the sake of migration, we will have as many TCE table
objects pre-created as many windows supported.
So we need a way to map windows dynamically onto a PCI bus
when migration of a table is completed but at this stage a
This makes use of the new "memory registering" feature. The idea is
to provide the userspace ability to notify the host kernel about pages
which are going to be used for DMA. Having this information, the host
kernel can pin them all once per user process, do locked pages
accounting (once) and not s
This adds support for Dynamic DMA Windows (DDW) option defined by
the SPAPR specification which allows to have additional DMA window(s)
This implements DDW for emulated and VFIO devices. As all TCE root regions
are mapped at 0 and 64bit long (and actual tables are child regions),
this replaces mem
Every IOMMU has some granularity which MemoryRegionIOMMUOps::translate
uses when translating, however this information is not available outside
the translate context for various checks.
This adds a get_page_sizes callback to MemoryRegionIOMMUOps and
a wrapper for it so IOMMU users (such as VFIO) c
On Tue, 1 Mar 2016 13:47:27 +0530
Bharata B Rao wrote:
> On Mon, Feb 29, 2016 at 04:15:25PM +0100, Igor Mammedov wrote:
> > On Mon, 29 Feb 2016 18:25:25 +0530
> > Bharata B Rao wrote:
> >
> > > On Mon, Feb 29, 2016 at 11:03:16AM +0100, Igor Mammedov wrote:
> > > > On Mon, 29 Feb 2016 11:20:
On 03/01/2016 05:08 PM, Michael S. Tsirkin wrote:
On Tue, Mar 01, 2016 at 04:53:23PM +0800, Xiao Guangrong wrote:
On 02/29/2016 05:38 PM, Michael S. Tsirkin wrote:
+/* Build NAME(, 0x) where 0x is encoded as a dword,
+ * and return the offset to 0x for runtime p
On Mon, Feb 29, 2016 at 11:46:42AM +0100, Igor Mammedov wrote:
> On Thu, 25 Feb 2016 21:52:41 +0530
> Bharata B Rao wrote:
>
> > Implement query cpu-slots that provides information about hot-plugged
> > as well as hot-pluggable CPU slots that the machine supports.
> >
> > TODO: As Eric suggested
On Tue, 1 Mar 2016 12:21:27 +1100
David Gibson wrote:
> On Mon, Feb 29, 2016 at 04:15:25PM +0100, Igor Mammedov wrote:
> > On Mon, 29 Feb 2016 18:25:25 +0530
> > Bharata B Rao wrote:
> > > On Mon, Feb 29, 2016 at 11:03:16AM +0100, Igor Mammedov wrote:
> > > > On Mon, 29 Feb 2016 11:20:19 +05
On Mon, 29 Feb 2016 19:33:15 +0100
Markus Armbruster wrote:
> Igor Mammedov writes:
>
> > if host_memory_backend_get_memory() were to return error and
>
> Start sentences with a capital letter, please.
>
> > NULL MemoryRegion, pc_dimm_check_memdev_is_busy() would crash
> > dereferrencing nu
If host_memory_backend_get_memory() were to return error and
NULL MemoryRegion, pc_dimm_check_memdev_is_busy() would crash
dereferencing NULL pointer in memory_region_is_mapped().
But if error is set and non NULL MemoryRegion is returned
then error_setg() will fail with "error already set" assertio
On 01/03/2016 07:18, Fam Zheng wrote:
> +/* Memory topology clips a memory region to 2^64, size.hi >= 0 means the
> + * section must cover any addr. */
Small improvement:
/* Memory topology clips a memory region to [0, 2^64); size.hi > 0 means
* the section must cover the entir
On 01/03/2016 07:18, Fam Zheng wrote:
> v2: In the optimization patch, factor out section_covers_addr() and use it.
> [Paolo, Peter]
> Check "ram_block == NULL" in patch 3. [Gonglei]
> Add Gonglei's rev-by in patches 1, 2, 4 and 5.
>
> The first four patches drop ram_addr from Memory
Signed-off-by: Cao jin
---
BTW: this doc seems little out of date, since pxb has already support Q35.
docs/pci_expander_bridge.txt| 6 +++---
hw/pci-bridge/pci_expander_bridge.c | 2 +-
2 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/docs/pci_expander_bridge.txt b/docs/pci
On Tue, Mar 01, 2016 at 10:16:18AM +0100, Igor Mammedov wrote:
> On Tue, 1 Mar 2016 13:47:27 +0530
> Bharata B Rao wrote:
>
> > On Mon, Feb 29, 2016 at 04:15:25PM +0100, Igor Mammedov wrote:
> > > On Mon, 29 Feb 2016 18:25:25 +0530
> > > Bharata B Rao wrote:
> > >
> > > > On Mon, Feb 29, 2016
On 03/01/2016 11:45 AM, Cao jin wrote:
Signed-off-by: Cao jin
---
BTW: this doc seems little out of date, since pxb has already support Q35.
Hi,
Yes, you are right, but we use a different device, pxb-pcie.
I will send a doc update about it, thanks for reminding me,
Marcel
docs/pci_expan
Fam Zheng writes:
> On Mon, 02/29 17:18, Alex Bennée wrote:
>>
>> Fam Zheng writes:
>>
>> > Signed-off-by: Fam Zheng
>>
>> I think we need to include the dtc libs in here unless we need to ship
>> all submodules in the tree as well.
>
> OK, will take a look.
>
>>
>> Is there a way to get a pla
Fam Zheng writes:
> On Mon, 02/29 17:08, Alex Bennée wrote:
>>
>> Fam Zheng writes:
>>
>> > This adds a group of make targets to run docker tests, all are available
>> > in source tree without running ./configure.
>> >
>> > The usage is shown by "make docker".
>> >
>> > Besides the fixed ones,
On 1 March 2016 at 05:07, Stefan Weil wrote:
> The rest of the code can use longjmp with stack unwinding.
>
> Signed-off-by: Stefan Weil
> ---
>
> This is a bug fix needed for 64 bit Windows.
>
> QEMU for Windows currently gets the wrong definition for
> sigsetjmp. It uses stack unwinding for lon
On Tue, Mar 01, 2016 at 12:37:14AM +0100, Max Reitz wrote:
> On 01.03.2016 00:24, Eric Blake wrote:
> > On 02/29/2016 04:19 PM, Max Reitz wrote:
> >> Turns out NBD is not so simple to do if you do it right. Anyway, this
> >> series adds blockdev-add support for NBD clients.
> >>
> >> Patches 1 and
On Thu, Feb 25, 2016 at 09:52:36PM +0530, Bharata B Rao wrote:
> Hi,
>
> This is an attempt to implement CPU hotplug for PowerPC sPAPR based on
> the approach suggested by Andreas. While I say that, I should also explicitly
> add that I have tried to follow Andreas' suggestions to the best of my
>
Am 01.03.2016 um 11:00 hat Daniel P. Berrange geschrieben:
> On Tue, Mar 01, 2016 at 12:37:14AM +0100, Max Reitz wrote:
> > On 01.03.2016 00:24, Eric Blake wrote:
> > > On 02/29/2016 04:19 PM, Max Reitz wrote:
> > >> Turns out NBD is not so simple to do if you do it right. Anyway, this
> > >> serie
On 29 February 2016 at 20:08, Jeff Cody wrote:
> The following changes since commit 071608b519adf62bc29c914343a21c5407ab1ac9:
>
> Merge remote-tracking branch 'remotes/kraxel/tags/pull-usb-20160229-1' into
> staging (2016-02-29 12:24:26 +)
>
> are available in the git repository at:
>
>
>
* Li Zhijian (lizhij...@cn.fujitsu.com) wrote:
>
>
> On 03/01/2016 04:04 AM, Dr. David Alan Gilbert wrote:
> >* Zhang Chen (zhangchen.f...@cn.fujitsu.com) wrote:
> >>From: zhangchen
> >>
> >>Hi,all
> >>
> >>This patch add an colo-proxy object, COLO-Proxy is a part of COLO,
> >>based on qemu netf
On Tue, 1 Mar 2016 12:19:21 +1100
David Gibson wrote:
> On Mon, Feb 29, 2016 at 04:42:58PM +0100, Igor Mammedov wrote:
> > On Thu, 25 Feb 2016 14:52:06 -0300
> > Eduardo Habkost wrote:
> >
> > > On Wed, Feb 24, 2016 at 03:42:18PM +0100, Igor Mammedov wrote:
> > > > On Tue, 23 Feb 2016 18:26
On Mon, Feb 29, 2016 at 7:40 PM, Markus Armbruster wrote:
> Option parameter "share" is missing. Without it, you get a *private*
> mmap(), which defeats ivshmem's purpose pretty thoroughly ;)
>
> While there, switch to the conventional mountpoint of hugetlbfs
> /dev/hugepages.
>
> Signed-off-by:
This patchset is against commit 2212ef27b342b98b220fe9 (fw-cfg: support
writeable blobs) on pci branch of Michael's git tree
and can be found at:
https://github.com/xiaogr/qemu.git nvdimm-acpi-v4
Changelog in v4:
- drop the unnecessary assert() in aml_concatenate() based on Igor's
suggesti
It will be used by nvdimm acpi
Signed-off-by: Xiao Guangrong
---
hw/acpi/aml-build.c | 14 ++
include/hw/acpi/aml-build.h | 2 ++
2 files changed, 16 insertions(+)
diff --git a/hw/acpi/aml-build.c b/hw/acpi/aml-build.c
index 6675535..45b7f0a 100644
--- a/hw/acpi/aml-build.c
32 bits IO port starting from 0x0a18 in guest is reserved for NVDIMM
ACPI emulation. The table, NVDIMM_DSM_MEM_FILE, will be patched into
NVDIMM ACPI binary code
OSPM uses this port to tell QEMU the final address of the DSM memory
and notify QEMU to emulate the DSM method
Signed-off-by: Xiao Guan
From: "Michael S. Tsirkin"
This is a very limited form of support for runtime patching -
similar in functionality to what we can do with ACPI_EXTRACT
macros in python, but implemented in C.
This is to allow ACPI code direct access to data tables -
which is exactly what DataTableRegion is there f
It will be used by nvdimm acpi
Signed-off-by: Xiao Guangrong
---
hw/acpi/aml-build.c | 7 +++
include/hw/acpi/aml-build.h | 1 +
2 files changed, 8 insertions(+)
diff --git a/hw/acpi/aml-build.c b/hw/acpi/aml-build.c
index 45b7f0a..bb0cf52 100644
--- a/hw/acpi/aml-build.c
+++ b/hw/a
As Igor suggested that we can report the BIOS patched operation region
so that OSPM could see that particular range is in use and be able to
notice conflicts if it happens some day
Signed-off-by: Xiao Guangrong
---
hw/acpi/nvdimm.c | 27 +++
1 file changed, 27 insertions(
The dsm memory is used to save the input parameters and store
the dsm result which is filled by QEMU.
The address of dsm memory is decided by bios and patched into
int32 object named "MEMA"
Signed-off-by: Xiao Guangrong
---
hw/acpi/nvdimm.c | 20 +---
1 file changed, 17 insertio
On 26/02/16 16:49, Peter Maydell wrote:
> On 26 February 2016 at 16:19, Leon Alrae wrote:
>> Cluster Power Controller (CPC) is responsible for power management in
>> multiprocessing system. It provides registers to control the power and the
>> clock frequency of the individual elements in the syst
If dsm memory is successfully patched, we let qemu fully emulate
the dsm method
This patch saves _DSM input parameters into dsm memory, tell dsm
memory address to QEMU, then fetch the result from the dsm memory
Signed-off-by: Xiao Guangrong
---
hw/acpi/nvdimm.c | 117 +++
Extend aml_operation_region() to use object as offset
Reviewed-by: Igor Mammedov
Signed-off-by: Xiao Guangrong
---
hw/acpi/aml-build.c | 4 ++--
hw/i386/acpi-build.c| 31 ---
include/hw/acpi/aml-build.h | 2 +-
3 files changed, 19 insertions(+), 18
Emulate dsm method after IO VM-exit
Currently, we only introduce the framework and no function is actually
supported
Signed-off-by: Xiao Guangrong
---
hw/acpi/aml-build.c | 2 +-
hw/acpi/nvdimm.c| 44
include/hw/acpi/aml-build.h
Hi
On Mon, Feb 29, 2016 at 7:40 PM, Markus Armbruster wrote:
> Event notifiers are designed for eventfd(2). They can fall back to
> pipes, but according to Paolo, event_notifier_init_fd() really
> requires the real thing, and should therefore be under #ifdef
> CONFIG_EVENTFD. Do that.
>
> Its o
On Mon, Feb 29, 2016 at 7:40 PM, Markus Armbruster wrote:
> qpci_pc_iomap() maps BARs one after the other, without padding. This
> is wrong. PCI Local Bus Specification Revision 3.0, 6.2.5.1. Address
> Maps: "all address spaces used are a power of two in size and are
> naturally aligned". That'
On Mon, Feb 29, 2016 at 7:40 PM, Markus Armbruster wrote:
> Test state of registers after reset.
>
> Test reading Interrupt Status clears it.
>
> Test (invalid) read of Doorbell.
>
> Add more comments.
>
> Signed-off-by: Markus Armbruster
Reviewed-by: Marc-André Lureau
> ---
> tests/ivshmem-
This patch implements record and replay of character devices.
It records chardevs communication in replay mode. Recorded information
include data read from backend and counter of bytes written
from frontend to backend to preserve frontend internal state.
If character device was configured through t
This patch introduces block driver that implement recording
and replaying of block devices' operations.
All block completion operations are added to the queue.
Queue is flushed at checkpoints and information about processed requests
is recorded to the log. In replay phase the queue is matched with
This set of patches is related to the reverse execution and deterministic
replay of qemu execution. It includes recording and replaying of serial devices
and block devices operations.
With these patches one can record and deterministically replay behavior
of the system with connected disk drives
qemu_clock_warp call in qemu_tcg_wait_io_event function is not needed
anymore, because it is called in every iteration of main_loop_wait.
Signed-off-by: Pavel Dovgalyuk
---
cpus.c |3 ---
1 files changed, 0 insertions(+), 3 deletions(-)
diff --git a/cpus.c b/cpus.c
index 898426c..01c9809 10
qemu_clock_warp function is called to update virtual clock when CPU
is sleeping. This function includes replay checkpoint to make execution
deterministic in icount mode.
Record/replay module flushes async event queue at checkpoints.
Some of the events (e.g., block devices operations) include intera
Lluís Vilanova writes:
> NOTE: This series should complete the framework for guest code tracing. From
> here on, other series can concurrently add actual events and improve the
> guest code tracing features and performance (e.g., control tracing
> independently on each vCPU).
>
On Mon, Feb 29, 2016 at 7:40 PM, Markus Armbruster wrote:
> test_ivshmem_server() waits until the first byte in BAR 2 contains the
> 0x42 we put into shared memory. Works because the byte reads zero
> until the device maps the shared memory gotten from the server.
>
> Check the IVPosition registe
This patch adds callback for flush request. This callback is responsible
for flushing whole block devices stack. bdrv_flush function does not
proceed to underlying devices. It should be performed by this callback
function, if needed.
Signed-off-by: Pavel Dovgalyuk
---
block/io.c|
On Mon, Feb 29, 2016 at 7:40 PM, Markus Armbruster wrote:
> Document missing test: behavior with MSI-X present but not enabled.
>
> For MSI-X, we test and clear the interrupt pending bit before testing
> the interrupt. For INTx, we only clear. Change to test and clear for
> consistency.
>
> Test
The segfault here is triggered by the driver notifying the stats queue
twice after adding a buffer to it. This effectively resets stats_vq_elem
back to NULL and QEMU crashes on the next stats timer tick in
balloon_stats_poll_cb.
This is a regression introduced in 51b19ebe4320f3dc, although admitte
motes/kraxel/tags/pull-usb-20160229-1' into
> staging (2016-02-29 12:24:26 +)
>
> are available in the git repository at:
>
>
> git://git.kraxel.org/qemu tags/pull-vga-20160301-1
>
> for you to fetch changes up to 05fa1c742fd6f66978b989ded0dd981ef11c4a0c:
>
>
On Do, 2016-02-18 at 13:38 +0200, Dimitris Aragiorgis wrote:
> In case of daemonize, use the logfile passed with the -D option in
> order to redirect stderr to it instead of /dev/null.
>
> Also remove some unused code in log.h.
Patch breaks interaction with libvirt. libvirt hangs on startup, whi
On Mon, Feb 29, 2016 at 7:40 PM, Markus Armbruster wrote:
> This started as an attempt to update ivshmem_device_spec.txt for
> clarity, accuracy and completeness while working on its code, and
> quickly became a full rewrite. Since the diff would be useless
> anyway, I'm using the opportunity to
On 29/02/2016 19:40, Markus Armbruster wrote:
> Event notifiers are designed for eventfd(2). They can fall back to
> pipes, but according to Paolo, event_notifier_init_fd() really
> requires the real thing, and should therefore be under #ifdef
> CONFIG_EVENTFD. Do that.
>
> Its only user is iv
On 29/02/2016 19:40, Markus Armbruster wrote:
> -if (!stat(path, &st) && S_ISDIR(st.st_mode)) {
> +ret = stat(path, &st);
> +if (!ret && S_ISDIR(st.st_mode)) {
> +/* path names a directory -> create a temporary file there */
> /* Make name safe to use with mkstemp by
On 29/02/2016 19:40, Markus Armbruster wrote:
> Option parameter "share" is missing. Without it, you get a *private*
> mmap(), which defeats ivshmem's purpose pretty thoroughly ;)
>
> While there, switch to the conventional mountpoint of hugetlbfs
> /dev/hugepages.
>
> Signed-off-by: Markus Ar
On 1 March 2016 at 09:38, Hunter Laux wrote:
> I was having trouble running golang on linux-user with an aarch64 target.
>
> It turns out that snappy is written in Go. When I tried the xenial aarch64
> preinstall image in qemu, Snappy was broken.
>
> For some reason, it calls sigaction on all the
On 29/02/2016 19:40, Markus Armbruster wrote:
> ivshmem has its very own code to create and map shared memory.
> Replace that with an implicitly created memory backend. Reduces the
> number of ways we create BAR 2 from three to two.
>
> Signed-off-by: Markus Armbruster
Very appreciated, but d
On 29/02/2016 19:40, Markus Armbruster wrote:
> ivshmem_realize() puts the shared memory region in a container region.
> Used to be necessary to permit delayed mapping of the shared memory.
> Now we don't do that anymore, the container is redundant. Drop it.
Can you explain why we don't do that
On 29/02/2016 19:40, Markus Armbruster wrote:
> -memory_region_init_ram_ptr(&s->ivshmem, OBJECT(s),
> +s->ivshmem_bar2 = g_new(MemoryRegion, 1);
> +memory_region_init_ram_ptr(s->ivshmem_bar2, OBJECT(s),
> "ivshmem.bar2", s->ivshmem_size, ptr);
> -qe
On Tue, Mar 01, 2016 at 12:15:21PM +0100, Gerd Hoffmann wrote:
> On Do, 2016-02-18 at 13:38 +0200, Dimitris Aragiorgis wrote:
> > In case of daemonize, use the logfile passed with the -D option in
> > order to redirect stderr to it instead of /dev/null.
> >
> > Also remove some unused code in log.
Signed-off-by: Paolo Bonzini
---
include/sysemu/os-win32.h | 1 +
util/log.c| 30 --
util/oslib-win32.c| 35 +++
3 files changed, 44 insertions(+), 22 deletions(-)
diff --git a/include/sysemu/os-win32.h b/includ
On 01/03/2016 12:15, Gerd Hoffmann wrote:
> On Do, 2016-02-18 at 13:38 +0200, Dimitris Aragiorgis wrote:
>> In case of daemonize, use the logfile passed with the -D option in
>> order to redirect stderr to it instead of /dev/null.
>>
>> Also remove some unused code in log.h.
>
> Patch breaks int
Commit 96c33a4 ("log: Redirect stderr to logfile if deamonized",
2016-02-22) wanted to move stderr of a daemonized QEMU to the file
specified with -D.
However, if -D was not passed, the patch had the side effect of not
redirecting stderr to /dev/null. This happened because qemu_logfile
was set to
On 01/03/2016 12:47, Daniel P. Berrange wrote:
> means that QEMU will never close stderr anymore, so libvirt things
> QEMU is still starting upforever.
>
> Given current libvirt behaviour / expectations, I think the only
> option is to revert this change.
Why not fix it instead? :)
Paolo
Sent by mistake, sorry.
Paolo
On 01/03/2016 12:48, Paolo Bonzini wrote:
> Signed-off-by: Paolo Bonzini
> ---
> include/sysemu/os-win32.h | 1 +
> util/log.c| 30 --
> util/oslib-win32.c| 35 +++
> 3 files chang
The new-in-ARMv8 YIELD instruction has been implemented to throw
an EXCP_YIELD back up to the QEMU main loop. In system emulation
we use this to decide to schedule a different guest CPU in SMP
configurations. In usermode emulation there is nothing to do,
so just ignore it and resume the guest.
Thi
From: David Hildenbrand
When mmu_translate debugging output is enabled, code won't compile.
Let's just use the same statement as in trigger_prot_fault().
Acked-by: Cornelia Huck
Signed-off-by: David Hildenbrand
Signed-off-by: Cornelia Huck
---
target-s390x/mmu_helper.c | 2 +-
1 file changed
This interface was only used by the old virtio machine and therefore
is not needed anymore.
Reviewed-by: David Hildenbrand
Reviewed-by: Halil Pasic
Signed-off-by: Cornelia Huck
---
target-s390x/cpu.h | 5 -
target-s390x/interrupt.c | 11 ---
target-s390x/kvm.c | 11 ---
From: David Hildenbrand
We can now also sync the fprs via kvm_run, avoiding one ioctl.
Reviewed-by: Christian Borntraeger
Signed-off-by: David Hildenbrand
Signed-off-by: Cornelia Huck
---
target-s390x/kvm.c | 11 +++
1 file changed, 11 insertions(+)
diff --git a/target-s390x/kvm.c b
From: Wei Yang
Even PCI_CAP_FLAGS has the same value as PCI_MSIX_FLAGS, the later one is
the more proper on retrieving MSIX entries.
This patch uses PCI_MSIX_FLAGS to retrieve the MSIX entries.
Signed-off-by: Wei Yang
CC: Cornelia Huck
CC: Christian Borntraeger
Message-Id: <1455895091-7589-3
The following changes since commit 9c74a853048f14fd9a3e2efa1e3a6935d00e7495:
Merge remote-tracking branch 'remotes/cody/tags/block-pull-request' into
staging (2016-03-01 09:54:53 +)
are available in the git repository at:
git://github.com/cohuck/qemu tags/s390x-20160301
From: Sascha Silbe
When configured to inject an NMI, watchdog_perform_action() may cause
the BQL to be temporarily relinquished (inject_nmi() → ... →
s390_nmi() → s390_cpu_restart() → run_on_cpu()). When the guest issues
diag 288 again in response to the NMI, the diag 288 operation will
race agai
From: Thomas Huth
The files have been deleted recently, no need to keep these entries
anymore.
Signed-off-by: Thomas Huth
Message-Id: <1456397100-22746-1-git-send-email-th...@redhat.com>
Signed-off-by: Cornelia Huck
---
MAINTAINERS | 1 -
1 file changed, 1 deletion(-)
diff --git a/MAINTAINER
From: Yi Min Zhao
Indicator refcounting interfaces are introduced. This patch fixes
introducing unneeded indicator mappings and failure to release
AISB mappings on deregistration.
Signed-off-by: Yi Min Zhao
Reviewed-by: Cornelia Huck
Signed-off-by: Cornelia Huck
---
hw/s390x/s390-pci-bus.h
1 - 100 of 394 matches
Mail list logo
