Re: [PATCH] dma-buf: use parameter structure for dma_buf_attach

Christian König <ckoenig.leichtzumerken@xxxxxxxxx> · Mon, 26 Mar 2018 12:47:53 +0200

Am 26.03.2018 um 10:36 schrieb Daniel Vetter:
On Sun, Mar 25, 2018 at 01:34:51PM +0200, Christian König wrote:
[SNIP]
-	attach->dev = dev;
+	attach->dev = info->dev;
  	attach->dmabuf = dmabuf;
+	attach->priv = info->priv;
The ->priv field is for the exporter, not the importer. See e.g.
drm_gem_map_attach. You can't let the importer set this now too, so needs
to be removed from the info struct.

Crap, in this case I need to add an importer_priv field because we now 
need to map from the attachment to it's importer object as well.

Thanks for noticing this.

Regards,
Christian.