Will do. Thanks!
Alex
On Mon, Oct 26, 2020 at 12:56 PM Alex Xu (Hello71) <alex_y_xu@xxxxxxxx> wrote:
>
> Excerpts from Alex Deucher's message of October 26, 2020 10:30 am:
> > It looks this was accidently lost in a follow up patch.
> > dc context is large and we don't need contiguous pages.
> >
> > Fixes: e4863f118a7d ("drm/amd/display: Multi display cause system lag on mode change")
> > Signed-off-by: Alex Deucher <alexander.deucher@xxxxxxx>
> > Cc: Aric Cyr <aric.cyr@xxxxxxx>
> > Cc: Alex Xu <alex_y_xu@xxxxxxxx>
> > ---
> > drivers/gpu/drm/amd/display/dc/core/dc.c | 4 ++--
> > 1 file changed, 2 insertions(+), 2 deletions(-)
> >
> > diff --git a/drivers/gpu/drm/amd/display/dc/core/dc.c b/drivers/gpu/drm/amd/display/dc/core/dc.c
> > index 7ff029143722..64da60450fb0 100644
> > --- a/drivers/gpu/drm/amd/display/dc/core/dc.c
> > +++ b/drivers/gpu/drm/amd/display/dc/core/dc.c
> > @@ -1564,8 +1564,8 @@ static void init_state(struct dc *dc, struct dc_state *context)
> >
> > struct dc_state *dc_create_state(struct dc *dc)
> > {
> > - struct dc_state *context = kzalloc(sizeof(struct dc_state),
> > - GFP_KERNEL);
> > + struct dc_state *context = kvzalloc(sizeof(struct dc_state),
> > + GFP_KERNEL);
> >
> > if (!context)
> > return NULL;
> > --
> > 2.25.4
> >
> >
>
> Did a quick test, it seems to resolve the issue for me. If you could
> apply one or both of:
>
> Reported-by: Alex Xu (Hello71) <alex_y_xu@xxxxxxxx>
> Tested-by: Alex Xu (Hello71) <alex_y_xu@xxxxxxxx>
>
> I'd appreciate it, but it's not a big deal. Thanks!
_______________________________________________
amd-gfx mailing list
amd-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/amd-gfx
