Skip to content

Commit

Permalink
drm/i915/selftests: Move local mock_ggtt allocations to the heap
Browse files Browse the repository at this point in the history
This struct appears quite large and pushes our stack frame over
1024 bytes -- too high for conservative setups. So move the mock_ggtt
struct to the heap.

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Cc: Matthew Auld <matthew.auld@intel.com>
Reviewed-by: Matthew Auld <matthew.auld@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20190217202518.24730-1-chris@chris-wilson.co.uk
  • Loading branch information
Chris Wilson committed Feb 17, 2019
1 parent 2a4a275 commit 83e3a21
Show file tree
Hide file tree
Showing 2 changed files with 22 additions and 10 deletions.
16 changes: 11 additions & 5 deletions drivers/gpu/drm/i915/selftests/i915_gem_gtt.c
Original file line number Diff line number Diff line change
Expand Up @@ -1681,25 +1681,31 @@ int i915_gem_gtt_mock_selftests(void)
SUBTEST(igt_gtt_insert),
};
struct drm_i915_private *i915;
struct i915_ggtt ggtt;
struct i915_ggtt *ggtt;
int err;

i915 = mock_gem_device();
if (!i915)
return -ENOMEM;

mock_init_ggtt(i915, &ggtt);
ggtt = kmalloc(sizeof(*ggtt), GFP_KERNEL);
if (!ggtt) {
err = -ENOMEM;
goto out_put;
}
mock_init_ggtt(i915, ggtt);

mutex_lock(&i915->drm.struct_mutex);
err = i915_subtests(tests, &ggtt);
err = i915_subtests(tests, ggtt);
mock_device_flush(i915);
mutex_unlock(&i915->drm.struct_mutex);

i915_gem_drain_freed_objects(i915);

mock_fini_ggtt(&ggtt);
mock_fini_ggtt(ggtt);
kfree(ggtt);
out_put:
drm_dev_put(&i915->drm);

return err;
}

Expand Down
16 changes: 11 additions & 5 deletions drivers/gpu/drm/i915/selftests/i915_vma.c
Original file line number Diff line number Diff line change
Expand Up @@ -725,24 +725,30 @@ int i915_vma_mock_selftests(void)
SUBTEST(igt_vma_partial),
};
struct drm_i915_private *i915;
struct i915_ggtt ggtt;
struct i915_ggtt *ggtt;
int err;

i915 = mock_gem_device();
if (!i915)
return -ENOMEM;

mock_init_ggtt(i915, &ggtt);
ggtt = kmalloc(sizeof(*ggtt), GFP_KERNEL);
if (!ggtt) {
err = -ENOMEM;
goto out_put;
}
mock_init_ggtt(i915, ggtt);

mutex_lock(&i915->drm.struct_mutex);
err = i915_subtests(tests, &ggtt);
err = i915_subtests(tests, ggtt);
mock_device_flush(i915);
mutex_unlock(&i915->drm.struct_mutex);

i915_gem_drain_freed_objects(i915);

mock_fini_ggtt(&ggtt);
mock_fini_ggtt(ggtt);
kfree(ggtt);
out_put:
drm_dev_put(&i915->drm);

return err;
}

0 comments on commit 83e3a21

Please sign in to comment.