Commit baa89ba3 authored by Wambui Karuga's avatar Wambui Karuga Committed by Jani Nikula

drm/i915/gem: initial conversion to new logging macros using coccinelle

First pass of conversion to the new struct drm_based device logging
macros in the drm/i915/gem directory. This conversion was achieved using
the following coccinelle script that transforms based on the existence
of a straightforward struct drm_i915_private device:

@rule1@
identifier fn, T;
@@

fn(struct drm_i915_private *T,...) {
<+...
(
-DRM_INFO(
+drm_info(&T->drm,
...)
|
-DRM_ERROR(
+drm_err(&T->drm,
...)
|
-DRM_WARN(
+drm_warn(&T->drm,
...)
|
-DRM_DEBUG(
+drm_dbg(&T->drm,
...)
|
-DRM_DEBUG_DRIVER(
+drm_dbg(&T->drm,
...)
|
-DRM_DEBUG_KMS(
+drm_dbg_kms(&T->drm,
...)
|
-DRM_DEBUG_ATOMIC(
+drm_dbg_atomic(&T->drm,
...)
)
...+>
}

@rule2@
identifier fn, T;
@@

fn(...) {
...
struct drm_i915_private *T = ...;
<+...
(
-DRM_INFO(
+drm_info(&T->drm,
...)
|
-DRM_ERROR(
+drm_err(&T->drm,
...)
|
-DRM_WARN(
+drm_warn(&T->drm,
...)
|
-DRM_DEBUG(
+drm_dbg(&T->drm,
...)
|
-DRM_DEBUG_KMS(
+drm_dbg_kms(&T->drm,
...)
|
-DRM_DEBUG_DRIVER(
+drm_dbg(&T->drm,
...)
|
-DRM_DEBUG_ATOMIC(
+drm_dbg_atomic(&T->drm,
...)
)
...+>
}

Checkpatch warnings were addressed manually.
Acked-by: default avatarChris Wilson <chris@chris-wilson.co.uk>
Signed-off-by: default avatarWambui Karuga <wambui.karugax@gmail.com>
Signed-off-by: default avatarJani Nikula <jani.nikula@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20200122125750.9737-2-wambui.karugax@gmail.com
parent 9b6320aa
...@@ -708,8 +708,8 @@ i915_gem_create_context(struct drm_i915_private *i915, unsigned int flags) ...@@ -708,8 +708,8 @@ i915_gem_create_context(struct drm_i915_private *i915, unsigned int flags)
ppgtt = i915_ppgtt_create(&i915->gt); ppgtt = i915_ppgtt_create(&i915->gt);
if (IS_ERR(ppgtt)) { if (IS_ERR(ppgtt)) {
DRM_DEBUG_DRIVER("PPGTT setup failed (%ld)\n", drm_dbg(&i915->drm, "PPGTT setup failed (%ld)\n",
PTR_ERR(ppgtt)); PTR_ERR(ppgtt));
context_close(ctx); context_close(ctx);
return ERR_CAST(ppgtt); return ERR_CAST(ppgtt);
} }
...@@ -751,9 +751,9 @@ static void init_contexts(struct i915_gem_contexts *gc) ...@@ -751,9 +751,9 @@ static void init_contexts(struct i915_gem_contexts *gc)
void i915_gem_init__contexts(struct drm_i915_private *i915) void i915_gem_init__contexts(struct drm_i915_private *i915)
{ {
init_contexts(&i915->gem.contexts); init_contexts(&i915->gem.contexts);
DRM_DEBUG_DRIVER("%s context support initialized\n", drm_dbg(&i915->drm, "%s context support initialized\n",
DRIVER_CAPS(i915)->has_logical_contexts ? DRIVER_CAPS(i915)->has_logical_contexts ?
"logical" : "fake"); "logical" : "fake");
} }
void i915_gem_driver_release__contexts(struct drm_i915_private *i915) void i915_gem_driver_release__contexts(struct drm_i915_private *i915)
...@@ -1595,6 +1595,7 @@ static int ...@@ -1595,6 +1595,7 @@ static int
set_engines(struct i915_gem_context *ctx, set_engines(struct i915_gem_context *ctx,
const struct drm_i915_gem_context_param *args) const struct drm_i915_gem_context_param *args)
{ {
struct drm_i915_private *i915 = ctx->i915;
struct i915_context_param_engines __user *user = struct i915_context_param_engines __user *user =
u64_to_user_ptr(args->value); u64_to_user_ptr(args->value);
struct set_engines set = { .ctx = ctx }; struct set_engines set = { .ctx = ctx };
...@@ -1616,8 +1617,8 @@ set_engines(struct i915_gem_context *ctx, ...@@ -1616,8 +1617,8 @@ set_engines(struct i915_gem_context *ctx,
BUILD_BUG_ON(!IS_ALIGNED(sizeof(*user), sizeof(*user->engines))); BUILD_BUG_ON(!IS_ALIGNED(sizeof(*user), sizeof(*user->engines)));
if (args->size < sizeof(*user) || if (args->size < sizeof(*user) ||
!IS_ALIGNED(args->size, sizeof(*user->engines))) { !IS_ALIGNED(args->size, sizeof(*user->engines))) {
DRM_DEBUG("Invalid size for engine array: %d\n", drm_dbg(&i915->drm, "Invalid size for engine array: %d\n",
args->size); args->size);
return -EINVAL; return -EINVAL;
} }
...@@ -1653,8 +1654,9 @@ set_engines(struct i915_gem_context *ctx, ...@@ -1653,8 +1654,9 @@ set_engines(struct i915_gem_context *ctx,
ci.engine_class, ci.engine_class,
ci.engine_instance); ci.engine_instance);
if (!engine) { if (!engine) {
DRM_DEBUG("Invalid engine[%d]: { class:%d, instance:%d }\n", drm_dbg(&i915->drm,
n, ci.engine_class, ci.engine_instance); "Invalid engine[%d]: { class:%d, instance:%d }\n",
n, ci.engine_class, ci.engine_instance);
__free_engines(set.engines, n); __free_engines(set.engines, n);
return -ENOENT; return -ENOENT;
} }
...@@ -2168,8 +2170,9 @@ int i915_gem_context_create_ioctl(struct drm_device *dev, void *data, ...@@ -2168,8 +2170,9 @@ int i915_gem_context_create_ioctl(struct drm_device *dev, void *data,
ext_data.fpriv = file->driver_priv; ext_data.fpriv = file->driver_priv;
if (client_is_banned(ext_data.fpriv)) { if (client_is_banned(ext_data.fpriv)) {
DRM_DEBUG("client %s[%d] banned from creating ctx\n", drm_dbg(&i915->drm,
current->comm, task_pid_nr(current)); "client %s[%d] banned from creating ctx\n",
current->comm, task_pid_nr(current));
return -EIO; return -EIO;
} }
...@@ -2191,7 +2194,7 @@ int i915_gem_context_create_ioctl(struct drm_device *dev, void *data, ...@@ -2191,7 +2194,7 @@ int i915_gem_context_create_ioctl(struct drm_device *dev, void *data,
goto err_ctx; goto err_ctx;
args->ctx_id = id; args->ctx_id = id;
DRM_DEBUG("HW context %d created\n", args->ctx_id); drm_dbg(&i915->drm, "HW context %d created\n", args->ctx_id);
return 0; return 0;
......
...@@ -420,6 +420,7 @@ eb_validate_vma(struct i915_execbuffer *eb, ...@@ -420,6 +420,7 @@ eb_validate_vma(struct i915_execbuffer *eb,
struct drm_i915_gem_exec_object2 *entry, struct drm_i915_gem_exec_object2 *entry,
struct i915_vma *vma) struct i915_vma *vma)
{ {
struct drm_i915_private *i915 = eb->i915;
if (unlikely(entry->flags & eb->invalid_flags)) if (unlikely(entry->flags & eb->invalid_flags))
return -EINVAL; return -EINVAL;
...@@ -443,8 +444,9 @@ eb_validate_vma(struct i915_execbuffer *eb, ...@@ -443,8 +444,9 @@ eb_validate_vma(struct i915_execbuffer *eb,
} }
if (unlikely(vma->exec_flags)) { if (unlikely(vma->exec_flags)) {
DRM_DEBUG("Object [handle %d, index %d] appears more than once in object list\n", drm_dbg(&i915->drm,
entry->handle, (int)(entry - eb->exec)); "Object [handle %d, index %d] appears more than once in object list\n",
entry->handle, (int)(entry - eb->exec));
return -EINVAL; return -EINVAL;
} }
...@@ -1330,6 +1332,7 @@ eb_relocate_entry(struct i915_execbuffer *eb, ...@@ -1330,6 +1332,7 @@ eb_relocate_entry(struct i915_execbuffer *eb,
struct i915_vma *vma, struct i915_vma *vma,
const struct drm_i915_gem_relocation_entry *reloc) const struct drm_i915_gem_relocation_entry *reloc)
{ {
struct drm_i915_private *i915 = eb->i915;
struct i915_vma *target; struct i915_vma *target;
int err; int err;
...@@ -1340,7 +1343,7 @@ eb_relocate_entry(struct i915_execbuffer *eb, ...@@ -1340,7 +1343,7 @@ eb_relocate_entry(struct i915_execbuffer *eb,
/* Validate that the target is in a valid r/w GPU domain */ /* Validate that the target is in a valid r/w GPU domain */
if (unlikely(reloc->write_domain & (reloc->write_domain - 1))) { if (unlikely(reloc->write_domain & (reloc->write_domain - 1))) {
DRM_DEBUG("reloc with multiple write domains: " drm_dbg(&i915->drm, "reloc with multiple write domains: "
"target %d offset %d " "target %d offset %d "
"read %08x write %08x", "read %08x write %08x",
reloc->target_handle, reloc->target_handle,
...@@ -1351,7 +1354,7 @@ eb_relocate_entry(struct i915_execbuffer *eb, ...@@ -1351,7 +1354,7 @@ eb_relocate_entry(struct i915_execbuffer *eb,
} }
if (unlikely((reloc->write_domain | reloc->read_domains) if (unlikely((reloc->write_domain | reloc->read_domains)
& ~I915_GEM_GPU_DOMAINS)) { & ~I915_GEM_GPU_DOMAINS)) {
DRM_DEBUG("reloc with read/write non-GPU domains: " drm_dbg(&i915->drm, "reloc with read/write non-GPU domains: "
"target %d offset %d " "target %d offset %d "
"read %08x write %08x", "read %08x write %08x",
reloc->target_handle, reloc->target_handle,
...@@ -1391,7 +1394,7 @@ eb_relocate_entry(struct i915_execbuffer *eb, ...@@ -1391,7 +1394,7 @@ eb_relocate_entry(struct i915_execbuffer *eb,
/* Check that the relocation address is valid... */ /* Check that the relocation address is valid... */
if (unlikely(reloc->offset > if (unlikely(reloc->offset >
vma->size - (eb->reloc_cache.use_64bit_reloc ? 8 : 4))) { vma->size - (eb->reloc_cache.use_64bit_reloc ? 8 : 4))) {
DRM_DEBUG("Relocation beyond object bounds: " drm_dbg(&i915->drm, "Relocation beyond object bounds: "
"target %d offset %d size %d.\n", "target %d offset %d size %d.\n",
reloc->target_handle, reloc->target_handle,
(int)reloc->offset, (int)reloc->offset,
...@@ -1399,7 +1402,7 @@ eb_relocate_entry(struct i915_execbuffer *eb, ...@@ -1399,7 +1402,7 @@ eb_relocate_entry(struct i915_execbuffer *eb,
return -EINVAL; return -EINVAL;
} }
if (unlikely(reloc->offset & 3)) { if (unlikely(reloc->offset & 3)) {
DRM_DEBUG("Relocation not 4-byte aligned: " drm_dbg(&i915->drm, "Relocation not 4-byte aligned: "
"target %d offset %d.\n", "target %d offset %d.\n",
reloc->target_handle, reloc->target_handle,
(int)reloc->offset); (int)reloc->offset);
...@@ -2075,6 +2078,7 @@ static int eb_parse_pipeline(struct i915_execbuffer *eb, ...@@ -2075,6 +2078,7 @@ static int eb_parse_pipeline(struct i915_execbuffer *eb,
static int eb_parse(struct i915_execbuffer *eb) static int eb_parse(struct i915_execbuffer *eb)
{ {
struct drm_i915_private *i915 = eb->i915;
struct intel_engine_pool_node *pool; struct intel_engine_pool_node *pool;
struct i915_vma *shadow, *trampoline; struct i915_vma *shadow, *trampoline;
unsigned int len; unsigned int len;
...@@ -2090,7 +2094,8 @@ static int eb_parse(struct i915_execbuffer *eb) ...@@ -2090,7 +2094,8 @@ static int eb_parse(struct i915_execbuffer *eb)
* post-scan tampering * post-scan tampering
*/ */
if (!eb->context->vm->has_read_only) { if (!eb->context->vm->has_read_only) {
DRM_DEBUG("Cannot prevent post-scan tampering without RO capable vm\n"); drm_dbg(&i915->drm,
"Cannot prevent post-scan tampering without RO capable vm\n");
return -EINVAL; return -EINVAL;
} }
} else { } else {
...@@ -2371,8 +2376,9 @@ eb_select_legacy_ring(struct i915_execbuffer *eb, ...@@ -2371,8 +2376,9 @@ eb_select_legacy_ring(struct i915_execbuffer *eb,
if (user_ring_id != I915_EXEC_BSD && if (user_ring_id != I915_EXEC_BSD &&
(args->flags & I915_EXEC_BSD_MASK)) { (args->flags & I915_EXEC_BSD_MASK)) {
DRM_DEBUG("execbuf with non bsd ring but with invalid " drm_dbg(&i915->drm,
"bsd dispatch flags: %d\n", (int)(args->flags)); "execbuf with non bsd ring but with invalid "
"bsd dispatch flags: %d\n", (int)(args->flags));
return -1; return -1;
} }
...@@ -2386,8 +2392,9 @@ eb_select_legacy_ring(struct i915_execbuffer *eb, ...@@ -2386,8 +2392,9 @@ eb_select_legacy_ring(struct i915_execbuffer *eb,
bsd_idx >>= I915_EXEC_BSD_SHIFT; bsd_idx >>= I915_EXEC_BSD_SHIFT;
bsd_idx--; bsd_idx--;
} else { } else {
DRM_DEBUG("execbuf with unknown bsd ring: %u\n", drm_dbg(&i915->drm,
bsd_idx); "execbuf with unknown bsd ring: %u\n",
bsd_idx);
return -1; return -1;
} }
...@@ -2395,7 +2402,8 @@ eb_select_legacy_ring(struct i915_execbuffer *eb, ...@@ -2395,7 +2402,8 @@ eb_select_legacy_ring(struct i915_execbuffer *eb,
} }
if (user_ring_id >= ARRAY_SIZE(user_ring_map)) { if (user_ring_id >= ARRAY_SIZE(user_ring_map)) {
DRM_DEBUG("execbuf with unknown ring: %u\n", user_ring_id); drm_dbg(&i915->drm, "execbuf with unknown ring: %u\n",
user_ring_id);
return -1; return -1;
} }
...@@ -2669,13 +2677,14 @@ i915_gem_do_execbuffer(struct drm_device *dev, ...@@ -2669,13 +2677,14 @@ i915_gem_do_execbuffer(struct drm_device *dev,
} }
if (unlikely(*eb.batch->exec_flags & EXEC_OBJECT_WRITE)) { if (unlikely(*eb.batch->exec_flags & EXEC_OBJECT_WRITE)) {
DRM_DEBUG("Attempting to use self-modifying batch buffer\n"); drm_dbg(&i915->drm,
"Attempting to use self-modifying batch buffer\n");
err = -EINVAL; err = -EINVAL;
goto err_vma; goto err_vma;
} }
if (eb.batch_start_offset > eb.batch->size || if (eb.batch_start_offset > eb.batch->size ||
eb.batch_len > eb.batch->size - eb.batch_start_offset) { eb.batch_len > eb.batch->size - eb.batch_start_offset) {
DRM_DEBUG("Attempting to use out-of-bounds batch\n"); drm_dbg(&i915->drm, "Attempting to use out-of-bounds batch\n");
err = -EINVAL; err = -EINVAL;
goto err_vma; goto err_vma;
} }
......
...@@ -110,8 +110,11 @@ static int i915_adjust_stolen(struct drm_i915_private *i915, ...@@ -110,8 +110,11 @@ static int i915_adjust_stolen(struct drm_i915_private *i915,
if (stolen[0].start != stolen[1].start || if (stolen[0].start != stolen[1].start ||
stolen[0].end != stolen[1].end) { stolen[0].end != stolen[1].end) {
DRM_DEBUG_DRIVER("GTT within stolen memory at %pR\n", &ggtt_res); drm_dbg(&i915->drm,
DRM_DEBUG_DRIVER("Stolen memory adjusted to %pR\n", dsm); "GTT within stolen memory at %pR\n",
&ggtt_res);
drm_dbg(&i915->drm, "Stolen memory adjusted to %pR\n",
dsm);
} }
} }
...@@ -142,8 +145,9 @@ static int i915_adjust_stolen(struct drm_i915_private *i915, ...@@ -142,8 +145,9 @@ static int i915_adjust_stolen(struct drm_i915_private *i915,
* range. Apparently this works. * range. Apparently this works.
*/ */
if (!r && !IS_GEN(i915, 3)) { if (!r && !IS_GEN(i915, 3)) {
DRM_ERROR("conflict detected with stolen region: %pR\n", drm_err(&i915->drm,
dsm); "conflict detected with stolen region: %pR\n",
dsm);
return -EBUSY; return -EBUSY;
} }
...@@ -171,8 +175,8 @@ static void g4x_get_stolen_reserved(struct drm_i915_private *i915, ...@@ -171,8 +175,8 @@ static void g4x_get_stolen_reserved(struct drm_i915_private *i915,
ELK_STOLEN_RESERVED); ELK_STOLEN_RESERVED);
resource_size_t stolen_top = i915->dsm.end + 1; resource_size_t stolen_top = i915->dsm.end + 1;
DRM_DEBUG_DRIVER("%s_STOLEN_RESERVED = %08x\n", drm_dbg(&i915->drm, "%s_STOLEN_RESERVED = %08x\n",
IS_GM45(i915) ? "CTG" : "ELK", reg_val); IS_GM45(i915) ? "CTG" : "ELK", reg_val);
if ((reg_val & G4X_STOLEN_RESERVED_ENABLE) == 0) if ((reg_val & G4X_STOLEN_RESERVED_ENABLE) == 0)
return; return;
...@@ -202,7 +206,7 @@ static void gen6_get_stolen_reserved(struct drm_i915_private *i915, ...@@ -202,7 +206,7 @@ static void gen6_get_stolen_reserved(struct drm_i915_private *i915,
{ {
u32 reg_val = intel_uncore_read(uncore, GEN6_STOLEN_RESERVED); u32 reg_val = intel_uncore_read(uncore, GEN6_STOLEN_RESERVED);
DRM_DEBUG_DRIVER("GEN6_STOLEN_RESERVED = %08x\n", reg_val); drm_dbg(&i915->drm, "GEN6_STOLEN_RESERVED = %08x\n", reg_val);
if (!(reg_val & GEN6_STOLEN_RESERVED_ENABLE)) if (!(reg_val & GEN6_STOLEN_RESERVED_ENABLE))
return; return;
...@@ -236,7 +240,7 @@ static void vlv_get_stolen_reserved(struct drm_i915_private *i915, ...@@ -236,7 +240,7 @@ static void vlv_get_stolen_reserved(struct drm_i915_private *i915,
u32 reg_val = intel_uncore_read(uncore, GEN6_STOLEN_RESERVED); u32 reg_val = intel_uncore_read(uncore, GEN6_STOLEN_RESERVED);
resource_size_t stolen_top = i915->dsm.end + 1; resource_size_t stolen_top = i915->dsm.end + 1;
DRM_DEBUG_DRIVER("GEN6_STOLEN_RESERVED = %08x\n", reg_val); drm_dbg(&i915->drm, "GEN6_STOLEN_RESERVED = %08x\n", reg_val);
if (!(reg_val & GEN6_STOLEN_RESERVED_ENABLE)) if (!(reg_val & GEN6_STOLEN_RESERVED_ENABLE))
return; return;
...@@ -264,7 +268,7 @@ static void gen7_get_stolen_reserved(struct drm_i915_private *i915, ...@@ -264,7 +268,7 @@ static void gen7_get_stolen_reserved(struct drm_i915_private *i915,
{ {
u32 reg_val = intel_uncore_read(uncore, GEN6_STOLEN_RESERVED); u32 reg_val = intel_uncore_read(uncore, GEN6_STOLEN_RESERVED);
DRM_DEBUG_DRIVER("GEN6_STOLEN_RESERVED = %08x\n", reg_val); drm_dbg(&i915->drm, "GEN6_STOLEN_RESERVED = %08x\n", reg_val);
if (!(reg_val & GEN6_STOLEN_RESERVED_ENABLE)) if (!(reg_val & GEN6_STOLEN_RESERVED_ENABLE))
return; return;
...@@ -291,7 +295,7 @@ static void chv_get_stolen_reserved(struct drm_i915_private *i915, ...@@ -291,7 +295,7 @@ static void chv_get_stolen_reserved(struct drm_i915_private *i915,
{ {
u32 reg_val = intel_uncore_read(uncore, GEN6_STOLEN_RESERVED); u32 reg_val = intel_uncore_read(uncore, GEN6_STOLEN_RESERVED);
DRM_DEBUG_DRIVER("GEN6_STOLEN_RESERVED = %08x\n", reg_val); drm_dbg(&i915->drm, "GEN6_STOLEN_RESERVED = %08x\n", reg_val);
if (!(reg_val & GEN6_STOLEN_RESERVED_ENABLE)) if (!(reg_val & GEN6_STOLEN_RESERVED_ENABLE))
return; return;
...@@ -325,7 +329,7 @@ static void bdw_get_stolen_reserved(struct drm_i915_private *i915, ...@@ -325,7 +329,7 @@ static void bdw_get_stolen_reserved(struct drm_i915_private *i915,
u32 reg_val = intel_uncore_read(uncore, GEN6_STOLEN_RESERVED); u32 reg_val = intel_uncore_read(uncore, GEN6_STOLEN_RESERVED);
resource_size_t stolen_top = i915->dsm.end + 1; resource_size_t stolen_top = i915->dsm.end + 1;
DRM_DEBUG_DRIVER("GEN6_STOLEN_RESERVED = %08x\n", reg_val); drm_dbg(&i915->drm, "GEN6_STOLEN_RESERVED = %08x\n", reg_val);
if (!(reg_val & GEN6_STOLEN_RESERVED_ENABLE)) if (!(reg_val & GEN6_STOLEN_RESERVED_ENABLE))
return; return;
...@@ -344,7 +348,7 @@ static void icl_get_stolen_reserved(struct drm_i915_private *i915, ...@@ -344,7 +348,7 @@ static void icl_get_stolen_reserved(struct drm_i915_private *i915,
{ {
u64 reg_val = intel_uncore_read64(uncore, GEN6_STOLEN_RESERVED); u64 reg_val = intel_uncore_read64(uncore, GEN6_STOLEN_RESERVED);
DRM_DEBUG_DRIVER("GEN6_STOLEN_RESERVED = 0x%016llx\n", reg_val); drm_dbg(&i915->drm, "GEN6_STOLEN_RESERVED = 0x%016llx\n", reg_val);
*base = reg_val & GEN11_STOLEN_RESERVED_ADDR_MASK; *base = reg_val & GEN11_STOLEN_RESERVED_ADDR_MASK;
...@@ -455,8 +459,9 @@ static int i915_gem_init_stolen(struct drm_i915_private *i915) ...@@ -455,8 +459,9 @@ static int i915_gem_init_stolen(struct drm_i915_private *i915)
* it likely means we failed to read the registers correctly. * it likely means we failed to read the registers correctly.
*/ */
if (!reserved_base) { if (!reserved_base) {
DRM_ERROR("inconsistent reservation %pa + %pa; ignoring\n", drm_err(&i915->drm,
&reserved_base, &reserved_size); "inconsistent reservation %pa + %pa; ignoring\n",
&reserved_base, &reserved_size);
reserved_base = stolen_top; reserved_base = stolen_top;
reserved_size = 0; reserved_size = 0;
} }
...@@ -465,8 +470,9 @@ static int i915_gem_init_stolen(struct drm_i915_private *i915) ...@@ -465,8 +470,9 @@ static int i915_gem_init_stolen(struct drm_i915_private *i915)
(struct resource)DEFINE_RES_MEM(reserved_base, reserved_size); (struct resource)DEFINE_RES_MEM(reserved_base, reserved_size);
if (!resource_contains(&i915->dsm, &i915->dsm_reserved)) { if (!resource_contains(&i915->dsm, &i915->dsm_reserved)) {
DRM_ERROR("Stolen reserved area %pR outside stolen memory %pR\n", drm_err(&i915->drm,
&i915->dsm_reserved, &i915->dsm); "Stolen reserved area %pR outside stolen memory %pR\n",
&i915->dsm_reserved, &i915->dsm);
return 0; return 0;
} }
...@@ -474,9 +480,10 @@ static int i915_gem_init_stolen(struct drm_i915_private *i915) ...@@ -474,9 +480,10 @@ static int i915_gem_init_stolen(struct drm_i915_private *i915)
* memory, so just consider the start. */ * memory, so just consider the start. */
reserved_total = stolen_top - reserved_base; reserved_total = stolen_top - reserved_base;
DRM_DEBUG_DRIVER("Memory reserved for graphics device: %lluK, usable: %lluK\n", drm_dbg(&i915->drm,
(u64)resource_size(&i915->dsm) >> 10, "Memory reserved for graphics device: %lluK, usable: %lluK\n",
((u64)resource_size(&i915->dsm) - reserved_total) >> 10); (u64)resource_size(&i915->dsm) >> 10,
((u64)resource_size(&i915->dsm) - reserved_total) >> 10);
i915->stolen_usable_size = i915->stolen_usable_size =
resource_size(&i915->dsm) - reserved_total; resource_size(&i915->dsm) - reserved_total;
...@@ -692,8 +699,9 @@ i915_gem_object_create_stolen_for_preallocated(struct drm_i915_private *i915, ...@@ -692,8 +699,9 @@ i915_gem_object_create_stolen_for_preallocated(struct drm_i915_private *i915,
if (!drm_mm_initialized(&i915->mm.stolen)) if (!drm_mm_initialized(&i915->mm.stolen))
return ERR_PTR(-ENODEV); return ERR_PTR(-ENODEV);
DRM_DEBUG_DRIVER("creating preallocated stolen object: stolen_offset=%pa, gtt_offset=%pa, size=%pa\n", drm_dbg(&i915->drm,
&stolen_offset, &gtt_offset, &size); "creating preallocated stolen object: stolen_offset=%pa, gtt_offset=%pa, size=%pa\n",
&stolen_offset, &gtt_offset, &size);
/* KISS and expect everything to be page-aligned */ /* KISS and expect everything to be page-aligned */
if (drm_WARN_ON(&i915->drm, size == 0) || if (drm_WARN_ON(&i915->drm, size == 0) ||
...@@ -712,14 +720,14 @@ i915_gem_object_create_stolen_for_preallocated(struct drm_i915_private *i915, ...@@ -712,14 +720,14 @@ i915_gem_object_create_stolen_for_preallocated(struct drm_i915_private *i915,
ret = drm_mm_reserve_node(&i915->mm.stolen, stolen); ret = drm_mm_reserve_node(&i915->mm.stolen, stolen);
mutex_unlock(&i915->mm.stolen_lock); mutex_unlock(&i915->mm.stolen_lock);
if (ret) { if (ret) {
DRM_DEBUG_DRIVER("failed to allocate stolen space\n"); drm_dbg(&i915->drm, "failed to allocate stolen space\n");
kfree(stolen); kfree(stolen);
return ERR_PTR(ret); return ERR_PTR(ret);
} }
obj = __i915_gem_object_create_stolen(mem, stolen); obj = __i915_gem_object_create_stolen(mem, stolen);
if (IS_ERR(obj)) { if (IS_ERR(obj)) {
DRM_DEBUG_DRIVER("failed to allocate stolen object\n"); drm_dbg(&i915->drm, "failed to allocate stolen object\n");
i915_gem_stolen_remove_node(i915, stolen); i915_gem_stolen_remove_node(i915, stolen);
kfree(stolen); kfree(stolen);
return obj; return obj;
...@@ -749,7 +757,7 @@ i915_gem_object_create_stolen_for_preallocated(struct drm_i915_private *i915, ...@@ -749,7 +757,7 @@ i915_gem_object_create_stolen_for_preallocated(struct drm_i915_private *i915,
size, gtt_offset, obj->cache_level, size, gtt_offset, obj->cache_level,
0); 0);
if (ret) { if (ret) {
DRM_DEBUG_DRIVER("failed to allocate stolen GTT space\n"); drm_dbg(&i915->drm, "failed to allocate stolen GTT space\n");
mutex_unlock(&ggtt->vm.mutex); mutex_unlock(&ggtt->vm.mutex);
goto err_pages; goto err_pages;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment