lkml.org 
[lkml]   [2020]   [May]   [12]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[RFC 11/17] drm/amdgpu: DC also loves to allocate stuff where it shouldn't
    Date
    Not going to bother with a complete&pretty commit message, just
    offending backtrace:

    kvmalloc_node+0x47/0x80
    dc_create_state+0x1f/0x60 [amdgpu]
    dc_commit_state+0xcb/0x9b0 [amdgpu]
    amdgpu_dm_atomic_commit_tail+0xd31/0x2010 [amdgpu]
    commit_tail+0xa4/0x140 [drm_kms_helper]
    drm_atomic_helper_commit+0x152/0x180 [drm_kms_helper]
    drm_client_modeset_commit_atomic+0x1ea/0x250 [drm]
    drm_client_modeset_commit_locked+0x55/0x190 [drm]
    drm_client_modeset_commit+0x24/0x40 [drm]

    v2: Found more in DC code, I'm just going to pile them all up.

    Cc: linux-media@vger.kernel.org
    Cc: linaro-mm-sig@lists.linaro.org
    Cc: linux-rdma@vger.kernel.org
    Cc: amd-gfx@lists.freedesktop.org
    Cc: intel-gfx@lists.freedesktop.org
    Cc: Chris Wilson <chris@chris-wilson.co.uk>
    Cc: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
    Cc: Christian König <christian.koenig@amd.com>
    Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
    ---
    drivers/gpu/drm/amd/amdgpu/atom.c | 2 +-
    drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 2 +-
    drivers/gpu/drm/amd/display/dc/core/dc.c | 4 +++-
    3 files changed, 5 insertions(+), 3 deletions(-)

    diff --git a/drivers/gpu/drm/amd/amdgpu/atom.c b/drivers/gpu/drm/amd/amdgpu/atom.c
    index 4cfc786699c7..1b0c674fab25 100644
    --- a/drivers/gpu/drm/amd/amdgpu/atom.c
    +++ b/drivers/gpu/drm/amd/amdgpu/atom.c
    @@ -1226,7 +1226,7 @@ static int amdgpu_atom_execute_table_locked(struct atom_context *ctx, int index,
    ectx.abort = false;
    ectx.last_jump = 0;
    if (ws)
    - ectx.ws = kcalloc(4, ws, GFP_KERNEL);
    + ectx.ws = kcalloc(4, ws, GFP_ATOMIC);
    else
    ectx.ws = NULL;

    diff --git a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
    index 4469a8c96b08..9bfaa4cad483 100644
    --- a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
    +++ b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
    @@ -6622,7 +6622,7 @@ static void amdgpu_dm_commit_planes(struct drm_atomic_state *state,
    struct dc_stream_update stream_update;
    } *bundle;

    - bundle = kzalloc(sizeof(*bundle), GFP_KERNEL);
    + bundle = kzalloc(sizeof(*bundle), GFP_ATOMIC);

    if (!bundle) {
    dm_error("Failed to allocate update bundle\n");
    diff --git a/drivers/gpu/drm/amd/display/dc/core/dc.c b/drivers/gpu/drm/amd/display/dc/core/dc.c
    index 401d1c66a411..a37a32442a5a 100644
    --- a/drivers/gpu/drm/amd/display/dc/core/dc.c
    +++ b/drivers/gpu/drm/amd/display/dc/core/dc.c
    @@ -1416,8 +1416,10 @@ bool dc_post_update_surfaces_to_stream(struct dc *dc)

    struct dc_state *dc_create_state(struct dc *dc)
    {
    + /* No you really cant allocate random crap here this late in
    + * atomic_commit_tail. */
    struct dc_state *context = kvzalloc(sizeof(struct dc_state),
    - GFP_KERNEL);
    + GFP_ATOMIC);

    if (!context)
    return NULL;
    --
    2.26.2
    \
     
     \ /
      Last update: 2020-05-12 11:01    [W:5.933 / U:0.064 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site