Messages in this thread | | | Date | Mon, 20 Jan 2020 08:36:28 +0800 | From | Wei Yang <> | Subject | Re: [PATCH 2/4] mm/page_alloc.c: rename free_pages_check_bad() to check_free_page_bad() |
| |
On Sun, Jan 19, 2020 at 02:07:04PM -0800, David Rientjes wrote: >On Sun, 19 Jan 2020, Wei Yang wrote: > >> free_pages_check_bad() is the counterpart of check_new_page_bad(), while >> their naming convention is a little different. >> >> Use verb at first and singular form. >> > >I think if you agree with the suggestion in patch 1/4 to fix the issue >with bad page reporting that it would likely be better to fold patches 2 >and 3 into that change.
I am ok with this, while would it be confusing for review?
-- Wei Yang Help you, Help me
| |