lkml.org 
[lkml]   [2015]   [Jul]   [12]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Subject[PATCH 0/9] MIPS: Remove "weak" usage
From
Date
These patches don't fix any problem I'm aware of, but I think they make the
code easier to analyze, and they reduce the likelihood of issues if MIPS
ever builds a multi-platform kernel.

Weak function declarations in header files are hard to use safely because
they make every definition weak. If the linker sees multiple weak
definitions, it silently chooses one based on link order. That's not a
very obvious criterion, and it can easily lead to running the wrong
version.

These patches remove the weak attribute from function declarations and
rework the code to match. I don't have any of these platforms, so I can't
test them, but my intent is that these should cause no functional change.

---

Bjorn Helgaas (9):
MIPS: CPC: Remove "weak" from mips_cpc_phys_base() and make it static
MIPS: Remove "weak" from platform_maar_init() declaration
MIPS: VPE: Exit vpe_release() early if vpe_run() isn't defined
MIPS: MT: Remove "weak" from vpe_run() declaration
MIPS: Remove "weak" from get_c0_perfcount_int() declaration
MIPS: Remove "weak" from get_c0_compare_int() declaration
MIPS: Remove "weak" from get_c0_fdc_int() declaration
MIPS: Remove "weak" from mips_cdmm_phys_base() declaration
MIPS: Remove "__weak" definition from arch-specific linkage.h


arch/mips/include/asm/cdmm.h | 4 ++--
arch/mips/include/asm/irq.h | 2 +-
arch/mips/include/asm/linkage.h | 1 -
arch/mips/include/asm/maar.h | 2 +-
arch/mips/include/asm/mips-cpc.h | 10 ----------
arch/mips/include/asm/time.h | 4 ++--
arch/mips/include/asm/vpe.h | 2 +-
arch/mips/kernel/cevt-r4k.c | 11 +++++++----
arch/mips/kernel/mips-cpc.c | 9 ++++++++-
arch/mips/kernel/perf_event_mipsxx.c | 7 +------
arch/mips/kernel/time.c | 10 +++++++++-
arch/mips/kernel/vpe.c | 7 ++++++-
arch/mips/oprofile/op_model_mipsxx.c | 8 +-------
drivers/bus/mips_cdmm.c | 14 +++++++++++++-
drivers/tty/mips_ejtag_fdc.c | 9 ++++++---
15 files changed, 58 insertions(+), 42 deletions(-)


\
 
 \ /
  Last update: 2015-07-13 01:41    [W:0.116 / U:0.936 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site