Differential D21837

bhyve: refactor NVMe PRP memcpy
ClosedPublic
Actions

Authored by chuck on Sep 29 2019, 3:23 PM.

Tags

None

Referenced Files

	F164613014: D21837.diff
	Sun, Aug 2, 11:37 AM

	F164578301: D21837.id69922.diff
	Sun, Aug 2, 4:38 AM

	F164548804: D21837.id62707.diff
	Sun, Aug 2, 1:10 AM

	Unknown Object (File)
	Sat, Aug 1, 8:31 AM

	Unknown Object (File)
	Wed, Jul 8, 10:38 AM

	Unknown Object (File)
	Jun 27 2026, 1:10 AM

	Unknown Object (File)
	Jun 24 2026, 10:29 PM

	Unknown Object (File)
	Jun 22 2026, 12:54 AM

Subscribers

Details

Reviewers

imp
ken
jhb

Group Reviewers

Commits

rS359362: bhyve: refactor NVMe PRP memcpy

Summary

Add a "copy direction" parameter to nvme_prp_memcpy such that data can
be copied to the memory specified by the PRP entries (current behavior)
or copied from the PRP entries (new behavior). The upcoming deallocate
functionality will use the copy from capability.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

chuck created this revision.Sep 29 2019, 3:23 PM

Herald added subscribers: bcran, rgrimes. · View Herald TranscriptSep 29 2019, 3:23 PM

Harbormaster completed remote builds in B26762: Diff 62707.Sep 29 2019, 3:23 PM

chuck mentioned this in D21838: bhyve: refactor NVMe namespace initialization.Sep 29 2019, 3:26 PM

chuck mentioned this in D21839: bhyve: implement NVMe deallocate command.Sep 29 2019, 3:29 PM

jhb accepted this revision.Oct 10 2019, 4:26 PM

This revision is now accepted and ready to land.Oct 10 2019, 4:26 PM

I came here from D21838 to say: looks good too!!! :)

Closed by commit rS359362: bhyve: refactor NVMe PRP memcpy (authored by chuck). · Explain WhyMar 27 2020, 3:28 PM

This revision was automatically updated to reflect the committed changes.

chuck added a commit: rS359362: bhyve: refactor NVMe PRP memcpy.

Herald added a subscriber: dab. · View Herald TranscriptMar 27 2020, 3:28 PM

Revision Contents
Changeset List

Path

Size

head/

usr.sbin/

bhyve/

46 lines

Diff 69922

head/usr.sbin/bhyve/pci_nvme.c

Loading...