Subject: Re: [Intel-wired-lan] [PATCH iwl-next v1 07/12] iavf: add support for indirect access to PHC time

From: kernel test robot <[email-masked]>
Date: Wed, 27 Mar 2024 08:29:19 +0800

Hi Mateusz,

kernel test robot noticed the following build errors:

[auto build test ERROR on tnguy-next-queue/dev-queue]

url:    https://github.com/intel-lab-lkp/linux/commits/Mateusz-Polchlopek/virtchnl-add-support-for-enabling-PTP-on-iAVF/20240326-200321
base:   https://git.kernel.org/pub/scm/linux/kernel/git/tnguy/next-queue.git dev-queue
patch link:    https://lore.kernel.org/r/20240326115116.10040-8-mateusz.polchlopek%40intel.com
patch subject: [Intel-wired-lan] [PATCH iwl-next v1 07/12] iavf: add support for indirect access to PHC time
config: openrisc-allyesconfig (https://download.01.org/0day-ci/archive/20240327/[email-masked]/config)
compiler: or1k-linux-gcc (GCC) 13.2.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20240327/[email-masked]/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 <[email-masked]>
| Closes: https://lore.kernel.org/oe-kbuild-all/[email-masked]/

All error/warnings (new ones prefixed by >>):

   In file included from include/linux/device.h:15,
                    from include/linux/pci.h:37,
                    from drivers/net/ethernet/intel/iavf/iavf.h:8,
                    from drivers/net/ethernet/intel/iavf/iavf_virtchnl.c:4:
   drivers/net/ethernet/intel/iavf/iavf_virtchnl.c: In function 'iavf_virtchnl_ptp_get_time':
>> drivers/net/ethernet/intel/iavf/iavf_virtchnl.c:2185:30: warning: format '%lu' expects argument of type 'long unsigned int', but argument 4 has type 'unsigned int' [-Wformat=]
    2185 |                              "Invalid VIRTCHNL_OP_1588_PTP_GET_TIME from PF. Got size %u, expected %lu\n",
         |                              ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   include/linux/dev_printk.h:110:30: note: in definition of macro 'dev_printk_index_wrap'
     110 |                 _p_func(dev, fmt, ##__VA_ARGS__);                       \
         |                              ^~~
   include/linux/dev_printk.h:144:56: note: in expansion of macro 'dev_fmt'
     144 |         dev_printk_index_wrap(_dev_err, KERN_ERR, dev, dev_fmt(fmt), ##__VA_ARGS__)
         |                                                        ^~~~~~~
   include/linux/dev_printk.h:174:17: note: in expansion of macro 'dev_err'
     174 |                 dev_level(dev, fmt, ##__VA_ARGS__);                     \
         |                 ^~~~~~~~~
   include/linux/dev_printk.h:192:9: note: in expansion of macro 'dev_level_once'
     192 |         dev_level_once(dev_err, dev, fmt, ##__VA_ARGS__)
         |         ^~~~~~~~~~~~~~
   drivers/net/ethernet/intel/iavf/iavf_virtchnl.c:2184:17: note: in expansion of macro 'dev_err_once'
    2184 |                 dev_err_once(&adapter->pdev->dev,
         |                 ^~~~~~~~~~~~
   drivers/net/ethernet/intel/iavf/iavf_virtchnl.c:2185:102: note: format string is defined here
    2185 |                              "Invalid VIRTCHNL_OP_1588_PTP_GET_TIME from PF. Got size %u, expected %lu\n",
         |                                                                                                    ~~^
         |                                                                                                      |
         |                                                                                                      long unsigned int
         |                                                                                                    %u
--
   arch/openrisc/kernel/head.o: in function `_dispatch_do_ipage_fault':
>> (.text+0x900): relocation truncated to fit: R_OR1K_INSN_REL_26 against `no symbol'
   (.text+0xa00): relocation truncated to fit: R_OR1K_INSN_REL_26 against `no symbol'
   arch/openrisc/kernel/head.o: in function `exit_with_no_dtranslation':
>> (.head.text+0x21bc): relocation truncated to fit: R_OR1K_INSN_REL_26 against `no symbol'
   arch/openrisc/kernel/head.o: in function `exit_with_no_itranslation':
   (.head.text+0x2264): relocation truncated to fit: R_OR1K_INSN_REL_26 against `no symbol'
   init/main.o: in function `initcall_blacklisted':
   main.c:(.text+0x5b8): relocation truncated to fit: R_OR1K_INSN_REL_26 against symbol `strcmp' defined in .text section in lib/string.o
   init/main.o: in function `trace_event_raw_event_initcall_level':
   main.c:(.text+0x7a8): relocation truncated to fit: R_OR1K_INSN_REL_26 against symbol `strlen' defined in .text section in lib/string.o
   init/main.o: in function `ktime_divns.constprop.0':
   main.c:(.text+0x8b0): relocation truncated to fit: R_OR1K_INSN_REL_26 against symbol `__muldi3' defined in .text section in ../lib/gcc/or1k-linux/13.2.0/libgcc.a(_muldi3.o)
   main.c:(.text+0x900): relocation truncated to fit: R_OR1K_INSN_REL_26 against symbol `__muldi3' defined in .text section in ../lib/gcc/or1k-linux/13.2.0/libgcc.a(_muldi3.o)
   main.c:(.text+0x930): relocation truncated to fit: R_OR1K_INSN_REL_26 against symbol `__muldi3' defined in .text section in ../lib/gcc/or1k-linux/13.2.0/libgcc.a(_muldi3.o)
   main.c:(.text+0x96c): relocation truncated to fit: R_OR1K_INSN_REL_26 against symbol `__muldi3' defined in .text section in ../lib/gcc/or1k-linux/13.2.0/libgcc.a(_muldi3.o)
   init/main.o: in function `trace_initcall_finish_cb':
   main.c:(.text+0xa5c): additional relocation overflows omitted from the output

vim +2185 drivers/net/ethernet/intel/iavf/iavf_virtchnl.c

  2163	
  2164	/**
  2165	 * iavf_virtchnl_ptp_get_time - Respond to VIRTCHNL_OP_1588_PTP_GET_TIME
  2166	 * @adapter: private adapter structure
  2167	 * @data: the message from the PF
  2168	 * @len: length of the message from the PF
  2169	 *
  2170	 * Handle the VIRTCHNL_OP_1588_PTP_GET_TIME message from the PF. This message
  2171	 * is sent by the PF in response to the same op as a request from the VF.
  2172	 * Extract the 64bit nanoseconds time from the message and store it in
  2173	 * cached_phc_time. Then, notify any thread that is waiting for the update via
  2174	 * the wait queue.
  2175	 */
  2176	static void iavf_virtchnl_ptp_get_time(struct iavf_adapter *adapter,
  2177					       void *data, u16 len)
  2178	{
  2179		struct virtchnl_phc_time *msg;
  2180	
  2181		if (len == sizeof(*msg)) {
  2182			msg = (struct virtchnl_phc_time *)data;
  2183		} else {
  2184			dev_err_once(&adapter->pdev->dev,
> 2185				     "Invalid VIRTCHNL_OP_1588_PTP_GET_TIME from PF. Got size %u, expected %lu\n",
  2186				     len, sizeof(*msg));
  2187			return;
  2188		}
  2189	
  2190		adapter->ptp.cached_phc_time = msg->time;
  2191		adapter->ptp.cached_phc_updated = jiffies;
  2192		adapter->ptp.phc_time_ready = true;
  2193	
  2194		wake_up(&adapter->ptp.phc_time_waitqueue);
  2195	}
  2196	

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