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
F102094221: D43479.diff
Thu, Nov 7, 1:23 PM
Unknown Object (File)
Tue, Oct 29, 12:49 PM
Unknown Object (File)
Fri, Oct 25, 11:11 PM
Unknown Object (File)
Wed, Oct 23, 4:51 PM
Unknown Object (File)
Mon, Oct 21, 5:30 AM
Unknown Object (File)
Mon, Oct 21, 4:41 AM
Unknown Object (File)
Sun, Oct 20, 4:56 PM
Unknown Object (File)
Sep 17 2024, 9:27 PM
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.