[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20201119171051.GA6464@ubuntu>
Date: Thu, 19 Nov 2020 18:10:51 +0100
From: Guennadi Liakhovetski <guennadi.liakhovetski@...ux.intel.com>
To: Arnaud POULIQUEN <arnaud.pouliquen@...com>
Cc: Mathieu Poirier <mathieu.poirier@...aro.org>,
"ohad@...ery.com" <ohad@...ery.com>,
"bjorn.andersson@...aro.org" <bjorn.andersson@...aro.org>,
"linux-remoteproc@...r.kernel.org" <linux-remoteproc@...r.kernel.org>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH v6 8/8] rpmsg: Turn name service into a stand alone driver
Hi Mathieu,
I haven't tested it yet, but I've compared it to the previous version
and all the changed except the one, mentioned by Arnaud, seem to be
there! So, let's just fix this one and should be good to go!
Thanks
Guennadi
On Thu, Nov 19, 2020 at 03:52:50PM +0100, Arnaud POULIQUEN wrote:
[snip]
> As Guennadi mentionned in a previous mail
> a kfree(vch) is missing on error
>
> @@ -949,10 +949,11 @@ static int rpmsg_probe(struct virtio_device *vdev)
> dev_info(&vdev->dev, "rpmsg host is online\n");
>
> return 0;
>
> free_coherent:
> + kfree(vch);
> dma_free_coherent(vdev->dev.parent, total_buf_space,
> bufs_va, vrp->bufs_dma);
> vqs_del:
>
> Otherwise i tested this series successfully.
>
> Thanks,
> Arnaud
Powered by blists - more mailing lists