在 2020/5/21 22:15, Arnaldo Carvalho de Melo 写道:
Em Thu, May 21, 2020 at 09:32:16PM +0800, Wei Li escreveu:
From: Li Bin
Fix leak of memory pointed to by t.sib_thr and t.sib_core in
svg_build_topology_map.
Signed-off-by: Li Bin
---
tools/perf/util/svghelper.c | 10 +++---
1 file chan
在 2018/4/19 10:25, Andy Lutomirski 写道:
On Apr 18, 2018, at 9:06 PM, Li Bin wrote:
The member auxv in prctl_mm_map structure which be shared with
userspace is pointer type, but the kernel supporting COMPAT didn't
handle it. This patch fix the compat handling for prctl syscall.
I would propos
在 2018/4/12 20:20, Steven Rostedt 写道:
On Thu, 12 Apr 2018 14:11:45 +0200
Peter Zijlstra wrote:
On Thu, Apr 12, 2018 at 06:58:53PM +0800, Li Bin wrote:
Li Bin (1):
sched/deadline.c: pick and check task if double_lock_balance() unlock
the rq
Zhou Chengming (1):
sched/rt.c: pick an
在 2018/4/11 18:26, Peter Zijlstra 写道:
On Tue, Apr 10, 2018 at 06:05:46PM -0400, Steven Rostedt wrote:
Peter,
Going through my inbox, I stumbled across this one. And it doesn't
appear to be addressed.
I think this patch is a reasonable solution.
Urgh, yeah, also seem to have forgotten about
Hi Takashi,
> -Original Message-
> From: Takashi Iwai [mailto:ti...@suse.de]
> Sent: Monday, March 14, 2016 3:23 PM
> To: kernel test robot
> Cc: l...@01.org; LKML; Wu, Fengguang; Yang, Libin; Lin, Mengdong
> Subject: Re: [lkp] [ALSA] ca53fbf3a6: WARNING: CPU: 0 PID:
same issue. I will check it
after my holiday.
Regards,
Libin
> -Original Message-
> From: Takashi Iwai [mailto:ti...@suse.de]
> Sent: Tuesday, February 09, 2016 7:44 PM
> To: Martin Kepplinger
> Cc: david.hennings...@canonical.com; Lu, Han; Yang, Libin;
> tred...@nvidia.
Thanks very much to Will and Steve for the wonderful comments, I will modify the
commit message, and remove the misleading comments about module text
disappearing case.
Thanks again,
Li Bin
on 2015/12/3 23:31, Steven Rostedt wrote:
> On Thu, 3 Dec 2015 15:09:26 +
> Will Deacon wrote:
>
>> Y
on 2015/12/2 21:16, Will Deacon wrote:
> On Wed, Dec 02, 2015 at 12:36:54PM +, Will Deacon wrote:
>> On Sat, Nov 28, 2015 at 03:50:09PM +0800, Li Bin wrote:
>>> On arm64, kstop_machine which is hugely disruptive to a running
>>> system is not needed to convert nops to ftrace calls or back,
>>
on 2015/12/2 20:36, Will Deacon wrote:
> On Sat, Nov 28, 2015 at 03:50:09PM +0800, Li Bin wrote:
>> On arm64, kstop_machine which is hugely disruptive to a running
>> system is not needed to convert nops to ftrace calls or back,
>> because that modifed code is a single 32bit instructions which
>>
Hi Josh,
on 2015/12/1 9:11, Josh Poimboeuf wrote:
> On Mon, Nov 30, 2015 at 11:54:37AM +0800, Li Bin wrote:
>> There is a potential race as following:
>>
>> CPU0 | CPU1
>> -|---
>> enabled_store() |
Sorry for my mistake. And please ignore this patch.
Thanks,
Li Bin
on 2015/11/30 17:53, Daniel K. wrote:
> On 11/30/2015 09:24 AM, Li Bin wrote:
>> s/ARCH_SUPPORT_FTARCE_OPS/ARCH_SUPPORTS_FTARCE_OPS
> How about s/FTARCE/FTRACE/ as well?
>
>
> Daniel K.
>
>
>
--
To unsubscribe from this list: se
on 2015/11/28 23:58, Steven Rostedt wrote:
> On Sat, 28 Nov 2015 15:50:09 +0800
> Li Bin wrote:
>
>> On arm64, kstop_machine which is hugely disruptive to a running
>> system is not needed to convert nops to ftrace calls or back,
>> because that modifed code is a single 32bit instructions which
In nop_mcount, shdr->sh_offset and welp->r_offset should handle
endianness properly, otherwise it will trigger Segmentation fault
if the recordmcount main and file.o have different endianness.
Cc: # 3.0+
Signed-off-by: Li Bin
---
scripts/recordmcount.h |2 +-
1 files changed, 1 insertions(+
on 2015/10/30 22:35, Sergei Shtylyov wrote:
> Hello.
>
> On 10/30/2015 11:31 AM, Li Bin wrote:
>
>> In nop_mcount, shdr->sh_offset and welp->r_offset should handle
>> endianness properly, otherwise it will trigger Segmentation fault
>> if the recordmcount main and file.o have different endianne
on 2015/10/30 16:31, Li Bin wrote:
> By now, the recordmcount only records the function that in
> following sections:
> .text/.ref.text/.sched.text/.spinlock.text/.irqentry.text/
> .kprobes.text/.text.unlikely
>
> For the function that not in these sections, the call mcount
> will be in place an
on 2015/10/30 16:31, Li Bin wrote:
> In nop_mcount, shdr->sh_offset and welp->r_offset should handle
> endianness properly, otherwise it will trigger Segmentation fault
> if the recordmcount main and file.o have different endianness.
>
For more information, please refer to
https://lkml.
在 2015/10/29 23:09, Will Deacon 写道:
> On Thu, Oct 29, 2015 at 09:35:42PM +0800, kbuild test robot wrote:
>> [auto build test ERROR on arm64/for-next/core -- if it's inappropriate base,
>> please suggest rules for selecting the more suitable base]
>>
>> url:
>> https://github.com/0day-ci/linu
在 2015/10/20 23:32, Catalin Marinas 写道:
On Thu, Oct 15, 2015 at 10:18:12AM -0400, Steven Rostedt wrote:
On Thu, 15 Oct 2015 13:51:33 +0100
Will Deacon wrote:
Is this the same old problem caused by e306dfd06fcb ("ARM64: unwind: Fix
PC calculation")? I've said previously that I'm happy to rev
id_cache(cpu)) & ASID_VERSION_MASK)
get_new_mmu_context(next, cpu);
write_c0_entryhi(cpu_asid(cpu, next));
...
In short, due to the truncation operation caused by inappropriate type
conversion,
making tlb flush failure, causing problems of COW, triggering bus error or
segment
From: Li Bin
The variable delay_time as a module parameter can be set a value
exceeding the limit of the udelay function. This patch checks the
delay_time value to select mdelay() or udelay().
Signed-off-by: Li Bin
---
drivers/net/ethernet/8390/pcnet_cs.c | 8 ++--
1 file changed, 6 insert
From: Li Bin
In __alloc_workqueue_key(), alloc_and_link_pwqs() already called
pwq_adjust_max_active() to update the pwq's max_active, but after
that there is a redundant call for pwq_adjust_max_active. It is
meanless, remove it.
Signed-off-by: Li Bin
---
kernel/workqueue.c | 5 -
1 file ch
From: Li Bin
s/wq_sysfs_register()/workqueue_sysfs_register()
Signed-off-by: Li Bin
---
include/linux/workqueue.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/include/linux/workqueue.h b/include/linux/workqueue.h
index 594521b..88d1e12 100644
--- a/include/linux/workqueu
From: Li Bin
This issue was introduced by 454c7999(sched/rt: Fix SCHED_RR across
cgroups) that missing the word 'not'. Fix it.
Signed-off-by: Li Bin
---
kernel/sched/rt.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/kernel/sched/rt.c b/kernel/sched/rt.c
index 01970c8
From: Li Bin
small codingstyle fix: '*' should be adjacent to the data name.
Signed-off-by: Li Bin
---
ipc/sem.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/ipc/sem.c b/ipc/sem.c
index 8c4f59b..7583fe8 100644
--- a/ipc/sem.c
+++ b/ipc/sem.c
@@ -2077,7 +2077,7 @@ void ex
On 2013/9/10 21:45, Tejun Heo wrote:
> On Tue, Sep 10, 2013 at 02:35:27PM +0800, Libin wrote:
>> From: Li Bin
>>
>> There is a redundant call for timer_stats_timer_set_start_info(),
>> because it is the responsibility of the 'timer a
From: Li Bin
There is a redundant call for timer_stats_timer_set_start_info(),
because it is the responsibility of the 'timer add' function:
add_timer_on()
|- timer_stats_timer_set_start_info()
add_timer()
|- mod_timer()
|- __mod_timer()
|- timer_stats_timer_set_start_in
From: Li Bin
When one work starts execution, the high bits of work's data contain
pool ID. It can represent a maximum of WORK_OFFQ_POOL_NONE. Pool ID
is assigned WORK_OFFQ_POOL_NONE when the work being initialized
indicating that no pool is associated and get_work_pool() uses it to
check the asso
Hello Tejun,
On 2013/9/9 22:14, Tejun Heo wrote:
> On Mon, Sep 09, 2013 at 01:12:14PM +0800, Libin wrote:
>> From: Li Bin
>>
>> When one work starts execution, the high bits of work's data contain
>> pool ID. It can represent a maximum of WORK_OFFQ_P
. Can you please roll
> that patch into this one with the suggested update?
Hello Tejun,
I will update it according to your suggestion.
Thanks!
Libin
>
> Thanks.
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to maj
From: Li Bin
2bd2d6f2dc ("sched: Replace use of entity_key()") had remove the
entity_key() which function is to subtract a tasks vruntime by
its groups minvruntime as the rbtree key.
The phrase "there is a subtraction using rq->cfs.min_vruntime
to account for possible wraparounds" in documentatio
From: Li Bin
It seems the "dying" should be "draining" here.
Signed-off-by: Li Bin
---
kernel/workqueue.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/kernel/workqueue.c b/kernel/workqueue.c
index 987293d..e390f3a 100644
--- a/kernel/workqueue.c
+++ b/kernel/workqueue.c
From: Li Bin
When one work starts execution, the high bits of work's data contain
pool ID. It can represent a maximum of WORK_OFFQ_POOL_NONE. Pool ID
is assigned WORK_OFFQ_POOL_NONE when the work being initialized
indicating that no pool is associated and get_work_pool() uses it to
check the asso
From: Li Bin
In the current implementation, this BUILD_BUG_ON makes no sense.
First of all, the number of worker pools needed cannot be determined
at compile time, because the number of backing pools for UNBOUND
workqueues is dynamic based on the assigned custom attributes.
Secondly, it should us
Commit-ID: 52239484bf8aec031afa84ae08aa88224d819b93
Gitweb: http://git.kernel.org/tip/52239484bf8aec031afa84ae08aa88224d819b93
Author: Libin
AuthorDate: Thu, 5 Sep 2013 18:57:56 +0800
Committer: Ingo Molnar
CommitDate: Thu, 5 Sep 2013 15:05:37 +0200
x86/smpboot: Fix announce_cpu() to
#15 #16 #17 OK
[0.890495] smpboot: Booting Node 1, Processors #18 #19 #20 #21 #22 #23
But the last OK is lost, because 'nr_cpu_ids-1' represents the maxinum possible
cpu id. It should use the maxinum present cpu id.
Signed-off-by: Libin
---
arch/x86/kernel/smpboot.c | 4 +++-
1 fi
different NUMA nodes.
Signed-off-by: Libin
---
kernel/workqueue.c | 16 +---
1 file changed, 13 insertions(+), 3 deletions(-)
diff --git a/kernel/workqueue.c b/kernel/workqueue.c
index 29b7985..42c6c29 100644
--- a/kernel/workqueue.c
+++ b/kernel/workqueue.c
@@ -249,6 +249,7
On 2013/9/4 14:51, Pekka Enberg wrote:
> On Sat, Aug 24, 2013 at 3:31 AM, Libin wrote:
>> Kmemcheck configuration menu location correction in Documentation/
>> kmemcheck.txt
>>
>> Signed-off-by: Libin
>
> Looks good to me. Andrew mind picking this up?
>
Hi
Hi all,
I'm so sorry, please ignore this patch set!
I have realized that there is no this problem in our kernel.
Preempt_schedule() has set PREEMPT_ACTIVE before calling
__schedule() and __schedule will check it if current state is not
TASK_RUNNING, avoiding this preemption.
Libin
On 2013
On 2013/8/29 21:57, Libin wrote:
> 2)Test:
> I have written a test module to trigger the problem by adding some
> synchronization condition. I will post it in the form of an attachment soon.
>
> Test result as follows:
> [103135.332683] wakeup_test: create two kernel
(set by the wake thread) validation.
Signed-off-by: Libin
---
kernel/irq/manage.c | 5 +
1 file changed, 5 insertions(+)
diff --git a/kernel/irq/manage.c b/kernel/irq/manage.c
index 514bcfd..09cb02f 100644
--- a/kernel/irq/manage.c
+++ b/kernel/irq/manage.c
@@ -655,6 +655,7 @@ static irqr
(waiter.woken)
break;
preempt_enable();
schedule();
preempt_disable();
}
__set_current_state(TASK_RUNNING);
preempt_enable();
...
Signed-off-by: Libin
---
lib/klist.c | 4
1 file changed, 4 insertions(+)
diff --git a/lib/klist.c b/lib/klist.c
index
ile (list_empty(&product_list)){
set_current_state(TASK_INTERRUPTIBLE);
preempt_enable();
schedule();
preempt_disable();
}
preempt_enable();
...
------
Libin (14):
kthread: Fix invalid wakeup in kthreadd
audit: Fix
time);
preempt_disable();
}
__set_current_state(TASK_RUNNING);
preempt_enable();
...
Signed-off-by: Libin
---
kernel/audit.c | 7 ++-
1 file changed, 6 insertions(+), 1 deletion(-)
diff --git a/kernel/audit.c b/kernel/audit.c
index a7d1346..48d2f76 100644
--- a/kernel/audit.c
+++ b/kerne
and before
__set_current_state(TASK_RUNNING), it triggers the invalid wakeup problem.
To solve this problem, using preempt_disable() to bound the operaion that
setting the task state and the conditions(set by the wake thread) validation.
Signed-off-by: Libin
---
kernel/workqueue.c | 92
and before
__set_current_state(TASK_RUNNING), it triggers the invalid wakeup problem.
To solve this problem, using preempt_disable() to bound the operaion that
setting the task state and the conditions(set by the wake thread) validation.
Signed-off-by: Libin
---
kernel/time/alarmtimer.c | 7
and before
__set_current_state(TASK_RUNNING), it triggers the invalid wakeup problem.
To solve this problem, using preempt_disable() to bound the operaion that
setting the task state and the conditions(set by the wake thread) validation.
Signed-off-by: Libin
---
kernel/exit.c | 7 +--
1 file
();
schedule();
preempt_disable();
set_current_state(TASK_INTERRUPTIBLE);
}
__set_current_state(TASK_RUNNING);
preempt_enable();
Signed-off-by: Libin
---
kernel/trace/ring_buffer_benchmark.c | 4
1 file changed, 4 insertions(+)
diff --git a/kernel/trace
);
}
...
Signed-off-by: Libin
---
kernel/kthread.c | 7 ++-
1 file changed, 6 insertions(+), 1 deletion(-)
diff --git a/kernel/kthread.c b/kernel/kthread.c
index 760e86d..25c3fed 100644
--- a/kernel/kthread.c
+++ b/kernel/kthread.c
@@ -456,10 +456,15 @@ int kthreadd(void *unused
and before
__set_current_state(TASK_RUNNING), it triggers the invalid wakeup problem.
To solve this problem, using preempt_disable() to bound the operaion that
setting the task state and the conditions(set by the wake thread) validation.
Signed-off-by: Libin
---
kernel/hrtimer.c | 7 ++-
1
preempt_disable();
}
__set_current_state(TASK_RUNNING);
preempt_enable();
...
Signed-off-by: Libin
---
kernel/module.c | 6 +-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/kernel/module.c b/kernel/module.c
index 2069158..22064e9 100644
--- a/kernel/module.c
+++ b/kernel/module.c
@@ -816,14 +8
and before
__set_current_state(TASK_RUNNING), it triggers the invalid wakeup problem.
To solve this problem, using preempt_disable() to bound the operaion that
setting the task state and the conditions(set by the wake thread) validation.
Signed-off-by: Libin
---
kernel/trace
_RUNNING);
preempt_enable();
...
}
Signed-off-by: Libin
---
kernel/audit.c | 4
1 file changed, 4 insertions(+)
diff --git a/kernel/audit.c b/kernel/audit.c
index 91e53d0..a7d1346 100644
--- a/kernel/audit.c
+++ b/kernel/audit.c
@@ -455,15 +455,19 @@ static int kauditd_thread(voi
eak;
schedule();
}
__set_current_state(TASK_RUNNING);
...
To solve this problem, using preempt_disable() to bound the operaion that
setting the task state and the conditions(set by the wake thread) validation.
Signed-off-by: Libin
---
kernel/pid_namespace.c | 4
1 file changed, 4 inserti
and before
__set_current_state(TASK_RUNNING), it triggers the invalid wakeup problem.
To solve this problem, using preempt_disable() to bound the operaion that
setting the task state and the conditions(set by the wake thread) validation.
Signed-off-by: Libin
---
kernel/rcutree.h | 4
1 file
Using return NULL to replace goto statement and avoiding a redundant
function call for free_workqueue_attrs(). If kzalloc for attrs of
struct workqueue_attrs is failed, alloc_workqueue_attrs return
NULL directedly. With no functional changs.
Signed-off-by: Libin
---
kernel/workqueue.c | 2 +-
1
Kmemcheck configuration menu location correction in Documentation/
kmemcheck.txt
Signed-off-by: Libin
---
Documentation/kmemcheck.txt | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/Documentation/kmemcheck.txt b/Documentation/kmemcheck.txt
index c28f828..9398a50 100644
On 2013/8/21 12:46, Dave Hansen wrote:
> On 08/20/2013 07:45 PM, Libin wrote:
>> [3.158023] [ cut here ]
>> [3.162626] WARNING: CPU: 0 PID: 1 at
>> arch/x86/mm/kmemcheck/kmemcheck.c:634 kmemcheck_fault+0xb1/0xc0()
> ...
>> [3.314
This patch set doesn't make any functional differences.
Libin (3):
workqueue: Comment correction in file header
workqueue: Fix manage_workers() RETURNS description
workqueue: Correct/Drop references to gcwq in Documentation
Documentation/workqueue.txt
No functional changes. This patch fixes the post gcwq comments in
Documentation/workqueue.txt.
Signed-off-by: Libin
---
Documentation/workqueue.txt | 72 ++---
1 file changed, 36 insertions(+), 36 deletions(-)
diff --git a/Documentation/workqueue.txt b
No functional change. There are two worker pools for each cpu in
current implementation(one for normal work items and the other for
high priority ones).
Signed-off-by: Libin
---
kernel/workqueue.c | 7 ---
1 file changed, 4 insertions(+), 3 deletions(-)
diff --git a/kernel/workqueue.c b
No functional change. The comment of function manage_workers()
RETURNS description is obvious wrong, same as the CONTEXT.
Fix it.
Signed-off-by: Libin
---
kernel/workqueue.c | 7 +--
1 file changed, 5 insertions(+), 2 deletions(-)
diff --git a/kernel/workqueue.c b/kernel/workqueue.c
index
On 2013/8/16 22:47, Tejun Heo wrote:
> On Fri, Aug 16, 2013 at 01:33:17PM +0800, Libin wrote:
>> No functional changes. This patch fixes the post gcwq comments in
>> Documentation/workqueue.txt.
>>
>> Signed-off-by: Libin
>
> Thanks a lot for doing this. Can y
No functional change. The comment of function manage_workers()
RETURNS description is obvious wrong, same as the CONTEXT.
Fix it.
Signed-off-by: Libin
---
kernel/workqueue.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/kernel/workqueue.c b/kernel/workqueue.c
index
No functional changes. This patch fixes the post gcwq comments in
Documentation/workqueue.txt.
Signed-off-by: Libin
---
Documentation/workqueue.txt | 60 ++---
1 file changed, 29 insertions(+), 31 deletions(-)
diff --git a/Documentation/workqueue.txt b
No functional change. There are two worker pools for each cpu in
current implementation(one for normal work items and the other for
high priority ones).
Signed-off-by: Libin
---
kernel/workqueue.c | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/kernel/workqueue.c b
(*->vm_end - *->vm_start) >> PAGE_SHIFT operation is implemented
as a inline funcion vma_pages() in linux/mm.h, so using it.
Signed-off-by: Libin
---
drivers/char/mspec.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/char/mspec.c b/drivers/char/m
On 2013/4/15 22:15, Jonghwan Choi wrote:
> From: libin
>
> This patch looks like it should be in the 3.8-stable tree, should we apply
> it?
>
yes, I think this patch should be applied to the 3.8-stable tree. Thanks.
Libin
> --
>
> Fr
(*->vm_end - *->vm_start) >> PAGE_SHIFT operation is implemented
as a inline funcion vma_pages() in linux/mm.h, so using it.
Signed-off-by: Libin
---
fs/ncpfs/mmap.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/fs/ncpfs/mmap.c b/fs/ncpfs/mmap.c
index ee24
(*->vm_end - *->vm_start) >> PAGE_SHIFT operation is implemented
as a inline funcion vma_pages() in linux/mm.h, so using it.
Signed-off-by: Libin
---
drivers/uio/uio.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/uio/uio.c b/drivers/uio/uio.c
i
(*->vm_end - *->vm_start) >> PAGE_SHIFT operation is implemented
as a inline funcion vma_pages() in linux/mm.h, so using it.
Signed-off-by: Libin
---
drivers/gpu/drm/ttm/ttm_bo_vm.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/ttm/ttm_bo_vm
(*->vm_end - *->vm_start) >> PAGE_SHIFT operation is implemented
as a inline funcion vma_pages() in linux/mm.h, so using it.
Signed-off-by: Libin
---
drivers/pci/pci-sysfs.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/pci/pci-sysfs.c b/drivers/pci
(*->vm_end - *->vm_start) >> PAGE_SHIFT operation is implemented
as a inline funcion vma_pages() in linux/mm.h, so using it.
Signed-off-by: Libin
---
mm/memory.c | 2 +-
mm/mmap.c | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/mm/memory.c b/mm/memory.c
i
(*->vm_end - *->vm_start) >> PAGE_SHIFT operation is implemented
as a inline funcion vma_pages() in linux/mm.h, so using it.
Signed-off-by: Libin
---
drivers/char/mspec.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/char/mspec.c b/drivers/char/m
Commit-ID: b9b0853a4b377f84a5e6ed091816a9a2d6b10918
Gitweb: http://git.kernel.org/tip/b9b0853a4b377f84a5e6ed091816a9a2d6b10918
Author: Libin
AuthorDate: Mon, 1 Apr 2013 19:14:01 +0800
Committer: Ingo Molnar
CommitDate: Wed, 10 Apr 2013 13:39:57 +0200
sched: Fix comment in
Commit-ID: fd9b86d37a600488dbd80fe60cca46b822bff1cd
Gitweb: http://git.kernel.org/tip/fd9b86d37a600488dbd80fe60cca46b822bff1cd
Author: libin
AuthorDate: Mon, 8 Apr 2013 14:39:12 +0800
Committer: Ingo Molnar
CommitDate: Mon, 8 Apr 2013 13:23:03 +0200
sched/debug: Fix sd->*_idx li
Commit 201c373e8e ("sched/debug: Limit sd->*_idx range on sysctl")
was an incomplete bug fix. This patch fixs sd->*_idx limit range to
[0 ~ CPU_LOAD_IDX_MAX - 1] avioding array overflow caused by setting
sd->*_idx to CPU_LOAD_IDX_MAX on sysctl.
Signed-off-by: Libin
---
Commit 201c373e8e ("sched/debug: Limit sd->*_idx range on sysctl")
was an incomplete bug fix. This patch fixs sd->*_idx limit range to
[0 ~ CPU_LOAD_IDX_MAX - 1] avioding array overflow caused by setting
sd->*_idx to CPU_LOAD_IDX_MAX on sysctl.
Signed-off-by: Libin
---
The comment of function rebalance_domains mentions arch_init_sched_domains,
but this function don't exist. The proper function is init_sched_domains.
Signed-off-by: Libin
---
kernel/sched/fair.c |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/kernel/sched/fai
78 matches
Mail list logo