[PATCH v7 19/31] powerpc/pci: Use pci_scan_host_bridge() for simplicity

2015-03-23 Thread Yijing Wang
Now we could use pci_scan_host_bridge() to scan pci buses, provide powerpc specific pci_host_bridge_ops. Signed-off-by: Yijing Wang wangyij...@huawei.com CC: Benjamin Herrenschmidt b...@kernel.crashing.org CC: linuxppc-dev@lists.ozlabs.org --- arch/powerpc/kernel/pci-common.c | 60

Re: [PATCH v3 1/2] PCI: One more parameter to pci_set_pcie_reset_state()

2015-03-23 Thread Alex Williamson
On Mon, 2015-03-23 at 15:40 +1100, Gavin Shan wrote: On Sun, Mar 22, 2015 at 10:06:01PM -0600, Alex Williamson wrote: On Mon, 2015-03-23 at 14:56 +1100, Gavin Shan wrote: On Sun, Mar 22, 2015 at 09:34:33PM -0600, Alex Williamson wrote: On Mon, 2015-03-23 at 14:02 +1100, Gavin Shan wrote:

Re: [PATCH v4 0/3] powerpc: Enable seccomp filter support

2015-03-23 Thread Purcareata Bogdan
On 27.02.2015 22:54, Benjamin Herrenschmidt wrote: On Fri, 2015-02-27 at 09:28 +0200, Purcareata Bogdan wrote: Ping? What is the ping for ? Ben. Hello Ben, I just wanted to check with you what's the current status of these patches. I noticed in patchwork [1][2][3] that the patches are

[PATCH 0/3] Reduce system overhead of automatic NUMA balancing

2015-03-23 Thread Mel Gorman
These are three follow-on patches based on the xfsrepair workload Dave Chinner reported was problematic in 4.0-rc1 due to changes in page table management -- https://lkml.org/lkml/2015/3/1/226. Much of the problem was reduced by commit 53da3bc2ba9e (mm: fix up numa read-only thread grouping

[PATCH 2/3] mm: numa: Preserve PTE write permissions across a NUMA hinting fault

2015-03-23 Thread Mel Gorman
Protecting a PTE to trap a NUMA hinting fault clears the writable bit and further faults are needed after trapping a NUMA hinting fault to set the writable bit again. This patch preserves the writable bit when trapping NUMA hinting faults. The impact is obvious from the number of minor faults

Re: [PATCH 4/4] mm: numa: Slow PTE scan rate if migration failures occur

2015-03-23 Thread Mel Gorman
On Fri, Mar 20, 2015 at 10:02:23AM -0700, Linus Torvalds wrote: On Thu, Mar 19, 2015 at 9:13 PM, Dave Chinner da...@fromorbit.com wrote: Testing now. It's a bit faster - three runs gave 7m35s, 7m20s and 7m36s. IOWs's a bit better, but not significantly. page migrations are pretty much

[PATCH 3/3] mm: numa: Slow PTE scan rate if migration failures occur

2015-03-23 Thread Mel Gorman
Dave Chinner reported the following on https://lkml.org/lkml/2015/3/1/226 Across the board the 4.0-rc1 numbers are much slower, and the degradation is far worse when using the large memory footprint configs. Perf points straight at the cause - this is from 4.0-rc1 on the -o bhash=101073

Re: Generic IOMMU pooled allocator

2015-03-23 Thread Benjamin Herrenschmidt
On Mon, 2015-03-23 at 07:04 +0100, Arnd Bergmann wrote: My guess is that the ARM code so far has been concerned mainly with getting things to work in the first place, but scalability problems will only be seen when there are faster CPU cores become available. In any case, I think this is

[PATCH 1/3] mm: numa: Group related processes based on VMA flags instead of page table flags

2015-03-23 Thread Mel Gorman
Threads that share writable data within pages are grouped together as related tasks. This decision is based on whether the PTE is marked dirty which is subject to timing races between the PTE scanner update and when the application writes the page. If the page is file-backed, then background

[RFC PATCH] powerpc/mm/thp: Make page table walk safe against thp split/collapse

2015-03-23 Thread Aneesh Kumar K.V
We can disable a THP split or a hugepage collapse by disabling irq. We do send IPI to all the cpus in the early part of split/collapse, and disabling local irq ensure we don't make progress with split/collapse. Before using the pte_t pointer returned by find_linux_pte_or_hugepte(), we need to make

Re: Generic IOMMU pooled allocator

2015-03-23 Thread Arnd Bergmann
On Sunday 22 March 2015, Benjamin Herrenschmidt wrote: On Sun, 2015-03-22 at 18:07 -0400, Sowmini Varadhan wrote: On (03/23/15 09:02), Benjamin Herrenschmidt wrote: How does this relate to the ARM implementation? There is currently an effort going on to make that one shared with ARM64

Re: [RFC PATCH] powerpc/mm/thp: Make page table walk safe against thp split/collapse

2015-03-23 Thread Aneesh Kumar K.V
Benjamin Herrenschmidt b...@kernel.crashing.org writes: On Mon, 2015-03-23 at 20:30 +0530, Aneesh Kumar K.V wrote: -static inline pte_t *lookup_linux_ptep(pgd_t *pgdir, unsigned long hva, +static inline pte_t lookup_linux_pte(pgd_t *pgdir, unsigned long hva,

Re: Build regressions/improvements in v4.0-rc5

2015-03-23 Thread Geert Uytterhoeven
On Mon, Mar 23, 2015 at 8:16 PM, Geert Uytterhoeven ge...@linux-m68k.org wrote: JFYI, when comparing v4.0-rc5[1] to v4.0-rc4[3], the summaries are: - build errors: +19/-7 + /home/kisskb/slave/src/drivers/gpu/drm/bridge/ptn3460.c: error: implicit declaration of function 'devm_gpiod_get'

Re: Generic IOMMU pooled allocator

2015-03-23 Thread Sowmini Varadhan
On (03/23/15 12:29), David Miller wrote: In order to elide the IOMMU flush as much as possible, I implemnented a scheme for sun4u wherein we always allocated from low IOMMU addresses to high IOMMU addresses. In this regime, we only need to flush the IOMMU when we rolled over back to low

[RESEND PATCH v2] powerpc/book3s: Fix the MCE code to use CONFIG_KVM_BOOK3S_64_HANDLER

2015-03-23 Thread Mahesh J Salgaonkar
From: Mahesh Salgaonkar mah...@linux.vnet.ibm.com commit id 2ba9f0d changed CONFIG_KVM_BOOK3S_64_HV to tristate to allow HV/PR bits to be built as modules. But the MCE code still depends on CONFIG_KVM_BOOK3S_64_HV which is wrong. When user selects CONFIG_KVM_BOOK3S_64_HV=m to build HV/PR bits as

Re: [PATCH v3 2/2] drivers/vfio: Support EEH error injection

2015-03-23 Thread Alex Williamson
On Mon, 2015-03-23 at 16:20 +1100, Gavin Shan wrote: On Mon, Mar 23, 2015 at 04:10:20PM +1100, David Gibson wrote: On Mon, Mar 23, 2015 at 04:03:59PM +1100, Gavin Shan wrote: On Mon, Mar 23, 2015 at 02:43:03PM +1100, David Gibson wrote: On Mon, Mar 23, 2015 at 12:56:36PM +1100, Gavin Shan

Re: Generic IOMMU pooled allocator

2015-03-23 Thread David Miller
From: Sowmini Varadhan sowmini.varad...@oracle.com Date: Mon, 23 Mar 2015 12:54:06 -0400 If it was only an optimization (i.e., removing it would not break any functionality), and if this was done for older hardware, and *if* we believe that the direction of most architectures is to follow

[RESEND PATCH] KVM: PPC: Book3S HV: Deliver machine check with MSR(RI=0) to guest as MCE.

2015-03-23 Thread Mahesh J Salgaonkar
From: Mahesh Salgaonkar mah...@linux.vnet.ibm.com For the machine check interrupt that happens while we are in the guest, kvm layer attempts the recovery, and then delivers the machine check interrupt directly to the guest if recovery fails. On successful recovery we go back to normal functioning

Re: [PATCH] powerpc/pseries: Introduce api_version to migration sysfs interface

2015-03-23 Thread Tyrel Datwyler
On 03/04/2015 06:25 PM, Tyrel Datwyler wrote: The /sys/kernel/mobility/migration interface was added all the way back in 2.6.37. However, the drmgr userspace tool was never augmented to use this interface to perfrom migrations. Instead it has continued using a faux rtas call coupled with

Re: Generic IOMMU pooled allocator

2015-03-23 Thread Sowmini Varadhan
On (03/23/15 15:05), David Miller wrote: Why add performance regressions to old machines who already are suffering too much from all the bloat we are constantly adding to the kernel? I have no personal opinion on this- it's a matter of choosing whether we want to have some extra baggage in

frequent lockups on ppc32

2015-03-23 Thread Pranith Kumar
Hello, I see frequent lockups with the latest rc5 kernel on a mac mini power pc 32 system. I see the following in dmesg. I could git bisect, but was wondering if you guys have seen this before and if anyone has any pointers which can reduce my bisect search space. Thanks! [ 5735.022209] Unable

Re: [PATCH v3 1/2] PCI: One more parameter to pci_set_pcie_reset_state()

2015-03-23 Thread cascardo
On Mon, Mar 23, 2015 at 06:40:34AM -0600, Alex Williamson wrote: On Mon, 2015-03-23 at 15:40 +1100, Gavin Shan wrote: On Sun, Mar 22, 2015 at 10:06:01PM -0600, Alex Williamson wrote: On Mon, 2015-03-23 at 14:56 +1100, Gavin Shan wrote: On Sun, Mar 22, 2015 at 09:34:33PM -0600, Alex

Re: [PATCH v2] powerpc/pseries: Simplify check for suspendability during suspend/migration

2015-03-23 Thread Tyrel Datwyler
On 03/04/2015 12:22 PM, Tyrel Datwyler wrote: During suspend/migration operation we must wait for the VASI state reported by the hypervisor to become Suspending prior to making the ibm,suspend-me RTAS call. Calling routines to rtas_ibm_supend_me() pass a vasi_state variable that exposes the

Re: [V6,1/9] elf: Add new powerpc specifc core note sections

2015-03-23 Thread Anshuman Khandual
On 03/19/2015 04:20 AM, Michael Neuling wrote: On Thu, 2015-03-19 at 09:45 +1100, Michael Neuling wrote: On Wed, 2015-03-18 at 13:53 +0100, Ulrich Weigand wrote: Michael Neuling mi...@neuling.org wrote on 23.02.2015 05:51:50: Sorry for the slow response. Same here :-( I'm going to break

Re: [PATCH v2] powerpc/book3s: Fix the MCE code to use CONFIG_KVM_BOOK3S_64_HANDLER

2015-03-23 Thread Alexander Graf
On 17.03.15 11:44, Mahesh J Salgaonkar wrote: From: Mahesh Salgaonkar mah...@linux.vnet.ibm.com commit id 2ba9f0d has changed CONFIG_KVM_BOOK3S_64_HV to tristate to allow HV/PR bits to be built as modules. But the MCE code still depends on CONFIG_KVM_BOOK3S_64_HV which is wrong. When user

Re: [PATCH 1/2] mm: Introducing arch_remap hook

2015-03-23 Thread Ingo Molnar
* Laurent Dufour lduf...@linux.vnet.ibm.com wrote: Some architecture would like to be triggered when a memory area is moved through the mremap system call. This patch is introducing a new arch_remap mm hook which is placed in the path of mremap, and is called before the old area is

Re: [PATCH 1/2] mm: Introducing arch_remap hook

2015-03-23 Thread Laurent Dufour
On 23/03/2015 09:52, Ingo Molnar wrote: * Laurent Dufour lduf...@linux.vnet.ibm.com wrote: Some architecture would like to be triggered when a memory area is moved through the mremap system call. This patch is introducing a new arch_remap mm hook which is placed in the path of mremap,

Re: [9/9] powerpc/hv-24x7: Add missing put_cpu_var()

2015-03-23 Thread Sukadev Bhattiprolu
Michael Ellerman [m...@ellerman.id.au] wrote: | On Tue, 2015-17-02 at 22:00:34 UTC, Sukadev Bhattiprolu wrote: | Add missing put_cpu_var() for 24x7 requests. | | When did it go missing? I assume in upstream, in which case this should be a | separate patch which I could merge for 4.0. It went

Re: Generic IOMMU pooled allocator

2015-03-23 Thread Benjamin Herrenschmidt
On Mon, 2015-03-23 at 15:05 -0400, David Miller wrote: From: Sowmini Varadhan sowmini.varad...@oracle.com Date: Mon, 23 Mar 2015 12:54:06 -0400 If it was only an optimization (i.e., removing it would not break any functionality), and if this was done for older hardware, and *if* we

Re: [PATCH v2] powerpc/book3s: Fix the MCE code to use CONFIG_KVM_BOOK3S_64_HANDLER

2015-03-23 Thread Michael Ellerman
On 23 March 2015 19:36:44 GMT+11:00, Alexander Graf ag...@suse.de wrote: On 17.03.15 11:44, Mahesh J Salgaonkar wrote: From: Mahesh Salgaonkar mah...@linux.vnet.ibm.com commit id 2ba9f0d has changed CONFIG_KVM_BOOK3S_64_HV to tristate to allow HV/PR bits to be built as modules. But the

Re: [PATCH v3 1/2] PCI: One more parameter to pci_set_pcie_reset_state()

2015-03-23 Thread Alex Williamson
On Mon, 2015-03-23 at 17:07 -0300, casca...@linux.vnet.ibm.com wrote: On Mon, Mar 23, 2015 at 06:40:34AM -0600, Alex Williamson wrote: On Mon, 2015-03-23 at 15:40 +1100, Gavin Shan wrote: On Sun, Mar 22, 2015 at 10:06:01PM -0600, Alex Williamson wrote: On Mon, 2015-03-23 at 14:56 +1100,

Re: [2/9] powerpc/hv24x7: Remove unnecessary parameter

2015-03-23 Thread Sukadev Bhattiprolu
Michael Ellerman [m...@ellerman.id.au] wrote: | On Tue, 2015-17-02 at 22:00:27 UTC, Sukadev Bhattiprolu wrote: | Use pr_notice_ratelimited() to log error messages and remove | the 'success_expected' parameter. | | I don't understand how this is equivalent? They are two unrelated changes that I

Re: [8/9] powerpc/hv-24x7: Break up single_24x7_request

2015-03-23 Thread Sukadev Bhattiprolu
Michael Ellerman [m...@ellerman.id.au] wrote: | +static void start_24x7_get_data(struct hv_24x7_request_buffer *request_buffer, | + struct hv_24x7_data_result_buffer *result_buffer) | +{ | | Just init_24x7_request() ? Sure. | | + | + memset(request_buffer, 0, 4096); |

Re: Generic IOMMU pooled allocator

2015-03-23 Thread Benjamin Herrenschmidt
On Mon, 2015-03-23 at 12:54 -0400, Sowmini Varadhan wrote: If it was only an optimization (i.e., removing it would not break any functionality), and if this was done for older hardware, and *if* we believe that the direction of most architectures is to follow the sun4v/HV model, then, given

Re: Generic IOMMU pooled allocator

2015-03-23 Thread David Miller
From: Sowmini Varadhan sowmini.varad...@oracle.com Date: Sun, 22 Mar 2015 15:27:26 -0400 That leaves only the odd iommu_flushall() hook, I'm trying to find the history behind that (needed for sun4u platforms, afaik, and not sure if there are other ways to achieve this). In order to elide the

Re: Generic IOMMU pooled allocator

2015-03-23 Thread Arnd Bergmann
On Monday 23 March 2015, Benjamin Herrenschmidt wrote: On Mon, 2015-03-23 at 07:04 +0100, Arnd Bergmann wrote: My guess is that the ARM code so far has been concerned mainly with getting things to work in the first place, but scalability problems will only be seen when there are faster

[PATCH v3 2/3] powerpc/powernv: Fundamental reset for PCI bus reset

2015-03-23 Thread Gavin Shan
Function pnv_pci_reset_secondary_bus() is used to reset specified PCI bus, which is leaded by root complex or PCI bridge. That means the function shouldn't be called on PCI root bus and the patch removes the logic for that case. Also, some adapters beneath the indicated PCI bus may require

[PATCH v3 1/3] powerpc/powernv: Use PCI slot reset infrastructure

2015-03-23 Thread Gavin Shan
For PowerNV platform, running on top of skiboot, all PE level reset should be routed to firmware if the bridge of the PE primary bus has device-node property ibm,reset-by-firmware. Otherwise, the kernel has to issue hot reset on PE's primary bus despite the requested reset types, which is the

Re: Generic IOMMU pooled allocator

2015-03-23 Thread Sowmini Varadhan
On (03/24/15 11:47), Benjamin Herrenschmidt wrote: Yes, pass a function pointer argument that can be NULL or just make it a member of the iommu_allocator struct (or whatever you call it) passed to the init function and that can be NULL. My point is we don't need a separate ops structure.

Re: [v7 2/2] powerpc/mpc85xx: Add FSL QorIQ DPAA QMan support to device tree(s)

2015-03-23 Thread Scott Wood
On Fri, 2015-02-27 at 09:16 -0600, Emil Medve wrote: From: Kumar Gala ga...@kernel.crashing.org Signed-off-by: Kumar Gala ga...@kernel.crashing.org Signed-off-by: Geoff Thorpe geoff.tho...@freescale.com Signed-off-by: Hai-Ying Wang haiying.w...@freescale.com Signed-off-by: Chunhe Lan

[PATCH v3 0/3] powerpc/powernv: Unified PCI slot reset

2015-03-23 Thread Gavin Shan
The patches intend to utilize the PCI slot reset capability exposed from the skiboot firmware, which requires corresponding changes as follows: Changelog = v2 - v3 * Rebased to Richard's SRIOV patchset. * Removed hotplug part, which needs rework. v1 - v2 * Keep opal_pci_reinit().

Re: Generic IOMMU pooled allocator

2015-03-23 Thread Sowmini Varadhan
On (03/24/15 09:21), Benjamin Herrenschmidt wrote: So we have two choices here that I can see: - Keep that old platform use the old/simpler allocator Problem with that approach is that the base struct iommu structure for sparc gets a split personality: the older one is used with the older

Re: [PATCH V14 17/21] powerpc/powernv: Shift VF resource with an offset

2015-03-23 Thread Daniel Axtens
On Fri, 2015-03-20 at 11:06 +0800, Wei Yang wrote: On PowerNV platform, resource position in M64 BAR implies the PE# the resource belongs to. In some cases, adjustment of a resource is necessary to locate it to a correct position in M64 BAR . This patch adds pnv_pci_vf_resource_shift() to

Re: [RFC PATCH] powerpc/mm/thp: Make page table walk safe against thp split/collapse

2015-03-23 Thread Benjamin Herrenschmidt
On Mon, 2015-03-23 at 20:30 +0530, Aneesh Kumar K.V wrote: -static inline pte_t *lookup_linux_ptep(pgd_t *pgdir, unsigned long hva, +static inline pte_t lookup_linux_pte(pgd_t *pgdir, unsigned long hva, unsigned long *pte_sizep) { pte_t *ptep;

Re: [PATCH v3 1/2] PCI: One more parameter to pci_set_pcie_reset_state()

2015-03-23 Thread Benjamin Herrenschmidt
On Mon, 2015-03-23 at 15:08 -0600, Alex Williamson wrote: I don't know what you're doing on POWER, I thought groups were equivalent to PEs, but on x86 we learn about isolation of PCI functions by standard PCI properties. Devices need to tell us that they're isolated via ACS capabilities (or

Re: powerpc/pseries: Introduce api_version to migration sysfs interface

2015-03-23 Thread Michael Ellerman
On Thu, 2015-05-03 at 02:25:38 UTC, Tyrel Datwyler wrote: The /sys/kernel/mobility/migration interface was added all the way back in 2.6.37. However, the drmgr userspace tool was never augmented to use this interface to perfrom migrations. Instead it has continued using a faux rtas call

Re: Generic IOMMU pooled allocator

2015-03-23 Thread Benjamin Herrenschmidt
On Mon, 2015-03-23 at 15:05 -0400, David Miller wrote: From: Sowmini Varadhan sowmini.varad...@oracle.com Date: Mon, 23 Mar 2015 12:54:06 -0400 If it was only an optimization (i.e., removing it would not break any functionality), and if this was done for older hardware, and *if* we

Re: Generic IOMMU pooled allocator

2015-03-23 Thread Sowmini Varadhan
On (03/24/15 09:36), Benjamin Herrenschmidt wrote: - One pool only - Whenever the allocation is before the previous hint, do a flush, that should only happen if a wrap around occurred or in some cases if the device DMA mask forced it. I think we always update the hint whenever we

Re: [PATCH 1/3] powerpc/mpc85xx: Add FMan clock nodes

2015-03-23 Thread Scott Wood
On Thu, 2015-02-26 at 09:26 -0600, Emil Medve wrote: From: Igal Liberman igal.liber...@freescale.com Signed-off-by: Igal Liberman igal.liber...@freescale.com --- arch/powerpc/boot/dts/fsl/b4si-post.dtsi| 11 +++ arch/powerpc/boot/dts/fsl/p2041si-post.dtsi | 8

Re: Generic IOMMU pooled allocator

2015-03-23 Thread Benjamin Herrenschmidt
On Mon, 2015-03-23 at 19:08 -0400, Sowmini Varadhan wrote: Sowmini, I see various options for the second choice. We could stick to 1 pool, and basically do as before, ie, if we fail on the first pass of alloc, it means we wrap around and do a flush, I don't think that will cause a

Re: [PATCH v3 2/2] drivers/vfio: Support EEH error injection

2015-03-23 Thread Gavin Shan
On Mon, Mar 23, 2015 at 10:14:59AM -0600, Alex Williamson wrote: On Mon, 2015-03-23 at 16:20 +1100, Gavin Shan wrote: On Mon, Mar 23, 2015 at 04:10:20PM +1100, David Gibson wrote: On Mon, Mar 23, 2015 at 04:03:59PM +1100, Gavin Shan wrote: On Mon, Mar 23, 2015 at 02:43:03PM +1100, David Gibson

Re: frequent lockups on ppc32

2015-03-23 Thread Benjamin Herrenschmidt
On Mon, 2015-03-23 at 15:28 -0400, Pranith Kumar wrote: Hello, I see frequent lockups with the latest rc5 kernel on a mac mini power pc 32 system. I see the following in dmesg. I could git bisect, but was wondering if you guys have seen this before and if anyone has any pointers which can

Re: [PATCH v3 1/2] PCI: One more parameter to pci_set_pcie_reset_state()

2015-03-23 Thread Gavin Shan
On Mon, Mar 23, 2015 at 06:40:34AM -0600, Alex Williamson wrote: On Mon, 2015-03-23 at 15:40 +1100, Gavin Shan wrote: On Sun, Mar 22, 2015 at 10:06:01PM -0600, Alex Williamson wrote: On Mon, 2015-03-23 at 14:56 +1100, Gavin Shan wrote: On Sun, Mar 22, 2015 at 09:34:33PM -0600, Alex Williamson

Re: Generic IOMMU pooled allocator

2015-03-23 Thread chase rayfield
On Mar 23, 2015 7:13 PM, Sowmini Varadhan sowmini.varad...@oracle.com wrote: On (03/24/15 09:21), Benjamin Herrenschmidt wrote: So we have two choices here that I can see: - Keep that old platform use the old/simpler allocator Problem with that approach is that the base struct iommu

Re: [PATCH v3 1/2] PCI: One more parameter to pci_set_pcie_reset_state()

2015-03-23 Thread Benjamin Herrenschmidt
On Mon, 2015-03-23 at 17:07 -0300, casca...@linux.vnet.ibm.com wrote: I agree with you here. I think the bigger issue is that we are not making sure VFIO is secure, allowing functions to be assigned separately to different guests, even when we cannot guarantee we can safely reset a single

[PATCH v3 3/3] powerpc/pci: Don't scan empty slot

2015-03-23 Thread Gavin Shan
In hotplug case, function pcibios_add_pci_devices() is called to rescan the specified PCI bus, which might not have any child devices. Access to the PCI bus's child device node will cause kernel crash without exception. The patch adds condition of skipping scanning PCI bus without child devices,

Re: Generic IOMMU pooled allocator

2015-03-23 Thread Benjamin Herrenschmidt
On Mon, 2015-03-23 at 19:19 -0400, Sowmini Varadhan wrote: What I've tried to do is to have a bool large_pool arg passed to iommu_tbl_pool_init. In my observation (instrumented for scsi, ixgbe), we never allocate more than 4 pages at a time, so I pass in large_pool == false for all the

Re: [PATCH v2] powerpc/pseries: Simplify check for suspendability during suspend/migration

2015-03-23 Thread Cyril Bur
On Wed, 2015-03-04 at 12:22 -0800, Tyrel Datwyler wrote: During suspend/migration operation we must wait for the VASI state reported by the hypervisor to become Suspending prior to making the ibm,suspend-me RTAS call. Calling routines to rtas_ibm_supend_me() pass a vasi_state variable that

Re: Generic IOMMU pooled allocator

2015-03-23 Thread David Miller
From: Benjamin Herrenschmidt b...@kernel.crashing.org Date: Tue, 24 Mar 2015 09:21:05 +1100 Dave, what's your feeling there ? Does anybody around still have some HW that we can test with ? I don't see what the actual problem is. Even if you use multiple pools, which we should for scalability

Re: Generic IOMMU pooled allocator

2015-03-23 Thread Sowmini Varadhan
benh It might be sufficient to add a flush counter and compare it between runs benh if actual wall-clock benchmarks are too hard to do (especially if you benh don't have things like very fast network cards at hand). benh benh Number of flush / number of packets might be a sufficient metric, it..

Re: [PATCH 3/3] selftests/powerpc: Add transactional syscall test

2015-03-23 Thread Michael Ellerman
On Tue, 2015-03-24 at 12:52 +1100, Sam Bobroff wrote: On 20/03/15 20:25, Anshuman Khandual wrote: On 03/19/2015 10:13 AM, Sam Bobroff wrote: Check that a syscall made during an active transaction will fail with the correct failure code and that one made during a suspended transaction will

Re: Generic IOMMU pooled allocator

2015-03-23 Thread Benjamin Herrenschmidt
On Mon, 2015-03-23 at 21:44 -0400, David Miller wrote: From: Benjamin Herrenschmidt b...@kernel.crashing.org Date: Tue, 24 Mar 2015 09:21:05 +1100 Dave, what's your feeling there ? Does anybody around still have some HW that we can test with ? I don't see what the actual problem is.

Re: Generic IOMMU pooled allocator

2015-03-23 Thread David Miller
From: Benjamin Herrenschmidt b...@kernel.crashing.org Date: Tue, 24 Mar 2015 13:08:10 +1100 For the large pool, we don't keep a hint so we don't know it's wrapped, in fact we purposefully don't use a hint to limit fragmentation on it, but then, it should be used rarely enough that flushing

Re: [PATCH 2/5] powerpc: Remove powerpc specific byteswap from bt8xx DVB driver

2015-03-23 Thread Benjamin Herrenschmidt
On Tue, 2015-02-03 at 16:36 +1100, David Gibson wrote: The bt8xx PCI DVB driver includes a powerpc specific hack, using one of the powerpc specific byteswapping functions in an IO helper macro. There's no reason to use the powerpc specific function instead of a generic byteswap, so this

Re: [PATCH 1/3] powerpc/tm: Abort syscalls in active transactions

2015-03-23 Thread Anshuman Khandual
On 03/24/2015 07:34 AM, Michael Ellerman wrote: On Fri, 2015-03-20 at 14:34 +0530, Anshuman Khandual wrote: On 03/19/2015 10:13 AM, Sam Bobroff wrote: This patch changes the syscall handler to doom (tabort) active transactions when a syscall is made and return immediately without performing

Re: [PATCH v3 2/2] drivers/vfio: Support EEH error injection

2015-03-23 Thread David Gibson
On Tue, Mar 24, 2015 at 09:47:54AM +1100, Gavin Shan wrote: On Mon, Mar 23, 2015 at 10:14:59AM -0600, Alex Williamson wrote: On Mon, 2015-03-23 at 16:20 +1100, Gavin Shan wrote: On Mon, Mar 23, 2015 at 04:10:20PM +1100, David Gibson wrote: On Mon, Mar 23, 2015 at 04:03:59PM +1100, Gavin Shan

[git pull] Please pull mpe/linux.git powerpc-4.0-3 tag

2015-03-23 Thread Michael Ellerman
Hi Linus, Please pull some powerpc fixes for 4.0: The following changes since commit 06e5801b8cb3fc057d88cb4dc03c0b64b2744cda: Linux 4.0-rc4 (2015-03-15 17:38:20 -0700) are available in the git repository at: git://git.kernel.org/pub/scm/linux/kernel/git/mpe/linux.git tags/powerpc-4.0-3

Re: [PATCH v3 2/2] drivers/vfio: Support EEH error injection

2015-03-23 Thread Gavin Shan
On Tue, Mar 24, 2015 at 12:22:25PM +1100, David Gibson wrote: On Tue, Mar 24, 2015 at 09:47:54AM +1100, Gavin Shan wrote: On Mon, Mar 23, 2015 at 10:14:59AM -0600, Alex Williamson wrote: On Mon, 2015-03-23 at 16:20 +1100, Gavin Shan wrote: On Mon, Mar 23, 2015 at 04:10:20PM +1100, David Gibson

Re: [PATCH V14 17/21] powerpc/powernv: Shift VF resource with an offset

2015-03-23 Thread Wei Yang
On Tue, Mar 24, 2015 at 10:22:26AM +1100, Daniel Axtens wrote: + +/* Do some magic shift */ +ret = pnv_pci_vf_resource_shift(pdev, pdn-offset); Given that you're already doing a version 15, would it be possible to include a more informative comment than Do some magic

Re: [PATCH 1/3] powerpc/tm: Abort syscalls in active transactions

2015-03-23 Thread Michael Ellerman
On Fri, 2015-03-20 at 14:34 +0530, Anshuman Khandual wrote: On 03/19/2015 10:13 AM, Sam Bobroff wrote: This patch changes the syscall handler to doom (tabort) active transactions when a syscall is made and return immediately without performing the syscall. Currently, the system call

Re: [PATCH 3/3] selftests/powerpc: Add transactional syscall test

2015-03-23 Thread Sam Bobroff
On 20/03/15 20:25, Anshuman Khandual wrote: On 03/19/2015 10:13 AM, Sam Bobroff wrote: Check that a syscall made during an active transaction will fail with the correct failure code and that one made during a suspended transaction will succeed. Signed-off-by: Sam Bobroff

[PATCH v8 19/30] powerpc/pci: Use pci_scan_host_bridge() for simplicity

2015-03-23 Thread Yijing Wang
Now we could use pci_scan_host_bridge() to scan pci buses, provide powerpc specific pci_host_bridge_ops. Signed-off-by: Yijing Wang wangyij...@huawei.com CC: Benjamin Herrenschmidt b...@kernel.crashing.org CC: linuxppc-dev@lists.ozlabs.org --- arch/powerpc/kernel/pci-common.c | 60