2021-02-25 13:40:16

by Arnd Bergmann

[permalink] [raw]
Subject: [PATCH] virt: acrn: add hotplug_cpu dependency

From: Arnd Bergmann <[email protected]>

Without CPU hotplug, acrn fails to build:

drivers/virt/acrn/hsm.c:389:3: error: implicit declaration of function 'remove_cpu' [-Werror,-Wimplicit-function-declaration]
remove_cpu(cpu);
^
drivers/virt/acrn/hsm.c:402:2: error: implicit declaration of function 'add_cpu' [-Werror,-Wimplicit-function-declaration]
add_cpu(cpu);
^

Enforce the dependency through Kconfig to avoid the build failure.

Fixes: 666834c47d3b ("virt: acrn: Introduce ACRN HSM basic driver")
Signed-off-by: Arnd Bergmann <[email protected]>
---
drivers/virt/acrn/Kconfig | 1 +
1 file changed, 1 insertion(+)

diff --git a/drivers/virt/acrn/Kconfig b/drivers/virt/acrn/Kconfig
index 3e1a61c9d8d8..fbb0e3234aaf 100644
--- a/drivers/virt/acrn/Kconfig
+++ b/drivers/virt/acrn/Kconfig
@@ -2,6 +2,7 @@
config ACRN_HSM
tristate "ACRN Hypervisor Service Module"
depends on ACRN_GUEST
+ depends on HOTPLUG_CPU
select EVENTFD
help
ACRN Hypervisor Service Module (HSM) is a kernel module which
--
2.29.2


2021-02-26 01:40:33

by Shuo Liu

[permalink] [raw]
Subject: Re: [PATCH] virt: acrn: add hotplug_cpu dependency

Hi Arnd,

There is already a patchset for this build issue. Please refer to
https://lore.kernel.org/lkml/[email protected]/.

Hi Greg,
Would you like accept that patchset in you tree?

Thanks
shuo

On Thu 25.Feb'21 at 14:38:30 +0100, Arnd Bergmann wrote:
>From: Arnd Bergmann <[email protected]>
>
>Without CPU hotplug, acrn fails to build:
>
>drivers/virt/acrn/hsm.c:389:3: error: implicit declaration of function 'remove_cpu' [-Werror,-Wimplicit-function-declaration]
> remove_cpu(cpu);
> ^
>drivers/virt/acrn/hsm.c:402:2: error: implicit declaration of function 'add_cpu' [-Werror,-Wimplicit-function-declaration]
> add_cpu(cpu);
> ^
>
>Enforce the dependency through Kconfig to avoid the build failure.
>
>Fixes: 666834c47d3b ("virt: acrn: Introduce ACRN HSM basic driver")
>Signed-off-by: Arnd Bergmann <[email protected]>
>---
> drivers/virt/acrn/Kconfig | 1 +
> 1 file changed, 1 insertion(+)
>
>diff --git a/drivers/virt/acrn/Kconfig b/drivers/virt/acrn/Kconfig
>index 3e1a61c9d8d8..fbb0e3234aaf 100644
>--- a/drivers/virt/acrn/Kconfig
>+++ b/drivers/virt/acrn/Kconfig
>@@ -2,6 +2,7 @@
> config ACRN_HSM
> tristate "ACRN Hypervisor Service Module"
> depends on ACRN_GUEST
>+ depends on HOTPLUG_CPU
> select EVENTFD
> help
> ACRN Hypervisor Service Module (HSM) is a kernel module which
>--
>2.29.2
>

2021-02-26 06:29:50

by Greg Kroah-Hartman

[permalink] [raw]
Subject: Re: [PATCH] virt: acrn: add hotplug_cpu dependency

On Fri, Feb 26, 2021 at 09:38:07AM +0800, Shuo A Liu wrote:
> Hi Arnd,
>
> There is already a patchset for this build issue. Please refer to
> https://lore.kernel.org/lkml/[email protected]/.
>
> Hi Greg, Would you like accept that patchset in you tree?

I can't take anything new in my tree until 5.12-rc1 is out. When that
happens I will go through everything submitted to me and merge fixes
like this one and get them sent to Linus in a week or so.

thanks,

greg k-h

2021-02-26 06:32:51

by Shuo Liu

[permalink] [raw]
Subject: Re: [PATCH] virt: acrn: add hotplug_cpu dependency

On Fri 26.Feb'21 at 7:25:18 +0100, Greg Kroah-Hartman wrote:
>On Fri, Feb 26, 2021 at 09:38:07AM +0800, Shuo A Liu wrote:
>> Hi Arnd,
>>
>> There is already a patchset for this build issue. Please refer to
>> https://lore.kernel.org/lkml/[email protected]/.
>>
>> Hi Greg, Would you like accept that patchset in you tree?
>
>I can't take anything new in my tree until 5.12-rc1 is out. When that
>happens I will go through everything submitted to me and merge fixes
>like this one and get them sent to Linus in a week or so.

OK. Thanks for the information.

Thanks
shuo

2021-03-05 00:06:57

by Greg Kroah-Hartman

[permalink] [raw]
Subject: Re: [PATCH] virt: acrn: add hotplug_cpu dependency

On Fri, Feb 26, 2021 at 02:30:18PM +0800, Shuo A Liu wrote:
> On Fri 26.Feb'21 at 7:25:18 +0100, Greg Kroah-Hartman wrote:
> > On Fri, Feb 26, 2021 at 09:38:07AM +0800, Shuo A Liu wrote:
> > > Hi Arnd,
> > >
> > > There is already a patchset for this build issue. Please refer to
> > > https://lore.kernel.org/lkml/[email protected]/.
> > >
> > > Hi Greg, Would you like accept that patchset in you tree?
> >
> > I can't take anything new in my tree until 5.12-rc1 is out. When that
> > happens I will go through everything submitted to me and merge fixes
> > like this one and get them sent to Linus in a week or so.
>
> OK. Thanks for the information.

Now in my tree and will show up in linux-next tomorrow.

thanks,

greg k-h