Re: [Qemu-devel] nbd: Possible regression in 2.9 RCs

2017-04-06 Thread Kevin Wolf
Am 05.04.2017 um 23:13 hat Paolo Bonzini geschrieben: > On 05/04/2017 13:01, Kevin Wolf wrote: > > Am 04.04.2017 um 17:09 hat Paolo Bonzini geschrieben: > >> On 04/04/2017 16:53, Kevin Wolf wrote: > The big question is how this fits into release management. We have > another important re

Re: [Qemu-devel] [PATCH v6] vfio error recovery: kernel support

2017-04-06 Thread Cao jin
On 04/06/2017 06:36 AM, Michael S. Tsirkin wrote: > On Wed, Apr 05, 2017 at 04:19:10PM -0600, Alex Williamson wrote: >> On Thu, 6 Apr 2017 00:50:22 +0300 >> "Michael S. Tsirkin" wrote: >> >>> On Wed, Apr 05, 2017 at 01:38:22PM -0600, Alex Williamson wrote: The previous intention of trying t

Re: [Qemu-devel] qemu_file_rate_limit

2017-04-06 Thread Peter Xu
On Thu, Apr 06, 2017 at 12:43:16PM +0430, ali saeedi wrote: > Hi > what does 'qemu_file_rate_limit' function do? > thanks It returns 1 if the stream is not full (so that we can continue send something), otherwise 0. Generally, it's used like: if (qemu_file_rate_limit(f)) { /* we can stil

Re: [Qemu-devel] [PATCH v6] vfio error recovery: kernel support

2017-04-06 Thread Cao jin
On 04/06/2017 03:38 AM, Alex Williamson wrote: > On Wed, 5 Apr 2017 16:54:33 +0800 > Cao jin wrote: > >> Sorry for late. Distracted by other problem for a while. >> >> On 03/31/2017 02:16 AM, Alex Williamson wrote: >>> On Thu, 30 Mar 2017 21:00:35 +0300 >> > >

Re: [Qemu-devel] [PATCH v6] vfio error recovery: kernel support

2017-04-06 Thread Cao jin
On 04/06/2017 05:56 AM, Michael S. Tsirkin wrote: > On Wed, Apr 05, 2017 at 04:54:33PM +0800, Cao jin wrote: >> Apparently, I don't have experience to induce non-fatal error, device >> error is more of a chance related with the environment(temperature, >> humidity, etc) as I understand. > > I'm

Re: [Qemu-devel] Emulating external registers

2017-04-06 Thread Peter Maydell
On 5 April 2017 at 22:03, Wu, Michael Y [US] (MS) wrote: > My current approach is to create a new MemoryRegion in the init function of > the g3beige source code (mac_oldworld.c). My idea is to set up some mmio to > certain address to represent those registers. For now I only care about > readin

[Qemu-devel] qemu_file_rate_limit

2017-04-06 Thread ali saeedi
Hi what does 'qemu_file_rate_limit' function do? thanks

Re: [Qemu-devel] [PATCH 08/21] ipmi: introduce an ipmi_bmc_sdr_find() API

2017-04-06 Thread Cédric Le Goater
On 04/06/2017 09:36 AM, Cédric Le Goater wrote: > On 04/06/2017 07:36 AM, David Gibson wrote: >> On Wed, Apr 05, 2017 at 02:41:33PM +0200, Cédric Le Goater wrote: >>> This patch exposes a new IPMI routine to query a sdr entry from the >>> sdr table maintained by the IPMI BMC simulator. The API is v

Re: [Qemu-devel] [PATCH v3 03/13] s390x: Drop useless casts

2017-04-06 Thread Cornelia Huck
On Wed, 5 Apr 2017 14:47:31 -0500 Eric Blake wrote: > An upcoming Coccinelle cleanup script wanted to reformat the casts > present in this file - but on closer look, we don't need the casts > at all because C automatically converts void* to any other pointer. > > Signed-off-by: Eric Blake > --

Re: [Qemu-devel] [PATCH 08/21] ipmi: introduce an ipmi_bmc_sdr_find() API

2017-04-06 Thread Cédric Le Goater
On 04/06/2017 07:36 AM, David Gibson wrote: > On Wed, Apr 05, 2017 at 02:41:33PM +0200, Cédric Le Goater wrote: >> This patch exposes a new IPMI routine to query a sdr entry from the >> sdr table maintained by the IPMI BMC simulator. The API is very >> similar to the internal sdr_find_entry() routi

Re: [Qemu-devel] [PATCH 10/21] ipmi: add SET_SENSOR_READING command

2017-04-06 Thread Cédric Le Goater
On 04/05/2017 04:41 PM, Corey Minyard wrote: > On 04/05/2017 07:41 AM, Cédric Le Goater wrote: >> SET_SENSOR_READING is a complex IPMI command (IPMI spec : "35.17 Set >> Sensor Reading And Event Status Command"). Here is a very minimum >> framework fitting the Open PowerNV platform needs. This comm

Re: [Qemu-devel] [PATCH 08/21] ipmi: introduce an ipmi_bmc_sdr_find() API

2017-04-06 Thread David Gibson
On Wed, Apr 05, 2017 at 02:41:33PM +0200, Cédric Le Goater wrote: > This patch exposes a new IPMI routine to query a sdr entry from the > sdr table maintained by the IPMI BMC simulator. The API is very > similar to the internal sdr_find_entry() routine and should be used > the same way to query one

[Qemu-devel] [PATCH v8 9/9] intel_iommu: enable remote IOTLB

2017-04-06 Thread Peter Xu
This patch is based on Aviv Ben-David ()'s patch upstream: "IOMMU: enable intel_iommu map and unmap notifiers" https://lists.gnu.org/archive/html/qemu-devel/2016-11/msg01453.html However I removed/fixed some content, and added my own codes. Instead of translate() every page for iotlb invalid

[Qemu-devel] [PATCH v8 6/9] intel_iommu: use the correct memory region for device IOTLB notification

2017-04-06 Thread Peter Xu
From: Jason Wang We have a specific memory region for DMAR now, so it's wrong to trigger the notifier with the root region. Cc: Michael S. Tsirkin Cc: Paolo Bonzini Cc: Richard Henderson Cc: Eduardo Habkost Signed-off-by: Jason Wang Reviewed-by: Peter Xu Signed-off-by: Peter Xu --- hw/i3

[Qemu-devel] [PATCH v8 8/9] intel_iommu: allow dynamic switch of IOMMU region

2017-04-06 Thread Peter Xu
This is preparation work to finally enabled dynamic switching ON/OFF for VT-d protection. The old VT-d codes is using static IOMMU address space, and that won't satisfy vfio-pci device listeners. Let me explain. vfio-pci devices depend on the memory region listener and IOMMU replay mechanism to m

[Qemu-devel] [PATCH v8 5/9] memory: add MemoryRegionIOMMUOps.replay() callback

2017-04-06 Thread Peter Xu
Originally we have one memory_region_iommu_replay() function, which is the default behavior to replay the translations of the whole IOMMU region. However, on some platform like x86, we may want our own replay logic for IOMMU regions. This patch add one more hook for IOMMUOps for the callback, and i

[Qemu-devel] [PATCH v8 7/9] intel_iommu: provide its own replay() callback

2017-04-06 Thread Peter Xu
The default replay() don't work for VT-d since vt-d will have a huge default memory region which covers address range 0-(2^64-1). This will normally consumes a lot of time (which looks like a dead loop). The solution is simple - we don't walk over all the regions. Instead, we jump over the regions

[Qemu-devel] [PATCH v8 1/9] memory: add section range info for IOMMU notifier

2017-04-06 Thread Peter Xu
In this patch, IOMMUNotifier.{start|end} are introduced to store section information for a specific notifier. When notification occurs, we not only check the notification type (MAP|UNMAP), but also check whether the notified iova range overlaps with the range of specific IOMMU notifier, and skip th

[Qemu-devel] [PATCH v8 4/9] memory: introduce memory_region_notify_one()

2017-04-06 Thread Peter Xu
Generalizing the notify logic in memory_region_notify_iommu() into a single function. This can be further used in customized replay() functions for IOMMUs. Reviewed-by: David Gibson Signed-off-by: Peter Xu --- include/exec/memory.h | 15 +++ memory.c | 40 ++

[Qemu-devel] [PATCH v8 2/9] memory: provide IOMMU_NOTIFIER_FOREACH macro

2017-04-06 Thread Peter Xu
Reviewed-by: David Gibson Signed-off-by: Peter Xu --- include/exec/memory.h | 3 +++ memory.c | 4 ++-- 2 files changed, 5 insertions(+), 2 deletions(-) diff --git a/include/exec/memory.h b/include/exec/memory.h index 0840c89..07e43da 100644 --- a/include/exec/memory.h +++ b/includ

[Qemu-devel] [PATCH v8 3/9] memory: provide iommu_replay_all()

2017-04-06 Thread Peter Xu
This is an "global" version of exising memory_region_iommu_replay() - we announce the translations to all the registered notifiers, instead of a specific one. Reviewed-by: David Gibson Signed-off-by: Peter Xu --- include/exec/memory.h | 8 memory.c | 9 + 2 files c

[Qemu-devel] [PATCH v8 0/9] VT-d: vfio enablement and misc enhances

2017-04-06 Thread Peter Xu
This is v8 of vt-d vfio enablement series. v8 - remove patches 1-9 since merged already - add David's r-b for all the patches - add Aviv's s-o-b in the last patch - rename iommu to iommu_dmar [Jason] - rename last patch subject to "remote IOTLB" [Jason] - pick up jason's two patches to fix vhost b

<    1   2   3   4