* [PATCH V1] rcu-tasks: Fix build error
@ 2023-02-20 18:13 Srinivasarao Pathipati
2023-02-20 18:54 ` Greg KH
0 siblings, 1 reply; 8+ messages in thread
From: Srinivasarao Pathipati @ 2023-02-20 18:13 UTC (permalink / raw)
To: stable, gregkh; +Cc: Srinivasarao Pathipati
Making show_rcu_tasks_rude_gp_kthread() function as 'inline' to
fix below compilation error.
This is applicable to only 5.10 kernels as code got modified
in latest kernels.
In file included from kernel/rcu/update.c:579:0:
kernel/rcu/tasks.h:710:13: error: ‘show_rcu_tasks_rude_gp_kthread’ defined but not used [-Werror=unused-function]
static void show_rcu_tasks_rude_gp_kthread(void) {}
Signed-off-by: Srinivasarao Pathipati <quic_c_spathi@quicinc.com>
---
kernel/rcu/tasks.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/kernel/rcu/tasks.h b/kernel/rcu/tasks.h
index 8b51e6a..53ddb4e 100644
--- a/kernel/rcu/tasks.h
+++ b/kernel/rcu/tasks.h
@@ -707,7 +707,7 @@ static void show_rcu_tasks_rude_gp_kthread(void)
#endif /* #ifndef CONFIG_TINY_RCU */
#else /* #ifdef CONFIG_TASKS_RUDE_RCU */
-static void show_rcu_tasks_rude_gp_kthread(void) {}
+static inline void show_rcu_tasks_rude_gp_kthread(void) {}
#endif /* #else #ifdef CONFIG_TASKS_RUDE_RCU */
////////////////////////////////////////////////////////////////////////
--
2.7.4
^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [PATCH V1] rcu-tasks: Fix build error
2023-02-20 18:13 [PATCH V1] rcu-tasks: Fix build error Srinivasarao Pathipati
@ 2023-02-20 18:54 ` Greg KH
2023-02-21 9:27 ` Srinivasarao Pathipati
0 siblings, 1 reply; 8+ messages in thread
From: Greg KH @ 2023-02-20 18:54 UTC (permalink / raw)
To: Srinivasarao Pathipati; +Cc: stable
On Mon, Feb 20, 2023 at 11:43:59PM +0530, Srinivasarao Pathipati wrote:
> Making show_rcu_tasks_rude_gp_kthread() function as 'inline' to
> fix below compilation error.
> This is applicable to only 5.10 kernels as code got modified
> in latest kernels.
>
> In file included from kernel/rcu/update.c:579:0:
> kernel/rcu/tasks.h:710:13: error: ‘show_rcu_tasks_rude_gp_kthread’ defined but not used [-Werror=unused-function]
> static void show_rcu_tasks_rude_gp_kthread(void) {}
>
> Signed-off-by: Srinivasarao Pathipati <quic_c_spathi@quicinc.com>
> ---
> kernel/rcu/tasks.h | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/kernel/rcu/tasks.h b/kernel/rcu/tasks.h
> index 8b51e6a..53ddb4e 100644
> --- a/kernel/rcu/tasks.h
> +++ b/kernel/rcu/tasks.h
> @@ -707,7 +707,7 @@ static void show_rcu_tasks_rude_gp_kthread(void)
> #endif /* #ifndef CONFIG_TINY_RCU */
>
> #else /* #ifdef CONFIG_TASKS_RUDE_RCU */
> -static void show_rcu_tasks_rude_gp_kthread(void) {}
> +static inline void show_rcu_tasks_rude_gp_kthread(void) {}
> #endif /* #else #ifdef CONFIG_TASKS_RUDE_RCU */
>
> ////////////////////////////////////////////////////////////////////////
> --
> 2.7.4
>
What commit id caused this problem?
And why isn't it an issue in newer kernels, what commit id fixed it and
why can't we just take that instead?
thanks,
greg k-h
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH V1] rcu-tasks: Fix build error
2023-02-20 18:54 ` Greg KH
@ 2023-02-21 9:27 ` Srinivasarao Pathipati
2023-02-21 10:37 ` Greg KH
0 siblings, 1 reply; 8+ messages in thread
From: Srinivasarao Pathipati @ 2023-02-21 9:27 UTC (permalink / raw)
To: Greg KH; +Cc: stable
On 2/21/2023 12:24 AM, Greg KH wrote:
> On Mon, Feb 20, 2023 at 11:43:59PM +0530, Srinivasarao Pathipati wrote:
>> Making show_rcu_tasks_rude_gp_kthread() function as 'inline' to
>> fix below compilation error.
>> This is applicable to only 5.10 kernels as code got modified
>> in latest kernels.
>>
>> In file included from kernel/rcu/update.c:579:0:
>> kernel/rcu/tasks.h:710:13: error: ‘show_rcu_tasks_rude_gp_kthread’ defined but not used [-Werror=unused-function]
>> static void show_rcu_tasks_rude_gp_kthread(void) {}
>>
>> Signed-off-by: Srinivasarao Pathipati <quic_c_spathi@quicinc.com>
>> ---
>> kernel/rcu/tasks.h | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/kernel/rcu/tasks.h b/kernel/rcu/tasks.h
>> index 8b51e6a..53ddb4e 100644
>> --- a/kernel/rcu/tasks.h
>> +++ b/kernel/rcu/tasks.h
>> @@ -707,7 +707,7 @@ static void show_rcu_tasks_rude_gp_kthread(void)
>> #endif /* #ifndef CONFIG_TINY_RCU */
>>
>> #else /* #ifdef CONFIG_TASKS_RUDE_RCU */
>> -static void show_rcu_tasks_rude_gp_kthread(void) {}
>> +static inline void show_rcu_tasks_rude_gp_kthread(void) {}
>> #endif /* #else #ifdef CONFIG_TASKS_RUDE_RCU */
>>
>> ////////////////////////////////////////////////////////////////////////
>> --
>> 2.7.4
>>
> What commit id caused this problem?
commit (8344496e8b49c4122c1808d6cd3f8dc71bccb595 rcu-tasks:
Conditionally compile show_rcu_tasks_gp_kthreads()) introduced this issue
This patch added conditional macros for definition of
show_rcu_tasks_rude_gp_kthread() but not for dummy definition.
> And why isn't it an issue in newer kernels, what commit id fixed it and
> why can't we just take that instead?
Later code got modified with patch (27c0f1448389 rcutorture: Make
grace-period kthread report match RCU flavor being tested) , with this
there won't be compilation issue.
This patch is part of below series, Not sure all these can be picked to
this 5.10 stable branch so fixed issue by simply making function inline.
if you think it is better to pick this series, please merge to 5.10 branch.
https://lore.kernel.org/lkml/20201105233900.GA20676@paulmck-ThinkPad-P72/
[1/4] e1eb075ccf37 rcutorture: Make preemptible TRACE02 enable lockdep
[2/4] 77dc174103fd rcu-tasks: Convert rcu_tasks_wait_gp() for-loop to
while-loop
[3/4] 27c0f1448389 rcutorture: Make grace-period kthread report match
RCU flavor being tested
[4/4] 75dc2da5ecd6 rcu-tasks: Make the units of ->init_fract be jiffies
>
> thanks,
>
> greg k-h
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH V1] rcu-tasks: Fix build error
2023-02-21 9:27 ` Srinivasarao Pathipati
@ 2023-02-21 10:37 ` Greg KH
2023-02-21 11:19 ` Srinivasarao Pathipati
0 siblings, 1 reply; 8+ messages in thread
From: Greg KH @ 2023-02-21 10:37 UTC (permalink / raw)
To: Srinivasarao Pathipati; +Cc: stable
On Tue, Feb 21, 2023 at 02:57:15PM +0530, Srinivasarao Pathipati wrote:
>
> On 2/21/2023 12:24 AM, Greg KH wrote:
> > On Mon, Feb 20, 2023 at 11:43:59PM +0530, Srinivasarao Pathipati wrote:
> > > Making show_rcu_tasks_rude_gp_kthread() function as 'inline' to
> > > fix below compilation error.
> > > This is applicable to only 5.10 kernels as code got modified
> > > in latest kernels.
> > >
> > > In file included from kernel/rcu/update.c:579:0:
> > > kernel/rcu/tasks.h:710:13: error: ‘show_rcu_tasks_rude_gp_kthread’ defined but not used [-Werror=unused-function]
> > > static void show_rcu_tasks_rude_gp_kthread(void) {}
> > >
> > > Signed-off-by: Srinivasarao Pathipati <quic_c_spathi@quicinc.com>
> > > ---
> > > kernel/rcu/tasks.h | 2 +-
> > > 1 file changed, 1 insertion(+), 1 deletion(-)
> > >
> > > diff --git a/kernel/rcu/tasks.h b/kernel/rcu/tasks.h
> > > index 8b51e6a..53ddb4e 100644
> > > --- a/kernel/rcu/tasks.h
> > > +++ b/kernel/rcu/tasks.h
> > > @@ -707,7 +707,7 @@ static void show_rcu_tasks_rude_gp_kthread(void)
> > > #endif /* #ifndef CONFIG_TINY_RCU */
> > > #else /* #ifdef CONFIG_TASKS_RUDE_RCU */
> > > -static void show_rcu_tasks_rude_gp_kthread(void) {}
> > > +static inline void show_rcu_tasks_rude_gp_kthread(void) {}
> > > #endif /* #else #ifdef CONFIG_TASKS_RUDE_RCU */
> > > ////////////////////////////////////////////////////////////////////////
> > > --
> > > 2.7.4
> > >
> > What commit id caused this problem?
>
> commit (8344496e8b49c4122c1808d6cd3f8dc71bccb595 rcu-tasks: Conditionally
> compile show_rcu_tasks_gp_kthreads()) introduced this issue
So this has been around for a very very long time.
Why is this suddenly an issue now, almost 3 years later? What changed
to cause this to become an issue?
And please put this information in the changelog text.
> This patch added conditional macros for definition of
> show_rcu_tasks_rude_gp_kthread() but not for dummy definition.
>
> > And why isn't it an issue in newer kernels, what commit id fixed it and
> > why can't we just take that instead?
>
> Later code got modified with patch (27c0f1448389 rcutorture: Make
> grace-period kthread report match RCU flavor being tested) , with this
> there won't be compilation issue.
>
> This patch is part of below series, Not sure all these can be picked to this
> 5.10 stable branch so fixed issue by simply making function inline.
>
> if you think it is better to pick this series, please merge to 5.10 branch.
>
> https://lore.kernel.org/lkml/20201105233900.GA20676@paulmck-ThinkPad-P72/
>
> [1/4] e1eb075ccf37 rcutorture: Make preemptible TRACE02 enable lockdep
> [2/4] 77dc174103fd rcu-tasks: Convert rcu_tasks_wait_gp() for-loop to
> while-loop
> [3/4] 27c0f1448389 rcutorture: Make grace-period kthread report match RCU
> flavor being tested
> [4/4] 75dc2da5ecd6 rcu-tasks: Make the units of ->init_fract be jiffies
I would need you to test and verify that these actually work properly
and do not cause any regressions and sign off on them before I could
take them.
thanks,
greg k-h
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH V1] rcu-tasks: Fix build error
2023-02-21 10:37 ` Greg KH
@ 2023-02-21 11:19 ` Srinivasarao Pathipati
2023-02-21 13:45 ` Greg KH
0 siblings, 1 reply; 8+ messages in thread
From: Srinivasarao Pathipati @ 2023-02-21 11:19 UTC (permalink / raw)
To: Greg KH; +Cc: stable
On 2/21/2023 4:07 PM, Greg KH wrote:
> On Tue, Feb 21, 2023 at 02:57:15PM +0530, Srinivasarao Pathipati wrote:
>> On 2/21/2023 12:24 AM, Greg KH wrote:
>>> On Mon, Feb 20, 2023 at 11:43:59PM +0530, Srinivasarao Pathipati wrote:
>>>> Making show_rcu_tasks_rude_gp_kthread() function as 'inline' to
>>>> fix below compilation error.
>>>> This is applicable to only 5.10 kernels as code got modified
>>>> in latest kernels.
>>>>
>>>> In file included from kernel/rcu/update.c:579:0:
>>>> kernel/rcu/tasks.h:710:13: error: ‘show_rcu_tasks_rude_gp_kthread’ defined but not used [-Werror=unused-function]
>>>> static void show_rcu_tasks_rude_gp_kthread(void) {}
>>>>
>>>> Signed-off-by: Srinivasarao Pathipati <quic_c_spathi@quicinc.com>
>>>> ---
>>>> kernel/rcu/tasks.h | 2 +-
>>>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>>>
>>>> diff --git a/kernel/rcu/tasks.h b/kernel/rcu/tasks.h
>>>> index 8b51e6a..53ddb4e 100644
>>>> --- a/kernel/rcu/tasks.h
>>>> +++ b/kernel/rcu/tasks.h
>>>> @@ -707,7 +707,7 @@ static void show_rcu_tasks_rude_gp_kthread(void)
>>>> #endif /* #ifndef CONFIG_TINY_RCU */
>>>> #else /* #ifdef CONFIG_TASKS_RUDE_RCU */
>>>> -static void show_rcu_tasks_rude_gp_kthread(void) {}
>>>> +static inline void show_rcu_tasks_rude_gp_kthread(void) {}
>>>> #endif /* #else #ifdef CONFIG_TASKS_RUDE_RCU */
>>>> ////////////////////////////////////////////////////////////////////////
>>>> --
>>>> 2.7.4
>>>>
>>> What commit id caused this problem?
>> commit (8344496e8b49c4122c1808d6cd3f8dc71bccb595 rcu-tasks: Conditionally
>> compile show_rcu_tasks_gp_kthreads()) introduced this issue
> So this has been around for a very very long time.
>
> Why is this suddenly an issue now, almost 3 years later? What changed
> to cause this to become an issue?
>
> And please put this information in the changelog text.
We observed this issue on Android kernels with arch=UM after picking
this patch (3fe617ccafd6f5bb33c2391d6f4eeb41c1fd0151 Enable '-Werror' by
default for all kernel builds ) recently into Android kernels.
Earlier it was just warning and this issue depends on specific
configuration, so it might got missed.
>
>> This patch added conditional macros for definition of
>> show_rcu_tasks_rude_gp_kthread() but not for dummy definition.
>>
>>> And why isn't it an issue in newer kernels, what commit id fixed it and
>>> why can't we just take that instead?
>> Later code got modified with patch (27c0f1448389 rcutorture: Make
>> grace-period kthread report match RCU flavor being tested) , with this
>> there won't be compilation issue.
>>
>> This patch is part of below series, Not sure all these can be picked to this
>> 5.10 stable branch so fixed issue by simply making function inline.
>>
>> if you think it is better to pick this series, please merge to 5.10 branch.
>>
>> https://lore.kernel.org/lkml/20201105233900.GA20676@paulmck-ThinkPad-P72/
>>
>> [1/4] e1eb075ccf37 rcutorture: Make preemptible TRACE02 enable lockdep
>> [2/4] 77dc174103fd rcu-tasks: Convert rcu_tasks_wait_gp() for-loop to
>> while-loop
>> [3/4] 27c0f1448389 rcutorture: Make grace-period kthread report match RCU
>> flavor being tested
>> [4/4] 75dc2da5ecd6 rcu-tasks: Make the units of ->init_fract be jiffies
> I would need you to test and verify that these actually work properly
> and do not cause any regressions and sign off on them before I could
> take them.
probably it would be better to pick this no risk patch instead of
picking above series as this is stable branch.
>
> thanks,
>
> greg k-h
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH V1] rcu-tasks: Fix build error
2023-02-21 11:19 ` Srinivasarao Pathipati
@ 2023-02-21 13:45 ` Greg KH
0 siblings, 0 replies; 8+ messages in thread
From: Greg KH @ 2023-02-21 13:45 UTC (permalink / raw)
To: Srinivasarao Pathipati; +Cc: stable
On Tue, Feb 21, 2023 at 04:49:17PM +0530, Srinivasarao Pathipati wrote:
>
> On 2/21/2023 4:07 PM, Greg KH wrote:
> > On Tue, Feb 21, 2023 at 02:57:15PM +0530, Srinivasarao Pathipati wrote:
> > > On 2/21/2023 12:24 AM, Greg KH wrote:
> > > > On Mon, Feb 20, 2023 at 11:43:59PM +0530, Srinivasarao Pathipati wrote:
> > > > > Making show_rcu_tasks_rude_gp_kthread() function as 'inline' to
> > > > > fix below compilation error.
> > > > > This is applicable to only 5.10 kernels as code got modified
> > > > > in latest kernels.
> > > > >
> > > > > In file included from kernel/rcu/update.c:579:0:
> > > > > kernel/rcu/tasks.h:710:13: error: ‘show_rcu_tasks_rude_gp_kthread’ defined but not used [-Werror=unused-function]
> > > > > static void show_rcu_tasks_rude_gp_kthread(void) {}
> > > > >
> > > > > Signed-off-by: Srinivasarao Pathipati <quic_c_spathi@quicinc.com>
> > > > > ---
> > > > > kernel/rcu/tasks.h | 2 +-
> > > > > 1 file changed, 1 insertion(+), 1 deletion(-)
> > > > >
> > > > > diff --git a/kernel/rcu/tasks.h b/kernel/rcu/tasks.h
> > > > > index 8b51e6a..53ddb4e 100644
> > > > > --- a/kernel/rcu/tasks.h
> > > > > +++ b/kernel/rcu/tasks.h
> > > > > @@ -707,7 +707,7 @@ static void show_rcu_tasks_rude_gp_kthread(void)
> > > > > #endif /* #ifndef CONFIG_TINY_RCU */
> > > > > #else /* #ifdef CONFIG_TASKS_RUDE_RCU */
> > > > > -static void show_rcu_tasks_rude_gp_kthread(void) {}
> > > > > +static inline void show_rcu_tasks_rude_gp_kthread(void) {}
> > > > > #endif /* #else #ifdef CONFIG_TASKS_RUDE_RCU */
> > > > > ////////////////////////////////////////////////////////////////////////
> > > > > --
> > > > > 2.7.4
> > > > >
> > > > What commit id caused this problem?
> > > commit (8344496e8b49c4122c1808d6cd3f8dc71bccb595 rcu-tasks: Conditionally
> > > compile show_rcu_tasks_gp_kthreads()) introduced this issue
> > So this has been around for a very very long time.
> >
> > Why is this suddenly an issue now, almost 3 years later? What changed
> > to cause this to become an issue?
> >
> > And please put this information in the changelog text.
>
> We observed this issue on Android kernels with arch=UM after picking this
> patch (3fe617ccafd6f5bb33c2391d6f4eeb41c1fd0151 Enable '-Werror' by default
> for all kernel builds ) recently into Android kernels.
But that commit is NOT in the 5.10.y tree, so why would we care about
this problem in the 5.10.y tree?
confused,
greg k-h
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH V1] rcu-tasks: Fix build error
2023-02-21 12:03 Srinivasarao Pathipati
@ 2023-02-21 13:44 ` Greg KH
0 siblings, 0 replies; 8+ messages in thread
From: Greg KH @ 2023-02-21 13:44 UTC (permalink / raw)
To: Srinivasarao Pathipati; +Cc: stable, gregkh, Srinivasarao Pathipati
On Tue, Feb 21, 2023 at 05:33:22PM +0530, Srinivasarao Pathipati wrote:
> From: Srinivasarao Pathipati <quic_c_spathi@quicinc.com>
>
> Making show_rcu_tasks_rude_gp_kthread() function as 'inline' to
> fix below compilation error.
> This is applicable to only 5.10 kernels as code got modified
> in latest kernels.
>
> In file included from kernel/rcu/update.c:579:0:
> kernel/rcu/tasks.h:710:13: error: ‘show_rcu_tasks_rude_gp_kthread’ defined but not used [-Werror=unused-function]
> static void show_rcu_tasks_rude_gp_kthread(void) {}
>
> Fixes: 8344496e8b49 ("rcu-tasks: Conditionally compile show_rcu_tasks_gp_kthreads()")
> Signed-off-by: Srinivasarao Pathipati <quic_c_spathi@quicinc.com>
> ---
> v1->v2:
But the subject says "v1"?
confused,
greg k-h
^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH V1] rcu-tasks: Fix build error
@ 2023-02-21 12:03 Srinivasarao Pathipati
2023-02-21 13:44 ` Greg KH
0 siblings, 1 reply; 8+ messages in thread
From: Srinivasarao Pathipati @ 2023-02-21 12:03 UTC (permalink / raw)
To: stable, gregkh; +Cc: Srinivasarao Pathipati
From: Srinivasarao Pathipati <quic_c_spathi@quicinc.com>
Making show_rcu_tasks_rude_gp_kthread() function as 'inline' to
fix below compilation error.
This is applicable to only 5.10 kernels as code got modified
in latest kernels.
In file included from kernel/rcu/update.c:579:0:
kernel/rcu/tasks.h:710:13: error: ‘show_rcu_tasks_rude_gp_kthread’ defined but not used [-Werror=unused-function]
static void show_rcu_tasks_rude_gp_kthread(void) {}
Fixes: 8344496e8b49 ("rcu-tasks: Conditionally compile show_rcu_tasks_gp_kthreads()")
Signed-off-by: Srinivasarao Pathipati <quic_c_spathi@quicinc.com>
---
v1->v2:
- Added Fixes tag
kernel/rcu/tasks.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/kernel/rcu/tasks.h b/kernel/rcu/tasks.h
index 8b51e6a..53ddb4e 100644
--- a/kernel/rcu/tasks.h
+++ b/kernel/rcu/tasks.h
@@ -707,7 +707,7 @@ static void show_rcu_tasks_rude_gp_kthread(void)
#endif /* #ifndef CONFIG_TINY_RCU */
#else /* #ifdef CONFIG_TASKS_RUDE_RCU */
-static void show_rcu_tasks_rude_gp_kthread(void) {}
+static inline void show_rcu_tasks_rude_gp_kthread(void) {}
#endif /* #else #ifdef CONFIG_TASKS_RUDE_RCU */
////////////////////////////////////////////////////////////////////////
--
2.7.4
^ permalink raw reply related [flat|nested] 8+ messages in thread
end of thread, other threads:[~2023-02-21 13:46 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-02-20 18:13 [PATCH V1] rcu-tasks: Fix build error Srinivasarao Pathipati
2023-02-20 18:54 ` Greg KH
2023-02-21 9:27 ` Srinivasarao Pathipati
2023-02-21 10:37 ` Greg KH
2023-02-21 11:19 ` Srinivasarao Pathipati
2023-02-21 13:45 ` Greg KH
2023-02-21 12:03 Srinivasarao Pathipati
2023-02-21 13:44 ` Greg KH
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.