Page MenuHomeFreeBSD

nvme: replace bus_space_[read|write]_4 with bus_[read|write]_4
ClosedPublic

Authored by ziaee on Mar 11 2026, 8:55 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Apr 29, 1:09 PM
Unknown Object (File)
Wed, Apr 29, 1:04 PM
Unknown Object (File)
Wed, Apr 29, 1:31 AM
Unknown Object (File)
Wed, Apr 29, 1:26 AM
Unknown Object (File)
Tue, Apr 28, 8:24 PM
Unknown Object (File)
Tue, Apr 28, 8:15 PM
Unknown Object (File)
Tue, Apr 28, 3:44 AM
Unknown Object (File)
Tue, Apr 28, 3:35 AM
Subscribers

Details

Summary

The goal this change is to remove the use of the tag and handle
needed by bus_space_[read|write]_4.

Proxying on behalf of Matt Delco <delco@google.com>
Requested by: jhb

Diff Detail

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

Event Timeline

ziaee requested review of this revision.Mar 11 2026, 8:55 PM
ziaee edited the summary of this revision. (Show Details)
ziaee edited the summary of this revision. (Show Details)
This revision is now accepted and ready to land.Mar 11 2026, 9:26 PM
sys/dev/nvme/nvme_private.h
328–340

Worth reformatting the code as part of it given they now fit on fewer lines, and whilst here, can fix the missing spaces around +, and the missing parentheses around val for nvme_mmio_write_8

sys/dev/nvme/nvme_private.h
328–340

(Phabricator mangles the whitespace, but an editor that visually distinguishes between tabs and spaces, and has a tab width of 8, should make it clear)

When committing, you can just use Matt's name as the commit author (you will still be the committer). I agree with Jess on reformatting the macros.

Attempt to improve whitespace. I'm not sure if I'm understanding @jrtc27's whitespace suggestion but also I've never done this part of it before.

When committing, you can just use Matt's name as the commit author (you will still be the committer). I agree with Jess on reformatting the macros.

Of course. Please note I merged over a hundred contributor reviews and believe I have never messed this up. I did however once misattribute emaste to having reviewed something that he did not, which still stings, though I do not believe he held it against me.

Thanks so much for helping with this!

This revision now requires review to proceed.Mar 14 2026, 6:27 PM

One tiny missing space, otherwise LGTM, feel free to fix on commit

sys/dev/nvme/nvme_private.h
332
This revision is now accepted and ready to land.Mar 14 2026, 9:37 PM