Page MenuHomeFreeBSD
Differential D54475

nvme: Be more correct in busdma usage
Needs ReviewPublic
Actions

Authored by imp on Jan 3 2026, 6:13 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Feb 1, 9:54 AM
Unknown Object (File)
Sun, Feb 1, 9:54 AM
Unknown Object (File)
Thu, Jan 22, 7:54 PM
Unknown Object (File)
Thu, Jan 22, 7:14 PM
Unknown Object (File)
Thu, Jan 22, 6:34 PM
Unknown Object (File)
Fri, Jan 16, 12:47 PM
Unknown Object (File)
Fri, Jan 16, 8:55 AM
Unknown Object (File)
Thu, Jan 15, 5:13 PM
View All 22 Files
Subscribers
dab
This revision needs review, but there are no reviewers specified.

Details

Reviewers
None
Summary

Only do a PRE/POST sync for READ or WRITE, but not both. Only do a PRE
sync for requests with payloads.

Sponsored by: Netflix

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 69594
Build 66477: arc lint + arc unit

Event Timeline

imp created this revision.Jan 3 2026, 6:13 PM
Herald added a subscriber: dab. · View Herald TranscriptJan 3 2026, 6:13 PM
imp requested review of this revision.Jan 3 2026, 6:13 PM
Harbormaster completed remote builds in B69590: Diff 168965.Jan 3 2026, 6:13 PM
mmel added a subscriber: mmel.Jan 3 2026, 6:30 PM
mmel removed a subscriber: mmel.Jan 3 2026, 7:28 PM
mmel added inline comments.
sys/dev/nvme/nvme_qpair.c
1071

This fragment looks wrong.

imp added inline comments.Jan 3 2026, 8:24 PM
sys/dev/nvme/nvme_qpair.c
1071

This fragment looks wrong.

What looks wrong?

imp updated this revision to Diff 168976.Sat, Jan 3, 10:54 PM

fix the little oops that mmel found

Harbormaster completed remote builds in B69594: Diff 168976.Sat, Jan 3, 10:54 PM

Revision Contents
Changeset List

PathSize
sys/
dev/
nvme/
10 lines
4 lines
7 lines
2 lines
12 lines
3 lines

Diff 168976

View Options

sys/dev/nvme/nvme_ctrlr.c

Loading...
View Options

sys/dev/nvme/nvme_ctrlr_cmd.c

Loading...
View Options

sys/dev/nvme/nvme_ns_cmd.c

Loading...
View Options

sys/dev/nvme/nvme_private.h

Loading...
View Options

sys/dev/nvme/nvme_qpair.c

Loading...
View Options

sys/dev/nvme/nvme_sim.c

Loading...