On Wed, Sep 25, 2019 at 03:34:01PM +0300, Jarkko Sakkinen wrote:

On Tue, Sep 24, 2019 at 07:12:40AM -0400, James Bottomley wrote:

quoted

I thought about that.  The main problem is that most of the
construct/append functions use the header, and these are the functions
most useful to the TPM2B operation.

The other thing that argues against this is that the TPM2B case would
save nothing if we eliminated the header, because we allocate a page
for all the data regardless.

It would be way more clean. There is absolutely nothing TPM2B specific.

Given the recent regression I'm detaching allocation from tpm_buf and
make it purely a decorator (sending patch today).

/Jarkko