Bug 54091 - ia64: linux-3.8.0/drivers/acpi/osl.c:629:29: error: ‘max_low_pfn_mapped’ undeclared
Summary: ia64: linux-3.8.0/drivers/acpi/osl.c:629:29: error: ‘max_low_pfn_mapped’ unde...
Status: CLOSED CODE_FIX
Alias: None
Product: ACPI
Classification: Unclassified
Component: Other (show other bugs)
Hardware: IA-64 Linux
: P1 normal
Assignee: Aaron Lu
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-02-19 11:49 UTC by Dennis Schridde
Modified: 2013-02-28 04:57 UTC (History)
2 users (show)

See Also:
Kernel Version: 3.8.0
Subsystem:
Regression: Yes
Bisected commit-id:

Attachments
config-3.8.0 (54.07 KB, application/octet-stream)
2013-02-19 11:49 UTC, Dennis Schridde 		Details
Add an attachment (proposed patch, testcase, etc.)

Description Dennis Schridde 2013-02-19 11:49:45 UTC 
Created attachment 93531 [details]
config-3.8.0

CC      drivers/acpi/osl.o
/usr/src/linux-3.8.0/drivers/acpi/osl.c: In function ‘acpi_initrd_override’:
/usr/src/linux-3.8.0/drivers/acpi/osl.c:629:29: error: ‘max_low_pfn_mapped’ undeclared (first use in this function)
/usr/src/linux-3.8.0/drivers/acpi/osl.c:629:29: note: each undeclared identifier is reported only once for each function it appears in
make[4]: *** [drivers/acpi/osl.o] Error 1
Comment 1 Dennis Schridde 2013-02-19 11:51:31 UTC 
From a quick grep it appears that max_low_pfn_mapped exists on x86 only, which is why it fails on my ia64 machine.
Comment 2 Aaron Lu 2013-02-21 05:40:17 UTC 
Hi Dennis,
Thanks for the bug report.

Hi Thomas,
Can you please take a look at this? Thanks.
Comment 3 Thomas Renninger 2013-02-22 13:13:27 UTC 
Patches sent, you should be added in CC.
Comment 4 Aaron Lu 2013-02-25 01:12:13 UTC 
Thanks Thomas.

Hi Dennis,
Can you please give the patch a test and if everything works OK, give your tested-by tag by replying to Thomas' patches? Thanks.
Comment 5 Dennis Schridde 2013-02-25 17:29:00 UTC 
Hi Thomas and Aaron!

The 2 patches make the kernel compile on my ia64 box. I am not sure how to formally "give my tested-by tag", so I just replied to the email, stating that the patches work.

Thanks a lot for the quick fix!
Comment 6 Aaron Lu 2013-02-26 00:26:49 UTC 
Hi Dennis,

It's OK, HPA will take these patches, thanks for the report.
Comment 7 Aaron Lu 2013-02-28 04:57:03 UTC 
commit 565d956a7e191e95d81f515196951715f2616e6b
Author: Thomas Renninger <trenn@suse.de>
Date:   Fri Feb 22 14:12:22 2013 +0100

    ACPI: Overriding ACPI tables via initrd only works with an initrd and on X86
Note You need to log in before you can comment on or make changes to this bug.