Re: [Intel-gfx] [PATCH 04/10] drm/i915: remove intel_memory_region avail

[Thomas Hellström]

On 5/25/22 20:43, Matthew Auld wrote:

No longer used.

Signed-off-by: Matthew Auld <matthew.a...@intel.com>
Cc: Thomas Hellström <thomas.hellst...@linux.intel.com>
Cc: Lionel Landwerlin <lionel.g.landwer...@intel.com>
Cc: Tvrtko Ursulin <tvrtko.ursu...@linux.intel.com>
Cc: Jon Bloomfield <jon.bloomfi...@intel.com>
Cc: Daniel Vetter <daniel.vet...@ffwll.ch>
Cc: Jordan Justen <jordan.l.jus...@intel.com>
Cc: Kenneth Graunke <kenn...@whitecape.org>
Cc: Akeem G Abodunrin <akeem.g.abodun...@intel.com>
---
  drivers/gpu/drm/i915/intel_memory_region.c | 4 +---
  drivers/gpu/drm/i915/intel_memory_region.h | 1 -
  2 files changed, 1 insertion(+), 4 deletions(-)

diff --git a/drivers/gpu/drm/i915/intel_memory_region.c 
b/drivers/gpu/drm/i915/intel_memory_region.c
index 94ee26e99549..9a4a7fb55582 100644
--- a/drivers/gpu/drm/i915/intel_memory_region.c
+++ b/drivers/gpu/drm/i915/intel_memory_region.c
@@ -198,8 +198,7 @@ void intel_memory_region_debug(struct intel_memory_region 
*mr,
        if (mr->region_private)
                ttm_resource_manager_debug(mr->region_private, printer);
        else
-               drm_printf(printer, "total:%pa, available:%pa bytes\n",
-                          &mr->total, &mr->avail);
+               drm_printf(printer, "total:%pa bytes\n", &mr->total);
  }

static int intel_memory_region_memtest(struct intel_memory_region *mem,

@@ -242,7 +241,6 @@ intel_memory_region_create(struct drm_i915_private *i915,
        mem->min_page_size = min_page_size;
        mem->ops = ops;
        mem->total = size;
-       mem->avail = mem->total;
        mem->type = type;
        mem->instance = instance;

diff --git a/drivers/gpu/drm/i915/intel_memory_region.h b/drivers/gpu/drm/i915/intel_memory_region.h

index 2214f251bec3..2953ed5c3248 100644
--- a/drivers/gpu/drm/i915/intel_memory_region.h
+++ b/drivers/gpu/drm/i915/intel_memory_region.h
@@ -75,7 +75,6 @@ struct intel_memory_region {
        resource_size_t io_size;
        resource_size_t min_page_size;
        resource_size_t total;
-       resource_size_t avail;

u16 type;

        u16 instance;

Reviewed-by: Thomas Hellström <thomas.hellst...@linux.intel.com>