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

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

Tags

None

Referenced Files

	F132910611: D37022.id111865.diff
	Tue, Oct 21, 2:46 AM

	Unknown Object (File)
	Mon, Oct 13, 4:31 AM

	Unknown Object (File)
	Mon, Oct 6, 3:36 AM

	Unknown Object (File)
	Fri, Sep 26, 1:58 AM

	Unknown Object (File)
	Wed, Sep 24, 3:57 PM

	Unknown Object (File)
	Sep 15 2025, 10:35 PM

	Unknown Object (File)
	Sep 8 2025, 11:16 PM

	Unknown Object (File)
	Aug 19 2025, 8:01 PM

View All 78 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."