[PATCH] powerpc: fix two build warnings
authorArnd Bergmann <arnd@arndb.de>
Fri, 16 Dec 2005 21:45:27 +0000 (22:45 +0100)
committerPaul Mackerras <paulus@samba.org>
Mon, 9 Jan 2006 04:14:14 +0000 (15:14 +1100)
Building the arch/powerpc tree currently gives me
two warnings with gcc-4.0:

arch/powerpc/mm/imalloc.c: In function '__im_get_area':
arch/powerpc/mm/imalloc.c:225: warning: 'tmp' may be used uninitialized in this function
arch/powerpc/mm/hugetlbpage.c: In function 'hugetlb_get_unmapped_area':
arch/powerpc/mm/hugetlbpage.c:608: warning: unused variable 'vma'

both fixes are trivial.

Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Signed-off-by: Paul Mackerras <paulus@samba.org>
arch/powerpc/mm/hugetlbpage.c
arch/powerpc/mm/imalloc.c

index f6fe3eaf87a3d5609c3479bfffb77a00bcac4779..b51bb28c054bbb5e14007af0d9efee7be0806a3a 100644 (file)
@@ -620,7 +620,6 @@ unsigned long hugetlb_get_unmapped_area(struct file *file, unsigned long addr,
 {
        int lastshift;
        u16 areamask, curareas;
-       struct vm_area_struct *vma;
 
        if (HPAGE_SHIFT == 0)
                return -EINVAL;
index f9587bcc6a48f89f375269db28f4d827174e1e38..8b0c132bc163f174b512b7d48515e12d071f3732 100644 (file)
@@ -107,6 +107,7 @@ static int im_region_status(unsigned long v_addr, unsigned long size,
                if (v_addr < (unsigned long) tmp->addr + tmp->size)
                        break;
 
+       *vm = NULL;
        if (tmp) {
                if (im_region_overlaps(v_addr, size, tmp))
                        return IM_REGION_OVERLAP;
@@ -127,7 +128,6 @@ static int im_region_status(unsigned long v_addr, unsigned long size,
                }
        }
 
-       *vm = NULL;
        return IM_REGION_UNUSED;
 }