Re: [peterz-queue:sched/psi 11/11] include/linux/cgroup-defs.h:432:38: error: 'NR_PSI_RESOURCES' undeclared here (not in a function)

[Chengming Zhou]

On 2022/9/7 16:06, Peter Zijlstra wrote:
> On Wed, Sep 07, 2022 at 10:45:55AM +0800, Chengming Zhou wrote:
>> On 2022/9/7 02:33, kernel test robot wrote:
>>> tree:   https://git.kernel.org/pub/scm/linux/kernel/git/peterz/queue.git sched/psi
>>> head:   51beb408c569e516780c84a2020920432ad4c5ed
>>> commit: 51beb408c569e516780c84a2020920432ad4c5ed [11/11] sched/psi: Per-cgroup PSI accounting disable/re-enable interface
>>> config: i386-randconfig-a001
>>> compiler: gcc-11 (Debian 11.3.0-5) 11.3.0
>>> reproduce (this is a W=1 build):
>>>         # https://git.kernel.org/pub/scm/linux/kernel/git/peterz/queue.git/commit/?id=51beb408c569e516780c84a2020920432ad4c5ed
>>>         git remote add peterz-queue https://git.kernel.org/pub/scm/linux/kernel/git/peterz/queue.git
>>>         git fetch --no-tags peterz-queue sched/psi
>>>         git checkout 51beb408c569e516780c84a2020920432ad4c5ed
>>>         # save the config file
>>>         mkdir build_dir && cp config build_dir/.config
>>>         make W=1 O=build_dir ARCH=i386 prepare
>>>
>>> If you fix the issue, kindly add following tag where applicable
>>> Reported-by: kernel test robot <lkp@xxxxxxxxx>
>>>
>>> All errors (new ones prefixed by >>):
>>>
>>>    In file included from include/linux/cgroup.h:28,
>>>                     from include/linux/memcontrol.h:13,
>>>                     from include/linux/swap.h:9,
>>>                     from include/linux/suspend.h:5,
>>>                     from arch/x86/kernel/asm-offsets.c:13:
>>>>> include/linux/cgroup-defs.h:432:38: error: 'NR_PSI_RESOURCES' undeclared here (not in a function)
>>>      432 |         struct cgroup_file psi_files[NR_PSI_RESOURCES];
>>
>> Sorry, looks like there are two problems here:
>>
>> 1. NR_PSI_RESOURCES is undeclared when !CONFIG_PSI
>>
>>    Should I send the below diff as a separate patch?
> 
> Please reply to the original patch with an updated version and I'll
> replace it.

Ok, I just sent. Thanks!