All of lore.kernel.org
 help / color / mirror / Atom feed
From: Russell King - ARM Linux admin <linux@armlinux.org.uk>
To: kbuild test robot <lkp@intel.com>
Cc: kbuild-all@lists.01.org, Peng Ma <peng.ma@nxp.com>,
	linux-arm-kernel@lists.infradead.org
Subject: Re: [arm:cex7 96/112] arch/sparc/include/asm/io_64.h:179:2: warning: 'px_is' may be used uninitialized in this function
Date: Tue, 3 Mar 2020 14:46:58 +0000	[thread overview]
Message-ID: <20200303144658.GN25745@shell.armlinux.org.uk> (raw)
In-Reply-To: <202003032214.pjEWVErC%lkp@intel.com>

This looks to me like a false positive.

On Tue, Mar 03, 2020 at 10:12:28PM +0800, kbuild test robot wrote:
> tree:   git://git.armlinux.org.uk/~rmk/linux-arm.git cex7
> head:   c333ba58ac979daa84d7ea59b7d4a04c15cce2e7
> commit: d45ed9abba2dce5bb6faeb3474264ba64ef2d610 [96/112] ahci: qoriq: workaround for errata A-379364 on lx2160a
> config: sparc64-randconfig-a001-20200303 (attached as .config)
> compiler: sparc64-linux-gcc (GCC) 7.5.0
> reproduce:
>         wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
>         chmod +x ~/bin/make.cross
>         git checkout d45ed9abba2dce5bb6faeb3474264ba64ef2d610
>         # save the attached .config to linux build tree
>         GCC_VERSION=7.5.0 make.cross ARCH=sparc64 
> 
> If you fix the issue, kindly add following tag
> Reported-by: kbuild test robot <lkp@intel.com>
> 
> Note: it may well be a FALSE warning. FWIW you are at least aware of it now.
> http://gcc.gnu.org/wiki/Better_Uninitialized_Warnings
> 
> All warnings (new ones prefixed by >>):
> 
>    In file included from arch/sparc/include/asm/io.h:5:0,
>                     from include/linux/io.h:13,
>                     from include/linux/of_address.h:7,
>                     from drivers//ata/ahci_qoriq.c:14:
>    drivers//ata/ahci_qoriq.c: In function 'ahci_qoriq_hardreset':
> >> arch/sparc/include/asm/io_64.h:179:2: warning: 'px_is' may be used uninitialized in this function [-Wmaybe-uninitialized]
>      __asm__ __volatile__("stwa\t%r0, [%1] %2\t/* pci_writel */"
>      ^~~~~~~
>    drivers//ata/ahci_qoriq.c:229:14: note: 'px_is' was declared here
>      u32 px_cmd, px_is, px_val;
>                  ^~~~~
>    In file included from arch/sparc/include/asm/io.h:5:0,
>                     from include/linux/io.h:13,
>                     from include/linux/of_address.h:7,
>                     from drivers//ata/ahci_qoriq.c:14:
> >> arch/sparc/include/asm/io_64.h:179:2: warning: 'px_cmd' may be used uninitialized in this function [-Wmaybe-uninitialized]
>      __asm__ __volatile__("stwa\t%r0, [%1] %2\t/* pci_writel */"
>      ^~~~~~~
>    drivers//ata/ahci_qoriq.c:229:6: note: 'px_cmd' was declared here
>      u32 px_cmd, px_is, px_val;
>          ^~~~~~
> --
>    In file included from arch/sparc/include/asm/io.h:5:0,
>                     from include/linux/io.h:13,
>                     from include/linux/of_address.h:7,
>                     from drivers/ata/ahci_qoriq.c:14:
>    drivers/ata/ahci_qoriq.c: In function 'ahci_qoriq_hardreset':
> >> arch/sparc/include/asm/io_64.h:179:2: warning: 'px_is' may be used uninitialized in this function [-Wmaybe-uninitialized]
>      __asm__ __volatile__("stwa\t%r0, [%1] %2\t/* pci_writel */"
>      ^~~~~~~
>    drivers/ata/ahci_qoriq.c:229:14: note: 'px_is' was declared here
>      u32 px_cmd, px_is, px_val;
>                  ^~~~~
>    In file included from arch/sparc/include/asm/io.h:5:0,
>                     from include/linux/io.h:13,
>                     from include/linux/of_address.h:7,
>                     from drivers/ata/ahci_qoriq.c:14:
> >> arch/sparc/include/asm/io_64.h:179:2: warning: 'px_cmd' may be used uninitialized in this function [-Wmaybe-uninitialized]
>      __asm__ __volatile__("stwa\t%r0, [%1] %2\t/* pci_writel */"
>      ^~~~~~~
>    drivers/ata/ahci_qoriq.c:229:6: note: 'px_cmd' was declared here
>      u32 px_cmd, px_is, px_val;
>          ^~~~~~
> 
> vim +/px_is +179 arch/sparc/include/asm/io_64.h
> 
> f5e706ad886b6a include/asm-sparc/io_64.h      Sam Ravnborg  2008-07-17  174  
> 79294d7eff89f6 arch/sparc/include/asm/io_64.h Sam Ravnborg  2014-07-20  175  #define writel writel
> 7c3969c3a4f359 arch/sparc/include/asm/io_64.h Arnd Bergmann 2014-11-19  176  #define writel_relaxed writel
> 79294d7eff89f6 arch/sparc/include/asm/io_64.h Sam Ravnborg  2014-07-20  177  static inline void writel(u32 l, volatile void __iomem *addr)
> f5e706ad886b6a include/asm-sparc/io_64.h      Sam Ravnborg  2008-07-17  178  {
> f5e706ad886b6a include/asm-sparc/io_64.h      Sam Ravnborg  2008-07-17 @179  	__asm__ __volatile__("stwa\t%r0, [%1] %2\t/* pci_writel */"
> f5e706ad886b6a include/asm-sparc/io_64.h      Sam Ravnborg  2008-07-17  180  			     : /* no outputs */
> f5e706ad886b6a include/asm-sparc/io_64.h      Sam Ravnborg  2008-07-17  181  			     : "Jr" (l), "r" (addr), "i" (ASI_PHYS_BYPASS_EC_E_L)
> f5e706ad886b6a include/asm-sparc/io_64.h      Sam Ravnborg  2008-07-17  182  			     : "memory");
> f5e706ad886b6a include/asm-sparc/io_64.h      Sam Ravnborg  2008-07-17  183  }
> f5e706ad886b6a include/asm-sparc/io_64.h      Sam Ravnborg  2008-07-17  184  
> 
> :::::: The code at line 179 was first introduced by commit
> :::::: f5e706ad886b6a5eb59637830110b09ccebf01c5 sparc: join the remaining header files
> 
> :::::: TO: Sam Ravnborg <sam@ravnborg.org>
> :::::: CC: David S. Miller <davem@davemloft.net>
> 
> ---
> 0-DAY CI Kernel Test Service, Intel Corporation
> https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org



-- 
RMK's Patch system: https://www.armlinux.org.uk/developer/patches/
FTTC broadband for 0.8mile line in suburbia: sync at 12.1Mbps down 622kbps up
According to speedtest.net: 11.9Mbps down 500kbps up

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

WARNING: multiple messages have this Message-ID (diff)
From: Russell King - ARM Linux admin <linux@armlinux.org.uk>
To: kbuild-all@lists.01.org
Subject: Re: [arm:cex7 96/112] arch/sparc/include/asm/io_64.h:179:2: warning: 'px_is' may be used uninitialized in this function
Date: Tue, 03 Mar 2020 14:46:58 +0000	[thread overview]
Message-ID: <20200303144658.GN25745@shell.armlinux.org.uk> (raw)
In-Reply-To: <202003032214.pjEWVErC%lkp@intel.com>

[-- Attachment #1: Type: text/plain, Size: 5327 bytes --]

This looks to me like a false positive.

On Tue, Mar 03, 2020 at 10:12:28PM +0800, kbuild test robot wrote:
> tree:   git://git.armlinux.org.uk/~rmk/linux-arm.git cex7
> head:   c333ba58ac979daa84d7ea59b7d4a04c15cce2e7
> commit: d45ed9abba2dce5bb6faeb3474264ba64ef2d610 [96/112] ahci: qoriq: workaround for errata A-379364 on lx2160a
> config: sparc64-randconfig-a001-20200303 (attached as .config)
> compiler: sparc64-linux-gcc (GCC) 7.5.0
> reproduce:
>         wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
>         chmod +x ~/bin/make.cross
>         git checkout d45ed9abba2dce5bb6faeb3474264ba64ef2d610
>         # save the attached .config to linux build tree
>         GCC_VERSION=7.5.0 make.cross ARCH=sparc64 
> 
> If you fix the issue, kindly add following tag
> Reported-by: kbuild test robot <lkp@intel.com>
> 
> Note: it may well be a FALSE warning. FWIW you are at least aware of it now.
> http://gcc.gnu.org/wiki/Better_Uninitialized_Warnings
> 
> All warnings (new ones prefixed by >>):
> 
>    In file included from arch/sparc/include/asm/io.h:5:0,
>                     from include/linux/io.h:13,
>                     from include/linux/of_address.h:7,
>                     from drivers//ata/ahci_qoriq.c:14:
>    drivers//ata/ahci_qoriq.c: In function 'ahci_qoriq_hardreset':
> >> arch/sparc/include/asm/io_64.h:179:2: warning: 'px_is' may be used uninitialized in this function [-Wmaybe-uninitialized]
>      __asm__ __volatile__("stwa\t%r0, [%1] %2\t/* pci_writel */"
>      ^~~~~~~
>    drivers//ata/ahci_qoriq.c:229:14: note: 'px_is' was declared here
>      u32 px_cmd, px_is, px_val;
>                  ^~~~~
>    In file included from arch/sparc/include/asm/io.h:5:0,
>                     from include/linux/io.h:13,
>                     from include/linux/of_address.h:7,
>                     from drivers//ata/ahci_qoriq.c:14:
> >> arch/sparc/include/asm/io_64.h:179:2: warning: 'px_cmd' may be used uninitialized in this function [-Wmaybe-uninitialized]
>      __asm__ __volatile__("stwa\t%r0, [%1] %2\t/* pci_writel */"
>      ^~~~~~~
>    drivers//ata/ahci_qoriq.c:229:6: note: 'px_cmd' was declared here
>      u32 px_cmd, px_is, px_val;
>          ^~~~~~
> --
>    In file included from arch/sparc/include/asm/io.h:5:0,
>                     from include/linux/io.h:13,
>                     from include/linux/of_address.h:7,
>                     from drivers/ata/ahci_qoriq.c:14:
>    drivers/ata/ahci_qoriq.c: In function 'ahci_qoriq_hardreset':
> >> arch/sparc/include/asm/io_64.h:179:2: warning: 'px_is' may be used uninitialized in this function [-Wmaybe-uninitialized]
>      __asm__ __volatile__("stwa\t%r0, [%1] %2\t/* pci_writel */"
>      ^~~~~~~
>    drivers/ata/ahci_qoriq.c:229:14: note: 'px_is' was declared here
>      u32 px_cmd, px_is, px_val;
>                  ^~~~~
>    In file included from arch/sparc/include/asm/io.h:5:0,
>                     from include/linux/io.h:13,
>                     from include/linux/of_address.h:7,
>                     from drivers/ata/ahci_qoriq.c:14:
> >> arch/sparc/include/asm/io_64.h:179:2: warning: 'px_cmd' may be used uninitialized in this function [-Wmaybe-uninitialized]
>      __asm__ __volatile__("stwa\t%r0, [%1] %2\t/* pci_writel */"
>      ^~~~~~~
>    drivers/ata/ahci_qoriq.c:229:6: note: 'px_cmd' was declared here
>      u32 px_cmd, px_is, px_val;
>          ^~~~~~
> 
> vim +/px_is +179 arch/sparc/include/asm/io_64.h
> 
> f5e706ad886b6a include/asm-sparc/io_64.h      Sam Ravnborg  2008-07-17  174  
> 79294d7eff89f6 arch/sparc/include/asm/io_64.h Sam Ravnborg  2014-07-20  175  #define writel writel
> 7c3969c3a4f359 arch/sparc/include/asm/io_64.h Arnd Bergmann 2014-11-19  176  #define writel_relaxed writel
> 79294d7eff89f6 arch/sparc/include/asm/io_64.h Sam Ravnborg  2014-07-20  177  static inline void writel(u32 l, volatile void __iomem *addr)
> f5e706ad886b6a include/asm-sparc/io_64.h      Sam Ravnborg  2008-07-17  178  {
> f5e706ad886b6a include/asm-sparc/io_64.h      Sam Ravnborg  2008-07-17 @179  	__asm__ __volatile__("stwa\t%r0, [%1] %2\t/* pci_writel */"
> f5e706ad886b6a include/asm-sparc/io_64.h      Sam Ravnborg  2008-07-17  180  			     : /* no outputs */
> f5e706ad886b6a include/asm-sparc/io_64.h      Sam Ravnborg  2008-07-17  181  			     : "Jr" (l), "r" (addr), "i" (ASI_PHYS_BYPASS_EC_E_L)
> f5e706ad886b6a include/asm-sparc/io_64.h      Sam Ravnborg  2008-07-17  182  			     : "memory");
> f5e706ad886b6a include/asm-sparc/io_64.h      Sam Ravnborg  2008-07-17  183  }
> f5e706ad886b6a include/asm-sparc/io_64.h      Sam Ravnborg  2008-07-17  184  
> 
> :::::: The code at line 179 was first introduced by commit
> :::::: f5e706ad886b6a5eb59637830110b09ccebf01c5 sparc: join the remaining header files
> 
> :::::: TO: Sam Ravnborg <sam@ravnborg.org>
> :::::: CC: David S. Miller <davem@davemloft.net>
> 
> ---
> 0-DAY CI Kernel Test Service, Intel Corporation
> https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org



-- 
RMK's Patch system: https://www.armlinux.org.uk/developer/patches/
FTTC broadband for 0.8mile line in suburbia: sync at 12.1Mbps down 622kbps up
According to speedtest.net: 11.9Mbps down 500kbps up

  reply	other threads:[~2020-03-03 14:47 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-03-03 14:12 [arm:cex7 96/112] arch/sparc/include/asm/io_64.h:179:2: warning: 'px_is' may be used uninitialized in this function kbuild test robot
2020-03-03 14:12 ` kbuild test robot
2020-03-03 14:46 ` Russell King - ARM Linux admin [this message]
2020-03-03 14:46   ` Russell King - ARM Linux admin

Reply instructions:

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

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

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

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

  git send-email \
    --in-reply-to=20200303144658.GN25745@shell.armlinux.org.uk \
    --to=linux@armlinux.org.uk \
    --cc=kbuild-all@lists.01.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=lkp@intel.com \
    --cc=peng.ma@nxp.com \
    /path/to/YOUR_REPLY

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

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.