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, Jan 20, 11:00 PM
Unknown Object (File)
Fri, Jan 17, 2:52 PM
Unknown Object (File)
Nov 28 2024, 4:03 AM
Unknown Object (File)
Nov 22 2024, 5:28 PM
Unknown Object (File)
Nov 7 2024, 1:23 PM
Unknown Object (File)
Oct 29 2024, 12:49 PM
Unknown Object (File)
Oct 25 2024, 11:11 PM
Unknown Object (File)
Oct 23 2024, 4:51 PM
Subscribers

Details

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 55465
Build 52354: arc lint + arc unit

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
123–124

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.