From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 3441BD10390 for ; Wed, 26 Nov 2025 09:13:34 +0000 (UTC) Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id CCC5C10E573; Wed, 26 Nov 2025 09:13:33 +0000 (UTC) Authentication-Results: gabe.freedesktop.org; dkim=pass (2048-bit key; unprotected) header.d=intel.com header.i=@intel.com header.b="HQNdIfmt"; dkim-atps=neutral Received: from mgamail.intel.com (mgamail.intel.com [198.175.65.19]) by gabe.freedesktop.org (Postfix) with ESMTPS id 6EFA310E54C; Wed, 26 Nov 2025 09:13:32 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1764148412; x=1795684412; h=from:to:cc:subject:in-reply-to:references:date: message-id:mime-version; bh=tfIBii/Wk+NhzAbLPhZMs1CNnSy2r2fqC9jyW+2vqqI=; b=HQNdIfmtY2o3+Ah944Yvv0Cagzrgx/DSAnSMPgnZaU+07y6KpeKWZM9s 2apXWCCyQFMFzrAle1IrYu8FaXQyptuk/jGDYEleY1CsGCKlUv41LlgF+ kcdjTeTmP5Bwm/zXPz8kenkWpLfa1okT+GtOSU2q/WY5B4a62uu6oLDMN mC9o7IJizN9IXkgmt3LAEmidU8EuR0H4tDtQ0ze2ZJFl87cNt0PNDPJ8Q DiDkBFTKfYZF7ND5VE5eC3/PD/2tSebVdP3RO/cEEP9cgRZghSqNuEE55 pZYDUiSm+AF3ODpyO5VZbzyHTVHUtt5nOjgxGakK08+brB9ws4HiOMMsn A==; X-CSE-ConnectionGUID: CihrSyE/TJigAKcFjfYXkg== X-CSE-MsgGUID: CZrPL+NiToydy4D2mCl7TQ== X-IronPort-AV: E=McAfee;i="6800,10657,11624"; a="66064315" X-IronPort-AV: E=Sophos;i="6.20,228,1758610800"; d="scan'208";a="66064315" Received: from fmviesa003.fm.intel.com ([10.60.135.143]) by orvoesa111.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 26 Nov 2025 01:13:32 -0800 X-CSE-ConnectionGUID: /ue8e6qfTeO9AlTWpRGkCg== X-CSE-MsgGUID: D1CoxtXXTE+WkinuzrI/hA== X-ExtLoop1: 1 Received: from lfiedoro-mobl.ger.corp.intel.com (HELO localhost) ([10.245.246.1]) by fmviesa003-auth.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 26 Nov 2025 01:13:26 -0800 From: Jani Nikula To: Yaroslav Bolyukin , Ville =?utf-8?B?U3lyasOkbMOk?= , Maarten Lankhorst , Maxime Ripard , Thomas Zimmermann , David Airlie , Simona Vetter Cc: Harry Wentland , Leo Li , Rodrigo Siqueira , Alex Deucher , Christian =?utf-8?Q?K=C3=B6nig?= , Wayne Lin , amd-gfx@lists.freedesktop.org, linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org, Yaroslav Bolyukin Subject: Re: [PATCH v6 2/7] drm/edid: prepare for VESA vendor-specific data block extension In-Reply-To: <20251126065126.54016-3-iam@lach.pw> Organization: Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo References: <20251126065126.54016-1-iam@lach.pw> <20251126065126.54016-3-iam@lach.pw> Date: Wed, 26 Nov 2025 11:13:23 +0200 Message-ID: <68c55e772424f8e001898cdd1edcf4856820461e@intel.com> MIME-Version: 1.0 Content-Type: text/plain X-BeenThere: amd-gfx@lists.freedesktop.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Discussion list for AMD gfx List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: amd-gfx-bounces@lists.freedesktop.org Sender: "amd-gfx" On Wed, 26 Nov 2025, Yaroslav Bolyukin wrote: > Current VESA vendor-specific block parsing expects real block size to be > the same as the defined struct size, use real offsets in conditionals > instead to add struct fields in future commits. > > Signed-off-by: Yaroslav Bolyukin I think this is something we want to backport, since MSO would break with bigger vendor-specific blocks, and that leads to black screens on MSO displays. Cc: stable@vger.kernel.org Reviewed-by: Jani Nikula > --- > drivers/gpu/drm/drm_edid.c | 28 ++++++++++++---------------- > 1 file changed, 12 insertions(+), 16 deletions(-) > > diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c > index 64f7a94dd9e4..a52fd6de9327 100644 > --- a/drivers/gpu/drm/drm_edid.c > +++ b/drivers/gpu/drm/drm_edid.c > @@ -6544,7 +6544,7 @@ static void drm_parse_vesa_specific_block(struct drm_connector *connector, > if (oui(vesa->oui[0], vesa->oui[1], vesa->oui[2]) != VESA_IEEE_OUI) > return; > > - if (sizeof(*vesa) != sizeof(*block) + block->num_bytes) { > + if (block->num_bytes < 5) { > drm_dbg_kms(connector->dev, > "[CONNECTOR:%d:%s] Unexpected VESA vendor block size\n", > connector->base.id, connector->name); > @@ -6567,24 +6567,20 @@ static void drm_parse_vesa_specific_block(struct drm_connector *connector, > break; > } > > - if (!info->mso_stream_count) { > - info->mso_pixel_overlap = 0; > - return; > - } > - > - info->mso_pixel_overlap = FIELD_GET(DISPLAYID_VESA_MSO_OVERLAP, vesa->mso); > - if (info->mso_pixel_overlap > 8) { > + if (info->mso_stream_count) { > + info->mso_pixel_overlap = FIELD_GET(DISPLAYID_VESA_MSO_OVERLAP, vesa->mso); > + if (info->mso_pixel_overlap > 8) { > + drm_dbg_kms(connector->dev, > + "[CONNECTOR:%d:%s] Reserved MSO pixel overlap value %u\n", > + connector->base.id, connector->name, > + info->mso_pixel_overlap); > + info->mso_pixel_overlap = 8; > + } > drm_dbg_kms(connector->dev, > - "[CONNECTOR:%d:%s] Reserved MSO pixel overlap value %u\n", > + "[CONNECTOR:%d:%s] MSO stream count %u, pixel overlap %u\n", > connector->base.id, connector->name, > - info->mso_pixel_overlap); > - info->mso_pixel_overlap = 8; > + info->mso_stream_count, info->mso_pixel_overlap); > } > - > - drm_dbg_kms(connector->dev, > - "[CONNECTOR:%d:%s] MSO stream count %u, pixel overlap %u\n", > - connector->base.id, connector->name, > - info->mso_stream_count, info->mso_pixel_overlap); > } > > static void drm_update_vesa_specific_block(struct drm_connector *connector, -- Jani Nikula, Intel