Re: Build failure on tpmdd master branch

["Jarkko Sakkinen" <jarkko@kernel.org>]

On Tue Aug 8, 2023 at 4:40 PM EEST, Nayna wrote:
> Hi Jarkko,
>
> I rebased the "Enable loading local and third party keys on PowerVM 
> guest"( 
> https://lore.kernel.org/linux-integrity/20230714153435.28155-1-nayna@linux.ibm.com/) 
> on top of tpmdd master branch.
>
> The build is failing due to commit cfa7522f280a ("sysctl: set variable 
> key_sysctls storage-class-specifier to static"). A patch with the same 
> subject line is already in your tree. Refer to commit 0de030b30823 
> ("sysctl: set variable key_sysctls storage-class-specifier to static").
>
> Thanks & Regards,
>
>       - Nayna

Thanks for reporting, I rebased my tree and did:

make ARCH=powerpc CROSS_COMPILE=powerpc64-linux-gnu- defconfig
make ARCH=powerpc CROSS_COMPILE=powerpc64-linux-gnu-

I get this now:

security/integrity/ima/ima_crypto.c: In function ‘ima_pcrread’:
security/integrity/ima/ima_crypto.c:785:13: error: too few arguments to function ‘tpm_pcr_read’
  785 |         if (tpm_pcr_read(ima_tpm_chip, idx, d) != 0)
      |             ^~~~~~~~~~~~
In file included from security/integrity/ima/ima.h:21,
                 from security/integrity/ima/ima_crypto.c:23:
./include/linux/tpm.h:428:12: note: declared here
  428 | extern int tpm_pcr_read(struct tpm_chip *chip, u32 pcr_idx,
      |            ^~~~~~~~~~~~

BTW, in future, please also provide both arch and .config. I made
my guesses because I had neither (not meaning to complain, just
telling what would be useful, no worries).

BR, Jarkko