* [PATCH] watchdog: Fix build error when CONFIG_SOFTLOCKUP_DETECTOR is not set
@ 2022-07-27 9:21 ` Laurent Dufour
0 siblings, 0 replies; 10+ messages in thread
From: Laurent Dufour @ 2022-07-27 9:21 UTC (permalink / raw)
To: mpe; +Cc: linux-kernel, linuxppc-dev, linux-next, Sachin Sant
Sachin reported the following build error when CONFIG_SOFTLOCKUP_DETECTOR
is not set:
kernel/watchdog.c:597:20: error: static declaration of 'lockup_detector_reconfigure' follows non-static declaration
static inline void lockup_detector_reconfigure(void)
^~~~~~~~~~~~~~~~~~~~~~~~~~~
In file included from kernel/watchdog.c:17:
./include/linux/nmi.h:125:6: note: previous declaration of 'lockup_detector_reconfigure' was here
void lockup_detector_reconfigure(void);
^~~~~~~~~~~~~~~~~~~~~~~~~~~
The definition of lockup_detector_reconfigure should be exported even in
that case, and __lockup_detector_reconfigure should remain static.
Fixes: 24a1260705b7 ("watchdog: export lockup_detector_reconfigure")
Reported-by: Sachin Sant <sachinp@linux.ibm.com>
Signed-off-by: Laurent Dufour <ldufour@linux.ibm.com>
---
kernel/watchdog.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/kernel/watchdog.c b/kernel/watchdog.c
index 90e6c41d5e33..41596c415111 100644
--- a/kernel/watchdog.c
+++ b/kernel/watchdog.c
@@ -590,7 +590,7 @@ static __init void lockup_detector_setup(void)
}
#else /* CONFIG_SOFTLOCKUP_DETECTOR */
-void __lockup_detector_reconfigure(void)
+static void __lockup_detector_reconfigure(void)
{
cpus_read_lock();
watchdog_nmi_stop();
@@ -598,7 +598,7 @@ void __lockup_detector_reconfigure(void)
watchdog_nmi_start();
cpus_read_unlock();
}
-static inline void lockup_detector_reconfigure(void)
+void lockup_detector_reconfigure(void)
{
__lockup_detector_reconfigure();
}
--
2.37.1
^ permalink raw reply related [flat|nested] 10+ messages in thread* [PATCH] watchdog: Fix build error when CONFIG_SOFTLOCKUP_DETECTOR is not set
@ 2022-07-27 9:21 ` Laurent Dufour
0 siblings, 0 replies; 10+ messages in thread
From: Laurent Dufour @ 2022-07-27 9:21 UTC (permalink / raw)
To: mpe; +Cc: linux-next, linuxppc-dev, linux-kernel, Sachin Sant
Sachin reported the following build error when CONFIG_SOFTLOCKUP_DETECTOR
is not set:
kernel/watchdog.c:597:20: error: static declaration of 'lockup_detector_reconfigure' follows non-static declaration
static inline void lockup_detector_reconfigure(void)
^~~~~~~~~~~~~~~~~~~~~~~~~~~
In file included from kernel/watchdog.c:17:
./include/linux/nmi.h:125:6: note: previous declaration of 'lockup_detector_reconfigure' was here
void lockup_detector_reconfigure(void);
^~~~~~~~~~~~~~~~~~~~~~~~~~~
The definition of lockup_detector_reconfigure should be exported even in
that case, and __lockup_detector_reconfigure should remain static.
Fixes: 24a1260705b7 ("watchdog: export lockup_detector_reconfigure")
Reported-by: Sachin Sant <sachinp@linux.ibm.com>
Signed-off-by: Laurent Dufour <ldufour@linux.ibm.com>
---
kernel/watchdog.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/kernel/watchdog.c b/kernel/watchdog.c
index 90e6c41d5e33..41596c415111 100644
--- a/kernel/watchdog.c
+++ b/kernel/watchdog.c
@@ -590,7 +590,7 @@ static __init void lockup_detector_setup(void)
}
#else /* CONFIG_SOFTLOCKUP_DETECTOR */
-void __lockup_detector_reconfigure(void)
+static void __lockup_detector_reconfigure(void)
{
cpus_read_lock();
watchdog_nmi_stop();
@@ -598,7 +598,7 @@ void __lockup_detector_reconfigure(void)
watchdog_nmi_start();
cpus_read_unlock();
}
-static inline void lockup_detector_reconfigure(void)
+void lockup_detector_reconfigure(void)
{
__lockup_detector_reconfigure();
}
--
2.37.1
^ permalink raw reply related [flat|nested] 10+ messages in thread* Re: [PATCH] watchdog: Fix build error when CONFIG_SOFTLOCKUP_DETECTOR is not set
2022-07-27 9:21 ` Laurent Dufour
@ 2022-07-27 10:10 ` Sachin Sant
-1 siblings, 0 replies; 10+ messages in thread
From: Sachin Sant @ 2022-07-27 10:10 UTC (permalink / raw)
To: Laurent Dufour; +Cc: Michael Ellerman, linux-kernel, linuxppc-dev, linux-next
> The definition of lockup_detector_reconfigure should be exported even in
> that case, and __lockup_detector_reconfigure should remain static.
>
> Fixes: 24a1260705b7 ("watchdog: export lockup_detector_reconfigure")
> Reported-by: Sachin Sant <sachinp@linux.ibm.com>
> Signed-off-by: Laurent Dufour <ldufour@linux.ibm.com>
> ---
> kernel/watchdog.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
Thanks Laurent for the fix.
Tested-by: Sachin Sant <sachinp@linux.ibm.com>
- Sachin
^ permalink raw reply [flat|nested] 10+ messages in thread* Re: [PATCH] watchdog: Fix build error when CONFIG_SOFTLOCKUP_DETECTOR is not set
@ 2022-07-27 10:10 ` Sachin Sant
0 siblings, 0 replies; 10+ messages in thread
From: Sachin Sant @ 2022-07-27 10:10 UTC (permalink / raw)
To: Laurent Dufour; +Cc: linux-next, linuxppc-dev, linux-kernel
> The definition of lockup_detector_reconfigure should be exported even in
> that case, and __lockup_detector_reconfigure should remain static.
>
> Fixes: 24a1260705b7 ("watchdog: export lockup_detector_reconfigure")
> Reported-by: Sachin Sant <sachinp@linux.ibm.com>
> Signed-off-by: Laurent Dufour <ldufour@linux.ibm.com>
> ---
> kernel/watchdog.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
Thanks Laurent for the fix.
Tested-by: Sachin Sant <sachinp@linux.ibm.com>
- Sachin
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] watchdog: Fix build error when CONFIG_SOFTLOCKUP_DETECTOR is not set
2022-07-27 9:21 ` Laurent Dufour
@ 2022-07-27 11:38 ` Michael Ellerman
-1 siblings, 0 replies; 10+ messages in thread
From: Michael Ellerman @ 2022-07-27 11:38 UTC (permalink / raw)
To: Laurent Dufour; +Cc: linux-kernel, linuxppc-dev, linux-next, Sachin Sant
Laurent Dufour <ldufour@linux.ibm.com> writes:
> Sachin reported the following build error when CONFIG_SOFTLOCKUP_DETECTOR
> is not set:
>
> kernel/watchdog.c:597:20: error: static declaration of 'lockup_detector_reconfigure' follows non-static declaration
> static inline void lockup_detector_reconfigure(void)
> ^~~~~~~~~~~~~~~~~~~~~~~~~~~
> In file included from kernel/watchdog.c:17:
> ./include/linux/nmi.h:125:6: note: previous declaration of 'lockup_detector_reconfigure' was here
> void lockup_detector_reconfigure(void);
> ^~~~~~~~~~~~~~~~~~~~~~~~~~~
>
> The definition of lockup_detector_reconfigure should be exported even in
> that case, and __lockup_detector_reconfigure should remain static.
>
> Fixes: 24a1260705b7 ("watchdog: export lockup_detector_reconfigure")
> Reported-by: Sachin Sant <sachinp@linux.ibm.com>
> Signed-off-by: Laurent Dufour <ldufour@linux.ibm.com>
> ---
> kernel/watchdog.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
Thanks.
I'll squash that in to the original commit.
cheers
^ permalink raw reply [flat|nested] 10+ messages in thread* Re: [PATCH] watchdog: Fix build error when CONFIG_SOFTLOCKUP_DETECTOR is not set
@ 2022-07-27 11:38 ` Michael Ellerman
0 siblings, 0 replies; 10+ messages in thread
From: Michael Ellerman @ 2022-07-27 11:38 UTC (permalink / raw)
To: Laurent Dufour; +Cc: linux-next, linuxppc-dev, linux-kernel, Sachin Sant
Laurent Dufour <ldufour@linux.ibm.com> writes:
> Sachin reported the following build error when CONFIG_SOFTLOCKUP_DETECTOR
> is not set:
>
> kernel/watchdog.c:597:20: error: static declaration of 'lockup_detector_reconfigure' follows non-static declaration
> static inline void lockup_detector_reconfigure(void)
> ^~~~~~~~~~~~~~~~~~~~~~~~~~~
> In file included from kernel/watchdog.c:17:
> ./include/linux/nmi.h:125:6: note: previous declaration of 'lockup_detector_reconfigure' was here
> void lockup_detector_reconfigure(void);
> ^~~~~~~~~~~~~~~~~~~~~~~~~~~
>
> The definition of lockup_detector_reconfigure should be exported even in
> that case, and __lockup_detector_reconfigure should remain static.
>
> Fixes: 24a1260705b7 ("watchdog: export lockup_detector_reconfigure")
> Reported-by: Sachin Sant <sachinp@linux.ibm.com>
> Signed-off-by: Laurent Dufour <ldufour@linux.ibm.com>
> ---
> kernel/watchdog.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
Thanks.
I'll squash that in to the original commit.
cheers
^ permalink raw reply [flat|nested] 10+ messages in thread* Re: [PATCH] watchdog: Fix build error when CONFIG_SOFTLOCKUP_DETECTOR is not set
2022-07-27 11:38 ` Michael Ellerman
@ 2022-07-27 14:20 ` Laurent Dufour
-1 siblings, 0 replies; 10+ messages in thread
From: Laurent Dufour @ 2022-07-27 14:20 UTC (permalink / raw)
To: Michael Ellerman; +Cc: linux-kernel, linuxppc-dev, linux-next, Sachin Sant
Le 27/07/2022 à 13:38, Michael Ellerman a écrit :
> Laurent Dufour <ldufour@linux.ibm.com> writes:
>> Sachin reported the following build error when CONFIG_SOFTLOCKUP_DETECTOR
>> is not set:
>>
>> kernel/watchdog.c:597:20: error: static declaration of 'lockup_detector_reconfigure' follows non-static declaration
>> static inline void lockup_detector_reconfigure(void)
>> ^~~~~~~~~~~~~~~~~~~~~~~~~~~
>> In file included from kernel/watchdog.c:17:
>> ./include/linux/nmi.h:125:6: note: previous declaration of 'lockup_detector_reconfigure' was here
>> void lockup_detector_reconfigure(void);
>> ^~~~~~~~~~~~~~~~~~~~~~~~~~~
>>
>> The definition of lockup_detector_reconfigure should be exported even in
>> that case, and __lockup_detector_reconfigure should remain static.
>>
>> Fixes: 24a1260705b7 ("watchdog: export lockup_detector_reconfigure")
>> Reported-by: Sachin Sant <sachinp@linux.ibm.com>
>> Signed-off-by: Laurent Dufour <ldufour@linux.ibm.com>
>> ---
>> kernel/watchdog.c | 4 ++--
>> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> Thanks.
>
> I'll squash that in to the original commit.
Thanks Michael,
I was thinking this is too late for such a squash.
Laurent.
^ permalink raw reply [flat|nested] 10+ messages in thread* Re: [PATCH] watchdog: Fix build error when CONFIG_SOFTLOCKUP_DETECTOR is not set
@ 2022-07-27 14:20 ` Laurent Dufour
0 siblings, 0 replies; 10+ messages in thread
From: Laurent Dufour @ 2022-07-27 14:20 UTC (permalink / raw)
To: Michael Ellerman; +Cc: linux-next, linuxppc-dev, linux-kernel, Sachin Sant
Le 27/07/2022 à 13:38, Michael Ellerman a écrit :
> Laurent Dufour <ldufour@linux.ibm.com> writes:
>> Sachin reported the following build error when CONFIG_SOFTLOCKUP_DETECTOR
>> is not set:
>>
>> kernel/watchdog.c:597:20: error: static declaration of 'lockup_detector_reconfigure' follows non-static declaration
>> static inline void lockup_detector_reconfigure(void)
>> ^~~~~~~~~~~~~~~~~~~~~~~~~~~
>> In file included from kernel/watchdog.c:17:
>> ./include/linux/nmi.h:125:6: note: previous declaration of 'lockup_detector_reconfigure' was here
>> void lockup_detector_reconfigure(void);
>> ^~~~~~~~~~~~~~~~~~~~~~~~~~~
>>
>> The definition of lockup_detector_reconfigure should be exported even in
>> that case, and __lockup_detector_reconfigure should remain static.
>>
>> Fixes: 24a1260705b7 ("watchdog: export lockup_detector_reconfigure")
>> Reported-by: Sachin Sant <sachinp@linux.ibm.com>
>> Signed-off-by: Laurent Dufour <ldufour@linux.ibm.com>
>> ---
>> kernel/watchdog.c | 4 ++--
>> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> Thanks.
>
> I'll squash that in to the original commit.
Thanks Michael,
I was thinking this is too late for such a squash.
Laurent.
^ permalink raw reply [flat|nested] 10+ messages in thread* Re: [PATCH] watchdog: Fix build error when CONFIG_SOFTLOCKUP_DETECTOR is not set
2022-07-27 14:20 ` Laurent Dufour
@ 2022-07-28 3:18 ` Michael Ellerman
-1 siblings, 0 replies; 10+ messages in thread
From: Michael Ellerman @ 2022-07-28 3:18 UTC (permalink / raw)
To: Laurent Dufour; +Cc: linux-kernel, linuxppc-dev, linux-next, Sachin Sant
Laurent Dufour <ldufour@linux.ibm.com> writes:
> Le 27/07/2022 à 13:38, Michael Ellerman a écrit :
>> Laurent Dufour <ldufour@linux.ibm.com> writes:
>>> Sachin reported the following build error when CONFIG_SOFTLOCKUP_DETECTOR
>>> is not set:
>>>
>>> kernel/watchdog.c:597:20: error: static declaration of 'lockup_detector_reconfigure' follows non-static declaration
>>> static inline void lockup_detector_reconfigure(void)
>>> ^~~~~~~~~~~~~~~~~~~~~~~~~~~
>>> In file included from kernel/watchdog.c:17:
>>> ./include/linux/nmi.h:125:6: note: previous declaration of 'lockup_detector_reconfigure' was here
>>> void lockup_detector_reconfigure(void);
>>> ^~~~~~~~~~~~~~~~~~~~~~~~~~~
>>>
>>> The definition of lockup_detector_reconfigure should be exported even in
>>> that case, and __lockup_detector_reconfigure should remain static.
>>>
>>> Fixes: 24a1260705b7 ("watchdog: export lockup_detector_reconfigure")
>>> Reported-by: Sachin Sant <sachinp@linux.ibm.com>
>>> Signed-off-by: Laurent Dufour <ldufour@linux.ibm.com>
>>> ---
>>> kernel/watchdog.c | 4 ++--
>>> 1 file changed, 2 insertions(+), 2 deletions(-)
>>
>> Thanks.
>>
>> I'll squash that in to the original commit.
>
> Thanks Michael,
>
> I was thinking this is too late for such a squash.
It usually would be, if I was organised, so sending a follow-up was the
correct thing for you to do :)
cheers
^ permalink raw reply [flat|nested] 10+ messages in thread* Re: [PATCH] watchdog: Fix build error when CONFIG_SOFTLOCKUP_DETECTOR is not set
@ 2022-07-28 3:18 ` Michael Ellerman
0 siblings, 0 replies; 10+ messages in thread
From: Michael Ellerman @ 2022-07-28 3:18 UTC (permalink / raw)
To: Laurent Dufour; +Cc: linux-next, linuxppc-dev, linux-kernel, Sachin Sant
Laurent Dufour <ldufour@linux.ibm.com> writes:
> Le 27/07/2022 à 13:38, Michael Ellerman a écrit :
>> Laurent Dufour <ldufour@linux.ibm.com> writes:
>>> Sachin reported the following build error when CONFIG_SOFTLOCKUP_DETECTOR
>>> is not set:
>>>
>>> kernel/watchdog.c:597:20: error: static declaration of 'lockup_detector_reconfigure' follows non-static declaration
>>> static inline void lockup_detector_reconfigure(void)
>>> ^~~~~~~~~~~~~~~~~~~~~~~~~~~
>>> In file included from kernel/watchdog.c:17:
>>> ./include/linux/nmi.h:125:6: note: previous declaration of 'lockup_detector_reconfigure' was here
>>> void lockup_detector_reconfigure(void);
>>> ^~~~~~~~~~~~~~~~~~~~~~~~~~~
>>>
>>> The definition of lockup_detector_reconfigure should be exported even in
>>> that case, and __lockup_detector_reconfigure should remain static.
>>>
>>> Fixes: 24a1260705b7 ("watchdog: export lockup_detector_reconfigure")
>>> Reported-by: Sachin Sant <sachinp@linux.ibm.com>
>>> Signed-off-by: Laurent Dufour <ldufour@linux.ibm.com>
>>> ---
>>> kernel/watchdog.c | 4 ++--
>>> 1 file changed, 2 insertions(+), 2 deletions(-)
>>
>> Thanks.
>>
>> I'll squash that in to the original commit.
>
> Thanks Michael,
>
> I was thinking this is too late for such a squash.
It usually would be, if I was organised, so sending a follow-up was the
correct thing for you to do :)
cheers
^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2022-07-28 3:18 UTC | newest]
Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-07-27 9:21 [PATCH] watchdog: Fix build error when CONFIG_SOFTLOCKUP_DETECTOR is not set Laurent Dufour
2022-07-27 9:21 ` Laurent Dufour
2022-07-27 10:10 ` Sachin Sant
2022-07-27 10:10 ` Sachin Sant
2022-07-27 11:38 ` Michael Ellerman
2022-07-27 11:38 ` Michael Ellerman
2022-07-27 14:20 ` Laurent Dufour
2022-07-27 14:20 ` Laurent Dufour
2022-07-28 3:18 ` Michael Ellerman
2022-07-28 3:18 ` Michael Ellerman
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.