Skip to contentSkip to conversation

mm/page_poison: update comment after code moved

PATCH In review Michael S. Tsirkin 2 participants 3 messages Feb 7, 2019 – Feb 8, 2019 Memory management

lore
Share & tools
lore

Michael S. Tsirkin

author
#0 lore raw

PATCH 1/1 mm/page_poison: update comment after code moved

mm/debug-pagealloc.c is no more, so of course header now needs to be updated. This seems like something checkpatch should be able to catch - worth looking into?

Cc [email protected]
Cc [email protected]
Cc Linus Torvalds <[email protected]>
Cc [email protected]
Fixes 8823b1dbc05f ("mm/page_poison.c: enable PAGE_POISONING as a separate option")
---
 include/linux/poison.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
include/linux/poison.h +1 -1
index 15927ebc22f2..5046bad0c1c5 100644
@@ -30,7 +30,7 @@
  */
 #define TIMER_ENTRY_STATIC	((void *) 0x300 + POISON_POINTER_DELTA)
 
-/********** mm/debug-pagealloc.c **********/
+/********** mm/page_poison.c **********/
 #ifdef CONFIG_PAGE_POISONING_ZERO
 #define PAGE_POISON 0x00
 #else
Signature · 1 line
MST
maintainer
#1 lore raw
Quoted 9 lines · On Thu, 7 Feb 2019 14:11:16 -0500 "Michael S. Tsirkin" <[email protected]> wrote:

mm/debug-pagealloc.c is no more, so of course header now needs to be updated. This seems like something checkpatch should be able to catch - worth looking into?

Cc [email protected]
Cc [email protected]
Cc Linus Torvalds <[email protected]>
Cc [email protected]
Fixes 8823b1dbc05f ("mm/page_poison.c: enable PAGE_POISONING as a separate option")

Please send along a signed-off-by: for this.

author
#2 lore raw
Quoted 6 lines · Andrew Morton
Quoted 4 lines · On Thu, 7 Feb 2019 14:11:16 -0500 "Michael S. Tsirkin" <[email protected]> wrote:

mm/debug-pagealloc.c is no more, so of course header now needs to be updated. This seems like something checkpatch should be able to catch - worth looking into?

Quoted 7 lines
Quoted 5 lines
Cc [email protected]
Cc [email protected]
Cc Linus Torvalds <[email protected]>
Cc [email protected]
Fixes 8823b1dbc05f ("mm/page_poison.c: enable PAGE_POISONING as a separate option")

Please send along a signed-off-by: for this.

Oh sorry.

Signed-off-by Michael S. Tsirkin <[email protected]>

Good enough or should I repost?