All of lore.kernel.org
 help / color / mirror / Atom feed
From: kernel test robot <lkp@intel.com>
To: kbuild-all@lists.01.org
Subject: drivers/crypto/sa2ul.c:432:1: warning: the frame size of 1088 bytes is larger than 1024 bytes
Date: Sun, 19 Dec 2021 07:52:53 +0800	[thread overview]
Message-ID: <202112190729.P81P29Nc-lkp@intel.com> (raw)

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

Hi Keerthy,

FYI, the error/warning still remains.

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head:   9eaa88c7036eda3f6c215f87ca693594cf90559b
commit: d2c8ac187fc922e73930a1b2f6a211e27f595d01 crypto: sa2ul - Add AEAD algorithm support
date:   1 year, 5 months ago
config: mips-randconfig-p002-20211218 (https://download.01.org/0day-ci/archive/20211219/202112190729.P81P29Nc-lkp(a)intel.com/config)
compiler: mips-linux-gcc (GCC) 11.2.0
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
        # https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=d2c8ac187fc922e73930a1b2f6a211e27f595d01
        git remote add linus https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
        git fetch --no-tags linus master
        git checkout d2c8ac187fc922e73930a1b2f6a211e27f595d01
        # save the config file to linux build tree
        mkdir build_dir
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.2.0 make.cross O=build_dir ARCH=mips SHELL=/bin/bash drivers/crypto/ drivers/opp/ drivers/pinctrl/ drivers/usb/chipidea/

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 >>):

   In file included from include/linux/device.h:15,
                    from include/linux/dmaengine.h:8,
                    from drivers/crypto/sa2ul.c:12:
   drivers/crypto/sa2ul.c: In function 'sa_sha_init':
   drivers/crypto/sa2ul.c:1486:47: warning: cast from pointer to integer of different size [-Wpointer-to-int-cast]
    1486 |                 crypto_ahash_digestsize(tfm), (u64)rctx);
         |                                               ^
   include/linux/dev_printk.h:123:61: note: in definition of macro 'dev_dbg'
     123 |                 dev_printk(KERN_DEBUG, dev, dev_fmt(fmt), ##__VA_ARGS__); \
         |                                                             ^~~~~~~~~~~
   drivers/crypto/sa2ul.c: In function 'sa_prepare_iopads':
>> drivers/crypto/sa2ul.c:432:1: warning: the frame size of 1088 bytes is larger than 1024 bytes [-Wframe-larger-than=]
     432 | }
         | ^


cppcheck possible warnings: (new ones prefixed by >>, may not real problems)

>> drivers/usb/chipidea/core.c:596:15: warning: Local variable ci_role shadows outer function [shadowFunction]
    enum ci_role ci_role = usb_role_to_ci_role(role);
                 ^
   drivers/usb/chipidea/ci.h:262:38: note: Shadowed declaration
   static inline struct ci_role_driver *ci_role(struct ci_hdrc *ci)
                                        ^
   drivers/usb/chipidea/core.c:596:15: note: Shadow variable
    enum ci_role ci_role = usb_role_to_ci_role(role);
                 ^
>> drivers/opp/core.c:1395:4: warning: Address of local auto-variable assigned to a function parameter. [autoVariables]
      *head = &opp->node;
      ^

vim +432 drivers/crypto/sa2ul.c

   407	
   408	static void sa_prepare_iopads(struct algo_data *data, const u8 *key,
   409				      u16 key_sz, __be32 *ipad, __be32 *opad)
   410	{
   411		SHASH_DESC_ON_STACK(shash, data->ctx->shash);
   412		int block_size = crypto_shash_blocksize(data->ctx->shash);
   413		int digest_size = crypto_shash_digestsize(data->ctx->shash);
   414		u8 k_ipad[SHA1_BLOCK_SIZE];
   415		u8 k_opad[SHA1_BLOCK_SIZE];
   416	
   417		shash->tfm = data->ctx->shash;
   418	
   419		prepare_kiopad(k_ipad, k_opad, key, key_sz);
   420	
   421		memzero_explicit(ipad, block_size);
   422		memzero_explicit(opad, block_size);
   423	
   424		crypto_shash_init(shash);
   425		crypto_shash_update(shash, k_ipad, block_size);
   426		sa_export_shash(shash, block_size, digest_size, ipad);
   427	
   428		crypto_shash_init(shash);
   429		crypto_shash_update(shash, k_opad, block_size);
   430	
   431		sa_export_shash(shash, block_size, digest_size, opad);
 > 432	}
   433	

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org

WARNING: multiple messages have this Message-ID (diff)
From: kernel test robot <lkp@intel.com>
To: Keerthy <j-keerthy@ti.com>
Cc: kbuild-all@lists.01.org, linux-kernel@vger.kernel.org,
	Herbert Xu <herbert@gondor.apana.org.au>,
	Tero Kristo <t-kristo@ti.com>
Subject: drivers/crypto/sa2ul.c:432:1: warning: the frame size of 1088 bytes is larger than 1024 bytes
Date: Sun, 19 Dec 2021 07:52:53 +0800	[thread overview]
Message-ID: <202112190729.P81P29Nc-lkp@intel.com> (raw)

Hi Keerthy,

FYI, the error/warning still remains.

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head:   9eaa88c7036eda3f6c215f87ca693594cf90559b
commit: d2c8ac187fc922e73930a1b2f6a211e27f595d01 crypto: sa2ul - Add AEAD algorithm support
date:   1 year, 5 months ago
config: mips-randconfig-p002-20211218 (https://download.01.org/0day-ci/archive/20211219/202112190729.P81P29Nc-lkp@intel.com/config)
compiler: mips-linux-gcc (GCC) 11.2.0
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
        # https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=d2c8ac187fc922e73930a1b2f6a211e27f595d01
        git remote add linus https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
        git fetch --no-tags linus master
        git checkout d2c8ac187fc922e73930a1b2f6a211e27f595d01
        # save the config file to linux build tree
        mkdir build_dir
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.2.0 make.cross O=build_dir ARCH=mips SHELL=/bin/bash drivers/crypto/ drivers/opp/ drivers/pinctrl/ drivers/usb/chipidea/

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 >>):

   In file included from include/linux/device.h:15,
                    from include/linux/dmaengine.h:8,
                    from drivers/crypto/sa2ul.c:12:
   drivers/crypto/sa2ul.c: In function 'sa_sha_init':
   drivers/crypto/sa2ul.c:1486:47: warning: cast from pointer to integer of different size [-Wpointer-to-int-cast]
    1486 |                 crypto_ahash_digestsize(tfm), (u64)rctx);
         |                                               ^
   include/linux/dev_printk.h:123:61: note: in definition of macro 'dev_dbg'
     123 |                 dev_printk(KERN_DEBUG, dev, dev_fmt(fmt), ##__VA_ARGS__); \
         |                                                             ^~~~~~~~~~~
   drivers/crypto/sa2ul.c: In function 'sa_prepare_iopads':
>> drivers/crypto/sa2ul.c:432:1: warning: the frame size of 1088 bytes is larger than 1024 bytes [-Wframe-larger-than=]
     432 | }
         | ^


cppcheck possible warnings: (new ones prefixed by >>, may not real problems)

>> drivers/usb/chipidea/core.c:596:15: warning: Local variable ci_role shadows outer function [shadowFunction]
    enum ci_role ci_role = usb_role_to_ci_role(role);
                 ^
   drivers/usb/chipidea/ci.h:262:38: note: Shadowed declaration
   static inline struct ci_role_driver *ci_role(struct ci_hdrc *ci)
                                        ^
   drivers/usb/chipidea/core.c:596:15: note: Shadow variable
    enum ci_role ci_role = usb_role_to_ci_role(role);
                 ^
>> drivers/opp/core.c:1395:4: warning: Address of local auto-variable assigned to a function parameter. [autoVariables]
      *head = &opp->node;
      ^

vim +432 drivers/crypto/sa2ul.c

   407	
   408	static void sa_prepare_iopads(struct algo_data *data, const u8 *key,
   409				      u16 key_sz, __be32 *ipad, __be32 *opad)
   410	{
   411		SHASH_DESC_ON_STACK(shash, data->ctx->shash);
   412		int block_size = crypto_shash_blocksize(data->ctx->shash);
   413		int digest_size = crypto_shash_digestsize(data->ctx->shash);
   414		u8 k_ipad[SHA1_BLOCK_SIZE];
   415		u8 k_opad[SHA1_BLOCK_SIZE];
   416	
   417		shash->tfm = data->ctx->shash;
   418	
   419		prepare_kiopad(k_ipad, k_opad, key, key_sz);
   420	
   421		memzero_explicit(ipad, block_size);
   422		memzero_explicit(opad, block_size);
   423	
   424		crypto_shash_init(shash);
   425		crypto_shash_update(shash, k_ipad, block_size);
   426		sa_export_shash(shash, block_size, digest_size, ipad);
   427	
   428		crypto_shash_init(shash);
   429		crypto_shash_update(shash, k_opad, block_size);
   430	
   431		sa_export_shash(shash, block_size, digest_size, opad);
 > 432	}
   433	

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org

             reply	other threads:[~2021-12-18 23:52 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-12-18 23:52 kernel test robot [this message]
2021-12-18 23:52 ` drivers/crypto/sa2ul.c:432:1: warning: the frame size of 1088 bytes is larger than 1024 bytes kernel test robot

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=202112190729.P81P29Nc-lkp@intel.com \
    --to=lkp@intel.com \
    --cc=kbuild-all@lists.01.org \
    /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.