Re: [PATCH v3 1/2] clk: Add consumer APIs for discovering possible parent clocks

2016-06-30 Thread Scott Wood
On 06/29/2016 10:02 PM, Yuantian Tang wrote: >> -Original Message- >> From: Rafael J. Wysocki [mailto:r...@rjwysocki.net] >> Sent: Thursday, June 30, 2016 10:24 AM >> To: Yuantian Tang >> Cc: Scott Wood ; Russell King ; >> Michael Turquette ; Stephen Boyd >> ; Viresh Kumar ; linux- >> c...

[PATCH v5 00/44] dma-mapping: Use unsigned long for dma_attrs

2016-06-30 Thread Krzysztof Kozlowski
Hi, This is fifth approach for replacing struct dma_attrs with unsigned long. The main patch (1/44) doing the change is split into many subpatches for easier review (2-42). They should be squashed together when applying. Rebased on v4.7-rc5. For easier testing the patchset is available here:

[PATCH v5 21/44] powerpc: dma-mapping: Use unsigned long for dma_attrs

2016-06-30 Thread Krzysztof Kozlowski
Split out subsystem specific changes for easier reviews. This will be squashed with main commit. Signed-off-by: Krzysztof Kozlowski --- arch/powerpc/include/asm/dma-mapping.h| 7 +++ arch/powerpc/include/asm/iommu.h | 10 +- arch/powerpc/kernel/dma-iommu.c | 1

[PATCH v5 43/44] dma-mapping: Remove dma_get_attr

2016-06-30 Thread Krzysztof Kozlowski
After switching DMA attributes to unsigned long it is easier to just compare the bits. Signed-off-by: Krzysztof Kozlowski [for avr32] Acked-by: Hans-Christian Noren Egtvedt [for arc] Acked-by: Vineet Gupta [for arm64 and dma-iommu] Acked-by: Robin Murphy --- Documentation/DMA-API.txt

Re: [PATCH v2 RFC] pasemi: Fix boot failure on 4.7-rc1

2016-06-30 Thread Aneesh Kumar K.V
Olof Johansson writes: > Hi, > > On Wed, Jun 29, 2016 at 1:06 PM, Darren Stevens > wrote: >> >> Commit:d6a9996e84ac4beb7713e9485f4563e100a9b03e (powerpc/mm: >> vmalloc abstraction in preparation for radix) turned kernel memory >> and IO addresses from #defined constants to variables

Re: [PATCHv2 0/7] eBPF JIT for PPC64

2016-06-30 Thread Andreas Ziegler
at line 667 (linux-next of today, i.e., next-20160630): #ifdef CONFIG_PPC64 /* Function descriptor nastiness: Address + TOC */ ((u64 *)image)[0] = (u64)code_base; ((u64 *)image)[1] = local_paca->kernel_toc; #endif From my understanding

Re: [PATCHv2 0/7] eBPF JIT for PPC64

2016-06-30 Thread Denis Kirjanov
pc/net/bpf_jit_comp.c, however, there is still an #ifdef > CONFIG_PPC64 block at line 667 (linux-next of today, i.e., next-20160630): > > #ifdef CONFIG_PPC64 > /* Function descriptor nastiness: Address + TOC */ > ((u64 *)image)[0] = (u64)code_base;

Re: [RFC 3/3] powerpc: tm: Enable transactional memory (TM) lazily for userspace

2016-06-30 Thread Laurent Dufour
On 29/06/2016 08:34, Cyril Bur wrote: > Currently the MSR TM bit is always set if the hardware is TM capable. > This adds extra overhead as it means the TM SPRS (TFHAR, TEXASR and > TFAIR) must be swapped for each process regardless of if they use TM. > > For processes that don't use TM the TM MSR

Re: [PATCHv2 0/7] eBPF JIT for PPC64

2016-06-30 Thread Naveen N. Rao
f_jit_comp64.c if CONFIG_PPC64 is enabled. > > > > Inside arch/powerpc/net/bpf_jit_comp.c, however, there is still an #ifdef > > CONFIG_PPC64 block at line 667 (linux-next of today, i.e., next-20160630): > > > > #ifdef CONFIG_PPC64 > > /* Functi

[PATCH v3 0/7] PCI: Add support for enforcing all MMIO BARs not to share PAGE_SIZE

2016-06-30 Thread Yongji Xie
This series aims to add an option for PCI resource allocator to force BARs not to share PAGE_SIZE. This would make sense to VFIO driver. Because current VFIO implementation disallows to mmap sub-page(size < PAGE_SIZE) MMIO BARs which may share the same page with other BARs for security reasons

[PATCH v3 1/7] PCI: Ignore enforced alignment when kernel uses existing firmware setup

2016-06-30 Thread Yongji Xie
PCI resources allocator will use firmware setup and not try to reassign resource when PCI_PROBE_ONLY or IORESOURCE_PCI_FIXED is set. The enforced alignment in pci_reassigndev_resource_alignment() should be ignored in this case. Otherwise, some PCI devices' resources would be released here and not

[PATCH v3 4/7] PCI: Add a new option for resource_alignment to reassign alignment

2016-06-30 Thread Yongji Xie
When using resource_alignment kernel parameter, the current implement reassigns the alignment by changing resources' size which can potentially break some drivers. For example, the driver uses the size to locate some register whose length is related to the size. This patch adds a new option "nores

[PATCH v3 3/7] PCI: Do not disable memory decoding in pci_reassigndev_resource_alignment()

2016-06-30 Thread Yongji Xie
We should not disable memory decoding when we reassign alignment in pci_reassigndev_resource_alignment(). It's meaningless and have some side effect. For example, some fixup functions such as quirk_e100_interrupt() read PCI_COMMAND_MEMORY to know whether the devices has been initialized by the firm

[PATCH v3 5/7] PCI: Do not use IORESOURCE_STARTALIGN to identify bridge resources

2016-06-30 Thread Yongji Xie
Now we use the IORESOURCE_STARTALIGN to identify bridge resources in __assign_resources_sorted(). That's quite fragile. We may also set flag IORESOURCE_STARTALIGN for SR-IOV resources in some cases, for example, using the option "noresize" of parameter "pci=resource_alignment". In this patch, we t

[PATCH v3 6/7] PCI: Add support for enforcing all MMIO BARs to be page aligned

2016-06-30 Thread Yongji Xie
When vfio passthrough a PCI device of which MMIO BARs are smaller than PAGE_SIZE, guest will not handle the mmio accesses to the BARs which leads to mmio emulations in host. This is because vfio will not allow to passthrough one BAR's mmio page which may be shared with other BARs. Otherwise, there

[PATCH v3 2/7] PCI: Ignore enforced alignment to VF BARs

2016-06-30 Thread Yongji Xie
VF BARs are read-only zeroes according to SRIOV spec, the normal way(writing BARs) of allocating resources wouldn't be applied to VFs. The VFs' resources would be allocated when we enable SR-IOV capability. So we should not try to reassign alignment after we enable VFs. It's meaningless and will re

[PATCH v3 7/7] PCI: Add a macro to set default alignment for all PCI devices

2016-06-30 Thread Yongji Xie
Now we can use something like "pci=resource_alignment=*:*:*.*:noresize" to enforce the alignment of all MMIO BARs to be at least PAGE_SIZE so that we can passthrough sub-page(size < PAGE_SIZE) BARs to guest in VFIO module. But sometimes we may want to enable this feature by default on some platfor

Re: [PATCH 2/2] cxl: Workaround XSL bug that does not clear the RA bit after a reset

2016-06-30 Thread Frederic Barrat
Le 29/06/2016 20:51, Ian Munsie a écrit : From: Ian Munsie An issue was noted in our debug logs where the XSL would leave the RA bit asserted after an AFU reset operation, which would effectively prevent further AFU reset operations from working. Workaround the issue by clearing the RA bit w

[PATCH] cxl: Ignore CAPI adapters misplaced in switched slots

2016-06-30 Thread Philippe Bergheaud
One should not attempt to switch a PHB into CAPI mode if there is a switch between the PHB and the adapter. This patch modifies the cxl driver to ignore CAPI adapters misplaced in switched slots. Signed-off-by: Philippe Bergheaud --- This patch fixes Bz 142217. drivers/misc/cxl/pci.c | 29 +

Re: [v7, 1/2] cxl: Add mechanism for delivering AFU driver specific events

2016-06-30 Thread Michael Ellerman
On Thu, 2016-23-06 at 13:03:53 UTC, Philippe Bergheaud wrote: > This adds an afu_driver_ops structure with fetch_event() and > event_delivered() callbacks. An AFU driver such as cxlflash can fill > this out and associate it with a context to enable passing custom AFU > specific events to userspace.

Re: [V5, 1/3] devicetree/bindings: Add binding for operator panel on FSP machines

2016-06-30 Thread Michael Ellerman
On Wed, 2016-29-06 at 03:38:37 UTC, Suraj Jitindar Singh wrote: > Add a binding to Documentation/devicetree/bindings/powerpc/opal > (oppanel-opal.txt) for the operator panel which is present on IBM > Power Systems machines with FSPs. > > Signed-off-by: Suraj Jitindar Singh > Acked-by: Rob Herring

Re: [RESEND, v7, 2/2] cxl: Add set and get private data to context struct

2016-06-30 Thread Michael Ellerman
On Fri, 2016-24-06 at 06:47:07 UTC, Philippe Bergheaud wrote: > From: Michael Neuling > > This provides AFU drivers a means to associate private data with a cxl > context. This is particularly intended for make the new callbacks for > driver specific events easier for AFU drivers to use, as they

Re: [V5, 2/3] powerpc/opal: Add inline function to get rc from an ASYNC_COMP opal_msg

2016-06-30 Thread Michael Ellerman
On Wed, 2016-29-06 at 03:38:38 UTC, Suraj Jitindar Singh wrote: > An opal_msg of type OPAL_MSG_ASYNC_COMP contains the return code in the > params[1] struct member. However this isn't intuitive or obvious when > reading the code and requires that a user look at the skiboot > documentation or opal-a

Re: [V5, 3/3] powerpc/drivers: Add driver for operator panel on FSP machines

2016-06-30 Thread Michael Ellerman
On Wed, 2016-29-06 at 03:38:39 UTC, Suraj Jitindar Singh wrote: > Implement new character device driver to allow access from user space > to the operator panel display present on IBM Power Systems machines > with FSPs. > > This will allow status information to be presented on the display which > i

Re: powerpc/powernv: spelling mistake: "Retrived" -> "Retrieved"

2016-06-30 Thread Michael Ellerman
On Thu, 2016-23-06 at 17:05:56 UTC, Colin King wrote: > From: Colin Ian King > > trivial fix to spelling mistake in pr_debug message > > Signed-off-by: Colin Ian King Applied to powerpc next, thanks. https://git.kernel.org/powerpc/c/6e8a9279a85abd07d05e932284 cheers _

Re: [V2] powerpc/fadump: trivial fix of spelling mistake, clean up message

2016-06-30 Thread Michael Ellerman
On Mon, 2016-27-06 at 11:07:41 UTC, Colin King wrote: > From: Colin Ian King > > Fix trivial spelling mistake "rgistration". Also use pr_err > instead of printk and unsplit the string to keep it all on one > line. > > Signed-off-by: Colin Ian King Applied to powerpc next, thanks. https://git.

Re: [PATCH v3 1/2] clk: Add consumer APIs for discovering possible parent clocks

2016-06-30 Thread Rafael J. Wysocki
On Thursday, June 30, 2016 05:46:42 AM Scott Wood wrote: > On 06/29/2016 10:02 PM, Yuantian Tang wrote: > >> -Original Message- > >> From: Rafael J. Wysocki [mailto:r...@rjwysocki.net] > >> Sent: Thursday, June 30, 2016 10:24 AM > >> To: Yuantian Tang > >> Cc: Scott Wood ; Russell King ; >

Re: [PATCH 1/2] cxl: Fix bug where AFU disable operation had no effect

2016-06-30 Thread Frederic Barrat
Hi Ian, -static int afu_control(struct cxl_afu *afu, u64 command, +static int afu_control(struct cxl_afu *afu, u64 command, u64 clear, u64 result, u64 mask, bool enabled) I'm not a big fan of the new "clear" argument, which forces us to pass an extra 0 most of the time.

Re: [PATCH v3] cpuidle: Fix last_residency division

2016-06-30 Thread Shreyas B Prabhu
On 06/29/2016 08:31 PM, Nicolas Pitre wrote: > On Wed, 29 Jun 2016, Daniel Lezcano wrote: > >> On 06/29/2016 09:06 AM, Shreyas B. Prabhu wrote: >>> diff --git a/drivers/cpuidle/cpuidle.h b/drivers/cpuidle/cpuidle.h >>> index f87f399..c8ea5ad 100644 >>> --- a/drivers/cpuidle/cpuidle.h >>> +++ b/d

[PATCH] cpuidle: Fix last_residency division

2016-06-30 Thread Shreyas B. Prabhu
Snooze is a poll idle state in powernv and pseries platforms. Snooze has a timeout so that if a cpu stays in snooze for more than target residency of the next available idle state, then it would exit thereby giving chance to the cpuidle governor to re-evaluate and promote the cpu to a deeper idle s

[PATCH v3] powerpc/boot: Add OPAL console to epapr wrappers

2016-06-30 Thread Oliver O'Halloran
This patch adds an OPAL console backend to the powerpc boot wrapper so that decompression failures inside the wrapper can be reported to the user. This is important since it typically indicates data corruption in the firmware and other nasty things. Currently this only works when building a little

Re: [PATCH] cpuidle: Fix last_residency division

2016-06-30 Thread Shreyas B Prabhu
Please ignore this mail. I have resent this post using the correct version number. Sorry for the noise. On 06/30/2016 07:57 PM, Shreyas B. Prabhu wrote: > Snooze is a poll idle state in powernv and pseries platforms. Snooze > has a timeout so that if a cpu stays in snooze for more than target > r

[PATCH v4] cpuidle: Fix last_residency division

2016-06-30 Thread Shreyas B. Prabhu
Snooze is a poll idle state in powernv and pseries platforms. Snooze has a timeout so that if a cpu stays in snooze for more than target residency of the next available idle state, then it would exit thereby giving chance to the cpuidle governor to re-evaluate and promote the cpu to a deeper idle s

Re: [PATCH v4] cpuidle: Fix last_residency division

2016-06-30 Thread Daniel Lezcano
On 06/30/2016 04:34 PM, Shreyas B. Prabhu wrote: Snooze is a poll idle state in powernv and pseries platforms. Snooze has a timeout so that if a cpu stays in snooze for more than target residency of the next available idle state, then it would exit thereby giving chance to the cpuidle governor to

Re: [PATCH v3 2/9] kexec_file: Generalize kexec_add_buffer.

2016-06-30 Thread Dave Young
On 06/29/16 at 06:18pm, Thiago Jung Bauermann wrote: > Am Mittwoch, 29 Juni 2016, 15:47:51 schrieb Dave Young: > > On 06/28/16 at 07:18pm, Thiago Jung Bauermann wrote: > > > diff --git a/include/linux/kexec.h b/include/linux/kexec.h > > > index e8acb2b43dd9..e16d845d587f 100644 > > > --- a/include/

Re: [PATCH] cpuidle: Fix last_residency division

2016-06-30 Thread Nicolas Pitre
On Thu, 30 Jun 2016, Shreyas B. Prabhu wrote: > Snooze is a poll idle state in powernv and pseries platforms. Snooze > has a timeout so that if a cpu stays in snooze for more than target > residency of the next available idle state, then it would exit thereby > giving chance to the cpuidle governo

Re: [PATCH 1/2] cxl: Fix bug where AFU disable operation had no effect

2016-06-30 Thread Ian Munsie
Excerpts from Frederic Barrat's message of 2016-06-30 16:19:54 +0200: > I'm not a big fan of the new "clear" argument, which forces us to pass > an extra 0 most of the time. Why not always clearing the "action" bits > of the register before applying the command? They are mutually > exclusive, so

Re: [PATCH v4] cpuidle: Fix last_residency division

2016-06-30 Thread Nicolas Pitre
On Thu, 30 Jun 2016, Daniel Lezcano wrote: > On 06/30/2016 04:34 PM, Shreyas B. Prabhu wrote: > > Snooze is a poll idle state in powernv and pseries platforms. Snooze > > has a timeout so that if a cpu stays in snooze for more than target > > residency of the next available idle state, then it wou

Re: [PATCH v3 3/9] kexec_file: Factor out kexec_locate_mem_hole from kexec_add_buffer.

2016-06-30 Thread Dave Young
On 06/29/16 at 06:09pm, Thiago Jung Bauermann wrote: > Am Mittwoch, 29 Juni 2016, 15:45:18 schrieb Dave Young: > > On 06/28/16 at 07:18pm, Thiago Jung Bauermann wrote: > > > Am Dienstag, 28 Juni 2016, 15:20:55 schrieb Dave Young: > > > > On 06/27/16 at 04:21pm, Dave Young wrote: > > > > Using one a

Re: [PATCH v3 2/9] kexec_file: Generalize kexec_add_buffer.

2016-06-30 Thread Thiago Jung Bauermann
Am Donnerstag, 30 Juni 2016, 11:07:00 schrieb Dave Young: > On 06/29/16 at 06:18pm, Thiago Jung Bauermann wrote: > > Am Mittwoch, 29 Juni 2016, 15:47:51 schrieb Dave Young: > > > On 06/28/16 at 07:18pm, Thiago Jung Bauermann wrote: > > > > +/** > > > > + * struct kexec_buf - parameters for finding

Re: [PATCH 1/2] cxl: Fix bug where AFU disable operation had no effect

2016-06-30 Thread Frederic Barrat
Le 30/06/2016 17:32, Ian Munsie a écrit : For dedicated mode, the CAIA recommends an explicit reset of the AFU >(section 2.1.1). True, I had forgotten that procedure was added to the document before it was made public - I'll update the comment and resend. Actually, my point was that for ded

Re: [PATCH v3 3/9] kexec_file: Factor out kexec_locate_mem_hole from kexec_add_buffer.

2016-06-30 Thread Thiago Jung Bauermann
Am Donnerstag, 30 Juni 2016, 11:41:19 schrieb Dave Young: > On 06/29/16 at 06:09pm, Thiago Jung Bauermann wrote: > > Am Mittwoch, 29 Juni 2016, 15:45:18 schrieb Dave Young: > > > On 06/28/16 at 07:18pm, Thiago Jung Bauermann wrote: > > > > Am Dienstag, 28 Juni 2016, 15:20:55 schrieb Dave Young: > >

Re: [PATCH v3 2/9] kexec_file: Generalize kexec_add_buffer.

2016-06-30 Thread Thiago Jung Bauermann
Am Donnerstag, 30 Juni 2016, 12:49:44 schrieb Thiago Jung Bauermann: > Am Donnerstag, 30 Juni 2016, 11:07:00 schrieb Dave Young: > > On 06/29/16 at 06:18pm, Thiago Jung Bauermann wrote: > > > I'm not following. The IMA buffer patchset doesn't use > > > kexec_locate_mem_hole nor struct kexec_buf. >

Re: [PATCH 1/2] cxl: Fix bug where AFU disable operation had no effect

2016-06-30 Thread Ian Munsie
Excerpts from Frederic Barrat's message of 2016-06-30 17:50:00 +0200: > > Le 30/06/2016 17:32, Ian Munsie a écrit : > >> For dedicated mode, the CAIA recommends an explicit reset of the AFU > >> >(section 2.1.1). > > True, I had forgotten that procedure was added to the document before it > > was

[PATCH v2] cxl: Fix bug where AFU disable operation had no effect

2016-06-30 Thread Ian Munsie
From: Ian Munsie The AFU disable operation has a bug where it will not clear the enable bit and therefore will have no effect. To date this has likely been masked by fact that we perform an AFU reset before the disable, which also has the effect of clearing the enable bit, making the following di

Re: [PATCH] cxl: Ignore CAPI adapters misplaced in switched slots

2016-06-30 Thread Ian Munsie
Thanks Philippe - this looks like a decent solution to the problem (and I intend to use this for the upcoming cx4 support as well). Acked-by: Ian Munsie Excerpts from Philippe Bergheaud's message of 2016-06-30 13:45:37 +0200: > One should not attempt to switch a PHB into CAPI mode if there is >

[PATCH 0/2] powerpc/pseries: Implemented indexed-count hotplug memory management

2016-06-30 Thread Sahil Mehta
Indexed-count memory management allows addition and removal of contiguous lmb blocks to be added and removed with a single command. When compared to the series of calls previously required to manage contiguous blocks, indexed-count decreases command frequency and complexity, consequently preventing

[PATCH 1/2] powerpc/pseries: Implemented indexed-count hotplug memory add

2016-06-30 Thread Sahil Mehta
Indexed-count add for memory hotplug guarantees that a contiguous block of lmbs beginning at a specified will be assigned (NOT that lmbs will be added). Because of Qemu's per-DIMM memory management, the addition of a contiguous block of memory currently requires a series of individual calls. Ind

[PATCH 2/2] powerpc/pseries: Implemented indexed-count hotplug memory remove

2016-06-30 Thread Sahil Mehta
Indexed-count remove for memory hotplug guarantees that a contiguous block of lmbs beginning at a specified will be unassigned (NOT that lmbs will be removed). Because of Qemu's per-DIMM memory management, the removal of a contiguous block of memory currently requires a series of individual call

Re: [PATCH v3 3/9] kexec_file: Factor out kexec_locate_mem_hole from kexec_add_buffer.

2016-06-30 Thread Dave Young
On 06/30/16 at 01:08pm, Thiago Jung Bauermann wrote: > Am Donnerstag, 30 Juni 2016, 11:41:19 schrieb Dave Young: > > On 06/29/16 at 06:09pm, Thiago Jung Bauermann wrote: > > > Am Mittwoch, 29 Juni 2016, 15:45:18 schrieb Dave Young: > > > > On 06/28/16 at 07:18pm, Thiago Jung Bauermann wrote: > > >

Re: [PATCH v3 2/9] kexec_file: Generalize kexec_add_buffer.

2016-06-30 Thread Dave Young
On 06/30/16 at 01:42pm, Thiago Jung Bauermann wrote: > Am Donnerstag, 30 Juni 2016, 12:49:44 schrieb Thiago Jung Bauermann: > > Am Donnerstag, 30 Juni 2016, 11:07:00 schrieb Dave Young: > > > On 06/29/16 at 06:18pm, Thiago Jung Bauermann wrote: > > > > I'm not following. The IMA buffer patchset doe

[PATCH] linuxppc/devtree: Parse new DRC mem/cpu/dev device tree elements

2016-06-30 Thread Michael Bringmann
Several properties in the DRC device tree format are replaced by more compact representations to allow, for example, for the encoding of vast amounts of memory, and or reduced duplication of information in related data structures. "ibm,drc-info": This property, when present, replaces the following

Re: [PATCH v2 04/12] axonrom: Generate uevent after attribute available

2016-06-30 Thread Dan Williams
On Wed, Jun 29, 2016 at 6:59 PM, Fam Zheng wrote: > It is documented that KOBJ_ADD should be generated after the object's > attributes and children are ready. We can achieve this with the new > disk_gen_uevents interface. > > Signed-off-by: Fam Zheng > --- > arch/powerpc/sysdev/axonram.c | 3 ++

[PATCH 1/2] powerpc: define FUNC_START/FUNC_END

2016-06-30 Thread Anton Blanchard
From: Anton Blanchard gcc provides FUNC_START/FUNC_END macros to help with creating assembly functions. Mirror these in the kernel so we can more easily share code between userspace and the kernel. FUNC_END is just a stub since we don't currently annotate the end of kernel functions. It might ma

[PATCH 2/2] crypto: powerpc: Add POWER8 optimised crc32c

2016-06-30 Thread Anton Blanchard
From: Anton Blanchard Use the vector polynomial multiply-sum instructions in POWER8 to speed up crc32c. This is just over 41x faster than the slice-by-8 method that it replaces. Measurements on a 4.1 GHz POWER8 show it sustaining 52 GiB/sec. A simple btrfs write performance test: dd if=/de

Re: [PATCH v3 1/7] PCI: Ignore enforced alignment when kernel uses existing firmware setup

2016-06-30 Thread Gavin Shan
On Thu, Jun 30, 2016 at 06:53:07PM +0800, Yongji Xie wrote: >PCI resources allocator will use firmware setup and not try to >reassign resource when PCI_PROBE_ONLY or IORESOURCE_PCI_FIXED >is set. > >The enforced alignment in pci_reassigndev_resource_alignment() >should be ignored in this case. Othe

Re: [PATCH v3 2/7] PCI: Ignore enforced alignment to VF BARs

2016-06-30 Thread Gavin Shan
On Thu, Jun 30, 2016 at 06:53:08PM +0800, Yongji Xie wrote: >VF BARs are read-only zeroes according to SRIOV spec, >the normal way(writing BARs) of allocating resources wouldn't >be applied to VFs. The VFs' resources would be allocated >when we enable SR-IOV capability. So we should not try to >rea

Re: [PATCH v3 3/7] PCI: Do not disable memory decoding in pci_reassigndev_resource_alignment()

2016-06-30 Thread Gavin Shan
On Thu, Jun 30, 2016 at 06:53:09PM +0800, Yongji Xie wrote: >We should not disable memory decoding when we reassign alignment >in pci_reassigndev_resource_alignment(). It's meaningless and >have some side effect. For example, some fixup functions such as >quirk_e100_interrupt() read PCI_COMMAND_MEM

Re: [PATCH v2 00/12] gendisk: Generate uevent after attribute available

2016-06-30 Thread Fam Zheng
On Wed, 06/29 23:38, Christoph Hellwig wrote: > On Thu, Jun 30, 2016 at 02:35:54PM +0800, Fam Zheng wrote: > > also more code and less flexible IMO. For example, we need at least two > > variants, for attribute_group and device_attribute separately, right? > > Yes, or maybe just a calling convent

Re: [PATCH v2 04/12] axonrom: Generate uevent after attribute available

2016-06-30 Thread Fam Zheng
On Thu, 06/30 15:10, Dan Williams wrote: > On Wed, Jun 29, 2016 at 6:59 PM, Fam Zheng wrote: > > It is documented that KOBJ_ADD should be generated after the object's > > attributes and children are ready. We can achieve this with the new > > disk_gen_uevents interface. > > > > Signed-off-by: Fam

Re: [PATCH v2 05/12] aoeblk: Generate uevent after attribute available

2016-06-30 Thread Ed Cashin
On 06/29/2016 09:59 PM, Fam Zheng wrote: It is documented that KOBJ_ADD should be generated after the object's attributes and children are ready. We can achieve this with the new disk_gen_uevents interface. Looks like an improvement, thanks! -- Ed __

Re: [PATCH v2 00/12] gendisk: Generate uevent after attribute available

2016-06-30 Thread Dan Williams
On Thu, Jun 30, 2016 at 6:01 PM, Fam Zheng wrote: > On Wed, 06/29 23:38, Christoph Hellwig wrote: >> On Thu, Jun 30, 2016 at 02:35:54PM +0800, Fam Zheng wrote: >> > also more code and less flexible IMO. For example, we need at least two >> > variants, for attribute_group and device_attribute sepa

Re: [PATCH v3 4/7] PCI: Add a new option for resource_alignment to reassign alignment

2016-06-30 Thread Gavin Shan
On Thu, Jun 30, 2016 at 06:53:10PM +0800, Yongji Xie wrote: >When using resource_alignment kernel parameter, the current >implement reassigns the alignment by changing resources' size >which can potentially break some drivers. For example, the driver >uses the size to locate some register whose len

[PATCH] powerpc/fadump: Fix compile error due to missing semicolon

2016-06-30 Thread Ian Munsie
From: Ian Munsie The commit "powerpc/fadump: trivial fix of spelling mistake, clean up message" removed a semicolon causing the following compile failure: arch/powerpc/kernel/fadump.c: In function ‘fadump_invalidate_dump’: arch/powerpc/kernel/fadump.c:1014:2: error: expected ‘;’ before ‘}’ token

Re: [PATCH v3 5/7] PCI: Do not use IORESOURCE_STARTALIGN to identify bridge resources

2016-06-30 Thread Gavin Shan
On Thu, Jun 30, 2016 at 06:53:11PM +0800, Yongji Xie wrote: >Now we use the IORESOURCE_STARTALIGN to identify bridge resources >in __assign_resources_sorted(). That's quite fragile. We may also >set flag IORESOURCE_STARTALIGN for SR-IOV resources in some cases, >for example, using the option "nores

Re: [PATCH v3 1/7] PCI: Ignore enforced alignment when kernel uses existing firmware setup

2016-06-30 Thread Yongji Xie
Hi Gavin, On 2016/7/1 8:28, Gavin Shan wrote: On Thu, Jun 30, 2016 at 06:53:07PM +0800, Yongji Xie wrote: PCI resources allocator will use firmware setup and not try to reassign resource when PCI_PROBE_ONLY or IORESOURCE_PCI_FIXED is set. The enforced alignment in pci_reassigndev_resource_ali

linux-next: build failure after merge of the powerpc tree

2016-06-30 Thread Stephen Rothwell
Hi all, After merging the powerpc tree, today's linux-next build (powerpc allyesconfig) failed like this: arch/powerpc/kernel/fadump.c: In function 'fadump_invalidate_dump': arch/powerpc/kernel/fadump.c:1014:2: error: expected ';' before '}' token } ^ Caused by commit 4a03749f140c ("power

Re: [PATCH v3 2/7] PCI: Ignore enforced alignment to VF BARs

2016-06-30 Thread Yongji Xie
Hi Gavin, On 2016/7/1 8:39, Gavin Shan wrote: On Thu, Jun 30, 2016 at 06:53:08PM +0800, Yongji Xie wrote: VF BARs are read-only zeroes according to SRIOV spec, the normal way(writing BARs) of allocating resources wouldn't be applied to VFs. The VFs' resources would be allocated when we enable

Re: [PATCH v3 2/7] PCI: Ignore enforced alignment to VF BARs

2016-06-30 Thread Gavin Shan
On Fri, Jul 01, 2016 at 01:27:17PM +0800, Yongji Xie wrote: >>On Thu, Jun 30, 2016 at 06:53:08PM +0800, Yongji Xie wrote: >>>VF BARs are read-only zeroes according to SRIOV spec, >>>the normal way(writing BARs) of allocating resources wouldn't >>>be applied to VFs. The VFs' resources would be alloc

[PATCH v6] powerpc/timer - large decrementer support

2016-06-30 Thread Oliver O'Halloran
Power ISAv3 adds a large decrementer (LD) mode which increases the size of the decrementer register. The size of the enlarged decrementer register is between 32 and 64 bits with the exact size being dependent on the implementation. When in LD mode, reads are sign extended to 64 bits and a decrement

Re: [PATCH v3 3/7] PCI: Do not disable memory decoding in pci_reassigndev_resource_alignment()

2016-06-30 Thread Yongji Xie
Hi Gavin, On 2016/7/1 8:50, Gavin Shan wrote: On Thu, Jun 30, 2016 at 06:53:09PM +0800, Yongji Xie wrote: We should not disable memory decoding when we reassign alignment in pci_reassigndev_resource_alignment(). It's meaningless and have some side effect. For example, some fixup functions such

Re: [PATCH v3 2/7] PCI: Ignore enforced alignment to VF BARs

2016-06-30 Thread Yongji Xie
Hi Gavin, On 2016/7/1 14:05, Gavin Shan wrote: On Fri, Jul 01, 2016 at 01:27:17PM +0800, Yongji Xie wrote: On Thu, Jun 30, 2016 at 06:53:08PM +0800, Yongji Xie wrote: VF BARs are read-only zeroes according to SRIOV spec, the normal way(writing BARs) of allocating resources wouldn't be applied

Re: [PATCH v3 4/7] PCI: Add a new option for resource_alignment to reassign alignment

2016-06-30 Thread Yongji Xie
Hi Gavin, On 2016/7/1 10:25, Gavin Shan wrote: On Thu, Jun 30, 2016 at 06:53:10PM +0800, Yongji Xie wrote: When using resource_alignment kernel parameter, the current implement reassigns the alignment by changing resources' size which can potentially break some drivers. For example, the driver

Re: [PATCH v3 1/2] clk: Add consumer APIs for discovering possible parent clocks

2016-06-30 Thread Scott Wood
On Thu, 2016-06-30 at 15:29 +0200, Rafael J. Wysocki wrote: > On Thursday, June 30, 2016 05:46:42 AM Scott Wood wrote: > > > > On 06/29/2016 10:02 PM, Yuantian Tang wrote: > > > > > > > > > > > -Original Message- > > > > From: Rafael J. Wysocki [mailto:r...@rjwysocki.net] > > > > Sent: T