truss: remove now-unused special case for MIPS
ClosedPublic
Actions

Authored by emaste on Oct 17 2022, 5:20 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Mon, Feb 10, 9:17 AM

	Unknown Object (File)
	Jan 13 2025, 8:27 PM

	Unknown Object (File)
	Jan 13 2025, 2:56 PM

	Unknown Object (File)
	Jan 10 2025, 9:12 AM

	Unknown Object (File)
	Jan 10 2025, 2:57 AM

	Unknown Object (File)
	Dec 28 2024, 9:02 AM

	Unknown Object (File)
	Dec 22 2024, 9:07 AM

	Unknown Object (File)
	Dec 19 2024, 7:07 PM

View All 45 Files

Subscribers

None

Details

Reviewers

arichardson
imp
mhorne

Commits

rG9a86a3cd9b4c: truss: remove now-unused special case for MIPS

Summary

Was introduced with 31dddc6a19042bad303cb3d3aa8f52a2cb22dd77. We could revert part of that commit instead (and inline user_ptr32_to_psaddr) but I think the fn serves some documentary purpose.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

emaste requested review of this revision.Oct 17 2022, 5:20 PM

emaste created this revision.

LGTM

This revision is now accepted and ready to land.Oct 17 2022, 5:44 PM

One comment, but either way this is a good change.

usr.bin/truss/syscalls.c
1559–1561	I think you can omit the second sentence.

emaste added inline comments.Oct 17 2022, 6:00 PM

usr.bin/truss/syscalls.c
1559–1561	Yeah, good point, although it's probably the "currently" that is superfluous. Maybe "Convert a 32-bit user-space pointer to psaddr_t by zero-extending."