Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1591454

Re: [tpmdd-devel] [PATCH v2] tpm: Apply a sane minimum adapterlimit value for retransmission.

Show all headers | View raw

Hi Andrew,

On 02/03/17 18:24, Andrew Lunn wrote:
>>>> Hi Enric
>>>>
>>>> Maybe you should remember that you need to use smaller transfers? If
>>>> you don't remember, but use the full size message every time and only
>>>> drop back on error, the i2c core is going to log rate limited
>>>> messages. By remembering, there will only be one such message in the
>>>> log.
>>>>
>>
> 
>> Maybe I did not explain well but this is what the code does, when
>> i2c-core fails with -EOPNOTSUPP because the msg is too long for this
>> adapter it loop with a smaller chunk of fixed size, so you only see
>> the i2c-core message once.
> 
> Hi Enric
> 
> Would it not be more accurate to say, that you only see the i2c-core
> message once, for this transfer request. Is the next transfer request
> again going to use the longer length? then fail, maybe generate
> another i2c core message, depending on rate limiting, and then use the
> lower message size? I think it does.
> 
> Which is why i suggested remembering the length.
> 

Ah ok, sorry I did not understand you, got it now. Yes, so with this every time you do, i.e. a tpm_getpubek, you will see the i2c-core message, right. Thanks. Sounds a good suggestion for me.

Cheers,
 Enric

>       Andrew
> 

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH v2] tpm: Apply a sane minimum adapterlimit value for retransmission. Enric Balletbo i Serra <enric.balletbo@collabora.com> - 2017-03-01 16:40 +0100
  Re: [PATCH v2] tpm: Apply a sane minimum adapterlimit value for  retransmission. Andrew Lunn <andrew@lunn.ch> - 2017-03-02 01:30 +0100
  Re: [tpmdd-devel] [PATCH v2] tpm: Apply a sane minimum adapterlimit  value for retransmission. Jarkko Sakkinen <jarkko.sakkinen@linux.intel.com> - 2017-03-02 14:00 +0100
    Re: [tpmdd-devel] [PATCH v2] tpm: Apply a sane minimum adapterlimit  value for retransmission. Enric Balletbo i Serra <enric.balletbo@collabora.com> - 2017-03-02 17:40 +0100
      Re: [tpmdd-devel] [PATCH v2] tpm: Apply a sane minimum adapterlimit  value for retransmission. Andrew Lunn <andrew@lunn.ch> - 2017-03-02 18:30 +0100
        Re: [tpmdd-devel] [PATCH v2] tpm: Apply a sane minimum adapterlimit  value for retransmission. Enric Balletbo i Serra <enric.balletbo@collabora.com> - 2017-03-02 21:40 +0100
    Re: [tpmdd-devel] [PATCH v2] tpm: Apply a sane minimum adapterlimit value for retransmission. Peter Huewe <peterhuewe@gmx.de> - 2017-03-02 18:50 +0100

csiph-web