All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] drm/i915: Remove obsolete starting offset from struct i915_address_space
@ 2016-01-15 11:57 Chris Wilson
  2016-01-15 12:20 ` ✗ Fi.CI.BAT: warning for " Patchwork
  2016-01-19 19:51 ` [PATCH] " Daniel Vetter
  0 siblings, 2 replies; 3+ messages in thread
From: Chris Wilson @ 2016-01-15 11:57 UTC (permalink / raw)
  To: intel-gfx; +Cc: Mika Kuoppala

Since the removal of UMS, we have always had sole ownership of the GTTs.
We no longer have to track the subsection we are allowed to use (denoted
by start + total). vGPU does restrict the range of the global GTT we can
use (as it has to share it amongst all the clients on the host), but
that is achieved by ballooning reserved node within the whole (so that
it could adjust available space dynamically). As such
i915_address_space.start is always 0 and we can remove having to worry
about such complications.

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Cc: Dave Gordon <david.s.gordon@intel.com>
Cc: Mika Kuoppala <mika.kuoppala@intel.com>
---
 drivers/gpu/drm/i915/i915_debugfs.c |  2 +-
 drivers/gpu/drm/i915/i915_gem_gtt.c | 24 ++++++++----------------
 drivers/gpu/drm/i915/i915_gem_gtt.h |  1 -
 drivers/gpu/drm/i915/i915_vgpu.c    | 17 +++++++----------
 4 files changed, 16 insertions(+), 28 deletions(-)

diff --git a/drivers/gpu/drm/i915/i915_debugfs.c b/drivers/gpu/drm/i915/i915_debugfs.c
index e3377abc0d4d..b1bd710f645a 100644
--- a/drivers/gpu/drm/i915/i915_debugfs.c
+++ b/drivers/gpu/drm/i915/i915_debugfs.c
@@ -495,7 +495,7 @@ static int i915_gem_object_info(struct seq_file *m, void* data)
 
 	seq_printf(m, "%llu [%llu] gtt total\n",
 		   dev_priv->gtt.base.total,
-		   (u64)dev_priv->gtt.mappable_end - dev_priv->gtt.base.start);
+		   dev_priv->gtt.mappable_end);
 
 	seq_putc(m, '\n');
 	print_batch_pool_stats(m, dev_priv);
diff --git a/drivers/gpu/drm/i915/i915_gem_gtt.c b/drivers/gpu/drm/i915/i915_gem_gtt.c
index 56f4f2e58d53..11080f00bda1 100644
--- a/drivers/gpu/drm/i915/i915_gem_gtt.c
+++ b/drivers/gpu/drm/i915/i915_gem_gtt.c
@@ -1443,7 +1443,7 @@ static void gen8_dump_pdp(struct i915_page_directory_pointer *pdp,
 static void gen8_dump_ppgtt(struct i915_hw_ppgtt *ppgtt, struct seq_file *m)
 {
 	struct i915_address_space *vm = &ppgtt->base;
-	uint64_t start = ppgtt->base.start;
+	uint64_t start = 0;
 	uint64_t length = ppgtt->base.total;
 	gen8_pte_t scratch_pte = gen8_pte_encode(px_dma(vm->scratch_page),
 						 I915_CACHE_LLC, true);
@@ -1507,7 +1507,6 @@ static int gen8_ppgtt_init(struct i915_hw_ppgtt *ppgtt)
 	if (ret)
 		return ret;
 
-	ppgtt->base.start = 0;
 	ppgtt->base.cleanup = gen8_ppgtt_cleanup;
 	ppgtt->base.allocate_va_range = gen8_alloc_va_range;
 	ppgtt->base.insert_entries = gen8_ppgtt_insert_entries;
@@ -1560,7 +1559,7 @@ static void gen6_dump_ppgtt(struct i915_hw_ppgtt *ppgtt, struct seq_file *m)
 	gen6_pte_t scratch_pte;
 	uint32_t pd_entry;
 	uint32_t  pte, pde, temp;
-	uint32_t start = ppgtt->base.start, length = ppgtt->base.total;
+	uint32_t start = 0, length = ppgtt->base.total;
 
 	scratch_pte = vm->pte_encode(px_dma(vm->scratch_page),
 				     I915_CACHE_LLC, true, 0);
@@ -2086,7 +2085,6 @@ static int gen6_ppgtt_init(struct i915_hw_ppgtt *ppgtt)
 	ppgtt->base.unbind_vma = ppgtt_unbind_vma;
 	ppgtt->base.bind_vma = ppgtt_bind_vma;
 	ppgtt->base.cleanup = gen6_ppgtt_cleanup;
-	ppgtt->base.start = 0;
 	ppgtt->base.total = I915_PDES * GEN6_PTES * PAGE_SIZE;
 	ppgtt->debug_dump = gen6_dump_ppgtt;
 
@@ -2123,7 +2121,7 @@ static int __hw_ppgtt_init(struct drm_device *dev, struct i915_hw_ppgtt *ppgtt)
 static void i915_address_space_init(struct i915_address_space *vm,
 				    struct drm_i915_private *dev_priv)
 {
-	drm_mm_init(&vm->mm, vm->start, vm->total);
+	drm_mm_init(&vm->mm, 0, vm->total);
 	vm->dev = dev_priv->dev;
 	INIT_LIST_HEAD(&vm->active_list);
 	INIT_LIST_HEAD(&vm->inactive_list);
@@ -2312,8 +2310,7 @@ void i915_gem_suspend_gtt_mappings(struct drm_device *dev)
 	i915_check_and_clear_faults(dev);
 
 	dev_priv->gtt.base.clear_range(&dev_priv->gtt.base,
-				       dev_priv->gtt.base.start,
-				       dev_priv->gtt.base.total,
+				       0, dev_priv->gtt.base.total,
 				       true);
 
 	i915_ggtt_flush(dev_priv);
@@ -2681,7 +2678,6 @@ static void i915_gtt_color_adjust(struct drm_mm_node *node,
 }
 
 static int i915_gem_setup_global_gtt(struct drm_device *dev,
-				     u64 start,
 				     u64 mappable_end,
 				     u64 end)
 {
@@ -2703,11 +2699,9 @@ static int i915_gem_setup_global_gtt(struct drm_device *dev,
 
 	BUG_ON(mappable_end > end);
 
-	ggtt_vm->start = start;
-
 	/* Subtract the guard page before address space initialization to
 	 * shrink the range used by drm_mm */
-	ggtt_vm->total = end - start - PAGE_SIZE;
+	ggtt_vm->total = end - PAGE_SIZE;
 	i915_address_space_init(ggtt_vm, dev_priv);
 	ggtt_vm->total += PAGE_SIZE;
 
@@ -2773,8 +2767,7 @@ static int i915_gem_setup_global_gtt(struct drm_device *dev,
 		}
 
 		ppgtt->base.clear_range(&ppgtt->base,
-					ppgtt->base.start,
-					ppgtt->base.total,
+					0, ppgtt->base.total,
 					true);
 
 		dev_priv->mm.aliasing_ppgtt = ppgtt;
@@ -2793,7 +2786,7 @@ void i915_gem_init_global_gtt(struct drm_device *dev)
 	gtt_size = dev_priv->gtt.base.total;
 	mappable_size = dev_priv->gtt.mappable_end;
 
-	i915_gem_setup_global_gtt(dev, 0, mappable_size, gtt_size);
+	i915_gem_setup_global_gtt(dev, mappable_size, gtt_size);
 }
 
 void i915_global_gtt_cleanup(struct drm_device *dev)
@@ -3212,8 +3205,7 @@ void i915_gem_restore_gtt_mappings(struct drm_device *dev)
 
 	/* First fill our portion of the GTT with scratch pages */
 	dev_priv->gtt.base.clear_range(&dev_priv->gtt.base,
-				       dev_priv->gtt.base.start,
-				       dev_priv->gtt.base.total,
+				       0, dev_priv->gtt.base.total,
 				       true);
 
 	/* Cache flush objects bound into GGTT and rebind them. */
diff --git a/drivers/gpu/drm/i915/i915_gem_gtt.h b/drivers/gpu/drm/i915/i915_gem_gtt.h
index b448ad832dcf..824dd966e5e4 100644
--- a/drivers/gpu/drm/i915/i915_gem_gtt.h
+++ b/drivers/gpu/drm/i915/i915_gem_gtt.h
@@ -273,7 +273,6 @@ struct i915_address_space {
 	struct drm_mm mm;
 	struct drm_device *dev;
 	struct list_head global_link;
-	u64 start;		/* Start offset always 0 for dri2 */
 	u64 total;		/* size addr space maps (ex. 2GB for ggtt) */
 
 	struct i915_page_scratch *scratch_page;
diff --git a/drivers/gpu/drm/i915/i915_vgpu.c b/drivers/gpu/drm/i915/i915_vgpu.c
index dea7429be4d0..81054c8fdc26 100644
--- a/drivers/gpu/drm/i915/i915_vgpu.c
+++ b/drivers/gpu/drm/i915/i915_vgpu.c
@@ -182,7 +182,7 @@ int intel_vgt_balloon(struct drm_device *dev)
 {
 	struct drm_i915_private *dev_priv = to_i915(dev);
 	struct i915_address_space *ggtt_vm = &dev_priv->gtt.base;
-	unsigned long ggtt_vm_end = ggtt_vm->start + ggtt_vm->total;
+	unsigned long ggtt_vm_end = ggtt_vm->total;
 
 	unsigned long mappable_base, mappable_size, mappable_end;
 	unsigned long unmappable_base, unmappable_size, unmappable_end;
@@ -202,8 +202,7 @@ int intel_vgt_balloon(struct drm_device *dev)
 	DRM_INFO("Unmappable graphic memory: base 0x%lx size %ldKiB\n",
 		 unmappable_base, unmappable_size / 1024);
 
-	if (mappable_base < ggtt_vm->start ||
-	    mappable_end > dev_priv->gtt.mappable_end ||
+	if (mappable_end > dev_priv->gtt.mappable_end ||
 	    unmappable_base < dev_priv->gtt.mappable_end ||
 	    unmappable_end > ggtt_vm_end) {
 		DRM_ERROR("Invalid ballooning configuration!\n");
@@ -235,14 +234,12 @@ int intel_vgt_balloon(struct drm_device *dev)
 	}
 
 	/* Mappable graphic memory ballooning */
-	if (mappable_base > ggtt_vm->start) {
-		ret = vgt_balloon_space(&ggtt_vm->mm,
-					&bl_info.space[0],
-					ggtt_vm->start, mappable_base);
+	ret = vgt_balloon_space(&ggtt_vm->mm,
+				&bl_info.space[0],
+				0, mappable_base);
 
-		if (ret)
-			goto err;
-	}
+	if (ret)
+		goto err;
 
 	if (mappable_end < dev_priv->gtt.mappable_end) {
 		ret = vgt_balloon_space(&ggtt_vm->mm,
-- 
2.7.0.rc3

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

^ permalink raw reply related	[flat|nested] 3+ messages in thread
* ✗ Fi.CI.BAT: warning for drm/i915: Remove obsolete starting offset from struct i915_address_space
  2016-01-15 11:57 [PATCH] drm/i915: Remove obsolete starting offset from struct i915_address_space Chris Wilson
@ 2016-01-15 12:20 ` Patchwork
  2016-01-19 19:51 ` [PATCH] " Daniel Vetter
  1 sibling, 0 replies; 3+ messages in thread
From: Patchwork @ 2016-01-15 12:20 UTC (permalink / raw)
  To: Chris Wilson; +Cc: intel-gfx

== Summary ==

Built on 615fbad7f4cea73b1a8eccdcc942c8ca1a708dab drm-intel-nightly: 2016y-01m-15d-09h-46m-32s UTC integration manifest

Test gem_storedw_loop:
        Subgroup basic-render:
                pass       -> DMESG-WARN (skl-i5k-2) UNSTABLE
                dmesg-warn -> PASS       (bdw-nuci7)
                pass       -> DMESG-WARN (skl-i7k-2) UNSTABLE
                pass       -> DMESG-WARN (bdw-ultra)
Test kms_flip:
        Subgroup basic-flip-vs-modeset:
                dmesg-warn -> PASS       (skl-i5k-2)

bdw-nuci7        total:138  pass:129  dwarn:0   dfail:0   fail:0   skip:9  
bdw-ultra        total:138  pass:131  dwarn:1   dfail:0   fail:0   skip:6  
bsw-nuc-2        total:141  pass:115  dwarn:2   dfail:0   fail:0   skip:24 
byt-nuc          total:141  pass:123  dwarn:3   dfail:0   fail:0   skip:15 
hsw-brixbox      total:141  pass:134  dwarn:0   dfail:0   fail:0   skip:7  
hsw-gt2          total:141  pass:137  dwarn:0   dfail:0   fail:0   skip:4  
ilk-hp8440p      total:141  pass:101  dwarn:3   dfail:0   fail:0   skip:37 
ivb-t430s        total:135  pass:122  dwarn:3   dfail:4   fail:0   skip:6  
skl-i5k-2        total:141  pass:131  dwarn:2   dfail:0   fail:0   skip:8  
skl-i7k-2        total:141  pass:131  dwarn:2   dfail:0   fail:0   skip:8  
snb-dellxps      total:141  pass:122  dwarn:5   dfail:0   fail:0   skip:14 
snb-x220t        total:141  pass:122  dwarn:5   dfail:0   fail:1   skip:13 

Results at /archive/results/CI_IGT_test/Patchwork_1196/

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

^ permalink raw reply	[flat|nested] 3+ messages in thread
* Re: [PATCH] drm/i915: Remove obsolete starting offset from struct i915_address_space
  2016-01-15 11:57 [PATCH] drm/i915: Remove obsolete starting offset from struct i915_address_space Chris Wilson
  2016-01-15 12:20 ` ✗ Fi.CI.BAT: warning for " Patchwork
@ 2016-01-19 19:51 ` Daniel Vetter
  1 sibling, 0 replies; 3+ messages in thread
From: Daniel Vetter @ 2016-01-19 19:51 UTC (permalink / raw)
  To: Chris Wilson; +Cc: intel-gfx, Mika Kuoppala

On Fri, Jan 15, 2016 at 11:57:49AM +0000, Chris Wilson wrote:
> Since the removal of UMS, we have always had sole ownership of the GTTs.
> We no longer have to track the subsection we are allowed to use (denoted
> by start + total). vGPU does restrict the range of the global GTT we can
> use (as it has to share it amongst all the clients on the host), but
> that is achieved by ballooning reserved node within the whole (so that
> it could adjust available space dynamically). As such
> i915_address_space.start is always 0 and we can remove having to worry
> about such complications.
> 
> Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
> Cc: Dave Gordon <david.s.gordon@intel.com>
> Cc: Mika Kuoppala <mika.kuoppala@intel.com>

Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch>

> ---
>  drivers/gpu/drm/i915/i915_debugfs.c |  2 +-
>  drivers/gpu/drm/i915/i915_gem_gtt.c | 24 ++++++++----------------
>  drivers/gpu/drm/i915/i915_gem_gtt.h |  1 -
>  drivers/gpu/drm/i915/i915_vgpu.c    | 17 +++++++----------
>  4 files changed, 16 insertions(+), 28 deletions(-)
> 
> diff --git a/drivers/gpu/drm/i915/i915_debugfs.c b/drivers/gpu/drm/i915/i915_debugfs.c
> index e3377abc0d4d..b1bd710f645a 100644
> --- a/drivers/gpu/drm/i915/i915_debugfs.c
> +++ b/drivers/gpu/drm/i915/i915_debugfs.c
> @@ -495,7 +495,7 @@ static int i915_gem_object_info(struct seq_file *m, void* data)
>  
>  	seq_printf(m, "%llu [%llu] gtt total\n",
>  		   dev_priv->gtt.base.total,
> -		   (u64)dev_priv->gtt.mappable_end - dev_priv->gtt.base.start);
> +		   dev_priv->gtt.mappable_end);
>  
>  	seq_putc(m, '\n');
>  	print_batch_pool_stats(m, dev_priv);
> diff --git a/drivers/gpu/drm/i915/i915_gem_gtt.c b/drivers/gpu/drm/i915/i915_gem_gtt.c
> index 56f4f2e58d53..11080f00bda1 100644
> --- a/drivers/gpu/drm/i915/i915_gem_gtt.c
> +++ b/drivers/gpu/drm/i915/i915_gem_gtt.c
> @@ -1443,7 +1443,7 @@ static void gen8_dump_pdp(struct i915_page_directory_pointer *pdp,
>  static void gen8_dump_ppgtt(struct i915_hw_ppgtt *ppgtt, struct seq_file *m)
>  {
>  	struct i915_address_space *vm = &ppgtt->base;
> -	uint64_t start = ppgtt->base.start;
> +	uint64_t start = 0;
>  	uint64_t length = ppgtt->base.total;
>  	gen8_pte_t scratch_pte = gen8_pte_encode(px_dma(vm->scratch_page),
>  						 I915_CACHE_LLC, true);
> @@ -1507,7 +1507,6 @@ static int gen8_ppgtt_init(struct i915_hw_ppgtt *ppgtt)
>  	if (ret)
>  		return ret;
>  
> -	ppgtt->base.start = 0;
>  	ppgtt->base.cleanup = gen8_ppgtt_cleanup;
>  	ppgtt->base.allocate_va_range = gen8_alloc_va_range;
>  	ppgtt->base.insert_entries = gen8_ppgtt_insert_entries;
> @@ -1560,7 +1559,7 @@ static void gen6_dump_ppgtt(struct i915_hw_ppgtt *ppgtt, struct seq_file *m)
>  	gen6_pte_t scratch_pte;
>  	uint32_t pd_entry;
>  	uint32_t  pte, pde, temp;
> -	uint32_t start = ppgtt->base.start, length = ppgtt->base.total;
> +	uint32_t start = 0, length = ppgtt->base.total;
>  
>  	scratch_pte = vm->pte_encode(px_dma(vm->scratch_page),
>  				     I915_CACHE_LLC, true, 0);
> @@ -2086,7 +2085,6 @@ static int gen6_ppgtt_init(struct i915_hw_ppgtt *ppgtt)
>  	ppgtt->base.unbind_vma = ppgtt_unbind_vma;
>  	ppgtt->base.bind_vma = ppgtt_bind_vma;
>  	ppgtt->base.cleanup = gen6_ppgtt_cleanup;
> -	ppgtt->base.start = 0;
>  	ppgtt->base.total = I915_PDES * GEN6_PTES * PAGE_SIZE;
>  	ppgtt->debug_dump = gen6_dump_ppgtt;
>  
> @@ -2123,7 +2121,7 @@ static int __hw_ppgtt_init(struct drm_device *dev, struct i915_hw_ppgtt *ppgtt)
>  static void i915_address_space_init(struct i915_address_space *vm,
>  				    struct drm_i915_private *dev_priv)
>  {
> -	drm_mm_init(&vm->mm, vm->start, vm->total);
> +	drm_mm_init(&vm->mm, 0, vm->total);
>  	vm->dev = dev_priv->dev;
>  	INIT_LIST_HEAD(&vm->active_list);
>  	INIT_LIST_HEAD(&vm->inactive_list);
> @@ -2312,8 +2310,7 @@ void i915_gem_suspend_gtt_mappings(struct drm_device *dev)
>  	i915_check_and_clear_faults(dev);
>  
>  	dev_priv->gtt.base.clear_range(&dev_priv->gtt.base,
> -				       dev_priv->gtt.base.start,
> -				       dev_priv->gtt.base.total,
> +				       0, dev_priv->gtt.base.total,
>  				       true);
>  
>  	i915_ggtt_flush(dev_priv);
> @@ -2681,7 +2678,6 @@ static void i915_gtt_color_adjust(struct drm_mm_node *node,
>  }
>  
>  static int i915_gem_setup_global_gtt(struct drm_device *dev,
> -				     u64 start,
>  				     u64 mappable_end,
>  				     u64 end)
>  {
> @@ -2703,11 +2699,9 @@ static int i915_gem_setup_global_gtt(struct drm_device *dev,
>  
>  	BUG_ON(mappable_end > end);
>  
> -	ggtt_vm->start = start;
> -
>  	/* Subtract the guard page before address space initialization to
>  	 * shrink the range used by drm_mm */
> -	ggtt_vm->total = end - start - PAGE_SIZE;
> +	ggtt_vm->total = end - PAGE_SIZE;
>  	i915_address_space_init(ggtt_vm, dev_priv);
>  	ggtt_vm->total += PAGE_SIZE;
>  
> @@ -2773,8 +2767,7 @@ static int i915_gem_setup_global_gtt(struct drm_device *dev,
>  		}
>  
>  		ppgtt->base.clear_range(&ppgtt->base,
> -					ppgtt->base.start,
> -					ppgtt->base.total,
> +					0, ppgtt->base.total,
>  					true);
>  
>  		dev_priv->mm.aliasing_ppgtt = ppgtt;
> @@ -2793,7 +2786,7 @@ void i915_gem_init_global_gtt(struct drm_device *dev)
>  	gtt_size = dev_priv->gtt.base.total;
>  	mappable_size = dev_priv->gtt.mappable_end;
>  
> -	i915_gem_setup_global_gtt(dev, 0, mappable_size, gtt_size);
> +	i915_gem_setup_global_gtt(dev, mappable_size, gtt_size);
>  }
>  
>  void i915_global_gtt_cleanup(struct drm_device *dev)
> @@ -3212,8 +3205,7 @@ void i915_gem_restore_gtt_mappings(struct drm_device *dev)
>  
>  	/* First fill our portion of the GTT with scratch pages */
>  	dev_priv->gtt.base.clear_range(&dev_priv->gtt.base,
> -				       dev_priv->gtt.base.start,
> -				       dev_priv->gtt.base.total,
> +				       0, dev_priv->gtt.base.total,
>  				       true);
>  
>  	/* Cache flush objects bound into GGTT and rebind them. */
> diff --git a/drivers/gpu/drm/i915/i915_gem_gtt.h b/drivers/gpu/drm/i915/i915_gem_gtt.h
> index b448ad832dcf..824dd966e5e4 100644
> --- a/drivers/gpu/drm/i915/i915_gem_gtt.h
> +++ b/drivers/gpu/drm/i915/i915_gem_gtt.h
> @@ -273,7 +273,6 @@ struct i915_address_space {
>  	struct drm_mm mm;
>  	struct drm_device *dev;
>  	struct list_head global_link;
> -	u64 start;		/* Start offset always 0 for dri2 */
>  	u64 total;		/* size addr space maps (ex. 2GB for ggtt) */
>  
>  	struct i915_page_scratch *scratch_page;
> diff --git a/drivers/gpu/drm/i915/i915_vgpu.c b/drivers/gpu/drm/i915/i915_vgpu.c
> index dea7429be4d0..81054c8fdc26 100644
> --- a/drivers/gpu/drm/i915/i915_vgpu.c
> +++ b/drivers/gpu/drm/i915/i915_vgpu.c
> @@ -182,7 +182,7 @@ int intel_vgt_balloon(struct drm_device *dev)
>  {
>  	struct drm_i915_private *dev_priv = to_i915(dev);
>  	struct i915_address_space *ggtt_vm = &dev_priv->gtt.base;
> -	unsigned long ggtt_vm_end = ggtt_vm->start + ggtt_vm->total;
> +	unsigned long ggtt_vm_end = ggtt_vm->total;
>  
>  	unsigned long mappable_base, mappable_size, mappable_end;
>  	unsigned long unmappable_base, unmappable_size, unmappable_end;
> @@ -202,8 +202,7 @@ int intel_vgt_balloon(struct drm_device *dev)
>  	DRM_INFO("Unmappable graphic memory: base 0x%lx size %ldKiB\n",
>  		 unmappable_base, unmappable_size / 1024);
>  
> -	if (mappable_base < ggtt_vm->start ||
> -	    mappable_end > dev_priv->gtt.mappable_end ||
> +	if (mappable_end > dev_priv->gtt.mappable_end ||
>  	    unmappable_base < dev_priv->gtt.mappable_end ||
>  	    unmappable_end > ggtt_vm_end) {
>  		DRM_ERROR("Invalid ballooning configuration!\n");
> @@ -235,14 +234,12 @@ int intel_vgt_balloon(struct drm_device *dev)
>  	}
>  
>  	/* Mappable graphic memory ballooning */
> -	if (mappable_base > ggtt_vm->start) {
> -		ret = vgt_balloon_space(&ggtt_vm->mm,
> -					&bl_info.space[0],
> -					ggtt_vm->start, mappable_base);
> +	ret = vgt_balloon_space(&ggtt_vm->mm,
> +				&bl_info.space[0],
> +				0, mappable_base);
>  
> -		if (ret)
> -			goto err;
> -	}
> +	if (ret)
> +		goto err;
>  
>  	if (mappable_end < dev_priv->gtt.mappable_end) {
>  		ret = vgt_balloon_space(&ggtt_vm->mm,
> -- 
> 2.7.0.rc3
> 
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/intel-gfx

-- 
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

^ permalink raw reply	[flat|nested] 3+ messages in thread
end of thread, other threads:[~2016-01-19 19:50 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-01-15 11:57 [PATCH] drm/i915: Remove obsolete starting offset from struct i915_address_space Chris Wilson
2016-01-15 12:20 ` ✗ Fi.CI.BAT: warning for " Patchwork
2016-01-19 19:51 ` [PATCH] " Daniel Vetter

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.