• Douglas Anderson's avatar
    watchdog/hardlockup: have the perf hardlockup use __weak functions more cleanly · d9b3629a
    Douglas Anderson authored
    The fact that there watchdog_hardlockup_enable(),
    watchdog_hardlockup_disable(), and watchdog_hardlockup_probe() are
    declared __weak means that the configured hardlockup detector can define
    non-weak versions of those functions if it needs to.  Instead of doing
    this, the perf hardlockup detector hooked itself into the default __weak
    implementation, which was a bit awkward.  Clean this up.
    
    From comments, it looks as if the original design was done because the
    __weak function were expected to implemented by the architecture and not
    by the configured hardlockup detector.  This got awkward when we tried to
    add the buddy lockup detector which was not arch-specific but wanted to
    hook into those same functions.
    
    This is not expected to have any functional impact.
    
    Link: https://lkml.kernel.org/r/20230519101840.v5.13.I847d9ec852449350997ba00401d2462a9cb4302b@changeidSigned-off-by: default avatarDouglas Anderson <dianders@chromium.org>
    Reviewed-by: default avatarPetr Mladek <pmladek@suse.com>
    Cc: Andi Kleen <ak@linux.intel.com>
    Cc: Catalin Marinas <catalin.marinas@arm.com>
    Cc: Chen-Yu Tsai <wens@csie.org>
    Cc: Christophe Leroy <christophe.leroy@csgroup.eu>
    Cc: Colin Cross <ccross@android.com>
    Cc: Daniel Thompson <daniel.thompson@linaro.org>
    Cc: "David S. Miller" <davem@davemloft.net>
    Cc: Guenter Roeck <groeck@chromium.org>
    Cc: Ian Rogers <irogers@google.com>
    Cc: Lecopzer Chen <lecopzer.chen@mediatek.com>
    Cc: Marc Zyngier <maz@kernel.org>
    Cc: Mark Rutland <mark.rutland@arm.com>
    Cc: Masayoshi Mizuma <msys.mizuma@gmail.com>
    Cc: Matthias Kaehlcke <mka@chromium.org>
    Cc: Michael Ellerman <mpe@ellerman.id.au>
    Cc: Nicholas Piggin <npiggin@gmail.com>
    Cc: Pingfan Liu <kernelfans@gmail.com>
    Cc: Randy Dunlap <rdunlap@infradead.org>
    Cc: "Ravi V. Shankar" <ravi.v.shankar@intel.com>
    Cc: Ricardo Neri <ricardo.neri@intel.com>
    Cc: Stephane Eranian <eranian@google.com>
    Cc: Stephen Boyd <swboyd@chromium.org>
    Cc: Sumit Garg <sumit.garg@linaro.org>
    Cc: Tzung-Bi Shih <tzungbi@chromium.org>
    Cc: Will Deacon <will@kernel.org>
    Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
    d9b3629a
watchdog.c 25.5 KB