On Fri, Aug 06, 2021 at 08:08:27AM -0400, Stefan Berger wrote: > > On 8/6/21 7:25 AM, Jarkko Sakkinen wrote: > > On Thu, Aug 05, 2021 at 05:52:55PM -0400, Stefan Berger wrote: > > > From: Stefan Berger <stefanb@xxxxxxxxxxxxx> > > > > > > Rename the field tpm_processing_cmd to tpm_status in ibmvtpm_dev and set > > > the TPM_STATUS_BUSY flag while the vTPM is busy processing a command. > > > > > > Fixes: 6674ff145eef ("tpm_ibmvtpm: properly handle interrupted packet receptions") > > > Signed-off-by: Stefan Berger <stefanb@xxxxxxxxxxxxx> > > > Cc: Nayna Jain <nayna@xxxxxxxxxxxxx> > > > Cc: George Wilson <gcwilson@xxxxxxxxxxxxx> > > Please put the bug fix first because otherwise it will be dependent of this > > patch, which is bad thing when it comes to backporting. > > Yes, and that's why I have this one here also with a Fix tag. I basically > don't want to logically '&' with the 'true' flag but want this > TPM_STATUS_BUSY flag first. > > Stefan You can then just change the type to 'u8'. /Jarkko