Re: [PATCH v3 2/2] powerpc/8xx: use _PAGE_RO instead of _PAGE_RW

2015-01-05 Thread leroy christophe
Le 05/01/2015 19:12, Joakim Tjernlund a écrit : On Mon, 2014-12-22 at 11:14 +0100, Christophe Leroy wrote: On powerpc 8xx, in TLB entries, 0x400 bit is set to 1 for read-only pages and is set to 0 for RW pages. So we should use _PAGE_RO instead of _PAGE_RW Signed-off-by: Christophe Leroy

Re: [PATCH 09/11] powerpc/8xx: dont save CR in SCRATCH registers

2015-01-05 Thread leroy christophe
Le 05/01/2015 19:30, Joakim Tjernlund a écrit : On Tue, 2014-12-16 at 16:03 +0100, Christophe Leroy wrote: CR only needs to be preserved when checking if we are handling a kernel address. So we can preserve CR in a register: - In ITLBMiss, check is done only when CONFIG_MODULES is defined.

Re: [PATCH 2/2] powerpc/config: enable mdio buses muxing via Memory-mapped device

2015-01-05 Thread Emil Medve
Hello Shao-Hui, On 01/03/2015 10:58 PM, Xie Shaohui-B21989 wrote: Hello Emil, The patches sent by Shruti were marked as Changes Requested over 5 months. to me if the patches do need changes, the possible changes seems not in 'config' but in corenet_generic.c. It was part of a series that

[PATCH 03/10] mm: Convert p[te|md]_numa users to p[te|md]_protnone_numa

2015-01-05 Thread Mel Gorman
Convert existing users of pte_numa and friends to the new helper. Note that the kernel is broken after this patch is applied until the other page table modifiers are also altered. This patch layout is to make review easier. Signed-off-by: Mel Gorman mgor...@suse.de Acked-by: Linus Torvalds

[PATCH 01/10] mm: numa: Do not dereference pmd outside of the lock during NUMA hinting fault

2015-01-05 Thread Mel Gorman
A transhuge NUMA hinting fault may find the page is migrating and should wait until migration completes. The check is race-prone because the pmd is deferenced outside of the page lock and while the race is tiny, it'll be larger if the PMD is cleared while marking PMDs for hinting fault. This patch

[PATCH 02/10] mm: Add p[te|md] protnone helpers for use by NUMA balancing

2015-01-05 Thread Mel Gorman
This is a preparatory patch that introduces protnone helpers for automatic NUMA balancing. Signed-off-by: Mel Gorman mgor...@suse.de Acked-by: Linus Torvalds torva...@linux-foundation.org Acked-by: Aneesh Kumar K.V aneesh.ku...@linux.vnet.ibm.com Tested-by: Sasha Levin sasha.le...@oracle.com ---

[PATCH 04/10] ppc64: Add paranoid warnings for unexpected DSISR_PROTFAULT

2015-01-05 Thread Mel Gorman
ppc64 should not be depending on DSISR_PROTFAULT and it's unexpected if they are triggered. This patch adds warnings just in case they are being accidentally depended upon. Signed-off-by: Mel Gorman mgor...@suse.de Acked-by: Aneesh Kumar K.V aneesh.ku...@linux.vnet.ibm.com Tested-by: Sasha Levin

[PATCH 0/10] Replace _PAGE_NUMA with PAGE_NONE protections v5

2015-01-05 Thread Mel Gorman
Changelog since V4 o Rebase to 3.19-rc2(mel) Changelog since V3 o Minor comment update (benh) o Add ack'ed bys Changelog since V2 o Rename *_protnone_numa to _protnone and extend docs (linus) o Rebase

[PATCH 06/10] mm: Remove remaining references to NUMA hinting bits and helpers

2015-01-05 Thread Mel Gorman
This patch removes the NUMA PTE bits and associated helpers. As a side-effect it increases the maximum possible swap space on x86-64. One potential source of problems is races between the marking of PTEs PROT_NONE, NUMA hinting faults and migration. It must be guaranteed that a PTE being

[PATCH 05/10] mm: Convert p[te|md]_mknonnuma and remaining page table manipulations

2015-01-05 Thread Mel Gorman
With PROT_NONE, the traditional page table manipulation functions are sufficient. Signed-off-by: Mel Gorman mgor...@suse.de Acked-by: Linus Torvalds torva...@linux-foundation.org Acked-by: Aneesh Kumar aneesh.ku...@linux.vnet.ibm.com Tested-by: Sasha Levin sasha.le...@oracle.com ---

[PATCH 07/10] mm: numa: Do not trap faults on the huge zero page

2015-01-05 Thread Mel Gorman
Faults on the huge zero page are pointless and there is a BUG_ON to catch them during fault time. This patch reintroduces a check that avoids marking the zero page PAGE_NONE. Signed-off-by: Mel Gorman mgor...@suse.de --- include/linux/huge_mm.h | 3 ++- mm/huge_memory.c| 13

[PATCH 08/10] x86: mm: Restore original pte_special check

2015-01-05 Thread Mel Gorman
Commit b38af4721f59 (x86,mm: fix pte_special versus pte_numa) adjusted the pte_special check to take into account that a special pte had SPECIAL and neither PRESENT nor PROTNONE. Now that NUMA hinting PTEs are no longer modifying _PAGE_PRESENT it should be safe to restore the original pte_special

RE: [PATCH 2/2] powerpc/config: enable mdio buses muxing via Memory-mapped device

2015-01-05 Thread Shaohui Xie
-Original Message- From: Emil Medve [mailto:emilian.me...@freescale.com] Sent: Monday, January 05, 2015 5:29 PM To: Xie Shaohui-B21989; linuxppc-dev@lists.ozlabs.org; Wood Scott-B07421 Cc: Andy Fleming; Kanetkar Shruti-B44454 Subject: Re: [PATCH 2/2] powerpc/config: enable mdio buses

[PATCH 10/10] mm: numa: Avoid unnecessary TLB flushes when setting NUMA hinting entries

2015-01-05 Thread Mel Gorman
If a PTE or PMD is already marked NUMA when scanning to mark entries for NUMA hinting then it is not necessary to update the entry and incur a TLB flush penalty. Avoid the avoidhead where possible. Signed-off-by: Mel Gorman mgor...@suse.de --- mm/huge_memory.c | 14 -- mm/mprotect.c

[PATCH 09/10] mm: numa: Add paranoid check around pte_protnone_numa

2015-01-05 Thread Mel Gorman
pte_protnone_numa is only safe to use after VMA checks for PROT_NONE are complete. Treating a real PROT_NONE PTE as a NUMA hinting fault is going to result in strangeness so add a check for it. BUG_ON looks like overkill but if this is hit then it's a serious bug that could result in corruption so

RE: [PATCH] [v3] power/fsl: add MDIO dt binding for FMan

2015-01-05 Thread Shaohui Xie
Hello Scott, Is this v3 OK? Best Regards, Shaohui Xie -Original Message- From: shh@gmail.com [mailto:shh@gmail.com] Sent: Tuesday, December 23, 2014 3:58 PM To: linuxppc-dev@lists.ozlabs.org; devicet...@vger.kernel.org; Wood Scott-B07421 Cc: Medve Emilian-EMMEDVE1; Xie

[PATCH V2] tty: 8250: Add 64byte UART support for FSL platforms

2015-01-05 Thread Vijay Rai
Some of FSL SoCs like T1040 has new version of UART controller which can support 64byte FiFo. To enable 64 byte support, following needs to be done: -FCR[EN64] needs to be programmed to 1 to enable it. -Also, when FCR[EN64]==1, RTL bits to be used as below to define various Receive Trigger Levels:

Re: [PATCH 11/13] ALSA: Deletion of checks before the function call iounmap

2015-01-05 Thread Dan Carpenter
On Sun, Jan 04, 2015 at 02:36:01PM +0100, SF Markus Elfring wrote: /* unmap PCI memory space, mapped during device init. */ - for (idx = 0; idx HPI_MAX_ADAPTER_MEM_SPACES; idx++) { - if (pci.ap_mem_base[idx]) - iounmap(pci.ap_mem_base[idx]); - }

[PATCH] powerpc/8xx: reduce pressure on TLB due to context switches

2015-01-05 Thread Christophe Leroy
For nohash powerpc, when we run out of contexts, contexts are freed by stealing used contexts in-turn. When a victim has been selected, the associated TLB entries are freed using _tlbil_pid(). Unfortunatly, on the PPC 8xx, _tlbil_pid() does a tlbia, hence flushes ALL TLB entries and not only the

Re: PROBLEM: USB isochronous urb leak on EHCI driver

2015-01-05 Thread Fabio Estevam
On Mon, Jan 5, 2015 at 1:12 PM, Michael Tessier michael.tess...@axiontech.ca wrote: If sound is ok when using only 1 codec and becomes choppy when adding a second codec, then it means that this issue is still in the 3.x kernel. This answer will tell me if it is worth working on using a newer

[RFC] PPC: MPIC: necessary readback after EOI?

2015-01-05 Thread Purcareata Bogdan
Hello, While doing some performance testing of a KVM guest on a PPC platform, I noticed that there's a read of the CPU_WHOAMI register after each MPIC EOI [1]. This has been present since the initial implementation of the MPIC driver [2]. In a KVM virtualized environment, this results in an

Re: [RFC] PPC: MPIC: necessary readback after EOI?

2015-01-05 Thread Andreas Mohr
Hi, I was curious why the mpic_cpu_read(MPIC_INFO(CPU_WHOAMI)) was there in the first place and if it's still needed. If it's still required, I guess a better approach is to eliminate the call only if the kernel is running on the KVM guest side, where the MPIC is emulated and no longer

Re: [RFC] PPC: MPIC: necessary readback after EOI?

2015-01-05 Thread Scott Wood
On Mon, 2015-01-05 at 18:46 +0100, Andreas Mohr wrote: Hi, I was curious why the mpic_cpu_read(MPIC_INFO(CPU_WHOAMI)) was there in the first place and if it's still needed. If it's still required, I guess a better approach is to eliminate the call only if the kernel is running on the

Re: [PATCH 04/11] powerpc/8xx: Take benefit of aligned PGDIR

2015-01-05 Thread Joakim Tjernlund
On Tue, 2014-12-16 at 16:03 +0100, Christophe Leroy wrote: L1 base address is now aligned so we can insert L1 index into r11 directly and then preserve r10 Signed-off-by: Christophe Leroy christophe.le...@c-s.fr Acked-by: Joakim Tjernlund joakim.tjernl...@transmode.se ---

Re: [PATCH v3 2/2] powerpc/8xx: use _PAGE_RO instead of _PAGE_RW

2015-01-05 Thread Joakim Tjernlund
On Mon, 2014-12-22 at 11:14 +0100, Christophe Leroy wrote: On powerpc 8xx, in TLB entries, 0x400 bit is set to 1 for read-only pages and is set to 0 for RW pages. So we should use _PAGE_RO instead of _PAGE_RW Signed-off-by: Christophe Leroy christophe.le...@c-s.fr Hi Christophe, been

Re: [PATCH 02/11] powerpc/8xx: remove tests on PGDIR entry validity

2015-01-05 Thread Joakim Tjernlund
On Tue, 2014-12-16 at 16:03 +0100, Christophe Leroy wrote: Kernel MMU handling code handles validity of entries via _PMD_PRESENT which corresponds to V bit in MD_TWC and MI_TWC. When the V bit is not set, MPC8xx triggers TLBError exception. So we don't have to check that and branch ourself to

Re: [PATCH 11/11] powerpc/8xx: Add support for TASK_SIZE greater than 0x80000000

2015-01-05 Thread Joakim Tjernlund
On Tue, 2014-12-16 at 16:03 +0100, Christophe Leroy wrote: By default, TASK_SIZE is set to 0x8000 for PPC_8xx, which is most likely sufficient for most cases. However, kernel configuration allows to set TASK_SIZE to another value, so the 8xx shall handle it. On 8xx I would be just as

Re: [PATCH 09/11] powerpc/8xx: dont save CR in SCRATCH registers

2015-01-05 Thread Joakim Tjernlund
On Tue, 2014-12-16 at 16:03 +0100, Christophe Leroy wrote: CR only needs to be preserved when checking if we are handling a kernel address. So we can preserve CR in a register: - In ITLBMiss, check is done only when CONFIG_MODULES is defined. Otherwise we don't need to do anything at all

Re: [PATCH 05/11] powerpc/8xx: Optimise access to swapper_pg_dir

2015-01-05 Thread Joakim Tjernlund
On Tue, 2014-12-16 at 16:03 +0100, Christophe Leroy wrote: All accessed to PGD entries are done via 0(r11). By using lower part of swapper_pg_dir as load index to r11, we can remove the ori instruction. Signed-off-by: Christophe Leroy christophe.le...@c-s.fr Nice :) Acked-by: Joakim

Re: [RFC] PPC: MPIC: necessary readback after EOI?

2015-01-05 Thread Andreas Mohr
[CC related ppl] On Mon, Jan 05, 2015 at 12:10:54PM -0600, Scott Wood wrote: On Mon, 2015-01-05 at 18:46 +0100, Andreas Mohr wrote: Hi, I was curious why the mpic_cpu_read(MPIC_INFO(CPU_WHOAMI)) was there in the first place and if it's still needed. If it's still required, I guess

RE: PROBLEM: USB isochronous urb leak on EHCI driver

2015-01-05 Thread Michael Tessier
On Mon, 15 Dec 2014, Michael Tessier wrote: Hi, I am dealing with a USB EHCI driver bug. Here is the info: My configuration: - Host: Freescale i.MX512 with ARM Cortex A8 (USB 2.0 host controller) Linux kernel: 2.6.31, using EHCI USB driver As mentioned by

[PATCH] powerpc/8xx: reduce pressure on TLB due to context switches

2015-01-05 Thread Christophe Leroy
For nohash powerpc, when we run out of contexts, contexts are freed by stealing used contexts in-turn. When a victim has been selected, the associated TLB entries are freed using _tlbil_pid(). Unfortunatly, on the PPC 8xx, _tlbil_pid() does a tlbia, hence flushes ALL TLB entries and not only the

RE: PROBLEM: USB isochronous urb leak on EHCI driver

2015-01-05 Thread Alan Stern
On Mon, 5 Jan 2015, Michael Tessier wrote: Hi, I am dealing with a USB EHCI driver bug. Here is the info: My configuration: - Host: Freescale i.MX512 with ARM Cortex A8 (USB 2.0 host controller) Linux kernel: 2.6.31, using EHCI USB driver As