Re: ✗ Fi.CI.BUILD: failure for drm/connector: add eld_mutex to protect connector->eld

2024-11-30 Thread Dmitry Baryshkov
On Sat, Nov 30, 2024 at 11:57:47PM +, Patchwork wrote: > == Series Details == > > Series: drm/connector: add eld_mutex to protect connector->eld > URL : https://patchwork.freedesktop.org/series/141958/ > State : failure > > == Summary == > > Error: patch > https://patchwork.freedesktop.or

[PATCH 00/10] drm/connector: add eld_mutex to protect connector->eld

2024-11-30 Thread Dmitry Baryshkov
The connector->eld is accessed by the .get_eld() callback. This access can collide with the drm_edid_to_eld() updating the data at the same time. Add drm_connector.eld_mutex to protect the data from concurrenct access. The individual drivers were just compile tested. I propose to merge the drm_con

✗ Fi.CI.BUILD: failure for drm/connector: add eld_mutex to protect connector->eld

2024-11-30 Thread Patchwork
== Series Details == Series: drm/connector: add eld_mutex to protect connector->eld URL : https://patchwork.freedesktop.org/series/141958/ State : failure == Summary == Error: patch https://patchwork.freedesktop.org/api/1.0/series/141958/revisions/1/mbox/ not applied Applying: drm/connector:

[PATCH 08/10] drm/radeon: use eld_mutex to protect access to connector->eld

2024-11-30 Thread Dmitry Baryshkov
Reading access to connector->eld can happen at the same time the drm_edid_to_eld() updates the data. Take the newly added eld_mutex in order to protect connector->eld from concurrent access. Signed-off-by: Dmitry Baryshkov --- drivers/gpu/drm/radeon/radeon_audio.c | 2 ++ 1 file changed, 2 inser

[PATCH 10/10] drm/vc4: hdmi: use eld_mutex to protect access to connector->eld

2024-11-30 Thread Dmitry Baryshkov
Reading access to connector->eld can happen at the same time the drm_edid_to_eld() updates the data. Take the newly added eld_mutex in order to protect connector->eld from concurrent access. Signed-off-by: Dmitry Baryshkov --- drivers/gpu/drm/vc4/vc4_hdmi.c | 4 ++-- 1 file changed, 2 insertions

[PATCH 03/10] drm/bridge: ite-it66121: use eld_mutex to protect access to connector->eld

2024-11-30 Thread Dmitry Baryshkov
Reading access to connector->eld can happen at the same time the drm_edid_to_eld() updates the data. Take the newly added eld_mutex in order to protect connector->eld from concurrent access. Signed-off-by: Dmitry Baryshkov --- drivers/gpu/drm/bridge/ite-it66121.c | 2 ++ 1 file changed, 2 insert

[PATCH 09/10] drm/sti: hdmi: use eld_mutex to protect access to connector->eld

2024-11-30 Thread Dmitry Baryshkov
Reading access to connector->eld can happen at the same time the drm_edid_to_eld() updates the data. Take the newly added eld_mutex in order to protect connector->eld from concurrent access. Signed-off-by: Dmitry Baryshkov --- drivers/gpu/drm/sti/sti_hdmi.c | 2 ++ 1 file changed, 2 insertions(+

[PATCH 07/10] drm/msm/dp: use eld_mutex to protect access to connector->eld

2024-11-30 Thread Dmitry Baryshkov
Reading access to connector->eld can happen at the same time the drm_edid_to_eld() updates the data. Take the newly added eld_mutex in order to protect connector->eld from concurrent access. Signed-off-by: Dmitry Baryshkov --- drivers/gpu/drm/msm/dp/dp_audio.c | 2 ++ 1 file changed, 2 insertion

[PATCH 06/10] drm/i915/audio: use eld_mutex to protect access to connector->eld

2024-11-30 Thread Dmitry Baryshkov
Reading access to connector->eld can happen at the same time the drm_edid_to_eld() updates the data. Take the newly added eld_mutex in order to protect connector->eld from concurrent access. Signed-off-by: Dmitry Baryshkov --- drivers/gpu/drm/i915/display/intel_audio.c | 3 +++ 1 file changed, 3

[PATCH 05/10] drm/exynos: hdmi: use eld_mutex to protect access to connector->eld

2024-11-30 Thread Dmitry Baryshkov
Reading access to connector->eld can happen at the same time the drm_edid_to_eld() updates the data. Take the newly added eld_mutex in order to protect connector->eld from concurrent access. Signed-off-by: Dmitry Baryshkov --- drivers/gpu/drm/exynos/exynos_hdmi.c | 2 ++ 1 file changed, 2 insert

[PATCH 04/10] drm/amd/display: use eld_mutex to protect access to connector->eld

2024-11-30 Thread Dmitry Baryshkov
Reading access to connector->eld can happen at the same time the drm_edid_to_eld() updates the data. Take the newly added eld_mutex in order to protect connector->eld from concurrent access. Signed-off-by: Dmitry Baryshkov --- drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 2 ++ 1 file chan

[PATCH 02/10] drm/bridge: anx7625: use eld_mutex to protect access to connector->eld

2024-11-30 Thread Dmitry Baryshkov
Reading access to connector->eld can happen at the same time the drm_edid_to_eld() updates the data. Take the newly added eld_mutex in order to protect connector->eld from concurrent access. Signed-off-by: Dmitry Baryshkov --- drivers/gpu/drm/bridge/analogix/anx7625.c | 2 ++ 1 file changed, 2 i

[PATCH 01/10] drm/connector: add mutex to protect ELD from concurrent access

2024-11-30 Thread Dmitry Baryshkov
The connector->eld is accessed by the .get_eld() callback. This access can collide with the drm_edid_to_eld() updating the data at the same time. Add drm_connector.eld_mutex to protect the data from concurrenct access. Individual drivers are not updated (to reduce possible issues while applying the

Re: [PATCH v2] drm/i915/guc: Flush ct receive tasklet during reset preparation

2024-11-30 Thread Rodrigo Vivi
On Mon, Nov 04, 2024 at 01:41:03PM -0800, Zhanjun Dong wrote: > GuC to host communication is interrupt driven, the handling has 3 > parts: interrupt context, tasklet and request queue worker. > During GuC reset prepare, interrupt is disabled before destroy > contexts steps start. The IRQ and worker

Re: [PATCH v2 1/3] drm/i915/dmc_wl: Extract intel_dmc_wl_flush_release_work()

2024-11-30 Thread Rodrigo Vivi
On Fri, Nov 29, 2024 at 01:25:14PM -0500, Rodrigo Vivi wrote: > On Fri, Nov 29, 2024 at 01:37:54PM -0300, Gustavo Sousa wrote: > > We will need to flush the release work from outside in an upcoming > > change. Let's put that into a public interface and call it > > intel_dmc_wl_flush_release_work().

Re: [PATCH 1/4] drm/i915/fb: Relax clear color alignment to 64 bytes

2024-11-30 Thread Xi Ruoyao
On Fri, 2024-11-29 at 08:50 +0200, Ville Syrjala wrote: > From: Ville Syrjälä > > Mesa changed its clear color alignment from 4k to 64 bytes > without informing the kernel side about the change. This > is now likely to cause framebuffer creation to fail. > > The only thing we do with the clear c

Re: [PATCH v10 1/4] drm: Introduce device wedged event

2024-11-30 Thread André Almeida
Hi Raag, Em 28/11/2024 12:37, Raag Jadav escreveu: Introduce device wedged event, which notifies userspace of 'wedged' (hanged/unusable) state of the DRM device through a uevent. This is useful especially in cases where the device is no longer operating as expected and has become unrecoverable f

Re: [PATCH 5/5] drm/i915: Add drm_panic support

2024-11-30 Thread kernel test robot
Hi Jocelyn, kernel test robot noticed the following build warnings: [auto build test WARNING on 44cff6c5b0b17a78bc0b30372bcd816cf6dd282a] url: https://github.com/intel-lab-lkp/linux/commits/Jocelyn-Falempe/drm-i915-fbdev-Add-intel_fbdev_getvaddr/20241130-002536 base

Re: [v2 09/25] drm: Add helper to initialize segmented 1D LUT

2024-11-30 Thread Dmitry Baryshkov
On Tue, Nov 26, 2024 at 06:57:14PM +0530, Uma Shankar wrote: > From: Chaitanya Kumar Borah > > Add helper to initialize 1D segmented LUT > > Signed-off-by: Chaitanya Kumar Borah > Signed-off-by: Uma Shankar > --- > drivers/gpu/drm/drm_colorop.c | 27 ++- > include/drm/

Re: [v2 07/25] drm: Add 1D LUT multi-segmented color op

2024-11-30 Thread Dmitry Baryshkov
On Tue, Nov 26, 2024 at 06:57:12PM +0530, Uma Shankar wrote: > From: Chaitanya Kumar Borah > > Add support for color ops that can be programmed > by 1 dimensional multi segmented Look Up Tables. > > Signed-off-by: Chaitanya Kumar Borah > Signed-off-by: Uma Shankar > --- > drivers/gpu/drm/drm_

Re: [v2 05/25] drm: Add Color ops capability property

2024-11-30 Thread Dmitry Baryshkov
On Tue, Nov 26, 2024 at 06:57:10PM +0530, Uma Shankar wrote: > Add capability property which a colorop can expose it's > hardware's abilities. It's a blob property that can be > filled with respective data structures depending on the > colorop. The user space is expected to read this property > and

Re: [v2 02/25] drm: Add support for 3x3 CTM

2024-11-30 Thread Dmitry Baryshkov
On Tue, Nov 26, 2024 at 06:57:07PM +0530, Uma Shankar wrote: > From: Chaitanya Kumar Borah > > Add support for 3x3 Color Transformation Matrices in Color Pipeline. > > Signed-off-by: Chaitanya Kumar Borah > Signed-off-by: Uma Shankar > --- > drivers/gpu/drm/drm_atomic.c | 3 +++ > drive