Re: [Freedreno] [PATCH 2/2] drm/msm: make msm_disp_state transient data struct

2021-04-26 Thread abhinavk
On 2021-04-26 15:14, Dmitry Baryshkov wrote: On Tue, 27 Apr 2021 at 01:03, wrote: On 2021-04-26 14:23, Dmitry Baryshkov wrote: > On 26/04/2021 23:50, abhin...@codeaurora.org wrote: >> On 2021-04-25 09:08, Dmitry Baryshkov wrote: >>> Instead of allocating snapshotting structure at the driver pr

Re: [Freedreno] [PATCH 2/2] drm/msm: make msm_disp_state transient data struct

2021-04-26 Thread Dmitry Baryshkov
On Tue, 27 Apr 2021 at 01:03, wrote: > > On 2021-04-26 14:23, Dmitry Baryshkov wrote: > > On 26/04/2021 23:50, abhin...@codeaurora.org wrote: > >> On 2021-04-25 09:08, Dmitry Baryshkov wrote: > >>> Instead of allocating snapshotting structure at the driver probe time > >>> and later handling concu

Re: [Freedreno] [PATCH 2/2] drm/msm: make msm_disp_state transient data struct

2021-04-26 Thread abhinavk
On 2021-04-26 14:23, Dmitry Baryshkov wrote: On 26/04/2021 23:50, abhin...@codeaurora.org wrote: On 2021-04-25 09:08, Dmitry Baryshkov wrote: Instead of allocating snapshotting structure at the driver probe time and later handling concurrent access, actual state, etc, make msm_disp_state transi

Re: [Freedreno] [PATCH 2/2] drm/msm: make msm_disp_state transient data struct

2021-04-26 Thread Dmitry Baryshkov
On 26/04/2021 23:50, abhin...@codeaurora.org wrote: On 2021-04-25 09:08, Dmitry Baryshkov wrote: Instead of allocating snapshotting structure at the driver probe time and later handling concurrent access, actual state, etc, make msm_disp_state transient struct. Allocate one when snapshotting hap

Re: [Freedreno] [PATCH 2/2] drm/msm: make msm_disp_state transient data struct

2021-04-26 Thread abhinavk
On 2021-04-25 09:08, Dmitry Baryshkov wrote: Instead of allocating snapshotting structure at the driver probe time and later handling concurrent access, actual state, etc, make msm_disp_state transient struct. Allocate one when snapshotting happens and free it after coredump data is read by users

[Freedreno] [PATCH 2/2] drm/msm: make msm_disp_state transient data struct

2021-04-25 Thread Dmitry Baryshkov
Instead of allocating snapshotting structure at the driver probe time and later handling concurrent access, actual state, etc, make msm_disp_state transient struct. Allocate one when snapshotting happens and free it after coredump data is read by userspace. Signed-off-by: Dmitry Baryshkov --- dr