Re: [PATCH] um: Allow multiple symbol definitions for GCOV builds
Alex Hung <[email protected]> Wed, 22 Jul 2026 10:09:06 -0600
| Newsgroups | org.infradead.lists.linux-um |
|---|---|
| Message-ID | <[email protected]> |
On 7/22/26 08:02, Johannes Berg wrote: > Hi, > > On Tue, 2026-07-21 at 16:23 -0600, Alex Hung wrote: >>> >>> I cannot see how that would possibly result in correct behaviour (beyond >>> being able to link)?? > >> Good point. --allow-multiple-definition silences the linker error, but >> picking kernel or libgcov's mangle_path() will break the other at runtime. >> >> I saw your previous patch >> (https://lkml.iu.edu/hypermail/linux/kernel/2103.1/06604.html) to rename >> mangle_path() to seq_mangle_path(). How about reviving it to avoid name >> conflicts instead? > > Honestly, I completely forgot about that. Looks like I ran into the > typical "nobody wants to merge it" problem ... > > johannes Is there any concerns or blockers to merge them? I sure want to revive them. The series may also have a solution for the other patch I sent (https://lore.kernel.org/linux-um/[email protected]/T/#u). I will test the series and report here later.