Re: [PATCH] staging: android: ion: minor cleanup for ion_page_pool

Hi Laura, On 2018/2/7 6:51, Laura Abbott wrote: > On 01/31/2018 01:50 AM, Yisheng Xie wrote: >> ion_page_pool.c now is used to apply pool APIs for system heap, which do >> not need do any initial at device_initcall. >> >> Meanwhile, this patch also remove some useless i

[PATCH 2/2] staging: android: ion: Return void instead of int

Now, nobody care about the return value of ion_page_pool_add, therefore we can just make it return void. Signed-off-by: Yisheng Xie <xieyishe...@huawei.com> --- drivers/staging/android/ion/ion_page_pool.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/s

[PATCH 1/2] staging: android: ion: Remove dead code in ion_page_pool_free

ion_page_pool_add will always return 0, however ion_page_pool_free will call ion_page_pool_free_pages when ion_page_pool_add's return value is not 0, so it is a dead code which can be removed. Signed-off-by: Yisheng Xie <xieyishe...@huawei.com> --- drivers/staging/android/ion/ion_page_pool

[PATCH 2/2] staging: android: ion: Return void instead of int

Now, nobody care about the return value of ion_page_pool_add, therefore we can just make it return void. Signed-off-by: Yisheng Xie --- drivers/staging/android/ion/ion_page_pool.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/staging/android/ion/ion_page_pool.c

[PATCH 1/2] staging: android: ion: Remove dead code in ion_page_pool_free

ion_page_pool_add will always return 0, however ion_page_pool_free will call ion_page_pool_free_pages when ion_page_pool_add's return value is not 0, so it is a dead code which can be removed. Signed-off-by: Yisheng Xie --- drivers/staging/android/ion/ion_page_pool.c | 6 +- 1 file changed

[PATCH] ftrace: nuke clear_ftrace_function

clear_ftrace_function is not used outside of ftrace.c and is not help to use a function, so nuke it per Steve's suggestion. Suggested-by: Steven Rostedt <rost...@goodmis.org> Signed-off-by: Yisheng Xie <xieyishe...@huawei.com> --- include/linux/ftrace.h | 2 -- kernel/trace/ft

[PATCH] ftrace: nuke clear_ftrace_function

clear_ftrace_function is not used outside of ftrace.c and is not help to use a function, so nuke it per Steve's suggestion. Suggested-by: Steven Rostedt Signed-off-by: Yisheng Xie --- include/linux/ftrace.h | 2 -- kernel/trace/ftrace.c | 13 + 2 files changed, 1 insertion(+), 14

[PATCH 1/2] staging: android: ion: Avoid NULL point in error path

of debug_root, then a NULL point will occur. Fix this by avoiding call dentry_path, but show the debug name only when failed to create debug file for shrinker. Signed-off-by: Yisheng Xie <xieyishe...@huawei.com> --- drivers/staging/android/ion/ion.c | 10 +++--- 1 file changed, 3 insertions

[PATCH 2/2] staging: android: ion: Remove lable debugfs_done

When failed to create debug_root, we will go on initail other part of ion, so we can just info this message to user and do not need a lable to jump. Signed-off-by: Yisheng Xie <xieyishe...@huawei.com> --- drivers/staging/android/ion/ion.c | 5 + 1 file changed, 1 insertion(+), 4 del

[PATCH 1/2] staging: android: ion: Avoid NULL point in error path

of debug_root, then a NULL point will occur. Fix this by avoiding call dentry_path, but show the debug name only when failed to create debug file for shrinker. Signed-off-by: Yisheng Xie --- drivers/staging/android/ion/ion.c | 10 +++--- 1 file changed, 3 insertions(+), 7 deletions(-) diff --git

[PATCH 2/2] staging: android: ion: Remove lable debugfs_done

When failed to create debug_root, we will go on initail other part of ion, so we can just info this message to user and do not need a lable to jump. Signed-off-by: Yisheng Xie --- drivers/staging/android/ion/ion.c | 5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/drivers

[PATCH resend 1/3] staging: android: ion: Remove unused declaration ion_buffer_fault_user_mappings

ion_buffer_fault_user_mappings's definition has been removed and not be used anymore, just remove its useless declaration. Signed-off-by: Yisheng Xie <xieyishe...@huawei.com> --- drivers/staging/android/ion/ion.h | 9 - 1 file changed, 9 deletions(-) diff --git a/drivers/staging/a

[PATCH resend 1/3] staging: android: ion: Remove unused declaration ion_buffer_fault_user_mappings

ion_buffer_fault_user_mappings's definition has been removed and not be used anymore, just remove its useless declaration. Signed-off-by: Yisheng Xie --- drivers/staging/android/ion/ion.h | 9 - 1 file changed, 9 deletions(-) diff --git a/drivers/staging/android/ion/ion.h b/drivers

[PATCH resend 3/3] staging: android: ion: Nuke ion_page_pool_init

ion_page_pool.c now is used to apply pool APIs for system heap, which do not need do any initial at device_initcall. Therefore ion_page_pool_init can be nuked. Signed-off-by: Yisheng Xie <xieyishe...@huawei.com> --- drivers/staging/android/ion/ion_page_pool.c | 7 --- 1 file chan

[PATCH resend 2/3] staging: android: ion: Remove unused include files for ion_page_pool.c

After rewrite of ion_page_pool, some of its include file is no need anymore, just remove it. Signed-off-by: Yisheng Xie <xieyishe...@huawei.com> --- drivers/staging/android/ion/ion_page_pool.c | 4 1 file changed, 4 deletions(-) diff --git a/drivers/staging/android/ion/ion_page_poo

[PATCH resend 3/3] staging: android: ion: Nuke ion_page_pool_init

ion_page_pool.c now is used to apply pool APIs for system heap, which do not need do any initial at device_initcall. Therefore ion_page_pool_init can be nuked. Signed-off-by: Yisheng Xie --- drivers/staging/android/ion/ion_page_pool.c | 7 --- 1 file changed, 7 deletions(-) diff --git

[PATCH resend 2/3] staging: android: ion: Remove unused include files for ion_page_pool.c

After rewrite of ion_page_pool, some of its include file is no need anymore, just remove it. Signed-off-by: Yisheng Xie --- drivers/staging/android/ion/ion_page_pool.c | 4 1 file changed, 4 deletions(-) diff --git a/drivers/staging/android/ion/ion_page_pool.c b/drivers/staging/android

Re: [PATCH] staging: android: ion: minor cleanup for ion_page_pool

Hi Greg, On 2018/1/31 19:32, Greg KH wrote: > On Wed, Jan 31, 2018 at 05:50:52PM +0800, Yisheng Xie wrote: >> ion_page_pool.c now is used to apply pool APIs for system heap, which do >> not need do any initial at device_initcall. >> >> Meanwhile, this patch also remove

Re: [PATCH] staging: android: ion: minor cleanup for ion_page_pool

Hi Greg, On 2018/1/31 19:32, Greg KH wrote: > On Wed, Jan 31, 2018 at 05:50:52PM +0800, Yisheng Xie wrote: >> ion_page_pool.c now is used to apply pool APIs for system heap, which do >> not need do any initial at device_initcall. >> >> Meanwhile, this patch also remove

[PATCH] staging: android: ion: minor cleanup for ion_page_pool

ion_page_pool.c now is used to apply pool APIs for system heap, which do not need do any initial at device_initcall. Meanwhile, this patch also remove some useless include files. Signed-off-by: Yisheng Xie <xieyishe...@huawei.com> --- drivers/staging/android/ion/ion_page_pool.

[PATCH] staging: android: ion: minor cleanup for ion_page_pool

ion_page_pool.c now is used to apply pool APIs for system heap, which do not need do any initial at device_initcall. Meanwhile, this patch also remove some useless include files. Signed-off-by: Yisheng Xie --- drivers/staging/android/ion/ion_page_pool.c | 11 --- 1 file changed, 11

[PATCH] staging: android: ion: Remove unused declaration ion_buffer_fault_user_mappings

ion_buffer_fault_user_mappings's definition has been removed and not be used anymore, just remove its useless declaration. Signed-off-by: Yisheng Xie <xieyishe...@huawei.com> --- drivers/staging/android/ion/ion.h | 9 - 1 file changed, 9 deletions(-) diff --git a/drivers/staging/a

[PATCH] staging: android: ion: Remove unused declaration ion_buffer_fault_user_mappings

ion_buffer_fault_user_mappings's definition has been removed and not be used anymore, just remove its useless declaration. Signed-off-by: Yisheng Xie --- drivers/staging/android/ion/ion.h | 9 - 1 file changed, 9 deletions(-) diff --git a/drivers/staging/android/ion/ion.h b/drivers

Re: [PATCH] ftrace: use helper function clear_ftrace_function

Hi Steve, On 2018/1/30 18:13, Steven Rostedt wrote: > On Tue, 30 Jan 2018 08:46:01 +0800 > Yisheng Xie <xieyishe...@huawei.com> wrote: > >> We have helper function clear_ftrace_function(), use it instead of hard >> code. And this should not have any function

Re: [PATCH] ftrace: use helper function clear_ftrace_function

Hi Steve, On 2018/1/30 18:13, Steven Rostedt wrote: > On Tue, 30 Jan 2018 08:46:01 +0800 > Yisheng Xie wrote: > >> We have helper function clear_ftrace_function(), use it instead of hard >> code. And this should not have any functional change. >> >> Signed-off-b

[PATCH] ftrace: use helper function clear_ftrace_function

We have helper function clear_ftrace_function(), use it instead of hard code. And this should not have any functional change. Signed-off-by: Yisheng Xie <xieyishe...@huawei.com> --- kernel/trace/ftrace.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/kernel/trace/ftra

[PATCH] ftrace: use helper function clear_ftrace_function

We have helper function clear_ftrace_function(), use it instead of hard code. And this should not have any functional change. Signed-off-by: Yisheng Xie --- kernel/trace/ftrace.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/kernel/trace/ftrace.c b/kernel/trace/ftrace.c

[PATCH v6] devres: combine function devm_ioremap*

Bytes in my compile environment. Suggested-by: Greg KH <gre...@linuxfoundation.org> Suggested-by: Christophe LEROY <christophe.le...@c-s.fr> Signed-off-by: Yisheng Xie <xieyishe...@huawei.com> --- v2: - use MARCO for ioremap v3: - kill dev_ioremap_nocache v4: - combine fun

[PATCH v6] devres: combine function devm_ioremap*

Bytes in my compile environment. Suggested-by: Greg KH Suggested-by: Christophe LEROY Signed-off-by: Yisheng Xie --- v2: - use MARCO for ioremap v3: - kill dev_ioremap_nocache v4: - combine function devm_ioremap* - per Christophe v5: - fix code style. - per

Re: [PATCH AUTOSEL for 4.14 078/100] kmemleak: add scheduling point to kmemleak_scan()

Hi Sasha, There is a fix patch for this patch: 13ab183 mm/kmemleak.c: make cond_resched() rate-limiting more efficient So if this patch is backported to stable, maybe the above patch is also need to be backported too. Thanks Yisheng On 2018/1/24 12:15, Sasha Levin wrote: > From: Yisheng

Re: [PATCH AUTOSEL for 4.14 078/100] kmemleak: add scheduling point to kmemleak_scan()

Hi Sasha, There is a fix patch for this patch: 13ab183 mm/kmemleak.c: make cond_resched() rate-limiting more efficient So if this patch is backported to stable, maybe the above patch is also need to be backported too. Thanks Yisheng On 2018/1/24 12:15, Sasha Levin wrote: > From: Yisheng

Re: [PATCH v5] devres: combine function devm_ioremap*

On 2018/1/23 16:42, Greg KH wrote: > On Tue, Jan 16, 2018 at 08:03:41PM +0800, Yisheng Xie wrote: >> When I tried to use devm_ioremap function and review related >> code, I found devm_ioremap_* almost have the similar realize >> with each other, which can be combined. >&g

Re: [PATCH v5] devres: combine function devm_ioremap*

On 2018/1/23 16:42, Greg KH wrote: > On Tue, Jan 16, 2018 at 08:03:41PM +0800, Yisheng Xie wrote: >> When I tried to use devm_ioremap function and review related >> code, I found devm_ioremap_* almost have the similar realize >> with each other, which can be combined. >&g

Re: [PATCH v3 11/20] arm64: mm: Map entry trampoline into trampoline and kernel page tables

Hi Will, On 2018/1/23 18:04, Will Deacon wrote: > On Tue, Jan 23, 2018 at 04:28:45PM +0800, Yisheng Xie wrote: >> On 2017/12/6 20:35, Will Deacon wrote: >>> +#ifdef CONFIG_UNMAP_KERNEL_AT_EL0 >>> +static int __init map_entry_trampoline(void) >>> +{ >>

Re: [PATCH v3 11/20] arm64: mm: Map entry trampoline into trampoline and kernel page tables

Hi Will, On 2018/1/23 18:04, Will Deacon wrote: > On Tue, Jan 23, 2018 at 04:28:45PM +0800, Yisheng Xie wrote: >> On 2017/12/6 20:35, Will Deacon wrote: >>> +#ifdef CONFIG_UNMAP_KERNEL_AT_EL0 >>> +static int __init map_entry_trampoline(void) >>> +{ >>

Re: [PATCH v3 11/20] arm64: mm: Map entry trampoline into trampoline and kernel page tables

Hi Will, On 2017/12/6 20:35, Will Deacon wrote: > +#ifdef CONFIG_UNMAP_KERNEL_AT_EL0 > +static int __init map_entry_trampoline(void) > +{ > + extern char __entry_tramp_text_start[]; > + > + pgprot_t prot = rodata_enabled ? PAGE_KERNEL_ROX : PAGE_KERNEL_EXEC; > + phys_addr_t pa_start =

Re: [PATCH v3 11/20] arm64: mm: Map entry trampoline into trampoline and kernel page tables

Hi Will, On 2017/12/6 20:35, Will Deacon wrote: > +#ifdef CONFIG_UNMAP_KERNEL_AT_EL0 > +static int __init map_entry_trampoline(void) > +{ > + extern char __entry_tramp_text_start[]; > + > + pgprot_t prot = rodata_enabled ? PAGE_KERNEL_ROX : PAGE_KERNEL_EXEC; > + phys_addr_t pa_start =

Re: [PATCH v2 07/11] arm64: Add skeleton to harden the branch predictor against aliasing attacks

Hi Will, On 2018/1/17 18:07, Will Deacon wrote: > On Wed, Jan 17, 2018 at 12:10:33PM +0800, Yisheng Xie wrote: >> Hi Will, >> >> On 2018/1/5 21:12, Will Deacon wrote: >>> diff --git a/arch/arm64/mm/context.c b/arch/arm64/mm/context.c >>> index 5f7097d0c

Re: [PATCH v2 07/11] arm64: Add skeleton to harden the branch predictor against aliasing attacks

Hi Will, On 2018/1/17 18:07, Will Deacon wrote: > On Wed, Jan 17, 2018 at 12:10:33PM +0800, Yisheng Xie wrote: >> Hi Will, >> >> On 2018/1/5 21:12, Will Deacon wrote: >>> diff --git a/arch/arm64/mm/context.c b/arch/arm64/mm/context.c >>> index 5f7097d0c

Re: [PATCH v2 07/11] arm64: Add skeleton to harden the branch predictor against aliasing attacks

Hi Will, On 2018/1/5 21:12, Will Deacon wrote: > diff --git a/arch/arm64/mm/context.c b/arch/arm64/mm/context.c > index 5f7097d0cd12..d99b36555a16 100644 > --- a/arch/arm64/mm/context.c > +++ b/arch/arm64/mm/context.c > @@ -246,6 +246,8 @@ asmlinkage void post_ttbr_update_workaround(void) >

Re: [PATCH v2 07/11] arm64: Add skeleton to harden the branch predictor against aliasing attacks

Hi Will, On 2018/1/5 21:12, Will Deacon wrote: > diff --git a/arch/arm64/mm/context.c b/arch/arm64/mm/context.c > index 5f7097d0cd12..d99b36555a16 100644 > --- a/arch/arm64/mm/context.c > +++ b/arch/arm64/mm/context.c > @@ -246,6 +246,8 @@ asmlinkage void post_ttbr_update_workaround(void) >

Re: [PATCH v3 06/20] arm64: mm: Fix and re-enable ARM64_SW_TTBR0_PAN

hardware do not support PAN, is SW_TTBR0_PAN is *must* option? or there maybe a security risk? Thanks Yisheng Xie

Re: [PATCH v3 06/20] arm64: mm: Fix and re-enable ARM64_SW_TTBR0_PAN

hardware do not support PAN, is SW_TTBR0_PAN is *must* option? or there maybe a security risk? Thanks Yisheng Xie

[PATCH v5] devres: combine function devm_ioremap*

to 7352Bytes in my compile environment. Suggested-by: Christophe LEROY <christophe.le...@c-s.fr> Signed-off-by: Yisheng Xie <xieyishe...@huawei.com> --- v2: - use MARCO for ioremap v3: - kill dev_ioremap_nocache v4: - combine function devm_ioremap* v5: - fix code style. include/lin

[PATCH v5] devres: combine function devm_ioremap*

to 7352Bytes in my compile environment. Suggested-by: Christophe LEROY Signed-off-by: Yisheng Xie --- v2: - use MARCO for ioremap v3: - kill dev_ioremap_nocache v4: - combine function devm_ioremap* v5: - fix code style. include/linux/io.h | 61 +++ lib/devres.c

Re: [PATCH v4] devres: combine function devm_ioremap*

Hi Christophe, Thanks for your review and comment, I will follow your suggestion listed in line. Thanks Yisheng On 2018/1/14 0:39, christophe leroy wrote: > Hi Yisheng, > > Le 12/01/2018 à 12:51, Yisheng Xie a écrit : >> When I tried to use devm_ioremap function and review rel

Re: [PATCH v4] devres: combine function devm_ioremap*

Hi Christophe, Thanks for your review and comment, I will follow your suggestion listed in line. Thanks Yisheng On 2018/1/14 0:39, christophe leroy wrote: > Hi Yisheng, > > Le 12/01/2018 à 12:51, Yisheng Xie a écrit : >> When I tried to use devm_ioremap function and review rel

[PATCH v4] devres: combine function devm_ioremap*

to 7352Bytes in my compile environment. Suggested-by: Christophe LEROY <christophe.le...@c-s.fr> Signed-off-by: Yisheng Xie <xieyishe...@huawei.com> --- v2: - use MARCO for ioremap v3: - kill dev_ioremap_nocache v4: - combine function devm_ioremap* include/linux/io.h | 33 +

[PATCH v4] devres: combine function devm_ioremap*

to 7352Bytes in my compile environment. Suggested-by: Christophe LEROY Signed-off-by: Yisheng Xie --- v2: - use MARCO for ioremap v3: - kill dev_ioremap_nocache v4: - combine function devm_ioremap* include/linux/io.h | 33 +++- lib/devres.c | 76

Re: [PATCH v3 19/27] drm: replace devm_ioremap_nocache with devm_ioremap

Hi Benjamin , On 2018/1/9 17:21, Benjamin Gaignard wrote: > 2017-12-23 12:01 GMT+01:00 Yisheng Xie <xieyishe...@huawei.com>: >> > Default ioremap is ioremap_nocache, so devm_ioremap has the same >> > function with devm_ioremap_nocache, which can just be killed to >

Re: [PATCH v3 19/27] drm: replace devm_ioremap_nocache with devm_ioremap

Hi Benjamin , On 2018/1/9 17:21, Benjamin Gaignard wrote: > 2017-12-23 12:01 GMT+01:00 Yisheng Xie : >> > Default ioremap is ioremap_nocache, so devm_ioremap has the same >> > function with devm_ioremap_nocache, which can just be killed to >> > save the size of devr

Re: [PATCH v3 00/27] kill devm_ioremap_nocache

Hi Christophe ， On 2018/1/4 16:05, Christophe LEROY wrote: > > > Le 25/12/2017 à 02:34, Yisheng Xie a écrit : >> >> >> On 2017/12/24 17:05, christophe leroy wrote: >>> >>> >>> Le 23/12/2017 à 14:48, Greg KH a écrit : >>>> On Sat,

Re: [PATCH v3 00/27] kill devm_ioremap_nocache

Hi Christophe ， On 2018/1/4 16:05, Christophe LEROY wrote: > > > Le 25/12/2017 à 02:34, Yisheng Xie a écrit : >> >> >> On 2017/12/24 17:05, christophe leroy wrote: >>> >>> >>> Le 23/12/2017 à 14:48, Greg KH a écrit : >>>> On Sat,

Re: [PATCH 05/23] x86, kaiser: unmap kernel from userspace page tables (core patch)

Hi Dave, On 2018/1/5 13:18, Dave Hansen wrote: > On 01/04/2018 08:16 PM, Yisheng Xie wrote: >>> === Page Table Poisoning === >>> >>> KAISER has two copies of the page tables: one for the kernel and >>> one for when running in userspace. >> >>

Re: [PATCH 05/23] x86, kaiser: unmap kernel from userspace page tables (core patch)

Hi Dave, On 2018/1/5 13:18, Dave Hansen wrote: > On 01/04/2018 08:16 PM, Yisheng Xie wrote: >>> === Page Table Poisoning === >>> >>> KAISER has two copies of the page tables: one for the kernel and >>> one for when running in userspace. >> >>

Re: [PATCH 05/23] x86, kaiser: unmap kernel from userspace page tables (core patch)

Hi Dava, On 2017/11/23 8:34, Dave Hansen wrote: > > From: Dave Hansen > > These patches are based on work from a team at Graz University of > Technology: https://github.com/IAIK/KAISER . This work would not have > been possible without their work as a starting

Re: [PATCH 05/23] x86, kaiser: unmap kernel from userspace page tables (core patch)

Hi Dava, On 2017/11/23 8:34, Dave Hansen wrote: > > From: Dave Hansen > > These patches are based on work from a team at Graz University of > Technology: https://github.com/IAIK/KAISER . This work would not have > been possible without their work as a starting point. > > KAISER is a

Re: [PATCH v3 00/27] kill devm_ioremap_nocache

+ cris/ia64/mn10300/openrisc maintainers On 2017/12/25 9:09, Yisheng Xie wrote: > hi Christophe and Greg, > > On 2017/12/24 16:55, christophe leroy wrote: >> >> >> Le 23/12/2017 à 16:57, Guenter Roeck a écrit : >>> On 12/23/2017 05:48 AM, Greg KH wrote: >

Re: [PATCH v3 00/27] kill devm_ioremap_nocache

+ cris/ia64/mn10300/openrisc maintainers On 2017/12/25 9:09, Yisheng Xie wrote: > hi Christophe and Greg, > > On 2017/12/24 16:55, christophe leroy wrote: >> >> >> Le 23/12/2017 à 16:57, Guenter Roeck a écrit : >>> On 12/23/2017 05:48 AM, Greg KH wrote: >

Re: [PATCH v3 18/27] pinctrl: replace devm_ioremap_nocache with devm_ioremap

On 2018/1/2 16:43, Linus Walleij wrote: > On Sat, Dec 23, 2017 at 12:00 PM, Yisheng Xie <xieyishe...@huawei.com> wrote: > >> Default ioremap is ioremap_nocache, so devm_ioremap has the same >> function with devm_ioremap_nocache, which can just be killed to >

Re: [PATCH v3 18/27] pinctrl: replace devm_ioremap_nocache with devm_ioremap

On 2018/1/2 16:43, Linus Walleij wrote: > On Sat, Dec 23, 2017 at 12:00 PM, Yisheng Xie wrote: > >> Default ioremap is ioremap_nocache, so devm_ioremap has the same >> function with devm_ioremap_nocache, which can just be killed to >> save the size of devres.o >&g

Re: [PATCH v3 06/27] gpio: replace devm_ioremap_nocache with devm_ioremap

On 2018/1/2 16:41, Linus Walleij wrote: > On Sat, Dec 23, 2017 at 11:58 AM, Yisheng Xie <xieyishe...@huawei.com> wrote: > >> Default ioremap is ioremap_nocache, so devm_ioremap has the same >> function with devm_ioremap_nocache, which can just be killed to >

Re: [PATCH v3 06/27] gpio: replace devm_ioremap_nocache with devm_ioremap

On 2018/1/2 16:41, Linus Walleij wrote: > On Sat, Dec 23, 2017 at 11:58 AM, Yisheng Xie wrote: > >> Default ioremap is ioremap_nocache, so devm_ioremap has the same >> function with devm_ioremap_nocache, which can just be killed to >> save the size of devres.o >&g

Re: [v3,21/27] watchdog: replace devm_ioremap_nocache with devm_ioremap

Hi Guenter, On 2017/12/27 1:28, Guenter Roeck wrote: > On Sat, Dec 23, 2017 at 07:01:37PM +0800, Yisheng Xie wrote: >> > Default ioremap is ioremap_nocache, so devm_ioremap has the same >> > function with devm_ioremap_nocache, which can just be killed to >>

Re: [v3,21/27] watchdog: replace devm_ioremap_nocache with devm_ioremap

Hi Guenter, On 2017/12/27 1:28, Guenter Roeck wrote: > On Sat, Dec 23, 2017 at 07:01:37PM +0800, Yisheng Xie wrote: >> > Default ioremap is ioremap_nocache, so devm_ioremap has the same >> > function with devm_ioremap_nocache, which can just be killed to >>

Re: [PATCH v3 27/27] devres: kill devm_ioremap_nocache

On 2017/12/23 21:45, Greg KH wrote: > On Sat, Dec 23, 2017 at 07:02:59PM +0800, Yisheng Xie wrote: >> --- a/lib/devres.c >> +++ b/lib/devres.c >> @@ -44,35 +44,6 @@ void __iomem *devm_ioremap(struct device *dev, >> resource_size_t offset, >

Re: [PATCH v3 27/27] devres: kill devm_ioremap_nocache

On 2017/12/23 21:45, Greg KH wrote: > On Sat, Dec 23, 2017 at 07:02:59PM +0800, Yisheng Xie wrote: >> --- a/lib/devres.c >> +++ b/lib/devres.c >> @@ -44,35 +44,6 @@ void __iomem *devm_ioremap(struct device *dev, >> resource_size_t offset, >

Re: [PATCH v3 08/27] PCI: replace devm_ioremap_nocache with devm_ioremap

Hi Fabio, Thanks for you reviewing and comment! On 2017/12/23 20:23, Fabio Estevam wrote: > On Sat, Dec 23, 2017 at 8:58 AM, Yisheng Xie <xieyishe...@huawei.com> wrote: >> Default ioremap is ioremap_nocache, so devm_ioremap has the same >> function with devm_ioremap_no

Re: [PATCH v3 08/27] PCI: replace devm_ioremap_nocache with devm_ioremap

Hi Fabio, Thanks for you reviewing and comment! On 2017/12/23 20:23, Fabio Estevam wrote: > On Sat, Dec 23, 2017 at 8:58 AM, Yisheng Xie wrote: >> Default ioremap is ioremap_nocache, so devm_ioremap has the same >> function with devm_ioremap_nocache, which can just be killed to &

Re: [PATCH v3 00/27] kill devm_ioremap_nocache

On 2017/12/24 17:05, christophe leroy wrote: > > > Le 23/12/2017 à 14:48, Greg KH a écrit : >> On Sat, Dec 23, 2017 at 06:55:25PM +0800, Yisheng Xie wrote: >>> Hi all, >>> >>> When I tried to use devm_ioremap function and review related code, I found

Re: [PATCH v3 00/27] kill devm_ioremap_nocache

On 2017/12/24 17:05, christophe leroy wrote: > > > Le 23/12/2017 à 14:48, Greg KH a écrit : >> On Sat, Dec 23, 2017 at 06:55:25PM +0800, Yisheng Xie wrote: >>> Hi all, >>> >>> When I tried to use devm_ioremap function and review related code, I found

Re: [PATCH v3 00/27] kill devm_ioremap_nocache

hi Christophe and Greg, On 2017/12/24 16:55, christophe leroy wrote: > > > Le 23/12/2017 à 16:57, Guenter Roeck a écrit : >> On 12/23/2017 05:48 AM, Greg KH wrote: >>> On Sat, Dec 23, 2017 at 06:55:25PM +0800, Yisheng Xie wrote: >>>> Hi all, >>>>

Re: [PATCH v3 00/27] kill devm_ioremap_nocache

hi Christophe and Greg, On 2017/12/24 16:55, christophe leroy wrote: > > > Le 23/12/2017 à 16:57, Guenter Roeck a écrit : >> On 12/23/2017 05:48 AM, Greg KH wrote: >>> On Sat, Dec 23, 2017 at 06:55:25PM +0800, Yisheng Xie wrote: >>>> Hi all, >>>>

[PATCH v3 27/27] devres: kill devm_ioremap_nocache

Now, nobody use devm_ioremap_nocache anymore, can it can just be removed. After this patch the size of devres.o will be reduced from 20304 bytes to 18992 bytes. Suggested-by: Greg KH <gre...@linuxfoundation.org> Signed-off-by: Yisheng Xie <xieyishe...@huawei.com> --- Documentation/

[PATCH v3 27/27] devres: kill devm_ioremap_nocache

Now, nobody use devm_ioremap_nocache anymore, can it can just be removed. After this patch the size of devres.o will be reduced from 20304 bytes to 18992 bytes. Suggested-by: Greg KH Signed-off-by: Yisheng Xie --- Documentation/driver-model/devres.txt | 1 - include/linux/io.h

[PATCH v3 26/27] ethernet: replace devm_ioremap_nocache with devm_ioremap

devm_ioremap_nocache. Cc: Jakub Kicinski <jakub.kicin...@netronome.com> Cc: "David S. Miller" <da...@davemloft.net> Cc: nios2-...@lists.rocketboards.org Cc: net...@vger.kernel.org Signed-off-by: Yisheng Xie <xieyishe...@huawei.com> --- drivers/net/ethernet/altera/altera_tse

[PATCH v3 26/27] ethernet: replace devm_ioremap_nocache with devm_ioremap

devm_ioremap_nocache. Cc: Jakub Kicinski Cc: "David S. Miller" Cc: nios2-...@lists.rocketboards.org Cc: net...@vger.kernel.org Signed-off-by: Yisheng Xie --- drivers/net/ethernet/altera/altera_tse_main.c | 3 +-- drivers/net/ethernet/ethoc.c | 8 drivers/ne

[PATCH v3 24/27] can: replace devm_ioremap_nocache with devm_ioremap

devm_ioremap_nocache. Cc: Wolfgang Grandegger <w...@grandegger.com> Cc: Marc Kleine-Budde <m...@pengutronix.de> Cc: linux-...@vger.kernel.org Cc: net...@vger.kernel.org Signed-off-by: Yisheng Xie <xieyishe...@huawei.com> --- drivers/net/can/sja1000/sja1000_platform.c | 4 ++--

[PATCH v3 25/27] wireless: replace devm_ioremap_nocache with devm_ioremap

devm_ioremap_nocache. Cc: Kalle Valo <kv...@codeaurora.org> Cc: linux-wirel...@vger.kernel.org Cc: net...@vger.kernel.org Signed-off-by: Yisheng Xie <xieyishe...@huawei.com> --- drivers/net/wireless/ath/ath9k/ahb.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/

[PATCH v3 24/27] can: replace devm_ioremap_nocache with devm_ioremap

devm_ioremap_nocache. Cc: Wolfgang Grandegger Cc: Marc Kleine-Budde Cc: linux-...@vger.kernel.org Cc: net...@vger.kernel.org Signed-off-by: Yisheng Xie --- drivers/net/can/sja1000/sja1000_platform.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/net/can/sja1000

[PATCH v3 25/27] wireless: replace devm_ioremap_nocache with devm_ioremap

devm_ioremap_nocache. Cc: Kalle Valo Cc: linux-wirel...@vger.kernel.org Cc: net...@vger.kernel.org Signed-off-by: Yisheng Xie --- drivers/net/wireless/ath/ath9k/ahb.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/wireless/ath/ath9k/ahb.c b/drivers/net/wireless/ath

[PATCH v3 23/27] MIPS: pci: replace devm_ioremap_nocache with devm_ioremap

devm_ioremap_nocache. Cc: Ralf Baechle <r...@linux-mips.org> Cc: linux-m...@linux-mips.org Signed-off-by: Yisheng Xie <xieyishe...@huawei.com> --- arch/mips/pci/pci-ar2315.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/arch/mips/pci/pci-ar2315.c b/arch/mips/pci/

[PATCH v3 23/27] MIPS: pci: replace devm_ioremap_nocache with devm_ioremap

devm_ioremap_nocache. Cc: Ralf Baechle Cc: linux-m...@linux-mips.org Signed-off-by: Yisheng Xie --- arch/mips/pci/pci-ar2315.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/arch/mips/pci/pci-ar2315.c b/arch/mips/pci/pci-ar2315.c index b4fa641..8f3baf1 100644 --- a/arch/mips

[PATCH v3 20/27] regulator: replace devm_ioremap_nocache with devm_ioremap

devm_ioremap_nocache. Cc: Liam Girdwood <lgirdw...@gmail.com> Cc: Mark Brown <broo...@kernel.org> Signed-off-by: Yisheng Xie <xieyishe...@huawei.com> --- drivers/regulator/ti-abb-regulator.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/drivers/regulator/

[PATCH v3 21/27] watchdog: replace devm_ioremap_nocache with devm_ioremap

devm_ioremap_nocache. Cc: Wim Van Sebroeck <w...@iguana.be> Cc: Guenter Roeck <li...@roeck-us.net> Cc: linux-watch...@vger.kernel.org Signed-off-by: Yisheng Xie <xieyishe...@huawei.com> --- drivers/watchdog/bcm63xx_wdt.c | 4 ++-- drivers/watchdog/rc32434_wdt.c | 4 ++-- 2 files cha

[PATCH v3 20/27] regulator: replace devm_ioremap_nocache with devm_ioremap

devm_ioremap_nocache. Cc: Liam Girdwood Cc: Mark Brown Signed-off-by: Yisheng Xie --- drivers/regulator/ti-abb-regulator.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/drivers/regulator/ti-abb-regulator.c b/drivers/regulator/ti-abb-regulator.c index d2f9942..5cec760

[PATCH v3 21/27] watchdog: replace devm_ioremap_nocache with devm_ioremap

devm_ioremap_nocache. Cc: Wim Van Sebroeck Cc: Guenter Roeck Cc: linux-watch...@vger.kernel.org Signed-off-by: Yisheng Xie --- drivers/watchdog/bcm63xx_wdt.c | 4 ++-- drivers/watchdog/rc32434_wdt.c | 4 ++-- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/watchdog

[PATCH v3 22/27] tools/testing/nvdimm: replace devm_ioremap_nocache with devm_ioremap

devm_ioremap_nocache. Cc: Dan Williams <dan.j.willi...@intel.com> Cc: Andy Shevchenko <andriy.shevche...@linux.intel.com> Signed-off-by: Yisheng Xie <xieyishe...@huawei.com> --- tools/testing/nvdimm/Kbuild | 2 +- tools/testing/nvdimm/test/iomap.c | 2 +- 2 files changed,

[PATCH v3 22/27] tools/testing/nvdimm: replace devm_ioremap_nocache with devm_ioremap

devm_ioremap_nocache. Cc: Dan Williams Cc: Andy Shevchenko Signed-off-by: Yisheng Xie --- tools/testing/nvdimm/Kbuild | 2 +- tools/testing/nvdimm/test/iomap.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/tools/testing/nvdimm/Kbuild b/tools/testing/nvdimm/Kbuild index

[PATCH v3 18/27] pinctrl: replace devm_ioremap_nocache with devm_ioremap

devm_ioremap_nocache. Cc: Linus Walleij <linus.wall...@linaro.org> Cc: linux-g...@vger.kernel.org Signed-off-by: Yisheng Xie <xieyishe...@huawei.com> --- drivers/pinctrl/bcm/pinctrl-ns2-mux.c | 2 +- drivers/pinctrl/bcm/pinctrl-nsp-mux.c | 4 ++-- drivers/pinctrl/freescale/

[PATCH v3 18/27] pinctrl: replace devm_ioremap_nocache with devm_ioremap

devm_ioremap_nocache. Cc: Linus Walleij Cc: linux-g...@vger.kernel.org Signed-off-by: Yisheng Xie --- drivers/pinctrl/bcm/pinctrl-ns2-mux.c | 2 +- drivers/pinctrl/bcm/pinctrl-nsp-mux.c | 4 ++-- drivers/pinctrl/freescale/pinctrl-imx1-core.c | 2 +- drivers/pinctrl/pinctrl-amd.c

[PATCH v3 19/27] drm: replace devm_ioremap_nocache with devm_ioremap

devm_ioremap_nocache. Cc: Daniel Vetter <daniel.vet...@intel.com> Cc: Jani Nikula <jani.nik...@linux.intel.com> Cc: Sean Paul <seanp...@chromium.org> Cc: David Airlie <airl...@linux.ie> Cc: dri-de...@lists.freedesktop.org Signed-off-by: Yisheng Xie <xieyishe...@huawei.com>

[PATCH v3 19/27] drm: replace devm_ioremap_nocache with devm_ioremap

devm_ioremap_nocache. Cc: Daniel Vetter Cc: Jani Nikula Cc: Sean Paul Cc: David Airlie Cc: dri-de...@lists.freedesktop.org Signed-off-by: Yisheng Xie --- drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_drv.c | 2 +- drivers/gpu/drm/msm/msm_drv.c | 2 +- drivers/gpu/drm/sti/sti_dvo.c

[PATCH v3 15/27] dmaengine: replace devm_ioremap_nocache with devm_ioremap

devm_ioremap_nocache. Cc: Vinod Koul <vinod.k...@intel.com> Cc: Dan Williams <dan.j.willi...@intel.com> Cc: dmaeng...@vger.kernel.org Signed-off-by: Yisheng Xie <xieyishe...@huawei.com> --- drivers/dma/altera-msgdma.c | 3 +-- drivers/dma/sprd-dma.c | 4 ++-- 2 files changed,

[PATCH v3 16/27] ata: replace devm_ioremap_nocache with devm_ioremap

devm_ioremap_nocache. Cc: Tejun Heo <t...@kernel.org> Cc: linux-...@vger.kernel.org Signed-off-by: Yisheng Xie <xieyishe...@huawei.com> --- drivers/ata/pata_arasan_cf.c | 3 +-- drivers/ata/pata_octeon_cf.c | 9 - drivers/ata/pata_rb532_cf.c | 2 +- 3 files changed, 6 inse

[PATCH v3 15/27] dmaengine: replace devm_ioremap_nocache with devm_ioremap

devm_ioremap_nocache. Cc: Vinod Koul Cc: Dan Williams Cc: dmaeng...@vger.kernel.org Signed-off-by: Yisheng Xie --- drivers/dma/altera-msgdma.c | 3 +-- drivers/dma/sprd-dma.c | 4 ++-- 2 files changed, 3 insertions(+), 4 deletions(-) diff --git a/drivers/dma/altera-msgdma.c b/drivers/dma

[PATCH v3 16/27] ata: replace devm_ioremap_nocache with devm_ioremap

devm_ioremap_nocache. Cc: Tejun Heo Cc: linux-...@vger.kernel.org Signed-off-by: Yisheng Xie --- drivers/ata/pata_arasan_cf.c | 3 +-- drivers/ata/pata_octeon_cf.c | 9 - drivers/ata/pata_rb532_cf.c | 2 +- 3 files changed, 6 insertions(+), 8 deletions(-) diff --git a/drivers/ata

[PATCH v3 17/27] irqchip: replace devm_ioremap_nocache with devm_ioremap

devm_ioremap_nocache. Cc: Thomas Gleixner <t...@linutronix.de> Cc: Jason Cooper <ja...@lakedaemon.net> Cc: Marc Zyngier <marc.zyng...@arm.com> Signed-off-by: Yisheng Xie <xieyishe...@huawei.com> --- drivers/irqchip/irq-renesas-intc-irqpin.c | 4 ++-- 1 file changed, 2 in

[PATCH v3 17/27] irqchip: replace devm_ioremap_nocache with devm_ioremap

devm_ioremap_nocache. Cc: Thomas Gleixner Cc: Jason Cooper Cc: Marc Zyngier Signed-off-by: Yisheng Xie --- drivers/irqchip/irq-renesas-intc-irqpin.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/irqchip/irq-renesas-intc-irqpin.c b/drivers/irqchip/irq-renesas-intc

[PATCH v3 14/27] mtd: nand: replace devm_ioremap_nocache with devm_ioremap

devm_ioremap_nocache. Cc: Boris Brezillon <boris.brezil...@free-electrons.com> Cc: Richard Weinberger <rich...@nod.at> Cc: Marek Vasut <marek.va...@gmail.com> Cc: Cyrille Pitchen <cyrille.pitc...@wedev4u.fr> Cc: linux-...@lists.infradead.org Signed-off-by: Yisheng Xie

<    1   2   3   4   5   6   7   8   9   >