Re: [PATCH] tpm: tpm_crb: relinquish locality on error path.

2018-05-11 Thread Jarkko Sakkinen
On Wed, May 09, 2018 at 09:49:49PM +, Winkler, Tomas wrote: > > > > On Sat, May 05, 2018 at 09:07:21PM +, Winkler, Tomas wrote: > > > There is already one 'out' label in that function, what would you prefer > > > for > > this new label name? > > > Thanks > > > Tomas > > > > Aah, of

Re: [PATCH] tpm: tpm_crb: relinquish locality on error path.

2018-05-11 Thread Jarkko Sakkinen
On Wed, May 09, 2018 at 09:49:49PM +, Winkler, Tomas wrote: > > > > On Sat, May 05, 2018 at 09:07:21PM +, Winkler, Tomas wrote: > > > There is already one 'out' label in that function, what would you prefer > > > for > > this new label name? > > > Thanks > > > Tomas > > > > Aah, of

RE: [PATCH] tpm: tpm_crb: relinquish locality on error path.

2018-05-09 Thread Winkler, Tomas
> > On Sat, May 05, 2018 at 09:07:21PM +, Winkler, Tomas wrote: > > There is already one 'out' label in that function, what would you prefer for > this new label name? > > Thanks > > Tomas > > Aah, of course. My bad that I did not notice it but please just point me if I > miss something

RE: [PATCH] tpm: tpm_crb: relinquish locality on error path.

2018-05-09 Thread Winkler, Tomas
> > On Sat, May 05, 2018 at 09:07:21PM +, Winkler, Tomas wrote: > > There is already one 'out' label in that function, what would you prefer for > this new label name? > > Thanks > > Tomas > > Aah, of course. My bad that I did not notice it but please just point me if I > miss something

Re: [PATCH] tpm: tpm_crb: relinquish locality on error path.

2018-05-09 Thread Jarkko Sakkinen
On Sat, May 05, 2018 at 09:07:21PM +, Winkler, Tomas wrote: > There is already one 'out' label in that function, what would you prefer for > this new label name? > Thanks > Tomas Aah, of course. My bad that I did not notice it but please just point me if I miss something obvious like that

Re: [PATCH] tpm: tpm_crb: relinquish locality on error path.

2018-05-09 Thread Jarkko Sakkinen
On Sat, May 05, 2018 at 09:07:21PM +, Winkler, Tomas wrote: > There is already one 'out' label in that function, what would you prefer for > this new label name? > Thanks > Tomas Aah, of course. My bad that I did not notice it but please just point me if I miss something obvious like that

RE: [PATCH] tpm: tpm_crb: relinquish locality on error path.

2018-05-05 Thread Winkler, Tomas
> > On Thu, May 03, 2018 at 06:42:26AM +, Winkler, Tomas wrote: > > > > > > > > > > > > > > > > On Tue, Apr 24, 2018 at 08:04:01PM +0000, Winkler, Tomas wrote: > > > > > > > Subject: Re: [PATCH] tpm: tpm_crb: reli

RE: [PATCH] tpm: tpm_crb: relinquish locality on error path.

2018-05-05 Thread Winkler, Tomas
> > On Thu, May 03, 2018 at 06:42:26AM +, Winkler, Tomas wrote: > > > > > > > > > > > > > > > > On Tue, Apr 24, 2018 at 08:04:01PM +0000, Winkler, Tomas wrote: > > > > > > > Subject: Re: [PATCH] tpm: tpm_crb: reli

Re: [PATCH] tpm: tpm_crb: relinquish locality on error path.

2018-05-04 Thread Jarkko Sakkinen
On Thu, May 03, 2018 at 06:42:26AM +, Winkler, Tomas wrote: > > > > > > > > > > > > On Tue, Apr 24, 2018 at 08:04:01PM +, Winkler, Tomas wrote: > > > > > > Subject: Re: [PATCH] tpm: tpm_crb: relinquish locality on error > > >

Re: [PATCH] tpm: tpm_crb: relinquish locality on error path.

2018-05-04 Thread Jarkko Sakkinen
On Thu, May 03, 2018 at 06:42:26AM +, Winkler, Tomas wrote: > > > > > > > > > > > > On Tue, Apr 24, 2018 at 08:04:01PM +, Winkler, Tomas wrote: > > > > > > Subject: Re: [PATCH] tpm: tpm_crb: relinquish locality on error > > >

RE: [PATCH] tpm: tpm_crb: relinquish locality on error path.

2018-05-03 Thread Winkler, Tomas
> > > > > > > > On Tue, Apr 24, 2018 at 08:04:01PM +, Winkler, Tomas wrote: > > > > > Subject: Re: [PATCH] tpm: tpm_crb: relinquish locality on error path. > > > > > > > > > > On Fri, Apr 20, 2018 at 01:19:12PM +, Wi

RE: [PATCH] tpm: tpm_crb: relinquish locality on error path.

2018-05-03 Thread Winkler, Tomas
> > > > > > > > On Tue, Apr 24, 2018 at 08:04:01PM +, Winkler, Tomas wrote: > > > > > Subject: Re: [PATCH] tpm: tpm_crb: relinquish locality on error path. > > > > > > > > > > On Fri, Apr 20, 2018 at 01:19:12PM +, Wi

Re: [PATCH] tpm: tpm_crb: relinquish locality on error path.

2018-05-03 Thread Jarkko Sakkinen
On Wed, May 02, 2018 at 01:35:47PM +, Winkler, Tomas wrote: > > > > > > On Tue, Apr 24, 2018 at 08:04:01PM +, Winkler, Tomas wrote: > > > > Subject: Re: [PATCH] tpm: tpm_crb: relinquish locality on error path. > > > > > > > > On

Re: [PATCH] tpm: tpm_crb: relinquish locality on error path.

2018-05-03 Thread Jarkko Sakkinen
On Wed, May 02, 2018 at 01:35:47PM +, Winkler, Tomas wrote: > > > > > > On Tue, Apr 24, 2018 at 08:04:01PM +, Winkler, Tomas wrote: > > > > Subject: Re: [PATCH] tpm: tpm_crb: relinquish locality on error path. > > > > > > > > On

RE: [PATCH] tpm: tpm_crb: relinquish locality on error path.

2018-05-02 Thread Winkler, Tomas
> > On Tue, Apr 24, 2018 at 08:04:01PM +, Winkler, Tomas wrote: > > > Subject: Re: [PATCH] tpm: tpm_crb: relinquish locality on error path. > > > > > > On Fri, Apr 20, 2018 at 01:19:12PM +, Winkler, Tomas wrote: > > > > > > On T

RE: [PATCH] tpm: tpm_crb: relinquish locality on error path.

2018-05-02 Thread Winkler, Tomas
> > On Tue, Apr 24, 2018 at 08:04:01PM +, Winkler, Tomas wrote: > > > Subject: Re: [PATCH] tpm: tpm_crb: relinquish locality on error path. > > > > > > On Fri, Apr 20, 2018 at 01:19:12PM +, Winkler, Tomas wrote: > > > > > > On T

Re: [PATCH] tpm: tpm_crb: relinquish locality on error path.

2018-05-02 Thread Jarkko Sakkinen
On Tue, Apr 24, 2018 at 08:04:01PM +, Winkler, Tomas wrote: > > Subject: Re: [PATCH] tpm: tpm_crb: relinquish locality on error path. > > > > On Fri, Apr 20, 2018 at 01:19:12PM +, Winkler, Tomas wrote: > > > > > On Tue, 2018-04-10 at

Re: [PATCH] tpm: tpm_crb: relinquish locality on error path.

2018-05-02 Thread Jarkko Sakkinen
On Tue, Apr 24, 2018 at 08:04:01PM +, Winkler, Tomas wrote: > > Subject: Re: [PATCH] tpm: tpm_crb: relinquish locality on error path. > > > > On Fri, Apr 20, 2018 at 01:19:12PM +, Winkler, Tomas wrote: > > > > > On Tue, 2018-04-10 at

RE: [PATCH] tpm: tpm_crb: relinquish locality on error path.

2018-04-24 Thread Winkler, Tomas
> On Tue, Apr 24, 2018 at 07:03:28PM +0300, Jarkko Sakkinen wrote: > > On Fri, Apr 20, 2018 at 01:19:12PM +, Winkler, Tomas wrote: > > > > > On Tue, 2018-04-10 at 09:00 +, Winkler, Tomas wrote: > > > > > > > > > > > > > > On Sat, 2018-04-07 at 19:12 +0300, Tomas Winkler wrote: > > > > > >

RE: [PATCH] tpm: tpm_crb: relinquish locality on error path.

2018-04-24 Thread Winkler, Tomas
> On Tue, Apr 24, 2018 at 07:03:28PM +0300, Jarkko Sakkinen wrote: > > On Fri, Apr 20, 2018 at 01:19:12PM +, Winkler, Tomas wrote: > > > > > On Tue, 2018-04-10 at 09:00 +, Winkler, Tomas wrote: > > > > > > > > > > > > > > On Sat, 2018-04-07 at 19:12 +0300, Tomas Winkler wrote: > > > > > >

RE: [PATCH] tpm: tpm_crb: relinquish locality on error path.

2018-04-24 Thread Winkler, Tomas
> Subject: Re: [PATCH] tpm: tpm_crb: relinquish locality on error path. > > On Fri, Apr 20, 2018 at 01:19:12PM +, Winkler, Tomas wrote: > > > > On Tue, 2018-04-10 at 09:00 +, Winkler, Tomas wrote: > > > > > > > > > > > &g

RE: [PATCH] tpm: tpm_crb: relinquish locality on error path.

2018-04-24 Thread Winkler, Tomas
> Subject: Re: [PATCH] tpm: tpm_crb: relinquish locality on error path. > > On Fri, Apr 20, 2018 at 01:19:12PM +, Winkler, Tomas wrote: > > > > On Tue, 2018-04-10 at 09:00 +, Winkler, Tomas wrote: > > > > > > > > > > > &g

Re: [PATCH] tpm: tpm_crb: relinquish locality on error path.

2018-04-24 Thread Jarkko Sakkinen
On Tue, Apr 24, 2018 at 07:03:28PM +0300, Jarkko Sakkinen wrote: > On Fri, Apr 20, 2018 at 01:19:12PM +, Winkler, Tomas wrote: > > > > On Tue, 2018-04-10 at 09:00 +, Winkler, Tomas wrote: > > > > > > > > > > > > On Sat, 2018-04-07 at 19:12 +0300, Tomas Winkler wrote: > > > > > > > In

Re: [PATCH] tpm: tpm_crb: relinquish locality on error path.

2018-04-24 Thread Jarkko Sakkinen
On Tue, Apr 24, 2018 at 07:03:28PM +0300, Jarkko Sakkinen wrote: > On Fri, Apr 20, 2018 at 01:19:12PM +, Winkler, Tomas wrote: > > > > On Tue, 2018-04-10 at 09:00 +, Winkler, Tomas wrote: > > > > > > > > > > > > On Sat, 2018-04-07 at 19:12 +0300, Tomas Winkler wrote: > > > > > > > In

Re: [PATCH] tpm: tpm_crb: relinquish locality on error path.

2018-04-24 Thread Jarkko Sakkinen
On Fri, Apr 20, 2018 at 01:19:12PM +, Winkler, Tomas wrote: > > > On Tue, 2018-04-10 at 09:00 +, Winkler, Tomas wrote: > > > > > > > > > > On Sat, 2018-04-07 at 19:12 +0300, Tomas Winkler wrote: > > > > > > In crb_map_io() function, __crb_request_locality() is called > > > > > > prior to

Re: [PATCH] tpm: tpm_crb: relinquish locality on error path.

2018-04-24 Thread Jarkko Sakkinen
On Fri, Apr 20, 2018 at 01:19:12PM +, Winkler, Tomas wrote: > > > On Tue, 2018-04-10 at 09:00 +, Winkler, Tomas wrote: > > > > > > > > > > On Sat, 2018-04-07 at 19:12 +0300, Tomas Winkler wrote: > > > > > > In crb_map_io() function, __crb_request_locality() is called > > > > > > prior to

RE: [PATCH] tpm: tpm_crb: relinquish locality on error path.

2018-04-20 Thread Winkler, Tomas
> > On Tue, 2018-04-10 at 09:00 +, Winkler, Tomas wrote: > > > > > > > > On Sat, 2018-04-07 at 19:12 +0300, Tomas Winkler wrote: > > > > > In crb_map_io() function, __crb_request_locality() is called > > > > > prior to crb_cmd_ready(), but if one of the consecutive function > > > > > fails the

RE: [PATCH] tpm: tpm_crb: relinquish locality on error path.

2018-04-20 Thread Winkler, Tomas
> > On Tue, 2018-04-10 at 09:00 +, Winkler, Tomas wrote: > > > > > > > > On Sat, 2018-04-07 at 19:12 +0300, Tomas Winkler wrote: > > > > > In crb_map_io() function, __crb_request_locality() is called > > > > > prior to crb_cmd_ready(), but if one of the consecutive function > > > > > fails the

RE: [PATCH] tpm: tpm_crb: relinquish locality on error path.

2018-04-10 Thread Winkler, Tomas
> > On Tue, 2018-04-10 at 09:00 +, Winkler, Tomas wrote: > > > > > > On Sat, 2018-04-07 at 19:12 +0300, Tomas Winkler wrote: > > > > In crb_map_io() function, __crb_request_locality() is called prior > > > > to crb_cmd_ready(), but if one of the consecutive function fails > > > > the flow

RE: [PATCH] tpm: tpm_crb: relinquish locality on error path.

2018-04-10 Thread Winkler, Tomas
> > On Tue, 2018-04-10 at 09:00 +, Winkler, Tomas wrote: > > > > > > On Sat, 2018-04-07 at 19:12 +0300, Tomas Winkler wrote: > > > > In crb_map_io() function, __crb_request_locality() is called prior > > > > to crb_cmd_ready(), but if one of the consecutive function fails > > > > the flow

Re: [PATCH] tpm: tpm_crb: relinquish locality on error path.

2018-04-10 Thread Jarkko Sakkinen
On Tue, 2018-04-10 at 09:00 +, Winkler, Tomas wrote: > > > > On Sat, 2018-04-07 at 19:12 +0300, Tomas Winkler wrote: > > > In crb_map_io() function, __crb_request_locality() is called prior to > > > crb_cmd_ready(), but if one of the consecutive function fails the flow > > > bails out instead

Re: [PATCH] tpm: tpm_crb: relinquish locality on error path.

2018-04-10 Thread Jarkko Sakkinen
On Tue, 2018-04-10 at 09:00 +, Winkler, Tomas wrote: > > > > On Sat, 2018-04-07 at 19:12 +0300, Tomas Winkler wrote: > > > In crb_map_io() function, __crb_request_locality() is called prior to > > > crb_cmd_ready(), but if one of the consecutive function fails the flow > > > bails out instead

RE: [PATCH] tpm: tpm_crb: relinquish locality on error path.

2018-04-10 Thread Winkler, Tomas
> > On Sat, 2018-04-07 at 19:12 +0300, Tomas Winkler wrote: > > In crb_map_io() function, __crb_request_locality() is called prior to > > crb_cmd_ready(), but if one of the consecutive function fails the flow > > bails out instead of trying to relinquish locality. > > This patch adds goto jump to

RE: [PATCH] tpm: tpm_crb: relinquish locality on error path.

2018-04-10 Thread Winkler, Tomas
> > On Sat, 2018-04-07 at 19:12 +0300, Tomas Winkler wrote: > > In crb_map_io() function, __crb_request_locality() is called prior to > > crb_cmd_ready(), but if one of the consecutive function fails the flow > > bails out instead of trying to relinquish locality. > > This patch adds goto jump to

Re: [PATCH] tpm: tpm_crb: relinquish locality on error path.

2018-04-09 Thread Jarkko Sakkinen
On Sat, 2018-04-07 at 19:12 +0300, Tomas Winkler wrote: > In crb_map_io() function, __crb_request_locality() is called prior > to crb_cmd_ready(), but if one of the consecutive function fails > the flow bails out instead of trying to relinquish locality. > This patch adds goto jump to

Re: [PATCH] tpm: tpm_crb: relinquish locality on error path.

2018-04-09 Thread Jarkko Sakkinen
On Sat, 2018-04-07 at 19:12 +0300, Tomas Winkler wrote: > In crb_map_io() function, __crb_request_locality() is called prior > to crb_cmd_ready(), but if one of the consecutive function fails > the flow bails out instead of trying to relinquish locality. > This patch adds goto jump to

[PATCH] tpm: tpm_crb: relinquish locality on error path.

2018-04-07 Thread Tomas Winkler
In crb_map_io() function, __crb_request_locality() is called prior to crb_cmd_ready(), but if one of the consecutive function fails the flow bails out instead of trying to relinquish locality. This patch adds goto jump to __crb_relinquish_locality() on the error path. Fixes: 888d867df441 (tpm:

[PATCH] tpm: tpm_crb: relinquish locality on error path.

2018-04-07 Thread Tomas Winkler
In crb_map_io() function, __crb_request_locality() is called prior to crb_cmd_ready(), but if one of the consecutive function fails the flow bails out instead of trying to relinquish locality. This patch adds goto jump to __crb_relinquish_locality() on the error path. Fixes: 888d867df441 (tpm: