lkml.org 
[lkml]   [2020]   [Nov]   [19]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [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

\
 
 \ /
  Last update: 2020-11-19 18:12    [W:0.134 / U:0.404 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site