diff options
| author | Qian Cai <cai@lca.pw> | 2019-03-01 09:20:40 -0500 |
|---|---|---|
| committer | Michael Ellerman <mpe@ellerman.id.au> | 2019-03-01 22:43:05 -0500 |
| commit | c38ca265525a00d635219450e8fcc858082ff630 (patch) | |
| tree | f839da0bab6e2e0ce29811b2ba7eeea0ef51721e | |
| parent | 8132cf115efc3b3684bb5fd3bfdf6860886f0e47 (diff) | |
powerpc/mm: fix "section_base" set but not used
The commit 24b6d4164348 ("mm: pass the vmem_altmap to vmemmap_free")
removed a line in vmemmap_free(),
altmap = to_vmem_altmap((unsigned long) section_base);
but left a variable no longer used.
arch/powerpc/mm/init_64.c: In function 'vmemmap_free':
arch/powerpc/mm/init_64.c:277:16: error: variable 'section_base' set but
not used [-Werror=unused-but-set-variable]
Signed-off-by: Qian Cai <cai@lca.pw>
Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
| -rw-r--r-- | arch/powerpc/mm/init_64.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/arch/powerpc/mm/init_64.c b/arch/powerpc/mm/init_64.c index a5091c034747..a4c155af1597 100644 --- a/arch/powerpc/mm/init_64.c +++ b/arch/powerpc/mm/init_64.c | |||
| @@ -274,7 +274,6 @@ void __ref vmemmap_free(unsigned long start, unsigned long end, | |||
| 274 | 274 | ||
| 275 | for (; start < end; start += page_size) { | 275 | for (; start < end; start += page_size) { |
| 276 | unsigned long nr_pages, addr; | 276 | unsigned long nr_pages, addr; |
| 277 | struct page *section_base; | ||
| 278 | struct page *page; | 277 | struct page *page; |
| 279 | 278 | ||
| 280 | /* | 279 | /* |
| @@ -290,7 +289,6 @@ void __ref vmemmap_free(unsigned long start, unsigned long end, | |||
| 290 | continue; | 289 | continue; |
| 291 | 290 | ||
| 292 | page = pfn_to_page(addr >> PAGE_SHIFT); | 291 | page = pfn_to_page(addr >> PAGE_SHIFT); |
| 293 | section_base = pfn_to_page(vmemmap_section_start(start)); | ||
| 294 | nr_pages = 1 << page_order; | 292 | nr_pages = 1 << page_order; |
| 295 | base_pfn = PHYS_PFN(addr); | 293 | base_pfn = PHYS_PFN(addr); |
| 296 | 294 | ||
