* drivers/gpio/gpio-dwapb.c:678:12: warning: stack frame size of 2064 bytes in function 'dwapb_gpio_probe'
@ 2021-05-02 22:54 kernel test robot
2021-05-03 8:39 ` Andy Shevchenko
0 siblings, 1 reply; 4+ messages in thread
From: kernel test robot @ 2021-05-02 22:54 UTC (permalink / raw)
To: Luo Jiaxing
Cc: kbuild-all, clang-built-linux, linux-kernel, Bartosz Golaszewski,
Andy Shevchenko
[-- Attachment #1: Type: text/plain, Size: 6699 bytes --]
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: 27787ba3fa4904422b3928b898d1bd3d74d98bea
commit: 60593df667e087b009ee0fc20d92e9c4c096a9b5 gpio: dwapb: fix NULL pointer dereference at dwapb_gpio_suspend()
date: 5 months ago
config: powerpc64-randconfig-r026-20210430 (attached as .config)
compiler: clang version 13.0.0 (https://github.com/llvm/llvm-project 8f5a2a5836cc8e4c1def2bdeb022e7b496623439)
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 powerpc64 cross compiling tool for clang build
# apt-get install binutils-powerpc64-linux-gnu
# https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=60593df667e087b009ee0fc20d92e9c4c096a9b5
git remote add linus https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
git fetch --no-tags linus master
git checkout 60593df667e087b009ee0fc20d92e9c4c096a9b5
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 ARCH=powerpc64
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/gpio/gpio-dwapb.c:678:12: warning: stack frame size of 2064 bytes in function 'dwapb_gpio_probe' [-Wframe-larger-than=]
static int dwapb_gpio_probe(struct platform_device *pdev)
^
1 warning generated.
vim +/dwapb_gpio_probe +678 drivers/gpio/gpio-dwapb.c
a72b8c4a63e2a3 Hoan Tran 2017-02-21 677
7779b34556978f Jamie Iles 2014-02-25 @678 static int dwapb_gpio_probe(struct platform_device *pdev)
7779b34556978f Jamie Iles 2014-02-25 679 {
3d2613c4289ff2 Weike Chen 2014-09-17 680 unsigned int i;
7779b34556978f Jamie Iles 2014-02-25 681 struct dwapb_gpio *gpio;
7779b34556978f Jamie Iles 2014-02-25 682 int err;
3d2613c4289ff2 Weike Chen 2014-09-17 683 struct device *dev = &pdev->dev;
3d2613c4289ff2 Weike Chen 2014-09-17 684 struct dwapb_platform_data *pdata = dev_get_platdata(dev);
3d2613c4289ff2 Weike Chen 2014-09-17 685
da9df93e9e0fd1 Axel Lin 2014-12-28 686 if (!pdata) {
4ba8cfa79f44a9 Jiang Qiu 2016-04-28 687 pdata = dwapb_gpio_get_pdata(dev);
3d2613c4289ff2 Weike Chen 2014-09-17 688 if (IS_ERR(pdata))
3d2613c4289ff2 Weike Chen 2014-09-17 689 return PTR_ERR(pdata);
3d2613c4289ff2 Weike Chen 2014-09-17 690 }
7779b34556978f Jamie Iles 2014-02-25 691
da9df93e9e0fd1 Axel Lin 2014-12-28 692 if (!pdata->nports)
da9df93e9e0fd1 Axel Lin 2014-12-28 693 return -ENODEV;
7779b34556978f Jamie Iles 2014-02-25 694
3d2613c4289ff2 Weike Chen 2014-09-17 695 gpio = devm_kzalloc(&pdev->dev, sizeof(*gpio), GFP_KERNEL);
da9df93e9e0fd1 Axel Lin 2014-12-28 696 if (!gpio)
da9df93e9e0fd1 Axel Lin 2014-12-28 697 return -ENOMEM;
da9df93e9e0fd1 Axel Lin 2014-12-28 698
3d2613c4289ff2 Weike Chen 2014-09-17 699 gpio->dev = &pdev->dev;
3d2613c4289ff2 Weike Chen 2014-09-17 700 gpio->nr_ports = pdata->nports;
3d2613c4289ff2 Weike Chen 2014-09-17 701
4731d80f5ea9b8 Serge Semin 2020-07-30 702 err = dwapb_get_reset(gpio);
4731d80f5ea9b8 Serge Semin 2020-07-30 703 if (err)
4731d80f5ea9b8 Serge Semin 2020-07-30 704 return err;
07901a94f9f9b1 Alan Tull 2017-10-11 705
3d2613c4289ff2 Weike Chen 2014-09-17 706 gpio->ports = devm_kcalloc(&pdev->dev, gpio->nr_ports,
7779b34556978f Jamie Iles 2014-02-25 707 sizeof(*gpio->ports), GFP_KERNEL);
da9df93e9e0fd1 Axel Lin 2014-12-28 708 if (!gpio->ports)
da9df93e9e0fd1 Axel Lin 2014-12-28 709 return -ENOMEM;
7779b34556978f Jamie Iles 2014-02-25 710
2a7194e9759586 Enrico Weigelt, metux IT consult 2019-03-11 711 gpio->regs = devm_platform_ioremap_resource(pdev, 0);
da9df93e9e0fd1 Axel Lin 2014-12-28 712 if (IS_ERR(gpio->regs))
da9df93e9e0fd1 Axel Lin 2014-12-28 713 return PTR_ERR(gpio->regs);
7779b34556978f Jamie Iles 2014-02-25 714
daa3f58d180c07 Serge Semin 2020-07-30 715 err = dwapb_get_clks(gpio);
daa3f58d180c07 Serge Semin 2020-07-30 716 if (err)
e6bf37736f6495 Phil Edworthy 2018-03-12 717 return err;
e6bf37736f6495 Phil Edworthy 2018-03-12 718
9826bbe1fef0b5 Andy Shevchenko 2020-04-15 719 gpio->flags = (uintptr_t)device_get_match_data(dev);
a72b8c4a63e2a3 Hoan Tran 2017-02-21 720
3d2613c4289ff2 Weike Chen 2014-09-17 721 for (i = 0; i < gpio->nr_ports; i++) {
3d2613c4289ff2 Weike Chen 2014-09-17 722 err = dwapb_gpio_add_port(gpio, &pdata->properties[i], i);
7779b34556978f Jamie Iles 2014-02-25 723 if (err)
7779b34556978f Jamie Iles 2014-02-25 724 return err;
7779b34556978f Jamie Iles 2014-02-25 725 }
7779b34556978f Jamie Iles 2014-02-25 726
60593df667e087 Luo Jiaxing 2020-11-27 727 platform_set_drvdata(pdev, gpio);
60593df667e087 Luo Jiaxing 2020-11-27 728
7779b34556978f Jamie Iles 2014-02-25 729 return 0;
7779b34556978f Jamie Iles 2014-02-25 730 }
7779b34556978f Jamie Iles 2014-02-25 731
:::::: The code at line 678 was first introduced by commit
:::::: 7779b34556978f6771a78bd87557523623e0405b gpio: add a driver for the Synopsys DesignWare APB GPIO block
:::::: TO: Jamie Iles <jamie@jamieiles.com>
:::::: CC: Linus Walleij <linus.walleij@linaro.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: 28842 bytes --]
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: drivers/gpio/gpio-dwapb.c:678:12: warning: stack frame size of 2064 bytes in function 'dwapb_gpio_probe'
2021-05-02 22:54 drivers/gpio/gpio-dwapb.c:678:12: warning: stack frame size of 2064 bytes in function 'dwapb_gpio_probe' kernel test robot
@ 2021-05-03 8:39 ` Andy Shevchenko
2021-05-03 20:49 ` Nick Desaulniers
0 siblings, 1 reply; 4+ messages in thread
From: Andy Shevchenko @ 2021-05-03 8:39 UTC (permalink / raw)
To: kernel test robot, Nick Desaulniers
Cc: Luo Jiaxing, kbuild-all, clang-built-linux,
Linux Kernel Mailing List, Bartosz Golaszewski
On Mon, May 3, 2021 at 1:54 AM kernel test robot <lkp@intel.com> wrote:
>
> tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
> head: 27787ba3fa4904422b3928b898d1bd3d74d98bea
> commit: 60593df667e087b009ee0fc20d92e9c4c096a9b5 gpio: dwapb: fix NULL pointer dereference at dwapb_gpio_suspend()
> date: 5 months ago
> config: powerpc64-randconfig-r026-20210430 (attached as .config)
> compiler: clang version 13.0.0 (https://github.com/llvm/llvm-project 8f5a2a5836cc8e4c1def2bdeb022e7b496623439)
> 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 powerpc64 cross compiling tool for clang build
> # apt-get install binutils-powerpc64-linux-gnu
> # https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=60593df667e087b009ee0fc20d92e9c4c096a9b5
> git remote add linus https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
> git fetch --no-tags linus master
> git checkout 60593df667e087b009ee0fc20d92e9c4c096a9b5
> # save the attached .config to linux build tree
> COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 ARCH=powerpc64
>
> 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/gpio/gpio-dwapb.c:678:12: warning: stack frame size of 2064 bytes in function 'dwapb_gpio_probe' [-Wframe-larger-than=]
> static int dwapb_gpio_probe(struct platform_device *pdev)
> ^
> 1 warning generated.
Another Clang bug?
> vim +/dwapb_gpio_probe +678 drivers/gpio/gpio-dwapb.c
>
> a72b8c4a63e2a3 Hoan Tran 2017-02-21 677
> 7779b34556978f Jamie Iles 2014-02-25 @678 static int dwapb_gpio_probe(struct platform_device *pdev)
> 7779b34556978f Jamie Iles 2014-02-25 679 {
> 3d2613c4289ff2 Weike Chen 2014-09-17 680 unsigned int i;
> 7779b34556978f Jamie Iles 2014-02-25 681 struct dwapb_gpio *gpio;
> 7779b34556978f Jamie Iles 2014-02-25 682 int err;
> 3d2613c4289ff2 Weike Chen 2014-09-17 683 struct device *dev = &pdev->dev;
> 3d2613c4289ff2 Weike Chen 2014-09-17 684 struct dwapb_platform_data *pdata = dev_get_platdata(dev);
> 3d2613c4289ff2 Weike Chen 2014-09-17 685
> da9df93e9e0fd1 Axel Lin 2014-12-28 686 if (!pdata) {
> 4ba8cfa79f44a9 Jiang Qiu 2016-04-28 687 pdata = dwapb_gpio_get_pdata(dev);
> 3d2613c4289ff2 Weike Chen 2014-09-17 688 if (IS_ERR(pdata))
> 3d2613c4289ff2 Weike Chen 2014-09-17 689 return PTR_ERR(pdata);
> 3d2613c4289ff2 Weike Chen 2014-09-17 690 }
> 7779b34556978f Jamie Iles 2014-02-25 691
> da9df93e9e0fd1 Axel Lin 2014-12-28 692 if (!pdata->nports)
> da9df93e9e0fd1 Axel Lin 2014-12-28 693 return -ENODEV;
> 7779b34556978f Jamie Iles 2014-02-25 694
> 3d2613c4289ff2 Weike Chen 2014-09-17 695 gpio = devm_kzalloc(&pdev->dev, sizeof(*gpio), GFP_KERNEL);
> da9df93e9e0fd1 Axel Lin 2014-12-28 696 if (!gpio)
> da9df93e9e0fd1 Axel Lin 2014-12-28 697 return -ENOMEM;
> da9df93e9e0fd1 Axel Lin 2014-12-28 698
> 3d2613c4289ff2 Weike Chen 2014-09-17 699 gpio->dev = &pdev->dev;
> 3d2613c4289ff2 Weike Chen 2014-09-17 700 gpio->nr_ports = pdata->nports;
> 3d2613c4289ff2 Weike Chen 2014-09-17 701
> 4731d80f5ea9b8 Serge Semin 2020-07-30 702 err = dwapb_get_reset(gpio);
> 4731d80f5ea9b8 Serge Semin 2020-07-30 703 if (err)
> 4731d80f5ea9b8 Serge Semin 2020-07-30 704 return err;
> 07901a94f9f9b1 Alan Tull 2017-10-11 705
> 3d2613c4289ff2 Weike Chen 2014-09-17 706 gpio->ports = devm_kcalloc(&pdev->dev, gpio->nr_ports,
> 7779b34556978f Jamie Iles 2014-02-25 707 sizeof(*gpio->ports), GFP_KERNEL);
> da9df93e9e0fd1 Axel Lin 2014-12-28 708 if (!gpio->ports)
> da9df93e9e0fd1 Axel Lin 2014-12-28 709 return -ENOMEM;
> 7779b34556978f Jamie Iles 2014-02-25 710
> 2a7194e9759586 Enrico Weigelt, metux IT consult 2019-03-11 711 gpio->regs = devm_platform_ioremap_resource(pdev, 0);
> da9df93e9e0fd1 Axel Lin 2014-12-28 712 if (IS_ERR(gpio->regs))
> da9df93e9e0fd1 Axel Lin 2014-12-28 713 return PTR_ERR(gpio->regs);
> 7779b34556978f Jamie Iles 2014-02-25 714
> daa3f58d180c07 Serge Semin 2020-07-30 715 err = dwapb_get_clks(gpio);
> daa3f58d180c07 Serge Semin 2020-07-30 716 if (err)
> e6bf37736f6495 Phil Edworthy 2018-03-12 717 return err;
> e6bf37736f6495 Phil Edworthy 2018-03-12 718
> 9826bbe1fef0b5 Andy Shevchenko 2020-04-15 719 gpio->flags = (uintptr_t)device_get_match_data(dev);
> a72b8c4a63e2a3 Hoan Tran 2017-02-21 720
> 3d2613c4289ff2 Weike Chen 2014-09-17 721 for (i = 0; i < gpio->nr_ports; i++) {
> 3d2613c4289ff2 Weike Chen 2014-09-17 722 err = dwapb_gpio_add_port(gpio, &pdata->properties[i], i);
> 7779b34556978f Jamie Iles 2014-02-25 723 if (err)
> 7779b34556978f Jamie Iles 2014-02-25 724 return err;
> 7779b34556978f Jamie Iles 2014-02-25 725 }
> 7779b34556978f Jamie Iles 2014-02-25 726
> 60593df667e087 Luo Jiaxing 2020-11-27 727 platform_set_drvdata(pdev, gpio);
> 60593df667e087 Luo Jiaxing 2020-11-27 728
> 7779b34556978f Jamie Iles 2014-02-25 729 return 0;
> 7779b34556978f Jamie Iles 2014-02-25 730 }
> 7779b34556978f Jamie Iles 2014-02-25 731
>
> :::::: The code at line 678 was first introduced by commit
> :::::: 7779b34556978f6771a78bd87557523623e0405b gpio: add a driver for the Synopsys DesignWare APB GPIO block
>
> :::::: TO: Jamie Iles <jamie@jamieiles.com>
> :::::: CC: Linus Walleij <linus.walleij@linaro.org>
>
> ---
> 0-DAY CI Kernel Test Service, Intel Corporation
> https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
--
With Best Regards,
Andy Shevchenko
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: drivers/gpio/gpio-dwapb.c:678:12: warning: stack frame size of 2064 bytes in function 'dwapb_gpio_probe'
2021-05-03 8:39 ` Andy Shevchenko
@ 2021-05-03 20:49 ` Nick Desaulniers
2021-05-03 23:39 ` Nick Desaulniers
0 siblings, 1 reply; 4+ messages in thread
From: Nick Desaulniers @ 2021-05-03 20:49 UTC (permalink / raw)
To: Andy Shevchenko
Cc: kernel test robot, Luo Jiaxing, kbuild-all, clang-built-linux,
Linux Kernel Mailing List, Bartosz Golaszewski
On Mon, May 3, 2021 at 1:40 AM Andy Shevchenko
<andy.shevchenko@gmail.com> wrote:
>
> On Mon, May 3, 2021 at 1:54 AM kernel test robot <lkp@intel.com> wrote:
> >
> > tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
> > head: 27787ba3fa4904422b3928b898d1bd3d74d98bea
> > commit: 60593df667e087b009ee0fc20d92e9c4c096a9b5 gpio: dwapb: fix NULL pointer dereference at dwapb_gpio_suspend()
> > date: 5 months ago
> > config: powerpc64-randconfig-r026-20210430 (attached as .config)
> > compiler: clang version 13.0.0 (https://github.com/llvm/llvm-project 8f5a2a5836cc8e4c1def2bdeb022e7b496623439)
> > 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 powerpc64 cross compiling tool for clang build
> > # apt-get install binutils-powerpc64-linux-gnu
> > # https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=60593df667e087b009ee0fc20d92e9c4c096a9b5
> > git remote add linus https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
> > git fetch --no-tags linus master
> > git checkout 60593df667e087b009ee0fc20d92e9c4c096a9b5
> > # save the attached .config to linux build tree
> > COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 ARCH=powerpc64
> >
> > 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/gpio/gpio-dwapb.c:678:12: warning: stack frame size of 2064 bytes in function 'dwapb_gpio_probe' [-Wframe-larger-than=]
> > static int dwapb_gpio_probe(struct platform_device *pdev)
> > ^
> > 1 warning generated.
>
> Another Clang bug?
We have a KI with ASAN's requirements on additional stack usage for
shadow stacks not being considered when making decisions whether or
not to inline. Looks like KASAN isn't enabled by this randconfig;
UBSAN is. I don't know if UBSAN has similar additional stack usage to
KASAN.
I can reproduce this warning on mainline with:
$ wget https://groups.google.com/group/clang-built-linux/attach/983d2ad77c1d/.config.gz\?part\=0.1
-O - | gunzip > .config
$ ARCH=powerpc CROSS_COMPILE=powerpc64le-linux-gnu- make CC=clang -j72
drivers/gpio/gpio-dwapb.o
If I enable CONFIG_DEBUG_INFO, then rebuild, I have a utility for
helping track these down from parsing the DWARF debug info.
Unfortunately, it depends on pyelftools which doesn't know any ppc64
relocations; let me see if I can implement that real quick, which
should help debug this futher.
>
> > vim +/dwapb_gpio_probe +678 drivers/gpio/gpio-dwapb.c
> >
> > a72b8c4a63e2a3 Hoan Tran 2017-02-21 677
> > 7779b34556978f Jamie Iles 2014-02-25 @678 static int dwapb_gpio_probe(struct platform_device *pdev)
> > 7779b34556978f Jamie Iles 2014-02-25 679 {
> > 3d2613c4289ff2 Weike Chen 2014-09-17 680 unsigned int i;
> > 7779b34556978f Jamie Iles 2014-02-25 681 struct dwapb_gpio *gpio;
> > 7779b34556978f Jamie Iles 2014-02-25 682 int err;
> > 3d2613c4289ff2 Weike Chen 2014-09-17 683 struct device *dev = &pdev->dev;
> > 3d2613c4289ff2 Weike Chen 2014-09-17 684 struct dwapb_platform_data *pdata = dev_get_platdata(dev);
> > 3d2613c4289ff2 Weike Chen 2014-09-17 685
> > da9df93e9e0fd1 Axel Lin 2014-12-28 686 if (!pdata) {
> > 4ba8cfa79f44a9 Jiang Qiu 2016-04-28 687 pdata = dwapb_gpio_get_pdata(dev);
> > 3d2613c4289ff2 Weike Chen 2014-09-17 688 if (IS_ERR(pdata))
> > 3d2613c4289ff2 Weike Chen 2014-09-17 689 return PTR_ERR(pdata);
> > 3d2613c4289ff2 Weike Chen 2014-09-17 690 }
> > 7779b34556978f Jamie Iles 2014-02-25 691
> > da9df93e9e0fd1 Axel Lin 2014-12-28 692 if (!pdata->nports)
> > da9df93e9e0fd1 Axel Lin 2014-12-28 693 return -ENODEV;
> > 7779b34556978f Jamie Iles 2014-02-25 694
> > 3d2613c4289ff2 Weike Chen 2014-09-17 695 gpio = devm_kzalloc(&pdev->dev, sizeof(*gpio), GFP_KERNEL);
> > da9df93e9e0fd1 Axel Lin 2014-12-28 696 if (!gpio)
> > da9df93e9e0fd1 Axel Lin 2014-12-28 697 return -ENOMEM;
> > da9df93e9e0fd1 Axel Lin 2014-12-28 698
> > 3d2613c4289ff2 Weike Chen 2014-09-17 699 gpio->dev = &pdev->dev;
> > 3d2613c4289ff2 Weike Chen 2014-09-17 700 gpio->nr_ports = pdata->nports;
> > 3d2613c4289ff2 Weike Chen 2014-09-17 701
> > 4731d80f5ea9b8 Serge Semin 2020-07-30 702 err = dwapb_get_reset(gpio);
> > 4731d80f5ea9b8 Serge Semin 2020-07-30 703 if (err)
> > 4731d80f5ea9b8 Serge Semin 2020-07-30 704 return err;
> > 07901a94f9f9b1 Alan Tull 2017-10-11 705
> > 3d2613c4289ff2 Weike Chen 2014-09-17 706 gpio->ports = devm_kcalloc(&pdev->dev, gpio->nr_ports,
> > 7779b34556978f Jamie Iles 2014-02-25 707 sizeof(*gpio->ports), GFP_KERNEL);
> > da9df93e9e0fd1 Axel Lin 2014-12-28 708 if (!gpio->ports)
> > da9df93e9e0fd1 Axel Lin 2014-12-28 709 return -ENOMEM;
> > 7779b34556978f Jamie Iles 2014-02-25 710
> > 2a7194e9759586 Enrico Weigelt, metux IT consult 2019-03-11 711 gpio->regs = devm_platform_ioremap_resource(pdev, 0);
> > da9df93e9e0fd1 Axel Lin 2014-12-28 712 if (IS_ERR(gpio->regs))
> > da9df93e9e0fd1 Axel Lin 2014-12-28 713 return PTR_ERR(gpio->regs);
> > 7779b34556978f Jamie Iles 2014-02-25 714
> > daa3f58d180c07 Serge Semin 2020-07-30 715 err = dwapb_get_clks(gpio);
> > daa3f58d180c07 Serge Semin 2020-07-30 716 if (err)
> > e6bf37736f6495 Phil Edworthy 2018-03-12 717 return err;
> > e6bf37736f6495 Phil Edworthy 2018-03-12 718
> > 9826bbe1fef0b5 Andy Shevchenko 2020-04-15 719 gpio->flags = (uintptr_t)device_get_match_data(dev);
> > a72b8c4a63e2a3 Hoan Tran 2017-02-21 720
> > 3d2613c4289ff2 Weike Chen 2014-09-17 721 for (i = 0; i < gpio->nr_ports; i++) {
> > 3d2613c4289ff2 Weike Chen 2014-09-17 722 err = dwapb_gpio_add_port(gpio, &pdata->properties[i], i);
> > 7779b34556978f Jamie Iles 2014-02-25 723 if (err)
> > 7779b34556978f Jamie Iles 2014-02-25 724 return err;
> > 7779b34556978f Jamie Iles 2014-02-25 725 }
> > 7779b34556978f Jamie Iles 2014-02-25 726
> > 60593df667e087 Luo Jiaxing 2020-11-27 727 platform_set_drvdata(pdev, gpio);
> > 60593df667e087 Luo Jiaxing 2020-11-27 728
> > 7779b34556978f Jamie Iles 2014-02-25 729 return 0;
> > 7779b34556978f Jamie Iles 2014-02-25 730 }
> > 7779b34556978f Jamie Iles 2014-02-25 731
> >
> > :::::: The code at line 678 was first introduced by commit
> > :::::: 7779b34556978f6771a78bd87557523623e0405b gpio: add a driver for the Synopsys DesignWare APB GPIO block
> >
> > :::::: TO: Jamie Iles <jamie@jamieiles.com>
> > :::::: CC: Linus Walleij <linus.walleij@linaro.org>
> >
> > ---
> > 0-DAY CI Kernel Test Service, Intel Corporation
> > https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
>
>
>
> --
> With Best Regards,
> Andy Shevchenko
>
> --
> You received this message because you are subscribed to the Google Groups "Clang Built Linux" group.
> To unsubscribe from this group and stop receiving emails from it, send an email to clang-built-linux+unsubscribe@googlegroups.com.
> To view this discussion on the web visit https://groups.google.com/d/msgid/clang-built-linux/CAHp75Vd9kXo7%3Dg35x_AosWBHJH0KBVeX4qgkpbTz31m3w3s3gA%40mail.gmail.com.
--
Thanks,
~Nick Desaulniers
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: drivers/gpio/gpio-dwapb.c:678:12: warning: stack frame size of 2064 bytes in function 'dwapb_gpio_probe'
2021-05-03 20:49 ` Nick Desaulniers
@ 2021-05-03 23:39 ` Nick Desaulniers
0 siblings, 0 replies; 4+ messages in thread
From: Nick Desaulniers @ 2021-05-03 23:39 UTC (permalink / raw)
To: Andy Shevchenko
Cc: kernel test robot, Luo Jiaxing, kbuild-all, clang-built-linux,
Linux Kernel Mailing List, Bartosz Golaszewski
On Mon, May 3, 2021 at 1:49 PM Nick Desaulniers <ndesaulniers@google.com> wrote:
>
> On Mon, May 3, 2021 at 1:40 AM Andy Shevchenko
> <andy.shevchenko@gmail.com> wrote:
> >
> > On Mon, May 3, 2021 at 1:54 AM kernel test robot <lkp@intel.com> wrote:
> > >
> > > tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
> > > head: 27787ba3fa4904422b3928b898d1bd3d74d98bea
> > > commit: 60593df667e087b009ee0fc20d92e9c4c096a9b5 gpio: dwapb: fix NULL pointer dereference at dwapb_gpio_suspend()
> > > date: 5 months ago
> > > config: powerpc64-randconfig-r026-20210430 (attached as .config)
> > > compiler: clang version 13.0.0 (https://github.com/llvm/llvm-project 8f5a2a5836cc8e4c1def2bdeb022e7b496623439)
> > > 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 powerpc64 cross compiling tool for clang build
> > > # apt-get install binutils-powerpc64-linux-gnu
> > > # https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=60593df667e087b009ee0fc20d92e9c4c096a9b5
> > > git remote add linus https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
> > > git fetch --no-tags linus master
> > > git checkout 60593df667e087b009ee0fc20d92e9c4c096a9b5
> > > # save the attached .config to linux build tree
> > > COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 ARCH=powerpc64
> > >
> > > 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/gpio/gpio-dwapb.c:678:12: warning: stack frame size of 2064 bytes in function 'dwapb_gpio_probe' [-Wframe-larger-than=]
> > > static int dwapb_gpio_probe(struct platform_device *pdev)
> > > ^
> > > 1 warning generated.
> >
> > Another Clang bug?
>
> We have a KI with ASAN's requirements on additional stack usage for
> shadow stacks not being considered when making decisions whether or
> not to inline. Looks like KASAN isn't enabled by this randconfig;
> UBSAN is. I don't know if UBSAN has similar additional stack usage to
> KASAN.
>
> I can reproduce this warning on mainline with:
> $ wget https://groups.google.com/group/clang-built-linux/attach/983d2ad77c1d/.config.gz\?part\=0.1
> -O - | gunzip > .config
> $ ARCH=powerpc CROSS_COMPILE=powerpc64le-linux-gnu- make CC=clang -j72
> drivers/gpio/gpio-dwapb.o
>
> If I enable CONFIG_DEBUG_INFO, then rebuild, I have a utility for
> helping track these down from parsing the DWARF debug info.
> Unfortunately, it depends on pyelftools which doesn't know any ppc64
> relocations; let me see if I can implement that real quick, which
> should help debug this further.
https://github.com/eliben/pyelftools/pull/360
--
Thanks,
~Nick Desaulniers
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2021-05-03 23:39 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-05-02 22:54 drivers/gpio/gpio-dwapb.c:678:12: warning: stack frame size of 2064 bytes in function 'dwapb_gpio_probe' kernel test robot
2021-05-03 8:39 ` Andy Shevchenko
2021-05-03 20:49 ` Nick Desaulniers
2021-05-03 23:39 ` Nick Desaulniers
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).