The new methods will be used by the coming pcib driver.
Details
Diff Detail
- Lint
Lint Skipped - Unit
Tests Skipped
Event Timeline
sys/dev/hyperv/vmbus/vmbus_if.m | ||
---|---|---|
49 ↗ | (On Diff #21876) | This replaced by: |
sys/dev/hyperv/vmbus/vmbus_if.m | ||
---|---|---|
49 ↗ | (On Diff #21876) | OK, will remove this method. |
removed the get instance guid method, and moved the get_vcpu_id method to a separate patch (https://reviews.freebsd.org/D8410)
This looks fine. Note that there is some existing code this somewhat duplicates. For example, the ACPI Host-PCI bridge driver also parses _CRS to determine valid ranges. It uses the pcib_host_res_* API to add those ranges to a list that it then iterates on allocation requests. I'm not sure if does the same trick of preferring > 4G ranges for 64-bit BARs (though if pcib_host_res_*() would otherwise work for you I'd be fine with fixing it to do the same).
Hmm, we probably can reuse the existing APIs, but do two pass decodes, the first pass adds the 64-bits BARs, the second one adds 32-bits BARs.
Hi jhb, Thanks for the suggestion! I'll update the patch with the pcib_host_res_* APIs.