[PATCH 1/2 v2] Add map_legacy interface

Guillem Jover guillem at hadrons.org
Fri Oct 7 01:30:38 PDT 2011


On Thu, 2011-10-06 at 20:51:26 -0700, Jeremy Huddleston wrote:
> This allows platforms to hand back mmaps of the low 1M (ISA) address
> space on a per-domain basis.

> diff --git a/src/common_interface.c b/src/common_interface.c
> index 94a0d21..e4fb66e 100644
> --- a/src/common_interface.c
> +++ b/src/common_interface.c
[...]
> +int
> +pci_device_map_legacy(struct pci_device *dev, pciaddr_t base, pciaddr_t size,
> +		      unsigned map_flags, void **addr)
> +{
> +    if (base > 1048576 || base + size > 1048576)
> +	return EINVAL;

I'd find 0x100000 easier to read as 1MiB than 1048576.

> +int
> +pci_device_unmap_legacy(struct pci_device *dev, void *addr, pciaddr_t size)
> +{
> +    return pci_sys->methods->unmap_legacy(dev, addr, size);
> +}

Shouldn't this be protected for a NULL unmap_legacy member too?

regards,
guillem



More information about the xorg-devel mailing list