upstream/opnsense-src
1
0
Fork 0
mirror of https://github.com/opnsense/src.git synced 2026-04-22 23:02:02 -04:00
Code Issues Projects Releases Packages Wiki Activity Actions

nvme: Initialize HMB entries before loading them into the controller

Browse source 
struct nvme_hmb_desc contains a pad field which was not getting
initialized before being synced.  This doesn't have much consequence but
triggers a report from KMSAN, which verifies that host-filled DMA memory
is initialized before it is made visible to the device.  So, let's just
initialize it properly.

Reported by:	KMSAN
Reviewed by:	mav, imp
MFC after:	1 week
Sponsored by:	Klara, Inc.
Sponsored by:	Juniper Networks, Inc.
Differential Revision:	https://reviews.freebsd.org/D43090

(cherry picked from commit d9b7301bb791faab48b6c7733c34078427b9a374)
This commit is contained in:
Mark Johnston 2023-12-18 17:45:24 -05:00
parent 2bc5d0db67
commit dcd747fda5
1 changed files with 2 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
sys/dev/nvme/nvme_ctrlr.c
View file

@ -1037,6 +1037,8 @@ again:
}
for (i = 0; i < ctrlr->hmb_nchunks; i++) {
memset(&ctrlr->hmb_desc_vaddr[i], 0,
sizeof(struct nvme_hmb_desc));
ctrlr->hmb_desc_vaddr[i].addr =
htole64(ctrlr->hmb_chunks[i].hmbc_paddr);
ctrlr->hmb_desc_vaddr[i].size = htole32(ctrlr->hmb_chunk / ctrlr->page_size);