Re: [PATCH 15/34] mm: migration: clean up unmap_and_move()

Greg KH
2012-07-25T18:03:49Z

On Wed, Jul 25, 2012 at 05:04:34PM +0100, Mel Gorman wrote:
> On Wed, Jul 25, 2012 at 08:45:26AM -0700, Greg KH wrote:
> > On Mon, Jul 23, 2012 at 02:38:28PM +0100, Mel Gorman wrote:
> > > commit 0dabec93de633a87adfbbe1d800a4c56cd19d73b upstream.
> > >
> > > Stable note: Not tracked in Bugzilla. This patch makes later patches
> > > easier to apply but has no other impact.
> > >
> > > unmap_and_move() is one a big messy function. Clean it up.
> > >
> > > Signed-off-by: Minchan Kim <minchan.kim@gmail.com>
> > > Reviewed-by: KOSAKI Motohiro <kosaki.motohiro@jp.fujitsu.com>
> > > Cc: Johannes Weiner <hannes@cmpxchg.org>
> > > Cc: KAMEZAWA Hiroyuki <kamezawa.hiroyu@jp.fujitsu.com>
> > > Cc: Mel Gorman <mgorman@suse.de>
> > > Cc: Rik van Riel <riel@redhat.com>
> > > Cc: Michal Hocko <mhocko@suse.cz>
> > > Cc: Andrea Arcangeli <aarcange@redhat.com>
> > > Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
> > > Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
> > > ---
> > > mm/migrate.c | 59 ++++++++++++++++++++++++++++++++--------------------------
> > > 1 file changed, 33 insertions(+), 26 deletions(-)
> >
> > Mel, you didn't sign-off-on this patch. Any reason why?
> >
>
> Another patch that was merged to the distribution kernel before picked
> up by mainline. In this case, I copied across the signed-off-bys and
> missed my own
>
> Signed-off-by: Mel Gorman <mgorman@suse.de>

Thanks, I've now added it.

greg k-h