rescue: Link libzutil after libzfs.
ClosedPublic
Actions

Authored by jhb on Sep 29 2022, 10:25 PM.

Tags

None

Referenced Files

	F110692967: D36811.id111384.diff
	Fri, Feb 21, 11:39 PM

	F110635324: D36811.id111228.diff
	Fri, Feb 21, 6:04 AM

	Unknown Object (File)
	Tue, Feb 11, 12:03 AM

	Unknown Object (File)
	Jan 21 2025, 6:18 AM

	Unknown Object (File)
	Jan 21 2025, 3:27 AM

	Unknown Object (File)
	Nov 23 2024, 1:01 AM

	Unknown Object (File)
	Nov 15 2024, 11:37 PM

	Unknown Object (File)
	Nov 3 2024, 1:02 AM

View All 54 Files

Subscribers

imp

Details

Reviewers

kib
emaste
imp

Commits

rG5e4854cbf8d8: rescue: Link libzutil after libzfs.

Summary

libzfs depends on symbols defined in libzutil. For static linking
ld.bfd is less forgiving than lld, so rescue was failing to link when
using ld.bfd due to unresolved symbols from libzutil used by libzfs.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

jhb created this revision.Sep 29 2022, 10:25 PM

Herald added a subscriber: imp. · View Herald TranscriptSep 29 2022, 10:25 PM

jhb requested review of this revision.Sep 29 2022, 10:25 PM

Harbormaster completed remote builds in B47625: Diff 111228.Sep 29 2022, 10:25 PM

I would argue that this is a bug in ld.lld.

This revision is now accepted and ready to land.Sep 30 2022, 12:27 AM

There is a note about this in ld.lld's man page under IMPLEMENTATION NOTES. There is a --warn-backrefs option documented there that we may want to investigate.

imp accepted this revision.Sep 30 2022, 3:07 AM

Closed by commit rG5e4854cbf8d8: rescue: Link libzutil after libzfs. (authored by jhb). · Explain WhyOct 3 2022, 11:13 PM

This revision was automatically updated to reflect the committed changes.

jhb added a commit: rG5e4854cbf8d8: rescue: Link libzutil after libzfs..