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@xxxxxxxxxxxxx/) > 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
