[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Xen-devel] RE: [PATCH] Add config check for cpu hotplug code



Seems I made a mistake on the experimental, please ignore it. Yes, you are 
right, it is because missing module license. With following patch, it can be 
load successfully.

Thanks
Yunhong Jiang

>From 08e246131feef5c7b6e5ac63916066244be3067a Mon Sep 17 00:00:00 2001
From: yjiang5 <yjiang5@xxxxxxxxxxxxxxxxxxxxxxxx>
Date: Fri, 11 Dec 2009 17:22:16 -0500
Subject: [PATCH] Add module license definition to xen's acpi_processor.

Add MODULE_GPL license definition to xen's acpi processor file, to make it can 
load sucessfully.

Signed-off-by: Jiang, Yunhong <yunhong.jiang@xxxxxxxxx>
---
 drivers/xen/acpi_processor.c |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/drivers/xen/acpi_processor.c b/drivers/xen/acpi_processor.c
index edc3ca0..f2f59cd 100644
--- a/drivers/xen/acpi_processor.c
+++ b/drivers/xen/acpi_processor.c
@@ -408,3 +408,4 @@ static int __init xen_acpi_processor_extcntl_init(void)
 }
 
 subsys_initcall(xen_acpi_processor_extcntl_init);
+MODULE_LICENSE("GPL");
-- 
1.6.3




>-----Original Message-----
>From: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
>[mailto:xen-devel-bounces@xxxxxxxxxxxxxxxxxxx] On Behalf Of Jiang, Yunhong
>Sent: Friday, December 11, 2009 3:46 PM
>To: Jeremy Fitzhardinge
>Cc: xen-devel@xxxxxxxxxxxxxxxxxxx; Yu, Ke
>Subject: [Xen-devel] RE: [PATCH] Add config check for cpu hotplug code
>
>I just tested disable acpi processor for my patch, so didn't notice this issue.
>
>Today I try to build the drivers/xen/acpi_processor.c as module and get 
>compile issue,
>seems it mainly because the xen code in acpi directory is not cleanup still.
>I will talk with Ke for the method. Ke has suggested me to make the
>processor_cntl_xen_notify as a callback registered.
>
>--jyh
>
>The patch I used to switch to module:
>--- a/drivers/xen/Kconfig
>+++ b/drivers/xen/Kconfig
>@@ -182,5 +182,5 @@ config XEN_MCE
> config ACPI_PROCESSOR_XEN
>    tristate
>    depends on XEN_DOM0 && ACPI_PROCESSOR && CPU_FREQ
>-   default y
>+   default m
>..
>The build error result:
>drivers/built-in.o: In function `acpi_processor_get_power_info':
>processor_idle.c:(.text+0x42698): undefined reference to
>`processor_cntl_xen_power_cache'
>drivers/built-in.o: In function `xen_acpi_processor_cst_has_changed':
>(.text+0x42a32): undefined reference to `processor_cntl_xen_notify'
>drivers/built-in.o: In function `xen_acpi_processor_get_performance':
>(.text+0x43d40): undefined reference to `processor_cntl_xen_notify'
>drivers/built-in.o: In function `xen_acpi_processor_ppc_has_changed':
>(.text+0x44462): undefined reference to `processor_cntl_xen_notify'
>drivers/built-in.o: In function `acpi_processor_hotplug_notify':
>processor_core.c:(.ref.text+0xb85): undefined reference to
>`processor_cntl_xen_notify'
>processor_core.c:(.ref.text+0xbc5): undefined reference to
>`processor_cntl_xen_notify'
>drivers/built-in.o:processor_core.c:(.ref.text+0xbee): more undefined 
>references to
>`processor_cntl_xen_notify' follow
>
>--jyh
>
>>-----Original Message-----
>>From: Jeremy Fitzhardinge [mailto:jeremy@xxxxxxxx]
>>Sent: Friday, December 11, 2009 5:43 AM
>>To: Jiang, Yunhong
>>Cc: xen-devel@xxxxxxxxxxxxxxxxxxx
>>Subject: Re: [PATCH] Add config check for cpu hotplug code
>>
>>On 12/09/09 20:14, Jiang, Yunhong wrote:
>>> For acpi_parser branch.
>>>
>>> Add CONFIG check to xen_get_apic_id() and xen_hotplug_notifier().
>>>
>>
>>xen/acpi_processor doesn't work as a module.  The module load fails with:
>>
>>acpi_processor: Unknown symbol xen_domain_type
>>acpi_processor: Unknown symbol xen_start_info
>>acpi_processor: Unknown symbol hypercall_page
>>
>>I think its missing a module license.
>>
>>     J
>
>_______________________________________________
>Xen-devel mailing list
>Xen-devel@xxxxxxxxxxxxxxxxxxx
>http://lists.xensource.com/xen-devel

Attachment: 0001-Add-module-license-definition-to-xen-s-acpi_processo.patch
Description: 0001-Add-module-license-definition-to-xen-s-acpi_processo.patch

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.