| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-Id: <1388646744-15608-15-git-send-email-minchan@kernel.org>
Date: Thu, 2 Jan 2014 16:12:22 +0900
From: Minchan Kim <minchan@...nel.org>
To: linux-mm@...ck.org, linux-kernel@...r.kernel.org
Cc: Andrew Morton <akpm@...ux-foundation.org>,
Mel Gorman <mgorman@...e.de>, Hugh Dickins <hughd@...gle.com>,
Dave Hansen <dave.hansen@...el.com>,
Rik van Riel <riel@...hat.com>,
KOSAKI Motohiro <kosaki.motohiro@...fujitsu.com>,
Michel Lespinasse <walken@...gle.com>,
Johannes Weiner <hannes@...xchg.org>,
John Stultz <john.stultz@...aro.org>,
Dhaval Giani <dhaval.giani@...il.com>,
"H. Peter Anvin" <hpa@...or.com>,
Android Kernel Team <kernel-team@...roid.com>,
Robert Love <rlove@...gle.com>, Mel Gorman <mel@....ul.ie>,
Dmitry Adamushko <dmitry.adamushko@...il.com>,
Dave Chinner <david@...morbit.com>, Neil Brown <neilb@...e.de>,
Andrea Righi <andrea@...terlinux.com>,
Andrea Arcangeli <aarcange@...hat.com>,
"Aneesh Kumar K.V" <aneesh.kumar@...ux.vnet.ibm.com>,
Mike Hommey <mh@...ndium.org>, Taras Glek <tglek@...illa.com>,
Jan Kara <jack@...e.cz>,
KOSAKI Motohiro <kosaki.motohiro@...il.com>,
Rob Clark <robdclark@...il.com>, Jason Evans <je@...com>,
Minchan Kim <minchan@...nel.org>
Subject: [PATCH v10 14/16] vrange: Change purged with hint
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 <mel@....ul.ie>
Cc: Hugh Dickins <hughd@...gle.com>
Cc: Dave Hansen <dave.hansen@...el.com>
Cc: Rik van Riel <riel@...hat.com>
Cc: KOSAKI Motohiro <kosaki.motohiro@...il.com>
Cc: Michel Lespinasse <walken@...gle.com>
Cc: Johannes Weiner <hannes@...xchg.org>
Cc: John Stultz <john.stultz@...aro.org>
Signed-off-by: Minchan Kim <minchan@...nel.org>
---
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@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists