Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751637AbaABHOI (ORCPT ); Thu, 2 Jan 2014 02:14:08 -0500 Received: from LGEMRELSE7Q.lge.com ([156.147.1.151]:43662 "EHLO LGEMRELSE7Q.lge.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751434AbaABHNQ (ORCPT ); Thu, 2 Jan 2014 02:13:16 -0500 X-AuditID: 9c930197-b7c4aae000003d84-59-52c5118b9ea9 From: Minchan Kim To: linux-mm@kvack.org, linux-kernel@vger.kernel.org Cc: Andrew Morton , Mel Gorman , Hugh Dickins , Dave Hansen , Rik van Riel , KOSAKI Motohiro , Michel Lespinasse , Johannes Weiner , John Stultz , Dhaval Giani , "H. Peter Anvin" , Android Kernel Team , Robert Love , Mel Gorman , Dmitry Adamushko , Dave Chinner , Neil Brown , Andrea Righi , Andrea Arcangeli , "Aneesh Kumar K.V" , Mike Hommey , Taras Glek , Jan Kara , KOSAKI Motohiro , Rob Clark , Jason Evans , Minchan Kim Subject: [PATCH v10 14/16] vrange: Change purged with hint Date: Thu, 2 Jan 2014 16:12:22 +0900 Message-Id: <1388646744-15608-15-git-send-email-minchan@kernel.org> X-Mailer: git-send-email 1.8.5.2 In-Reply-To: <1388646744-15608-1-git-send-email-minchan@kernel.org> References: <1388646744-15608-1-git-send-email-minchan@kernel.org> X-Brightmail-Tracker: AAAAAA== Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 4692 Lines: 155 struct vrange has a purged field which is just flag to express the range was purged or not so what we need is just a bit. It means it's too bloated. This patch changes the name with hint so upcoming patch will use other extra bitfield for other purpose. Cc: Mel Gorman Cc: Hugh Dickins Cc: Dave Hansen Cc: Rik van Riel Cc: KOSAKI Motohiro Cc: Michel Lespinasse Cc: Johannes Weiner Cc: John Stultz Signed-off-by: Minchan Kim --- include/linux/vrange_types.h | 3 ++- mm/vrange.c | 39 ++++++++++++++++++++++++++++++--------- 2 files changed, 32 insertions(+), 10 deletions(-) diff --git a/include/linux/vrange_types.h b/include/linux/vrange_types.h index c4ef8b69a0a1..d42b0e7d7343 100644 --- a/include/linux/vrange_types.h +++ b/include/linux/vrange_types.h @@ -20,7 +20,8 @@ struct vrange_root { struct vrange { struct interval_tree_node node; struct vrange_root *owner; - int purged; + /* purged */ + unsigned long hint; struct list_head lru; atomic_t refcount; }; diff --git a/mm/vrange.c b/mm/vrange.c index 4e0775b722af..df01c6b084bf 100644 --- a/mm/vrange.c +++ b/mm/vrange.c @@ -29,6 +29,24 @@ struct vrange_walker { struct list_head *pagelist; }; +#define VRANGE_PURGED_MARK 0 + +void mark_purge(struct vrange *range) +{ + range->hint |= (1 << VRANGE_PURGED_MARK); +} + +void clear_purge(struct vrange *range) +{ + range->hint &= ~(1 << VRANGE_PURGED_MARK); +} + +bool vrange_purged(struct vrange *range) +{ + bool purged = range->hint & (1 << VRANGE_PURGED_MARK); + return purged; +} + static inline unsigned long vrange_size(struct vrange *range) { return range->node.last + 1 - range->node.start; @@ -217,7 +235,7 @@ static struct vrange *__vrange_alloc(gfp_t flags) return vrange; vrange->owner = NULL; - vrange->purged = 0; + vrange->hint = 0; INIT_LIST_HEAD(&vrange->lru); atomic_set(&vrange->refcount, 1); @@ -288,14 +306,17 @@ static inline void __vrange_set(struct vrange *range, { range->node.start = start_idx; range->node.last = end_idx; - range->purged = purged; + if (purged) + mark_purge(range); + else + clear_purge(range); } static inline void __vrange_resize(struct vrange *range, unsigned long start_idx, unsigned long end_idx) { struct vrange_root *vroot = range->owner; - bool purged = range->purged; + bool purged = vrange_purged(range); __vrange_remove(range); __vrange_lru_del(range); @@ -341,7 +362,7 @@ static int vrange_add(struct vrange_root *vroot, start_idx = min_t(unsigned long, start_idx, node->start); end_idx = max_t(unsigned long, end_idx, node->last); - purged |= range->purged; + purged |= vrange_purged(range); __vrange_remove(range); __vrange_put(range); @@ -383,7 +404,7 @@ static int vrange_remove(struct vrange_root *vroot, next = interval_tree_iter_next(node, start_idx, end_idx); range = vrange_from_node(node); - *purged |= range->purged; + *purged |= vrange_purged(range); if (start_idx <= node->start && end_idx >= node->last) { /* argumented range covers the range fully */ @@ -409,7 +430,7 @@ static int vrange_remove(struct vrange_root *vroot, used_new = true; __vrange_resize(range, node->start, start_idx - 1); __vrange_set(new_range, end_idx + 1, last, - range->purged); + vrange_purged(range)); __vrange_add(new_range, vroot); break; } @@ -492,7 +513,7 @@ int vrange_fork(struct mm_struct *new_mm, struct mm_struct *old_mm) if (!new_range) goto fail; __vrange_set(new_range, range->node.start, - range->node.last, range->purged); + range->node.last, vrange_purged(range)); __vrange_add(new_range, new); } @@ -736,7 +757,7 @@ bool vrange_addr_purged(struct vm_area_struct *vma, unsigned long addr) vrange_lock(vroot); range = __vrange_find(vroot, vstart_idx, vstart_idx); - if (range && range->purged) + if (range && vrange_purged(range)) ret = true; vrange_unlock(vroot); @@ -753,7 +774,7 @@ static void do_purge(struct vrange_root *vroot, node = interval_tree_iter_first(&vroot->v_rb, start_idx, end_idx); while (node) { range = container_of(node, struct vrange, node); - range->purged = true; + mark_purge(range); node = interval_tree_iter_next(node, start_idx, end_idx); } } -- 1.7.9.5 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/