Re: [Xen-devel] [PATCH] VTD: Remove the check for reserved device scope type

[Keir Fraser <keir.xen@xxxxxxxxx>]

On 16/04/2013 07:18, "Yang Zhang" <yang.z.zhang@xxxxxxxxx> wrote:

> From: Yang Zhang <yang.z.zhang@xxxxxxxxx>
> 
> Though we only have four valid types now, the new type may be added in future.
> It's better to remove the check and only deal with the type that we can
> recognize.
> 
> Signed-off-by: Yang Zhang <yang.z.zhang@xxxxxxxxx>
> Signed-off-by: Xiantao Zhang <xiantao.zhang@xxxxxxxxx>

Acked-by: Keir Fraser <keir@xxxxxxx>

> ---
>  xen/drivers/passthrough/vtd/dmar.c |    7 +++++--
>  xen/include/acpi/actbl2.h          |    1 -
>  2 files changed, 5 insertions(+), 3 deletions(-)
> 
> diff --git a/xen/drivers/passthrough/vtd/dmar.c
> b/xen/drivers/passthrough/vtd/dmar.c
> index 370d5d6..eda04a3 100644
> --- a/xen/drivers/passthrough/vtd/dmar.c
> +++ b/xen/drivers/passthrough/vtd/dmar.c
> @@ -283,8 +283,7 @@ static int __init scope_device_count(const void *start,
> const void *end)
>      while ( start < end )
>      {
>          scope = start;
> -        if ( (scope->length < MIN_SCOPE_LEN) ||
> -             (scope->entry_type >= ACPI_DMAR_SCOPE_TYPE_RESERVED) )
> +        if ( scope->length < MIN_SCOPE_LEN )
>          {
>              dprintk(XENLOG_WARNING VTDPREFIX, "Invalid device scope.\n");
>              return -EINVAL;
> @@ -409,6 +408,10 @@ static int __init acpi_parse_dev_scope(
>              }
>  
>              break;
> +
> +        default:
> +            start += acpi_scope->length;
> +            continue;
>          }
>          scope->devices[didx++] = PCI_BDF(bus, path->dev, path->fn);
>          start += acpi_scope->length;
> diff --git a/xen/include/acpi/actbl2.h b/xen/include/acpi/actbl2.h
> index a28226a..87bc6b3 100644
> --- a/xen/include/acpi/actbl2.h
> +++ b/xen/include/acpi/actbl2.h
> @@ -303,7 +303,6 @@ enum acpi_dmar_scope_type {
> ACPI_DMAR_SCOPE_TYPE_BRIDGE = 2,
> ACPI_DMAR_SCOPE_TYPE_IOAPIC = 3,
> ACPI_DMAR_SCOPE_TYPE_HPET = 4,
> - ACPI_DMAR_SCOPE_TYPE_RESERVED = 5 /* 5 and greater are reserved */
>  };
>  
>  struct acpi_dmar_pci_path {



_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel