Signed-off-by: Matthew Auld <matthew.a...@intel.com>
Cc: Joonas Lahtinen <joonas.lahti...@linux.intel.com>
Cc: Abdiel Janulgue <abdiel.janul...@linux.intel.com>
---
 drivers/gpu/drm/i915/gem/i915_gem_phys.c      |  5 +-
 drivers/gpu/drm/i915/gem/i915_gem_region.c    | 14 +++-
 drivers/gpu/drm/i915/gem/i915_gem_shmem.c     | 74 ++++++++++++++-----
 drivers/gpu/drm/i915/i915_drv.h               |  2 +
 drivers/gpu/drm/i915/i915_gem.c               |  9 ---
 drivers/gpu/drm/i915/i915_gem_gtt.c           |  3 +-
 drivers/gpu/drm/i915/i915_pci.c               | 29 ++++++--
 .../gpu/drm/i915/selftests/mock_gem_device.c  |  6 +-
 8 files changed, 99 insertions(+), 43 deletions(-)

diff --git a/drivers/gpu/drm/i915/gem/i915_gem_phys.c 
b/drivers/gpu/drm/i915/gem/i915_gem_phys.c
index 768356908160..8043ff63d73f 100644
--- a/drivers/gpu/drm/i915/gem/i915_gem_phys.c
+++ b/drivers/gpu/drm/i915/gem/i915_gem_phys.c
@@ -16,6 +16,7 @@
 #include "gt/intel_gt.h"
 #include "i915_drv.h"
 #include "i915_gem_object.h"
+#include "i915_gem_region.h"
 #include "i915_scatterlist.h"
 
 static int i915_gem_object_get_pages_phys(struct drm_i915_gem_object *obj)
@@ -191,8 +192,10 @@ int i915_gem_object_attach_phys(struct drm_i915_gem_object 
*obj, int align)
        /* Perma-pin (until release) the physical set of pages */
        __i915_gem_object_pin_pages(obj);
 
-       if (!IS_ERR_OR_NULL(pages))
+       if (!IS_ERR_OR_NULL(pages)) {
                i915_gem_shmem_ops.put_pages(obj, pages);
+               i915_gem_object_release_memory_region(obj);
+       }
        mutex_unlock(&obj->mm.lock);
        return 0;
 
diff --git a/drivers/gpu/drm/i915/gem/i915_gem_region.c 
b/drivers/gpu/drm/i915/gem/i915_gem_region.c
index 663254b3da21..0c03c2c40dbc 100644
--- a/drivers/gpu/drm/i915/gem/i915_gem_region.c
+++ b/drivers/gpu/drm/i915/gem/i915_gem_region.c
@@ -6,6 +6,7 @@
 #include "intel_memory_region.h"
 #include "i915_gem_region.h"
 #include "i915_drv.h"
+#include "i915_trace.h"
 
 void
 i915_gem_object_put_pages_buddy(struct drm_i915_gem_object *obj,
@@ -146,11 +147,22 @@ i915_gem_object_create_region(struct intel_memory_region 
*mem,
        GEM_BUG_ON(!size);
        GEM_BUG_ON(!IS_ALIGNED(size, I915_GTT_MIN_ALIGNMENT));
 
+       /*
+        * There is a prevalence of the assumption that we fit the object's
+        * page count inside a 32bit _signed_ variable. Let's document this and
+        * catch if we ever need to fix it. In the meantime, if you do spot
+        * such a local variable, please consider fixing!
+        */
+
        if (size >> PAGE_SHIFT > INT_MAX)
                return ERR_PTR(-E2BIG);
 
        if (overflows_type(size, obj->base.size))
                return ERR_PTR(-E2BIG);
 
-       return mem->ops->create_object(mem, size, flags);
+       obj = mem->ops->create_object(mem, size, flags);
+       if (!IS_ERR(obj))
+               trace_i915_gem_object_create(obj);
+
+       return obj;
 }
diff --git a/drivers/gpu/drm/i915/gem/i915_gem_shmem.c 
b/drivers/gpu/drm/i915/gem/i915_gem_shmem.c
index 4c4954e8ce0a..f468a3424df4 100644
--- a/drivers/gpu/drm/i915/gem/i915_gem_shmem.c
+++ b/drivers/gpu/drm/i915/gem/i915_gem_shmem.c
@@ -7,7 +7,9 @@
 #include <linux/pagevec.h>
 #include <linux/swap.h>
 
+#include "gem/i915_gem_region.h"
 #include "i915_drv.h"
+#include "i915_gemfs.h"
 #include "i915_gem_object.h"
 #include "i915_scatterlist.h"
 #include "i915_trace.h"
@@ -26,6 +28,7 @@ static void check_release_pagevec(struct pagevec *pvec)
 static int shmem_get_pages(struct drm_i915_gem_object *obj)
 {
        struct drm_i915_private *i915 = to_i915(obj->base.dev);
+       struct intel_memory_region *mem = obj->mm.region;
        const unsigned long page_count = obj->base.size / PAGE_SIZE;
        unsigned long i;
        struct address_space *mapping;
@@ -52,7 +55,7 @@ static int shmem_get_pages(struct drm_i915_gem_object *obj)
         * If there's no chance of allocating enough pages for the whole
         * object, bail early.
         */
-       if (page_count > totalram_pages())
+       if (obj->base.size > resource_size(&mem->region))
                return -ENOMEM;
 
        st = kmalloc(sizeof(*st), GFP_KERNEL);
@@ -417,6 +420,8 @@ shmem_pwrite(struct drm_i915_gem_object *obj,
 
 static void shmem_release(struct drm_i915_gem_object *obj)
 {
+       i915_gem_object_release_memory_region(obj);
+
        fput(obj->base.filp);
 }
 
@@ -434,9 +439,9 @@ const struct drm_i915_gem_object_ops i915_gem_shmem_ops = {
        .release = shmem_release,
 };
 
-static int create_shmem(struct drm_i915_private *i915,
-                       struct drm_gem_object *obj,
-                       size_t size)
+static int __create_shmem(struct drm_i915_private *i915,
+                         struct drm_gem_object *obj,
+                         size_t size)
 {
        unsigned long flags = VM_NORESERVE;
        struct file *filp;
@@ -455,31 +460,23 @@ static int create_shmem(struct drm_i915_private *i915,
        return 0;
 }
 
-struct drm_i915_gem_object *
-i915_gem_object_create_shmem(struct drm_i915_private *i915, u64 size)
+static struct drm_i915_gem_object *
+create_shmem(struct intel_memory_region *mem,
+            resource_size_t size,
+            unsigned int flags)
 {
+       struct drm_i915_private *i915 = mem->i915;
        struct drm_i915_gem_object *obj;
        struct address_space *mapping;
        unsigned int cache_level;
        gfp_t mask;
        int ret;
 
-       /* There is a prevalence of the assumption that we fit the object's
-        * page count inside a 32bit _signed_ variable. Let's document this and
-        * catch if we ever need to fix it. In the meantime, if you do spot
-        * such a local variable, please consider fixing!
-        */
-       if (size >> PAGE_SHIFT > INT_MAX)
-               return ERR_PTR(-E2BIG);
-
-       if (overflows_type(size, obj->base.size))
-               return ERR_PTR(-E2BIG);
-
        obj = i915_gem_object_alloc();
        if (!obj)
                return ERR_PTR(-ENOMEM);
 
-       ret = create_shmem(i915, &obj->base, size);
+       ret = __create_shmem(i915, &obj->base, size);
        if (ret)
                goto fail;
 
@@ -518,7 +515,7 @@ i915_gem_object_create_shmem(struct drm_i915_private *i915, 
u64 size)
 
        i915_gem_object_set_cache_coherency(obj, cache_level);
 
-       trace_i915_gem_object_create(obj);
+       i915_gem_object_init_memory_region(obj, mem, 0);
 
        return obj;
 
@@ -527,6 +524,13 @@ i915_gem_object_create_shmem(struct drm_i915_private 
*i915, u64 size)
        return ERR_PTR(ret);
 }
 
+struct drm_i915_gem_object *
+i915_gem_object_create_shmem(struct drm_i915_private *i915, u64 size)
+{
+       return 
i915_gem_object_create_region(i915->mm.regions[INTEL_MEMORY_SMEM],
+                                            size, 0);
+}
+
 /* Allocate a new GEM object and fill it with the supplied data */
 struct drm_i915_gem_object *
 i915_gem_object_create_shmem_from_data(struct drm_i915_private *dev_priv,
@@ -577,3 +581,35 @@ i915_gem_object_create_shmem_from_data(struct 
drm_i915_private *dev_priv,
        i915_gem_object_put(obj);
        return ERR_PTR(err);
 }
+
+static int init_shmem(struct intel_memory_region *mem)
+{
+       int err;
+
+       err = i915_gemfs_init(mem->i915);
+       if (err) {
+               DRM_NOTE("Unable to create a private tmpfs mount, hugepage 
support will be disabled(%d).\n",
+                        err);
+       }
+
+       return 0; /* Don't error, we can simply fallback to the kernel mnt */
+}
+
+static void release_shmem(struct intel_memory_region *mem)
+{
+       i915_gemfs_fini(mem->i915);
+}
+
+static const struct intel_memory_region_ops shmem_region_ops = {
+       .init = init_shmem,
+       .release = release_shmem,
+       .create_object = create_shmem,
+};
+
+struct intel_memory_region *i915_gem_shmem_setup(struct drm_i915_private *i915)
+{
+       return intel_memory_region_create(i915, 0,
+                                         totalram_pages() << PAGE_SHIFT,
+                                         I915_GTT_PAGE_SIZE_4K, 0,
+                                         &shmem_region_ops);
+}
diff --git a/drivers/gpu/drm/i915/i915_drv.h b/drivers/gpu/drm/i915/i915_drv.h
index 0233265405b4..3f7f407c4ca4 100644
--- a/drivers/gpu/drm/i915/i915_drv.h
+++ b/drivers/gpu/drm/i915/i915_drv.h
@@ -2222,6 +2222,8 @@ void i915_gem_cleanup_early(struct drm_i915_private 
*dev_priv);
 int i915_gem_freeze(struct drm_i915_private *dev_priv);
 int i915_gem_freeze_late(struct drm_i915_private *dev_priv);
 
+struct intel_memory_region *i915_gem_shmem_setup(struct drm_i915_private 
*i915);
+
 static inline void i915_gem_drain_freed_objects(struct drm_i915_private *i915)
 {
        /*
diff --git a/drivers/gpu/drm/i915/i915_gem.c b/drivers/gpu/drm/i915/i915_gem.c
index 0ddbd3a5fb8d..9cc5ee3b7f2f 100644
--- a/drivers/gpu/drm/i915/i915_gem.c
+++ b/drivers/gpu/drm/i915/i915_gem.c
@@ -45,7 +45,6 @@
 #include "gem/i915_gem_context.h"
 #include "gem/i915_gem_ioctls.h"
 #include "gem/i915_gem_pm.h"
-#include "gem/i915_gemfs.h"
 #include "gt/intel_engine_user.h"
 #include "gt/intel_gt.h"
 #include "gt/intel_gt_pm.h"
@@ -1441,16 +1440,10 @@ static void i915_gem_init__mm(struct drm_i915_private 
*i915)
 
 void i915_gem_init_early(struct drm_i915_private *dev_priv)
 {
-       int err;
-
        i915_gem_init__mm(dev_priv);
        i915_gem_init__pm(dev_priv);
 
        spin_lock_init(&dev_priv->fb_tracking.lock);
-
-       err = i915_gemfs_init(dev_priv);
-       if (err)
-               DRM_NOTE("Unable to create a private tmpfs mount, hugepage 
support will be disabled(%d).\n", err);
 }
 
 void i915_gem_cleanup_early(struct drm_i915_private *dev_priv)
@@ -1459,8 +1452,6 @@ void i915_gem_cleanup_early(struct drm_i915_private 
*dev_priv)
        GEM_BUG_ON(!llist_empty(&dev_priv->mm.free_list));
        GEM_BUG_ON(atomic_read(&dev_priv->mm.free_count));
        WARN_ON(dev_priv->mm.shrink_count);
-
-       i915_gemfs_fini(dev_priv);
 }
 
 int i915_gem_freeze(struct drm_i915_private *dev_priv)
diff --git a/drivers/gpu/drm/i915/i915_gem_gtt.c 
b/drivers/gpu/drm/i915/i915_gem_gtt.c
index 450ec8779252..32ee220d2837 100644
--- a/drivers/gpu/drm/i915/i915_gem_gtt.c
+++ b/drivers/gpu/drm/i915/i915_gem_gtt.c
@@ -2780,7 +2780,8 @@ int i915_gem_init_memory_regions(struct drm_i915_private 
*i915)
 
                type = MEMORY_TYPE_FROM_REGION(intel_region_map[i]);
                switch (type) {
-               default:
+               case INTEL_SMEM:
+                       mem = i915_gem_shmem_setup(i915);
                        break;
                }
 
diff --git a/drivers/gpu/drm/i915/i915_pci.c b/drivers/gpu/drm/i915/i915_pci.c
index 1cbf3998b361..e55ab2724996 100644
--- a/drivers/gpu/drm/i915/i915_pci.c
+++ b/drivers/gpu/drm/i915/i915_pci.c
@@ -151,6 +151,9 @@
 #define GEN_DEFAULT_PAGE_SIZES \
        .page_sizes = I915_GTT_PAGE_SIZE_4K
 
+#define GEN_DEFAULT_REGIONS \
+       .memory_regions = REGION_SMEM
+
 #define I830_FEATURES \
        GEN(2), \
        .is_mobile = 1, \
@@ -168,7 +171,8 @@
        I9XX_PIPE_OFFSETS, \
        I9XX_CURSOR_OFFSETS, \
        I9XX_COLORS, \
-       GEN_DEFAULT_PAGE_SIZES
+       GEN_DEFAULT_PAGE_SIZES, \
+       GEN_DEFAULT_REGIONS
 
 #define I845_FEATURES \
        GEN(2), \
@@ -185,7 +189,8 @@
        I845_PIPE_OFFSETS, \
        I845_CURSOR_OFFSETS, \
        I9XX_COLORS, \
-       GEN_DEFAULT_PAGE_SIZES
+       GEN_DEFAULT_PAGE_SIZES, \
+       GEN_DEFAULT_REGIONS
 
 static const struct intel_device_info intel_i830_info = {
        I830_FEATURES,
@@ -219,7 +224,8 @@ static const struct intel_device_info intel_i865g_info = {
        I9XX_PIPE_OFFSETS, \
        I9XX_CURSOR_OFFSETS, \
        I9XX_COLORS, \
-       GEN_DEFAULT_PAGE_SIZES
+       GEN_DEFAULT_PAGE_SIZES, \
+       GEN_DEFAULT_REGIONS
 
 static const struct intel_device_info intel_i915g_info = {
        GEN3_FEATURES,
@@ -304,7 +310,8 @@ static const struct intel_device_info intel_pineview_m_info 
= {
        I9XX_PIPE_OFFSETS, \
        I9XX_CURSOR_OFFSETS, \
        I965_COLORS, \
-       GEN_DEFAULT_PAGE_SIZES
+       GEN_DEFAULT_PAGE_SIZES, \
+       GEN_DEFAULT_REGIONS
 
 static const struct intel_device_info intel_i965g_info = {
        GEN4_FEATURES,
@@ -354,7 +361,8 @@ static const struct intel_device_info intel_gm45_info = {
        I9XX_PIPE_OFFSETS, \
        I9XX_CURSOR_OFFSETS, \
        ILK_COLORS, \
-       GEN_DEFAULT_PAGE_SIZES
+       GEN_DEFAULT_PAGE_SIZES, \
+       GEN_DEFAULT_REGIONS
 
 static const struct intel_device_info intel_ironlake_d_info = {
        GEN5_FEATURES,
@@ -384,7 +392,8 @@ static const struct intel_device_info intel_ironlake_m_info 
= {
        I9XX_PIPE_OFFSETS, \
        I9XX_CURSOR_OFFSETS, \
        ILK_COLORS, \
-       GEN_DEFAULT_PAGE_SIZES
+       GEN_DEFAULT_PAGE_SIZES, \
+       GEN_DEFAULT_REGIONS
 
 #define SNB_D_PLATFORM \
        GEN6_FEATURES, \
@@ -432,7 +441,8 @@ static const struct intel_device_info 
intel_sandybridge_m_gt2_info = {
        IVB_PIPE_OFFSETS, \
        IVB_CURSOR_OFFSETS, \
        IVB_COLORS, \
-       GEN_DEFAULT_PAGE_SIZES
+       GEN_DEFAULT_PAGE_SIZES, \
+       GEN_DEFAULT_REGIONS
 
 #define IVB_D_PLATFORM \
        GEN7_FEATURES, \
@@ -493,6 +503,7 @@ static const struct intel_device_info intel_valleyview_info 
= {
        I9XX_CURSOR_OFFSETS,
        I965_COLORS,
        GEN_DEFAULT_PAGE_SIZES,
+       GEN_DEFAULT_REGIONS,
 };
 
 #define G75_FEATURES  \
@@ -587,6 +598,7 @@ static const struct intel_device_info intel_cherryview_info 
= {
        CHV_CURSOR_OFFSETS,
        CHV_COLORS,
        GEN_DEFAULT_PAGE_SIZES,
+       GEN_DEFAULT_REGIONS,
 };
 
 #define GEN9_DEFAULT_PAGE_SIZES \
@@ -661,7 +673,8 @@ static const struct intel_device_info 
intel_skylake_gt4_info = {
        HSW_PIPE_OFFSETS, \
        IVB_CURSOR_OFFSETS, \
        IVB_COLORS, \
-       GEN9_DEFAULT_PAGE_SIZES
+       GEN9_DEFAULT_PAGE_SIZES, \
+       GEN_DEFAULT_REGIONS
 
 static const struct intel_device_info intel_broxton_info = {
        GEN9_LP_FEATURES,
diff --git a/drivers/gpu/drm/i915/selftests/mock_gem_device.c 
b/drivers/gpu/drm/i915/selftests/mock_gem_device.c
index 6d47f0aeac0c..c50234f938a4 100644
--- a/drivers/gpu/drm/i915/selftests/mock_gem_device.c
+++ b/drivers/gpu/drm/i915/selftests/mock_gem_device.c
@@ -72,8 +72,6 @@ static void mock_device_release(struct drm_device *dev)
        mock_fini_ggtt(&i915->ggtt);
        destroy_workqueue(i915->wq);
 
-       i915_gemfs_fini(i915);
-
        i915_gem_cleanup_memory_regions(i915);
 
        drm_mode_config_cleanup(&i915->drm);
@@ -165,6 +163,8 @@ struct drm_i915_private *mock_gem_device(void)
                I915_GTT_PAGE_SIZE_64K |
                I915_GTT_PAGE_SIZE_2M;
 
+       mkwrite_device_info(i915)->memory_regions = REGION_SMEM;
+
        mock_uncore_init(&i915->uncore);
        i915_gem_init__mm(i915);
        intel_gt_init_early(&i915->gt, i915);
@@ -196,8 +196,6 @@ struct drm_i915_private *mock_gem_device(void)
 
        intel_engines_driver_register(i915);
 
-       WARN_ON(i915_gemfs_init(i915));
-
        err = i915_gem_init_memory_regions(i915);
        if (err)
                goto err_context;
-- 
2.20.1

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

Reply via email to