Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
IronPort-SDR: o0aHOT3rPfclYcOXhsalnykM8+A1FQUIMiiE4ukVkYQXjUZiQ0iArDK3cGnC/FiM5MPHXavl/0
 bPx3Vh0JH8kQ==
IronPort-SDR: bM3ptaXzkv/iFqswyOPD70Rz4tqdTi+PMKIg5vi40sI67LQ4/qPXSa60A2xRHnj27/iaPcCfRE
 o0NPBu98S4NQ==
Date:   Mon, 2 Nov 2020 14:16:37 +0200
From:   Jarkko Sakkinen <jarkko.sakkinen@linux.intel.com>
To:     "Daniel P. Smith" <dpsmith@apertussolutions.com>
Cc:     linux-kernel@vger.kernel.org, x86@kernel.org,
        linux-integrity@vger.kernel.org, ross.philipson@oracle.com,
        tglx@linutronix.de, mingo@redhat.com, bp@alien8.de, hpa@zytor.com,
        luto@amacapital.net, trenchboot-devel@googlegroups.com
Subject: Re: [RFC PATCH 3/4] tpm: Conditionally use static buffer in TPM
 buffer management
Message-ID: <20201102121637.GB5242@linux.intel.com>
References: <20201031165122.21539-1-dpsmith@apertussolutions.com>
 <20201031165122.21539-4-dpsmith@apertussolutions.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20201031165122.21539-4-dpsmith@apertussolutions.com>
Organization: Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo
Precedence: bulk

On Sat, Oct 31, 2020 at 12:51:21PM -0400, Daniel P. Smith wrote:
> Memory management calls cannot be made in the compressed kernel
> environment to dynamically allocate TPM buffer space. For the Secure
> Launch early PCR extend code, use a static buffer instead.
> 
> Signed-off-by: Daniel P. Smith <dpsmith@apertussolutions.com>
> Signed-off-by: Ross Philipson <ross.philipson@oracle.com>
> ---

This patch is not necessary.

Just assign tb->data to the static buffer where you use the static
buffer and use tpm_buf_reset().

tpm_buf_init() and tpm_buf_destroy() are optional.

/Jarkko

>  include/linux/tpm_buffer.h | 12 ++++++++++++
>  1 file changed, 12 insertions(+)
> 
> diff --git a/include/linux/tpm_buffer.h b/include/linux/tpm_buffer.h
> index 8144a52fbc0a..c9482edf6618 100644
> --- a/include/linux/tpm_buffer.h
> +++ b/include/linux/tpm_buffer.h
> @@ -18,6 +18,10 @@
>  #ifndef __LINUX_TPM_BUFFER_H__
>  #define __LINUX_TPM_BUFFER_H__
>  
> +#ifdef COMPRESSED_KERNEL
> +static u8 _tpm_buffer[PAGE_SIZE] = {0};
> +#endif
> +
>  struct tpm_header {
>  	__be16 tag;
>  	__be32 length;
> @@ -52,7 +56,11 @@ static inline void tpm_buf_reset(struct tpm_buf *buf, u16 tag, u32 ordinal)
>  
>  static inline int tpm_buf_init(struct tpm_buf *buf, u16 tag, u32 ordinal)
>  {
> +#ifdef COMPRESSED_KERNEL
> +	buf->data = _tpm_buffer;
> +#else
>  	buf->data = (u8 *)__get_free_page(GFP_KERNEL);
> +#endif
>  	if (!buf->data)
>  		return -ENOMEM;
>  
> @@ -63,7 +71,9 @@ static inline int tpm_buf_init(struct tpm_buf *buf, u16 tag, u32 ordinal)
>  
>  static inline void tpm_buf_destroy(struct tpm_buf *buf)
>  {
> +#ifndef COMPRESSED_KERNEL
>  	free_page((unsigned long)buf->data);
> +#endif
>  }
>  
>  static inline u32 tpm_buf_length(struct tpm_buf *buf)
> @@ -92,7 +102,9 @@ static inline void tpm_buf_append(struct tpm_buf *buf,
>  		return;
>  
>  	if ((len + new_len) > PAGE_SIZE) {
> +#ifndef COMPRESSED_KERNEL
>  		WARN(1, "tpm_buf: overflow\n");
> +#endif
>  		buf->flags |= TPM_BUF_OVERFLOW;
>  		return;
>  	}
> -- 
> 2.11.0
> 

/Jarkko