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)
Mon, Dec 30, 6:11 PM
Unknown Object (File)
Sun, Dec 29, 6:39 PM
Unknown Object (File)
Sat, Dec 28, 7:33 PM
Unknown Object (File)
Sat, Dec 28, 1:06 AM
Unknown Object (File)
Nov 22 2024, 5:36 PM
Unknown Object (File)
Nov 22 2024, 12:08 PM
Unknown Object (File)
Nov 19 2024, 7:03 AM
Unknown Object (File)
Nov 17 2024, 9:56 PM
View All 37 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 Skipped
Unit
Tests Skipped
Build Status
Buildable 29959

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

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

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
usr.sbin/
bhyve/
18 lines

Diff 69557

View Options

usr.sbin/bhyve/pci_nvme.c

Loading...