On Wed, Jun 14, 2023 at 05:21:43PM +0100, Russell King (Oracle) wrote: > What I'm trying to get at is that we have arch_vma_name in > arch/arm/kernel/process.c and also a weak function in kernel/signal.c. > > Both of these end up adding an entry into the __ksymtab_strings > section and a ___ksymtab section for this symbol. So we end up with > two entries in each. > > Now, if the one from kernel/signal.c points at its own weak function, > and that is found first, then that's the function that is going to be > bound, not the function that's overriding it. > > If, instead, the export in kernel/signal.c ends up pointing at the > overriden function, then the export in arch/arm/kernel/process.c is > entirely redundant. > > So, you need to get to the bottom of this... and until you do I'm > afraid I'll have to NAK this patch. I think the patch should be NAKed indefinitely. I had a quick look at the user, and it seems like something is being done in the kernel that should be done in userspace.
