Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 7c14aa85 authored by Mel Gorman's avatar Mel Gorman Committed by Gerrit - the friendly Code Review server
Browse files

mm, compaction: rearrange compact_control 

compact_control spans two cache lines with write-intensive lines on
both.  Rearrange so the most write-intensive fields are in the same
cache line.  This has a negligible impact on the overall performance of
compaction and is more a tidying exercise than anything.

Change-Id: Id20738a737e3ca5e355b3f183f6b1d71ba85189b
Link: http://lkml.kernel.org/r/20190118175136.31341-3-mgorman@techsingularity.net


Signed-off-by: default avatarMel Gorman <mgorman@techsingularity.net>
Acked-by: default avatarVlastimil Babka <vbabka@suse.cz>
Cc: Andrea Arcangeli <aarcange@redhat.com>
Cc: Dan Carpenter <dan.carpenter@oracle.com>
Cc: David Rientjes <rientjes@google.com>
Cc: YueHaibing <yuehaibing@huawei.com>
Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
Git-commit: c5943b9c5312d4fa23175ff146e901b865e4a60a
Git-repo: git://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git


Signed-off-by: default avatarVinayak Menon <vinmenon@codeaurora.org>
parent 0cbe7fd4

mm/internal.h

+3 −3
Original line number Diff line number Diff line
@@ -203,14 +203,14 @@ extern int user_min_free_kbytes; 
struct compact_control {

	struct list_head freepages;	/* List of free pages to migrate to */

	struct list_head migratepages;	/* List of pages being migrated */

	struct zone *zone;

	unsigned int nr_freepages;	/* Number of isolated free pages */

	unsigned int nr_migratepages;	/* Number of pages to migrate */

	unsigned long total_migrate_scanned;

	unsigned long total_free_scanned;

	unsigned long free_pfn;		/* isolate_freepages search base */

	unsigned long migrate_pfn;	/* isolate_migratepages search base */

	unsigned long last_migrated_pfn;/* Not yet flushed page being freed */

	struct zone *zone;

	unsigned long total_migrate_scanned;

	unsigned long total_free_scanned;

	const gfp_t gfp_mask;		/* gfp mask of a direct compactor */

	int order;			/* order a direct compactor needs */

	int migratetype;		/* migratetype of direct compactor */