On 2016/7/19 22:14, Vlastimil Babka wrote:
> On 07/19/2016 03:48 PM, Xishi Qiu wrote:
>> On 2016/7/19 21:17, Vlastimil Babka wrote:
>>
>>> On 07/19/2016 02:43 PM, Yisheng Xie wrote:
>>>> hi all,
>>>> I'm getting a 2-order page allocation
On 2016/9/13 19:18, Matt Fleming wrote:
> On Wed, 07 Sep, at 07:15:48PM, Yisheng Xie wrote:
>>
>>
>> On 2016/9/5 21:57, Matt Fleming wrote:
>>> On Fri, 02 Sep, at 11:26:18AM, Will Deacon wrote:
>>>> On Fri, Sep 02, 2016 at 06:18:39PM +0800,
hi all,
When I enable NUMA in BIOS for arm64, it failed to boot on
v4.8-rc4-162-g071e31e.
For the crash log, it seems caused by error number of cpumask.
Any ideas about it?
Thanks.
The related config and detail dmesg can be seen in the attachment.
--- crash messages ---
[1.279155] -
On 2016/9/19 22:07, Mark Rutland wrote:
> [adding LAKML, arm64 maintainers]
>
> On Mon, Sep 19, 2016 at 09:05:26PM +0800, Yisheng Xie wrote:
>> hi all,
>
> Can you modify the warning in cpumask.h to dump the bad CPU number? That
> would make it fairly clear if that&#
On 2016/9/20 10:01, Ming Lei wrote:
> On Mon, Sep 19, 2016 at 9:05 PM, Yisheng Xie wrote:
>> hi all,
>> When I enable NUMA in BIOS for arm64, it failed to boot on
>> v4.8-rc4-162-g071e31e.
>> For the crash log, it seems caused by error number of cpumask.
>>
2.575210] [] cpuhp_thread_fun+0x13c/0x148
> [2.581027] [] smpboot_thread_fn+0x19c/0x1a8
> [2.586929] [] kthread+0xdc/0xf0
> [2.591776] [] ret_from_fork+0x10/0x50
> [2.597147] Code: b00057e1 91304021 91005021 b8626822 (b8606821)
> [ 2.603464] ---[ end trace 58c0cd
There's an early memmap leak in efi_init error path, fix it.
Signed-off-by: Yisheng Xie
---
drivers/firmware/efi/arm-init.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/drivers/firmware/efi/arm-init.c b/drivers/firmware/efi/arm-init.c
index 9f8d996..7c2fe57 1
g the function and out-lining its contents.
>
> Suggested-by: Robert Richter
> fixes: 1a2db300348b ("arm64, numa: Add NUMA support for arm64 platforms.")
> Cc: # 4.7.x-
> Signed-off-by: David Daney
> ---
Tested-by: Yisheng Xie
Thanks.
> arch/arm64/kern
On 2016/9/5 21:57, Matt Fleming wrote:
> On Fri, 02 Sep, at 11:26:18AM, Will Deacon wrote:
>> On Fri, Sep 02, 2016 at 06:18:39PM +0800, Xie Yisheng wrote:
>>> From: Yisheng Xie
>>>
>>> diff --git a/drivers/firmware/efi/arm-init.c
>>> b/drivers/firmw
is common issue, and I also can reproduce at certain config on X86-64
e.g. CONFIG_NODES_SHIFT=3 and have 8 node in the system.
IMO, if nbits=4, 0x0 or 0x10, 0xFF..F0 should not a subset of anything, so
following
patch may fix this problem:
From: Yisheng Xie
Date: Tue, 17 Oct 2017 20:53:55
node_empty check in
SYSC_migrate_pages.
Reported-by: Tan Xiaojun
Signed-off-by: Yisheng Xie
---
mm/mempolicy.c | 5 +
1 file changed, 5 insertions(+)
diff --git a/mm/mempolicy.c b/mm/mempolicy.c
index a2af6d5..1dfd3cc 100644
--- a/mm/mempolicy.c
+++ b/mm/mempolicy.c
@@ -1388,6 +1388,11 @@ static
Hi Will,
On 2017/10/17 21:19, Yisheng Xie wrote:
> Hi Will,
>
> On 2017/10/17 17:23, Will Deacon wrote:
>> On Tue, Oct 17, 2017 at 10:58:53AM +0800, Tan Xiaojun wrote:
>>> I'm not sure if this is the problem on arm64 numa. What do you think ?
>>> By the w
Hi Vlastimil,
Thanks for comment!
On 2017/10/31 16:34, Vlastimil Babka wrote:
> On 10/27/2017 12:14 PM, Yisheng Xie wrote:
>> It appears there is a nodemask miscalculation in the get_nodes()
>> function in mm/mempolicy.c. This bug has two effects:
>>
>> 1. It is impos
After commit 9e8e865bbe29 ("arm64: unify idmap removal"), we no need to
flush tlb in suspend.c, so the included file tlbflush.h can be removed.
Signed-off-by: Yisheng Xie
---
arch/arm64/kernel/suspend.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/arch/arm64/kernel/suspend.c b/
megasas_alloc_cmds is to alloc cmd_list of instance instead of fusion,
and fusion is useless in this function. Just remove it.
Signed-off-by: Yisheng Xie
---
drivers/scsi/megaraid/megaraid_sas_base.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/drivers/scsi/megaraid/megaraid_sas_base.c
ned-off-by: Yisheng Xie
---
drivers/scsi/megaraid/megaraid_sas_fusion.c | 17 -
1 file changed, 16 insertions(+), 1 deletion(-)
diff --git a/drivers/scsi/megaraid/megaraid_sas_fusion.c
b/drivers/scsi/megaraid/megaraid_sas_fusion.c
index 11bd2e6..9a1be45 100644
--- a/drivers/scs
Hi Vlastimil,
On 2017/10/31 17:46, Vlastimil Babka wrote:
> +CC Andi and Christoph
>
> On 10/27/2017 12:14 PM, Yisheng Xie wrote:
>> As manpage of migrate_pages, the errno should be set to EINVAL when none
>> of the specified nodes contain memory. However, when new_nodes
Hi Vlastimil,
Thanks for your comment!
On 2017/10/18 15:54, Vlastimil Babka wrote:
> +CC linux-api
>
> On 10/18/2017 03:37 AM, Yisheng Xie wrote:
>> As Xiaojun reported the ltp of migrate_pages01 will failed on ARCH arm64
>> system whoes has 4 nodes[0...3], all have memory a
Hi Vlastimil,
Thanks for you comment!
On 2017/10/18 17:34, Yisheng Xie wrote:
> Hi Vlastimil,
>
> Thanks for your comment!
> On 2017/10/18 15:54, Vlastimil Babka wrote:
>> +CC linux-api
>>
>> On 10/18/2017 03:37 AM, Yisheng Xie wrote:
>>> As Xiaojun r
We have already checked whether maxnode is a page worth of bits, by:
maxnode > PAGE_SIZE*BITS_PER_BYTE
So no need to check it once more.
Signed-off-by: Yisheng Xie
---
mm/mempolicy.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/mm/mempolicy.c b/mm/mempolicy.c
index 613e9d0..3b51
, old_nodes, new_nodes, MAX);
The ret will be 0 and no errno is set.
This patch is to add nodes_empty check to fix above case.
Signed-off-by: Yisheng Xie
---
mm/mempolicy.c | 5 +
1 file changed, 5 insertions(+)
diff --git a/mm/mempolicy.c b/mm/mempolicy.c
index 8798ecb..58352cc 100644
ff-by: Yisheng Xie
---
mm/mempolicy.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/mm/mempolicy.c b/mm/mempolicy.c
index a2af6d5..613e9d0 100644
--- a/mm/mempolicy.c
+++ b/mm/mempolicy.c
@@ -1265,7 +1265,6 @@ static int get_nodes(nodemask_t *nodes, const unsigned
long __user *nmask,
uns
ind.2.html
[3] http://man7.org/linux/man-pages/man2/migrate_pages.2.html
Reported-by: Tan Xiaojun
Signed-off-by: Yisheng Xie
---
mm/mempolicy.c | 23 ---
1 file changed, 20 insertions(+), 3 deletions(-)
diff --git a/mm/mempolicy.c b/mm/mempolicy.c
index 3b51bb3..8798ecb 1
m to small
patches for easy to review and discussion. For more detail, please look
into each patches.
Change logs of v2:
* fix get_nodes's mask miscalculation
* remove redundant check in get_nodes
* fix the check of nodemask from user - per Vlastimil
Any comment and complain is welome.
Thanks
what about the Patch 1/2, which is just a minor clean up?
Thanks
Yisheng Xie
On 2017/10/31 17:30, Vlastimil Babka wrote:
> On 10/27/2017 12:14 PM, Yisheng Xie wrote:
>> +/*
>> + * When the user specified more nodes than supported just check
>> + * if the non supported part is all zero.
>> + *
>> + * If maxnode have mo
Hi Andy,
Sorry for late response. I will take your suggestion in next version.
Thanks
Yisheng
On 2018/6/6 13:01, Andy Shevchenko wrote:
> On Wed, Jun 6, 2018 at 5:19 AM, Yisheng Xie wrote:
>> match_string() returns the index of an array for a matching string,
>> which can be
.
Cc: "David S. Miller"
Cc: Anthony Yznaga
Cc: Pavel Tatashin
Cc: sparcli...@vger.kernel.org
Signed-off-by: Yisheng Xie
---
arch/sparc/kernel/setup_64.c | 24 +---
1 file changed, 9 insertions(+), 15 deletions(-)
diff --git a/arch/sparc/kernel/setup_64.c b/arch/sparc/
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)
>>> +{
>>
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
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
hi all,
When doing FUZZ test with strict writes enabled:
- $ 80 00 > /proc/sys/kernel/hung_task_timeout_secs
- $ cat /proc/sys/kernel/hung_task_timeout_secs
- $ 80
I have a silly question about this:
Should we return -EINVAL in this case both for strict writes enabled and
disabled?
Thanks
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,
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
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
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
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/dr
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
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
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 a
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/dr
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
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
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
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 =
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 counterme
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.
>>
>>
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
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
hardware do not support PAN, is SW_TTBR0_PAN is *must* option? or
there maybe a security risk?
Thanks
Yisheng Xie
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)
>
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
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
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
We have brought perf_default_config to the very beginning at main(), so
it no need to call perf_default_config() once more for most of config in
perf-record but only for record.call-graph.
Signed-off-by: Yisheng Xie
---
tools/perf/builtin-record.c | 8 +---
1 file changed, 5 insertions
perf_default_config to the very
beginning at main())
Signed-off-by: Yisheng Xie
---
tools/perf/builtin-top.c | 6 --
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/tools/perf/builtin-top.c b/tools/perf/builtin-top.c
index 35ac016..fd6e238 100644
--- a/tools/perf/builtin-top.c
+++ b/tools/perf
hi Steven and all,
Ping, for month passed, do you have any comment about this patch?
Thanks & Sorry to disturb
Yisheng
On 2018/2/2 10:14, Yisheng Xie wrote:
> clear_ftrace_function is not used outside of ftrace.c and is not help to
> use a function, so nuke it per Steve&#x
In overwrite mode, start will be set to head in perf_mmap__read_init.
Therefore, it is no need to set the start one more in
overwrite_rb_find_range and *start can be used as head instead of
passing head to overwrite_rb_find_range.
Signed-off-by: Yisheng Xie
---
tools/perf/util/mmap.c | 15
Hi Steven,
On 2018/3/7 6:02, Steven Rostedt wrote:
> On Tue, 6 Mar 2018 19:30:28 +0800
> Yisheng Xie wrote:
>
>> hi Steven and all,
>> Ping, for month passed, do you have any comment about this patch?
>
> Sorry you got me in the midst of traveling and other issues.
Hi jirka,
On 2018/3/7 18:21, Jiri Olsa wrote:
> On Tue, Mar 06, 2018 at 07:44:23PM +0800, Yisheng Xie wrote:
>> In overwrite mode, start will be set to head in perf_mmap__read_init.
>> Therefore, it is no need to set the start one more in
>> overwrite_rb_find_range and *star
Hi all,
We meet a problem with linux 3.4, which trigger BUG on lib/prio_tree.c:280!
kernel BUG at
/usr/src/packages/BUILD/kernel-default-3.4.24.25/linux-3.4/lib/prio_tree.c:280!
[...]
Process: grep (pid: 64867, threadinfo: 88000501, task:
8800022d5c80) on CPU: 2
Pid: 64867, comm: gr
Hi Vivek,
On 2018/4/11 13:15, Vivek Gautam wrote:
> Hi Yisheng,
>
>
> On 4/11/2018 6:52 AM, Yisheng Xie wrote:
>> Hi Tomasz,
>>
>> On 2018/4/10 21:14, Tomasz Figa wrote:
>>> Hi Yisheng,
>>>
>>> Sorry, I think we missed your question
Hi Tomasz,
On 2018/4/10 21:14, Tomasz Figa wrote:
> Hi Yisheng,
>
> Sorry, I think we missed your question here.
>
> On Wed, Mar 28, 2018 at 3:12 PM Yisheng Xie wrote:
>
>> Hi Vivek,
>
>> On 2018/3/28 12:37, Vivek Gautam wrote:
>>> Hi Yisheng
>
Hi Arnaldo,
On 2018/3/7 21:58, Arnaldo Carvalho de Melo wrote:
> Em Wed, Mar 07, 2018 at 11:21:57AM +0100, Jiri Olsa escreveu:
>> On Tue, Mar 06, 2018 at 07:44:23PM +0800, Yisheng Xie wrote:
>>> In overwrite mode, start will be set to head in perf_mmap__read_init.
>>>
Hi Mike,
On 2018/3/8 7:59, Mike Kravetz wrote:
> A vma with vm_pgoff large enough to overflow a loff_t type when
> converted to a byte offset can be passed via the remap_file_pages
> system call. The hugetlbfs mmap routine uses the byte offset to
> calculate reservations and file size.
>
> A seq
whether it is MPOL_F_LOCAL or not.
Reported-by: Alexander Potapenko
Signed-off-by: Yisheng Xie
---
mm/mempolicy.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/mm/mempolicy.c b/mm/mempolicy.c
index d879f1d..641545e 100644
--- a/mm/mempolicy.c
+++ b/mm/mempolicy.c
@@ -2124,6 +2124,9 @@ bool
Hi Alexander ,
On 2018/3/9 18:49, Alexander Potapenko wrote:
> On Fri, Mar 9, 2018 at 6:21 AM, Yisheng Xie wrote:
>> Alexander reported an use of uninitialized memory in __mpol_equal(),
>> which is caused by incorrect use of preferred_node.
>>
>> When mempolicy in mod
In overwrite mode, start will be set to head in perf_mmap__read_init.
Therefore, it is no need to set the start one more in
overwrite_rb_find_range and *start can be used as head instead of
passing head to overwrite_rb_find_range.
Signed-off-by: Yisheng Xie
---
v2:
- rebase to perf/core
tools
When use --quiet to disable log, we will set quiet first, then check the
quiet to decide whether need to call perf_quiet_option, so no need to set
the quiet once more.
Signed-off-by: Yisheng Xie
---
- This is a new add-on patch
tools/perf/util/debug.c | 1 -
1 file changed, 1 deletion
match_string() returns the index of an array for a matching string,
which can be used instead of open coded variant.
Cc: Liam Girdwood
Cc: Mark Brown
Cc: Jaroslav Kysela
Cc: Takashi Iwai
Cc: alsa-de...@alsa-project.org
Signed-off-by: Yisheng Xie
---
v2:
- split ret=xxx and move assignment
match_string() returns the index of an array for a matching string,
which can be used instead of open coded variant.
Cc: Benjamin Herrenschmidt
Cc: Paul Mackerras
Cc: Michael Ellerman
Cc: linuxppc-...@lists.ozlabs.org
Signed-off-by: Yisheng Xie
---
arch/powerpc/xmon/xmon.c | 23
match_string() returns the index of an array for a matching string,
which can be used instead of open coded variant.
Cc: Kent Overstreet
Cc: linux-bca...@vger.kernel.org
Signed-off-by: Yisheng Xie
---
drivers/md/bcache/util.c | 9 ++---
1 file changed, 2 insertions(+), 7 deletions
match_string() returns the index of an array for a matching string,
which can be used instead of open coded variant.
Cc: Clemens Ladisch
Cc: Jaroslav Kysela
Cc: Takashi Iwai
Cc: alsa-de...@alsa-project.org
Signed-off-by: Yisheng Xie
---
v2:
- do not change the type of i - per Andy
sound
match_string() returns the index of an array for a matching string,
which can be used instead of open coded variant.
Reviewed-by: Andy Shevchenko
Cc: Liam Girdwood
Cc: Mark Brown
Cc: Jaroslav Kysela
Cc: Takashi Iwai
Cc: alsa-de...@alsa-project.org
Signed-off-by: Yisheng Xie
---
v2:
- add
match_string() returns the index of an array for a matching string,
which can be used instead of open coded variant.
Cc: Liam Girdwood
Cc: Mark Brown
Cc: Jaroslav Kysela
Cc: Takashi Iwai
Cc: alsa-de...@alsa-project.org
Signed-off-by: Yisheng Xie
---
v2:
- split ret= and move assignment
match_string() returns the index of an array for a matching string,
which can be used instead of open coded variant.
Cc: "David S. Miller"
Cc: Anthony Yznaga
Cc: Pavel Tatashin
Cc: sparcli...@vger.kernel.org
Signed-off-by: Yisheng Xie
---
arch/sparc/kernel/setup_64.c | 7 +++--
match_string() returns the index of an array for a matching string,
which can be used instead of open coded variant.
Reviewed-by: Andy Shevchenko
Cc: Stuart Yoder
Cc: Laurentiu Tudor
Signed-off-by: Yisheng Xie
---
v2:
- add Reviewed-by tag.
drivers/bus/fsl-mc/fsl-mc-allocator.c | 24
match_string() returns the index of an array for a matching string,
which can be used instead of open coded variant.
Reviewed-by: Andy Shevchenko
Cc: Srinivas Pandruvada
Cc: Len Brown
Cc: "Rafael J. Wysocki"
Cc: Viresh Kumar
Cc: linux...@vger.kernel.org
Signed-off-by: Yisheng X
match_string() returns the index of an array for a matching string,
which can be used instead of open coded variant.
Cc: Ganesh Goudar
Cc: net...@vger.kernel.org
Signed-off-by: Yisheng Xie
---
v2:
- no change from v1.
drivers/net/ethernet/chelsio/cxgb4/cudbg_lib.c | 14 --
1 file
match_string() returns the index of an array for a matching string,
which can be used instead of open coded variant.
Cc: Ingo Molnar
Cc: Peter Zijlstra
Signed-off-by: Yisheng Xie
---
v2:
- rename i to ret to show the change in returned value meaning - per Andy
kernel/sched/debug.c | 31
: linux-arm-ker...@lists.infradead.org
Signed-off-by: Yisheng Xie
---
v2:
- donot change the type of i - per Andy
drivers/clk/bcm/clk-bcm2835.c | 13 ++---
1 file changed, 6 insertions(+), 7 deletions(-)
diff --git a/drivers/clk/bcm/clk-bcm2835.c b/drivers/clk/bcm/clk-bcm2835.c
index
Cc: linux-security-mod...@vger.kernel.org
Signed-off-by: Yisheng Xie
---
v2:
- add Reviewed-by tag.
security/integrity/ima/ima_main.c | 11 ---
1 file changed, 4 insertions(+), 7 deletions(-)
diff --git a/security/integrity/ima/ima_main.c
b/security/integrity/ima/ima_main.c
index 74d0bd
match_string() returns the index of an array for a matching string,
which can be used instead of open coded variant.
Cc: Michael Turquette
Cc: Stephen Boyd
Cc: linux-...@vger.kernel.org
Signed-off-by: Yisheng Xie
---
v2:
- leave second parameter on the first line - per Andy
drivers/clk
Patches which Andy already sent out, or maintainer already
picked up.
- Add Reviewed-by or Acked-by tags for some patchs.
- Add some new patches[19-21].
Yisheng Xie (21):
usb: phy: use match_string() helper
mfd: omap-usb-host: use match_string() helper
Staging: gdm724x: use match_string
match_string() returns the index of an array for a matching string,
which can be used instead of open coded variant.
Acked-by: Tony Lindgren
Reviewed-by: Andy Shevchenko
Cc: Tony Lindgren
Cc: Lee Jones
Cc: linux-o...@vger.kernel.org
Signed-off-by: Yisheng Xie
---
v2:
- add Acked-by and
match_string() returns the index of an array for a matching string,
which can be used instead of open coded variant.
Cc: Greg Kroah-Hartman
Cc: Quytelda Kahja
Cc: de...@driverdev.osuosl.org
Signed-off-by: Yisheng Xie
---
v2:
- const DRIVER_STRING instead - per Andy
drivers/staging/gdm724x
Hi Greg,
On 2018/5/31 19:44, Greg Kroah-Hartman wrote:
> On Thu, May 31, 2018 at 07:11:08PM +0800, Yisheng Xie wrote:
>> match_string() returns the index of an array for a matching string,
>> which can be used instead of open coded variant.
>>
>> Cc: Greg Kroah-Har
+ Mark
On 2018/5/31 19:11, Yisheng Xie wrote:
> Andy introduce helper function match_string() which can be used to return
> the index of array for a matching string. so we can use it in many places
> instead of open coded variant.
>
> I just try to make this API be used more com
Hi Mark,
On 2018/5/31 19:49, Mark Brown wrote:
> On Thu, May 31, 2018 at 07:11:21PM +0800, Yisheng Xie wrote:
>> match_string() returns the index of an array for a matching string,
>> which can be used instead of open coded variant.
>
> I don't have either the cover
Hi Mark,
On 2018/6/1 0:02, Mark Brown wrote:
> On Thu, May 31, 2018 at 08:25:39PM +0800, Yisheng Xie wrote:
>
>> Anyway, Here is the cover letter of v1 and I have add v2's cover letter to
>> you:
>> https://lkml.org/lkml/2018/5/21/303
>
>> Each patch i
Hi Coly,
On 2018/6/1 11:45, Coly Li wrote:
> On 2018/5/31 7:11 PM, Yisheng Xie wrote:
>> match_string() returns the index of an array for a matching string,
>> which can be used instead of open coded variant.
>>
>> Cc: Kent Overstreet
>> Cc: linux-bca...@
Hi Andy,
On 2018/6/1 19:34, Andy Shevchenko wrote:
> On Thu, May 31, 2018 at 2:11 PM, Yisheng Xie wrote:
>> match_string() returns the index of an array for a matching string,
>> which can be used instead of open coded variant.
>
>> @@ -512,10 +512,9 @@ sta
match_string() returns the index of an array for a matching string,
which can be used to simplify the code.
Signed-off-by: Yisheng Xie
---
kernel/trace/trace.c | 15 +--
1 file changed, 5 insertions(+), 10 deletions(-)
diff --git a/kernel/trace/trace.c b/kernel/trace/trace.c
index
Hi Jani,
On 2018/5/22 16:36, Jani Nikula wrote:
> On Tue, 22 May 2018, Andy Shevchenko wrote:
>> On Mon, May 21, 2018 at 2:57 PM, Yisheng Xie wrote:
>>> match_string() returns the index of an array for a matching string,
>>> which can be used intead of open
Hi,
On 2018/5/22 18:02, Greg Kroah-Hartman wrote:
> On Tue, May 22, 2018 at 01:20:01AM +0300, Andy Shevchenko wrote:
>> On Mon, May 21, 2018 at 2:57 PM, Yisheng Xie wrote:
>>> match_string() returns the index of an array for a matching string,
>>> which can be used i
Hi Andy,
On 2018/5/22 6:13, Andy Shevchenko wrote:
> On Mon, May 21, 2018 at 2:58 PM, Yisheng Xie wrote:
>> match_string() returns the index of an array for a matching string,
>> which can be used intead of open coded variant.
>>
>> Cc: Ingo Molnar
>> Cc:
Hi Andy,
On 2018/5/22 6:24, Andy Shevchenko wrote:
> On Mon, May 21, 2018 at 2:58 PM, Yisheng Xie wrote:
>> match_string() returns the index of an array for a matching string,
>> which can be used intead of open coded variant.
>
>> static int max98088_get_channel(
Hi Andy,
On 2018/5/22 6:00, Andy Shevchenko wrote:
> On Mon, May 21, 2018 at 2:57 PM, Yisheng Xie wrote:
>> match_string() returns the index of an array for a matching string,
>> which can be used intead of open coded variant.
>
>> + i = match_string((const char
copy_from_user out of ashmem_mutex.
[1] https://www.spinics.net/lists/kernel/msg2733200.html
Fixes: ce8a3a9e76d0 (staging: android: ashmem: Fix a race condition in pin
ioctls)
Reported-by: syzbot+d7a918a7a8e1c952b...@syzkaller.appspotmail.com
Signed-off-by: Yisheng Xie
---
drivers/staging
orks abnomal. Fix
it by adding check for syzbot.
Signed-off-by: Yisheng Xie
---
scripts/checkpatch.pl | 6 +-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl
index 3d40403..19f3e26 100755
--- a/scripts/checkpatch.pl
+++ b/scripts/chec
1 - 100 of 403 matches
Mail list logo
