Skip to content

Commit

Permalink
mm: memory-failure: minor cleanup for comments and codestyle
Browse files Browse the repository at this point in the history
Fix some wrong function names and grammar error in comments. Also remove
unneeded space after for_each_process. No functional change intended.

Link: https://lkml.kernel.org/r/20230711055016.2286677-7-linmiaohe@huawei.com
Signed-off-by: Miaohe Lin <linmiaohe@huawei.com>
Acked-by: Naoya Horiguchi <naoya.horiguchi@nec.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
  • Loading branch information
Miaohe Lin authored and Andrew Morton committed Aug 18, 2023
1 parent e9c36f7 commit 5885c6a
Showing 1 changed file with 6 additions and 6 deletions.
12 changes: 6 additions & 6 deletions mm/memory-failure.c
Original file line number Diff line number Diff line change
Expand Up @@ -608,7 +608,7 @@ static void collect_procs_anon(struct page *page, struct list_head *to_kill,

pgoff = page_to_pgoff(page);
read_lock(&tasklist_lock);
for_each_process (tsk) {
for_each_process(tsk) {
struct anon_vma_chain *vmac;
struct task_struct *t = task_early_kill(tsk, force_early);

Expand Down Expand Up @@ -652,7 +652,7 @@ static void collect_procs_file(struct page *page, struct list_head *to_kill,
/*
* Send early kill signal to tasks where a vma covers
* the page but the corrupted page is not necessarily
* mapped it in its pte.
* mapped in its pte.
* Assume applications who requested early kill want
* to be informed of all such data corruptions.
*/
Expand Down Expand Up @@ -2117,7 +2117,7 @@ static DEFINE_MUTEX(mf_mutex);
* detected by a background scrubber)
*
* Must run in process context (e.g. a work queue) with interrupts
* enabled and no spinlocks hold.
* enabled and no spinlocks held.
*
* Return: 0 for successfully handled the memory error,
* -EOPNOTSUPP for hwpoison_filter() filtered the error event,
Expand Down Expand Up @@ -2221,7 +2221,7 @@ int memory_failure(unsigned long pfn, int flags)
* otherwise it may race with THP split.
* And the flag can't be set in get_hwpoison_page() since
* it is called by soft offline too and it is just called
* for !MF_COUNT_INCREASE. So here seems to be the best
* for !MF_COUNT_INCREASED. So here seems to be the best
* place.
*
* Don't need care about the above error handling paths for
Expand Down Expand Up @@ -2578,10 +2578,10 @@ static bool isolate_page(struct page *page, struct list_head *pagelist)

/*
* If we succeed to isolate the page, we grabbed another refcount on
* the page, so we can safely drop the one we got from get_any_pages().
* the page, so we can safely drop the one we got from get_any_page().
* If we failed to isolate the page, it means that we cannot go further
* and we will return an error, so drop the reference we got from
* get_any_pages() as well.
* get_any_page() as well.
*/
put_page(page);
return isolated;
Expand Down

0 comments on commit 5885c6a

Please sign in to comment.