rman resource kvaddr_t
Needs ReviewPublic
Actions

Authored by br on Mar 10 2022, 6:02 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Jan 1 2024, 12:27 AM

	Unknown Object (File)
	Jun 20 2023, 6:37 PM

	Unknown Object (File)
	Mar 5 2023, 5:32 PM

	Unknown Object (File)
	Feb 17 2023, 2:38 AM

	Unknown Object (File)
	Feb 11 2023, 7:57 PM

	Unknown Object (File)
	Jan 10 2023, 3:35 AM

	Unknown Object (File)
	Jan 5 2023, 3:15 PM

	Unknown Object (File)
	Dec 14 2022, 2:29 AM

Subscribers

None

Details

Reviewers

jhb
jrtc27
kib

Summary

Replace pointer members of structs u_resource and u_rman to kvaddr_t.
These structs exported to userspace, but the members never used as pointers in userspace (but instead used as unique idenitifiers).

These breaks a bit of ABI but we believe that only libdevinfo uses it (?).

Diff Detail

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

br requested review of this revision.Mar 10 2022, 6:02 PM

br created this revision.

Motivation is so the sysctl works for non-native ABIs.

Justification for the ABI break is I can't find any references to the structs via code search.debian.net, libdevinfo is the only in-tree consumer and it only affects 32-bit architectures.