linux-kernel - [PATCH v4 0/3] Fix clock division overflow problem

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

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [thread-next>] [day] [month] [year] [list]

Message-ID: <20231024161931.78567-1-sebastian.reichel@collabora.com>
Date:   Tue, 24 Oct 2023 18:18:14 +0200
From:   Sebastian Reichel <sebastian.reichel@...labora.com>
To:     Andy Shevchenko <andriy.shevchenko@...ux.intel.com>,
        Michael Turquette <mturquette@...libre.com>,
        Stephen Boyd <sboyd@...nel.org>,
        David Laight <David.Laight@...LAB.COM>,
        linux-clk@...r.kernel.org, linux-kernel@...r.kernel.org
Cc:     Vasily Gorbik <gor@...ux.ibm.com>,
        Niklas Schnelle <schnelle@...ux.ibm.com>,
        Linus Torvalds <torvalds@...ux-foundation.org>,
        Sebastian Reichel <sebastian.reichel@...labora.com>,
        kernel@...labora.com
Subject: [PATCH v4 0/3] Fix clock division overflow problem

Hi,

I noticed the overflow issue while debugging an issue related to the Rockchip
RK3588 eMMC. It's not an issue on that particular platform and it does not seem
this has been an issue on any other platform so far, so I did not add any
stable Tags. Considering 64 bit platforms are the standard nowadays and 3GHz is
a reasonable small value, I expect this to become an issue soon, though.

Changes since PATCHv3:
 * https://lore.kernel.org/all/20230630183835.464216-1-sebastian.reichel@collabora.com/
 * Add some people to Cc, because similar overflow issue came up during s390 pull request
 * Split DIV_ROUND_UP_NO_OVERFLOW into its own patch
 * Add patch replacing open-coded abs_diff() from the previously applied patch

Changes since PATCHv2:
 * https://lore.kernel.org/all/20230526171057.66876-1-sebastian.reichel@collabora.com/
 * Drop first patch (applied)
 * Update second patch to use newly introduced DIV_ROUND_UP_NO_OVERFLOW

Changes since PATCHv1:
 * https://lore.kernel.org/linux-clk/20230519190522.194729-1-sebastian.reichel@collabora.com/
 * Add Christopher Obbard's Reviewed-by to the first patch
 * Update the second patch to use DIV_ROUND_UP instead of DIV64_U64_ROUND_UP

Greetings,

-- Sebastian


Sebastian Reichel (3):
  math.h: add DIV_ROUND_UP_NO_OVERFLOW
  clk: divider: Fix divisor masking on 64 bit platforms
  clk: composite: replace open-coded abs_diff()

 drivers/clk/clk-composite.c |  6 ++----
 drivers/clk/clk-divider.c   |  6 +++---
 include/linux/math.h        | 11 +++++++++++
 3 files changed, 16 insertions(+), 7 deletions(-)

-- 
2.42.0