CC: kbuild-...@lists.01.org
BCC: l...@intel.com
CC: linux-ker...@vger.kernel.org
TO: Srinivas Kandagatla <srinivas.kandaga...@linaro.org>
CC: Mark Brown <broo...@kernel.org>
CC: "Pierre-Louis Bossart" <pierre-louis.boss...@linux.intel.com>

Hi Srinivas,

First bad commit (maybe != root cause):

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git 
master
head:   34e047aa16c0123bbae8e2f6df33e5ecc1f56601
commit: cf989b68fcadbeeea1446e50fd8b2f24a0f1275c ASoC: qdsp6: audioreach: add 
Kconfig and Makefile
date:   5 months ago
:::::: branch date: 22 hours ago
:::::: commit date: 5 months ago
config: openrisc-randconfig-m031-20220319 
(https://download.01.org/0day-ci/archive/20220320/202203200147.9wzmpdx8-...@intel.com/config)
compiler: or1k-linux-gcc (GCC) 11.2.0

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <l...@intel.com>
Reported-by: Dan Carpenter <dan.carpen...@oracle.com>

smatch warnings:
sound/soc/qcom/qdsp6/audioreach.c:212 __audioreach_alloc_pkt() warn: is 'p' 
large enough for 'struct gpr_pkt'? s32min
sound/soc/qcom/qdsp6/audioreach.c:228 __audioreach_alloc_pkt() warn: is 'p' 
large enough for 'struct apm_cmd_header'? 0

vim +212 sound/soc/qcom/qdsp6/audioreach.c

44c28dbdb6195b Srinivas Kandagatla 2021-10-26  196  
44c28dbdb6195b Srinivas Kandagatla 2021-10-26  197  static void 
*__audioreach_alloc_pkt(int payload_size, uint32_t opcode, uint32_t token,
44c28dbdb6195b Srinivas Kandagatla 2021-10-26  198                              
    uint32_t src_port, uint32_t dest_port, bool has_cmd_hdr)
44c28dbdb6195b Srinivas Kandagatla 2021-10-26  199  {
44c28dbdb6195b Srinivas Kandagatla 2021-10-26  200      struct gpr_pkt *pkt;
44c28dbdb6195b Srinivas Kandagatla 2021-10-26  201      void *p;
44c28dbdb6195b Srinivas Kandagatla 2021-10-26  202      int pkt_size = 
GPR_HDR_SIZE + payload_size;
44c28dbdb6195b Srinivas Kandagatla 2021-10-26  203  
44c28dbdb6195b Srinivas Kandagatla 2021-10-26  204      if (has_cmd_hdr)
44c28dbdb6195b Srinivas Kandagatla 2021-10-26  205              pkt_size += 
APM_CMD_HDR_SIZE;
44c28dbdb6195b Srinivas Kandagatla 2021-10-26  206  
44c28dbdb6195b Srinivas Kandagatla 2021-10-26  207      p = kzalloc(pkt_size, 
GFP_KERNEL);
44c28dbdb6195b Srinivas Kandagatla 2021-10-26  208      if (!p)
44c28dbdb6195b Srinivas Kandagatla 2021-10-26  209              return 
ERR_PTR(-ENOMEM);
44c28dbdb6195b Srinivas Kandagatla 2021-10-26  210  
44c28dbdb6195b Srinivas Kandagatla 2021-10-26  211      pkt = p;
44c28dbdb6195b Srinivas Kandagatla 2021-10-26 @212      pkt->hdr.version = 
GPR_PKT_VER;
44c28dbdb6195b Srinivas Kandagatla 2021-10-26  213      pkt->hdr.hdr_size = 
GPR_PKT_HEADER_WORD_SIZE;
44c28dbdb6195b Srinivas Kandagatla 2021-10-26  214      pkt->hdr.pkt_size = 
pkt_size;
44c28dbdb6195b Srinivas Kandagatla 2021-10-26  215      pkt->hdr.dest_port = 
dest_port;
44c28dbdb6195b Srinivas Kandagatla 2021-10-26  216      pkt->hdr.src_port = 
src_port;
44c28dbdb6195b Srinivas Kandagatla 2021-10-26  217  
44c28dbdb6195b Srinivas Kandagatla 2021-10-26  218      pkt->hdr.dest_domain = 
GPR_DOMAIN_ID_ADSP;
44c28dbdb6195b Srinivas Kandagatla 2021-10-26  219      pkt->hdr.src_domain = 
GPR_DOMAIN_ID_APPS;
44c28dbdb6195b Srinivas Kandagatla 2021-10-26  220      pkt->hdr.token = token;
44c28dbdb6195b Srinivas Kandagatla 2021-10-26  221      pkt->hdr.opcode = 
opcode;
44c28dbdb6195b Srinivas Kandagatla 2021-10-26  222  
44c28dbdb6195b Srinivas Kandagatla 2021-10-26  223      if (has_cmd_hdr) {
44c28dbdb6195b Srinivas Kandagatla 2021-10-26  224              struct 
apm_cmd_header *cmd_header;
44c28dbdb6195b Srinivas Kandagatla 2021-10-26  225  
44c28dbdb6195b Srinivas Kandagatla 2021-10-26  226              p = p + 
GPR_HDR_SIZE;
44c28dbdb6195b Srinivas Kandagatla 2021-10-26  227              cmd_header = p;
44c28dbdb6195b Srinivas Kandagatla 2021-10-26 @228              
cmd_header->payload_size = payload_size;
44c28dbdb6195b Srinivas Kandagatla 2021-10-26  229      }
44c28dbdb6195b Srinivas Kandagatla 2021-10-26  230  
44c28dbdb6195b Srinivas Kandagatla 2021-10-26  231      return pkt;
44c28dbdb6195b Srinivas Kandagatla 2021-10-26  232  }
44c28dbdb6195b Srinivas Kandagatla 2021-10-26  233  

:::::: The code at line 212 was first introduced by commit
:::::: 44c28dbdb6195b2a92e1fcb2946d1e987658f8b5 ASoC: qdsp6: audioreach: add 
basic pkt alloc support

:::::: TO: Srinivas Kandagatla <srinivas.kandaga...@linaro.org>
:::::: CC: Mark Brown <broo...@kernel.org>

-- 
0-DAY CI Kernel Test Service
https://01.org/lkp
_______________________________________________
kbuild mailing list -- kbuild@lists.01.org
To unsubscribe send an email to kbuild-le...@lists.01.org

Reply via email to