netdev - Re: [PATCH v2 net-next 6/7] net: dsa: sja1105: Configure the Time-Aware Scheduler via tc-taprio offload

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:   Sat, 14 Sep 2019 17:47:12 +0800
From:   kbuild test robot <lkp@...el.com>
To:     Vladimir Oltean <olteanv@...il.com>
Cc:     kbuild-all@...org, f.fainelli@...il.com, vivien.didelot@...il.com,
        andrew@...n.ch, davem@...emloft.net, vinicius.gomes@...el.com,
        vedang.patel@...el.com, richardcochran@...il.com,
        weifeng.voon@...el.com, jiri@...lanox.com, m-karicheri2@...com,
        Jose.Abreu@...opsys.com, ilias.apalodimas@...aro.org,
        jhs@...atatu.com, xiyou.wangcong@...il.com,
        kurt.kanzenbach@...utronix.de, joergen.andreasen@...rochip.com,
        netdev@...r.kernel.org, Vladimir Oltean <olteanv@...il.com>
Subject: Re: [PATCH v2 net-next 6/7] net: dsa: sja1105: Configure the
 Time-Aware Scheduler via tc-taprio offload

Hi Vladimir,

I love your patch! Yet something to improve:

[auto build test ERROR on net-next/master]

url:    https://github.com/0day-ci/linux/commits/Vladimir-Oltean/tc-taprio-offload-for-SJA1105-DSA/20190914-154650
config: i386-randconfig-b001-201936 (attached as .config)
compiler: gcc-7 (Debian 7.4.0-11) 7.4.0
reproduce:
        # save the attached .config to linux build tree
        make ARCH=i386 

If you fix the issue, kindly add following tag
Reported-by: kbuild test robot <lkp@...el.com>

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

   In file included from drivers/net/dsa/sja1105/sja1105.h:23:0,
                    from drivers/net/dsa/sja1105/sja1105_spi.c:8:
>> drivers/net/dsa/sja1105/sja1105_tas.h:38:45: warning: 'struct sja1105_private' declared inside parameter list will not be visible outside of this definition or declaration
    static inline void sja1105_tas_setup(struct sja1105_private *priv) { }
                                                ^~~~~~~~~~~~~~~
   drivers/net/dsa/sja1105/sja1105_tas.h:40:48: warning: 'struct sja1105_private' declared inside parameter list will not be visible outside of this definition or declaration
    static inline void sja1105_tas_teardown(struct sja1105_private *priv) { }
                                                   ^~~~~~~~~~~~~~~
--
   In file included from drivers/net/dsa/sja1105/sja1105.h:23:0,
                    from drivers/net/dsa/sja1105/sja1105_main.c:24:
>> drivers/net/dsa/sja1105/sja1105_tas.h:38:45: warning: 'struct sja1105_private' declared inside parameter list will not be visible outside of this definition or declaration
    static inline void sja1105_tas_setup(struct sja1105_private *priv) { }
                                                ^~~~~~~~~~~~~~~
   drivers/net/dsa/sja1105/sja1105_tas.h:40:48: warning: 'struct sja1105_private' declared inside parameter list will not be visible outside of this definition or declaration
    static inline void sja1105_tas_teardown(struct sja1105_private *priv) { }
                                                   ^~~~~~~~~~~~~~~
   drivers/net/dsa/sja1105/sja1105_main.c: In function 'sja1105_teardown':
>> drivers/net/dsa/sja1105/sja1105_main.c:1731:23: error: passing argument 1 of 'sja1105_tas_teardown' from incompatible pointer type [-Werror=incompatible-pointer-types]
     sja1105_tas_teardown(priv);
                          ^~~~
   In file included from drivers/net/dsa/sja1105/sja1105.h:23:0,
                    from drivers/net/dsa/sja1105/sja1105_main.c:24:
   drivers/net/dsa/sja1105/sja1105_tas.h:40:20: note: expected 'struct sja1105_private *' but argument is of type 'struct sja1105_private *'
    static inline void sja1105_tas_teardown(struct sja1105_private *priv) { }
                       ^~~~~~~~~~~~~~~~~~~~
   drivers/net/dsa/sja1105/sja1105_main.c: In function 'sja1105_probe':
>> drivers/net/dsa/sja1105/sja1105_main.c:2215:20: error: passing argument 1 of 'sja1105_tas_setup' from incompatible pointer type [-Werror=incompatible-pointer-types]
     sja1105_tas_setup(priv);
                       ^~~~
   In file included from drivers/net/dsa/sja1105/sja1105.h:23:0,
                    from drivers/net/dsa/sja1105/sja1105_main.c:24:
   drivers/net/dsa/sja1105/sja1105_tas.h:38:20: note: expected 'struct sja1105_private *' but argument is of type 'struct sja1105_private *'
    static inline void sja1105_tas_setup(struct sja1105_private *priv) { }
                       ^~~~~~~~~~~~~~~~~
   cc1: some warnings being treated as errors

vim +/sja1105_tas_teardown +1731 drivers/net/dsa/sja1105/sja1105_main.c

  1726	
  1727	static void sja1105_teardown(struct dsa_switch *ds)
  1728	{
  1729		struct sja1105_private *priv = ds->priv;
  1730	
> 1731		sja1105_tas_teardown(priv);
  1732		cancel_work_sync(&priv->tagger_data.rxtstamp_work);
  1733		skb_queue_purge(&priv->tagger_data.skb_rxtstamp_queue);
  1734		sja1105_ptp_clock_unregister(priv);
  1735		sja1105_static_config_free(&priv->static_config);
  1736	}
  1737	

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation

Download attachment ".config.gz" of type "application/gzip" (31701 bytes)