All of lore.kernel.org
 help / color / mirror / Atom feed
From: kernel test robot <lkp@intel.com>
To: James Morse <james.morse@arm.com>
Cc: oe-kbuild-all@lists.linux.dev
Subject: [morse:mpam/snapshot/v6.11-rc1 114/128] arch/x86/include/asm/resctrl.h:205:21: error: conflicting types for 'resctrl_arch_mon_ctx_alloc'; have 'void *(struct rdt_resource *, int)'
Date: Sat, 3 Aug 2024 21:31:50 +0800	[thread overview]
Message-ID: <202408032113.y76zoLS3-lkp@intel.com> (raw)

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/morse/linux.git mpam/snapshot/v6.11-rc1
head:   063bc546005302904092c158fe3228dc8af798a9
commit: f6cf53b477a878020ffbcbfcfc4bc520d6b5332a [114/128] arm64: mpam: Select ARCH_HAS_CPU_RESCTRL
config: i386-allmodconfig (https://download.01.org/0day-ci/archive/20240803/202408032113.y76zoLS3-lkp@intel.com/config)
compiler: gcc-12 (Debian 12.2.0-14) 12.2.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20240803/202408032113.y76zoLS3-lkp@intel.com/reproduce)

If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202408032113.y76zoLS3-lkp@intel.com/

All errors (new ones prefixed by >>):

         |                    ^~~~~~~~~~~~~~~~~~~~~~~~~
   include/linux/arm_mpam.h:92:20: note: previous definition of 'resctrl_arch_enable_alloc' with type 'void(void)'
      92 | static inline void resctrl_arch_enable_alloc(void) { }
         |                    ^~~~~~~~~~~~~~~~~~~~~~~~~
   arch/x86/include/asm/resctrl.h:64:20: error: redefinition of 'resctrl_arch_disable_alloc'
      64 | static inline void resctrl_arch_disable_alloc(void)
         |                    ^~~~~~~~~~~~~~~~~~~~~~~~~~
   include/linux/arm_mpam.h:93:20: note: previous definition of 'resctrl_arch_disable_alloc' with type 'void(void)'
      93 | static inline void resctrl_arch_disable_alloc(void) { }
         |                    ^~~~~~~~~~~~~~~~~~~~~~~~~~
   arch/x86/include/asm/resctrl.h:70:20: error: static declaration of 'resctrl_arch_mon_capable' follows non-static declaration
      70 | static inline bool resctrl_arch_mon_capable(void)
         |                    ^~~~~~~~~~~~~~~~~~~~~~~~
   include/linux/arm_mpam.h:55:6: note: previous declaration of 'resctrl_arch_mon_capable' with type 'bool(void)' {aka '_Bool(void)'}
      55 | bool resctrl_arch_mon_capable(void);
         |      ^~~~~~~~~~~~~~~~~~~~~~~~
   arch/x86/include/asm/resctrl.h:75:20: error: redefinition of 'resctrl_arch_enable_mon'
      75 | static inline void resctrl_arch_enable_mon(void)
         |                    ^~~~~~~~~~~~~~~~~~~~~~~
   include/linux/arm_mpam.h:90:20: note: previous definition of 'resctrl_arch_enable_mon' with type 'void(void)'
      90 | static inline void resctrl_arch_enable_mon(void) { }
         |                    ^~~~~~~~~~~~~~~~~~~~~~~
   arch/x86/include/asm/resctrl.h:81:20: error: redefinition of 'resctrl_arch_disable_mon'
      81 | static inline void resctrl_arch_disable_mon(void)
         |                    ^~~~~~~~~~~~~~~~~~~~~~~~
   include/linux/arm_mpam.h:91:20: note: previous definition of 'resctrl_arch_disable_mon' with type 'void(void)'
      91 | static inline void resctrl_arch_disable_mon(void) { }
         |                    ^~~~~~~~~~~~~~~~~~~~~~~~
   arch/x86/include/asm/resctrl.h:87:20: error: static declaration of 'resctrl_arch_is_llc_occupancy_enabled' follows non-static declaration
      87 | static inline bool resctrl_arch_is_llc_occupancy_enabled(void)
         |                    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   include/linux/arm_mpam.h:56:6: note: previous declaration of 'resctrl_arch_is_llc_occupancy_enabled' with type 'bool(void)' {aka '_Bool(void)'}
      56 | bool resctrl_arch_is_llc_occupancy_enabled(void);
         |      ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   arch/x86/include/asm/resctrl.h:92:20: error: static declaration of 'resctrl_arch_is_mbm_total_enabled' follows non-static declaration
      92 | static inline bool resctrl_arch_is_mbm_total_enabled(void)
         |                    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   include/linux/arm_mpam.h:58:6: note: previous declaration of 'resctrl_arch_is_mbm_total_enabled' with type 'bool(void)' {aka '_Bool(void)'}
      58 | bool resctrl_arch_is_mbm_total_enabled(void);
         |      ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   arch/x86/include/asm/resctrl.h:97:20: error: static declaration of 'resctrl_arch_is_mbm_local_enabled' follows non-static declaration
      97 | static inline bool resctrl_arch_is_mbm_local_enabled(void)
         |                    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   include/linux/arm_mpam.h:57:6: note: previous declaration of 'resctrl_arch_is_mbm_local_enabled' with type 'bool(void)' {aka '_Bool(void)'}
      57 | bool resctrl_arch_is_mbm_local_enabled(void);
         |      ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   arch/x86/include/asm/resctrl.h:146:28: error: redefinition of 'resctrl_arch_round_mon_val'
     146 | static inline unsigned int resctrl_arch_round_mon_val(unsigned int val)
         |                            ^~~~~~~~~~~~~~~~~~~~~~~~~~
   include/linux/arm_mpam.h:43:28: note: previous definition of 'resctrl_arch_round_mon_val' with type 'unsigned int(unsigned int)'
      43 | static inline unsigned int resctrl_arch_round_mon_val(unsigned int val)
         |                            ^~~~~~~~~~~~~~~~~~~~~~~~~~
   arch/x86/include/asm/resctrl.h:155:20: error: static declaration of 'resctrl_arch_set_cpu_default_closid_rmid' follows non-static declaration
     155 | static inline void resctrl_arch_set_cpu_default_closid_rmid(int cpu, u32 closid,
         |                    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   include/linux/arm_mpam.h:65:6: note: previous declaration of 'resctrl_arch_set_cpu_default_closid_rmid' with type 'void(int,  u32,  u32)' {aka 'void(int,  unsigned int,  unsigned int)'}
      65 | void resctrl_arch_set_cpu_default_closid_rmid(int cpu, u32 closid, u32 pmg);
         |      ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   arch/x86/include/asm/resctrl.h:162:20: error: static declaration of 'resctrl_arch_set_closid_rmid' follows non-static declaration
     162 | static inline void resctrl_arch_set_closid_rmid(struct task_struct *tsk,
         |                    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
   include/linux/arm_mpam.h:64:6: note: previous declaration of 'resctrl_arch_set_closid_rmid' with type 'void(struct task_struct *, u32,  u32)' {aka 'void(struct task_struct *, unsigned int,  unsigned int)'}
      64 | void resctrl_arch_set_closid_rmid(struct task_struct *tsk, u32 closid, u32 rmid);
         |      ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
   arch/x86/include/asm/resctrl.h:169:20: error: static declaration of 'resctrl_arch_match_closid' follows non-static declaration
     169 | static inline bool resctrl_arch_match_closid(struct task_struct *tsk, u32 closid)
         |                    ^~~~~~~~~~~~~~~~~~~~~~~~~
   include/linux/arm_mpam.h:69:6: note: previous declaration of 'resctrl_arch_match_closid' with type 'bool(struct task_struct *, u32)' {aka '_Bool(struct task_struct *, unsigned int)'}
      69 | bool resctrl_arch_match_closid(struct task_struct *tsk, u32 closid);
         |      ^~~~~~~~~~~~~~~~~~~~~~~~~
   arch/x86/include/asm/resctrl.h:174:20: error: static declaration of 'resctrl_arch_match_rmid' follows non-static declaration
     174 | static inline bool resctrl_arch_match_rmid(struct task_struct *tsk, u32 ignored,
         |                    ^~~~~~~~~~~~~~~~~~~~~~~
   include/linux/arm_mpam.h:70:6: note: previous declaration of 'resctrl_arch_match_rmid' with type 'bool(struct task_struct *, u32,  u32)' {aka '_Bool(struct task_struct *, unsigned int,  unsigned int)'}
      70 | bool resctrl_arch_match_rmid(struct task_struct *tsk, u32 closid, u32 rmid);
         |      ^~~~~~~~~~~~~~~~~~~~~~~
   arch/x86/include/asm/resctrl.h:180:20: error: static declaration of 'resctrl_arch_sched_in' follows non-static declaration
     180 | static inline void resctrl_arch_sched_in(struct task_struct *tsk)
         |                    ^~~~~~~~~~~~~~~~~~~~~
   include/linux/arm_mpam.h:66:6: note: previous declaration of 'resctrl_arch_sched_in' with type 'void(struct task_struct *)'
      66 | void resctrl_arch_sched_in(struct task_struct *tsk);
         |      ^~~~~~~~~~~~~~~~~~~~~
   arch/x86/include/asm/resctrl.h:186:19: error: static declaration of 'resctrl_arch_system_num_rmid_idx' follows non-static declaration
     186 | static inline u32 resctrl_arch_system_num_rmid_idx(void)
         |                   ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   include/linux/arm_mpam.h:73:5: note: previous declaration of 'resctrl_arch_system_num_rmid_idx' with type 'u32(void)' {aka 'unsigned int(void)'}
      73 | u32 resctrl_arch_system_num_rmid_idx(void);
         |     ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   arch/x86/include/asm/resctrl.h:192:20: error: static declaration of 'resctrl_arch_rmid_idx_decode' follows non-static declaration
     192 | static inline void resctrl_arch_rmid_idx_decode(u32 idx, u32 *closid, u32 *rmid)
         |                    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
   include/linux/arm_mpam.h:72:6: note: previous declaration of 'resctrl_arch_rmid_idx_decode' with type 'void(u32,  u32 *, u32 *)' {aka 'void(unsigned int,  unsigned int *, unsigned int *)'}
      72 | void resctrl_arch_rmid_idx_decode(u32 idx, u32 *closid, u32 *rmid);
         |      ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
   arch/x86/include/asm/resctrl.h:198:19: error: static declaration of 'resctrl_arch_rmid_idx_encode' follows non-static declaration
     198 | static inline u32 resctrl_arch_rmid_idx_encode(u32 ignored, u32 rmid)
         |                   ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
   include/linux/arm_mpam.h:71:5: note: previous declaration of 'resctrl_arch_rmid_idx_encode' with type 'u32(u32,  u32)' {aka 'unsigned int(unsigned int,  unsigned int)'}
      71 | u32 resctrl_arch_rmid_idx_encode(u32 closid, u32 rmid);
         |     ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
>> arch/x86/include/asm/resctrl.h:205:21: error: conflicting types for 'resctrl_arch_mon_ctx_alloc'; have 'void *(struct rdt_resource *, int)'
     205 | static inline void *resctrl_arch_mon_ctx_alloc(struct rdt_resource *r, int evtid)
         |                     ^~~~~~~~~~~~~~~~~~~~~~~~~~
   include/linux/arm_mpam.h:76:7: note: previous declaration of 'resctrl_arch_mon_ctx_alloc' with type 'void *(struct rdt_resource *, enum resctrl_event_id)'
      76 | void *resctrl_arch_mon_ctx_alloc(struct rdt_resource *r, enum resctrl_event_id evtid);
         |       ^~~~~~~~~~~~~~~~~~~~~~~~~~
>> arch/x86/include/asm/resctrl.h:211:20: error: conflicting types for 'resctrl_arch_mon_ctx_free'; have 'void(struct rdt_resource *, int,  void *)'
     211 | static inline void resctrl_arch_mon_ctx_free(struct rdt_resource *r, int evtid,
         |                    ^~~~~~~~~~~~~~~~~~~~~~~~~
   include/linux/arm_mpam.h:77:6: note: previous declaration of 'resctrl_arch_mon_ctx_free' with type 'void(struct rdt_resource *, enum resctrl_event_id,  void *)'
      77 | void resctrl_arch_mon_ctx_free(struct rdt_resource *r, enum resctrl_event_id evtid, void *ctx);
         |      ^~~~~~~~~~~~~~~~~~~~~~~~~
>> drivers/platform/arm64/mpam/mpam_resctrl.c:19:10: fatal error: asm/mpam.h: No such file or directory
      19 | #include <asm/mpam.h>
         |          ^~~~~~~~~~~~
   compilation terminated.


vim +205 arch/x86/include/asm/resctrl.h

6791e0ea30711b James Morse 2024-02-13  202  
e557999f80a5ee James Morse 2024-02-13  203  /* x86 can always read an rmid, nothing needs allocating */
e557999f80a5ee James Morse 2024-02-13  204  struct rdt_resource;
e557999f80a5ee James Morse 2024-02-13 @205  static inline void *resctrl_arch_mon_ctx_alloc(struct rdt_resource *r, int evtid)
e557999f80a5ee James Morse 2024-02-13  206  {
e557999f80a5ee James Morse 2024-02-13  207  	might_sleep();
e557999f80a5ee James Morse 2024-02-13  208  	return NULL;
e557999f80a5ee James Morse 2024-02-13  209  };
e557999f80a5ee James Morse 2024-02-13  210  
e557999f80a5ee James Morse 2024-02-13 @211  static inline void resctrl_arch_mon_ctx_free(struct rdt_resource *r, int evtid,
e557999f80a5ee James Morse 2024-02-13  212  					     void *ctx) { };
e557999f80a5ee James Morse 2024-02-13  213  

:::::: The code at line 205 was first introduced by commit
:::::: e557999f80a5ee4ec812f594ab42bb76c3ec4eb2 x86/resctrl: Allow arch to allocate memory needed in resctrl_arch_rmid_read()

:::::: TO: James Morse <james.morse@arm.com>
:::::: CC: Borislav Petkov (AMD) <bp@alien8.de>

-- 
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki

                 reply	other threads:[~2024-08-03 13:32 UTC|newest]

Thread overview: [no followups] expand[flat|nested]  mbox.gz  Atom feed

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=202408032113.y76zoLS3-lkp@intel.com \
    --to=lkp@intel.com \
    --cc=james.morse@arm.com \
    --cc=oe-kbuild-all@lists.linux.dev \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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.