On Thu, Dec 15, 2016 at 09:11:09PM +0530, Bart Van Assche wrote: > On 12/11/2016 05:35 PM, Varun Prakash wrote: > > If iscsit_tpg_add_network_portal() fails then > > return error code instead of 0 to user space. > > > > If iscsi-target returns 0 then user space keeps > > on retrying same command infinitely, targetcli or > > echo hangs till command completes with non zero > > return value. In some cases it is possible that > > add network portal command never completes with > > success even after retrying multiple times, > > for example - cxgbit_setup_np() always returns > > -EINVAL if portal IP does not belong to Chelsio > > adapter interface. > > Hello Varun, > > This patch looks fine to me. Since you posted it as an RFC does this > mean that you are not completely confident that this patch is right? Thanks for reviewing, I am completely confident that this patch is correct and I tested it before posting. -- 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