sys/dev/nvme/nvme_sim.c
462	FWIW, in nvmf I just have a single `namespace_changed` callback. It parses the list of changed namespaces for the AEN and then calls `nvmf_rescan_ns` for each namespace. For the /dev/nvmeXnsY devices it does do some extra work to detect add vs remove vs change, but for the CAM SIM, it just always calls `nvmf_sim_rescan_ns` that just triggers an `xpt_rescan` for the namespace.

imp added inline comments.Jul 23 2025, 7:51 PM

sys/dev/nvme/nvme_sim.c
462	Yea. 'new' is called at startup, while my delta is called for the AER change event. I don't have a rm event yet. I'm happy to tweak things, especially if an xpt_rescan does the right thing.
462	Also, that's how it is with these changes, but not set in stone if we can make things simpler.

imp retitled this revision from nvme_sim: Connect to events broadcast with nvmec_if to nvme_sim: Connect to events broadcast with nvme_if.Thu, Nov 13, 11:30 PM

Update per review, add missing change

Harbormaster completed remote builds in B68654: Diff 166475.Fri, Nov 14, 7:55 PM

Update per review and rebase over resize changes... woof.

Harbormaster completed remote builds in B69089: Diff 167619.Fri, Dec 5, 11:28 PM

dab accepted this revision.Sun, Dec 7, 5:12 PM

This revision is now accepted and ready to land.Sun, Dec 7, 5:12 PM

Closed by commit rGa21f12840f5f: nvme_sim: Connect to events broadcast with nvme_if (authored by imp). · Explain WhyWed, Dec 10, 10:54 PM

This revision was automatically updated to reflect the committed changes.

imp added a commit: rGa21f12840f5f: nvme_sim: Connect to events broadcast with nvme_if.