[PATCH 3/3] powerpc: ps3: Add missing set_freezable() for ps3_probe_thread()

The kernel thread function ps3_probe_thread() invokes the try_to_freeze() in its loop. But all the kernel threads are non-freezable by default. So if we want to make a kernel thread to be freezable, we have to invoke set_freezable() explicitly. Signed-off-by: Kevin Hao --- arch/powerpc

[PATCH 2/3] powerpc: mpc83xx: Use wait_event_freezable() for freezable kthread

wait_event_freezable() and then eliminate a function call. Signed-off-by: Kevin Hao --- arch/powerpc/platforms/83xx/suspend.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/arch/powerpc/platforms/83xx/suspend.c b/arch/powerpc/platforms/83xx/suspend.c index eed325ed08cc

[PATCH 1/3] powerpc: mpc83xx: Add the missing set_freezable() for agent_thread_fn()

The kernel thread function agent_thread_fn() invokes the try_to_freeze() in its loop. But all the kernel threads are non-freezable by default. So if we want to make a kernel thread to be freezable, we have to invoke set_freezable() explicitly. Signed-off-by: Kevin Hao --- arch/powerpc/platforms

[PATCH 0/3] powerpc: Fixes and optimization for the freezable kthread

Hi, The main changes include: - Invoke set_freezable() for the kthread which could be frozen - Drop redundant try_to_freeze() invocation Kevin Hao (3): powerpc: mpc83xx: Add the missing set_freezable() for agent_thread_fn() powerpc: mpc83xx: Use wait_event_freezable() for freezable

[PATCH] powerpc: Export mmu_feature_keys[] as non-GPL

ature_keys[] to the non-GPL modules too. Reported-by: Nathaniel Filardo Signed-off-by: Kevin Hao --- arch/powerpc/kernel/cputable.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/powerpc/kernel/cputable.c b/arch/powerpc/kernel/cputable.c index ae0fdef0ac11..3a8cd40b6

Re: [PATCH 3/5] of/platform: introduce a generic way to declare a platform bus

On Thu, Aug 25, 2016 at 08:44:56AM -0500, Rob Herring wrote: > On Mon, Aug 22, 2016 at 9:06 PM, Kevin Hao <haoke...@gmail.com> wrote: > > The specific buses which need to be probed at boot time are different > > between platforms. Instead of put all the b

[PATCH 5/5] powerpc/83xx: enable the default of probe

Use the default of probe function of_platform_default_populate_init() to kill the arch specific duplicated codes. Signed-off-by: Kevin Hao <haoke...@gmail.com> --- arch/powerpc/platforms/83xx/asp834x.c | 2 -- arch/powerpc/platforms/83xx/km83xx.c | 2 -- arch/powerpc/platform

[PATCH 4/5] powerpc/83xx: factor out the common codes of setup arch functions

function even there is no such invocation in its original setup arch function. Signed-off-by: Kevin Hao <haoke...@gmail.com> --- arch/powerpc/platforms/83xx/asp834x.c | 4 +--- arch/powerpc/platforms/83xx/km83xx.c | 5 + arch/powerpc/platforms/83xx/misc.c| 8 arch/p

[PATCH 3/5] of/platform: introduce a generic way to declare a platform bus

The specific buses which need to be probed at boot time are different between platforms. Instead of put all the buses into the default of_default_bus_match_table[] match tables, this patch introduces a general way to declare a platform bus. Signed-off-by: Kevin Hao <haoke...@gmail.

[PATCH 2/5] powerpc: introduce arch_enable_default_of_probe()

We can use this function to enable the default of probe for one board. Signed-off-by: Kevin Hao <haoke...@gmail.com> --- arch/powerpc/include/asm/setup.h | 1 + arch/powerpc/kernel/of_platform.c | 9 - 2 files changed, 9 insertions(+), 1 deletion(-) diff --git a/arch/powerpc/i

[PATCH 1/5] of/platform: introduce arch_want_default_of_probe()

k function arch_want_default_of_probe(), we can override it in arch specific code to enable/disable the default of probe per board. Signed-off-by: Kevin Hao <haoke...@gmail.com> --- arch/powerpc/kernel/of_platform.c | 5 + drivers/of/platform.c | 9 ++--- include/linux/of_platform.

[PATCH 0/5] ppc32: use the default of_platform_default_populate_init() for 83xx boards

this method for the of probe. [1] https://patchwork.ozlabs.org/patch/658531/ Kevin Hao (5): of/platform: introduce arch_want_default_of_probe() powerpc: introduce arch_enable_default_of_probe() of/platform: introduce a generic way to declare a platform bus powerpc/83xx: factor out the common

Re: [PATCH] powerpc: populate the default bus with machine_arch_initcall

On Fri, Aug 12, 2016 at 02:39:32PM +1000, Michael Ellerman wrote: > Kevin Hao <haoke...@gmail.com> writes: > > > With the commit 44a7185c2ae6 ("of/platform: Add common method to > > populate default bus"), a default function is introduced to populate >

Re: [PATCH] powerpc: populate the default bus with machine_arch_initcall

On Thu, Aug 11, 2016 at 08:17:52AM -0500, Rob Herring wrote: > On Thu, Aug 11, 2016 at 6:09 AM, Kevin Hao <haoke...@gmail.com> wrote: > > With the commit 44a7185c2ae6 ("of/platform: Add common method to > > populate default bus"), a default function is introduced

[PATCH] powerpc: populate the default bus with machine_arch_initcall

at a lower level than arch_initcall_sync. Since not all powerpc specific buses are added to the of_default_bus_match_table[], this causes some powerpc specific bus are not probed. Fix this by using a more preceding initcall. Signed-off-by: Kevin Hao <haoke...@gmail.com> --- Of course we can ad

Re: [PATCH for-4.8 V2 08/10] powerpc: use the jump label for cpu_has_feature

On Mon, Jul 25, 2016 at 04:28:49PM +1000, Nicholas Piggin wrote: > On Sat, 23 Jul 2016 14:42:41 +0530 > "Aneesh Kumar K.V" <aneesh.ku...@linux.vnet.ibm.com> wrote: > > > From: Kevin Hao <haoke...@gmail.com> > > > > The cpu features

Re: [PATCH v2 0/3] powerpc: tweak the kernel options for CRASH_DUMP and RELOCATABLE

Please ignore this. Forgot the CC list. Thanks, Kevin On Wed, Jul 13, 2016 at 09:12:53AM +0800, Kevin Hao wrote: > v2: > Compare to v1 [1], the main differences are: > - Still automatically select RELOCATABLE when CRASH_DUMP is enabled. > - Add a new patch to kill RELOCATABLE_PP

[PATCH v2 3/3] powerpc32: kill RELOCATABLE_PPC32

It is seldom used in the kernel code and can be easily replaced by either RELOCATABLE or PPC32. So there is no reason to keep a separate kernel option for this. Signed-off-by: Kevin Hao <haoke...@gmail.com> --- arch/powerpc/Kconfig | 4 arch/powerpc/include/asm/page.h

[PATCH v2 2/3] powerpc: merge the RELOCATABLE config entries for ppc32 and ppc64

It makes no sense to keep two separate RELOCATABLE config entries for ppc32 and ppc64 respectively. Merge them into one and move it to a common place. Signed-off-by: Kevin Hao <haoke...@gmail.com> --- arch/powerpc/Kconfig | 65 ++-- 1 file c

[PATCH v2 1/3] powerpc32: booke: fix the build error when CRASH_DUMP is enabled

reasons to make the RELOCATABLE depend on ADVANCED_OPTIONS. So remove this dependency to fix this issue. Signed-off-by: Kevin Hao <haoke...@gmail.com> --- arch/powerpc/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/powerpc/Kconfig b/arch/powerpc/Kconfig inde

[PATCH v2 0/3] powerpc: tweak the kernel options for CRASH_DUMP and RELOCATABLE

cleanup for RELOCATABLE option. Kevin Hao (3): powerpc32: booke: fix the build error when CRASH_DUMP is enabled powerpc: merge the RELOCATABLE config entries for ppc32 and ppc64 powerpc32: kill RELOCATABLE_PPC32 arch/powerpc/Kconfig | 61 +++ arch

[PATCH v2 0/3] powerpc: tweak the kernel options for CRASH_DUMP and RELOCATABLE

cleanup for RELOCATABLE option. Kevin Hao (3): powerpc32: booke: fix the build error when CRASH_DUMP is enabled powerpc: merge the RELOCATABLE config entries for ppc32 and ppc64 powerpc32: kill RELOCATABLE_PPC32 arch/powerpc/Kconfig | 61 +++ arch

Re: [v2] powerpc/e6500: hw tablewalk: make sure we invalidate and write to the same tlb entry

On Fri, Oct 16, 2015 at 07:01:55PM -0500, Scott Wood wrote: > On Tue, Aug 18, 2015 at 03:55:56PM +0800, Kevin Hao wrote: > > diff --git a/arch/powerpc/mm/tlb_low_64e.S b/arch/powerpc/mm/tlb_low_64e.S > > index e4185581c5a7..3a5b89dfb5a1 100644 > > --- a/arch/powerpc/mm/tlb_low

[PATCH v3] powerpc/e6500: hw tablewalk: make sure we invalidate and write to the same tlb entry

the race that we may get the wrong value and overwrite the tlb entry just updated by the other thread. We observe better performance using option b. So reserve an additional register to save the value of the esel_next. Signed-off-by: Kevin Hao <haoke...@gmail.com> --- v3: Move th

Re: [PATCH] powerpc: fsl_pci: cast the regs->nip to void * when passing it to probe_kernel_address()

On Tue, Sep 22, 2015 at 12:08:17PM +1000, Michael Ellerman wrote: > On Mon, 2015-09-21 at 17:22 +0800, Kevin Hao wrote: > > With the reimplementation of probe_kernel_address() in commit > > ecc83243e1d4 ("uaccess: reimplement probe_kernel_address() using > > probe_kernel

Re: [PATCH] powerpc: fsl_pci: cast the regs->nip to void * when passing it to probe_kernel_address()

On Mon, Sep 21, 2015 at 09:16:59PM -0500, Scott Wood wrote: > On Mon, 2015-09-21 at 17:22 +0800, Kevin Hao wrote: > > With the reimplementation of probe_kernel_address() in commit > > ecc83243e1d4 ("uaccess: reimplement probe_kernel_address() using > > probe_kernel

[PATCH] powerpc: fsl_pci: cast the regs->nip to void * when passing it to probe_kernel_address()

or] ret = probe_kernel_address(regs->nip, inst); ^ Signed-off-by: Kevin Hao <haoke...@gmail.com> --- This is against linux-next. arch/powerpc/sysdev/fsl_pci.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/powerpc/sysdev/fsl_pci.c b/arch/powerpc/sysdev/fsl_pc

Re: [v2,3/6] powerpc: kill mfvtb()

On Mon, Sep 07, 2015 at 07:29:42PM +1000, Michael Ellerman wrote: > On Mon, 2015-24-08 at 11:20:25 UTC, Kevin Hao wrote: > > This function is only used by get_vtb(). They are almost the same > > except the reading from the real register. Move the mfspr() to > > get_vtb()

[PATCH v2 0/6] powerpc: use jump label for {cpu,mmu}_has_feature()

/pipermail/linuxppc-dev/2013-September/111026.html Kevin Hao (6): jump_label: make it possible for the archs to invoke jump_label_init() much earlier powerpc: invoke jump_label_init() in a much earlier stage powerpc: kill mfvtb() powerpc: move the cpu_has_feature to a separate file powerpc

[PATCH v2 1/6] jump_label: make it possible for the archs to invoke jump_label_init() much earlier

For some archs (such as powerpc) would want to invoke jump_label_init() in a much earlier stage. So check static_key_initialized in order to make sure this function run only once. Signed-off-by: Kevin Hao haoke...@gmail.com --- v2: No change. kernel/jump_label.c | 3 +++ 1 file changed, 3

[PATCH v2 2/6] powerpc: invoke jump_label_init() in a much earlier stage

So we can use static_key for cpu_has_feature() and mmu_has_feature(). Signed-off-by: Kevin Hao haoke...@gmail.com --- v2: No change. arch/powerpc/kernel/setup_32.c | 2 ++ arch/powerpc/kernel/setup_64.c | 2 ++ 2 files changed, 4 insertions(+) diff --git a/arch/powerpc/kernel/setup_32.c b/arch

[PATCH v2 3/6] powerpc: kill mfvtb()

of jump label for cpu_has_feature(). Signed-off-by: Kevin Hao haoke...@gmail.com --- v2: No change. arch/powerpc/include/asm/reg.h | 9 - arch/powerpc/include/asm/time.h | 2 +- 2 files changed, 1 insertion(+), 10 deletions(-) diff --git a/arch/powerpc/include/asm/reg.h b/arch/powerpc

[PATCH v2 4/6] powerpc: move the cpu_has_feature to a separate file

will introduces various recursive inclusion. And it is very hard to fix that. So we choose to move the function cpu_has_feature to a separate header file before using the jump label for it. No functional change. Signed-off-by: Kevin Hao haoke...@gmail.com --- v2: No change. arch/powerpc/include

[PATCH v2 5/6] powerpc: use the jump label for cpu_has_feature

is enabled b xxx if CPU_FTR_XXX is not enabled Signed-off-by: Kevin Hao haoke...@gmail.com --- v2: Use the open-coded definition and initialization for cpu_feat_keys[]. arch/powerpc/include/asm/cpufeatures.h | 20 arch/powerpc/include/asm/cputable.h| 8

[PATCH v2 6/6] powerpc: use jump label for mmu_has_feature

is enabled b xxx if MMU_FTR_XXX is not enabled Signed-off-by: Kevin Hao haoke...@gmail.com --- v2: Use the open-coded definition and initialization for mmu_feat_keys[]. arch/powerpc/include/asm/mmu.h | 29 + arch/powerpc/kernel/cputable.c | 17

Re: [PATCH 3/8] jump_label: introduce DEFINE_STATIC_KEY_{TRUE,FALSE}_ARRAY macros

On Fri, Aug 21, 2015 at 08:28:26AM +0200, Ingo Molnar wrote: * Kevin Hao haoke...@gmail.com wrote: These are used to define a static_key_{true,false} array. Signed-off-by: Kevin Hao haoke...@gmail.com --- include/linux/jump_label.h | 6 ++ 1 file changed, 6 insertions

Re: [PATCH 3/8] jump_label: introduce DEFINE_STATIC_KEY_{TRUE,FALSE}_ARRAY macros

On Fri, Aug 21, 2015 at 08:40:59AM +0200, Ingo Molnar wrote: Please also Cc: peterz and me to the next submission of the series - static key (and jump label) changes go through the locking tree normally, and there's a number of changes pending already for v4.3: Sure. Thanks, Kevin

Re: [PATCH 3/8] jump_label: introduce DEFINE_STATIC_KEY_{TRUE,FALSE}_ARRAY macros

On Thu, Aug 20, 2015 at 08:31:58PM +0200, Peter Zijlstra wrote: On Thu, Aug 20, 2015 at 08:14:31PM +0800, Kevin Hao wrote: These are used to define a static_key_{true,false} array. Yes but why... there might have been some clue in the patches you didn't send me, but since you didn't send

[PATCH 3/8] jump_label: introduce DEFINE_STATIC_KEY_{TRUE, FALSE}_ARRAY macros

These are used to define a static_key_{true,false} array. Signed-off-by: Kevin Hao haoke...@gmail.com --- include/linux/jump_label.h | 6 ++ 1 file changed, 6 insertions(+) diff --git a/include/linux/jump_label.h b/include/linux/jump_label.h index 7f653e8f6690..5c1d6a49dd6b 100644

[PATCH 8/8] powerpc: use jump label for mmu_has_feature

is enabled b xxx if MMU_FTR_XXX is not enabled Signed-off-by: Kevin Hao haoke...@gmail.com --- arch/powerpc/include/asm/mmu.h | 29 + arch/powerpc/kernel/cputable.c | 15 +++ arch/powerpc/kernel/setup_32.c | 1 + arch/powerpc/kernel/setup_64.c | 1 + 4

[PATCH 7/8] powerpc: use the jump label for cpu_has_feature

is enabled b xxx if CPU_FTR_XXX is not enabled Signed-off-by: Kevin Hao haoke...@gmail.com --- arch/powerpc/include/asm/cpufeatures.h | 20 arch/powerpc/include/asm/cputable.h| 8 arch/powerpc/kernel/cputable.c | 18 ++ arch/powerpc

[PATCH 0/8] powerpc: use jump label for {cpu,mmu}_has_feature()

(qemu). This patch series is against linux-next. [1] https://lists.ozlabs.org/pipermail/linuxppc-dev/2013-September/111026.html Kevin Hao (8): jump_label: no need to acquire the jump_label_mutex in jump_lable_init() jump_label: make it possible for the archs to invoke jump_label_init

[PATCH 1/8] jump_label: no need to acquire the jump_label_mutex in jump_lable_init()

The jump_label_init() run in a very early stage, even before the sched_init(). So there is no chance for concurrent access of the jump label table. Signed-off-by: Kevin Hao haoke...@gmail.com --- kernel/jump_label.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/kernel/jump_label.c b

[PATCH 5/8] powerpc: kill mfvtb()

of jump label for cpu_has_feature(). Signed-off-by: Kevin Hao haoke...@gmail.com --- arch/powerpc/include/asm/reg.h | 9 - arch/powerpc/include/asm/time.h | 2 +- 2 files changed, 1 insertion(+), 10 deletions(-) diff --git a/arch/powerpc/include/asm/reg.h b/arch/powerpc/include/asm/reg.h

[PATCH 2/8] jump_label: make it possible for the archs to invoke jump_label_init() much earlier

For some archs (such as powerpc) would want to invoke jump_label_init() in a much earlier stage. So check static_key_initialized in order to make sure this function run only once. Signed-off-by: Kevin Hao haoke...@gmail.com --- kernel/jump_label.c | 3 +++ 1 file changed, 3 insertions(+) diff

[PATCH 4/8] powerpc: invoke jump_label_init() in a much earlier stage

So we can use static_key for cpu_has_feature() and mmu_has_feature(). Signed-off-by: Kevin Hao haoke...@gmail.com --- arch/powerpc/kernel/setup_32.c | 2 ++ arch/powerpc/kernel/setup_64.c | 2 ++ 2 files changed, 4 insertions(+) diff --git a/arch/powerpc/kernel/setup_32.c b/arch/powerpc/kernel

[PATCH 6/8] powerpc: move the cpu_has_feature to a separate file

will introduces various recursive inclusion. And it is very hard to fix that. So we choose to move the function cpu_has_feature to a separate header file before using the jump label for it. No functional change. Signed-off-by: Kevin Hao haoke...@gmail.com --- arch/powerpc/include/asm/cacheflush.h

Re: [PATCH 1/8] jump_label: no need to acquire the jump_label_mutex in jump_lable_init()

On Thu, Aug 20, 2015 at 08:29:03PM +0200, Peter Zijlstra wrote: On Thu, Aug 20, 2015 at 08:14:29PM +0800, Kevin Hao wrote: The jump_label_init() run in a very early stage, even before the sched_init(). So there is no chance for concurrent access of the jump label table. It also doesn't

[PATCH v2] powerpc/e6500: hw tablewalk: make sure we invalidate and write to the same tlb entry

the race that we may get the wrong value and overwrite the tlb entry just updated by the other thread. We observe better performance using option b. So reserve an additional register to save the value of the esel_next. Signed-off-by: Kevin Hao haoke...@gmail.com --- v2: Use an additional

Re: [PATCH 2/3] powerpc/e6500: hw tablewalk: optimize a bit for tcd lock acquiring codes

On Fri, Aug 14, 2015 at 09:44:28PM -0500, Scott Wood wrote: I tried a couple different benchmarks and didn't find a significant difference, relative to the variability of the results running on the same kernel. A patch that claims to optimize a bit as its main purpose ought to show some

Re: [PATCH 3/3] powerpc/e6500: hw tablewalk: order the memory access when acquire/release tcd lock

On Fri, Aug 14, 2015 at 07:44:23PM -0500, Scott Wood wrote: On Fri, 2015-08-14 at 15:13 +0800, Kevin Hao wrote: On Thu, Aug 13, 2015 at 10:39:19PM -0500, Scott Wood wrote: On Thu, 2015-08-13 at 19:51 +0800, Kevin Hao wrote: I didn't find anything unusual. But I think we do need to order

Re: [PATCH 2/3] powerpc/e6500: hw tablewalk: optimize a bit for tcd lock acquiring codes

On Thu, Aug 13, 2015 at 01:44:43PM -0500, Scott Wood wrote: On Thu, 2015-08-13 at 19:51 +0800, Kevin Hao wrote: It makes no sense to put the instructions for calculating the lock value (cpu number + 1) and the clearing of eq bit of cr1 in lbarx/stbcx loop. And when the lock is acquired

Re: [PATCH 3/3] powerpc/e6500: hw tablewalk: order the memory access when acquire/release tcd lock

On Thu, Aug 13, 2015 at 10:39:19PM -0500, Scott Wood wrote: On Thu, 2015-08-13 at 19:51 +0800, Kevin Hao wrote: I didn't find anything unusual. But I think we do need to order the load/store of esel_next when acquire/release tcd lock. For acquire, add a data dependency to order the loads

[PATCH 2/3] powerpc/e6500: hw tablewalk: optimize a bit for tcd lock acquiring codes

the comparing for these two lock values in the lbz/bne loop. Signed-off-by: Kevin Hao haoke...@gmail.com --- arch/powerpc/mm/tlb_low_64e.S | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/arch/powerpc/mm/tlb_low_64e.S b/arch/powerpc/mm/tlb_low_64e.S index 765b419883f2

[PATCH 3/3] powerpc/e6500: hw tablewalk: order the memory access when acquire/release tcd lock

order. So we still need lwsync for the two stores for lock and esel_next. Signed-off-by: Kevin Hao haoke...@gmail.com --- arch/powerpc/mm/tlb_low_64e.S | 3 +++ 1 file changed, 3 insertions(+) diff --git a/arch/powerpc/mm/tlb_low_64e.S b/arch/powerpc/mm/tlb_low_64e.S index e4185581c5a7..964754911987

[PATCH 1/3] powerpc/e6500: remove the stale TCD_LOCK macro

Since we moved the lock to be the first element of struct tlb_core_data in commit 82d86de25b9c (powerpc/e6500: Make TLB lock recursive), this macro is not used by any code. Just delete it. Signed-off-by: Kevin Hao haoke...@gmail.com --- arch/powerpc/kernel/asm-offsets.c | 1 - 1 file changed, 1

[PATCH] powerpc: pci: use %pR for printing struct resource

Use %pR to simplify the debug code. This also make the debug info more readable. Signed-off-by: Kevin Hao haoke...@gmail.com --- arch/powerpc/kernel/pci-common.c | 72 +++- 1 file changed, 19 insertions(+), 53 deletions(-) diff --git a/arch/powerpc/kernel/pci

[PATCH] powerpc: mpc85xx: flush the l1 cache before cpu down in kexec

for this support [1]. Hope we can fix this issue for e500mc after that merged. [1] https://lists.ozlabs.org/pipermail/linuxppc-dev/2014-March/115830.html Signed-off-by: Kevin Hao haoke...@gmail.com --- arch/powerpc/platforms/85xx/smp.c | 51 +-- 1 file changed

Re: [PATCH 1/2] powerpc: fix the dependency issue for CRASH_DUMP

On Mon, May 04, 2015 at 05:17:17PM -0500, Scott Wood wrote: On Thu, 2015-04-30 at 20:29 +0800, Kevin Hao wrote: In the current code, the RELOCATABLE will be forcedly enabled when enabling CRASH_DUMP. But for ppc32, the RELOCABLE also depend on ADVANCED_OPTIONS and select NONSTATIC_KERNEL

[PATCH 0/2] powerpc: tweak the kernel options for CRASH_DUMP and RELOCATABLE

Hi, The first patch fixes a build error when CRASH_DUMP=y ADVANCED_OPTIONS=n for ppc32. The second does some cleanup for RELOCATABLE option. Kevin Hao (2): powerpc: fix the dependency issue for CRASH_DUMP powerpc: merge the RELOCATABLE config entries for ppc32 and ppc64 arch/powerpc

[PATCH 1/2] powerpc: fix the dependency issue for CRASH_DUMP

is only for non-visible symbols and for symbols with no dependencies. As for a symbol like RELOCATABLE, it is definitely not suitable to select it. So choose to depend on it. Also enable the RELOCATABLE explicitly for the defconfigs which has CRASH_DUMP enabled. Signed-off-by: Kevin Hao haoke

[PATCH 2/2] powerpc: merge the RELOCATABLE config entries for ppc32 and ppc64

It makes no sense to keep two separate RELOCATABLE config entries for ppc32 and ppc64 respectively. Merge them into one and move it to a common place. The dependency on ADVANCED_OPTIONS for ppc32 seems unnecessary, also drop it. Signed-off-by: Kevin Hao haoke...@gmail.com --- arch/powerpc

[PATCH 1/3] powerpc: introduce PPC_HAS_LOCK_OWNER

, we unconditionally encode the lock owner into the lock token for all the ppc64 platforms no matter it is a shared processor or not. This introduces a new kernel option to distinguish the platforms which need this hack. Signed-off-by: Kevin Hao haoke...@gmail.com --- arch/powerpc/Kconfig

[PATCH 2/3] powerpc: spinlock: refactor codes wrapped by PPC_HAS_LOCK_OWNER

Move all of them to one place. No function change. Signed-off-by: Kevin Hao haoke...@gmail.com --- arch/powerpc/include/asm/spinlock.h | 71 - 1 file changed, 30 insertions(+), 41 deletions(-) diff --git a/arch/powerpc/include/asm/spinlock.h b/arch/powerpc

Re: [RFC] powerpc: use ticket spin lock for !CONFIG_PPC_SPLPAR

On Thu, Mar 12, 2015 at 04:24:10PM +0100, Torsten Duwe wrote: But generally, which platforms would benefit most from this change? In theory, the more cpus the platform has, the more serious the thundering herd problem is. So the latest platforms which has more cpus would benefit most. Thanks,

Re: [RFC] powerpc: use ticket spin lock for !CONFIG_PPC_SPLPAR

On Thu, Mar 12, 2015 at 10:13:27PM +1100, Benjamin Herrenschmidt wrote: I would do the ifdef'ing differently, something like CONFIG_PPC_HAS_LOCK_OWNER CONFIG_PPC_TICKET_LOCKS depends on !PPC_HAS_LOCK_OWNER and use these two in the code... with SPLPAR select'ing HAS_LOCK_OWNER OK, will

[RFC] powerpc: use ticket spin lock for !CONFIG_PPC_SPLPAR

/pipermail/linuxppc-dev/2014-February/115195.html [2] http://patchwork.ozlabs.org/patch/447563/ Signed-off-by: Kevin Hao haoke...@gmail.com --- arch/powerpc/include/asm/spinlock.h | 79 ++- arch/powerpc/include/asm/spinlock_types.h | 16 +++ arch/powerpc/lib/locks.c

[PATCH v3 0/9] powerpc: kill PPC_OF

if we put this series into 4.1. v3: No codes change, just add the acks from Ben and Tomi. Kevin Hao (9): sata_svw: remove the dependency on PPC_OF fbdev: aty128fb: replace PPC_OF with PPC fbdev: radeon: replace PPC_OF with PPC fbdev: imsttfb: remove the dependency on PPC_OF fbdev: nvidia

[PATCH v3 1/9] sata_svw: remove the dependency on PPC_OF

The OF functionality has moved to a common place and be used by many archs. So we don't need to include the ppc arch specific header files and depend on PPC_OF option any more. This is a preparation for killing PPC_OF. Signed-off-by: Kevin Hao haoke...@gmail.com Acked-by: Tejun Heo t

[PATCH v3 2/9] fbdev: aty128fb: replace PPC_OF with PPC

-by: Kevin Hao haoke...@gmail.com Acked-by: Benjamin Herrenschmidt b...@kernel.crashing.org Acked-by: Tomi Valkeinen tomi.valkei...@ti.com --- drivers/video/fbdev/aty/aty128fb.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/video/fbdev/aty/aty128fb.c b/drivers

[PATCH v3 3/9] fbdev: radeon: replace PPC_OF with PPC

-by: Kevin Hao haoke...@gmail.com Acked-by: Benjamin Herrenschmidt b...@kernel.crashing.org Acked-by: Tomi Valkeinen tomi.valkei...@ti.com --- drivers/video/fbdev/Kconfig | 2 +- drivers/video/fbdev/aty/radeon_base.c| 24 drivers/video/fbdev/aty

[PATCH v3 4/9] fbdev: imsttfb: remove the dependency on PPC_OF

The OF functionality has moved to a common place and be used by many archs. So we don't need to depend on PPC_OF option any more. This is a preparation for killing PPC_OF. Signed-off-by: Kevin Hao haoke...@gmail.com Acked-by: Benjamin Herrenschmidt b...@kernel.crashing.org Acked-by: Tomi

[PATCH v3 5/9] fbdev: nvidia: remove the dependency on PPC_OF

The OF functionality has moved to a common place and be used by many archs. So we don't need to include the ppc arch specific header files and depend on PPC_OF option any more. This is a preparation for killing PPC_OF. Signed-off-by: Kevin Hao haoke...@gmail.com Acked-by: Benjamin Herrenschmidt b

[PATCH v3 6/9] fbdev: riva: remove the dependency on PPC_OF

The OF functionality has moved to a common place and be used by many archs. So we don't need to include the ppc arch specific header files and depend on PPC_OF option any more. This is a preparation for killing PPC_OF. Signed-off-by: Kevin Hao haoke...@gmail.com Acked-by: Benjamin Herrenschmidt b

[PATCH v3 7/9] fbdev: remove the unnecessary includes of ppc specific header files

In the current kernel, we don't need to include these arch specific header files for ppc. Signed-off-by: Kevin Hao haoke...@gmail.com Acked-by: Benjamin Herrenschmidt b...@kernel.crashing.org Acked-by: Tomi Valkeinen tomi.valkei...@ti.com --- drivers/video/fbdev/core/fbmon.c | 4 1 file

[PATCH v3 8/9] fbdev: kconfig: replace PPC_OF with PPC

-by: Kevin Hao haoke...@gmail.com Acked-by: Benjamin Herrenschmidt b...@kernel.crashing.org Acked-by: Tomi Valkeinen tomi.valkei...@ti.com --- drivers/video/fbdev/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/video/fbdev/Kconfig b/drivers/video/fbdev/Kconfig

[PATCH v3 9/9] powerpc: kill PPC_OF

to keep such a option in the current kernel. Just kill it. Signed-off-by: Kevin Hao haoke...@gmail.com Acked-by: Benjamin Herrenschmidt b...@kernel.crashing.org --- arch/powerpc/Kconfig | 3 --- arch/powerpc/Kconfig.debug | 2 +- arch/powerpc/kernel/Makefile | 4 ++-- 3 files changed, 3

Re: [PATCH] powerpc: book3e_64: fix the align size for paca_struct

On Mon, Mar 09, 2015 at 06:31:25PM +1100, Benjamin Herrenschmidt wrote: On Mon, 2015-03-09 at 09:13 +0800, Kevin Hao wrote: On Sun, Mar 08, 2015 at 08:13:26PM +1100, Benjamin Herrenschmidt wrote: On Sat, 2015-03-07 at 19:14 +0800, Kevin Hao wrote: All the cache line size of the current

[PATCH v2] powerpc: book3e_64: fix the align size for paca_struct

*/ /* sum members: 1667, holes: 4, sum holes: 13 */ /* padding: 48 */ Signed-off-by: Kevin Hao haoke...@gmail.com --- v2: Only update the commit log. arch/powerpc/include/asm/paca.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/arch/powerpc/include/asm/paca.h b

Re: [PATCH] powerpc64: use fixed lock token for !CONFIG_PPC_SPLPAR

On Tue, Mar 10, 2015 at 11:15:18AM +1100, Michael Ellerman wrote: On Mon, 2015-03-09 at 17:53 +1100, Benjamin Herrenschmidt wrote: On Sat, 2015-03-07 at 19:19 +0800, Kevin Hao wrote: It makes no sense to use a variant lock token on a platform which doesn't support for shared-processor

Re: [PATCH 03/15] fbdev: aty128fb: replace PPC_OF with PPC

On Tue, Mar 10, 2015 at 02:23:12PM +0200, Tomi Valkeinen wrote: I just sent out a v2 [1] a few hours earlier with some minor updates. We plan to merge this patch series via the powerpc tree in 4.1 cycle if I can collect all the acks from the corresponding driver maintainers. Fbdev

Re: [PATCH] powerpc: book3e_64: fix the align size for paca_struct

On Sun, Mar 08, 2015 at 08:13:26PM +1100, Benjamin Herrenschmidt wrote: On Sat, 2015-03-07 at 19:14 +0800, Kevin Hao wrote: All the cache line size of the current book3e 64bit SoCs are 64 bytes. So we should use this size to align the member of paca_struct. With this change we save 192

[PATCH] powerpc64: use fixed lock token for !CONFIG_PPC_SPLPAR

It makes no sense to use a variant lock token on a platform which doesn't support for shared-processor logical partitions. Actually we can eliminate a memory load by using a fixed lock token on these platforms. Signed-off-by: Kevin Hao haoke...@gmail.com --- arch/powerpc/include/asm/spinlock.h

[PATCH] powerpc: book3e_64: fix the align size for paca_struct

, cachelines: 30, members: 46 */ /* sum members: 1667, holes: 6, sum holes: 141 */ /* padding: 112 */ After: /* size: 1728, cachelines: 27, members: 46 */ /* sum members: 1667, holes: 4, sum holes: 13 */ /* padding: 48 */ Signed-off-by: Kevin Hao haoke

[PATCH v2 00/11] powerpc: kill PPC_OF

pci_device_to_OF_node() stub for !CONFIG_OF) - Add the ack in v1 cycle. - Resolve the comments in v1 cycle. We plan to merge this patch series via the powerpc tree in 4.1 cycle. So please ack the corresponding patches if you are OK with these changes. Kevin Hao (11): sata_svw: remove the dependency

[PATCH v2 01/11] sata_svw: remove the dependency on PPC_OF

The OF functionality has moved to a common place and be used by many archs. So we don't need to include the ppc arch specific header files and depend on PPC_OF option any more. This is a preparation for killing PPC_OF. Signed-off-by: Kevin Hao haoke...@gmail.com Acked-by: Tejun Heo t

[PATCH v2 02/11] fbdev: aty128fb: replace PPC_OF with PPC

-by: Kevin Hao haoke...@gmail.com --- v2: No change. drivers/video/fbdev/aty/aty128fb.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/video/fbdev/aty/aty128fb.c b/drivers/video/fbdev/aty/aty128fb.c index aedf2fbf9bf6..0156954bf340 100644 --- a/drivers/video/fbdev

[PATCH v2 03/11] fbdev: radeon: replace PPC_OF with PPC

-by: Kevin Hao haoke...@gmail.com --- v2: No change. drivers/video/fbdev/Kconfig | 2 +- drivers/video/fbdev/aty/radeon_base.c| 24 drivers/video/fbdev/aty/radeon_monitor.c | 20 ++-- drivers/video/fbdev/aty/radeon_pm.c | 16

[PATCH v2 04/11] fbdev: imsttfb: remove the dependency on PPC_OF

The OF functionality has moved to a common place and be used by many archs. So we don't need to depend on PPC_OF option any more. This is a preparation for killing PPC_OF. Signed-off-by: Kevin Hao haoke...@gmail.com --- v2: Only print the error log when CONFIG_OF is enabled. drivers/video/fbdev

[PATCH v2 05/11] fbdev: nvidia: remove the dependency on PPC_OF

The OF functionality has moved to a common place and be used by many archs. So we don't need to include the ppc arch specific header files and depend on PPC_OF option any more. This is a preparation for killing PPC_OF. Signed-off-by: Kevin Hao haoke...@gmail.com --- v2: No change. drivers/video

[PATCH v2 06/11] fbdev: riva: remove the dependency on PPC_OF

The OF functionality has moved to a common place and be used by many archs. So we don't need to include the ppc arch specific header files and depend on PPC_OF option any more. This is a preparation for killing PPC_OF. Signed-off-by: Kevin Hao haoke...@gmail.com --- v2: No change. drivers/video

[PATCH v2 07/11] fbdev: remove the unnecessary includes of ppc specific header files

In the current kernel, we don't need to include these arch specific header files for ppc. Signed-off-by: Kevin Hao haoke...@gmail.com --- v2: No change. drivers/video/fbdev/core/fbmon.c | 4 1 file changed, 4 deletions(-) diff --git a/drivers/video/fbdev/core/fbmon.c b/drivers/video/fbdev

[PATCH v2 08/11] fbdev: kconfig: replace PPC_OF with PPC

-by: Kevin Hao haoke...@gmail.com --- v2: No change. drivers/video/fbdev/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/video/fbdev/Kconfig b/drivers/video/fbdev/Kconfig index 3b818d7a0983..109462303087 100644 --- a/drivers/video/fbdev/Kconfig +++ b/drivers

[PATCH v2 09/11] mmc: kconfig: replace PPC_OF with PPC

-by: Kevin Hao haoke...@gmail.com --- v2: No change. drivers/mmc/host/Kconfig | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/mmc/host/Kconfig b/drivers/mmc/host/Kconfig index 61ac63a3776a..7858d7a52818 100644 --- a/drivers/mmc/host/Kconfig +++ b/drivers/mmc/host

[PATCH v2 10/11] tty: kconfig: remove the superfluous dependency on PPC_OF

In the current kernel, the CONFIG_PPC_OF is always 'y' for the ppc arch. So we don't need to check it with other ppc specific options. Signed-off-by: Kevin Hao haoke...@gmail.com --- v2: No change. drivers/tty/serial/Kconfig | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

[PATCH v2 11/11] powerpc: kill PPC_OF

to keep such a option in the current kernel. Just kill it. Signed-off-by: Kevin Hao haoke...@gmail.com --- v2: No change. arch/powerpc/Kconfig | 3 --- arch/powerpc/Kconfig.debug | 2 +- arch/powerpc/kernel/Makefile | 4 ++-- 3 files changed, 3 insertions(+), 6 deletions(-) diff --git a/arch

Re: [PATCH 03/15] fbdev: aty128fb: replace PPC_OF with PPC

On Fri, Feb 27, 2015 at 11:11:15AM +1100, Benjamin Herrenschmidt wrote: On Sat, 2015-01-31 at 21:47 +0800, Kevin Hao wrote: The PPC_OF is a ppc specific option which is used to mean that the firmware device tree access functions are available. Since all the ppc platforms have a device tree

Re: [PATCH 1/3] powerpc/dma: Always configure ZONE_DMA32 correctly

On Sun, Feb 22, 2015 at 09:23:51AM +1100, Benjamin Herrenschmidt wrote: No it won't, or am I missing something ? Ie, limit_zone_pfn() will only reduce the size of the window, never increase it. Yes, you are right. Just overlook this. Sorry for the noise. Thanks, Kevin pgpH0IxyneZ4n.pgp

Re: [PATCH 1/3] powerpc/dma: Always configure ZONE_DMA32 correctly

On Fri, Feb 20, 2015 at 07:35:25PM +1100, Benjamin Herrenschmidt wrote: We do this for consistency and also in order to support the use of a consistent mask smaller than the dma mask in subsequent patches. Signed-off-by: Benjamin Herrenschmidt b...@kernel.crashing.org ---

[PATCH] mmc: sdhci-pltfm: remove the unneeded check of disabled device

, it just doesn't parse the dtb for the infos such as quirks or clock, but it will continue to try to init the disabled device after that check. So just remove it. Signed-off-by: Kevin Hao haoke...@gmail.com --- drivers/mmc/host/sdhci-pltfm.c | 54 -- 1 file

  1   2   3   4   >