On Mon, Nov 9, 2015 at 9:26 PM, Leon Romanovsky <[email protected]> wrote: > On Mon, Nov 09, 2015 at 06:30:56PM +0200, Matan Barak wrote: >> + >> + if (uhw->outlen) { >> + err = ib_copy_to_udata(uhw, &resp, resp.response_length); >> + if (err) >> + return err; >> + } >> + >> return 0; > What do you think about to rewrite this part of code to be something > like that? > + int ret = 0; > ..... > + if (uhw->outlen) > + ret = ib_copy_to_udata(uhw, &resp, resp.response_length); > + return ret;
I'll change that. Thanks. > -- > To unsubscribe from this list: send the line "unsubscribe linux-rdma" in > the body of a message to [email protected] > More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
