Page MenuHomeFreeBSD

hyperv vmbus_pcib: Use pci_domain_*_bus for PCI_RES_BUS resources
ClosedPublic

Authored by jhb on Jan 12 2024, 10:32 PM.
Tags
None
Referenced Files
Unknown Object (File)
Oct 3 2024, 2:55 PM
Unknown Object (File)
Oct 3 2024, 12:16 PM
Unknown Object (File)
Oct 1 2024, 2:30 PM
Unknown Object (File)
Sep 27 2024, 10:24 PM
Unknown Object (File)
Sep 23 2024, 4:05 PM
Unknown Object (File)
Sep 22 2024, 12:02 PM
Unknown Object (File)
Sep 16 2024, 10:50 PM
Unknown Object (File)
Sep 9 2024, 6:45 PM
Subscribers
None

Details

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

jhb requested review of this revision.Jan 12 2024, 10:32 PM
jhb created this revision.
sys/dev/hyperv/pcib/vmbus_pcib.c
2011–2012

Please check the indentation here.

sys/dev/hyperv/pcib/vmbus_pcib.c
2011–2012

The raw version looks fine. I think the previous version was not fully consistent. Here is the patched method table:

	/* Bus interface */
	DEVMETHOD(bus_read_ivar,		vmbus_pcib_read_ivar),
	DEVMETHOD(bus_write_ivar,		vmbus_pcib_write_ivar),
	DEVMETHOD(bus_alloc_resource,		vmbus_pcib_alloc_resource),
	DEVMETHOD(bus_adjust_resource,		vmbus_pcib_adjust_resource),
	DEVMETHOD(bus_release_resource,		vmbus_pcib_release_resource),
	DEVMETHOD(bus_activate_resource,   	vmbus_pcib_activate_resource),
	DEVMETHOD(bus_deactivate_resource, 	vmbus_pcib_deactivate_resource),
	DEVMETHOD(bus_setup_intr,	   bus_generic_setup_intr),
	DEVMETHOD(bus_teardown_intr,	   bus_generic_teardown_intr),
	DEVMETHOD(bus_get_cpus,			vmbus_pcib_get_cpus),
This revision was not accepted when it landed; it landed in state Needs Review.Jan 23 2024, 5:46 PM
This revision was automatically updated to reflect the committed changes.

Code looks fine to me, but has the new functions been tested on Azure ? As in local Hyper-V we mostly dont use pci passthrough.

Code looks fine to me, but has the new functions been tested on Azure ? As in local Hyper-V we mostly dont use pci passthrough.

I do not have a means to test it, no. It is part of a larger API sweep/cleanup series.