arch/powerpc/platforms/pseries/hotplug-memory.c: fix section handling code
authorAndrew Morton <akpm@linux-foundation.org>
Wed, 10 Oct 2012 22:53:52 +0000 (15:53 -0700)
committerLinus Torvalds <torvalds@linux-foundation.org>
Wed, 10 Oct 2012 23:50:13 +0000 (08:50 +0900)
Fix

  arch/powerpc/platforms/pseries/hotplug-memory.c: In function 'pseries_remove_memblock':
  arch/powerpc/platforms/pseries/hotplug-memory.c:103:17: error: unused variable 'pfn' [-Werror=unused-variable]

Caused by commit d760afd4d257 ("memory-hotplug: suppress "Trying to free
nonexistent resource <XXXXXXXXXXXXXXXX-YYYYYYYYYYYYYYYY>" warning").

Reported-by: Stephen Rothwell <sfr@canb.auug.org.au>
Cc: Yasuaki Ishimatsu <isimatu.yasuaki@jp.fujitsu.com>
Tested-by: Stephen Rothwell <sfr@canb.auug.org.au>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Tested-by: Nathan Fontenot <nfont@linux.vnet.ibm.com>
Tested-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
arch/powerpc/platforms/pseries/hotplug-memory.c

index dc0a035e63bb5c549adbfa58660b15ef16abec90..2fe690fcaa1d89367455e8cb4c13fc1b7c7028a0 100644 (file)
@@ -101,7 +101,7 @@ static int pseries_remove_memblock(unsigned long base, unsigned int memblock_siz
        sections_to_remove = (memblock_size >> PAGE_SHIFT) / PAGES_PER_SECTION;
        for (i = 0; i < sections_to_remove; i++) {
                unsigned long pfn = start_pfn + i * PAGES_PER_SECTION;
-               ret = __remove_pages(zone, start_pfn,  PAGES_PER_SECTION);
+               ret = __remove_pages(zone, pfn, PAGES_PER_SECTION);
                if (ret)
                        return ret;
        }