Re: [PATCH V5 02/10] perf/amd/iommu: Consolidate and move perf_event_amd_iommu header

2016-03-20 Thread Borislav Petkov
On Fri, Mar 18, 2016 at 04:09:33PM +0700, Suravee Suthikulpanit wrote: > But the whole point is that since we are moving it to consolidate these > duplicated declarations, I think we should just put it in the most common > place. The include/linux/amd-iommu.h file is already there. It's not like

Re: [Patch V2 1/4] iommu/vt-d: replace *hdr with hdr[0] in struct dmar_drhd_unit

2016-03-20 Thread Thomas Gleixner
On Sun, 20 Mar 2016, Wei Yang wrote: > hdr in struct dmar_drhd_unit is used to point the DMAR hardware unit copied > at the end of struct dmar_drhd_unit. One zero-sized array may be more > elegant for this purpose. You forget to tell why. > This patch replace *hdr with hdr[0] in struct

Re: [PATCH 1/2] iommu/vt-d: replace *hdr with hdr[0] in struct dmar_drhd_unit

2016-03-20 Thread Wei Yang
Obsolete this one. V2 is sent. On Sat, Mar 12, 2016 at 09:57:38PM +, Wei Yang wrote: >hdr in struct dmar_drhd_unit is used to point the DMAR hardware unit copied >at the end of struct dmar_drhd_unit. One zero-sized array may be more >elegant for this purpose. > >This patch replace *hdr with

[Patch V2 2/4] iommu/vt-d: use zero-sized array in DMAR related ACPI structures

2016-03-20 Thread Wei Yang
1. DMAR table has variable number of remapping entries 2. DMAR hardware unit has variable number of device scope 3. DMAR device scope has variable number of pci path In current implementation, we use (head + 1) to access these variable number elements, which may not be obvious for audience.

[Patch V2 4/4] iommu/vt-d: refine dmar_acpi_dev_scope_init() with dmar_walk_dmar_table()

2016-03-20 Thread Wei Yang
dmar_acpi_dev_scope_init() iterates on the remapping structure and just do proper job for ANDD structure. This is the what dmar_walk_dmar_table() does. This patch improves the code with dmar_walk_dmar_table(). Signed-off-by: Wei Yang --- drivers/iommu/dmar.c | 56

[Patch V2 1/4] iommu/vt-d: replace *hdr with hdr[0] in struct dmar_drhd_unit

2016-03-20 Thread Wei Yang
hdr in struct dmar_drhd_unit is used to point the DMAR hardware unit copied at the end of struct dmar_drhd_unit. One zero-sized array may be more elegant for this purpose. This patch replace *hdr with hdr[0] in struct dmar_drhd_unit. Besides this, this patch includes other two changes: 1. remove

[Patch V2 0/4] Code refine for Intel IOMMU

2016-03-20 Thread Wei Yang
These four patches try to refine the Intel IOMMU. Patch 1/2 tries to make it more user friendly by add a zero-sized array in some dmar data structure. Patch 3 move the ckeck of Register Base Address ahead to avoid cleanup when it is NULL. Patch 4 re-use dmar_walk_dmar_table() to make the code

[PATCH 06/10] iommu/ipmmu-vmsa: Add optional root device feature

2016-03-20 Thread Magnus Damm
From: Magnus Damm Add root device handling to the IPMMU driver by allowing certain DT compat strings to enable has_cache_leaf_nodes that in turn will support both root devices with interrupts and leaf devices that face the actual IPMMU consumer devices.

[PATCH v2 0/2] iommu/vt-d: Fault logging improvements

2016-03-20 Thread Alex Williamson
Ratelimit and improve formatting. v2: - Use a single ratelimit state as suggested by Joe Perches, except I chose to move it up to dmar_fault() so that it includes the "handling fault status reg" pr_err and we can avoid collecting entries for logging if we don't plan to print them. -

Re: [v6, 5/5] mmc: sdhci-of-esdhc: fix host version for T4240-R1.0-R2.0

2016-03-20 Thread Arnd Bergmann
On Thursday 17 March 2016 12:01:01 Rob Herring wrote: > On Mon, Mar 14, 2016 at 05:45:43PM +, Scott Wood wrote: > > >> This makes the driver non-portable. Better identify the specific > > >> workarounds based on the compatible string for this device, or add a > > >> boolean DT property for

Re: [PATCH v5 6/9] dmaengine: rcar-dmac: group slave configuration

2016-03-20 Thread Laurent Pinchart
Hi Niklas, Thank you for the patch. On Tuesday 08 March 2016 03:42:51 Niklas Söderlund wrote: > Group slave address and transfer size in own structs for source and > destination. This is in preparation for hooking up the dma-mapping API > to the slave addresses. > > Signed-off-by: Niklas

Re: [PATCH 0/7] Intel IOMMU scalability improvements

2016-03-20 Thread Benjamin Serebrin via iommu
There are nice. Thanks very much for doing this work! We have some preliminary results, looking at scaling to high core counts. We tested the patches on a 2-socket high core count SNB-EP server with a Broadcomm NIC. Our benchmark uses 200 threads of TCP_RR. We see similar performance for IOMMU

[PATCH] iommu: io-pgtable: fixed a brace coding style issue.

2016-03-20 Thread Cosmin-Gabriel Samoila
Fixed a coding style issue. Signed-off-by: Cosmin-Gabriel Samoila --- drivers/iommu/io-pgtable.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/iommu/io-pgtable.c b/drivers/iommu/io-pgtable.c index 6f2e319..a4a9a42 100644 ---

[PATCH 05/10] iommu/ipmmu-vmsa: Introduce features, break out alias

2016-03-20 Thread Magnus Damm
From: Magnus Damm Introduce struct ipmmu_features to track various hardware and software implementation changes inside the driver for different kinds of IPMMU hardware. Add use_ns_alias_offset as a first example of a feature to control if the secure register bank

Re: [PATCH V5 02/10] perf/amd/iommu: Consolidate and move perf_event_amd_iommu header

2016-03-20 Thread Borislav Petkov
On Fri, Mar 18, 2016 at 12:11:01PM +0100, Joerg Roedel wrote: > On Fri, Mar 18, 2016 at 11:39:18AM +0100, Borislav Petkov wrote: > > Yeah, so arch/x86/include/asm/ has all the x86-specific stuff which is > > not exported to userspace, so moving stuff there makes sense to me. > > While the AMD