2013-07-15 14:23:46

by Julien Grall

[permalink] [raw]
Subject: [PATCH] xen/arm: enable PV control for ARM

Enable power management from the toolstack for ARM guest.

Signed-off-by: Julien Grall <[email protected]>
---
drivers/xen/Makefile | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/drivers/xen/Makefile b/drivers/xen/Makefile
index 2bf461a..a5f12bd 100644
--- a/drivers/xen/Makefile
+++ b/drivers/xen/Makefile
@@ -1,9 +1,8 @@
ifneq ($(CONFIG_ARM),y)
-obj-y += manage.o
obj-$(CONFIG_HOTPLUG_CPU) += cpu_hotplug.o
endif
obj-$(CONFIG_X86) += fallback.o
-obj-y += grant-table.o features.o events.o balloon.o time.o
+obj-y += grant-table.o features.o events.o balloon.o time.o manage.o
obj-y += xenbus/

nostackp := $(call cc-option, -fno-stack-protector)
--
1.7.10.4


2013-07-17 13:40:46

by Stefano Stabellini

[permalink] [raw]
Subject: Re: [PATCH] xen/arm: enable PV control for ARM

On Mon, 15 Jul 2013, Julien Grall wrote:
> Enable power management from the toolstack for ARM guest.
>
> Signed-off-by: Julien Grall <[email protected]>

Considering that now we support both ARM and ARM64, could you please
add an ifneq ($(CONFIG_ARM64),y) too around cpu_hotplug.o, since you are
at it?


> drivers/xen/Makefile | 3 +--
> 1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/drivers/xen/Makefile b/drivers/xen/Makefile
> index 2bf461a..a5f12bd 100644
> --- a/drivers/xen/Makefile
> +++ b/drivers/xen/Makefile
> @@ -1,9 +1,8 @@
> ifneq ($(CONFIG_ARM),y)
> -obj-y += manage.o
> obj-$(CONFIG_HOTPLUG_CPU) += cpu_hotplug.o
> endif
> obj-$(CONFIG_X86) += fallback.o
> -obj-y += grant-table.o features.o events.o balloon.o time.o
> +obj-y += grant-table.o features.o events.o balloon.o time.o manage.o
> obj-y += xenbus/
>
> nostackp := $(call cc-option, -fno-stack-protector)
> --
> 1.7.10.4
>

2013-07-18 17:19:21

by Julien Grall

[permalink] [raw]
Subject: Re: [PATCH] xen/arm: enable PV control for ARM

On 17 July 2013 14:40, Stefano Stabellini
<[email protected]> wrote:
> On Mon, 15 Jul 2013, Julien Grall wrote:
>> Enable power management from the toolstack for ARM guest.
>>
>> Signed-off-by: Julien Grall <[email protected]>
>
> Considering that now we support both ARM and ARM64, could you please
> add an ifneq ($(CONFIG_ARM64),y) too around cpu_hotplug.o, since you are
> at it?

Yes. I will resend the patch.

--
Julien Grall