]> asedeno.scripts.mit.edu Git - linux.git/commitdiff
drm/i915: Split GEM object type definition to its own header
authorChris Wilson <chris@chris-wilson.co.uk>
Tue, 28 May 2019 09:29:42 +0000 (10:29 +0100)
committerChris Wilson <chris@chris-wilson.co.uk>
Tue, 28 May 2019 11:45:29 +0000 (12:45 +0100)
For convenience in avoiding inline spaghetti, keep the type definition
as a separate header.

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Reviewed-by: Matthew Auld <matthew.auld@intel.com>
Acked-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
Acked-by: Jani Nikula <jani.nikula@intel.com>
Acked-by: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20190528092956.14910-1-chris@chris-wilson.co.uk
drivers/gpu/drm/i915/Makefile
drivers/gpu/drm/i915/gem/Makefile [new file with mode: 0644]
drivers/gpu/drm/i915/gem/Makefile.header-test [new file with mode: 0644]
drivers/gpu/drm/i915/gem/i915_gem_object_types.h [new file with mode: 0644]
drivers/gpu/drm/i915/gt/intel_engine_types.h
drivers/gpu/drm/i915/i915_drv.h
drivers/gpu/drm/i915/i915_gem_batch_pool.h
drivers/gpu/drm/i915/i915_gem_gtt.h
drivers/gpu/drm/i915/i915_gem_object.h

index 139a0fc1939020a9cce19531b2c20c97f5a6621b..23a45d7baabbe3944455f145e6a97bc3162acc35 100644 (file)
@@ -85,6 +85,7 @@ gt-$(CONFIG_DRM_I915_SELFTEST) += \
 i915-y += $(gt-y)
 
 # GEM (Graphics Execution Management) code
+obj-y += gem/
 i915-y += \
          i915_active.o \
          i915_cmd_parser.o \
diff --git a/drivers/gpu/drm/i915/gem/Makefile b/drivers/gpu/drm/i915/gem/Makefile
new file mode 100644 (file)
index 0000000..07e7b8b
--- /dev/null
@@ -0,0 +1 @@
+include $(src)/Makefile.header-test # Extra header tests
diff --git a/drivers/gpu/drm/i915/gem/Makefile.header-test b/drivers/gpu/drm/i915/gem/Makefile.header-test
new file mode 100644 (file)
index 0000000..61e06cb
--- /dev/null
@@ -0,0 +1,16 @@
+# SPDX-License-Identifier: MIT
+# Copyright © 2019 Intel Corporation
+
+# Test the headers are compilable as standalone units
+header_test := $(notdir $(wildcard $(src)/*.h))
+
+quiet_cmd_header_test = HDRTEST $@
+      cmd_header_test = echo "\#include \"$(<F)\"" > $@
+
+header_test_%.c: %.h
+       $(call cmd,header_test)
+
+extra-$(CONFIG_DRM_I915_WERROR) += \
+       $(foreach h,$(header_test),$(patsubst %.h,header_test_%.o,$(h)))
+
+clean-files += $(foreach h,$(header_test),$(patsubst %.h,header_test_%.c,$(h)))
diff --git a/drivers/gpu/drm/i915/gem/i915_gem_object_types.h b/drivers/gpu/drm/i915/gem/i915_gem_object_types.h
new file mode 100644 (file)
index 0000000..fe3b2a2
--- /dev/null
@@ -0,0 +1,285 @@
+/*
+ * SPDX-License-Identifier: MIT
+ *
+ * Copyright © 2016 Intel Corporation
+ */
+
+#ifndef __I915_GEM_OBJECT_TYPES_H__
+#define __I915_GEM_OBJECT_TYPES_H__
+
+#include <linux/reservation.h>
+
+#include <drm/drm_gem.h>
+
+#include "i915_active.h"
+#include "i915_selftest.h"
+
+struct drm_i915_gem_object;
+
+/*
+ * struct i915_lut_handle tracks the fast lookups from handle to vma used
+ * for execbuf. Although we use a radixtree for that mapping, in order to
+ * remove them as the object or context is closed, we need a secondary list
+ * and a translation entry (i915_lut_handle).
+ */
+struct i915_lut_handle {
+       struct list_head obj_link;
+       struct list_head ctx_link;
+       struct i915_gem_context *ctx;
+       u32 handle;
+};
+
+struct drm_i915_gem_object_ops {
+       unsigned int flags;
+#define I915_GEM_OBJECT_HAS_STRUCT_PAGE        BIT(0)
+#define I915_GEM_OBJECT_IS_SHRINKABLE  BIT(1)
+#define I915_GEM_OBJECT_IS_PROXY       BIT(2)
+#define I915_GEM_OBJECT_ASYNC_CANCEL   BIT(3)
+
+       /* Interface between the GEM object and its backing storage.
+        * get_pages() is called once prior to the use of the associated set
+        * of pages before to binding them into the GTT, and put_pages() is
+        * called after we no longer need them. As we expect there to be
+        * associated cost with migrating pages between the backing storage
+        * and making them available for the GPU (e.g. clflush), we may hold
+        * onto the pages after they are no longer referenced by the GPU
+        * in case they may be used again shortly (for example migrating the
+        * pages to a different memory domain within the GTT). put_pages()
+        * will therefore most likely be called when the object itself is
+        * being released or under memory pressure (where we attempt to
+        * reap pages for the shrinker).
+        */
+       int (*get_pages)(struct drm_i915_gem_object *obj);
+       void (*put_pages)(struct drm_i915_gem_object *obj,
+                         struct sg_table *pages);
+
+       int (*pwrite)(struct drm_i915_gem_object *obj,
+                     const struct drm_i915_gem_pwrite *arg);
+
+       int (*dmabuf_export)(struct drm_i915_gem_object *obj);
+       void (*release)(struct drm_i915_gem_object *obj);
+};
+
+struct drm_i915_gem_object {
+       struct drm_gem_object base;
+
+       const struct drm_i915_gem_object_ops *ops;
+
+       struct {
+               /**
+                * @vma.lock: protect the list/tree of vmas
+                */
+               spinlock_t lock;
+
+               /**
+                * @vma.list: List of VMAs backed by this object
+                *
+                * The VMA on this list are ordered by type, all GGTT vma are
+                * placed at the head and all ppGTT vma are placed at the tail.
+                * The different types of GGTT vma are unordered between
+                * themselves, use the @vma.tree (which has a defined order
+                * between all VMA) to quickly find an exact match.
+                */
+               struct list_head list;
+
+               /**
+                * @vma.tree: Ordered tree of VMAs backed by this object
+                *
+                * All VMA created for this object are placed in the @vma.tree
+                * for fast retrieval via a binary search in
+                * i915_vma_instance(). They are also added to @vma.list for
+                * easy iteration.
+                */
+               struct rb_root tree;
+       } vma;
+
+       /**
+        * @lut_list: List of vma lookup entries in use for this object.
+        *
+        * If this object is closed, we need to remove all of its VMA from
+        * the fast lookup index in associated contexts; @lut_list provides
+        * this translation from object to context->handles_vma.
+        */
+       struct list_head lut_list;
+
+       /** Stolen memory for this object, instead of being backed by shmem. */
+       struct drm_mm_node *stolen;
+       union {
+               struct rcu_head rcu;
+               struct llist_node freed;
+       };
+
+       /**
+        * Whether the object is currently in the GGTT mmap.
+        */
+       unsigned int userfault_count;
+       struct list_head userfault_link;
+
+       struct list_head batch_pool_link;
+       I915_SELFTEST_DECLARE(struct list_head st_link);
+
+       unsigned long flags;
+
+       /**
+        * Have we taken a reference for the object for incomplete GPU
+        * activity?
+        */
+#define I915_BO_ACTIVE_REF 0
+
+       /*
+        * Is the object to be mapped as read-only to the GPU
+        * Only honoured if hardware has relevant pte bit
+        */
+       unsigned int cache_level:3;
+       unsigned int cache_coherent:2;
+#define I915_BO_CACHE_COHERENT_FOR_READ BIT(0)
+#define I915_BO_CACHE_COHERENT_FOR_WRITE BIT(1)
+       unsigned int cache_dirty:1;
+
+       /**
+        * @read_domains: Read memory domains.
+        *
+        * These monitor which caches contain read/write data related to the
+        * object. When transitioning from one set of domains to another,
+        * the driver is called to ensure that caches are suitably flushed and
+        * invalidated.
+        */
+       u16 read_domains;
+
+       /**
+        * @write_domain: Corresponding unique write memory domain.
+        */
+       u16 write_domain;
+
+       atomic_t frontbuffer_bits;
+       unsigned int frontbuffer_ggtt_origin; /* write once */
+       struct i915_active_request frontbuffer_write;
+
+       /** Current tiling stride for the object, if it's tiled. */
+       unsigned int tiling_and_stride;
+#define FENCE_MINIMUM_STRIDE 128 /* See i915_tiling_ok() */
+#define TILING_MASK (FENCE_MINIMUM_STRIDE - 1)
+#define STRIDE_MASK (~TILING_MASK)
+
+       /** Count of VMA actually bound by this object */
+       unsigned int bind_count;
+       unsigned int active_count;
+       /** Count of how many global VMA are currently pinned for use by HW */
+       unsigned int pin_global;
+
+       struct {
+               struct mutex lock; /* protects the pages and their use */
+               atomic_t pages_pin_count;
+
+               struct sg_table *pages;
+               void *mapping;
+
+               /* TODO: whack some of this into the error state */
+               struct i915_page_sizes {
+                       /**
+                        * The sg mask of the pages sg_table. i.e the mask of
+                        * of the lengths for each sg entry.
+                        */
+                       unsigned int phys;
+
+                       /**
+                        * The gtt page sizes we are allowed to use given the
+                        * sg mask and the supported page sizes. This will
+                        * express the smallest unit we can use for the whole
+                        * object, as well as the larger sizes we may be able
+                        * to use opportunistically.
+                        */
+                       unsigned int sg;
+
+                       /**
+                        * The actual gtt page size usage. Since we can have
+                        * multiple vma associated with this object we need to
+                        * prevent any trampling of state, hence a copy of this
+                        * struct also lives in each vma, therefore the gtt
+                        * value here should only be read/write through the vma.
+                        */
+                       unsigned int gtt;
+               } page_sizes;
+
+               I915_SELFTEST_DECLARE(unsigned int page_mask);
+
+               struct i915_gem_object_page_iter {
+                       struct scatterlist *sg_pos;
+                       unsigned int sg_idx; /* in pages, but 32bit eek! */
+
+                       struct radix_tree_root radix;
+                       struct mutex lock; /* protects this cache */
+               } get_page;
+
+               /**
+                * Element within i915->mm.unbound_list or i915->mm.bound_list,
+                * locked by i915->mm.obj_lock.
+                */
+               struct list_head link;
+
+               /**
+                * Advice: are the backing pages purgeable?
+                */
+               unsigned int madv:2;
+
+               /**
+                * This is set if the object has been written to since the
+                * pages were last acquired.
+                */
+               bool dirty:1;
+
+               /**
+                * This is set if the object has been pinned due to unknown
+                * swizzling.
+                */
+               bool quirked:1;
+       } mm;
+
+       /** Breadcrumb of last rendering to the buffer.
+        * There can only be one writer, but we allow for multiple readers.
+        * If there is a writer that necessarily implies that all other
+        * read requests are complete - but we may only be lazily clearing
+        * the read requests. A read request is naturally the most recent
+        * request on a ring, so we may have two different write and read
+        * requests on one ring where the write request is older than the
+        * read request. This allows for the CPU to read from an active
+        * buffer by only waiting for the write to complete.
+        */
+       struct reservation_object *resv;
+
+       /** References from framebuffers, locks out tiling changes. */
+       unsigned int framebuffer_references;
+
+       /** Record of address bit 17 of each page at last unbind. */
+       unsigned long *bit_17;
+
+       union {
+               struct i915_gem_userptr {
+                       uintptr_t ptr;
+
+                       struct i915_mm_struct *mm;
+                       struct i915_mmu_object *mmu_object;
+                       struct work_struct *work;
+               } userptr;
+
+               unsigned long scratch;
+
+               void *gvt_info;
+       };
+
+       /** for phys allocated objects */
+       struct drm_dma_handle *phys_handle;
+
+       struct reservation_object __builtin_resv;
+};
+
+static inline struct drm_i915_gem_object *
+to_intel_bo(struct drm_gem_object *gem)
+{
+       /* Assert that to_intel_bo(NULL) == NULL */
+       BUILD_BUG_ON(offsetof(struct drm_i915_gem_object, base));
+
+       return container_of(gem, struct drm_i915_gem_object, base);
+}
+
+#endif
index 40e774acc2cd20341904cb989ca8d200136f63f1..01223864237a8fa1cbbc46cc47107442ac0d6f09 100644 (file)
@@ -29,6 +29,7 @@
 #define I915_CMD_HASH_ORDER 9
 
 struct dma_fence;
+struct drm_i915_gem_object;
 struct drm_i915_reg_table;
 struct i915_gem_context;
 struct i915_request;
index 87cc80c7b6269e244c3eb92597a6f7b8394e419f..9a744d39d1cd189b7d8a4d4efab422808b49c639 100644 (file)
@@ -82,7 +82,6 @@
 #include "i915_gem.h"
 #include "i915_gem_context.h"
 #include "i915_gem_fence_reg.h"
-#include "i915_gem_object.h"
 #include "i915_gem_gtt.h"
 #include "i915_gpu_error.h"
 #include "i915_request.h"
@@ -137,6 +136,8 @@ bool i915_error_injected(void);
        __i915_printk(i915, i915_error_injected() ? KERN_DEBUG : KERN_ERR, \
                      fmt, ##__VA_ARGS__)
 
+struct drm_i915_gem_object;
+
 enum hpd_pin {
        HPD_NONE = 0,
        HPD_TV = HPD_NONE,     /* TV is known to be unreliable */
index 56947daaaf65b68c62e9635be51207fddf531f69..feeeeeaa54d8964667a8cc948234e1d8515f307d 100644 (file)
@@ -9,6 +9,7 @@
 
 #include <linux/types.h>
 
+struct drm_i915_gem_object;
 struct intel_engine_cs;
 
 struct i915_gem_batch_pool {
@@ -19,7 +20,7 @@ struct i915_gem_batch_pool {
 void i915_gem_batch_pool_init(struct i915_gem_batch_pool *pool,
                              struct intel_engine_cs *engine);
 void i915_gem_batch_pool_fini(struct i915_gem_batch_pool *pool);
-struct drm_i915_gem_object*
+struct drm_i915_gem_object *
 i915_gem_batch_pool_get(struct i915_gem_batch_pool *pool, size_t size);
 
 #endif /* I915_GEM_BATCH_POOL_H */
index 38496039456b7f3430e70836c24ac3ebd074c7cf..811fa05c0322d07000a73e8d1287cb6fa689f168 100644 (file)
@@ -61,6 +61,7 @@
 
 struct drm_i915_file_private;
 struct drm_i915_fence_reg;
+struct drm_i915_gem_object;
 struct i915_vma;
 
 typedef u32 gen6_pte_t;
index ca93a40c0c878885110e36f675b0ba79c5050a62..3666b0c5f6ca9e6767bbe8e5085201427733c7a4 100644 (file)
 /*
- * Copyright © 2016 Intel Corporation
- *
- * Permission is hereby granted, free of charge, to any person obtaining a
- * copy of this software and associated documentation files (the "Software"),
- * to deal in the Software without restriction, including without limitation
- * the rights to use, copy, modify, merge, publish, distribute, sublicense,
- * and/or sell copies of the Software, and to permit persons to whom the
- * Software is furnished to do so, subject to the following conditions:
- *
- * The above copyright notice and this permission notice (including the next
- * paragraph) shall be included in all copies or substantial portions of the
- * Software.
- *
- * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
- * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
- * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.  IN NO EVENT SHALL
- * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
- * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
- * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS
- * IN THE SOFTWARE.
+ * SPDX-License-Identifier: MIT
  *
+ * Copyright © 2016 Intel Corporation
  */
 
 #ifndef __I915_GEM_OBJECT_H__
 #define __I915_GEM_OBJECT_H__
 
-#include <linux/reservation.h>
-
-#include <drm/drm_vma_manager.h>
 #include <drm/drm_gem.h>
 #include <drm/drm_file.h>
 #include <drm/drm_device.h>
 
 #include <drm/i915_drm.h>
 
-#include "i915_request.h"
-#include "i915_selftest.h"
-
-struct drm_i915_gem_object;
-
-/*
- * struct i915_lut_handle tracks the fast lookups from handle to vma used
- * for execbuf. Although we use a radixtree for that mapping, in order to
- * remove them as the object or context is closed, we need a secondary list
- * and a translation entry (i915_lut_handle).
- */
-struct i915_lut_handle {
-       struct list_head obj_link;
-       struct list_head ctx_link;
-       struct i915_gem_context *ctx;
-       u32 handle;
-};
-
-struct drm_i915_gem_object_ops {
-       unsigned int flags;
-#define I915_GEM_OBJECT_HAS_STRUCT_PAGE        BIT(0)
-#define I915_GEM_OBJECT_IS_SHRINKABLE  BIT(1)
-#define I915_GEM_OBJECT_IS_PROXY       BIT(2)
-#define I915_GEM_OBJECT_ASYNC_CANCEL   BIT(3)
-
-       /* Interface between the GEM object and its backing storage.
-        * get_pages() is called once prior to the use of the associated set
-        * of pages before to binding them into the GTT, and put_pages() is
-        * called after we no longer need them. As we expect there to be
-        * associated cost with migrating pages between the backing storage
-        * and making them available for the GPU (e.g. clflush), we may hold
-        * onto the pages after they are no longer referenced by the GPU
-        * in case they may be used again shortly (for example migrating the
-        * pages to a different memory domain within the GTT). put_pages()
-        * will therefore most likely be called when the object itself is
-        * being released or under memory pressure (where we attempt to
-        * reap pages for the shrinker).
-        */
-       int (*get_pages)(struct drm_i915_gem_object *);
-       void (*put_pages)(struct drm_i915_gem_object *, struct sg_table *);
-
-       int (*pwrite)(struct drm_i915_gem_object *,
-                     const struct drm_i915_gem_pwrite *);
-
-       int (*dmabuf_export)(struct drm_i915_gem_object *);
-       void (*release)(struct drm_i915_gem_object *);
-};
-
-struct drm_i915_gem_object {
-       struct drm_gem_object base;
-
-       const struct drm_i915_gem_object_ops *ops;
-
-       struct {
-               /**
-                * @vma.lock: protect the list/tree of vmas
-                */
-               spinlock_t lock;
-
-               /**
-                * @vma.list: List of VMAs backed by this object
-                *
-                * The VMA on this list are ordered by type, all GGTT vma are
-                * placed at the head and all ppGTT vma are placed at the tail.
-                * The different types of GGTT vma are unordered between
-                * themselves, use the @vma.tree (which has a defined order
-                * between all VMA) to quickly find an exact match.
-                */
-               struct list_head list;
-
-               /**
-                * @vma.tree: Ordered tree of VMAs backed by this object
-                *
-                * All VMA created for this object are placed in the @vma.tree
-                * for fast retrieval via a binary search in
-                * i915_vma_instance(). They are also added to @vma.list for
-                * easy iteration.
-                */
-               struct rb_root tree;
-       } vma;
-
-       /**
-        * @lut_list: List of vma lookup entries in use for this object.
-        *
-        * If this object is closed, we need to remove all of its VMA from
-        * the fast lookup index in associated contexts; @lut_list provides
-        * this translation from object to context->handles_vma.
-        */
-       struct list_head lut_list;
-
-       /** Stolen memory for this object, instead of being backed by shmem. */
-       struct drm_mm_node *stolen;
-       union {
-               struct rcu_head rcu;
-               struct llist_node freed;
-       };
-
-       /**
-        * Whether the object is currently in the GGTT mmap.
-        */
-       unsigned int userfault_count;
-       struct list_head userfault_link;
-
-       struct list_head batch_pool_link;
-       I915_SELFTEST_DECLARE(struct list_head st_link);
-
-       unsigned long flags;
-
-       /**
-        * Have we taken a reference for the object for incomplete GPU
-        * activity?
-        */
-#define I915_BO_ACTIVE_REF 0
-
-       /*
-        * Is the object to be mapped as read-only to the GPU
-        * Only honoured if hardware has relevant pte bit
-        */
-       unsigned int cache_level:3;
-       unsigned int cache_coherent:2;
-#define I915_BO_CACHE_COHERENT_FOR_READ BIT(0)
-#define I915_BO_CACHE_COHERENT_FOR_WRITE BIT(1)
-       unsigned int cache_dirty:1;
-
-       /**
-        * @read_domains: Read memory domains.
-        *
-        * These monitor which caches contain read/write data related to the
-        * object. When transitioning from one set of domains to another,
-        * the driver is called to ensure that caches are suitably flushed and
-        * invalidated.
-        */
-       u16 read_domains;
-
-       /**
-        * @write_domain: Corresponding unique write memory domain.
-        */
-       u16 write_domain;
-
-       atomic_t frontbuffer_bits;
-       unsigned int frontbuffer_ggtt_origin; /* write once */
-       struct i915_active_request frontbuffer_write;
-
-       /** Current tiling stride for the object, if it's tiled. */
-       unsigned int tiling_and_stride;
-#define FENCE_MINIMUM_STRIDE 128 /* See i915_tiling_ok() */
-#define TILING_MASK (FENCE_MINIMUM_STRIDE-1)
-#define STRIDE_MASK (~TILING_MASK)
-
-       /** Count of VMA actually bound by this object */
-       unsigned int bind_count;
-       unsigned int active_count;
-       /** Count of how many global VMA are currently pinned for use by HW */
-       unsigned int pin_global;
-
-       struct {
-               struct mutex lock; /* protects the pages and their use */
-               atomic_t pages_pin_count;
-
-               struct sg_table *pages;
-               void *mapping;
-
-               /* TODO: whack some of this into the error state */
-               struct i915_page_sizes {
-                       /**
-                        * The sg mask of the pages sg_table. i.e the mask of
-                        * of the lengths for each sg entry.
-                        */
-                       unsigned int phys;
-
-                       /**
-                        * The gtt page sizes we are allowed to use given the
-                        * sg mask and the supported page sizes. This will
-                        * express the smallest unit we can use for the whole
-                        * object, as well as the larger sizes we may be able
-                        * to use opportunistically.
-                        */
-                       unsigned int sg;
-
-                       /**
-                        * The actual gtt page size usage. Since we can have
-                        * multiple vma associated with this object we need to
-                        * prevent any trampling of state, hence a copy of this
-                        * struct also lives in each vma, therefore the gtt
-                        * value here should only be read/write through the vma.
-                        */
-                       unsigned int gtt;
-               } page_sizes;
-
-               I915_SELFTEST_DECLARE(unsigned int page_mask);
-
-               struct i915_gem_object_page_iter {
-                       struct scatterlist *sg_pos;
-                       unsigned int sg_idx; /* in pages, but 32bit eek! */
-
-                       struct radix_tree_root radix;
-                       struct mutex lock; /* protects this cache */
-               } get_page;
-
-               /**
-                * Element within i915->mm.unbound_list or i915->mm.bound_list,
-                * locked by i915->mm.obj_lock.
-                */
-               struct list_head link;
-
-               /**
-                * Advice: are the backing pages purgeable?
-                */
-               unsigned int madv:2;
-
-               /**
-                * This is set if the object has been written to since the
-                * pages were last acquired.
-                */
-               bool dirty:1;
-
-               /**
-                * This is set if the object has been pinned due to unknown
-                * swizzling.
-                */
-               bool quirked:1;
-       } mm;
-
-       /** Breadcrumb of last rendering to the buffer.
-        * There can only be one writer, but we allow for multiple readers.
-        * If there is a writer that necessarily implies that all other
-        * read requests are complete - but we may only be lazily clearing
-        * the read requests. A read request is naturally the most recent
-        * request on a ring, so we may have two different write and read
-        * requests on one ring where the write request is older than the
-        * read request. This allows for the CPU to read from an active
-        * buffer by only waiting for the write to complete.
-        */
-       struct reservation_object *resv;
-
-       /** References from framebuffers, locks out tiling changes. */
-       unsigned int framebuffer_references;
-
-       /** Record of address bit 17 of each page at last unbind. */
-       unsigned long *bit_17;
-
-       union {
-               struct i915_gem_userptr {
-                       uintptr_t ptr;
-
-                       struct i915_mm_struct *mm;
-                       struct i915_mmu_object *mmu_object;
-                       struct work_struct *work;
-               } userptr;
-
-               unsigned long scratch;
-
-               void *gvt_info;
-       };
-
-       /** for phys allocated objects */
-       struct drm_dma_handle *phys_handle;
-
-       struct reservation_object __builtin_resv;
-};
-
-static inline struct drm_i915_gem_object *
-to_intel_bo(struct drm_gem_object *gem)
-{
-       /* Assert that to_intel_bo(NULL) == NULL */
-       BUILD_BUG_ON(offsetof(struct drm_i915_gem_object, base));
-
-       return container_of(gem, struct drm_i915_gem_object, base);
-}
+#include "gem/i915_gem_object_types.h"
 
 struct drm_i915_gem_object *i915_gem_object_alloc(void);
 void i915_gem_object_free(struct drm_i915_gem_object *obj);