Messages in this thread Patch in this message | | | Date | Thu, 25 Oct 2007 19:14:31 -0700 (PDT) | From | David Rientjes <> | Subject | [patch 2/3] mempolicy: mpol_rebind_policy cleanup |
| |
Set the memory policy nodemask to the new nodemask on rebind only in one place. The only memory policy that does not need an associated mpolmask is MPOL_DEFAULT.
Cc: Andi Kleen <ak@suse.de> Cc: Christoph Lameter <clameter@sgi.com> Cc: Lee Schermerhorn <Lee.Schermerhorn@hp.com> Signed-off-by: Paul Jackson <pj@sgi.com> Signed-off-by: David Rientjes <rientjes@google.com> --- mm/mempolicy.c | 5 ++--- 1 files changed, 2 insertions(+), 3 deletions(-)
diff --git a/mm/mempolicy.c b/mm/mempolicy.c --- a/mm/mempolicy.c +++ b/mm/mempolicy.c @@ -1741,14 +1741,12 @@ static void mpol_rebind_policy(struct mempolicy *pol, case MPOL_INTERLEAVE: nodes_remap(tmp, pol->v.nodes, *mpolmask, *newmask); pol->v.nodes = tmp; - *mpolmask = *newmask; current->il_next = node_remap(current->il_next, *mpolmask, *newmask); break; case MPOL_PREFERRED: pol->v.preferred_node = node_remap(pol->v.preferred_node, *mpolmask, *newmask); - *mpolmask = *newmask; break; case MPOL_BIND: { nodemask_t nodes; @@ -1773,13 +1771,14 @@ static void mpol_rebind_policy(struct mempolicy *pol, kfree(pol->v.zonelist); pol->v.zonelist = zonelist; } - *mpolmask = *newmask; break; } default: BUG(); break; } + if (pol->policy != MPOL_DEFAULT) + *mpolmask = *newmask; } /* - 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/
| |