Since the commit 55627e3cd22c ("sched/core: Remove rq->cpu_load[]"),
we don't need this any more.
Signed-off-by: Hui Su
---
kernel/sched/sched.h | 5 -
1 file changed, 5 deletions(-)
diff --git a/kernel/sched/sched.h b/kernel/sched/sched.h
index 10a1522b1e30..223202
sed_result [-Wunused-result]
154 | kstrtobool(str, _depot_disable);
| ^
Signed-off-by: Hui Su
---
lib/stackdepot.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/lib/stackdepot.c b/lib/stackdepot.c
index cc21116512a7..acd13f574
The following commit has been merged into the sched/core branch of tip:
Commit-ID: 65bcf072e20ed7597caa902f170f293662b0af3c
Gitweb:
https://git.kernel.org/tip/65bcf072e20ed7597caa902f170f293662b0af3c
Author:Hui Su
AuthorDate:Sat, 31 Oct 2020 01:32:23 +08:00
Committer
Hi, Matthew:
On Wed, Dec 30, 2020 at 12:42:33PM +, Matthew Wilcox wrote:
> On Wed, Dec 30, 2020 at 07:40:14PM +0800, Hui Su wrote:
> > local variable node_order do not need the static here.
>
> It bloody well does. It can be up to 2^10 entries on x86 (and larger
> on
local variable node_order do not need the static here.
Signed-off-by: Hui Su
---
mm/page_alloc.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/mm/page_alloc.c b/mm/page_alloc.c
index bdbec4c98173..45e049ccf117 100644
--- a/mm/page_alloc.c
+++ b/mm/page_alloc.c
@@ -5864,7
map_sg()")
> url:
> https://github.com/0day-ci/linux/commits/Hui-Su/blokc-blk-merge-remove-the-next_bvec-label-in-__blk_bios_map_sg/20201223-202618
> base: https://git.kernel.org/cgit/linux/kernel/git/axboe/linux-block.git
> for-next
>
> in testcase: xfstests
> version:
remove the next_bvec label in __blk_bios_map_sg(),
simplify the logic of traversal bvec.
Signed-off-by: Hui Su
---
block/blk-merge.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/block/blk-merge.c b/block/blk-merge.c
index 808768f6b174..aa113cbc0f35 100644
On Wed, Dec 23, 2020 at 08:33:07AM +, Christoph Hellwig wrote:
> On Wed, Dec 23, 2020 at 12:31:58PM +0800, sh wrote:
> > remove the next_bvec label in __blk_bios_map_sg(), simplify the logic
> > of traversal bvec.
>
> What makes you believe that this simplifies anything?
1. this change
Hi, all:
Please ignore this change, i will resend a patch V2 later.
The macro irqs_disabled is always defined in include/linux/irqflags.h,
so we don't need the macro check.
Signed-off-by: Hui Su
---
fs/buffer.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/fs/buffer.c b/fs/buffer.c
index 32647d2011df..34b505542d96 100644
--- a/fs/buffer.c
+++ b/fs
the rcu_read_lock and rcu_read_unlock is unnecessary in:
lock_page_lruvec()
lock_page_lruvec_irq()
lock_page_lruvec_irqsave()
Signed-off-by: Hui Su
---
mm/memcontrol.c | 6 --
1 file changed, 6 deletions(-)
diff --git a/mm/memcontrol.c b/mm/memcontrol.c
index
The following commit has been merged into the core/rcu branch of tip:
Commit-ID: c386e29d43728778ddd642fa73cc582bee684171
Gitweb:
https://git.kernel.org/tip/c386e29d43728778ddd642fa73cc582bee684171
Author:Hui Su
AuthorDate:Thu, 15 Oct 2020 22:13:34 +08:00
Committer
cing: Attempted to kill init!
exitcode=0x0009 ]---
qemu-system-x86_64: terminating on signal 15 from pid 2836242 (pkill)
The next tree for Dec 10 does not contain this problem, and the tree for Dec 11
can reproduce this problem stably(always panic when poweroff).
Reported-by: Hui Su
we don't need pde_get()'s return value, so make
pde_get() return nothing
Signed-off-by: Hui Su
---
fs/proc/internal.h | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/fs/proc/internal.h b/fs/proc/internal.h
index 917cc85e3466..0abbd93a4a08 100644
--- a/fs/proc/internal.h
On Mon, Dec 07, 2020 at 09:28:46AM -0800, Shakeel Butt wrote:
> On Mon, Dec 7, 2020 at 6:22 AM Hui Su wrote:
>
> The reason to keep __memcg_kmem_[un]charge_page functions is that they
> were called in the very hot path. Can you please check the performance
> impact
Since the commit 60cd4bcd6238 ("memcg: localize memcg_kmem_enabled()
check"), we have supplied the api which users don't have to explicitly
check memcg_kmem_enabled().
Signed-off-by: Hui Su
---
mm/page_alloc.c | 12 ++--
1 file changed, 6 insertions(+), 6 deletions(-)
diff
of
function ‘set_memory_valid’ [-Werror=implicit-function-declaration]
12 | set_memory_valid(addr, 1, !protect);
| ^~~~
cc1: some warnings being treated as errors
which introduced by commit d54febeba2ff ("kfence: use pt_regs to
generate stack trace on faults").
Signed-off-
defer_compaction() and compaction_deferred() and
compaction_restarting() in mm/compaction.c won't
be used in other files, so make them static, and
remove the declaration in the header file.
Take the chance to fix a typo.
Signed-off-by: Hui Su
---
include/linux/compaction.h | 12
c) Ideally: find the original patch on the mailing list and do a
> > reply-to-all to that, adding suitable additional cc's
> >
> > *** Remember to use Documentation/process/submit-checklist.rst when testing
> > your code ***
> >
> > The -mm tree is in
On Tue, Nov 17, 2020 at 05:59:00PM +, Matthew Wilcox wrote:
> On Wed, Nov 18, 2020 at 01:56:58AM +0800, Hui Su wrote:
> > On Tue, Nov 17, 2020 at 05:47:27PM +, Matthew Wilcox wrote:
> > > On Wed, Nov 18, 2020 at 01:46:33AM +0800, Hui Su wrote:
> > > > On T
The following commit has been merged into the x86/cleanups branch of tip:
Commit-ID: 09a217c10504bcaef911cf2af74e424338efe629
Gitweb:
https://git.kernel.org/tip/09a217c10504bcaef911cf2af74e424338efe629
Author:Hui Su
AuthorDate:Fri, 13 Nov 2020 21:39:43 +08:00
Committer
On Tue, Nov 17, 2020 at 05:59:00PM +, Matthew Wilcox wrote:
> On Wed, Nov 18, 2020 at 01:56:58AM +0800, Hui Su wrote:
> > On Tue, Nov 17, 2020 at 05:47:27PM +, Matthew Wilcox wrote:
> > > On Wed, Nov 18, 2020 at 01:46:33AM +0800, Hui Su wrote:
> > > > On T
On Tue, Nov 17, 2020 at 05:41:17PM +, Matthew Wilcox wrote:
> On Wed, Nov 18, 2020 at 01:12:42AM +0800, Hui Su wrote:
> > lru_list lru bit 0 can tell whether the list is
> > avtive lru-list or not.
> > lru_list lru bit 1 can tell whether the list is
> > file lru-li
On Tue, Nov 17, 2020 at 05:47:27PM +, Matthew Wilcox wrote:
> On Wed, Nov 18, 2020 at 01:46:33AM +0800, Hui Su wrote:
> > On Tue, Nov 17, 2020 at 05:41:17PM +, Matthew Wilcox wrote:
> > > On Wed, Nov 18, 2020 at 01:12:42AM +0800, Hui Su wrote:
> > > > lru_l
lru_list lru bit 0 can tell whether the list is
avtive lru-list or not.
lru_list lru bit 1 can tell whether the list is
file lru-list or not.
And fix some define type in shrink_active_list()
and get_scan_count().
v1->v2:
correct the commit message, and fix the define type.
Signed-off-by: Hui
On Wed, Nov 18, 2020 at 12:49:19AM +0800, Hui Su wrote:
> page->lru bit 0 can tell whether the page is
> avtive page or not.
> page->lru bit 1 can tell whether the page is
> file page or not.
>
> Signed-off-by: Hui Su
> ---
> include/linux/mmzone.h | 4 ++--
> 1
page->lru bit 0 can tell whether the page is
avtive page or not.
page->lru bit 1 can tell whether the page is
file page or not.
Signed-off-by: Hui Su
---
include/linux/mmzone.h | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/include/linux/mmzone.h b/include
timens_on_fork() always return 0, and maybe not
need to judge the return value in copy_namespaces().
So make timens_on_fork() return nothing and do not
judge its return val in copy_namespaces().
Signed-off-by: Hui Su
---
include/linux/time_namespace.h | 6 +++---
kernel/nsproxy.c
Please ignore this change.
timens_on_fork() always return 0, and maybe not
need to judge the return value in copy_namespaces().
So make timens_on_fork() return nothing and do not
judge its return val in copy_namespaces().
Signed-off-by: Hui Su
---
include/linux/time_namespace.h | 6 +++---
kernel/nsproxy.c
On Tue, Nov 17, 2020 at 12:30:26AM -0800, Andrey Vagin wrote:
> On Sun, Nov 15, 2020 at 10:36 AM Hui Su wrote:
> >
> > timens_on_fork() always return 0, and maybe not
> > need to judge the return value in copy_namespaces().
>
> Thank you for cleaning this up. I t
that, adding suitable additional cc's
> > >
> > > *** Remember to use Documentation/process/submit-checklist.rst when
> > > testing your code ***
> > >
> > > The -mm tree is included into linux-next and is updated
> > > there every 3-4 workin
Since commit 837d026d560c ("mm/compaction: more
trace to understand when/why compaction start/finish"),
the comment place is not suitable.
So move compaction_suitable's comment to right place.
Signed-off-by: Hui Su
---
mm/compaction.c | 14 +++---
1 file changed, 7 insert
timens_on_fork() always return 0, and maybe not
need to judge the return value in copy_namespaces().
Signed-off-by: Hui Su
---
kernel/nsproxy.c | 7 +--
1 file changed, 1 insertion(+), 6 deletions(-)
diff --git a/kernel/nsproxy.c b/kernel/nsproxy.c
index 12dd41b39a7f..e2e6c5dc433f 100644
Use put_nsproxy() instead of '
if (atomic_dec_and_test(>count)) {
free_nsproxy(ns);
}' in switch_task_namespaces().
and remove the whitespace by the way.
Signed-off-by: Hui Su
---
kernel/nsproxy.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/ker
_inode().
Signed-off-by: Hui Su
---
mm/shmem.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/mm/shmem.c b/mm/shmem.c
index 537c137698f8..b84adda45461 100644
--- a/mm/shmem.c
+++ b/mm/shmem.c
@@ -2308,7 +2308,6 @@ static struct inode *shmem_get_inode(struct super_block
*sb,
ion in shmem_fs.h
v2->v3:
make shmem_aops global, and export it to modules.
Signed-off-by: Hui Su
---
include/linux/shmem_fs.h | 6 +-
mm/shmem.c | 16 ++--
2 files changed, 11 insertions(+), 11 deletions(-)
diff --git a/include/linux/shmem_fs.h b/include/linux/shmem_fs.h
On Sat, Nov 14, 2020 at 10:50:39AM -0800, Andrew Morton wrote:
> On Sat, 14 Nov 2020 13:51:34 +0800 Hui Su wrote:
>
> > inline the shmem_mapping(), and use shmem_mapping()
> > instead of 'inode->i_mapping->a_ops == _aops'
> > in shmem_evict_inode().
> &g
inline the shmem_mapping(), and use shmem_mapping()
instead of 'inode->i_mapping->a_ops == _aops'
in shmem_evict_inode().
v1->v2:
remove the inline for func declaration in shmem_fs.h
Reviewed-by: Pankaj Gupta
Reported-by: kernel test robot
Signed-off-by: Hui Su
---
mm/shmem.c |
On Sat, Nov 14, 2020 at 12:54:47AM +0800, kernel test robot wrote:
> Hi Hui,
>
> Thank you for the patch! Perhaps something to improve:
>
> [auto build test WARNING on mmotm/master]
>
> url:
> https://github.com/0day-ci/linux/commits/Hui-Su/mm-shmem-c-make-shmem
inline the shmem_mapping(), and use shmem_mapping()
instead of 'inode->i_mapping->a_ops == _aops'
in shmem_evict_inode().
Signed-off-by: Hui Su
---
include/linux/shmem_fs.h | 2 +-
mm/shmem.c | 4 ++--
2 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/include
use page_counter_read() in page_counter_set_max().
Signed-off-by: Hui Su
---
mm/page_counter.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/mm/page_counter.c b/mm/page_counter.c
index b24a60b28bb0..c6860f51b6c6 100644
--- a/mm/page_counter.c
+++ b/mm/page_counter.c
show_trace_log_lvl() won't be used by other files, so
make it static and remove the declaration from the header
file.
Signed-off-by: Hui Su
---
arch/x86/include/asm/stacktrace.h | 3 ---
arch/x86/kernel/dumpstack.c | 2 +-
2 files changed, 1 insertion(+), 4 deletions(-)
diff --git a/arch
In both kernfs_node_from_dentry() and in
kernfs_dentry_node(), we will check the dentry->inode
is NULL or not, which is superfluous.
So remove the check in kernfs_node_from_dentry().
Signed-off-by: Hui Su
---
fs/kernfs/dir.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --
The following commit has been merged into the sched/core branch of tip:
Commit-ID: cdb310474dece99985e4cdd2b96b1324e39c1c9d
Gitweb:
https://git.kernel.org/tip/cdb310474dece99985e4cdd2b96b1324e39c1c9d
Author:Hui Su
AuthorDate:Fri, 30 Oct 2020 22:46:21 +08:00
Committer
cgroup'.
Acked-by: Michal Hocko
Acked-by: Johannes Weiner
Reviewed-by: Shakeel Butt
Signed-off-by: Hui Su
---
include/linux/memcontrol.h | 16 +--
mm/memcontrol.c| 40 --
2 files changed, 14 insertions(+), 42 deletions(-)
diff --git a/i
we have supplied the inline function: of_cft() in cgroup.h.
So replace the direct use 'of->kn->priv' with inline func
of_cft(), which is more readable.
Signed-off-by: Hui Su
---
kernel/cgroup/cgroup.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/kernel/
We have supplied the inline func: mem_cgroup_is_root().
So we may use mem_cgroup_is_root() instead of using
'memcg == root_mem_cgroup' or 'memcg != root_mem_cgroup'
directly, which is more readable.
Signed-off-by: Hui Su
---
mm/memcontrol.c | 16
1 file changed, 8 insertions
mem_cgroup_page_lruvec() in memcontrol.c and
mem_cgroup_lruvec() in memcontrol.h is very similar
except for the param(page and memcg) which also can be
convert to each other.
So rewrite mem_cgroup_page_lruvec() with mem_cgroup_lruvec().
Signed-off-by: Hui Su
---
include/linux/memcontrol.h | 18
k/cgroup_unlock have been killed.
So update the cgroup-v1 Docs.
Signed-off-by: Hui Su
---
Documentation/admin-guide/cgroup-v1/cgroups.rst | 13 +
1 file changed, 5 insertions(+), 8 deletions(-)
diff --git a/Documentation/admin-guide/cgroup-v1/cgroups.rst
b/Documentation/admin-guide/c
On Mon, Nov 02, 2020 at 08:53:41AM -0500, Phil Auld wrote:
> On Fri, Oct 30, 2020 at 10:16:29PM + David Laight wrote:
> > From: Benjamin Segall
> > > Sent: 30 October 2020 18:48
> > >
> > > Hui Su writes:
> > >
> > > > Since 'ab9
These flag is meant the bit numbers, they are
used like '(type & SLOTS_CACHE)' and so on.
Define these flags using bit numbers instead of
hardcoding powers of 2, which maybe better.
No change in the actual values.
Signed-off-by: Hui Su
---
mm/swap_slots.c | 4 ++--
mm/swapfile.c
Change the comment of is_dump_unreclaim_slabs(), it just check
whether nr_unreclaimable slabs amount is greater than user
memory, and explain why we dump unreclaim slabs.
Rename it to should_dump_unreclaim_slab() maybe better.
Signed-off-by: Hui Su
---
mm/oom_kill.c | 14 --
1 file
We have supplied the macro: 'task_current()', and we should
all use task_current() instaed of 'rq->curr == p',
which is more readable.
No functional change.
Signed-off-by: Hui Su
---
kernel/sched/deadline.c | 2 +-
kernel/sched/debug.c| 2 +-
kernel/sched/fair.c | 6 +++---
ker
Since 'ab93a4bc955b ("sched/fair: Remove
distribute_running fromCFS bandwidth")',there is
nothing to protect between raw_spin_lock_irqsave/store()
in do_sched_cfs_slack_timer().
So remove it.
Signed-off-by: Hui Su
---
kernel/sched/fair.c | 3 ---
1 file changed, 3 deletions(-)
to should_dump_unreclaim_slabs which should make it
clear what it is meant to do and drop the comment as the purpose
should be pretty evident now.
Signed-off-by: Hui Su
Acked-by: Michal Hocko
---
mm/oom_kill.c | 8 ++--
1 file changed, 2 insertions(+), 6 deletions(-)
diff --git a/mm
On Tue, Oct 27, 2020 at 07:45:53PM +0100, Vlastimil Babka wrote:
> On 10/27/20 6:04 PM, Hui Su wrote:
> > In list_lru_walk(), nr_to_walk type is 'unsigned long',
> > so nr_to_walk won't be '< 0'.
> >
> > In list_lru_walk_node(), nr_to_walk type is 'unsigned long',
&g
In list_lru_walk(), nr_to_walk type is 'unsigned long',
so nr_to_walk won't be '< 0'.
In list_lru_walk_node(), nr_to_walk type is 'unsigned long',
so *nr_to_walk won't be '< 0' too.
We can use '!nr_to_walk' instead of 'nr_to_walk <= 0', which
is more precise.
Signed-off-by: Hui Su
---
Use task_group_is_root() instead of '!tg->se[0]'
and 'tg == _task_group' to judge whether a
task_group is root_task_group.
Signed-off-by: Hui Su
---
kernel/sched/autogroup.c | 2 +-
kernel/sched/core.c | 11 ---
kernel/sched/fair.c | 4 ++--
kernel/sched/rt.c|
On Tue, Oct 27, 2020 at 03:58:14PM +0100, Michal Hocko wrote:
> On Tue 27-10-20 22:45:29, Hui Su wrote:
> > is_dump_unreclaim_slabs() just check whether nr_unreclaimable
> > slabs amount is greater than user memory, not match witch comment.
>
> As I've tried to
is_dump_unreclaim_slabs() just check whether nr_unreclaimable
slabs amount is greater than user memory, not match witch comment.
So delete the comment, and rename it to should_dump_unreclaim_slabs().
Signed-off-by: Hui Su
---
mm/oom_kill.c | 8 ++--
1 file changed, 2 insertions(+), 6
On Tue, Oct 27, 2020 at 08:11:18AM +0100, Michal Hocko wrote:
> On Sat 26-09-20 12:15:26, Hui Su wrote:
> > fix the comment of is_dump_unreclaim_slabs(), it just check
> > whether nr_unreclaimable slabs amount is greater than user
> > memory.
>
> The original c
Use the list_is_singular(_se->run_list) api instead of
'rt_se->run_list.prev != rt_se->run_list.next'.
Fix a comment by the way, and make the comment more clearly.
Signed-off-by: Hui Su
---
kernel/sched/rt.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git
list_is_singular() can tell whether a list has just one entry.
So we use list_is_singular() here.
Signed-off-by: Hui Su
---
net/atm/br2684.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/net/atm/br2684.c b/net/atm/br2684.c
index 3e17a5ecaa94..398f7e086cf4 100644
(dl_rq_of_se()):
dl_se ==> p ==> rq ==> dl
||
rq <==
it looks a little redundant.
So add rq_of_se(dl_se) to instead of rq_of_dl_rq(dl_rq_of_se(dl_se)).
rq_of_se():dl_se ==> p ==> rq
Signed-off
we can get the next dl_entity just from struct dl_rq,
and do not need the param struct rq.
So remove it.
Like the change: https://lkml.org/lkml/2020/9/29/2030
Signed-off-by: Hui Su
---
kernel/sched/deadline.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/kernel/sched
On Tue, Oct 20, 2020 at 10:27:05AM -0700, David Riley wrote:
> I don't think it's worth making debug_file_exist a separate function. It's
> more clear to just do the check for the file, especially since you then log
> that path in the failure case.
>
> On Fri, Oct 16, 2020 at
off-by: Hui Su
---
tools/time/udelay_test.sh | 42 ++-
1 file changed, 24 insertions(+), 18 deletions(-)
diff --git a/tools/time/udelay_test.sh b/tools/time/udelay_test.sh
index 6779d7e55d85..c8458d5b7fcd 100755
--- a/tools/time/udelay_test.sh
+++ b/tools/t
different from rt_sched_class and fair_sched_class,
dl_sched_class have no sched group.
We can direct get dl_rq according to cpu, so remove the
param dl_rq in print_dl_rq().
Signed-off-by: Hui Su
---
kernel/sched/deadline.c | 2 +-
kernel/sched/debug.c| 3 ++-
kernel/sched/sched.h| 2
1.reduce the critical section in elevator_get().
2.reduce the critical section in elevator_get_by_features().
3.remove the found variable.
the elv_list_lock is used to protect the elv_list,
and the operations of elevator_type does not need
to be protected.
Signed-off-by: Hui Su
---
block
use REQ_OP_WRITE instead of hard code in
op_is_write().
Signed-off-by: Hui Su
---
include/linux/blk_types.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/include/linux/blk_types.h b/include/linux/blk_types.h
index 7d7c13238fdb..7b9b02378c24 100644
--- a/include/linux
format the example code.
Signed-off-by: Hui Su
---
Documentation/admin-guide/cpu-load.rst | 63 ++
1 file changed, 33 insertions(+), 30 deletions(-)
diff --git a/Documentation/admin-guide/cpu-load.rst
b/Documentation/admin-guide/cpu-load.rst
index ebdecf864080
:~/workspace/linux-stable/tools/time$ sudo ./udelay_test.sh
modprobe: FATAL: Module udelay_test not found in directory /lib/modules/5.4.44
ERROR, can not access /sys/kernel/debug/udelay_test.
modprobe: FATAL: Module udelay_test not found.
Signed-off-by: Hui Su
---
tools/time/udelay_test.sh | 51
update the api of call_rcu()
Signed-off-by: Hui Su
---
Documentation/RCU/whatisRCU.rst | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/Documentation/RCU/whatisRCU.rst b/Documentation/RCU/whatisRCU.rst
index c7f147b8034f..aa7d5ed20da5 100644
--- a/Documentation/RCU
use #elif instead of #end and #elif.
Signed-off-by: Hui Su
---
kernel/acct.c | 7 ++-
1 file changed, 2 insertions(+), 5 deletions(-)
diff --git a/kernel/acct.c b/kernel/acct.c
index b0c5b3a9f5af..24681372af07 100644
--- a/kernel/acct.c
+++ b/kernel/acct.c
@@ -381,9 +381,7 @@ static
The judgement of first do not need to be in the critical region.
So move it out.
Signed-off-by: Hui Su
---
kernel/async.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/kernel/async.c b/kernel/async.c
index 33258e6e20f8..9d33ffcc84c7 100644
--- a/kernel/async.c
+++ b/kernel
use family->version instead of the hrad code(TASKSTATS_GENL_VERSION).
Signed-off-by: Hui Su
---
kernel/taskstats.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/kernel/taskstats.c b/kernel/taskstats.c
index e2ac0e37c4ae..8364663ca07d 100644
--- a/kernel/taskstats.c
++
it.
Signed-off-by: Hui Su
---
drivers/base/node.c | 4 ++--
mm/hugetlb.c| 6 +++---
2 files changed, 5 insertions(+), 5 deletions(-)
diff --git a/drivers/base/node.c b/drivers/base/node.c
index 50af16e68d98..b5453c372c5b 100644
--- a/drivers/base/node.c
+++ b/drivers/base/node.c
@@ -430,8 +430,8
On Tue, Sep 29, 2020 at 11:56:42AM +0200, Dietmar Eggemann wrote:
> On 25/09/2020 21:10, Hui Su wrote:
> > Macro for_each_leaf_cfs_rq_safe() use list_for_each_entry_safe(),
> > which can against removal of list entry, but we only
> > print the cfs_rq data and won't
On Thu, Oct 01, 2020 at 09:17:29AM -0700, Joe Perches wrote:
> On Thu, 2020-10-01 at 14:27 +0200, David Hildenbrand wrote:
> > On 25.09.20 18:06, Hui Su wrote:
> > > 1. the cpuset.c has been moved from kernel/cpuset.c to
> > > kernel/cgroup/cpuset.c long time ag
On Mon, Sep 28, 2020 at 11:04:34AM -0700, Ira Weiny wrote:
> On Mon, Sep 28, 2020 at 12:33:37AM +0800, Hui Su wrote:
> > As the comments said, if @addr is NULL, no operation
> > is performed, check the addr first in vfree() and
> > vfree_atomic() maybe a better choice.
we test the page reference count is zero or not here,
it can be a bug here if page refercence count is not zero.
So we can just use put_page_testzero() instead of
page_count().
Signed-off-by: Hui Su
---
mm/hugetlb.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/mm
struct rq is not necessary for pick_next_rt_entity(),
we can get next sched_rt_entity just from struct rt_rq.
Signed-off-by: Hui Su
---
kernel/sched/rt.c | 9 +++--
1 file changed, 3 insertions(+), 6 deletions(-)
diff --git a/kernel/sched/rt.c b/kernel/sched/rt.c
index f215eea6a966
handle exceptions gracefully, and avoid using
if (0) .
Signed-off-by: Hui Su
---
kernel/sched/rt.c | 13 +++--
1 file changed, 7 insertions(+), 6 deletions(-)
diff --git a/kernel/sched/rt.c b/kernel/sched/rt.c
index f215eea6a966..1dd6cbd67e6e 100644
--- a/kernel/sched/rt.c
+++ b/kernel
arch/x86: fix some typos in xen_pagetable_p2m_free():
s/Fortunatly/Fortunately
Signed-off-by: Hui Su
---
arch/x86/xen/mmu_pv.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/x86/xen/mmu_pv.c b/arch/x86/xen/mmu_pv.c
index 3273c985d3dd..fbee8f50088b 100644
--- a/arch/x86
fix the comment of find_vm_area() and get_vm_area()
Signed-off-by: Hui Su
---
mm/vmalloc.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/mm/vmalloc.c b/mm/vmalloc.c
index be4724b916b3..a2d7900024be 100644
--- a/mm/vmalloc.c
+++ b/mm/vmalloc.c
@@ -2133,7 +2133,7
As the comments said, if @addr is NULL, no operation
is performed, check the addr first in vfree() and
vfree_atomic() maybe a better choice.
Signed-off-by: Hui Su
---
mm/vmalloc.c | 11 ++-
1 file changed, 6 insertions(+), 5 deletions(-)
diff --git a/mm/vmalloc.c b/mm/vmalloc.c
index
There is some typos in cpuidle_idle_call(), so fix
it.
s/stratight/straight
Signed-off-by: Hui Su
---
kernel/sched/idle.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/kernel/sched/idle.c b/kernel/sched/idle.c
index f324dc36fc43..4710fef20678 100644
--- a/kernel/sched
fix the comment of perf_sched_init():
interator --> iterator
Signed-off-by: Hui Su
---
arch/x86/events/core.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/x86/events/core.c b/arch/x86/events/core.c
index 1cbf57dc2ac8..74d430b31d78 100644
--- a/arch/x86/events/cor
fix comments for isolate_lru_page():
s/fundamentnal/fundamental
Signed-off-by: Hui Su
---
mm/vmscan.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/mm/vmscan.c b/mm/vmscan.c
index 466fc3144fff..def9d2b3057b 100644
--- a/mm/vmscan.c
+++ b/mm/vmscan.c
@@ -1751,7 +1751,7
fix the comment of radix_tree_next_slot():
interator --> iterator.
Signed-off-by: Hui Su
---
include/linux/radix-tree.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/include/linux/radix-tree.h b/include/linux/radix-tree.h
index c2a9f7c90727..03591b18215f 100
since c67dc624757 commit, the __vunmap() have been
changed to __vfree(), so update the confusing comment().
Signed-off-by: Hui Su
---
mm/vmalloc.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/mm/vmalloc.c b/mm/vmalloc.c
index be4724b916b3..689e7ef08a5d 100644
--- a/mm
alloc_slots() allocate memory for slots by kmem_cache_alloc(),
then memset it, we can just use kmem_cache_zalloc() api.
Signed-off-by: Hui Su
---
mm/z3fold.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/mm/z3fold.c b/mm/z3fold.c
index 460b0feced26..18feaa0bc537 100644
dump_unreclaimable_slab() acquires the slab_mutex first,
and it won't remove any slab_caches list entry when
itering the slab_caches lists.
Thus, we do not need list_for_each_entry_safe here,
which is against removal of list entry.
Signed-off-by: Hui Su
---
mm/slab_common.c | 4 ++--
1 file
fix the comment of is_dump_unreclaim_slabs(), it just check
whether nr_unreclaimable slabs amount is greater than user
memory.
Signed-off-by: Hui Su
---
mm/oom_kill.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/mm/oom_kill.c b/mm/oom_kill.c
index e90f25d6385d
() in
print_cfs_stats().
Signed-off-by: Hui Su
---
kernel/sched/fair.c | 11 ---
1 file changed, 8 insertions(+), 3 deletions(-)
diff --git a/kernel/sched/fair.c b/kernel/sched/fair.c
index 1a68a0536add..d40dfb4349b0 100644
--- a/kernel/sched/fair.c
+++ b/kernel/sched/fair.c
@@ -391,11
kmemleak-test.c is just a kmemleak test module, which
also can not be used as a built-in kernel module.
Thus, i think it may should not be in mm dir, and move
the kmemleak-test.c to samples/kmemleak/kmemleak-test.c.
Fix the spelling of built-in by the way.
Signed-off-by: Hui Su
just fix the spelling of enqueued.
Signed-off-by: Hui Su
---
kernel/sched/fair.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/kernel/sched/fair.c b/kernel/sched/fair.c
index 1a68a0536add..fadee2b05df3 100644
--- a/kernel/sched/fair.c
+++ b/kernel/sched/fair.c
@@ -753,7
1. the cpuset.c has been moved from kernel/cpuset.c to
kernel/cgroup/cpuset.c long time ago, but the comment is stale,
so we update it.
2. get_page_from_freelist() may alloc many pages according to
order, we may use pages for better.
Signed-off-by: Hui Su
---
kernel/cgroup/cpuset.c | 2 +-
mm
the struct name was modified long ago, but the comment still
use struct handle_s.
Signed-off-by: Hui Su
---
include/linux/jbd2.h | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/include/linux/jbd2.h b/include/linux/jbd2.h
index 08f904943ab2..a1ef05412acf 100644
100 matches
Mail list logo