Hi Jarkko,

I love your patch! Perhaps something to improve:

[auto build test WARNING on jss-tpmdd/next]
[also build test WARNING on next-20181102]
[cannot apply to v4.19]
[if your patch is applied to the wrong git tree, please drop us a note to help 
improve the system]

url:    
https://github.com/0day-ci/linux/commits/Jarkko-Sakkinen/Detach-TPM-space-code-from-tpm_transmit-flow/20181103-183010
base:   git://git.infradead.org/users/jjs/linux-tpmdd next
config: i386-randconfig-x002-201843 (attached as .config)
compiler: gcc-7 (Debian 7.3.0-1) 7.3.0
reproduce:
        # save the attached .config to linux build tree
        make ARCH=i386 

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

   drivers/char/tpm/tpm2-space.c: In function 'tpm2_prepare_space':
>> drivers/char/tpm/tpm2-space.c:335:16: warning: 'cc' may be used 
>> uninitialized in this function [-Wmaybe-uninitialized]
     chip->last_cc = cc;
     ~~~~~~~~~~~~~~^~~~

vim +/cc +335 drivers/char/tpm/tpm2-space.c

   302  
   303  int tpm2_prepare_space(struct tpm_chip *chip, struct tpm_space *space, 
u8 *cmd,
   304                         size_t cmdsiz)
   305  {
   306          int rc;
   307          int cc;
   308  
   309          if (!space)
   310                  return 0;
   311  
   312          cc = tpm_validate_command(chip, space, cmd, cmdsiz);
   313          if (cc < 0)
   314                  return cc;
   315  
   316          memcpy(&chip->work_space.context_tbl, &space->context_tbl,
   317                 sizeof(space->context_tbl));
   318          memcpy(&chip->work_space.session_tbl, &space->session_tbl,
   319                 sizeof(space->session_tbl));
   320          memcpy(chip->work_space.context_buf, space->context_buf, 
PAGE_SIZE);
   321          memcpy(chip->work_space.session_buf, space->session_buf, 
PAGE_SIZE);
   322  
   323          rc = tpm2_load_space(chip);
   324          if (rc) {
   325                  tpm2_flush_space(chip);
   326                  return rc;
   327          }
   328  
   329          rc = tpm2_map_command(chip, cc, cmd);
   330          if (rc) {
   331                  tpm2_flush_space(chip);
   332                  return rc;
   333          }
   334  
 > 335          chip->last_cc = cc;
   336          return 0;
   337  }
   338  

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation

Attachment: .config.gz
Description: application/gzip

Reply via email to