* [PATCH 1/2] mempolicy:del case MPOL_INTERLEAVE in policy_zonelist()
@ 2010-03-18 12:46 Bob Liu
2010-03-19 8:48 ` David Rientjes
0 siblings, 1 reply; 2+ messages in thread
From: Bob Liu @ 2010-03-18 12:46 UTC (permalink / raw)
To: akpm; +Cc: linux-mm, andi, rientjes, lee.schermerhorn, Bob Liu
From: Bob Liu <lliubbo@gmail.com>
In policy_zonelist() mode MPOL_INTERLEAVE shouldn't happen,
so fall through to BUG() instead of break to return.I also fix
the comment.
Signed-off-by: Bob Liu <lliubbo@gmail.com>
---
mm/mempolicy.c | 4 +---
1 files changed, 1 insertions(+), 3 deletions(-)
diff --git a/mm/mempolicy.c b/mm/mempolicy.c
index 643f66e..b88e914 100644
--- a/mm/mempolicy.c
+++ b/mm/mempolicy.c
@@ -1441,15 +1441,13 @@ static struct zonelist *policy_zonelist(gfp_t gfp, struct mempolicy *policy)
/*
* Normally, MPOL_BIND allocations are node-local within the
* allowed nodemask. However, if __GFP_THISNODE is set and the
- * current node is part of the mask, we use the zonelist for
+ * current node isn't part of the mask, we use the zonelist for
* the first node in the mask instead.
*/
if (unlikely(gfp & __GFP_THISNODE) &&
unlikely(!node_isset(nd, policy->v.nodes)))
nd = first_node(policy->v.nodes);
break;
- case MPOL_INTERLEAVE: /* should not happen */
- break;
default:
BUG();
}
--
1.5.6.3
--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org. For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH 1/2] mempolicy:del case MPOL_INTERLEAVE in policy_zonelist()
2010-03-18 12:46 [PATCH 1/2] mempolicy:del case MPOL_INTERLEAVE in policy_zonelist() Bob Liu
@ 2010-03-19 8:48 ` David Rientjes
0 siblings, 0 replies; 2+ messages in thread
From: David Rientjes @ 2010-03-19 8:48 UTC (permalink / raw)
To: Bob Liu; +Cc: Andrew Morton, linux-mm, Andi Kleen, Lee Schermerhorn
On Thu, 18 Mar 2010, Bob Liu wrote:
> From: Bob Liu <lliubbo@gmail.com>
>
> In policy_zonelist() mode MPOL_INTERLEAVE shouldn't happen,
> so fall through to BUG() instead of break to return.I also fix
> the comment.
>
> Signed-off-by: Bob Liu <lliubbo@gmail.com>
Acked-by: David Rientjes <rientjes@google.com>
--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org. For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2010-03-19 8:48 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2010-03-18 12:46 [PATCH 1/2] mempolicy:del case MPOL_INTERLEAVE in policy_zonelist() Bob Liu
2010-03-19 8:48 ` David Rientjes
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.