Received: by 2002:a05:6a10:8c0a:0:0:0:0 with SMTP id go10csp914238pxb; Wed, 3 Mar 2021 20:53:01 -0800 (PST) X-Google-Smtp-Source: ABdhPJxZMNFI3DsbBZFsXgEJvlz+eStR6Gwq7UNj3NAo7XMTGkP7JTGQXRSrImeD2K55dzBMJsAb X-Received: by 2002:a17:906:6558:: with SMTP id u24mr2277244ejn.185.1614833580840; Wed, 03 Mar 2021 20:53:00 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1614833580; cv=none; d=google.com; s=arc-20160816; b=a2s1+j44GA/3pkCYK+Qti8jDhbPQ4EakTIXZJK0iiNPEEfl/oV+b3oApTu5Y3FPFLE fBohKnQ0Agdp6iNjcFI8bz+SJpCUdlHHy7E5o3NNCQQ+7fBQ7gZqZcSgZlokWSFSE3Jt NzZPggFu0H5laDbHuVK+0+NlRGi38bpqlNFb3VSZQruRRry7d4imNUhcVb/8Jn+RB3mZ TB+PDHFi7qq/sXkpscy/5gbEcee2ldPh3kmfkk39EKDuYr0q8OUgvohBzZPNUodDkM3o wKd3uu27/I+poEwal4z7+S6TrfjSaCKWFp5XdULFYesFlW1ce6R/iKEOKquEGvpo6Dw0 h/xQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :user-agent:references:in-reply-to:message-id:date:subject:cc:to :from:dkim-signature; bh=RBQZ5C0wQiTFQlWu7nf0h4UMR/vVBgEtYlPvUTJMFzo=; b=iN4W8AdYeml/TJ28Cd+XwMZKRC2i0wHfOS5mcQAzXF//0PaWCHbnQNbkKd6Mc4bv2H o5RqVP+vlFuRJJGgo5ZSJMhw064VKJxQ7eNXYirzsqaVopiPPAnA9y4D3pu9oEa+HxsZ 9ccb5aUz4KlwnPstKx6cdYUKtexTy8IfdfrHAEv0apJXUXlz0uj3LGiMzUfkXOtIR29Y 8G3wqRwWusi5B1w0CFmFiW6ptE1SyuLKFa5D8kv9/ZZczNPQgyvc8Ahz+JNMAjIBKN8e bc+JOUuP9m9U+/4+gB2dd6OOAaPPLCmufx3V9Bf0WLRtWjlsg6cPnx3BNqSYPwlZHOy+ bn7A== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@linuxfoundation.org header.s=korg header.b="u/3mwueM"; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linuxfoundation.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id v4si18523891edj.37.2021.03.03.20.52.38; Wed, 03 Mar 2021 20:53:00 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@linuxfoundation.org header.s=korg header.b="u/3mwueM"; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linuxfoundation.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S241842AbhCBCKP (ORCPT + 99 others); Mon, 1 Mar 2021 21:10:15 -0500 Received: from mail.kernel.org ([198.145.29.99]:50942 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S242151AbhCATf1 (ORCPT ); Mon, 1 Mar 2021 14:35:27 -0500 Received: by mail.kernel.org (Postfix) with ESMTPSA id 8505E65077; Mon, 1 Mar 2021 17:27:00 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=linuxfoundation.org; s=korg; t=1614619621; bh=yGdYWvNuFZncbwZNiLQfD7jXjmOmiTOQhKTxlaNTqVg=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=u/3mwueMXz72OVILAejUU90z/3gfJakavkVNzmjZOF70HOAvzE8IUk+f9zyub+5YR /Uy5QnKjYWoIVFZvtF1R960hTTMieH/Rq49ELN1h7PU7bvSF+rBIT56FfWWcmXykct U0BfBviSq7d/UvGaN1uALwN9kjGh74Ca3qeRG3tY= From: Greg Kroah-Hartman To: linux-kernel@vger.kernel.org Cc: Greg Kroah-Hartman , stable@vger.kernel.org, Randy Dunlap , =?UTF-8?q?Ville=20Syrj=C3=A4l=C3=A4?= , Chris Wilson Subject: [PATCH 5.10 519/663] drm/modes: Switch to 64bit maths to avoid integer overflow Date: Mon, 1 Mar 2021 17:12:47 +0100 Message-Id: <20210301161207.522823064@linuxfoundation.org> X-Mailer: git-send-email 2.30.1 In-Reply-To: <20210301161141.760350206@linuxfoundation.org> References: <20210301161141.760350206@linuxfoundation.org> User-Agent: quilt/0.66 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org From: Ville Syrjälä commit 5b34ab52401f0f1f191bcb83a182c83b506f4763 upstream. The new >8k CEA modes have dotclocks reaching 5.94 GHz, which means our clock*1000 will now overflow the 32bit unsigned integer. Switch to 64bit maths to avoid it. Cc: stable@vger.kernel.org Reported-by: Randy Dunlap Signed-off-by: Ville Syrjälä Link: https://patchwork.freedesktop.org/patch/msgid/20201022194256.30978-1-ville.syrjala@linux.intel.com Tested-by: Randy Dunlap Reviewed-by: Chris Wilson Signed-off-by: Greg Kroah-Hartman --- drivers/gpu/drm/drm_modes.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) --- a/drivers/gpu/drm/drm_modes.c +++ b/drivers/gpu/drm/drm_modes.c @@ -762,7 +762,7 @@ int drm_mode_vrefresh(const struct drm_d if (mode->htotal == 0 || mode->vtotal == 0) return 0; - num = mode->clock * 1000; + num = mode->clock; den = mode->htotal * mode->vtotal; if (mode->flags & DRM_MODE_FLAG_INTERLACE) @@ -772,7 +772,7 @@ int drm_mode_vrefresh(const struct drm_d if (mode->vscan > 1) den *= mode->vscan; - return DIV_ROUND_CLOSEST(num, den); + return DIV_ROUND_CLOSEST_ULL(mul_u32_u32(num, 1000), den); } EXPORT_SYMBOL(drm_mode_vrefresh);