kpriv/linux-hardened
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

drivers:hv: Track allocations of children of hv_vmbus in private resource tree

Browse source 
This patch changes vmbus_allocate_mmio() and vmbus_free_mmio() so
that when child paravirtual devices allocate memory-mapped I/O
space, they allocate it privately from a resource tree pointed
at by hyperv_mmio and also by the public resource tree
iomem_resource.  This allows the region to be marked as "busy"
in the private tree, but a "bridge window" in the public tree,
guaranteeing that no two bridge windows will overlap each other
but while also allowing the PCI device children of the bridge
windows to overlap that window.

One might conclude that this belongs in the pnp layer, rather
than in this driver.  Rafael Wysocki, the maintainter of the
pnp layer, has previously asked that we not modify the pnp layer
as it is considered deprecated.  This patch is thus essentially
a workaround.

Signed-off-by: Jake Oshins <jakeo@microsoft.com>
Signed-off-by: K. Y. Srinivasan <kys@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:
Jake Oshins 2016-04-05 10:22:54 -07:00 committed by Greg Kroah-Hartman
parent 23a0683186
commit be000f93e5
1 changed files with 21 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

22
drivers/hv/vmbus_drv.c
View file

@ -1128,7 +1128,7 @@ int vmbus_allocate_mmio(struct resource **new, struct hv_device *device_obj,
resource_size_t size, resource_size_t align, resource_size_t size, resource_size_t align,
bool fb_overlap_ok) bool fb_overlap_ok)
{ {
struct resource *iter; struct resource *iter, *shadow;
resource_size_t range_min, range_max, start, local_min, local_max; resource_size_t range_min, range_max, start, local_min, local_max;
const char *dev_n = dev_name(&device_obj->device); const char *dev_n = dev_name(&device_obj->device);
u32 fb_end = screen_info.lfb_base + (screen_info.lfb_size << 1); u32 fb_end = screen_info.lfb_base + (screen_info.lfb_size << 1);
@ -1170,12 +1170,22 @@ int vmbus_allocate_mmio(struct resource **new, struct hv_device *device_obj,
start = (local_min + align - 1) & ~(align - 1); start = (local_min + align - 1) & ~(align - 1);
for (; start + size - 1 <= local_max; start += align) { for (; start + size - 1 <= local_max; start += align) {
shadow = __request_region(iter, start,
size,
NULL,
IORESOURCE_BUSY);
if (!shadow)
continue;
*new = request_mem_region_exclusive(start, size, *new = request_mem_region_exclusive(start, size,
dev_n); dev_n);
if (*new) { if (*new) {
shadow->name = (char *)*new;
retval = 0; retval = 0;
goto exit; goto exit;
} }
__release_region(iter, start, size);
} }
} }
} }
@ -1196,7 +1206,17 @@ EXPORT_SYMBOL_GPL(vmbus_allocate_mmio);
*/ */
void vmbus_free_mmio(resource_size_t start, resource_size_t size) void vmbus_free_mmio(resource_size_t start, resource_size_t size)
{ {
struct resource *iter;
down(&hyperv_mmio_lock);
for (iter = hyperv_mmio; iter; iter = iter->sibling) {
if ((iter->start >= start + size) || (iter->end <= start))
continue;
__release_region(iter, start, size);
}
release_mem_region(start, size); release_mem_region(start, size);
up(&hyperv_mmio_lock);
} }
EXPORT_SYMBOL_GPL(vmbus_free_mmio); EXPORT_SYMBOL_GPL(vmbus_free_mmio);