Commit 5885c6a6 authored by Miaohe Lin's avatar Miaohe Lin Committed by Andrew Morton
Browse files

mm: memory-failure: minor cleanup for comments and codestyle 

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: default avatarMiaohe Lin <linmiaohe@huawei.com>
Acked-by: default avatarNaoya Horiguchi <naoya.horiguchi@nec.com>
Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
parent e9c36f7a

mm/memory-failure.c

+6 −6
Original line number Diff line number Diff line
@@ -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.

			 */
@@ -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,
@@ -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
@@ -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;