Page MenuHomeFreeBSD
Differential D54475

nvme: Be more correct in busdma usage
Needs ReviewPublic
Actions

Authored by imp on Sat, Jan 3, 6:13 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Jan 10, 4:46 PM
Unknown Object (File)
Sat, Jan 10, 5:33 AM
Unknown Object (File)
Fri, Jan 9, 1:27 AM
Unknown Object (File)
Thu, Jan 8, 12:45 AM
Unknown Object (File)
Wed, Jan 7, 5:50 AM
Unknown Object (File)
Mon, Jan 5, 12:07 PM
Unknown Object (File)
Mon, Jan 5, 6:10 AM
Unknown Object (File)
Mon, Jan 5, 5:58 AM
View All 11 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.Sat, Jan 3, 6:13 PM
Herald added a subscriber: dab. · View Herald TranscriptSat, Jan 3, 6:13 PM
imp requested review of this revision.Sat, Jan 3, 6:13 PM
Harbormaster completed remote builds in B69590: Diff 168965.Sat, Jan 3, 6:13 PM
mmel added a subscriber: mmel.Sat, Jan 3, 6:30 PM
mmel removed a subscriber: mmel.Sat, Jan 3, 7:28 PM
mmel added inline comments.
sys/dev/nvme/nvme_qpair.c
1071

This fragment looks wrong.

imp added inline comments.Sat, Jan 3, 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...