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)
Mon, Apr 29, 12:26 AM
Unknown Object (File)
Sun, Apr 28, 8:30 AM
Unknown Object (File)
Sat, Apr 27, 4:05 PM
Unknown Object (File)
Sun, Apr 21, 11:07 AM
Unknown Object (File)
Sat, Apr 20, 8:12 AM
Unknown Object (File)
Sat, Apr 20, 4:50 AM
Unknown Object (File)
Mar 14 2024, 3:14 PM
Unknown Object (File)
Mar 2 2024, 4:42 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.