* [linux-stable-rc:linux-5.10.y 3189/5262] drivers/usb/dwc2/hcd.c:3835:6: warning: stack frame size of 1032 bytes in function 'dwc2_hcd_dump_state'
@ 2021-05-19 19:48 ` kernel test robot
0 siblings, 0 replies; 14+ messages in thread
From: kernel test robot @ 2021-05-19 19:48 UTC (permalink / raw)
To: Arnd Bergmann
Cc: kbuild-all, clang-built-linux, Greg Kroah-Hartman,
Nathan Chancellor, Kees Cook, Andrew Morton,
Linux Memory Management List
[-- Attachment #1: Type: text/plain, Size: 16904 bytes --]
tree: https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.git linux-5.10.y
head: 689e89aee55c565fe90fcdf8a7e53f2f976c5946
commit: 5f2f616343b1d62d26f24316fbfc107ecf0983a9 [3189/5262] linux/compiler-clang.h: define HAVE_BUILTIN_BSWAP*
config: mips-randconfig-r015-20210519 (attached as .config)
compiler: clang version 13.0.0 (https://github.com/llvm/llvm-project 8e93d10633d751a3e9169bf9fa68326925ffa097)
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# install mips cross compiling tool for clang build
# apt-get install binutils-mips-linux-gnu
# https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.git/commit/?id=5f2f616343b1d62d26f24316fbfc107ecf0983a9
git remote add linux-stable-rc https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.git
git fetch --no-tags linux-stable-rc linux-5.10.y
git checkout 5f2f616343b1d62d26f24316fbfc107ecf0983a9
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=mips
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>
All warnings (new ones prefixed by >>):
>> drivers/usb/dwc2/hcd.c:3835:6: warning: stack frame size of 1032 bytes in function 'dwc2_hcd_dump_state' [-Wframe-larger-than=]
void dwc2_hcd_dump_state(struct dwc2_hsotg *hsotg)
^
1 warning generated.
vim +/dwc2_hcd_dump_state +3835 drivers/usb/dwc2/hcd.c
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3830
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3831 /*
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3832 * NOTE: This function will be removed once the peripheral controller code
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3833 * is integrated and the driver is stable
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3834 */
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 @3835 void dwc2_hcd_dump_state(struct dwc2_hsotg *hsotg)
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3836 {
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3837 #ifdef DEBUG
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3838 struct dwc2_host_chan *chan;
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3839 struct dwc2_hcd_urb *urb;
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3840 struct dwc2_qtd *qtd;
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3841 int num_channels;
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3842 u32 np_tx_status;
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3843 u32 p_tx_status;
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3844 int i;
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3845
bea8e86c51cf9c drivers/usb/dwc2/hcd.c John Youn 2016-11-03 3846 num_channels = hsotg->params.host_channels;
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3847 dev_dbg(hsotg->dev, "\n");
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3848 dev_dbg(hsotg->dev,
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3849 "************************************************************\n");
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3850 dev_dbg(hsotg->dev, "HCD State:\n");
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3851 dev_dbg(hsotg->dev, " Num channels: %d\n", num_channels);
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3852
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3853 for (i = 0; i < num_channels; i++) {
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3854 chan = hsotg->hc_ptr_array[i];
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3855 dev_dbg(hsotg->dev, " Channel %d:\n", i);
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3856 dev_dbg(hsotg->dev,
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3857 " dev_addr: %d, ep_num: %d, ep_is_in: %d\n",
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3858 chan->dev_addr, chan->ep_num, chan->ep_is_in);
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3859 dev_dbg(hsotg->dev, " speed: %d\n", chan->speed);
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3860 dev_dbg(hsotg->dev, " ep_type: %d\n", chan->ep_type);
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3861 dev_dbg(hsotg->dev, " max_packet: %d\n", chan->max_packet);
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3862 dev_dbg(hsotg->dev, " data_pid_start: %d\n",
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3863 chan->data_pid_start);
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3864 dev_dbg(hsotg->dev, " multi_count: %d\n", chan->multi_count);
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3865 dev_dbg(hsotg->dev, " xfer_started: %d\n",
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3866 chan->xfer_started);
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3867 dev_dbg(hsotg->dev, " xfer_buf: %p\n", chan->xfer_buf);
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3868 dev_dbg(hsotg->dev, " xfer_dma: %08lx\n",
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3869 (unsigned long)chan->xfer_dma);
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3870 dev_dbg(hsotg->dev, " xfer_len: %d\n", chan->xfer_len);
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3871 dev_dbg(hsotg->dev, " xfer_count: %d\n", chan->xfer_count);
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3872 dev_dbg(hsotg->dev, " halt_on_queue: %d\n",
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3873 chan->halt_on_queue);
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3874 dev_dbg(hsotg->dev, " halt_pending: %d\n",
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3875 chan->halt_pending);
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3876 dev_dbg(hsotg->dev, " halt_status: %d\n", chan->halt_status);
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3877 dev_dbg(hsotg->dev, " do_split: %d\n", chan->do_split);
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3878 dev_dbg(hsotg->dev, " complete_split: %d\n",
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3879 chan->complete_split);
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3880 dev_dbg(hsotg->dev, " hub_addr: %d\n", chan->hub_addr);
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3881 dev_dbg(hsotg->dev, " hub_port: %d\n", chan->hub_port);
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3882 dev_dbg(hsotg->dev, " xact_pos: %d\n", chan->xact_pos);
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3883 dev_dbg(hsotg->dev, " requests: %d\n", chan->requests);
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3884 dev_dbg(hsotg->dev, " qh: %p\n", chan->qh);
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3885
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3886 if (chan->xfer_started) {
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3887 u32 hfnum, hcchar, hctsiz, hcint, hcintmsk;
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3888
f25c42b8d604fb drivers/usb/dwc2/hcd.c Gevorg Sahakyan 2018-07-26 3889 hfnum = dwc2_readl(hsotg, HFNUM);
f25c42b8d604fb drivers/usb/dwc2/hcd.c Gevorg Sahakyan 2018-07-26 3890 hcchar = dwc2_readl(hsotg, HCCHAR(i));
f25c42b8d604fb drivers/usb/dwc2/hcd.c Gevorg Sahakyan 2018-07-26 3891 hctsiz = dwc2_readl(hsotg, HCTSIZ(i));
f25c42b8d604fb drivers/usb/dwc2/hcd.c Gevorg Sahakyan 2018-07-26 3892 hcint = dwc2_readl(hsotg, HCINT(i));
f25c42b8d604fb drivers/usb/dwc2/hcd.c Gevorg Sahakyan 2018-07-26 3893 hcintmsk = dwc2_readl(hsotg, HCINTMSK(i));
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3894 dev_dbg(hsotg->dev, " hfnum: 0x%08x\n", hfnum);
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3895 dev_dbg(hsotg->dev, " hcchar: 0x%08x\n", hcchar);
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3896 dev_dbg(hsotg->dev, " hctsiz: 0x%08x\n", hctsiz);
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3897 dev_dbg(hsotg->dev, " hcint: 0x%08x\n", hcint);
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3898 dev_dbg(hsotg->dev, " hcintmsk: 0x%08x\n", hcintmsk);
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3899 }
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3900
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3901 if (!(chan->xfer_started && chan->qh))
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3902 continue;
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3903
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3904 list_for_each_entry(qtd, &chan->qh->qtd_list, qtd_list_entry) {
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3905 if (!qtd->in_process)
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3906 break;
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3907 urb = qtd->urb;
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3908 dev_dbg(hsotg->dev, " URB Info:\n");
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3909 dev_dbg(hsotg->dev, " qtd: %p, urb: %p\n",
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3910 qtd, urb);
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3911 if (urb) {
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3912 dev_dbg(hsotg->dev,
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3913 " Dev: %d, EP: %d %s\n",
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3914 dwc2_hcd_get_dev_addr(&urb->pipe_info),
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3915 dwc2_hcd_get_ep_num(&urb->pipe_info),
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3916 dwc2_hcd_is_pipe_in(&urb->pipe_info) ?
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3917 "IN" : "OUT");
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3918 dev_dbg(hsotg->dev,
babd183915e91a drivers/usb/dwc2/hcd.c Douglas Anderson 2019-05-31 3919 " Max packet size: %d (%d mult)\n",
babd183915e91a drivers/usb/dwc2/hcd.c Douglas Anderson 2019-05-31 3920 dwc2_hcd_get_maxp(&urb->pipe_info),
babd183915e91a drivers/usb/dwc2/hcd.c Douglas Anderson 2019-05-31 3921 dwc2_hcd_get_maxp_mult(&urb->pipe_info));
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3922 dev_dbg(hsotg->dev,
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3923 " transfer_buffer: %p\n",
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3924 urb->buf);
157dfaac1f2922 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-14 3925 dev_dbg(hsotg->dev,
157dfaac1f2922 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-14 3926 " transfer_dma: %08lx\n",
157dfaac1f2922 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-14 3927 (unsigned long)urb->dma);
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3928 dev_dbg(hsotg->dev,
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3929 " transfer_buffer_length: %d\n",
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3930 urb->length);
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3931 dev_dbg(hsotg->dev, " actual_length: %d\n",
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3932 urb->actual_length);
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3933 }
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3934 }
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3935 }
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3936
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3937 dev_dbg(hsotg->dev, " non_periodic_channels: %d\n",
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3938 hsotg->non_periodic_channels);
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3939 dev_dbg(hsotg->dev, " periodic_channels: %d\n",
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3940 hsotg->periodic_channels);
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3941 dev_dbg(hsotg->dev, " periodic_usecs: %d\n", hsotg->periodic_usecs);
f25c42b8d604fb drivers/usb/dwc2/hcd.c Gevorg Sahakyan 2018-07-26 3942 np_tx_status = dwc2_readl(hsotg, GNPTXSTS);
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3943 dev_dbg(hsotg->dev, " NP Tx Req Queue Space Avail: %d\n",
d6ec53e04bf790 drivers/staging/dwc2/hcd.c Matthijs Kooijman 2013-08-30 3944 (np_tx_status & TXSTS_QSPCAVAIL_MASK) >> TXSTS_QSPCAVAIL_SHIFT);
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3945 dev_dbg(hsotg->dev, " NP Tx FIFO Space Avail: %d\n",
d6ec53e04bf790 drivers/staging/dwc2/hcd.c Matthijs Kooijman 2013-08-30 3946 (np_tx_status & TXSTS_FSPCAVAIL_MASK) >> TXSTS_FSPCAVAIL_SHIFT);
f25c42b8d604fb drivers/usb/dwc2/hcd.c Gevorg Sahakyan 2018-07-26 3947 p_tx_status = dwc2_readl(hsotg, HPTXSTS);
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3948 dev_dbg(hsotg->dev, " P Tx Req Queue Space Avail: %d\n",
d6ec53e04bf790 drivers/staging/dwc2/hcd.c Matthijs Kooijman 2013-08-30 3949 (p_tx_status & TXSTS_QSPCAVAIL_MASK) >> TXSTS_QSPCAVAIL_SHIFT);
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3950 dev_dbg(hsotg->dev, " P Tx FIFO Space Avail: %d\n",
d6ec53e04bf790 drivers/staging/dwc2/hcd.c Matthijs Kooijman 2013-08-30 3951 (p_tx_status & TXSTS_FSPCAVAIL_MASK) >> TXSTS_FSPCAVAIL_SHIFT);
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3952 dwc2_dump_global_registers(hsotg);
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3953 dwc2_dump_host_registers(hsotg);
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3954 dev_dbg(hsotg->dev,
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3955 "************************************************************\n");
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3956 dev_dbg(hsotg->dev, "\n");
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3957 #endif
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3958 }
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3959
:::::: The code at line 3835 was first introduced by commit
:::::: 7359d482eb4d3967cc8be354405ae6be6eaf732c staging: HCD files for the DWC2 driver
:::::: TO: Paul Zimmerman <Paul.Zimmerman@synopsys.com>
:::::: CC: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 27115 bytes --]
^ permalink raw reply [flat|nested] 14+ messages in thread
* [linux-stable-rc:linux-5.10.y 3189/5262] drivers/usb/dwc2/hcd.c:3835:6: warning: stack frame size of 1032 bytes in function 'dwc2_hcd_dump_state'
@ 2021-05-19 19:48 ` kernel test robot
0 siblings, 0 replies; 14+ messages in thread
From: kernel test robot @ 2021-05-19 19:48 UTC (permalink / raw)
To: kbuild-all
[-- Attachment #1: Type: text/plain, Size: 17076 bytes --]
tree: https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.git linux-5.10.y
head: 689e89aee55c565fe90fcdf8a7e53f2f976c5946
commit: 5f2f616343b1d62d26f24316fbfc107ecf0983a9 [3189/5262] linux/compiler-clang.h: define HAVE_BUILTIN_BSWAP*
config: mips-randconfig-r015-20210519 (attached as .config)
compiler: clang version 13.0.0 (https://github.com/llvm/llvm-project 8e93d10633d751a3e9169bf9fa68326925ffa097)
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# install mips cross compiling tool for clang build
# apt-get install binutils-mips-linux-gnu
# https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.git/commit/?id=5f2f616343b1d62d26f24316fbfc107ecf0983a9
git remote add linux-stable-rc https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.git
git fetch --no-tags linux-stable-rc linux-5.10.y
git checkout 5f2f616343b1d62d26f24316fbfc107ecf0983a9
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=mips
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>
All warnings (new ones prefixed by >>):
>> drivers/usb/dwc2/hcd.c:3835:6: warning: stack frame size of 1032 bytes in function 'dwc2_hcd_dump_state' [-Wframe-larger-than=]
void dwc2_hcd_dump_state(struct dwc2_hsotg *hsotg)
^
1 warning generated.
vim +/dwc2_hcd_dump_state +3835 drivers/usb/dwc2/hcd.c
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3830
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3831 /*
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3832 * NOTE: This function will be removed once the peripheral controller code
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3833 * is integrated and the driver is stable
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3834 */
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 @3835 void dwc2_hcd_dump_state(struct dwc2_hsotg *hsotg)
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3836 {
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3837 #ifdef DEBUG
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3838 struct dwc2_host_chan *chan;
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3839 struct dwc2_hcd_urb *urb;
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3840 struct dwc2_qtd *qtd;
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3841 int num_channels;
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3842 u32 np_tx_status;
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3843 u32 p_tx_status;
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3844 int i;
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3845
bea8e86c51cf9c drivers/usb/dwc2/hcd.c John Youn 2016-11-03 3846 num_channels = hsotg->params.host_channels;
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3847 dev_dbg(hsotg->dev, "\n");
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3848 dev_dbg(hsotg->dev,
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3849 "************************************************************\n");
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3850 dev_dbg(hsotg->dev, "HCD State:\n");
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3851 dev_dbg(hsotg->dev, " Num channels: %d\n", num_channels);
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3852
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3853 for (i = 0; i < num_channels; i++) {
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3854 chan = hsotg->hc_ptr_array[i];
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3855 dev_dbg(hsotg->dev, " Channel %d:\n", i);
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3856 dev_dbg(hsotg->dev,
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3857 " dev_addr: %d, ep_num: %d, ep_is_in: %d\n",
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3858 chan->dev_addr, chan->ep_num, chan->ep_is_in);
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3859 dev_dbg(hsotg->dev, " speed: %d\n", chan->speed);
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3860 dev_dbg(hsotg->dev, " ep_type: %d\n", chan->ep_type);
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3861 dev_dbg(hsotg->dev, " max_packet: %d\n", chan->max_packet);
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3862 dev_dbg(hsotg->dev, " data_pid_start: %d\n",
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3863 chan->data_pid_start);
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3864 dev_dbg(hsotg->dev, " multi_count: %d\n", chan->multi_count);
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3865 dev_dbg(hsotg->dev, " xfer_started: %d\n",
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3866 chan->xfer_started);
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3867 dev_dbg(hsotg->dev, " xfer_buf: %p\n", chan->xfer_buf);
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3868 dev_dbg(hsotg->dev, " xfer_dma: %08lx\n",
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3869 (unsigned long)chan->xfer_dma);
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3870 dev_dbg(hsotg->dev, " xfer_len: %d\n", chan->xfer_len);
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3871 dev_dbg(hsotg->dev, " xfer_count: %d\n", chan->xfer_count);
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3872 dev_dbg(hsotg->dev, " halt_on_queue: %d\n",
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3873 chan->halt_on_queue);
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3874 dev_dbg(hsotg->dev, " halt_pending: %d\n",
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3875 chan->halt_pending);
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3876 dev_dbg(hsotg->dev, " halt_status: %d\n", chan->halt_status);
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3877 dev_dbg(hsotg->dev, " do_split: %d\n", chan->do_split);
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3878 dev_dbg(hsotg->dev, " complete_split: %d\n",
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3879 chan->complete_split);
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3880 dev_dbg(hsotg->dev, " hub_addr: %d\n", chan->hub_addr);
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3881 dev_dbg(hsotg->dev, " hub_port: %d\n", chan->hub_port);
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3882 dev_dbg(hsotg->dev, " xact_pos: %d\n", chan->xact_pos);
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3883 dev_dbg(hsotg->dev, " requests: %d\n", chan->requests);
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3884 dev_dbg(hsotg->dev, " qh: %p\n", chan->qh);
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3885
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3886 if (chan->xfer_started) {
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3887 u32 hfnum, hcchar, hctsiz, hcint, hcintmsk;
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3888
f25c42b8d604fb drivers/usb/dwc2/hcd.c Gevorg Sahakyan 2018-07-26 3889 hfnum = dwc2_readl(hsotg, HFNUM);
f25c42b8d604fb drivers/usb/dwc2/hcd.c Gevorg Sahakyan 2018-07-26 3890 hcchar = dwc2_readl(hsotg, HCCHAR(i));
f25c42b8d604fb drivers/usb/dwc2/hcd.c Gevorg Sahakyan 2018-07-26 3891 hctsiz = dwc2_readl(hsotg, HCTSIZ(i));
f25c42b8d604fb drivers/usb/dwc2/hcd.c Gevorg Sahakyan 2018-07-26 3892 hcint = dwc2_readl(hsotg, HCINT(i));
f25c42b8d604fb drivers/usb/dwc2/hcd.c Gevorg Sahakyan 2018-07-26 3893 hcintmsk = dwc2_readl(hsotg, HCINTMSK(i));
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3894 dev_dbg(hsotg->dev, " hfnum: 0x%08x\n", hfnum);
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3895 dev_dbg(hsotg->dev, " hcchar: 0x%08x\n", hcchar);
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3896 dev_dbg(hsotg->dev, " hctsiz: 0x%08x\n", hctsiz);
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3897 dev_dbg(hsotg->dev, " hcint: 0x%08x\n", hcint);
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3898 dev_dbg(hsotg->dev, " hcintmsk: 0x%08x\n", hcintmsk);
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3899 }
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3900
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3901 if (!(chan->xfer_started && chan->qh))
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3902 continue;
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3903
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3904 list_for_each_entry(qtd, &chan->qh->qtd_list, qtd_list_entry) {
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3905 if (!qtd->in_process)
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3906 break;
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3907 urb = qtd->urb;
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3908 dev_dbg(hsotg->dev, " URB Info:\n");
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3909 dev_dbg(hsotg->dev, " qtd: %p, urb: %p\n",
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3910 qtd, urb);
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3911 if (urb) {
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3912 dev_dbg(hsotg->dev,
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3913 " Dev: %d, EP: %d %s\n",
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3914 dwc2_hcd_get_dev_addr(&urb->pipe_info),
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3915 dwc2_hcd_get_ep_num(&urb->pipe_info),
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3916 dwc2_hcd_is_pipe_in(&urb->pipe_info) ?
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3917 "IN" : "OUT");
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3918 dev_dbg(hsotg->dev,
babd183915e91a drivers/usb/dwc2/hcd.c Douglas Anderson 2019-05-31 3919 " Max packet size: %d (%d mult)\n",
babd183915e91a drivers/usb/dwc2/hcd.c Douglas Anderson 2019-05-31 3920 dwc2_hcd_get_maxp(&urb->pipe_info),
babd183915e91a drivers/usb/dwc2/hcd.c Douglas Anderson 2019-05-31 3921 dwc2_hcd_get_maxp_mult(&urb->pipe_info));
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3922 dev_dbg(hsotg->dev,
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3923 " transfer_buffer: %p\n",
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3924 urb->buf);
157dfaac1f2922 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-14 3925 dev_dbg(hsotg->dev,
157dfaac1f2922 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-14 3926 " transfer_dma: %08lx\n",
157dfaac1f2922 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-14 3927 (unsigned long)urb->dma);
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3928 dev_dbg(hsotg->dev,
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3929 " transfer_buffer_length: %d\n",
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3930 urb->length);
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3931 dev_dbg(hsotg->dev, " actual_length: %d\n",
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3932 urb->actual_length);
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3933 }
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3934 }
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3935 }
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3936
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3937 dev_dbg(hsotg->dev, " non_periodic_channels: %d\n",
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3938 hsotg->non_periodic_channels);
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3939 dev_dbg(hsotg->dev, " periodic_channels: %d\n",
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3940 hsotg->periodic_channels);
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3941 dev_dbg(hsotg->dev, " periodic_usecs: %d\n", hsotg->periodic_usecs);
f25c42b8d604fb drivers/usb/dwc2/hcd.c Gevorg Sahakyan 2018-07-26 3942 np_tx_status = dwc2_readl(hsotg, GNPTXSTS);
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3943 dev_dbg(hsotg->dev, " NP Tx Req Queue Space Avail: %d\n",
d6ec53e04bf790 drivers/staging/dwc2/hcd.c Matthijs Kooijman 2013-08-30 3944 (np_tx_status & TXSTS_QSPCAVAIL_MASK) >> TXSTS_QSPCAVAIL_SHIFT);
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3945 dev_dbg(hsotg->dev, " NP Tx FIFO Space Avail: %d\n",
d6ec53e04bf790 drivers/staging/dwc2/hcd.c Matthijs Kooijman 2013-08-30 3946 (np_tx_status & TXSTS_FSPCAVAIL_MASK) >> TXSTS_FSPCAVAIL_SHIFT);
f25c42b8d604fb drivers/usb/dwc2/hcd.c Gevorg Sahakyan 2018-07-26 3947 p_tx_status = dwc2_readl(hsotg, HPTXSTS);
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3948 dev_dbg(hsotg->dev, " P Tx Req Queue Space Avail: %d\n",
d6ec53e04bf790 drivers/staging/dwc2/hcd.c Matthijs Kooijman 2013-08-30 3949 (p_tx_status & TXSTS_QSPCAVAIL_MASK) >> TXSTS_QSPCAVAIL_SHIFT);
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3950 dev_dbg(hsotg->dev, " P Tx FIFO Space Avail: %d\n",
d6ec53e04bf790 drivers/staging/dwc2/hcd.c Matthijs Kooijman 2013-08-30 3951 (p_tx_status & TXSTS_FSPCAVAIL_MASK) >> TXSTS_FSPCAVAIL_SHIFT);
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3952 dwc2_dump_global_registers(hsotg);
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3953 dwc2_dump_host_registers(hsotg);
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3954 dev_dbg(hsotg->dev,
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3955 "************************************************************\n");
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3956 dev_dbg(hsotg->dev, "\n");
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3957 #endif
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3958 }
7359d482eb4d39 drivers/staging/dwc2/hcd.c Paul Zimmerman 2013-03-11 3959
:::::: The code at line 3835 was first introduced by commit
:::::: 7359d482eb4d3967cc8be354405ae6be6eaf732c staging: HCD files for the DWC2 driver
:::::: TO: Paul Zimmerman <Paul.Zimmerman@synopsys.com>
:::::: CC: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org
[-- Attachment #2: config.gz --]
[-- Type: application/gzip, Size: 27115 bytes --]
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [linux-stable-rc:linux-5.10.y 3189/5262] drivers/usb/dwc2/hcd.c:3835:6: warning: stack frame size of 1032 bytes in function 'dwc2_hcd_dump_state'
2021-05-19 19:48 ` kernel test robot
@ 2021-05-19 20:28 ` Arnd Bergmann
-1 siblings, 0 replies; 14+ messages in thread
From: Arnd Bergmann @ 2021-05-19 20:28 UTC (permalink / raw)
To: kernel test robot
Cc: kbuild-all, clang-built-linux, Greg Kroah-Hartman,
Nathan Chancellor, Kees Cook, Andrew Morton,
Linux Memory Management List
On Wed, May 19, 2021 at 9:48 PM kernel test robot <lkp@intel.com> wrote:
>
> tree: https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.git linux-5.10.y
> head: 689e89aee55c565fe90fcdf8a7e53f2f976c5946
> commit: 5f2f616343b1d62d26f24316fbfc107ecf0983a9 [3189/5262] linux/compiler-clang.h: define HAVE_BUILTIN_BSWAP*
> config: mips-randconfig-r015-20210519 (attached as .config)
> compiler: clang version 13.0.0 (https://github.com/llvm/llvm-project 8e93d10633d751a3e9169bf9fa68326925ffa097)
> reproduce (this is a W=1 build):
> wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
> chmod +x ~/bin/make.cross
> # install mips cross compiling tool for clang build
> # apt-get install binutils-mips-linux-gnu
> # https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.git/commit/?id=5f2f616343b1d62d26f24316fbfc107ecf0983a9
> git remote add linux-stable-rc https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.git
> git fetch --no-tags linux-stable-rc linux-5.10.y
> git checkout 5f2f616343b1d62d26f24316fbfc107ecf0983a9
> # save the attached .config to linux build tree
> COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=mips
>
> If you fix the issue, kindly add following tag as appropriate
> Reported-by: kernel test robot <lkp@intel.com>
>
> All warnings (new ones prefixed by >>):
>
I analyzed another bug like this today, and I think this is again just
CONFIG_UBSAN_ALIGNMENT=y
on mips/clang causing tons of these warnings.
I don't think this should be considered a regression, but we may want
to change either the
kernel or the 0day bot from test building this combination of options.
Arnd
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [linux-stable-rc:linux-5.10.y 3189/5262] drivers/usb/dwc2/hcd.c:3835:6: warning: stack frame size of 1032 bytes in function 'dwc2_hcd_dump_state'
@ 2021-05-19 20:28 ` Arnd Bergmann
0 siblings, 0 replies; 14+ messages in thread
From: Arnd Bergmann @ 2021-05-19 20:28 UTC (permalink / raw)
To: kbuild-all
[-- Attachment #1: Type: text/plain, Size: 1833 bytes --]
On Wed, May 19, 2021 at 9:48 PM kernel test robot <lkp@intel.com> wrote:
>
> tree: https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.git linux-5.10.y
> head: 689e89aee55c565fe90fcdf8a7e53f2f976c5946
> commit: 5f2f616343b1d62d26f24316fbfc107ecf0983a9 [3189/5262] linux/compiler-clang.h: define HAVE_BUILTIN_BSWAP*
> config: mips-randconfig-r015-20210519 (attached as .config)
> compiler: clang version 13.0.0 (https://github.com/llvm/llvm-project 8e93d10633d751a3e9169bf9fa68326925ffa097)
> reproduce (this is a W=1 build):
> wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
> chmod +x ~/bin/make.cross
> # install mips cross compiling tool for clang build
> # apt-get install binutils-mips-linux-gnu
> # https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.git/commit/?id=5f2f616343b1d62d26f24316fbfc107ecf0983a9
> git remote add linux-stable-rc https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.git
> git fetch --no-tags linux-stable-rc linux-5.10.y
> git checkout 5f2f616343b1d62d26f24316fbfc107ecf0983a9
> # save the attached .config to linux build tree
> COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=mips
>
> If you fix the issue, kindly add following tag as appropriate
> Reported-by: kernel test robot <lkp@intel.com>
>
> All warnings (new ones prefixed by >>):
>
I analyzed another bug like this today, and I think this is again just
CONFIG_UBSAN_ALIGNMENT=y
on mips/clang causing tons of these warnings.
I don't think this should be considered a regression, but we may want
to change either the
kernel or the 0day bot from test building this combination of options.
Arnd
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [linux-stable-rc:linux-5.10.y 3189/5262] drivers/usb/dwc2/hcd.c:3835:6: warning: stack frame size of 1032 bytes in function 'dwc2_hcd_dump_state'
2021-05-19 20:28 ` Arnd Bergmann
@ 2021-05-20 9:20 ` Rong Chen
-1 siblings, 0 replies; 14+ messages in thread
From: Rong Chen @ 2021-05-20 9:20 UTC (permalink / raw)
To: Arnd Bergmann, kernel test robot
Cc: kbuild-all, clang-built-linux, Greg Kroah-Hartman,
Nathan Chancellor, Kees Cook, Andrew Morton,
Linux Memory Management List
On 5/20/21 4:28 AM, Arnd Bergmann wrote:
> On Wed, May 19, 2021 at 9:48 PM kernel test robot <lkp@intel.com> wrote:
>> tree: https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.git linux-5.10.y
>> head: 689e89aee55c565fe90fcdf8a7e53f2f976c5946
>> commit: 5f2f616343b1d62d26f24316fbfc107ecf0983a9 [3189/5262] linux/compiler-clang.h: define HAVE_BUILTIN_BSWAP*
>> config: mips-randconfig-r015-20210519 (attached as .config)
>> compiler: clang version 13.0.0 (https://github.com/llvm/llvm-project 8e93d10633d751a3e9169bf9fa68326925ffa097)
>> reproduce (this is a W=1 build):
>> wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
>> chmod +x ~/bin/make.cross
>> # install mips cross compiling tool for clang build
>> # apt-get install binutils-mips-linux-gnu
>> # https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.git/commit/?id=5f2f616343b1d62d26f24316fbfc107ecf0983a9
>> git remote add linux-stable-rc https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.git
>> git fetch --no-tags linux-stable-rc linux-5.10.y
>> git checkout 5f2f616343b1d62d26f24316fbfc107ecf0983a9
>> # save the attached .config to linux build tree
>> COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=mips
>>
>> If you fix the issue, kindly add following tag as appropriate
>> Reported-by: kernel test robot <lkp@intel.com>
>>
>> All warnings (new ones prefixed by >>):
>>
> I analyzed another bug like this today, and I think this is again just
> CONFIG_UBSAN_ALIGNMENT=y
> on mips/clang causing tons of these warnings.
>
> I don't think this should be considered a regression, but we may want
> to change either the
> kernel or the 0day bot from test building this combination of options.
>
> Arnd
>
Hi Arnd,
We have disabled CONFIG_UBSAN_ALIGNMENT for mips/clang in our kconfigs
to avoid reporting such warnings again.
Best Regards,
Rong Chen
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [linux-stable-rc:linux-5.10.y 3189/5262] drivers/usb/dwc2/hcd.c:3835:6: warning: stack frame size of 1032 bytes in function 'dwc2_hcd_dump_state'
@ 2021-05-20 9:20 ` Rong Chen
0 siblings, 0 replies; 14+ messages in thread
From: Rong Chen @ 2021-05-20 9:20 UTC (permalink / raw)
To: kbuild-all
[-- Attachment #1: Type: text/plain, Size: 2083 bytes --]
On 5/20/21 4:28 AM, Arnd Bergmann wrote:
> On Wed, May 19, 2021 at 9:48 PM kernel test robot <lkp@intel.com> wrote:
>> tree: https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.git linux-5.10.y
>> head: 689e89aee55c565fe90fcdf8a7e53f2f976c5946
>> commit: 5f2f616343b1d62d26f24316fbfc107ecf0983a9 [3189/5262] linux/compiler-clang.h: define HAVE_BUILTIN_BSWAP*
>> config: mips-randconfig-r015-20210519 (attached as .config)
>> compiler: clang version 13.0.0 (https://github.com/llvm/llvm-project 8e93d10633d751a3e9169bf9fa68326925ffa097)
>> reproduce (this is a W=1 build):
>> wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
>> chmod +x ~/bin/make.cross
>> # install mips cross compiling tool for clang build
>> # apt-get install binutils-mips-linux-gnu
>> # https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.git/commit/?id=5f2f616343b1d62d26f24316fbfc107ecf0983a9
>> git remote add linux-stable-rc https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.git
>> git fetch --no-tags linux-stable-rc linux-5.10.y
>> git checkout 5f2f616343b1d62d26f24316fbfc107ecf0983a9
>> # save the attached .config to linux build tree
>> COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=mips
>>
>> If you fix the issue, kindly add following tag as appropriate
>> Reported-by: kernel test robot <lkp@intel.com>
>>
>> All warnings (new ones prefixed by >>):
>>
> I analyzed another bug like this today, and I think this is again just
> CONFIG_UBSAN_ALIGNMENT=y
> on mips/clang causing tons of these warnings.
>
> I don't think this should be considered a regression, but we may want
> to change either the
> kernel or the 0day bot from test building this combination of options.
>
> Arnd
>
Hi Arnd,
We have disabled CONFIG_UBSAN_ALIGNMENT for mips/clang in our kconfigs
to avoid reporting such warnings again.
Best Regards,
Rong Chen
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [linux-stable-rc:linux-5.10.y 3189/5262] drivers/usb/dwc2/hcd.c:3835:6: warning: stack frame size of 1032 bytes in function 'dwc2_hcd_dump_state'
2021-05-20 9:20 ` Rong Chen
@ 2021-05-20 9:27 ` Arnd Bergmann
-1 siblings, 0 replies; 14+ messages in thread
From: Arnd Bergmann @ 2021-05-20 9:27 UTC (permalink / raw)
To: Rong Chen
Cc: kernel test robot, kbuild-all, clang-built-linux,
Greg Kroah-Hartman, Nathan Chancellor, Kees Cook, Andrew Morton,
Linux Memory Management List
On Thu, May 20, 2021 at 11:22 AM Rong Chen <rong.a.chen@intel.com> wrote:
> On 5/20/21 4:28 AM, Arnd Bergmann wrote:
> > On Wed, May 19, 2021 at 9:48 PM kernel test robot <lkp@intel.com> wrote:
> >> tree: https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.git linux-5.10.y
> >> head: 689e89aee55c565fe90fcdf8a7e53f2f976c5946
> >> commit: 5f2f616343b1d62d26f24316fbfc107ecf0983a9 [3189/5262] linux/compiler-clang.h: define HAVE_BUILTIN_BSWAP*
> >> config: mips-randconfig-r015-20210519 (attached as .config)
> >> compiler: clang version 13.0.0 (https://github.com/llvm/llvm-project 8e93d10633d751a3e9169bf9fa68326925ffa097)
> >> reproduce (this is a W=1 build):
> >> wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
> >> chmod +x ~/bin/make.cross
> >> # install mips cross compiling tool for clang build
> >> # apt-get install binutils-mips-linux-gnu
> >> # https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.git/commit/?id=5f2f616343b1d62d26f24316fbfc107ecf0983a9
> >> git remote add linux-stable-rc https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.git
> >> git fetch --no-tags linux-stable-rc linux-5.10.y
> >> git checkout 5f2f616343b1d62d26f24316fbfc107ecf0983a9
> >> # save the attached .config to linux build tree
> >> COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=mips
> >>
> >> If you fix the issue, kindly add following tag as appropriate
> >> Reported-by: kernel test robot <lkp@intel.com>
> >>
> >> All warnings (new ones prefixed by >>):
> >>
> > I analyzed another bug like this today, and I think this is again just
> > CONFIG_UBSAN_ALIGNMENT=y
> > on mips/clang causing tons of these warnings.
> >
> > I don't think this should be considered a regression, but we may want
> > to change either the
> > kernel or the 0day bot from test building this combination of options.
> >
> > Arnd
> >
>
> Hi Arnd,
>
> We have disabled CONFIG_UBSAN_ALIGNMENT for mips/clang in our kconfigs
> to avoid reporting such warnings again.
Thanks a lot, that helps.
Have you considered always enforcing CONFIG_COMPILE_TEST=y for any
randconfig build? That would address this issue along with a number of similar
ones. I have sent a number of patches to add COMPILE_TEST dependencies
for things that otherwise break in randconfig builds but could still be useful
in more specific cases when users know better.
Arnd
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [linux-stable-rc:linux-5.10.y 3189/5262] drivers/usb/dwc2/hcd.c:3835:6: warning: stack frame size of 1032 bytes in function 'dwc2_hcd_dump_state'
@ 2021-05-20 9:27 ` Arnd Bergmann
0 siblings, 0 replies; 14+ messages in thread
From: Arnd Bergmann @ 2021-05-20 9:27 UTC (permalink / raw)
To: kbuild-all
[-- Attachment #1: Type: text/plain, Size: 2596 bytes --]
On Thu, May 20, 2021 at 11:22 AM Rong Chen <rong.a.chen@intel.com> wrote:
> On 5/20/21 4:28 AM, Arnd Bergmann wrote:
> > On Wed, May 19, 2021 at 9:48 PM kernel test robot <lkp@intel.com> wrote:
> >> tree: https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.git linux-5.10.y
> >> head: 689e89aee55c565fe90fcdf8a7e53f2f976c5946
> >> commit: 5f2f616343b1d62d26f24316fbfc107ecf0983a9 [3189/5262] linux/compiler-clang.h: define HAVE_BUILTIN_BSWAP*
> >> config: mips-randconfig-r015-20210519 (attached as .config)
> >> compiler: clang version 13.0.0 (https://github.com/llvm/llvm-project 8e93d10633d751a3e9169bf9fa68326925ffa097)
> >> reproduce (this is a W=1 build):
> >> wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
> >> chmod +x ~/bin/make.cross
> >> # install mips cross compiling tool for clang build
> >> # apt-get install binutils-mips-linux-gnu
> >> # https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.git/commit/?id=5f2f616343b1d62d26f24316fbfc107ecf0983a9
> >> git remote add linux-stable-rc https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.git
> >> git fetch --no-tags linux-stable-rc linux-5.10.y
> >> git checkout 5f2f616343b1d62d26f24316fbfc107ecf0983a9
> >> # save the attached .config to linux build tree
> >> COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=mips
> >>
> >> If you fix the issue, kindly add following tag as appropriate
> >> Reported-by: kernel test robot <lkp@intel.com>
> >>
> >> All warnings (new ones prefixed by >>):
> >>
> > I analyzed another bug like this today, and I think this is again just
> > CONFIG_UBSAN_ALIGNMENT=y
> > on mips/clang causing tons of these warnings.
> >
> > I don't think this should be considered a regression, but we may want
> > to change either the
> > kernel or the 0day bot from test building this combination of options.
> >
> > Arnd
> >
>
> Hi Arnd,
>
> We have disabled CONFIG_UBSAN_ALIGNMENT for mips/clang in our kconfigs
> to avoid reporting such warnings again.
Thanks a lot, that helps.
Have you considered always enforcing CONFIG_COMPILE_TEST=y for any
randconfig build? That would address this issue along with a number of similar
ones. I have sent a number of patches to add COMPILE_TEST dependencies
for things that otherwise break in randconfig builds but could still be useful
in more specific cases when users know better.
Arnd
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [kbuild-all] Re: [linux-stable-rc:linux-5.10.y 3189/5262] drivers/usb/dwc2/hcd.c:3835:6: warning: stack frame size of 1032 bytes in function 'dwc2_hcd_dump_state'
2021-05-20 9:27 ` Arnd Bergmann
@ 2021-05-20 9:56 ` Rong Chen
-1 siblings, 0 replies; 14+ messages in thread
From: Rong Chen @ 2021-05-20 9:56 UTC (permalink / raw)
To: Arnd Bergmann
Cc: kernel test robot, kbuild-all, clang-built-linux,
Greg Kroah-Hartman, Nathan Chancellor, Kees Cook, Andrew Morton,
Linux Memory Management List
On 5/20/21 5:27 PM, Arnd Bergmann wrote:
> On Thu, May 20, 2021 at 11:22 AM Rong Chen <rong.a.chen@intel.com> wrote:
>> On 5/20/21 4:28 AM, Arnd Bergmann wrote:
>>> On Wed, May 19, 2021 at 9:48 PM kernel test robot <lkp@intel.com> wrote:
>>>> tree: https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.git linux-5.10.y
>>>> head: 689e89aee55c565fe90fcdf8a7e53f2f976c5946
>>>> commit: 5f2f616343b1d62d26f24316fbfc107ecf0983a9 [3189/5262] linux/compiler-clang.h: define HAVE_BUILTIN_BSWAP*
>>>> config: mips-randconfig-r015-20210519 (attached as .config)
>>>> compiler: clang version 13.0.0 (https://github.com/llvm/llvm-project 8e93d10633d751a3e9169bf9fa68326925ffa097)
>>>> reproduce (this is a W=1 build):
>>>> wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
>>>> chmod +x ~/bin/make.cross
>>>> # install mips cross compiling tool for clang build
>>>> # apt-get install binutils-mips-linux-gnu
>>>> # https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.git/commit/?id=5f2f616343b1d62d26f24316fbfc107ecf0983a9
>>>> git remote add linux-stable-rc https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.git
>>>> git fetch --no-tags linux-stable-rc linux-5.10.y
>>>> git checkout 5f2f616343b1d62d26f24316fbfc107ecf0983a9
>>>> # save the attached .config to linux build tree
>>>> COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=mips
>>>>
>>>> If you fix the issue, kindly add following tag as appropriate
>>>> Reported-by: kernel test robot <lkp@intel.com>
>>>>
>>>> All warnings (new ones prefixed by >>):
>>>>
>>> I analyzed another bug like this today, and I think this is again just
>>> CONFIG_UBSAN_ALIGNMENT=y
>>> on mips/clang causing tons of these warnings.
>>>
>>> I don't think this should be considered a regression, but we may want
>>> to change either the
>>> kernel or the 0day bot from test building this combination of options.
>>>
>>> Arnd
>>>
>> Hi Arnd,
>>
>> We have disabled CONFIG_UBSAN_ALIGNMENT for mips/clang in our kconfigs
>> to avoid reporting such warnings again.
> Thanks a lot, that helps.
>
> Have you considered always enforcing CONFIG_COMPILE_TEST=y for any
> randconfig build? That would address this issue along with a number of similar
> ones. I have sent a number of patches to add COMPILE_TEST dependencies
> for things that otherwise break in randconfig builds but could still be useful
> in more specific cases when users know better.
>
> Arnd
Hi Arnd,
Thanks for the advice, we'll consider it in the near future
after assessing the existing configures.
Best Regards,
Rong Chen
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [linux-stable-rc:linux-5.10.y 3189/5262] drivers/usb/dwc2/hcd.c:3835:6: warning: stack frame size of 1032 bytes in function 'dwc2_hcd_dump_state'
@ 2021-05-20 9:56 ` Rong Chen
0 siblings, 0 replies; 14+ messages in thread
From: Rong Chen @ 2021-05-20 9:56 UTC (permalink / raw)
To: kbuild-all
[-- Attachment #1: Type: text/plain, Size: 2817 bytes --]
On 5/20/21 5:27 PM, Arnd Bergmann wrote:
> On Thu, May 20, 2021 at 11:22 AM Rong Chen <rong.a.chen@intel.com> wrote:
>> On 5/20/21 4:28 AM, Arnd Bergmann wrote:
>>> On Wed, May 19, 2021 at 9:48 PM kernel test robot <lkp@intel.com> wrote:
>>>> tree: https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.git linux-5.10.y
>>>> head: 689e89aee55c565fe90fcdf8a7e53f2f976c5946
>>>> commit: 5f2f616343b1d62d26f24316fbfc107ecf0983a9 [3189/5262] linux/compiler-clang.h: define HAVE_BUILTIN_BSWAP*
>>>> config: mips-randconfig-r015-20210519 (attached as .config)
>>>> compiler: clang version 13.0.0 (https://github.com/llvm/llvm-project 8e93d10633d751a3e9169bf9fa68326925ffa097)
>>>> reproduce (this is a W=1 build):
>>>> wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
>>>> chmod +x ~/bin/make.cross
>>>> # install mips cross compiling tool for clang build
>>>> # apt-get install binutils-mips-linux-gnu
>>>> # https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.git/commit/?id=5f2f616343b1d62d26f24316fbfc107ecf0983a9
>>>> git remote add linux-stable-rc https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.git
>>>> git fetch --no-tags linux-stable-rc linux-5.10.y
>>>> git checkout 5f2f616343b1d62d26f24316fbfc107ecf0983a9
>>>> # save the attached .config to linux build tree
>>>> COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=mips
>>>>
>>>> If you fix the issue, kindly add following tag as appropriate
>>>> Reported-by: kernel test robot <lkp@intel.com>
>>>>
>>>> All warnings (new ones prefixed by >>):
>>>>
>>> I analyzed another bug like this today, and I think this is again just
>>> CONFIG_UBSAN_ALIGNMENT=y
>>> on mips/clang causing tons of these warnings.
>>>
>>> I don't think this should be considered a regression, but we may want
>>> to change either the
>>> kernel or the 0day bot from test building this combination of options.
>>>
>>> Arnd
>>>
>> Hi Arnd,
>>
>> We have disabled CONFIG_UBSAN_ALIGNMENT for mips/clang in our kconfigs
>> to avoid reporting such warnings again.
> Thanks a lot, that helps.
>
> Have you considered always enforcing CONFIG_COMPILE_TEST=y for any
> randconfig build? That would address this issue along with a number of similar
> ones. I have sent a number of patches to add COMPILE_TEST dependencies
> for things that otherwise break in randconfig builds but could still be useful
> in more specific cases when users know better.
>
> Arnd
Hi Arnd,
Thanks for the advice, we'll consider it in the near future
after assessing the existing configures.
Best Regards,
Rong Chen
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [kbuild-all] Re: [linux-stable-rc:linux-5.10.y 3189/5262] drivers/usb/dwc2/hcd.c:3835:6: warning: stack frame size of 1032 bytes in function 'dwc2_hcd_dump_state'
2021-05-20 9:56 ` Rong Chen
@ 2021-05-21 0:57 ` Philip Li
-1 siblings, 0 replies; 14+ messages in thread
From: Philip Li @ 2021-05-21 0:57 UTC (permalink / raw)
To: Rong Chen
Cc: Arnd Bergmann, kernel test robot, kbuild-all, clang-built-linux,
Greg Kroah-Hartman, Nathan Chancellor, Kees Cook, Andrew Morton,
Linux Memory Management List
On Thu, May 20, 2021 at 05:56:35PM +0800, Rong Chen wrote:
>
>
> On 5/20/21 5:27 PM, Arnd Bergmann wrote:
> > On Thu, May 20, 2021 at 11:22 AM Rong Chen <rong.a.chen@intel.com> wrote:
> > > On 5/20/21 4:28 AM, Arnd Bergmann wrote:
> > > > On Wed, May 19, 2021 at 9:48 PM kernel test robot <lkp@intel.com> wrote:
> > > > > tree: https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.git linux-5.10.y
> > > > > head: 689e89aee55c565fe90fcdf8a7e53f2f976c5946
> > > > > commit: 5f2f616343b1d62d26f24316fbfc107ecf0983a9 [3189/5262] linux/compiler-clang.h: define HAVE_BUILTIN_BSWAP*
> > > > > config: mips-randconfig-r015-20210519 (attached as .config)
> > > > > compiler: clang version 13.0.0 (https://github.com/llvm/llvm-project 8e93d10633d751a3e9169bf9fa68326925ffa097)
> > > > > reproduce (this is a W=1 build):
> > > > > wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
> > > > > chmod +x ~/bin/make.cross
> > > > > # install mips cross compiling tool for clang build
> > > > > # apt-get install binutils-mips-linux-gnu
> > > > > # https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.git/commit/?id=5f2f616343b1d62d26f24316fbfc107ecf0983a9
> > > > > git remote add linux-stable-rc https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.git
> > > > > git fetch --no-tags linux-stable-rc linux-5.10.y
> > > > > git checkout 5f2f616343b1d62d26f24316fbfc107ecf0983a9
> > > > > # save the attached .config to linux build tree
> > > > > COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=mips
> > > > >
> > > > > If you fix the issue, kindly add following tag as appropriate
> > > > > Reported-by: kernel test robot <lkp@intel.com>
> > > > >
> > > > > All warnings (new ones prefixed by >>):
> > > > >
> > > > I analyzed another bug like this today, and I think this is again just
> > > > CONFIG_UBSAN_ALIGNMENT=y
> > > > on mips/clang causing tons of these warnings.
> > > >
> > > > I don't think this should be considered a regression, but we may want
> > > > to change either the
> > > > kernel or the 0day bot from test building this combination of options.
> > > >
> > > > Arnd
> > > >
> > > Hi Arnd,
> > >
> > > We have disabled CONFIG_UBSAN_ALIGNMENT for mips/clang in our kconfigs
> > > to avoid reporting such warnings again.
> > Thanks a lot, that helps.
> >
> > Have you considered always enforcing CONFIG_COMPILE_TEST=y for any
> > randconfig build? That would address this issue along with a number of similar
Hi Arnd, thanks for feedback, we have talked about this internally. We use randconfig
to cover different combination, some randconfig would have CONFIG_COMPILE_TEST=y, some
is not. For now, we also want to check the kernel build status when COMPILE_TEST is
not set (as if this is a valid kconfig combination). Thus we may not always enforcing
the y here, but keep it as random as possible.
> > ones. I have sent a number of patches to add COMPILE_TEST dependencies
> > for things that otherwise break in randconfig builds but could still be useful
> > in more specific cases when users know better.
> >
> > Arnd
>
> Hi Arnd,
>
> Thanks for the advice, we'll consider it in the near future
> after assessing the existing configures.
>
> Best Regards,
> Rong Chen
> _______________________________________________
> kbuild-all mailing list -- kbuild-all@lists.01.org
> To unsubscribe send an email to kbuild-all-leave@lists.01.org
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [linux-stable-rc:linux-5.10.y 3189/5262] drivers/usb/dwc2/hcd.c:3835:6: warning: stack frame size of 1032 bytes in function 'dwc2_hcd_dump_state'
@ 2021-05-21 0:57 ` Philip Li
0 siblings, 0 replies; 14+ messages in thread
From: Philip Li @ 2021-05-21 0:57 UTC (permalink / raw)
To: kbuild-all
[-- Attachment #1: Type: text/plain, Size: 3668 bytes --]
On Thu, May 20, 2021 at 05:56:35PM +0800, Rong Chen wrote:
>
>
> On 5/20/21 5:27 PM, Arnd Bergmann wrote:
> > On Thu, May 20, 2021 at 11:22 AM Rong Chen <rong.a.chen@intel.com> wrote:
> > > On 5/20/21 4:28 AM, Arnd Bergmann wrote:
> > > > On Wed, May 19, 2021 at 9:48 PM kernel test robot <lkp@intel.com> wrote:
> > > > > tree: https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.git linux-5.10.y
> > > > > head: 689e89aee55c565fe90fcdf8a7e53f2f976c5946
> > > > > commit: 5f2f616343b1d62d26f24316fbfc107ecf0983a9 [3189/5262] linux/compiler-clang.h: define HAVE_BUILTIN_BSWAP*
> > > > > config: mips-randconfig-r015-20210519 (attached as .config)
> > > > > compiler: clang version 13.0.0 (https://github.com/llvm/llvm-project 8e93d10633d751a3e9169bf9fa68326925ffa097)
> > > > > reproduce (this is a W=1 build):
> > > > > wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
> > > > > chmod +x ~/bin/make.cross
> > > > > # install mips cross compiling tool for clang build
> > > > > # apt-get install binutils-mips-linux-gnu
> > > > > # https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.git/commit/?id=5f2f616343b1d62d26f24316fbfc107ecf0983a9
> > > > > git remote add linux-stable-rc https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.git
> > > > > git fetch --no-tags linux-stable-rc linux-5.10.y
> > > > > git checkout 5f2f616343b1d62d26f24316fbfc107ecf0983a9
> > > > > # save the attached .config to linux build tree
> > > > > COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=mips
> > > > >
> > > > > If you fix the issue, kindly add following tag as appropriate
> > > > > Reported-by: kernel test robot <lkp@intel.com>
> > > > >
> > > > > All warnings (new ones prefixed by >>):
> > > > >
> > > > I analyzed another bug like this today, and I think this is again just
> > > > CONFIG_UBSAN_ALIGNMENT=y
> > > > on mips/clang causing tons of these warnings.
> > > >
> > > > I don't think this should be considered a regression, but we may want
> > > > to change either the
> > > > kernel or the 0day bot from test building this combination of options.
> > > >
> > > > Arnd
> > > >
> > > Hi Arnd,
> > >
> > > We have disabled CONFIG_UBSAN_ALIGNMENT for mips/clang in our kconfigs
> > > to avoid reporting such warnings again.
> > Thanks a lot, that helps.
> >
> > Have you considered always enforcing CONFIG_COMPILE_TEST=y for any
> > randconfig build? That would address this issue along with a number of similar
Hi Arnd, thanks for feedback, we have talked about this internally. We use randconfig
to cover different combination, some randconfig would have CONFIG_COMPILE_TEST=y, some
is not. For now, we also want to check the kernel build status when COMPILE_TEST is
not set (as if this is a valid kconfig combination). Thus we may not always enforcing
the y here, but keep it as random as possible.
> > ones. I have sent a number of patches to add COMPILE_TEST dependencies
> > for things that otherwise break in randconfig builds but could still be useful
> > in more specific cases when users know better.
> >
> > Arnd
>
> Hi Arnd,
>
> Thanks for the advice, we'll consider it in the near future
> after assessing the existing configures.
>
> Best Regards,
> Rong Chen
> _______________________________________________
> kbuild-all mailing list -- kbuild-all(a)lists.01.org
> To unsubscribe send an email to kbuild-all-leave(a)lists.01.org
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [kbuild-all] Re: [linux-stable-rc:linux-5.10.y 3189/5262] drivers/usb/dwc2/hcd.c:3835:6: warning: stack frame size of 1032 bytes in function 'dwc2_hcd_dump_state'
2021-05-21 0:57 ` Philip Li
@ 2021-05-21 8:09 ` Arnd Bergmann
-1 siblings, 0 replies; 14+ messages in thread
From: Arnd Bergmann @ 2021-05-21 8:09 UTC (permalink / raw)
To: Philip Li
Cc: Rong Chen, kernel test robot, kbuild-all, clang-built-linux,
Greg Kroah-Hartman, Nathan Chancellor, Kees Cook, Andrew Morton,
Linux Memory Management List, Masahiro Yamada
On Fri, May 21, 2021 at 3:00 AM Philip Li <philip.li@intel.com> wrote:
> On Thu, May 20, 2021 at 05:56:35PM +0800, Rong Chen wrote:
> > On 5/20/21 5:27 PM, Arnd Bergmann wrote:
> > > On Thu, May 20, 2021 at 11:22 AM Rong Chen <rong.a.chen@intel.com> wrote:
> > > >
> > > > We have disabled CONFIG_UBSAN_ALIGNMENT for mips/clang in our kconfigs
> > > > to avoid reporting such warnings again.
> > > Thanks a lot, that helps.
> > >
> > > Have you considered always enforcing CONFIG_COMPILE_TEST=y for any
> > > randconfig build? That would address this issue along with a number of similar
>
> Hi Arnd, thanks for feedback, we have talked about this internally. We use randconfig
> to cover different combination, some randconfig would have CONFIG_COMPILE_TEST=y, some
> is not. For now, we also want to check the kernel build status when COMPILE_TEST is
> not set (as if this is a valid kconfig combination). Thus we may not always enforcing
> the y here, but keep it as random as possible.
Ok, I see.
(Adding Masahiro Yamada)
I think there was a patch in the past would let Kconfig detect whether
a kernel is built using
'make randconfig', and add further dependencies based on this, to
detect combinations
that we want to avoid testing in such environments. Maybe we should
revisit that and
replace the 'depends on !COMPILE_TEST' checks.
Arnd
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [linux-stable-rc:linux-5.10.y 3189/5262] drivers/usb/dwc2/hcd.c:3835:6: warning: stack frame size of 1032 bytes in function 'dwc2_hcd_dump_state'
@ 2021-05-21 8:09 ` Arnd Bergmann
0 siblings, 0 replies; 14+ messages in thread
From: Arnd Bergmann @ 2021-05-21 8:09 UTC (permalink / raw)
To: kbuild-all
[-- Attachment #1: Type: text/plain, Size: 1385 bytes --]
On Fri, May 21, 2021 at 3:00 AM Philip Li <philip.li@intel.com> wrote:
> On Thu, May 20, 2021 at 05:56:35PM +0800, Rong Chen wrote:
> > On 5/20/21 5:27 PM, Arnd Bergmann wrote:
> > > On Thu, May 20, 2021 at 11:22 AM Rong Chen <rong.a.chen@intel.com> wrote:
> > > >
> > > > We have disabled CONFIG_UBSAN_ALIGNMENT for mips/clang in our kconfigs
> > > > to avoid reporting such warnings again.
> > > Thanks a lot, that helps.
> > >
> > > Have you considered always enforcing CONFIG_COMPILE_TEST=y for any
> > > randconfig build? That would address this issue along with a number of similar
>
> Hi Arnd, thanks for feedback, we have talked about this internally. We use randconfig
> to cover different combination, some randconfig would have CONFIG_COMPILE_TEST=y, some
> is not. For now, we also want to check the kernel build status when COMPILE_TEST is
> not set (as if this is a valid kconfig combination). Thus we may not always enforcing
> the y here, but keep it as random as possible.
Ok, I see.
(Adding Masahiro Yamada)
I think there was a patch in the past would let Kconfig detect whether
a kernel is built using
'make randconfig', and add further dependencies based on this, to
detect combinations
that we want to avoid testing in such environments. Maybe we should
revisit that and
replace the 'depends on !COMPILE_TEST' checks.
Arnd
^ permalink raw reply [flat|nested] 14+ messages in thread
end of thread, other threads:[~2021-05-21 8:11 UTC | newest]
Thread overview: 14+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-05-19 19:48 [linux-stable-rc:linux-5.10.y 3189/5262] drivers/usb/dwc2/hcd.c:3835:6: warning: stack frame size of 1032 bytes in function 'dwc2_hcd_dump_state' kernel test robot
2021-05-19 19:48 ` kernel test robot
2021-05-19 20:28 ` Arnd Bergmann
2021-05-19 20:28 ` Arnd Bergmann
2021-05-20 9:20 ` Rong Chen
2021-05-20 9:20 ` Rong Chen
2021-05-20 9:27 ` Arnd Bergmann
2021-05-20 9:27 ` Arnd Bergmann
2021-05-20 9:56 ` [kbuild-all] " Rong Chen
2021-05-20 9:56 ` Rong Chen
2021-05-21 0:57 ` [kbuild-all] " Philip Li
2021-05-21 0:57 ` Philip Li
2021-05-21 8:09 ` [kbuild-all] " Arnd Bergmann
2021-05-21 8:09 ` Arnd Bergmann
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.