Merge branch 'for-airlied' of git://people.freedesktop.org/~danvet/drm-intel into drm-next

Daniel writes:
"New stuff for -next. Highlights:
- prep patches for the modeset rework. Note that one of those patches
  touches the fb helper in the common drm code.
- hasw hdmi audio support (Wang Xingchao)
- improved instdone dumping for gen7 (Ben)
- unbound tracking and a few follow-up patches from Chris
- dma_buf->begin/end_cpu_access plus fix for drm/udl (Dave)
- improve mmio error reporting for hsw
- prep patch for WQ_NON_REENTRANT removal (Tejun Heo)
"

* 'for-airlied' of git://people.freedesktop.org/~danvet/drm-intel: (41 commits)
  drm/i915: Remove __GFP_NO_KSWAPD
  drm/i915: disable rc6 on ilk when vt-d is enabled
  drm/i915: Avoid unbinding due to an interrupted pin_and_fence during execbuffer
  drm/i915: Use new INSTDONE registers (Gen7+)
  drm/i915: Add new INSTDONE registers
  drm/i915: Extract reading INSTDONE
  drm/i915: Use a non-blocking wait for set-to-domain ioctl
  drm/i915: Juggle code order to ease flow of the next patch
  drm/i915: Use cpu relocations if the object is in the GTT but not mappable
  drm/i915: Extract general object init routine
  drm/i915: Protect private gem objects from truncate (such as imported dmabuf)
  drm/i915: Only pwrite through the GTT if there is space in the aperture
  i915: use alloc_ordered_workqueue() instead of explicit UNBOUND w/ max_active = 1
  drm/i915: Find unclaimed MMIO writes.
  drm/i915: Add ERR_INT to gen7 error state
  drm/i915: Cantiga+ cannot handle a hsync front porch of 0
  drm/i915: fix reassignment of variable "intel_dp->DP"
  drm/i915: Try harder to allocate an mmap_offset
  drm/i915: Show pin count in debugfs
  drm/i915: Show (count, size) of purgeable objects in i915_gem_objects
  ...

2 files changed, 20 insertions, 9 deletions

diff --git a/drivers/gpu/drm/udl/udl_fb.c b/drivers/gpu/drm/udl/udl_fb.c
index ce9a6117992..b8c00ed3305 100644
--- a/drivers/gpu/drm/udl/udl_fb.c
+++ b/drivers/gpu/drm/udl/udl_fb.c
@@ -13,6 +13,7 @@
 #include <linux/module.h>
 #include <linux/slab.h>
 #include <linux/fb.h>
+#include <linux/dma-buf.h>
 
 #include "drmP.h"
 #include "drm.h"
@@ -377,16 +378,33 @@ static int udl_user_framebuffer_dirty(struct drm_framebuffer *fb,
 {
         struct udl_framebuffer *ufb = to_udl_fb(fb);
         int i;
+        int ret = 0;
 
         if (!ufb->active_16)
                 return 0;
 
+        if (ufb->obj->base.import_attach) {
+                ret = dma_buf_begin_cpu_access(ufb->obj->base.import_attach->dmabuf,
+                                               0, ufb->obj->base.size,
+                                               DMA_FROM_DEVICE);
+                if (ret)
+                        return ret;
+        }
+
         for (i = 0; i < num_clips; i++) {
-                udl_handle_damage(ufb, clips[i].x1, clips[i].y1,
+                ret = udl_handle_damage(ufb, clips[i].x1, clips[i].y1,
                                   clips[i].x2 - clips[i].x1,
                                   clips[i].y2 - clips[i].y1);
+                if (ret)
+                        break;
         }
-        return 0;
+
+        if (ufb->obj->base.import_attach) {
+                dma_buf_end_cpu_access(ufb->obj->base.import_attach->dmabuf,
+                                       0, ufb->obj->base.size,
+                                       DMA_FROM_DEVICE);
+        }
+        return ret;
 }
 
 static void udl_user_framebuffer_destroy(struct drm_framebuffer *fb)
diff --git a/drivers/gpu/drm/udl/udl_gem.c b/drivers/gpu/drm/udl/udl_gem.c
index 291ecc14558..47b25637583 100644
--- a/drivers/gpu/drm/udl/udl_gem.c
+++ b/drivers/gpu/drm/udl/udl_gem.c
@@ -181,11 +181,6 @@ int udl_gem_vmap(struct udl_gem_object *obj)
         int ret;
 
         if (obj->base.import_attach) {
-                ret = dma_buf_begin_cpu_access(obj->base.import_attach->dmabuf,
-                                               0, obj->base.size, DMA_BIDIRECTIONAL);
-                if (ret)
-                        return -EINVAL;
-
                 obj->vmapping = dma_buf_vmap(obj->base.import_attach->dmabuf);
                 if (!obj->vmapping)
                         return -ENOMEM;
@@ -206,8 +201,6 @@ void udl_gem_vunmap(struct udl_gem_object *obj)
 {
         if (obj->base.import_attach) {
                 dma_buf_vunmap(obj->base.import_attach->dmabuf, obj->vmapping);
-                dma_buf_end_cpu_access(obj->base.import_attach->dmabuf, 0,
-                                       obj->base.size, DMA_BIDIRECTIONAL);
                 return;
         }