linux/psi.h is included more than once.
Signed-off-by: Yan Yan <[email protected]>
---
kernel/sched/build_utility.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/kernel/sched/build_utility.c b/kernel/sched/build_utility.c
index 99bdd96f454f..80a3df49ab47 100644
--- a/kernel/sched/build_utility.c
+++ b/kernel/sched/build_utility.c
@@ -34,7 +34,6 @@
#include <linux/nospec.h>
#include <linux/proc_fs.h>
#include <linux/psi.h>
-#include <linux/psi.h>
#include <linux/ptrace_api.h>
#include <linux/sched_clock.h>
#include <linux/security.h>
--
2.32.0.3.g01195cf9f
Ping.
It seems that removing the two same included lines is ok. Because the
following "sched.h" also includes psi.h.
Any comments?
Thanks,
-Yan Yan
在 2023/5/6 23:39, 晏艳(采苓) 写道:
> linux/psi.h is included more than once.
>
> Signed-off-by: Yan Yan <[email protected]>
> ---
> kernel/sched/build_utility.c | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/kernel/sched/build_utility.c b/kernel/sched/build_utility.c
> index 99bdd96f454f..80a3df49ab47 100644
> --- a/kernel/sched/build_utility.c
> +++ b/kernel/sched/build_utility.c
> @@ -34,7 +34,6 @@
> #include <linux/nospec.h>
> #include <linux/proc_fs.h>
> #include <linux/psi.h>
> -#include <linux/psi.h>
> #include <linux/ptrace_api.h>
> #include <linux/sched_clock.h>
> #include <linux/security.h>
Ping, any comments?
thanks,
-Yan Yan
在 2023/5/16 00:00, Yan Yan 写道:
> Ping.
>
> It seems that removing the two same included lines is ok. Because the
> following "sched.h" also includes psi.h.
>
> Any comments?
>
> Thanks,
>
> -Yan Yan
>
> 在 2023/5/6 23:39, 晏艳(采苓) 写道:
>> linux/psi.h is included more than once.
>>
>> Signed-off-by: Yan Yan <[email protected]>
>> ---
>> kernel/sched/build_utility.c | 1 -
>> 1 file changed, 1 deletion(-)
>>
>> diff --git a/kernel/sched/build_utility.c b/kernel/sched/build_utility.c
>> index 99bdd96f454f..80a3df49ab47 100644
>> --- a/kernel/sched/build_utility.c
>> +++ b/kernel/sched/build_utility.c
>> @@ -34,7 +34,6 @@
>> #include <linux/nospec.h>
>> #include <linux/proc_fs.h>
>> #include <linux/psi.h>
>> -#include <linux/psi.h>
>> #include <linux/ptrace_api.h>
>> #include <linux/sched_clock.h>
>> #include <linux/security.h>
Le 22/05/2023 à 04:26, Yan Yan(cailing) a écrit :
> Ping, any comments?
>
> thanks,
>
> -Yan Yan
>
> 在 2023/5/16 00:00, Yan Yan 写道:
>> Ping.
>>
>> It seems that removing the two same included lines is ok. Because the
>> following "sched.h" also includes psi.h.
>>
>> Any comments?
FWIW, yes, looks good.
If you want to ga that way, their are some other potential removal
candidates in build_utility.c:
linux/sched/mm.h
linux/cpufreq.h
linux/ctype.h
linux/proc_fs.h
linux/spinlock_api.h
linux/wait_api.h
linux/workqueue_api.h
CJ
>>
>> Thanks,
>>
>> -Yan Yan
>>
>> 在 2023/5/6 23:39, 晏艳(采苓) 写道:
>>> linux/psi.h is included more than once.
>>>
>>> Signed-off-by: Yan Yan <[email protected]>
>>> ---
>>> kernel/sched/build_utility.c | 1 -
>>> 1 file changed, 1 deletion(-)
>>>
>>> diff --git a/kernel/sched/build_utility.c b/kernel/sched/build_utility.c
>>> index 99bdd96f454f..80a3df49ab47 100644
>>> --- a/kernel/sched/build_utility.c
>>> +++ b/kernel/sched/build_utility.c
>>> @@ -34,7 +34,6 @@
>>> #include <linux/nospec.h>
>>> #include <linux/proc_fs.h>
>>> #include <linux/psi.h>
>>> -#include <linux/psi.h>
>>> #include <linux/ptrace_api.h>
>>> #include <linux/sched_clock.h>
>>> #include <linux/security.h>
>