Re: [PATCH 0/3] IB-iSER: Adjustments for three function implementations

2018-01-28 Thread Jason Gunthorpe 
On Sun, Jan 28, 2018 at 04:41:16PM +0200, Max Gurtovoy wrote:
> 
> 
> On 1/27/2018 8:17 PM, SF Markus Elfring wrote:
> >From: Markus Elfring 
> >Date: Sat, 27 Jan 2018 19:02:34 +0100
> >
> >Three update suggestions were taken into account
> >from static source code analysis.
> >
> >Markus Elfring (3):
> >   Delete an error message for a failed memory allocation in 
> > iser_send_data_out()
> >   Delete an unnecessary variable initialisation in iser_send_data_out()
> >   Combine substrings for three messages
> >
> >  drivers/infiniband/ulp/iser/iser_initiator.c | 16 ++--
> >  1 file changed, 6 insertions(+), 10 deletions(-)
> >
> 
> This series looks good to me,
> 
> Reviewed-by: Max Gurtovoy 

Okay, applied to for-next, thanks both

Jason

Re: [PATCH 0/3] IB-iSER: Adjustments for three function implementations

2018-01-28 Thread Jason Gunthorpe 
On Sun, Jan 28, 2018 at 04:41:16PM +0200, Max Gurtovoy wrote:
> 
> 
> On 1/27/2018 8:17 PM, SF Markus Elfring wrote:
> >From: Markus Elfring 
> >Date: Sat, 27 Jan 2018 19:02:34 +0100
> >
> >Three update suggestions were taken into account
> >from static source code analysis.
> >
> >Markus Elfring (3):
> >   Delete an error message for a failed memory allocation in 
> > iser_send_data_out()
> >   Delete an unnecessary variable initialisation in iser_send_data_out()
> >   Combine substrings for three messages
> >
> >  drivers/infiniband/ulp/iser/iser_initiator.c | 16 ++--
> >  1 file changed, 6 insertions(+), 10 deletions(-)
> >
> 
> This series looks good to me,
> 
> Reviewed-by: Max Gurtovoy 

Okay, applied to for-next, thanks both

Jason

Re: [PATCH 0/3] IB-iSER: Adjustments for three function implementations

2018-01-28 Thread Max Gurtovoy 



On 1/27/2018 8:17 PM, SF Markus Elfring wrote:

From: Markus Elfring 
Date: Sat, 27 Jan 2018 19:02:34 +0100

Three update suggestions were taken into account
from static source code analysis.

Markus Elfring (3):
   Delete an error message for a failed memory allocation in 
iser_send_data_out()
   Delete an unnecessary variable initialisation in iser_send_data_out()
   Combine substrings for three messages

  drivers/infiniband/ulp/iser/iser_initiator.c | 16 ++--
  1 file changed, 6 insertions(+), 10 deletions(-)



This series looks good to me,

Reviewed-by: Max Gurtovoy

Re: [PATCH 0/3] IB-iSER: Adjustments for three function implementations

2018-01-28 Thread Max Gurtovoy 



On 1/27/2018 8:17 PM, SF Markus Elfring wrote:

From: Markus Elfring 
Date: Sat, 27 Jan 2018 19:02:34 +0100

Three update suggestions were taken into account
from static source code analysis.

Markus Elfring (3):
   Delete an error message for a failed memory allocation in 
iser_send_data_out()
   Delete an unnecessary variable initialisation in iser_send_data_out()
   Combine substrings for three messages

  drivers/infiniband/ulp/iser/iser_initiator.c | 16 ++--
  1 file changed, 6 insertions(+), 10 deletions(-)



This series looks good to me,

Reviewed-by: Max Gurtovoy

[PATCH 0/3] IB-iSER: Adjustments for three function implementations

2018-01-27 Thread SF Markus Elfring 
From: Markus Elfring 
Date: Sat, 27 Jan 2018 19:02:34 +0100

Three update suggestions were taken into account
from static source code analysis.

Markus Elfring (3):
  Delete an error message for a failed memory allocation in iser_send_data_out()
  Delete an unnecessary variable initialisation in iser_send_data_out()
  Combine substrings for three messages

 drivers/infiniband/ulp/iser/iser_initiator.c | 16 ++--
 1 file changed, 6 insertions(+), 10 deletions(-)

-- 
2.16.1

[PATCH 0/3] IB-iSER: Adjustments for three function implementations

2018-01-27 Thread SF Markus Elfring 
From: Markus Elfring 
Date: Sat, 27 Jan 2018 19:02:34 +0100

Three update suggestions were taken into account
from static source code analysis.

Markus Elfring (3):
  Delete an error message for a failed memory allocation in iser_send_data_out()
  Delete an unnecessary variable initialisation in iser_send_data_out()
  Combine substrings for three messages

 drivers/infiniband/ulp/iser/iser_initiator.c | 16 ++--
 1 file changed, 6 insertions(+), 10 deletions(-)

-- 
2.16.1