From mboxrd@z Thu Jan  1 00:00:00 1970
From: Jon Mason <jdmason@kudzu.us>
Date: Wed, 24 Jan 2018 04:07:29 +0000
Subject: Re: [PATCH] ntb_perf: Fix an error code in perf_copy_chunk()
Message-Id: <20180124040728.GC20474@kudzu.us>
List-Id: <kernel-janitors.vger.kernel.org>
References: <20180123083356.GC10091@mwanda>
In-Reply-To: <20180123083356.GC10091@mwanda>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: kernel-janitors@vger.kernel.org

On Tue, Jan 23, 2018 at 11:55:31AM +0300, Serge Semin wrote:
> On Tue, Jan 23, 2018 at 11:33:56AM +0300, Dan Carpenter <dan.carpenter@oracle.com> wrote:
> > We accidentally return success if dmaengine_submit() fails.  The fix is
> > to preserve the error code from dma_submit_error().
> > 
> 
> Yep, absolutely right. Thanks!
> 
> Acked-by: Serge Semin <fancer.lancer@gmail.com>

Applied to ntb-next

Thanks,
Jon

> 
> > Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
> > 
> > diff --git a/drivers/ntb/test/ntb_perf.c b/drivers/ntb/test/ntb_perf.c
> > index 8de72f3fba4d..387e685063fc 100644
> > --- a/drivers/ntb/test/ntb_perf.c
> > +++ b/drivers/ntb/test/ntb_perf.c
> > @@ -828,7 +828,8 @@ static int perf_copy_chunk(struct perf_thread *pthr,
> >  	tx->callback_param = pthr;
> >  	dma_set_unmap(tx, unmap);
> >  
> > -	if (dma_submit_error(dmaengine_submit(tx))) {
> > +	ret = dma_submit_error(dmaengine_submit(tx));
> > +	if (ret) {
> >  		dmaengine_unmap_put(unmap);
> >  		goto err_free_resource;
> >  	}