Bug 6538 - memory.c:(.text+0x4cb21): undefined reference to `remove_memory'
Summary: memory.c:(.text+0x4cb21): undefined reference to `remove_memory'
Status: CLOSED CODE_FIX
Alias: None
Product: Drivers
Classification: Unclassified
Component: Other (show other bugs)
Hardware: i386 Linux
: P2 low
Assignee: drivers_other
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2006-05-12 12:09 UTC by Toralf Förster
Modified: 2006-05-13 08:48 UTC (History)
1 user (show)

See Also:
Kernel Version: 2.6.17-rc4
Subsystem:
Regression: ---
Bisected commit-id:

Attachments
kernel config (21.19 KB, text/plain)
2006-05-12 12:10 UTC, Toralf Förster 		Details
make log (38.78 KB, text/plain)
2006-05-12 12:10 UTC, Toralf Förster 		Details
fix hotplug build (477 bytes, patch)
2006-05-12 13:45 UTC, Nishanth Aravamudan 		Details | Diff
beter fix, maybe? (503 bytes, patch)
2006-05-12 13:49 UTC, Nishanth Aravamudan 		Details | Diff
Show Obsolete (1) Add an attachment (proposed patch, testcase, etc.)

Description Toralf Förster 2006-05-12 12:09:59 UTC 
Playing with different kernel configs I got:
...
  UPD     include/linux/compile.h
  CC      init/version.o
  LD      init/built-in.o
  LD      .tmp_vmlinux1
drivers/built-in.o: In function `memory_block_action':
memory.c:(.text+0x4cb21): undefined reference to `remove_memory'
make: *** [.tmp_vmlinux1] Error 1
Comment 1 Toralf Förster 2006-05-12 12:10:25 UTC 
Created attachment 8098 [details]
kernel config
Comment 2 Toralf Förster 2006-05-12 12:10:42 UTC 
Created attachment 8099 [details]
make log
Comment 3 Nishanth Aravamudan 2006-05-12 13:45:02 UTC 
Created attachment 8102 [details]
fix hotplug build

Does the attached patch fix it for you?

Thanks,
Nish
Comment 4 Nishanth Aravamudan 2006-05-12 13:49:49 UTC 
Created attachment 8105 [details]
beter fix, maybe?

There is a typo in the file, it's CONFIG_MEMORY_HOTPLUG, not
CONFIG_HOTPLUG_MEMORY, see if this fixes it, please?
Comment 5 Toralf Förster 2006-05-12 23:51:23 UTC 
right, with the better fix it compiles fine :-)
Comment 6 Nishanth Aravamudan 2006-05-13 08:48:48 UTC 
Great, thanks for the test, I have submitted the patch to mainline.

Thanks,
Nish
Note You need to log in before you can comment on or make changes to this bug.