>>> On 05.12.16 at 20:36, <suravee.suthikulpa...@amd.com> wrote:
> +unsigned int ioapic_id_to_index(unsigned int apic_id)
> +{
> + unsigned int idx;
> +
> + if ( !nr_ioapic_sbdf )
> + return MAX_IO_APICS;
This is pointless (redundant with ...
> + for ( idx = 0 ; idx < nr_ioapic_sbdf; idx++ )
> + if ( ioapic_sbdf[idx].id == apic_id )
> + break;
> +
> + if ( idx == nr_ioapic_sbdf )
> + return MAX_IO_APICS;
... this), so I think I'll take the liberty of removing it while committing.
With that
Reviewed-by: Jan Beulich <jbeul...@suse.com>
And I see that you've got away without bumping the size of the
array.
Jan
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel
