Commit b7358132 authored by Alex Deucher's avatar Alex Deucher

drm/amd/display: use vmalloc for struct dc_state

This is a big structure so use vmalloc as malloc can
fail when there is memory pressure.

Closes: https://gitlab.freedesktop.org/drm/amd/-/issues/3454Reviewed-by: default avatarMario Limonciello <mario.limonciello@amd.com>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent 15eb8573
...@@ -23,6 +23,7 @@ ...@@ -23,6 +23,7 @@
* *
*/ */
#include <linux/vmalloc.h>
#include <drm/display/drm_dp_helper.h> #include <drm/display/drm_dp_helper.h>
#include <drm/display/drm_dp_mst_helper.h> #include <drm/display/drm_dp_mst_helper.h>
#include <drm/drm_atomic.h> #include <drm/drm_atomic.h>
...@@ -1491,9 +1492,10 @@ int pre_validate_dsc(struct drm_atomic_state *state, ...@@ -1491,9 +1492,10 @@ int pre_validate_dsc(struct drm_atomic_state *state,
* from dm_state->context. * from dm_state->context.
*/ */
local_dc_state = kmemdup(dm_state->context, sizeof(struct dc_state), GFP_KERNEL); local_dc_state = vmalloc(sizeof(struct dc_state));
if (!local_dc_state) if (!local_dc_state)
return -ENOMEM; return -ENOMEM;
memcpy(local_dc_state, dm_state->context, sizeof(struct dc_state));
for (i = 0; i < local_dc_state->stream_count; i++) { for (i = 0; i < local_dc_state->stream_count; i++) {
struct dc_stream_state *stream = dm_state->context->streams[i]; struct dc_stream_state *stream = dm_state->context->streams[i];
...@@ -1563,7 +1565,7 @@ int pre_validate_dsc(struct drm_atomic_state *state, ...@@ -1563,7 +1565,7 @@ int pre_validate_dsc(struct drm_atomic_state *state,
dc_stream_release(local_dc_state->streams[i]); dc_stream_release(local_dc_state->streams[i]);
} }
kfree(local_dc_state); vfree(local_dc_state);
return ret; return ret;
} }
......
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