[PATCH] mm: clean up local variables
Clean up a local variable with the same name as a variable in a larger block. Also move a variable into the block where it's actually used. Spotted by http://linuxicc.sourceforge.net/ Signed-off-by: Tobias Klauser <tklauser@nuerscht.ch> Signed-off-by: Andrew Morton <akpm@osdl.org> Signed-off-by: Linus Torvalds <torvalds@osdl.org>
This commit is contained in:
parent
152194aaa6
commit
cd105df459
2 changed files with 8 additions and 7 deletions
|
@ -854,7 +854,6 @@ static int __devinit cpuup_callback(struct notifier_block *nfb,
|
|||
struct kmem_list3 *l3 = NULL;
|
||||
int node = cpu_to_node(cpu);
|
||||
int memsize = sizeof(struct kmem_list3);
|
||||
struct array_cache *nc = NULL;
|
||||
|
||||
switch (action) {
|
||||
case CPU_UP_PREPARE:
|
||||
|
@ -891,6 +890,8 @@ static int __devinit cpuup_callback(struct notifier_block *nfb,
|
|||
/* Now we can go ahead with allocating the shared array's
|
||||
& array cache's */
|
||||
list_for_each_entry(cachep, &cache_chain, next) {
|
||||
struct array_cache *nc;
|
||||
|
||||
nc = alloc_arraycache(node, cachep->limit,
|
||||
cachep->batchcount);
|
||||
if (!nc)
|
||||
|
|
|
@ -1493,7 +1493,7 @@ asmlinkage long sys_swapon(const char __user * specialfile, int swap_flags)
|
|||
goto bad_swap;
|
||||
if (swap_header->info.nr_badpages > MAX_SWAP_BADPAGES)
|
||||
goto bad_swap;
|
||||
|
||||
|
||||
/* OK, set up the swap map and apply the bad block list */
|
||||
if (!(p->swap_map = vmalloc(maxpages * sizeof(short)))) {
|
||||
error = -ENOMEM;
|
||||
|
@ -1502,17 +1502,17 @@ asmlinkage long sys_swapon(const char __user * specialfile, int swap_flags)
|
|||
|
||||
error = 0;
|
||||
memset(p->swap_map, 0, maxpages * sizeof(short));
|
||||
for (i=0; i<swap_header->info.nr_badpages; i++) {
|
||||
int page = swap_header->info.badpages[i];
|
||||
if (page <= 0 || page >= swap_header->info.last_page)
|
||||
for (i = 0; i < swap_header->info.nr_badpages; i++) {
|
||||
int page_nr = swap_header->info.badpages[i];
|
||||
if (page_nr <= 0 || page_nr >= swap_header->info.last_page)
|
||||
error = -EINVAL;
|
||||
else
|
||||
p->swap_map[page] = SWAP_MAP_BAD;
|
||||
p->swap_map[page_nr] = SWAP_MAP_BAD;
|
||||
}
|
||||
nr_good_pages = swap_header->info.last_page -
|
||||
swap_header->info.nr_badpages -
|
||||
1 /* header page */;
|
||||
if (error)
|
||||
if (error)
|
||||
goto bad_swap;
|
||||
}
|
||||
|
||||
|
|
Loading…
Reference in a new issue