Page Menu
Home
FreeBSD
Search
Configure Global Search
Log In
Files
F165189297
D27448.id80237.diff
No One
Temporary
Actions
View File
Edit File
Delete File
View Transforms
Subscribe
Mute Notifications
Flag For Later
Award Token
Size
830 B
Referenced Files
None
Subscribers
None
D27448.id80237.diff
View Options
Index: sys/dev/nvme/nvme_qpair.c
===================================================================
--- sys/dev/nvme/nvme_qpair.c
+++ sys/dev/nvme/nvme_qpair.c
@@ -984,8 +984,12 @@
BUS_DMASYNC_PREREAD | BUS_DMASYNC_PREWRITE);
#if !defined( __powerpc__) && !defined( __aarch64__) && !defined( __arm__)
/*
- * powerpc's bus_dmamap_sync() already includes a heavyweight sync, but
- * no other archs do.
+ * Ensure proper ordering of writes to the submission queue (where the
+ * tailq write to the device's MMIO area must follow writes to the
+ * submission entries in host memory). On x86, this translates to a
+ * sfence operation to ensure all prior writes are complete. PowerPC
+ * and arm don't need this since their bus_dmamap_sync() functions
+ * already ensure this post condition.
*/
wmb();
#endif
File Metadata
Details
Attached
Mime Type
text/plain
Expires
Fri, Aug 7, 3:23 PM (8 h, 9 m)
Storage Engine
blob
Storage Format
Raw Data
Storage Handle
36186975
Default Alt Text
D27448.id80237.diff (830 B)
Attached To
Mode
D27448: Two commits to cleanup needless wmb() in nvme driver
Attached
Detach File
Event Timeline
Log In to Comment