* [PATCH] PSI: select KERNFS as needed
@ 2023-07-31 3:07 Randy Dunlap
2023-07-31 18:14 ` Suren Baghdasaryan
0 siblings, 1 reply; 5+ messages in thread
From: Randy Dunlap @ 2023-07-31 3:07 UTC (permalink / raw)
To: linux-kernel
Cc: Randy Dunlap, kernel test robot, Johannes Weiner,
Suren Baghdasaryan, Ingo Molnar, Peter Zijlstra, Juri Lelli,
Vincent Guittot
Users of KERNFS should select it to enforce its being built, so
do this to prevent a build error.
In file included from ../kernel/sched/build_utility.c:97:
../kernel/sched/psi.c: In function 'psi_trigger_poll':
../kernel/sched/psi.c:1479:17: error: implicit declaration of function 'kernfs_generic_poll' [-Werror=implicit-function-declaration]
1479 | kernfs_generic_poll(t->of, wait);
Fixes: aff037078eca ("sched/psi: use kernfs polling functions for PSI trigger polling")
Signed-off-by: Randy Dunlap <rdunlap@infradead.org>
Reported-by: kernel test robot <lkp@intel.com>
Link: lore.kernel.org/r/202307310732.r65EQFY0-lkp@intel.com
Cc: Johannes Weiner <hannes@cmpxchg.org>
Cc: Suren Baghdasaryan <surenb@google.com>
Cc: Ingo Molnar <mingo@redhat.com>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: Juri Lelli <juri.lelli@redhat.com>
Cc: Vincent Guittot <vincent.guittot@linaro.org>
---
init/Kconfig | 1 +
1 file changed, 1 insertion(+)
diff -- a/init/Kconfig b/init/Kconfig
--- a/init/Kconfig
+++ b/init/Kconfig
@@ -629,6 +629,7 @@ config TASK_IO_ACCOUNTING
config PSI
bool "Pressure stall information tracking"
+ select KERNFS
help
Collect metrics that indicate how overcommitted the CPU, memory,
and IO capacity are in the system.
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] PSI: select KERNFS as needed
2023-07-31 3:07 [PATCH] PSI: select KERNFS as needed Randy Dunlap
@ 2023-07-31 18:14 ` Suren Baghdasaryan
2023-07-31 18:17 ` Suren Baghdasaryan
0 siblings, 1 reply; 5+ messages in thread
From: Suren Baghdasaryan @ 2023-07-31 18:14 UTC (permalink / raw)
To: Randy Dunlap
Cc: linux-kernel, kernel test robot, Johannes Weiner, Ingo Molnar,
Peter Zijlstra, Juri Lelli, Vincent Guittot
On Sun, Jul 30, 2023 at 8:07 PM Randy Dunlap <rdunlap@infradead.org> wrote:
>
> Users of KERNFS should select it to enforce its being built, so
> do this to prevent a build error.
>
> In file included from ../kernel/sched/build_utility.c:97:
> ../kernel/sched/psi.c: In function 'psi_trigger_poll':
> ../kernel/sched/psi.c:1479:17: error: implicit declaration of function 'kernfs_generic_poll' [-Werror=implicit-function-declaration]
> 1479 | kernfs_generic_poll(t->of, wait);
>
> Fixes: aff037078eca ("sched/psi: use kernfs polling functions for PSI trigger polling")
> Signed-off-by: Randy Dunlap <rdunlap@infradead.org>
> Reported-by: kernel test robot <lkp@intel.com>
> Link: lore.kernel.org/r/202307310732.r65EQFY0-lkp@intel.com
> Cc: Johannes Weiner <hannes@cmpxchg.org>
> Cc: Suren Baghdasaryan <surenb@google.com>
> Cc: Ingo Molnar <mingo@redhat.com>
> Cc: Peter Zijlstra <peterz@infradead.org>
> Cc: Juri Lelli <juri.lelli@redhat.com>
> Cc: Vincent Guittot <vincent.guittot@linaro.org>
Acked-by: Suren Baghdasaryan <surenb@google.com>
Thanks!
> ---
> init/Kconfig | 1 +
> 1 file changed, 1 insertion(+)
>
> diff -- a/init/Kconfig b/init/Kconfig
> --- a/init/Kconfig
> +++ b/init/Kconfig
> @@ -629,6 +629,7 @@ config TASK_IO_ACCOUNTING
>
> config PSI
> bool "Pressure stall information tracking"
> + select KERNFS
> help
> Collect metrics that indicate how overcommitted the CPU, memory,
> and IO capacity are in the system.
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] PSI: select KERNFS as needed
2023-07-31 18:14 ` Suren Baghdasaryan
@ 2023-07-31 18:17 ` Suren Baghdasaryan
2023-07-31 23:13 ` Randy Dunlap
0 siblings, 1 reply; 5+ messages in thread
From: Suren Baghdasaryan @ 2023-07-31 18:17 UTC (permalink / raw)
To: Randy Dunlap
Cc: linux-kernel, kernel test robot, Johannes Weiner, Ingo Molnar,
Peter Zijlstra, Juri Lelli, Vincent Guittot
On Mon, Jul 31, 2023 at 11:14 AM Suren Baghdasaryan <surenb@google.com> wrote:
>
> On Sun, Jul 30, 2023 at 8:07 PM Randy Dunlap <rdunlap@infradead.org> wrote:
> >
> > Users of KERNFS should select it to enforce its being built, so
> > do this to prevent a build error.
> >
> > In file included from ../kernel/sched/build_utility.c:97:
> > ../kernel/sched/psi.c: In function 'psi_trigger_poll':
> > ../kernel/sched/psi.c:1479:17: error: implicit declaration of function 'kernfs_generic_poll' [-Werror=implicit-function-declaration]
> > 1479 | kernfs_generic_poll(t->of, wait);
> >
> > Fixes: aff037078eca ("sched/psi: use kernfs polling functions for PSI trigger polling")
> > Signed-off-by: Randy Dunlap <rdunlap@infradead.org>
> > Reported-by: kernel test robot <lkp@intel.com>
shouldn't this include:
Closes: https://lore.kernel.org/oe-kbuild-all/202307310732.r65EQFY0-lkp@intel.com/
?
> > Link: lore.kernel.org/r/202307310732.r65EQFY0-lkp@intel.com
> > Cc: Johannes Weiner <hannes@cmpxchg.org>
> > Cc: Suren Baghdasaryan <surenb@google.com>
> > Cc: Ingo Molnar <mingo@redhat.com>
> > Cc: Peter Zijlstra <peterz@infradead.org>
> > Cc: Juri Lelli <juri.lelli@redhat.com>
> > Cc: Vincent Guittot <vincent.guittot@linaro.org>
>
> Acked-by: Suren Baghdasaryan <surenb@google.com>
>
> Thanks!
>
>
> > ---
> > init/Kconfig | 1 +
> > 1 file changed, 1 insertion(+)
> >
> > diff -- a/init/Kconfig b/init/Kconfig
> > --- a/init/Kconfig
> > +++ b/init/Kconfig
> > @@ -629,6 +629,7 @@ config TASK_IO_ACCOUNTING
> >
> > config PSI
> > bool "Pressure stall information tracking"
> > + select KERNFS
> > help
> > Collect metrics that indicate how overcommitted the CPU, memory,
> > and IO capacity are in the system.
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] PSI: select KERNFS as needed
2023-07-31 18:17 ` Suren Baghdasaryan
@ 2023-07-31 23:13 ` Randy Dunlap
2023-07-31 23:16 ` Suren Baghdasaryan
0 siblings, 1 reply; 5+ messages in thread
From: Randy Dunlap @ 2023-07-31 23:13 UTC (permalink / raw)
To: Suren Baghdasaryan
Cc: linux-kernel, kernel test robot, Johannes Weiner, Ingo Molnar,
Peter Zijlstra, Juri Lelli, Vincent Guittot
On 7/31/23 11:17, Suren Baghdasaryan wrote:
> On Mon, Jul 31, 2023 at 11:14 AM Suren Baghdasaryan <surenb@google.com> wrote:
>>
>> On Sun, Jul 30, 2023 at 8:07 PM Randy Dunlap <rdunlap@infradead.org> wrote:
>>>
>>> Users of KERNFS should select it to enforce its being built, so
>>> do this to prevent a build error.
>>>
>>> In file included from ../kernel/sched/build_utility.c:97:
>>> ../kernel/sched/psi.c: In function 'psi_trigger_poll':
>>> ../kernel/sched/psi.c:1479:17: error: implicit declaration of function 'kernfs_generic_poll' [-Werror=implicit-function-declaration]
>>> 1479 | kernfs_generic_poll(t->of, wait);
>>>
>>> Fixes: aff037078eca ("sched/psi: use kernfs polling functions for PSI trigger polling")
>>> Signed-off-by: Randy Dunlap <rdunlap@infradead.org>
>>> Reported-by: kernel test robot <lkp@intel.com>
>
> shouldn't this include:
> Closes: https://lore.kernel.org/oe-kbuild-all/202307310732.r65EQFY0-lkp@intel.com/
> ?
>
That's the same URL as the Link: below.
The difference(s) in Closes: and Link: are not clear to me,
other than Closes: is newer.
>>> Link: lore.kernel.org/r/202307310732.r65EQFY0-lkp@intel.com
>>> Cc: Johannes Weiner <hannes@cmpxchg.org>
>>> Cc: Suren Baghdasaryan <surenb@google.com>
>>> Cc: Ingo Molnar <mingo@redhat.com>
>>> Cc: Peter Zijlstra <peterz@infradead.org>
>>> Cc: Juri Lelli <juri.lelli@redhat.com>
>>> Cc: Vincent Guittot <vincent.guittot@linaro.org>
>>
>> Acked-by: Suren Baghdasaryan <surenb@google.com>
>>
>> Thanks!
>>
>>
>>> ---
>>> init/Kconfig | 1 +
>>> 1 file changed, 1 insertion(+)
>>>
>>> diff -- a/init/Kconfig b/init/Kconfig
>>> --- a/init/Kconfig
>>> +++ b/init/Kconfig
>>> @@ -629,6 +629,7 @@ config TASK_IO_ACCOUNTING
>>>
>>> config PSI
>>> bool "Pressure stall information tracking"
>>> + select KERNFS
>>> help
>>> Collect metrics that indicate how overcommitted the CPU, memory,
>>> and IO capacity are in the system.
--
~Randy
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] PSI: select KERNFS as needed
2023-07-31 23:13 ` Randy Dunlap
@ 2023-07-31 23:16 ` Suren Baghdasaryan
0 siblings, 0 replies; 5+ messages in thread
From: Suren Baghdasaryan @ 2023-07-31 23:16 UTC (permalink / raw)
To: Randy Dunlap
Cc: linux-kernel, kernel test robot, Johannes Weiner, Ingo Molnar,
Peter Zijlstra, Juri Lelli, Vincent Guittot
On Mon, Jul 31, 2023 at 4:13 PM Randy Dunlap <rdunlap@infradead.org> wrote:
>
>
>
> On 7/31/23 11:17, Suren Baghdasaryan wrote:
> > On Mon, Jul 31, 2023 at 11:14 AM Suren Baghdasaryan <surenb@google.com> wrote:
> >>
> >> On Sun, Jul 30, 2023 at 8:07 PM Randy Dunlap <rdunlap@infradead.org> wrote:
> >>>
> >>> Users of KERNFS should select it to enforce its being built, so
> >>> do this to prevent a build error.
> >>>
> >>> In file included from ../kernel/sched/build_utility.c:97:
> >>> ../kernel/sched/psi.c: In function 'psi_trigger_poll':
> >>> ../kernel/sched/psi.c:1479:17: error: implicit declaration of function 'kernfs_generic_poll' [-Werror=implicit-function-declaration]
> >>> 1479 | kernfs_generic_poll(t->of, wait);
> >>>
> >>> Fixes: aff037078eca ("sched/psi: use kernfs polling functions for PSI trigger polling")
> >>> Signed-off-by: Randy Dunlap <rdunlap@infradead.org>
> >>> Reported-by: kernel test robot <lkp@intel.com>
> >
> > shouldn't this include:
> > Closes: https://lore.kernel.org/oe-kbuild-all/202307310732.r65EQFY0-lkp@intel.com/
> > ?
> >
>
> That's the same URL as the Link: below.
> The difference(s) in Closes: and Link: are not clear to me,
> other than Closes: is newer.
Either way, LGTM. Thanks!
>
> >>> Link: lore.kernel.org/r/202307310732.r65EQFY0-lkp@intel.com
> >>> Cc: Johannes Weiner <hannes@cmpxchg.org>
> >>> Cc: Suren Baghdasaryan <surenb@google.com>
> >>> Cc: Ingo Molnar <mingo@redhat.com>
> >>> Cc: Peter Zijlstra <peterz@infradead.org>
> >>> Cc: Juri Lelli <juri.lelli@redhat.com>
> >>> Cc: Vincent Guittot <vincent.guittot@linaro.org>
> >>
> >> Acked-by: Suren Baghdasaryan <surenb@google.com>
> >>
> >> Thanks!
> >>
> >>
> >>> ---
> >>> init/Kconfig | 1 +
> >>> 1 file changed, 1 insertion(+)
> >>>
> >>> diff -- a/init/Kconfig b/init/Kconfig
> >>> --- a/init/Kconfig
> >>> +++ b/init/Kconfig
> >>> @@ -629,6 +629,7 @@ config TASK_IO_ACCOUNTING
> >>>
> >>> config PSI
> >>> bool "Pressure stall information tracking"
> >>> + select KERNFS
> >>> help
> >>> Collect metrics that indicate how overcommitted the CPU, memory,
> >>> and IO capacity are in the system.
>
> --
> ~Randy
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2023-07-31 23:16 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-07-31 3:07 [PATCH] PSI: select KERNFS as needed Randy Dunlap
2023-07-31 18:14 ` Suren Baghdasaryan
2023-07-31 18:17 ` Suren Baghdasaryan
2023-07-31 23:13 ` Randy Dunlap
2023-07-31 23:16 ` Suren Baghdasaryan
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).