diff options
| author | Tim Gardner <tim.gardner@canonical.com> | 2012-12-07 07:54:26 -0700 |
|---|---|---|
| committer | Daniel Vetter <daniel.vetter@ffwll.ch> | 2012-12-10 10:13:35 +0100 |
| commit | 3ac18232946aacf0a9807c3143f8449ed4aa68f4 (patch) | |
| tree | e84d135a791f856002505e6ac039a28db42c64a4 /drivers/gpu/drm/i915/i915_gem.c | |
| parent | 378a6a77ae59d3312627b996ded94e23166d9e63 (diff) | |
| download | op-kernel-dev-3ac18232946aacf0a9807c3143f8449ed4aa68f4.zip op-kernel-dev-3ac18232946aacf0a9807c3143f8449ed4aa68f4.tar.gz | |
i915: intel_set_mode: Reduce stack allocation from 500 bytes to 2 pointers
smatch warning:
drivers/gpu/drm/i915/intel_display.c:7019 intel_set_mode() warn: function puts
500 bytes on stack
Refactor so that saved_mode and saved_hwmode are dynamically allocated as opposed
to being automatic variables. 500 bytes seems like it could run the potential for blowing
the kernel stack.
Cc: Daniel Vetter <daniel.vetter@ffwll.ch>
Cc: David Airlie <airlied@linux.ie>
Cc: dri-devel@lists.freedesktop.org
Signed-off-by: Tim Gardner <tim.gardner@canonical.com>
Reviewed-by: Jani Nikula <jani.nikula@intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Diffstat (limited to 'drivers/gpu/drm/i915/i915_gem.c')
0 files changed, 0 insertions, 0 deletions
