All of lore.kernel.org
 help / color / mirror / Atom feed
* Re: [PATCH RESEND] infiniband:core:Fix error handling in the function cm_lap_handler
       [not found] <1456167558-10123-1-git-send-email-xerofoify@gmail.com>
@ 2016-02-23  8:54     ` Or Gerlitz
  0 siblings, 0 replies; 2+ messages in thread
From: Or Gerlitz @ 2016-02-23  8:54 UTC (permalink / raw)
  To: Nicholas Krause
  Cc: dledford-H+wXaHxf7aLQT0dZR+AlfA,
	sean.hefty-ral2JQCrhuEAvxtiuMwx3w,
	hal.rosenstock-Re5JQEeQqe8AvxtiuMwx3w,
	matanb-VPRAkNaXOzVWk0Htik3J/w, haggaie-VPRAkNaXOzVWk0Htik3J/w,
	ira.weiny-ral2JQCrhuEAvxtiuMwx3w,
	jgunthorpe-ePGOBjL8dl3ta4EC/59zMFaTQe2KTcn/,
	yun.wang-EIkl63zCoXaH+58JC4qpiA, erezsh-VPRAkNaXOzVWk0Htik3J/w,
	david.ahern-QHcLZuEGTsvQT0dZR+AlfA,
	linux-rdma-u79uwXL29TY76Z2rM5mHXA,
	linux-kernel-u79uwXL29TY76Z2rM5mHXA

On 2/22/2016 8:59 PM, Nicholas Krause wrote:
> This fixes error handling in the function cm_lap_handler to properly
> check if the internal call to the function cm_init_av_for_response
> has failed by returning a error code and if so exit immediately from
> this particular function by freeing all previously allocated
> resources before returning the error code to this function's caller
> in order to allow the caller to handle the error properly in it's
>
> Signed-off-by: Nicholas Krause<xerofoify-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>

I think this is the longest sentence I have ever read in English, and it 
even doesn't have a end... please break it and tell us the (hopefully) 
happy end.

Use IB/core: prefix and point to the commit you are fixing

Fixes: xxx ('yyy')

where xxx is the 12 digit abbrev-iated git short log  and yyy the commit 
title

--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [PATCH RESEND] infiniband:core:Fix error handling in the function cm_lap_handler
@ 2016-02-23  8:54     ` Or Gerlitz
  0 siblings, 0 replies; 2+ messages in thread
From: Or Gerlitz @ 2016-02-23  8:54 UTC (permalink / raw)
  To: Nicholas Krause
  Cc: dledford, sean.hefty, hal.rosenstock, matanb, haggaie, ira.weiny,
	jgunthorpe, yun.wang, erezsh, david.ahern, linux-rdma,
	linux-kernel

On 2/22/2016 8:59 PM, Nicholas Krause wrote:
> This fixes error handling in the function cm_lap_handler to properly
> check if the internal call to the function cm_init_av_for_response
> has failed by returning a error code and if so exit immediately from
> this particular function by freeing all previously allocated
> resources before returning the error code to this function's caller
> in order to allow the caller to handle the error properly in it's
>
> Signed-off-by: Nicholas Krause<xerofoify@gmail.com>

I think this is the longest sentence I have ever read in English, and it 
even doesn't have a end... please break it and tell us the (hopefully) 
happy end.

Use IB/core: prefix and point to the commit you are fixing

Fixes: xxx ('yyy')

where xxx is the 12 digit abbrev-iated git short log  and yyy the commit 
title

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2016-02-23 10:29 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <1456167558-10123-1-git-send-email-xerofoify@gmail.com>
     [not found] ` <1456167558-10123-1-git-send-email-xerofoify-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2016-02-23  8:54   ` [PATCH RESEND] infiniband:core:Fix error handling in the function cm_lap_handler Or Gerlitz
2016-02-23  8:54     ` Or Gerlitz

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.