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
