Re: [PATCH] mm: put activate_page_pvecs and others pagevec together

From: Andrew Morton
Date: Wed Apr 27 2016 - 17:04:07 EST


On Mon, 11 Apr 2016 09:36:46 +0800 "Ming Li" <mingli199x@xxxxxx> wrote:

> hi, i`m sorry, I made some mistakes in last email. I have been studying mm and learn age_activate_anon(), at the very beginning I felt confuse when I saw activate_page_pvecs, after I learned the whole thing I understood that it's similar with other pagevec's function. Can we put it with other pagevec together? I think it is easier for newbies to read and understand.

Welcome to kernel development ;)

Your email client is mangling the patches in several ways. Please sort
that out for next time - mail yourself a patch, check that it applies.

The changelog text wasn't very conventional. I rewrote it to

: Put the activate_page_pvecs definition next to those of the other
: pagevecs, for clarity.

and I changed the title to

Subject: mm/swap.c: put activate_page_pvecs and other pagevecs together

because "mm" is a big place!