On 4/28/26 8:57 AM, Stefan Berger wrote:


On 4/28/26 3:01 AM, Arun Menon wrote:


With MAX() it can now be bigger than TPM_TIS_BUFFER_MAX if the backend says
so -- hm...

TPM_TIS_BUFFER_MAX is still 4096.

Oh, I had still based my patches on your TPM_TIS_BUFFER_MAX increase to 8192 bytes. Let me fixes this along with a few other things. I will let you know.

I have another series that I will post now that can be applied to master. It's adding a test for TIS over I2C but I will need to extend that one also with the large transfer test case then.



So I have resolved this issue now along with a few other things. My branch is here: https://github.com/stefanberger/qemu-tpm/tree/work-tpm-for-11.1

- I have applied the i2c swtpm test case series first since it could be easily upstreamed first
- Then your patch "migration/vmstate: Add VMState support for GByteArray"
- Then the CRB chunk + TIS extended buffer support series. I modified my patches (last 4 in that series) to
  - increased MAX_TIS_BUFFER_SIZE to 8192
- added migration blockers dynamically for TIS whenever >4096 bytes are either in the request or response; remove them later on again when device goes into ready state for example
  - added large transfer test also for i2c

Please pick up those patches for v6 posting, or otherwise you can split your v5 series up into CRB-only support for v6 and I post my (last 4) patches later on.


Reply via email to