mbox series

[v3,0/2] PSP TEE driver update and bug fixes

Message ID cover.1615796554.git.Rijo-john.Thomas@amd.com (mailing list archive)
Headers show
Series PSP TEE driver update and bug fixes | expand

Message

Rijo Thomas March 15, 2021, 8:25 a.m. UTC
The first patch helps to improve the response time by reducing the
polling time of the tee command status variable.

Second patch is a bug fix to handle multi-threaded use-case.
During testing, race condition was seen due to missing synchronisation
in writes to the TEE ring buffer. This patch helps to resolve that.

v3:
 * Fixed checkpatch.pl warning

v2:
 * Updated copyright year as a part of code change

Rijo Thomas (2):
  crypto: ccp - reduce tee command status polling interval from 5ms to
    1ms
  crypto: ccp - fix command queuing to TEE ring buffer

 drivers/crypto/ccp/tee-dev.c | 57 ++++++++++++++++++++++++------------
 drivers/crypto/ccp/tee-dev.h | 20 +++++++++++--
 2 files changed, 57 insertions(+), 20 deletions(-)

Comments

Herbert Xu March 26, 2021, 9:29 a.m. UTC | #1
On Mon, Mar 15, 2021 at 01:55:27PM +0530, Rijo Thomas wrote:
> The first patch helps to improve the response time by reducing the
> polling time of the tee command status variable.
> 
> Second patch is a bug fix to handle multi-threaded use-case.
> During testing, race condition was seen due to missing synchronisation
> in writes to the TEE ring buffer. This patch helps to resolve that.
> 
> v3:
>  * Fixed checkpatch.pl warning
> 
> v2:
>  * Updated copyright year as a part of code change
> 
> Rijo Thomas (2):
>   crypto: ccp - reduce tee command status polling interval from 5ms to
>     1ms
>   crypto: ccp - fix command queuing to TEE ring buffer
> 
>  drivers/crypto/ccp/tee-dev.c | 57 ++++++++++++++++++++++++------------
>  drivers/crypto/ccp/tee-dev.h | 20 +++++++++++--
>  2 files changed, 57 insertions(+), 20 deletions(-)

All applied.  Thanks.
Rijo Thomas April 5, 2021, 6:40 a.m. UTC | #2
On 26/03/21 2:59 pm, Herbert Xu wrote:
> On Mon, Mar 15, 2021 at 01:55:27PM +0530, Rijo Thomas wrote:
>> The first patch helps to improve the response time by reducing the
>> polling time of the tee command status variable.
>>
>> Second patch is a bug fix to handle multi-threaded use-case.
>> During testing, race condition was seen due to missing synchronisation
>> in writes to the TEE ring buffer. This patch helps to resolve that.
>>
>> v3:
>>  * Fixed checkpatch.pl warning
>>
>> v2:
>>  * Updated copyright year as a part of code change
>>
>> Rijo Thomas (2):
>>   crypto: ccp - reduce tee command status polling interval from 5ms to
>>     1ms
>>   crypto: ccp - fix command queuing to TEE ring buffer
>>
>>  drivers/crypto/ccp/tee-dev.c | 57 ++++++++++++++++++++++++------------
>>  drivers/crypto/ccp/tee-dev.h | 20 +++++++++++--
>>  2 files changed, 57 insertions(+), 20 deletions(-)
> 
> All applied.  Thanks.
> 

Thanks for pulling in the changes.

Cheers,
Rijo