diff options
| author | KOSAKI Motohiro <kosaki.motohiro@jp.fujitsu.com> | 2010-03-05 16:41:59 -0500 |
|---|---|---|
| committer | Linus Torvalds <torvalds@linux-foundation.org> | 2010-03-06 14:26:25 -0500 |
| commit | da0aa138944311e6745a00ac3d88f03e8d9a46c4 (patch) | |
| tree | f8a88b12652fb2e4ed6775b0cc32da837ca2d949 | |
| parent | d96ae5309165d9ed7c008a178238977b73595cd9 (diff) | |
mm/mempolicy.c: fix indentation of the comments of do_migrate_pages
Currently, do_migrate_pages() have very long comment and this is not
indent properly. I often misunderstand it is function starting commnents
and confused it.
this patch fixes it.
note: this patch doesn't break 80 column rule. I guess original
author intended this indentaion, but an accident corrupted it.
Signed-off-by: KOSAKI Motohiro <kosaki.motohiro@jp.fujitsu.com>
Reviewed-by: Christoph Lameter <cl@linux-foundation.org>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
| -rw-r--r-- | mm/mempolicy.c | 60 |
1 files changed, 30 insertions, 30 deletions
diff --git a/mm/mempolicy.c b/mm/mempolicy.c index 44dd9d1521ec..bda230e52acd 100644 --- a/mm/mempolicy.c +++ b/mm/mempolicy.c | |||
| @@ -888,36 +888,36 @@ int do_migrate_pages(struct mm_struct *mm, | |||
| 888 | if (err) | 888 | if (err) |
| 889 | goto out; | 889 | goto out; |
| 890 | 890 | ||
| 891 | /* | 891 | /* |
| 892 | * Find a 'source' bit set in 'tmp' whose corresponding 'dest' | 892 | * Find a 'source' bit set in 'tmp' whose corresponding 'dest' |
| 893 | * bit in 'to' is not also set in 'tmp'. Clear the found 'source' | 893 | * bit in 'to' is not also set in 'tmp'. Clear the found 'source' |
| 894 | * bit in 'tmp', and return that <source, dest> pair for migration. | 894 | * bit in 'tmp', and return that <source, dest> pair for migration. |
| 895 | * The pair of nodemasks 'to' and 'from' define the map. | 895 | * The pair of nodemasks 'to' and 'from' define the map. |
| 896 | * | 896 | * |
| 897 | * If no pair of bits is found that way, fallback to picking some | 897 | * If no pair of bits is found that way, fallback to picking some |
| 898 | * pair of 'source' and 'dest' bits that are not the same. If the | 898 | * pair of 'source' and 'dest' bits that are not the same. If the |
| 899 | * 'source' and 'dest' bits are the same, this represents a node | 899 | * 'source' and 'dest' bits are the same, this represents a node |
| 900 | * that will be migrating to itself, so no pages need move. | 900 | * that will be migrating to itself, so no pages need move. |
| 901 | * | 901 | * |
| 902 | * If no bits are left in 'tmp', or if all remaining bits left | 902 | * If no bits are left in 'tmp', or if all remaining bits left |
| 903 | * in 'tmp' correspond to the same bit in 'to', return false | 903 | * in 'tmp' correspond to the same bit in 'to', return false |
| 904 | * (nothing left to migrate). | 904 | * (nothing left to migrate). |
| 905 | * | 905 | * |
| 906 | * This lets us pick a pair of nodes to migrate between, such that | 906 | * This lets us pick a pair of nodes to migrate between, such that |
| 907 | * if possible the dest node is not already occupied by some other | 907 | * if possible the dest node is not already occupied by some other |
| 908 | * source node, minimizing the risk of overloading the memory on a | 908 | * source node, minimizing the risk of overloading the memory on a |
| 909 | * node that would happen if we migrated incoming memory to a node | 909 | * node that would happen if we migrated incoming memory to a node |
| 910 | * before migrating outgoing memory source that same node. | 910 | * before migrating outgoing memory source that same node. |
| 911 | * | 911 | * |
| 912 | * A single scan of tmp is sufficient. As we go, we remember the | 912 | * A single scan of tmp is sufficient. As we go, we remember the |
| 913 | * most recent <s, d> pair that moved (s != d). If we find a pair | 913 | * most recent <s, d> pair that moved (s != d). If we find a pair |
| 914 | * that not only moved, but what's better, moved to an empty slot | 914 | * that not only moved, but what's better, moved to an empty slot |
| 915 | * (d is not set in tmp), then we break out then, with that pair. | 915 | * (d is not set in tmp), then we break out then, with that pair. |
| 916 | * Otherwise when we finish scannng from_tmp, we at least have the | 916 | * Otherwise when we finish scannng from_tmp, we at least have the |
| 917 | * most recent <s, d> pair that moved. If we get all the way through | 917 | * most recent <s, d> pair that moved. If we get all the way through |
| 918 | * the scan of tmp without finding any node that moved, much less | 918 | * the scan of tmp without finding any node that moved, much less |
| 919 | * moved to an empty node, then there is nothing left worth migrating. | 919 | * moved to an empty node, then there is nothing left worth migrating. |
| 920 | */ | 920 | */ |
| 921 | 921 | ||
| 922 | tmp = *from_nodes; | 922 | tmp = *from_nodes; |
| 923 | while (!nodes_empty(tmp)) { | 923 | while (!nodes_empty(tmp)) { |
