Latest tip of the tree issues

Latest tip of the tree issues

[Stefan Berger]

[-- Attachment #1.1: Type: text/plain, Size: 2261 bytes --]

Jarkko,

  when building the latest tree I have the following two issues when 
building with modules: 

- missing exported symbols tpm_getcap and tpm2_get_tpm_pt. See the patch 
below that should be squashed into the patch(es) that introduced the 
problem (so one can bisect)

- the probing of TPM 1.2 doesn't work anymore ; the bad patch seems to be 
the tip:

commit 783fa18d7bb7bac20b8591cf575418fce31dd397
Author: Jarkko Sakkinen <jarkko.sakkinen-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>
Date:   Fri Jun 17 13:12:20 2016 +0200

    tpm: use tpm_pcr_read_dev() in tpm_do_selftest()
 
    Instead of a  ad-hoc protocol message construction it is better to
    call tpm_pcr_read_dev().
 
    Signed-off-by: Jarkko Sakkinen <jarkko.sakkinen-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>
    Reviewed-by: Jason Gunthorpe <jgunthorpe-ePGOBjL8dl3ta4EC/59zMFaTQe2KTcn/@public.gmane.org>


# dmesg | grep tpm
[    4.385205] tpm_tis 00:05: 1.2 TPM (device-id 0x1, rev-id 1)
[    4.424537] tpm tpm0: TPM self test failed
[    4.426630] tpm_tis: probe of 00:05 failed with error -14

The reason for this is that rc in the following is 0.

       do {
                /* Attempt to read a PCR value */
                rc = tpm_pcr_read_dev(chip, 0, dummy);

Looking at tpm_pcr_read_dev, returning 0 is quite ok.

Stefan



diff --git a/drivers/char/tpm/tpm-interface.c 
b/drivers/char/tpm/tpm-interface.c
index f43cc3f..3722fc0 100644
--- a/drivers/char/tpm/tpm-interface.c
+++ b/drivers/char/tpm/tpm-interface.c
@@ -458,6 +458,7 @@ ssize_t tpm_getcap(struct tpm_chip *chip, __be32 
subcap_id, cap_t *cap,
                *cap = tpm_cmd.params.getcap_out.cap;
        return rc;
 }
+EXPORT_SYMBOL_GPL(tpm_getcap);
 
 #define TPM_ORD_STARTUP cpu_to_be32(153)
 #define TPM_ST_CLEAR cpu_to_be16(1)
diff --git a/drivers/char/tpm/tpm2-cmd.c b/drivers/char/tpm/tpm2-cmd.c
index e1db404..499f405 100644
--- a/drivers/char/tpm/tpm2-cmd.c
+++ b/drivers/char/tpm/tpm2-cmd.c
@@ -707,6 +707,7 @@ ssize_t tpm2_get_tpm_pt(struct tpm_chip *chip, u32 
property_id,  u32 *value,
 
        return rc;
 }
+EXPORT_SYMBOL_GPL(tpm2_get_tpm_pt);
 
 #define TPM2_STARTUP_IN_SIZE \
        (sizeof(struct tpm_input_header) + \



[-- Attachment #1.2: Type: text/html, Size: 4601 bytes --]

[-- Attachment #2: Type: text/plain, Size: 79 bytes --]

------------------------------------------------------------------------------

[-- Attachment #3: Type: text/plain, Size: 192 bytes --]

_______________________________________________
tpmdd-devel mailing list
tpmdd-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org
https://lists.sourceforge.net/lists/listinfo/tpmdd-devel

Re: Latest tip of the tree issues

[Jarkko Sakkinen]

Stefan,

Since this patch is not going to the next release (it's not merged into
next nor has pull request containing it been sent) can this hold until
next week when I'm back from my four week leave? Thanks.

I'll prioritize this right after I've got through pile of emails early
next week. Thanks and sorry for the inconvience. Next branch should be
ok.


/Jarkko
On Thu, Aug 04, 2016 at 11:40:48AM -0400, Stefan Berger wrote:
>    Jarkko,
> 
>      when building the latest tree I have the following two issues when
>    building with modules:
> 
>    - missing exported symbols tpm_getcap and tpm2_get_tpm_pt. See the patch
>    below that should be squashed into the patch(es) that introduced the
>    problem (so one can bisect)
> 
>    - the probing of TPM 1.2 doesn't work anymore ; the bad patch seems to be
>    the tip:
> 
>    commit 783fa18d7bb7bac20b8591cf575418fce31dd397
>    Author: Jarkko Sakkinen <jarkko.sakkinen-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>
>    Date:   Fri Jun 17 13:12:20 2016 +0200
> 
>        tpm: use tpm_pcr_read_dev() in tpm_do_selftest()
>       
>        Instead of a  ad-hoc protocol message construction it is better to
>        call tpm_pcr_read_dev().
>       
>        Signed-off-by: Jarkko Sakkinen <jarkko.sakkinen-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>
>        Reviewed-by: Jason Gunthorpe <jgunthorpe-ePGOBjL8dl3ta4EC/59zMFaTQe2KTcn/@public.gmane.org>
> 
>    # dmesg | grep tpm
>    [    4.385205] tpm_tis 00:05: 1.2 TPM (device-id 0x1, rev-id 1)
>    [    4.424537] tpm tpm0: TPM self test failed
>    [    4.426630] tpm_tis: probe of 00:05 failed with error -14
> 
>    The reason for this is that rc in the following is 0.
> 
>           do {
>                    /* Attempt to read a PCR value */
>                    rc = tpm_pcr_read_dev(chip, 0, dummy);
> 
>    Looking at tpm_pcr_read_dev, returning 0 is quite ok.
> 
>    Stefan
> 
>    diff --git a/drivers/char/tpm/tpm-interface.c
>    b/drivers/char/tpm/tpm-interface.c
>    index f43cc3f..3722fc0 100644
>    --- a/drivers/char/tpm/tpm-interface.c
>    +++ b/drivers/char/tpm/tpm-interface.c
>    @@ -458,6 +458,7 @@ ssize_t tpm_getcap(struct tpm_chip *chip, __be32
>    subcap_id, cap_t *cap,
>                    *cap = tpm_cmd.params.getcap_out.cap;
>            return rc;
>     }
>    +EXPORT_SYMBOL_GPL(tpm_getcap);
>     
>     #define TPM_ORD_STARTUP cpu_to_be32(153)
>     #define TPM_ST_CLEAR cpu_to_be16(1)
>    diff --git a/drivers/char/tpm/tpm2-cmd.c b/drivers/char/tpm/tpm2-cmd.c
>    index e1db404..499f405 100644
>    --- a/drivers/char/tpm/tpm2-cmd.c
>    +++ b/drivers/char/tpm/tpm2-cmd.c
>    @@ -707,6 +707,7 @@ ssize_t tpm2_get_tpm_pt(struct tpm_chip *chip, u32
>    property_id,  u32 *value,
>     
>            return rc;
>     }
>    +EXPORT_SYMBOL_GPL(tpm2_get_tpm_pt);
>     
>     #define TPM2_STARTUP_IN_SIZE \
>            (sizeof(struct tpm_input_header) + \

------------------------------------------------------------------------------