[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Xen-devel] [PATCH v3 7/9] xen/gntdev: Add initial support for dma-buf UAPI
 
- To: Boris Ostrovsky <boris.ostrovsky@xxxxxxxxxx>, Oleksandr Andrushchenko <andr2000@xxxxxxxxx>, xen-devel@xxxxxxxxxxxxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx, dri-devel@xxxxxxxxxxxxxxxxxxxxx, linux-media@xxxxxxxxxxxxxxx, jgross@xxxxxxxx, konrad.wilk@xxxxxxxxxx
 
- From: Oleksandr Andrushchenko <Oleksandr_Andrushchenko@xxxxxxxx>
 
- Date: Wed, 13 Jun 2018 11:17:17 +0300
 
- Authentication-results: spf=none (sender IP is ) smtp.mailfrom=Oleksandr_Andrushchenko@xxxxxxxx; 
 
- Cc: daniel.vetter@xxxxxxxxx, matthew.d.roper@xxxxxxxxx, dongwon.kim@xxxxxxxxx
 
- Delivery-date: Wed, 13 Jun 2018 08:17:35 +0000
 
- List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
 
- Spamdiagnosticmetadata: NSPM
 
- Spamdiagnosticoutput: 1:99
 
 
 
On 06/13/2018 04:49 AM, Boris Ostrovsky wrote:
 
On 06/12/2018 09:41 AM, Oleksandr Andrushchenko wrote:
 
diff --git a/drivers/xen/gntdev.c b/drivers/xen/gntdev.c
index a09db23e9663..e82660d81d7e 100644
--- a/drivers/xen/gntdev.c
+++ b/drivers/xen/gntdev.c
@@ -48,6 +48,9 @@
  #include <asm/xen/hypercall.h>
    #include "gntdev-common.h"
+#ifdef CONFIG_XEN_GNTDEV_DMABUF
+#include "gntdev-dmabuf.h"
+#endif
    MODULE_LICENSE("GPL");
  MODULE_AUTHOR("Derek G. Murray <Derek.Murray@xxxxxxxxxxxx>, "
@@ -566,6 +569,15 @@ static int gntdev_open(struct inode *inode, 
struct file *flip)
      INIT_LIST_HEAD(&priv->freeable_maps);
      mutex_init(&priv->lock);
  +#ifdef CONFIG_XEN_GNTDEV_DMABUF
+    priv->dmabuf_priv = gntdev_dmabuf_init();
+    if (IS_ERR(priv->dmabuf_priv)) {
+        ret = PTR_ERR(priv->dmabuf_priv);
+        kfree(priv);
+        return ret;
+    }
+#endif
+
      if (use_ptemod) {
          priv->mm = get_task_mm(current);
          if (!priv->mm) {
@@ -616,8 +628,13 @@ static int gntdev_release(struct inode *inode, 
struct file *flip)
      WARN_ON(!list_empty(&priv->freeable_maps));
      mutex_unlock(&priv->lock);
  +#ifdef CONFIG_XEN_GNTDEV_DMABUF
+    gntdev_dmabuf_fini(priv->dmabuf_priv);
+#endif
+
      if (use_ptemod)
          mmu_notifier_unregister(&priv->mn, priv->mm);
+
      kfree(priv);
      return 0;
  }
 @@ -987,6 +1004,107 @@ static long gntdev_ioctl_grant_copy(struct 
gntdev_priv *priv, void __user *u)
      return ret;
  }
  +#ifdef CONFIG_XEN_GNTDEV_DMABUF
+static long
+gntdev_ioctl_dmabuf_exp_from_refs(struct gntdev_priv *priv,
+                  struct ioctl_gntdev_dmabuf_exp_from_refs __user *u)
 
 
Didn't we agree that this code moves to gntdev-dmabuf.c ?
 
 
Sure, didn't think we want IOCTL's code to be moved as well,
but that does make sense - will move all
 
-boris
 
 
Thank you,
Oleksandr
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel 
 
    
     |