On Wed, Mar 02, 2016 at 02:24:22PM +0300, Dan Carpenter wrote: > On Wed, Mar 02, 2016 at 04:19:38PM +0530, Varun Prakash wrote: > > > cxgb4_remove_server() can return NET_XMIT_DROP, I will add a check > > for positive return value. > > Why does cxgb4_remove_server() return NET_XMIT_DROP, though? There is > no comment, and the callers just check for zero vs non-zero. Ok, I will add comment and submit the patch. -- To unsubscribe from this list: send the line "unsubscribe target-devel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html