WARNING - OLD ARCHIVES

This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-devel

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

To: "Jiang, Yunhong" <yunhong.jiang@xxxxxxxxx>, Jeremy Fitzhardinge <jeremy@xxxxxxxx>
Subject: [Xen-devel] RE: [PATCH] Add config check for cpu hotplug code
From: "Jiang, Yunhong" <yunhong.jiang@xxxxxxxxx>
Date: Fri, 11 Dec 2009 17:54:56 +0800
Accept-language: en-US
Acceptlanguage: en-US
Cc: "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>, "Yu, Ke" <ke.yu@xxxxxxxxx>
Delivery-date: Fri, 11 Dec 2009 01:56:42 -0800
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <E2263E4A5B2284449EEBD0AAB751098418E63BD1DC@xxxxxxxxxxxxxxxxxxxxxxxxxxxx>
List-help: <mailto:xen-devel-request@lists.xensource.com?subject=help>
List-id: Xen developer discussion <xen-devel.lists.xensource.com>
List-post: <mailto:xen-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
References: <E2263E4A5B2284449EEBD0AAB751098418E634ABE6@xxxxxxxxxxxxxxxxxxxxxxxxxxxx> <4B216B67.5040104@xxxxxxxx> <E2263E4A5B2284449EEBD0AAB751098418E63BD1DC@xxxxxxxxxxxxxxxxxxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Thread-index: Acp54cPh9ZD+ied7TQaUIMW+R1/dJAAUea4AAATpSYA=
Thread-topic: [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
<Prev in Thread] Current Thread [Next in Thread>