* [PATCH] irqchip/gic-v3: use __init where appropriate
@ 2018-12-29 14:14 Stefan Agner
2019-01-03 11:08 ` Marc Zyngier
0 siblings, 1 reply; 3+ messages in thread
From: Stefan Agner @ 2018-12-29 14:14 UTC (permalink / raw)
To: tglx, jason, marc.zyngier
Cc: linux-kernel, linux-arm-kernel-bounces, Stefan Agner
Compiling with section mismatch debugging enabled prints the
following warning:
WARNING: vmlinux.o(.text+0xffa1b8): Section mismatch in reference from
the function gic_smp_init() to the function .init.text:set_smp_cross_call()
Avoid this warning by adding __init to gic_smp_init(). While add it,
also add it where appropriate.
Signed-off-by: Stefan Agner <stefan@agner.ch>
---
drivers/irqchip/irq-gic-v3.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/irqchip/irq-gic-v3.c b/drivers/irqchip/irq-gic-v3.c
index 0868a9d81c3c..3b33c94041e5 100644
--- a/drivers/irqchip/irq-gic-v3.c
+++ b/drivers/irqchip/irq-gic-v3.c
@@ -768,7 +768,7 @@ static void gic_raise_softirq(const struct cpumask *mask, unsigned int irq)
isb();
}
-static void gic_smp_init(void)
+static void __init gic_smp_init(void)
{
set_smp_cross_call(gic_raise_softirq);
cpuhp_setup_state_nocalls(CPUHP_AP_IRQ_GIC_STARTING,
@@ -848,7 +848,7 @@ static struct notifier_block gic_cpu_pm_notifier_block = {
.notifier_call = gic_cpu_pm_notifier,
};
-static void gic_cpu_pm_init(void)
+static void __init gic_cpu_pm_init(void)
{
cpu_pm_register_notifier(&gic_cpu_pm_notifier_block);
}
--
2.20.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] irqchip/gic-v3: use __init where appropriate
2018-12-29 14:14 [PATCH] irqchip/gic-v3: use __init where appropriate Stefan Agner
@ 2019-01-03 11:08 ` Marc Zyngier
2019-01-03 13:22 ` Stefan Agner
0 siblings, 1 reply; 3+ messages in thread
From: Marc Zyngier @ 2019-01-03 11:08 UTC (permalink / raw)
To: Stefan Agner, tglx, jason; +Cc: linux-kernel, linux-arm-kernel-bounces
Hi Stephan,
On 29/12/2018 14:14, Stefan Agner wrote:
> Compiling with section mismatch debugging enabled prints the
> following warning:
> WARNING: vmlinux.o(.text+0xffa1b8): Section mismatch in reference from
> the function gic_smp_init() to the function .init.text:set_smp_cross_call()
>
> Avoid this warning by adding __init to gic_smp_init(). While add it,
> also add it where appropriate.
>
> Signed-off-by: Stefan Agner <stefan@agner.ch>
> ---
> drivers/irqchip/irq-gic-v3.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/irqchip/irq-gic-v3.c b/drivers/irqchip/irq-gic-v3.c
> index 0868a9d81c3c..3b33c94041e5 100644
> --- a/drivers/irqchip/irq-gic-v3.c
> +++ b/drivers/irqchip/irq-gic-v3.c
> @@ -768,7 +768,7 @@ static void gic_raise_softirq(const struct cpumask *mask, unsigned int irq)
> isb();
> }
>
> -static void gic_smp_init(void)
> +static void __init gic_smp_init(void)
> {
> set_smp_cross_call(gic_raise_softirq);
> cpuhp_setup_state_nocalls(CPUHP_AP_IRQ_GIC_STARTING,
> @@ -848,7 +848,7 @@ static struct notifier_block gic_cpu_pm_notifier_block = {
> .notifier_call = gic_cpu_pm_notifier,
> };
>
> -static void gic_cpu_pm_init(void)
> +static void __init gic_cpu_pm_init(void)
> {
> cpu_pm_register_notifier(&gic_cpu_pm_notifier_block);
> }
>
Interesting. I don't see this warning at all. How do you trigger this?
Thanks,
M.
--
Jazz is not dead. It just smells funny...
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] irqchip/gic-v3: use __init where appropriate
2019-01-03 11:08 ` Marc Zyngier
@ 2019-01-03 13:22 ` Stefan Agner
0 siblings, 0 replies; 3+ messages in thread
From: Stefan Agner @ 2019-01-03 13:22 UTC (permalink / raw)
To: Marc Zyngier; +Cc: tglx, jason, linux-kernel, linux-arm-kernel-bounces
On 03.01.2019 12:08, Marc Zyngier wrote:
> Hi Stephan,
>
> On 29/12/2018 14:14, Stefan Agner wrote:
>> Compiling with section mismatch debugging enabled prints the
>> following warning:
>> WARNING: vmlinux.o(.text+0xffa1b8): Section mismatch in reference from
>> the function gic_smp_init() to the function .init.text:set_smp_cross_call()
>>
>> Avoid this warning by adding __init to gic_smp_init(). While add it,
>> also add it where appropriate.
>>
>> Signed-off-by: Stefan Agner <stefan@agner.ch>
>> ---
>> drivers/irqchip/irq-gic-v3.c | 4 ++--
>> 1 file changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/drivers/irqchip/irq-gic-v3.c b/drivers/irqchip/irq-gic-v3.c
>> index 0868a9d81c3c..3b33c94041e5 100644
>> --- a/drivers/irqchip/irq-gic-v3.c
>> +++ b/drivers/irqchip/irq-gic-v3.c
>> @@ -768,7 +768,7 @@ static void gic_raise_softirq(const struct cpumask *mask, unsigned int irq)
>> isb();
>> }
>>
>> -static void gic_smp_init(void)
>> +static void __init gic_smp_init(void)
>> {
>> set_smp_cross_call(gic_raise_softirq);
>> cpuhp_setup_state_nocalls(CPUHP_AP_IRQ_GIC_STARTING,
>> @@ -848,7 +848,7 @@ static struct notifier_block gic_cpu_pm_notifier_block = {
>> .notifier_call = gic_cpu_pm_notifier,
>> };
>>
>> -static void gic_cpu_pm_init(void)
>> +static void __init gic_cpu_pm_init(void)
>> {
>> cpu_pm_register_notifier(&gic_cpu_pm_notifier_block);
>> }
>>
>
> Interesting. I don't see this warning at all. How do you trigger this?
I noticed that it suddenly disappeared again too. I used some crazy
configurations while doing tests with LLVM/Clang, I think it was with
allyesconfig.
--
Stefan
>
> Thanks,
>
> M.
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2019-01-03 13:22 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-12-29 14:14 [PATCH] irqchip/gic-v3: use __init where appropriate Stefan Agner
2019-01-03 11:08 ` Marc Zyngier
2019-01-03 13:22 ` Stefan Agner
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).