| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-Id: <20251224-limit-infoframes-2-v3-0-7fd3cacfefed@oss.qualcomm.com>
Date: Wed, 24 Dec 2025 03:02:49 +0200
From: Dmitry Baryshkov <dmitry.baryshkov@....qualcomm.com>
To: Maarten Lankhorst <maarten.lankhorst@...ux.intel.com>,
Maxime Ripard <mripard@...nel.org>,
Thomas Zimmermann <tzimmermann@...e.de>,
David Airlie <airlied@...il.com>, Simona Vetter <simona@...ll.ch>,
Dave Stevenson <dave.stevenson@...pberrypi.com>,
Maíra Canal <mcanal@...lia.com>,
Raspberry Pi Kernel Maintenance <kernel-list@...pberrypi.com>,
Chen-Yu Tsai <wens@...nel.org>,
Jernej Skrabec <jernej.skrabec@...il.com>,
Samuel Holland <samuel@...lland.org>,
Andrzej Hajda <andrzej.hajda@...el.com>,
Neil Armstrong <neil.armstrong@...aro.org>,
Robert Foss <rfoss@...nel.org>,
Laurent Pinchart <Laurent.pinchart@...asonboard.com>,
Jonas Karlman <jonas@...boo.se>, Liu Ying <victor.liu@....com>,
Chun-Kuang Hu <chunkuang.hu@...nel.org>,
Philipp Zabel <p.zabel@...gutronix.de>,
Matthias Brugger <matthias.bgg@...il.com>,
AngeloGioacchino Del Regno <angelogioacchino.delregno@...labora.com>,
Rob Clark <robin.clark@....qualcomm.com>,
Dmitry Baryshkov <lumag@...nel.org>,
Abhinav Kumar <abhinav.kumar@...ux.dev>,
Jessica Zhang <jesszhan0024@...il.com>, Sean Paul <sean@...rly.run>,
Marijn Suijten <marijn.suijten@...ainline.org>,
Sandy Huang <hjc@...k-chips.com>,
Heiko Stübner <heiko@...ech.de>,
Andy Yan <andy.yan@...k-chips.com>
Cc: dri-devel@...ts.freedesktop.org, linux-kernel@...r.kernel.org,
linux-arm-kernel@...ts.infradead.org, linux-sunxi@...ts.linux.dev,
linux-mediatek@...ts.infradead.org, linux-arm-msm@...r.kernel.org,
freedreno@...ts.freedesktop.org, linux-rockchip@...ts.infradead.org
Subject: [PATCH v3 00/10] drm/connector: hdmi: limit infoframes per driver
capabilities, second approach
It's not uncommon for the particular device to support only a subset of
HDMI InfoFrames. Currently it's mostly ignored by the framework: it
calls write_infoframe() / clear_infoframe() callbacks for all frames and
expects them to return success even if the InfoFrame is not supported.
Sort that out, making sure that all interfaces are consistent:
- split function interfaces, having a pair of callbacks per each
InfoFrame type.
- write_infoframe() / clear_infoframe() functions return -EOPNOTSUPP
for unsupported InfoFrames.
- don't create and enable unsupported InfoFrames.
- limit debugfs to provide supported InfoFrames only.
Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@....qualcomm.com>
---
Changes in v3:
- Added DRM KUnit tests verifying InfoFrames behaviour (Maxime)
- Reworked the patchset, having per-infoframe callbacks rather than
extra flags (Maxime)
- Link to v2: https://lore.kernel.org/r/20250928-limit-infoframes-2-v2-0-6f8f5fd04214@oss.qualcomm.com
Changes in v2:
- Fixed build issue in common code
- Fixed comments regarding HDR / audio frames (Diedrik)
- In adv7511 actually check for the HDR frame (the comment was correct)
rather than the audio infoframe (c&p error).
- Link to v1: https://lore.kernel.org/r/20250927-limit-infoframes-2-v1-0-697511bd050b@oss.qualcomm.com
---
Dmitry Baryshkov (10):
drm/tests: hdmi: check the infoframes behaviour
drm/vc4: hdmi: implement clear_infoframe
drm/sun4i: hdmi_enc: implement clear_infoframe stub
drm/connector: make clear_infoframe callback mandatory for HDMI connectors
drm/bridge: refactor HDMI InfoFrame callbacks
drm/display: hdmi_state_helper: split InfoFrame functions per type
drm/display: hdmi_state_helper: reject Audio IF updates if it's not supported
drm/display: hdmi_state_helper: don't generate unsupported InfoFrames
drm/display: bridge_connector: dynamically generate HDMI callbacks
drm/debug: don't register files for unsupported HDMI InfoFrames
drivers/gpu/drm/bridge/adv7511/adv7511_drv.c | 180 ++++---
drivers/gpu/drm/bridge/ite-it6263.c | 95 ++--
drivers/gpu/drm/bridge/lontium-lt9611.c | 143 +++---
drivers/gpu/drm/bridge/synopsys/dw-hdmi-qp.c | 110 +++--
drivers/gpu/drm/display/drm_bridge_connector.c | 190 +++++++-
drivers/gpu/drm/display/drm_hdmi_state_helper.c | 94 ++--
drivers/gpu/drm/drm_connector.c | 6 +
drivers/gpu/drm/drm_debugfs.c | 7 +
drivers/gpu/drm/mediatek/mtk_hdmi_common.c | 8 +-
drivers/gpu/drm/mediatek/mtk_hdmi_v2.c | 110 +++--
drivers/gpu/drm/msm/hdmi/hdmi_bridge.c | 195 ++++----
drivers/gpu/drm/rockchip/inno_hdmi.c | 47 +-
drivers/gpu/drm/rockchip/rk3066_hdmi.c | 47 +-
drivers/gpu/drm/sun4i/sun4i_hdmi_enc.c | 42 +-
drivers/gpu/drm/tests/drm_client_modeset_test.c | 3 +
drivers/gpu/drm/tests/drm_connector_test.c | 19 +
drivers/gpu/drm/tests/drm_hdmi_state_helper_test.c | 525 +++++++++++++++++++++
drivers/gpu/drm/tests/drm_kunit_edid.h | 119 +++++
drivers/gpu/drm/vc4/vc4_hdmi.c | 105 ++++-
include/drm/drm_bridge.h | 127 ++++-
include/drm/drm_connector.h | 105 +++--
21 files changed, 1762 insertions(+), 515 deletions(-)
---
base-commit: e05b08d7d0162cf77fff119367fb1a2d5ab3e669
change-id: 20250927-limit-infoframes-2-6b93e599e79a
Best regards,
--
With best wishes
Dmitry
Powered by blists - more mailing lists