[PATCH] um: Allow multiple symbol definitions for GCOV builds
Johannes Berg
johannes at sipsolutions.net
Wed Jul 22 09:14:10 PDT 2026
On Wed, 2026-07-22 at 10:09 -0600, Alex Hung wrote:
>
> 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 don't _think_ so, I guess it ended up mostly procedural, since it was
across different maintainers etc., and I lost interest because we
stopped trying to use gcov.
> I sure want to revive them.
Sure, sounds reasonable, feel free, of course!
> The series may also have a solution for the other patch I sent
> (https://lore.kernel.org/linux-um/eea36454-cbbe-4690-980d-b4ec9423f3a0@amd.com/T/#u).
That one we should just apply for ARCH=um, seems reasonable to me.
> I will test the series and report here later.
>
Thanks. Please just resend whatever you think should be included, maybe
first all the UML stuff separately.
johannes
More information about the linux-um
mailing list