Page MenuHomeFreeBSD
Differential D24081

bhyve: use STAILQ in NVMe emulation
ClosedPublic
Actions

Authored by chuck on Mar 15 2020, 11:40 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Feb 9, 10:27 PM
Unknown Object (File)
Mon, Jan 27, 12:50 AM
Unknown Object (File)
Sun, Jan 26, 12:27 AM
Unknown Object (File)
Thu, Jan 23, 1:13 PM
Unknown Object (File)
Jan 11 2025, 10:19 PM
Unknown Object (File)
Dec 30 2024, 6:11 PM
Unknown Object (File)
Dec 29 2024, 6:39 PM
Unknown Object (File)
Dec 28 2024, 7:33 PM
View All 42 Files
Subscribers
bcran
dab
rgrimes

Details

Reviewers
jhb
imp
Group Reviewers
bhyve
Commits
rS359365: bhyve: use STAILQ in NVMe emulation
Summary

Use the standard queue(3) macros instead of hand-crafted linked list
code.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

chuck created this revision.Mar 15 2020, 11:40 PM
Herald added subscribers: bcran, rgrimes. · View Herald TranscriptMar 15 2020, 11:40 PM
Harbormaster completed remote builds in B29959: Diff 69557.Mar 15 2020, 11:40 PM
imp accepted this revision.Mar 15 2020, 11:42 PM
This revision is now accepted and ready to land.Mar 15 2020, 11:42 PM
jhb accepted this revision.Mar 26 2020, 5:38 PM
jhb added inline comments.
usr.sbin/bhyve/pci_nvme.c
1317 ↗(On Diff #69557)

Either move the assert() above the REMOVE similar to the old code or axe it. STAILQ_REMOVE_HEAD will core dump when it dereferences the head pointer before getting to the assert() in the current code.

Herald added a subscriber: dab. · View Herald TranscriptMar 26 2020, 5:38 PM
chuck marked an inline comment as done.Mar 26 2020, 6:49 PM
chuck added inline comments.
usr.sbin/bhyve/pci_nvme.c
1317 ↗(On Diff #69557)

Bah, good catch. Thanks

Closed by commit rS359365: bhyve: use STAILQ in NVMe emulation (authored by chuck). · Explain WhyMar 27 2020, 3:28 PM
This revision was automatically updated to reflect the committed changes.
chuck marked an inline comment as done.
chuck added a commit: rS359365: bhyve: use STAILQ in NVMe emulation.

Revision Contents
Changeset List

PathSize
head/
usr.sbin/
bhyve/
18 lines

Diff 69925

View Options

head/usr.sbin/bhyve/pci_nvme.c

Loading...