Re: [PATCH 15/20] cpuidle/powernv: Add support for POWER ISA v3 idle states

2016-11-28 Thread Ram Pai
Sorry. please ignore this email. RP On Mon, Nov 28, 2016 at 11:08:59PM -0800, linux...@us.ibm.com wrote: > From: Shreyas B. Prabhu > > POWER ISA v3 defines a new idle processor core mechanism. In summary, > a) new instruction named stop is added. > b) new per thread SPR named PSSCR is added wh

Re: [PATCH v3 0/3] powernv:stop: Use psscr_val,mask provided by firmware

2016-11-28 Thread Balbir Singh
On 10/11/16 18:54, Gautham R. Shenoy wrote: > From: "Gautham R. Shenoy" > > This is the third iteration of the patchset to use the psscr_val and > psscr_mask provided by the firmware for each of the stop states. > > The previous version can be found here: > > [v2]: https://lkml.org/lkml/2016/

Re: [PATCH v3 3/3] powernv: Pass PSSCR value and mask to power9_idle_stop

2016-11-28 Thread Gautham R Shenoy
Hi Michael, On Wed, Nov 23, 2016 at 08:51:10PM +1100, Michael Ellerman wrote: > "Gautham R. Shenoy" writes: > > > From: "Gautham R. Shenoy" > > > > The power9_idle_stop method currently takes only the requested stop > > level as a parameter and picks up the rest of the PSSCR bits from a > > han

Re: [PATCH v3 2/3] cpuidle:powernv: Add helper function to populate powernv idle states.

2016-11-28 Thread Gautham R Shenoy
Hi Michael, On Wed, Nov 23, 2016 at 08:49:08PM +1100, Michael Ellerman wrote: > "Gautham R. Shenoy" writes: > > > diff --git a/drivers/cpuidle/cpuidle-powernv.c > > b/drivers/cpuidle/cpuidle-powernv.c > > index 7fe442c..9240e08 100644 > > --- a/drivers/cpuidle/cpuidle-powernv.c > > +++ b/driver

Re: [PATCH kernel v6 6/7] vfio/spapr: Reference mm in tce_container

2016-11-28 Thread David Gibson
On Thu, Nov 24, 2016 at 04:48:09PM +1100, Alexey Kardashevskiy wrote: > In some situations the userspace memory context may live longer than > the userspace process itself so if we need to do proper memory context > cleanup, we better have tce_container take a reference to mm_struct and > use it la

Re: [PATCH kernel v6 7/7] powerpc/mm/iommu, vfio/spapr: Put pages on VFIO container shutdown

2016-11-28 Thread David Gibson
On Thu, Nov 24, 2016 at 04:48:10PM +1100, Alexey Kardashevskiy wrote: > At the moment the userspace tool is expected to request pinning of > the entire guest RAM when VFIO IOMMU SPAPR v2 driver is present. > When the userspace process finishes, all the pinned pages need to > be put; this is done as

Re: [PATCH v4 7/7] PCI: Add comments about ROM BAR updating

2016-11-28 Thread Gavin Shan
On Mon, Nov 28, 2016 at 10:16:48PM -0600, Bjorn Helgaas wrote: >pci_update_resource() updates a hardware BAR so its address matches the >kernel's struct resource UNLESS it's a disabled ROM BAR. We only update >those when we enable the ROM. > >It's not obvious from the code why ROM BARs should be h

Re: [PATCH v4 6/7] PCI: Decouple IORESOURCE_ROM_ENABLE and PCI_ROM_ADDRESS_ENABLE

2016-11-28 Thread Gavin Shan
On Mon, Nov 28, 2016 at 10:16:07PM -0600, Bjorn Helgaas wrote: >Remove the assumption that IORESOURCE_ROM_ENABLE == PCI_ROM_ADDRESS_ENABLE. >PCI_ROM_ADDRESS_ENABLE is the ROM enable bit defined by the PCI spec, so if >we're reading or writing a BAR register value, that's what we should use. >IORESO

Re: [PATCH v4 5/7] PCI: Remove pci_resource_bar() and pci_iov_resource_bar()

2016-11-28 Thread Gavin Shan
On Mon, Nov 28, 2016 at 10:15:42PM -0600, Bjorn Helgaas wrote: >pci_std_update_resource() only deals with standard BARs, so we don't have >to worry about the complications of VF BARs in an SR-IOV capability. > >Compute the BAR address inline and remove pci_resource_bar(). That makes >pci_iov_resou

Re: [mm v2 0/3] Support memory cgroup hotplug

2016-11-28 Thread Balbir Singh
On 29/11/16 11:42, Tejun Heo wrote: > Hello, Balbir. > > On Tue, Nov 29, 2016 at 11:09:26AM +1100, Balbir Singh wrote: >> On 29/11/16 08:10, Tejun Heo wrote: >>> On Thu, Nov 24, 2016 at 12:05:12AM +1100, Balbir Singh wrote: On my desktop NODES_SHIFT is 6, many distro kernels have it a 9. I'

Re: [PATCH v4 4/7] PCI: Don't update VF BARs while VF memory space is enabled

2016-11-28 Thread Gavin Shan
On Mon, Nov 28, 2016 at 10:15:21PM -0600, Bjorn Helgaas wrote: >If we update a VF BAR while it's enabled, there are two potential problems: > > 1) Any driver that's using the VF has a cached BAR value that is stale > after the update, and > > 2) We can't update 64-bit BARs atomically, so the

Re: [PATCH v4 3/7] PCI: Separate VF BAR updates from standard BAR updates

2016-11-28 Thread Gavin Shan
On Mon, Nov 28, 2016 at 10:15:06PM -0600, Bjorn Helgaas wrote: >Previously pci_update_resource() used the same code path for updating >standard BARs and VF BARs in SR-IOV capabilities. > >Split the VF BAR update into a new pci_iov_update_resource() internal >interface, which makes it simpler to com

Re: [PATCH v3 4/4] powerpc/perf: macros for PowerISA v3.0 format encoding

2016-11-28 Thread Michael Ellerman
Madhavan Srinivasan writes: > diff --git a/arch/powerpc/perf/isa207-common.c > b/arch/powerpc/perf/isa207-common.c > index 2a2040ea5f99..e747bbf06661 100644 > --- a/arch/powerpc/perf/isa207-common.c > +++ b/arch/powerpc/perf/isa207-common.c > @@ -55,6 +55,81 @@ static inline bool event_is_fab_ma

Re: [PATCH v4 2/7] PCI: Ignore BAR updates on virtual functions

2016-11-28 Thread Gavin Shan
On Mon, Nov 28, 2016 at 10:14:29PM -0600, Bjorn Helgaas wrote: >VF BARs are read-only zero, so updating VF BARs will not have any effect. >See the SR-IOV spec r1.1, sec 3.4.1.11. > >We already ignore these updates because of 70675e0b6a1a ("PCI: Don't try to >restore VF BARs"); this merely restructu

Re: [PATCH kernel v6 3/7] vfio/spapr: Postpone allocation of userspace version of TCE table

2016-11-28 Thread David Gibson
On Thu, Nov 24, 2016 at 04:48:06PM +1100, Alexey Kardashevskiy wrote: > The iommu_table struct manages a hardware TCE table and a vmalloc'd > table with corresponding userspace addresses. Both are allocated when > the default DMA window is created and this happens when the very first > group is att

Re: [PATCH kernel v6 5/7] vfio/spapr: Postpone default window creation

2016-11-28 Thread David Gibson
On Thu, Nov 24, 2016 at 04:48:08PM +1100, Alexey Kardashevskiy wrote: > We are going to allow the userspace to configure container in > one memory context and pass container fd to another so > we are postponing memory allocations accounted against > the locked memory limit. One of previous patches

Re: [PATCH kernel v6 4/7] vfio/spapr: Add a helper to create default DMA window

2016-11-28 Thread David Gibson
On Thu, Nov 24, 2016 at 04:48:07PM +1100, Alexey Kardashevskiy wrote: > There is already a helper to create a DMA window which does allocate > a table and programs it to the IOMMU group. However > tce_iommu_take_ownership_ddw() did not use it and did these 2 calls > itself to simplify error path. >

[PATCH 4/4] powerpc: link warning for orphan sections

2016-11-28 Thread Nicholas Piggin
Add --orphan-handling=warn to final link flags. This ensures we can handle all sections explicitly. This would have caught subtle breakage such as 7de3b27bac47da9de08409df1d69664acbb72197 at build-time. Also bring some wayward sections into the fold: - .text.hot and .text.unlikely are compiler gen

[PATCH 3/4] powerpc/64: do not create new section for save/restore functions

2016-11-28 Thread Nicholas Piggin
There is no need to create a new section for these. Consolidate with 32-bit and just use .text. Signed-off-by: Nicholas Piggin --- arch/powerpc/lib/crtsavres.S | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/arch/powerpc/lib/crtsavres.S b/arch/powerpc/lib/crtsavres.S in

[PATCH 2/4] powerpc/64: do not link crtsavres routines into vmlinux

2016-11-28 Thread Nicholas Piggin
The 64-bit linker creates save/restore functions on demand. crtsavres.o is still required to link against modules (which do not get their save/restore created automatically, due to not being a final link). Make crtsavres.o extra-y on 64-bit, rather than obj-y. Signed-off-by: Nicholas Piggin ---

[PATCH 1/4] powerpc/64: place sfpr section explicitly with the linker script

2016-11-28 Thread Nicholas Piggin
Signed-off-by: Nicholas Piggin --- arch/powerpc/kernel/vmlinux.lds.S | 8 1 file changed, 8 insertions(+) diff --git a/arch/powerpc/kernel/vmlinux.lds.S b/arch/powerpc/kernel/vmlinux.lds.S index 861f4e1..f710d15 100644 --- a/arch/powerpc/kernel/vmlinux.lds.S +++ b/arch/powerpc/kernel/v

[PATCH 0/4] Try orphan section warning again

2016-11-28 Thread Nicholas Piggin
Hi, The orphan section warning patch had a minor issue with unhandled .text.save.restore last time I sent it. I went back and tidied up crtsavres a bit, so after these, I think we can enable orphan section warnings. There is a further step we can take which is to enable --save-restore-funcs for m

Re: [PATCH v3 3/4] powerpc/perf: PowerISA v3.0 raw event format encoding

2016-11-28 Thread Michael Ellerman
Madhavan Srinivasan writes: > Patch to update the PowerISA v3.0 raw event encoding format > information and add support for the same in Power9-pmu.c. I'm not sure if calling this the "PowerISA v3.0" encoding is right. It's not actually specified in the ISA anywhere right? Maybe it should just b

[PATCH v4 7/7] PCI: Add comments about ROM BAR updating

2016-11-28 Thread Bjorn Helgaas
pci_update_resource() updates a hardware BAR so its address matches the kernel's struct resource UNLESS it's a disabled ROM BAR. We only update those when we enable the ROM. It's not obvious from the code why ROM BARs should be handled specially. Apparently there are Matrox devices with defective

[PATCH v4 6/7] PCI: Decouple IORESOURCE_ROM_ENABLE and PCI_ROM_ADDRESS_ENABLE

2016-11-28 Thread Bjorn Helgaas
Remove the assumption that IORESOURCE_ROM_ENABLE == PCI_ROM_ADDRESS_ENABLE. PCI_ROM_ADDRESS_ENABLE is the ROM enable bit defined by the PCI spec, so if we're reading or writing a BAR register value, that's what we should use. IORESOURCE_ROM_ENABLE is a corresponding bit in struct resource flags. S

[PATCH v4 5/7] PCI: Remove pci_resource_bar() and pci_iov_resource_bar()

2016-11-28 Thread Bjorn Helgaas
pci_std_update_resource() only deals with standard BARs, so we don't have to worry about the complications of VF BARs in an SR-IOV capability. Compute the BAR address inline and remove pci_resource_bar(). That makes pci_iov_resource_bar() unused, so remove that as well. Signed-off-by: Bjorn Helg

[PATCH v4 4/7] PCI: Don't update VF BARs while VF memory space is enabled

2016-11-28 Thread Bjorn Helgaas
If we update a VF BAR while it's enabled, there are two potential problems: 1) Any driver that's using the VF has a cached BAR value that is stale after the update, and 2) We can't update 64-bit BARs atomically, so the intermediate state (new lower dword with old upper dword) may co

[PATCH v4 3/7] PCI: Separate VF BAR updates from standard BAR updates

2016-11-28 Thread Bjorn Helgaas
Previously pci_update_resource() used the same code path for updating standard BARs and VF BARs in SR-IOV capabilities. Split the VF BAR update into a new pci_iov_update_resource() internal interface, which makes it simpler to compute the BAR address (we can get rid of pci_resource_bar() and pci_i

[PATCH v4 2/7] PCI: Ignore BAR updates on virtual functions

2016-11-28 Thread Bjorn Helgaas
VF BARs are read-only zero, so updating VF BARs will not have any effect. See the SR-IOV spec r1.1, sec 3.4.1.11. We already ignore these updates because of 70675e0b6a1a ("PCI: Don't try to restore VF BARs"); this merely restructures it slightly to make it easier to split updates for standard and

[PATCH v4 1/7] PCI: Do any VF BAR updates before enabling the BARs

2016-11-28 Thread Bjorn Helgaas
From: Gavin Shan Previously we enabled VFs and enable their memory space before calling pcibios_sriov_enable(). But pcibios_sriov_enable() may update the VF BARs: for example, on PPC PowerNV we may change them to manage the association of VFs to PEs. Because 64-bit BARs cannot be updated atomic

[PATCH v4 0/7] Disable VF's memory space on updating IOV BARs

2016-11-28 Thread Bjorn Helgaas
This is a v4 of Gavin's series for handling VF BAR updates. The important piece is the first patch ("PCI: Do any VF BAR updates before enabling the BARs"). That makes sure that if we update VF BARs, we do it before enabling the VFs, and that is unchanged from v3. The second patch in Gavin's seri

Re: [PATCH v3 0/2] Disable VF's memory space on updating IOV BARs

2016-11-28 Thread Bjorn Helgaas
On Wed, Nov 23, 2016 at 05:49:10PM -0600, Bjorn Helgaas wrote: > On Wed, Oct 26, 2016 at 12:15:34PM +1100, Gavin Shan wrote: > > This moves pcibios_sriov_enable() to the point before VF and VF BARs > > are enabled on PowerNV platform. Also, pci_update_resource() is used > > to update IOV BARs on Po

Re: [PATCH] powerpc/boot: request no dynamic linker for boot wrapper

2016-11-28 Thread Michael Ellerman
Nicholas Piggin writes: > On Mon, 28 Nov 2016 22:07:39 +1100 > Michael Ellerman wrote: >> Nicholas Piggin writes: >> > diff --git a/arch/powerpc/boot/wrapper b/arch/powerpc/boot/wrapper >> > index 404b3aa..cd941a8 100755 >> > --- a/arch/powerpc/boot/wrapper >> > +++ b/arch/powerpc/boot/wrapper >

Re: [PATCH v2 2/4] usb: dwc2: Add binding for AHB burst

2016-11-28 Thread John Youn
On 11/22/2016 12:51 PM, Christian Lamparter wrote: > On Monday, November 21, 2016 7:32:30 PM CET John Youn wrote: >> On 11/21/2016 1:10 PM, Christian Lamparter wrote: >>> On Monday, November 21, 2016 12:16:31 PM CET John Youn wrote: On 11/18/2016 12:18 PM, Christian Lamparter wrote: > On F

[PATCH] EDAC: mpc85xx: Add T2080 l2-cache support

2016-11-28 Thread Chris Packham
The l2-cache controller on the T2080 SoC has similar capabilities to the others already supported by the mpc85xx_edac driver. Add it to the list of compatible devices. Signed-off-by: Chris Packham --- arch/powerpc/boot/dts/fsl/t2081si-post.dtsi | 1 + drivers/edac/mpc85xx_edac.c

Re: [PATCH] powerpc/mm: Fix lazy icache flush on pre-POWER5

2016-11-28 Thread Aneesh Kumar K.V
Benjamin Herrenschmidt writes: > On 64-bit CPUs with no-execute support and non-snooping icache, such as > 970 or POWER4, we have a software mechanism to ensure coherency of the > cache (using exec faults when needed). > > This was broken due to a logic inversion when that code was rewritten > fr

[PATCH] powerpc/mm: Fix lazy icache flush on pre-POWER5

2016-11-28 Thread Benjamin Herrenschmidt
On 64-bit CPUs with no-execute support and non-snooping icache, such as 970 or POWER4, we have a software mechanism to ensure coherency of the cache (using exec faults when needed). This was broken due to a logic inversion when that code was rewritten from assembly to C. Signed-off-by: Benjamin H

Re: [PATCH] powerpc/boot: request no dynamic linker for boot wrapper

2016-11-28 Thread Nicholas Piggin
On Mon, 28 Nov 2016 14:25:31 + Nick Clifton wrote: > Hi Nicholas, > > >> ... this actually seems like a better fix to me. If you do not want the > >> PT_INTERP segment, then telling this linker this is a good idea. So > >> wouldn't > >> a patch like this be a better solution to the probl

[PATCH v2 14/14] cxlflash: Migrate scsi command pointer to AFU command

2016-11-28 Thread Uma Krishnan
From: "Matthew R. Ochs" Currently, when sending a SCSI command, the pointer is stored in a reserved field of the AFU command descriptor for retrieval once the SCSI command has completed. In order to support new descriptor formats that make use of the reserved field, the pointer is migrated to out

[PATCH v2 13/14] cxlflash: Migrate IOARRIN specific routines to function pointers

2016-11-28 Thread Uma Krishnan
From: "Matthew R. Ochs" As staging for supporting hardware with a different queuing mechanism, move the send_cmd() and context_reset() routines to function pointers that are configured when the AFU is initialized. In addition, rename the existing routines to better reflect the queue model they su

[PATCH v2 12/14] cxlflash: Cleanup queuecommand()

2016-11-28 Thread Uma Krishnan
From: "Matthew R. Ochs" The queuecommand routine is disorganized where it populates the private command and also contains some logic/statements that are not needed given that cxlflash devices do not (and likely never will) support scatter-gather. Restructure the code to remove the unnecessary lo

[PATCH v2 11/14] cxlflash: Cleanup send_tmf()

2016-11-28 Thread Uma Krishnan
From: "Matthew R. Ochs" The send_tmf() routine includes some copy/paste cruft that can be removed as well as the setting of an AFU command-specific while holding the tmf_slock. While not a bug, it is out of place and should be shifted down alongside the other command initialization statements for

[PATCH v2 10/14] cxlflash: Remove AFU command lock

2016-11-28 Thread Uma Krishnan
From: "Matthew R. Ochs" The original design of the cxlflash driver required AFU commands to convey state information across multiple threads. The IOASA "host use" byte was used to track if a command was done, errored, or timed out. A per-command spin lock was used to serialize access to this byte

[PATCH v2 09/14] cxlflash: Wait for active AFU commands to timeout upon tear down

2016-11-28 Thread Uma Krishnan
From: "Matthew R. Ochs" With the removal of the static private command pool, the ability to 'complete' outstanding commands was lost. While not an issue for the commands originating outside the driver, internal AFU commands are synchronous and therefore have a timeout associated with them. To avo

[PATCH v2 08/14] cxlflash: Remove private command pool

2016-11-28 Thread Uma Krishnan
From: "Matthew R. Ochs" Clean up and remove the remaining private command pool infrastructure that is no longer required. Signed-off-by: Matthew R. Ochs --- drivers/scsi/cxlflash/common.h | 7 - drivers/scsi/cxlflash/main.c | 68 -- 2 files change

[PATCH v2 07/14] cxlflash: Use cmd_size for private commands

2016-11-28 Thread Uma Krishnan
From: "Matthew R. Ochs" Instead of using a private pool of AFU commands, use cmd_size to prime the private pool of SCSI commands such that they are allocated with a size large enough to contain an aligned AFU command. Use scsi_cmd_priv() to derive the aligned/zeroed private command on queuecomman

[PATCH v2 06/14] cxlflash: Allocate memory instead of using command pool for AFU sync

2016-11-28 Thread Uma Krishnan
From: "Matthew R. Ochs" As staging for the removal of the AFU command pool, remove the reliance upon the pool for the internal AFU sync command. Instead of obtaining an AFU command from the pool, dynamically allocate memory with the appropriate alignment requirements. Since the AFU sync service i

[PATCH v2 05/14] cxlflash: Remove unused buffer from AFU command

2016-11-28 Thread Uma Krishnan
From: "Matthew R. Ochs" The cxlflash driver originally required a per-command 4K buffer that hosted data passed to the AFU. When the routines that initiate AFU and internal SCSI commands were refactored to use scsi_execute(), the need for this buffer became obsolete. As it is no longer necessary,

Re: [mm v2 0/3] Support memory cgroup hotplug

2016-11-28 Thread Tejun Heo
Hello, Balbir. On Tue, Nov 29, 2016 at 11:09:26AM +1100, Balbir Singh wrote: > On 29/11/16 08:10, Tejun Heo wrote: > > On Thu, Nov 24, 2016 at 12:05:12AM +1100, Balbir Singh wrote: > >> On my desktop NODES_SHIFT is 6, many distro kernels have it a 9. I've known > >> of solutions that use fake NUMA

[PATCH v2 04/14] cxlflash: Avoid command room violation

2016-11-28 Thread Uma Krishnan
During test, a command room violation interrupt is occasionally seen for the master context when the CXL flash devices are stressed. After studying the code, there could be gaps in the way command room value is being cached in cxlflash. When the cached command room is zero the thread attempting to

[PATCH v2 03/14] cxlflash: Improve context_reset() logic

2016-11-28 Thread Uma Krishnan
Currently, the context reset routine waits for command room to be available before sending the reset request. Per review of the SISLite specification and clarifications from the CXL Flash AFU designers, this wait is unnecessary. The reset request can be sent anytime regardless of command room, so l

[PATCH v2 02/14] cxlflash: Fix crash in cxlflash_restore_luntable()

2016-11-28 Thread Uma Krishnan
During test, the following crash was observed: [34538.981505] Faulting instruction address: 0xd7c9c870 cpu 0x9: Vector: 300 (Data Access) at [c007f1e8f590] pc: d7c9c870: cxlflash_restore_luntable+0x70/0x1d0 [cxlflash] lr: d7c9c84c: cxlflash_restore_luntable+0x4c

[PATCH v2 01/14] cxlflash: Set sg_tablesize to 1 instead of SG_NONE

2016-11-28 Thread Uma Krishnan
The following Oops is encountered when blk_mq is enabled with the cxlflash driver: [ 2960.817172] Oops: Kernel access of bad area, sig: 11 [#5] [ 2960.817309] NIP __blk_mq_run_hw_queue+0x278/0x4c0 [ 2960.817313] LR __blk_mq_run_hw_queue+0x2bc/0x4c0 [ 2960.817314] Call Trace: [ 2960.817320] __blk_

[PATCH v2 00/14] cxlflash: Fixes, enhancements, cleanup and staging

2016-11-28 Thread Uma Krishnan
The first four patches in this patch series include fixes for command room violation and lun table management. The remaining patches remove the reliance upon an internally maintained private command pool in favor of private commands being allocated alongside the SCSI commands. Several cleanup oppo

Re: [mm v2 0/3] Support memory cgroup hotplug

2016-11-28 Thread Balbir Singh
On 29/11/16 08:10, Tejun Heo wrote: > On Thu, Nov 24, 2016 at 12:05:12AM +1100, Balbir Singh wrote: >> On my desktop NODES_SHIFT is 6, many distro kernels have it a 9. I've known >> of solutions that use fake NUMA for partitioning and need as many nodes as >> possible. > > It was a crude kludge

Re: [mm v2 0/3] Support memory cgroup hotplug

2016-11-28 Thread Tejun Heo
On Thu, Nov 24, 2016 at 12:05:12AM +1100, Balbir Singh wrote: > On my desktop NODES_SHIFT is 6, many distro kernels have it a 9. I've known > of solutions that use fake NUMA for partitioning and need as many nodes as > possible. It was a crude kludge that people used before memcg. If people still

[PATCH v2] adb: properly mark continued kernel messages

2016-11-28 Thread Andreas Schwab
Use pr_cont where appropriate, and switch to pr_foo throughout. Additionally, lower messages in adb_probe_task to debug level. Signed-off-by: Andreas Schwab --- drivers/macintosh/adb.c| 24 +--- drivers/macintosh/adbhid.c | 56 +++--- 2

Re: [PATCH net 11/16] net: ethernet: marvell: mvneta: fix fixed-link phydev leaks

2016-11-28 Thread Thomas Petazzoni
Hello, On Mon, 28 Nov 2016 19:25:04 +0100, Johan Hovold wrote: > Make sure to deregister and free any fixed-link PHY registered using > of_phy_register_fixed_link() on probe errors and on driver unbind. > > Fixes: 83895bedeee6 ("net: mvneta: add support for fixed links") > Signed-off-by: Johan Ho

[PATCH net 12/16] net: ethernet: mediatek: fix fixed-link phydev leaks

2016-11-28 Thread Johan Hovold
Make sure to deregister and free any fixed-link PHY registered using of_phy_register_fixed_link() on initialisation errors and on uninit. Fixes: 0c72c50f6f93 ("net-next: mediatek: add fixed-phy support") Signed-off-by: Johan Hovold --- drivers/net/ethernet/mediatek/mtk_eth_soc.c | 4 1 file

[PATCH] mail: improve bash completion

2016-11-28 Thread Greg Kurz
This adds completion of the --to, --cc and --bcc options based on the content of the [mail "alias"] section of GIT configuration files. Signed-off-by: Greg Kurz --- stgit/argparse.py |1 + stgit/commands/mail.py |6 +++--- stgit/completion.py|2 ++ 3 files changed, 6 inserti

[PATCH net 15/16] net: ethernet: ti: davinci_emac: fix fixed-link phydev and of-node leaks

2016-11-28 Thread Johan Hovold
Make sure to deregister and free any fixed-link PHY registered using of_phy_register_fixed_link() on probe errors and on driver unbind. Also remember to put the of-node reference on probe errors. Fixes: 1bb6aa56bb38 ("net: davinci_emac: Add support for fixed-link PHY") Signed-off-by: Johan Hovold

[PATCH net 16/16] net: dsa: slave: fix fixed-link phydev leaks

2016-11-28 Thread Johan Hovold
Make sure to deregister and free any fixed-link PHY registered using of_phy_register_fixed_link() on slave-setup errors and on slave destroy. Fixes: 0d8bcdd383b8 ("net: dsa: allow for more complex PHY setups") Signed-off-by: Johan Hovold --- net/dsa/slave.c | 12 +++- 1 file changed, 11

[PATCH net 14/16] net: ethernet: dwc_eth_qos: fix fixed-link phydev leaks

2016-11-28 Thread Johan Hovold
Make sure to deregister and free any fixed-link PHY registered using of_phy_register_fixed_link() on probe errors and on driver unbind. Fixes: 077742dac2c7 ("dwc_eth_qos: Add support for Synopsys DWC Ethernet QoS") Signed-off-by: Johan Hovold --- drivers/net/ethernet/synopsys/dwc_eth_qos.c | 20

[PATCH net 13/16] net: ethernet: renesas: ravb: fix fixed-link phydev leaks

2016-11-28 Thread Johan Hovold
Make sure to deregister and free any fixed-link PHY registered using of_phy_register_fixed_link() on initialisation errors and on device close after having disconnected the PHY. Fixes: b4bc88a868ed ("ravb: Add fixed-link support") Signed-off-by: Johan Hovold --- drivers/net/ethernet/renesas/ravb

[PATCH net 08/16] net: ethernet: fs_enet: fix fixed-link phydev leaks

2016-11-28 Thread Johan Hovold
Make sure to deregister and free any fixed-link PHY registered using of_phy_register_fixed_link() on probe errors and on driver unbind. Fixes: bb74d9a4a87b ("fs_enet: use the new fixed PHY helpers") Signed-off-by: Johan Hovold --- drivers/net/ethernet/freescale/fs_enet/fs_enet-main.c | 7 ++-

[PATCH net 10/16] net: ethernet: ucc_geth: fix fixed-link phydev leaks

2016-11-28 Thread Johan Hovold
Make sure to deregister and free any fixed-link PHY registered using of_phy_register_fixed_link() on probe errors and on driver unbind. Fixes: 87009814cdbb ("ucc_geth: use the new fixed PHY helpers") Signed-off-by: Johan Hovold --- drivers/net/ethernet/freescale/ucc_geth.c | 23 -

[PATCH net 07/16] net: ethernet: fec: fix fixed-link phydev leaks

2016-11-28 Thread Johan Hovold
Make sure to deregister and free any fixed-link PHY registered using of_phy_register_fixed_link() on probe errors and on driver unbind. Fixes: 407066f8f371 ("net: fec: Support phys probed from devicetree and fixed-link") Signed-off-by: Johan Hovold --- drivers/net/ethernet/freescale/fec_main.c |

[PATCH net 11/16] net: ethernet: marvell: mvneta: fix fixed-link phydev leaks

2016-11-28 Thread Johan Hovold
Make sure to deregister and free any fixed-link PHY registered using of_phy_register_fixed_link() on probe errors and on driver unbind. Fixes: 83895bedeee6 ("net: mvneta: add support for fixed links") Signed-off-by: Johan Hovold --- drivers/net/ethernet/marvell/mvneta.c | 5 + 1 file changed

[PATCH net 06/16] net: ethernet: bcmgenet: fix fixed-link phydev leaks

2016-11-28 Thread Johan Hovold
Make sure to deregister and free any fixed-link PHY registered using of_phy_register_fixed_link() on probe errors and on driver unbind. Note that we're still leaking any fixed-link PHY registered in the non-OF probe path. Fixes: 9abf0c2b717a ("net: bcmgenet: use the new fixed PHY helpers") Signed

[PATCH net 09/16] net: ethernet: gianfar: fix fixed-link phydev leaks

2016-11-28 Thread Johan Hovold
Make sure to deregister and free any fixed-link PHY registered using of_phy_register_fixed_link() on probe errors and on driver unbind. Fixes: be40364544bd ("gianfar: use the new fixed PHY helpers") Signed-off-by: Johan Hovold --- drivers/net/ethernet/freescale/gianfar.c | 8 1 file cha

[PATCH net 05/16] net: ethernet: bcmsysport: fix fixed-link phydev leaks

2016-11-28 Thread Johan Hovold
Make sure to deregister and free any fixed-link PHY registered using of_phy_register_fixed_link() on probe errors and on driver unbind. Fixes: 186534a3f832 ("net: systemport: use the new fixed PHY helpers") Signed-off-by: Johan Hovold --- drivers/net/ethernet/broadcom/bcmsysport.c | 17 +

[PATCH net 04/16] net: ethernet: aurora: nb8800: fix fixed-link phydev leaks

2016-11-28 Thread Johan Hovold
Make sure to deregister and free any fixed-link PHY registered using of_phy_register_fixed_link() on probe errors and on driver unbind. Fixes: c7dfe3abf40e ("net: ethernet: nb8800: support fixed-link DT node") Signed-off-by: Johan Hovold --- drivers/net/ethernet/aurora/nb8800.c | 9 +++-- 1

[PATCH net 01/16] net: dsa: slave: fix of-node leak and phy priority

2016-11-28 Thread Johan Hovold
Make sure to drop the reference taken by of_parse_phandle() before returning from dsa_slave_phy_setup(). Note that this also modifies the PHY priority so that any fixed-link node is only parsed when no phy-handle is given, which is in accordance with the common scheme for this. Fixes: 0d8bcdd383b

[PATCH net 02/16] of_mdio: add helper to deregister fixed-link PHYs

2016-11-28 Thread Johan Hovold
Add helper to deregister fixed-link PHYs registered using of_phy_register_fixed_link(). Convert the two drivers that care to deregister their fixed-link PHYs to use the new helper, but note that most drivers currently fail to do so. Signed-off-by: Johan Hovold --- drivers/net/ethernet/ti/cpsw.c

[PATCH net 00/16] net: fix fixed-link phydev leaks

2016-11-28 Thread Johan Hovold
This series fixes failures to deregister and free fixed-link phydevs that have been registered using the of_phy_register_fixed_link() interface. All but two drivers currently fail to do this and this series fixes most of them with the exception of a staging driver and the stmmac drivers which will

[PATCH net 03/16] net: ethernet: altera: fix fixed-link phydev leaks

2016-11-28 Thread Johan Hovold
Make sure to deregister and free any fixed-link PHY registered using of_phy_register_fixed_link() on probe errors and on driver unbind. Fixes: 7cdbc6f74f8e ("altera tse: add support for fixed-links.") Signed-off-by: Johan Hovold --- drivers/net/ethernet/altera/altera_tse_main.c | 9 - 1

Re: [PATCH] powerpc/boot: request no dynamic linker for boot wrapper

2016-11-28 Thread Nick Clifton
Hi Nicholas, >> ... this actually seems like a better fix to me. If you do not want the >> PT_INTERP segment, then telling this linker this is a good idea. So wouldn't >> a patch like this be a better solution to the problem ? > > Yes, I wasn't asking for the binutils change to be reverted. O

Re: [PATCH] powerpc/boot: request no dynamic linker for boot wrapper

2016-11-28 Thread Nick Clifton
Hi Nicholas, >>> The boot wrapper performs its own relocations and does not require >>> PT_INTERP segment. >>> >>> Without this option, binutils 2.28 and newer tries to create a program >>> header segment due to PT_INTERP, and the link fails because there is no >>> space for it. >> >> 2.28 is no

[PATCH] powerpc/pserie: Use lmb_is_removable() to check removability

2016-11-28 Thread Nathan Fontenot
We should be using lmb_is_removable() to validate that enough LMBs are available to remove when doing a remove by count. This will check that the LMB is owned by the system and it is considered removable. This patch also adds a pr_info() notification to report the LMB count to remove was not satisf

Re: [RFC][PATCH] Update ppc disassembly in xmon

2016-11-28 Thread Peter Bergner
On 11/23/16 11:52 PM, Balbir Singh wrote: On 24/11/16 16:14, Andrew Donnellan wrote: On 24/11/16 13:05, Balbir Singh wrote: 9. The license for these files is now GPL v3 or later As much as I love the GPLv3, isn't this an instant NAK? Thats why I called it out, my bad though I should have

Re: [PATCH] cpuidle/powernv: staticise powernv_idle_driver

2016-11-28 Thread Rafael J. Wysocki
On Tue, Nov 22, 2016 at 6:08 AM, Andrew Donnellan wrote: > powernv_idle_driver isn't exported, it can be made static. Found by sparse. > > Signed-off-by: Andrew Donnellan Applied. Thanks, Rafael

Re: [PATCH] powerpc/boot: request no dynamic linker for boot wrapper

2016-11-28 Thread Nicholas Piggin
On Mon, 28 Nov 2016 12:39:38 + Nick Clifton wrote: > Hi Nicholas, > > >>> The boot wrapper performs its own relocations and does not require > >>> PT_INTERP segment. > >>> > >>> Without this option, binutils 2.28 and newer tries to create a program > >>> header segment due to PT_INTERP, and

Re: powerpc/eeh/of: use builtin_platform_driver

2016-11-28 Thread Michael Ellerman
On Wed, 2016-11-23 at 14:58:56 UTC, Geliang Tang wrote: > Use builtin_platform_driver() helper to simplify the code. > > Signed-off-by: Geliang Tang > Acked-by: Russell Currey Applied to powerpc next, thanks. https://git.kernel.org/powerpc/c/ebb242d56bbe14af6ea25cf5e45440 cheers

Re: powerpc: Fix __cmpxchg() to take a volatile ptr again

2016-11-28 Thread Michael Ellerman
On Fri, 2016-11-25 at 03:07:09 UTC, Michael Ellerman wrote: > In commit d0563a1297e2 ("powerpc: Implement {cmp}xchg for u8 and u16") > we removed the volatile from __cmpxchg(). > > This is leading to warnings such as: > > drivers/gpu/drm/drm_lock.c: In function ‘drm_lock_take’: > arch/pow

Re: [powerpc,v6,1/3] Setup AMOR in HV mode

2016-11-28 Thread Michael Ellerman
On Tue, 2016-11-15 at 06:56:14 UTC, Balbir Singh wrote: > AMOR should be setup in HV mode, we set it up once > and let the generic kernel handle IAMR. This patch is > used to enable storage keys in a following patch as > defined in ISA 3. We don't setup AMOR in DD1, since we > can't setup IAMR in D

Re: powernv: Clear SPRN_PSSCR when a POWER9 CPU comes online

2016-11-28 Thread Michael Ellerman
On Tue, 2016-11-22 at 18:06:40 UTC, "Gautham R. Shenoy" wrote: > From: "Gautham R. Shenoy" > > Ensure that PSSCR is set to a safe value corresponding to no > state-loss each time a POWER9 CPU comes online. > > Signed-off-by: Gautham R. Shenoy > Acked-By: Michael Neuling Applied to powerpc nex

Re: cxl: drop duplicate header sched.h

2016-11-28 Thread Michael Ellerman
On Wed, 2016-11-23 at 15:27:38 UTC, Geliang Tang wrote: > Drop duplicate header sched.h from native.c. > > Signed-off-by: Geliang Tang > Reviewed-by: Andrew Donnellan Applied to powerpc next, thanks. https://git.kernel.org/powerpc/c/7184bc2ddb15d9539c701668d6a5be cheers

Re: powerpc/xmon: Add 'dt' command to dump trace buffers

2016-11-28 Thread Michael Ellerman
On Fri, 2015-11-06 at 02:21:17 UTC, Michael Ellerman wrote: > There is a nice interface for asking ftrace to dump all its tracing > buffers. The only down side for use in xmon is that it uses printk. > Depending on circumstances printk may not work when in xmon, but it also > may, so add a 'dt' com

Re: powerpc/mm: Fixup kernel read only mapping

2016-11-28 Thread Michael Ellerman
On Thu, 2016-11-24 at 09:39:54 UTC, "Aneesh Kumar K.V" wrote: > With commit e58e87adc8bf9 ("powerpc/mm: Update _PAGE_KERNEL_RO") we started > using the ppp value 0b110 to map kernel readonly. But that facility > was only added as part of ISA 2.04. For earlier ISA version only supported ppp > bit va

Re: [PATCH] powerpc/boot: request no dynamic linker for boot wrapper

2016-11-28 Thread Nicholas Piggin
On Mon, 28 Nov 2016 22:07:39 +1100 Michael Ellerman wrote: > Nicholas Piggin writes: > > > The boot wrapper performs its own relocations and does not require > > PT_INTERP segment. > > > > Without this option, binutils 2.28 and newer tries to create a program > > header segment due to PT_INTERP

Re: [PATCH v7 4/7] powerpc/mm: Add radix__tlb_flush_pte

2016-11-28 Thread Michael Ellerman
"Aneesh Kumar K.V" writes: > diff --git a/arch/powerpc/mm/tlb-radix.c b/arch/powerpc/mm/tlb-radix.c > index 3493cf4e0452..7648952e4f08 100644 > --- a/arch/powerpc/mm/tlb-radix.c > +++ b/arch/powerpc/mm/tlb-radix.c > @@ -428,3 +428,21 @@ void radix__flush_tlb_all(void) >: : "r"

Re: [PATCH] powerpc/boot: request no dynamic linker for boot wrapper

2016-11-28 Thread Michael Ellerman
Nicholas Piggin writes: > The boot wrapper performs its own relocations and does not require > PT_INTERP segment. > > Without this option, binutils 2.28 and newer tries to create a program > header segment due to PT_INTERP, and the link fails because there is no > space for it. 2.28 is not relea

Re: [PATCH kernel v6 5/7] vfio/spapr: Postpone default window creation

2016-11-28 Thread Alexey Kardashevskiy
On 25/11/16 22:37, David Gibson wrote: > On Fri, Nov 25, 2016 at 05:38:26PM +1100, Alexey Kardashevskiy wrote: >> On 25/11/16 15:39, David Gibson wrote: >>> On Thu, Nov 24, 2016 at 04:48:08PM +1100, Alexey Kardashevskiy wrote: We are going to allow the userspace to configure container in

Re: [GIT PULL] Please pull powerpc/linux.git powerpc-4.9-6 tag

2016-11-28 Thread Michael Ellerman
Linus Torvalds writes: > On Sat, Nov 26, 2016 at 12:11 AM, Michael Ellerman > wrote: >> >> powerpc fixes for 4.9 #6 >> >> Fixes marked for stable: >> - Set missing wakeup bit in LPCR on POWER9 (Benjamin Herrenschmidt) >> - Fix the early OPAL console wrappers (Oliver O'Halloran) >> - Fixup ke

Re: [PATCH] crypto: vmx - rebuild generated asm when target changes

2016-11-28 Thread Nicholas Piggin
On Mon, 28 Nov 2016 13:51:36 +0530 "Naveen N. Rao" wrote: > On 2016/11/26 03:24PM, Nicholas Piggin wrote: > > Switching from big endian to little endian can fail to regenerate > > the crypto assembly properly. Switch to using standard form of > > kbuild dependency checking (i.e., use FORCE and if

Re: [PATCH] crypto: vmx - rebuild generated asm when target changes

2016-11-28 Thread Naveen N. Rao
On 2016/11/26 03:24PM, Nicholas Piggin wrote: > Switching from big endian to little endian can fail to regenerate > the crypto assembly properly. Switch to using standard form of > kbuild dependency checking (i.e., use FORCE and if_changed). > > Signed-off-by: Nicholas Piggin Hi Nick, A similar