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)
Sun, May 12, 5:01 AM
Unknown Object (File)
Sun, May 12, 5:01 AM
Unknown Object (File)
Fri, May 10, 10:16 PM
Unknown Object (File)
Thu, May 9, 9:24 PM
Unknown Object (File)
Thu, May 9, 10:39 AM
Unknown Object (File)
Thu, May 9, 5:53 AM
Unknown Object (File)
Thu, May 9, 12:57 AM
Unknown Object (File)
Wed, May 8, 10:58 PM
Subscribers

Details

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 55466
Build 52355: 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
128

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.