Page MenuHomeFreeBSD

arm64: fix db_read_bytes() for size == 8
ClosedPublic

Authored by mhorne on Jan 17 2024, 4:51 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Apr 22, 9:21 AM
Unknown Object (File)
Fri, Apr 19, 8:42 AM
Unknown Object (File)
Sat, Apr 6, 10:34 PM
Unknown Object (File)
Sat, Apr 6, 1:05 PM
Unknown Object (File)
Mar 26 2024, 11:20 PM
Unknown Object (File)
Feb 28 2024, 9:47 PM
Unknown Object (File)
Jan 22 2024, 5:51 AM
Unknown Object (File)
Jan 19 2024, 4:43 AM
Subscribers

Details

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

On a second look, the native-sized reads were not an optimization but rather to enable reading from device memory, per a67687fcd8f53.

So, just fix the cast.

mhorne edited the summary of this revision. (Show Details)
This revision is now accepted and ready to land.Jan 17 2024, 5:14 PM

Thanks fixing my bugs..

Seems like there ought to be a comment before the if size == 8 / else if / else if block.

jhb added inline comments.
sys/arm64/arm64/db_interface.c
134

I think const uint64_t * would be clearer here, and similarly for the other casts below.

This revision was automatically updated to reflect the committed changes.