Re: [Xen-devel] [PATCH 1/6] AMD IOMMU: allocate IRTE entries instead of using a static mapping

["Suthikulpanit, Suravee" <Suravee.Suthikulpanit@xxxxxxx>]

Yes.  I have tested this and check the interrupt remapping tables.  They look 
fine and working now.

Suravee

PS: Are you going to check in the patch to add the debug key also?  (You could 
change it to "V" as you pointed out.)


-----Original Message-----
From: Jan Beulich [mailto:JBeulich@xxxxxxxx] 
Sent: Monday, April 29, 2013 2:31 AM
To: Suthikulpanit, Suravee
Cc: Shin, Jacob; xiantao.zhang@xxxxxxxxx; xen-devel; Konrad Rzeszutek Wilk
Subject: Re: [PATCH 1/6] AMD IOMMU: allocate IRTE entries instead of using a 
static mapping
Importance: High

>>> On 26.04.13 at 19:13, Suravee Suthikulanit 
>>> <suravee.suthikulpanit@xxxxxxx>
wrote:
> On 4/19/2013 5:57 AM, Jan Beulich wrote:
>> --- a/xen/drivers/passthrough/amd/iommu_acpi.c
>> +++ b/xen/drivers/passthrough/amd/iommu_acpi.c
>>
>> @@ -691,14 +694,16 @@ static u16 __init parse_ivhd_device_spec
>>                   ioapic_sbdf[special->handle].bdf = bdf;
>>                   ioapic_sbdf[special->handle].seg = seg;
>>   
>> -                ioapic_sbdf[special->handle].pin_setup = xzalloc_array(
>> -                    unsigned long, BITS_TO_LONGS(nr_ioapic_entries[apic]));
>> +                ioapic_sbdf[special->handle].pin_2_idx = xmalloc_array(
>> +                    u16, nr_ioapic_entries[apic]);
>>                   if ( nr_ioapic_entries[apic] &&
>> -                     !ioapic_sbdf[IO_APIC_ID(apic)].pin_setup )
>> +                     !ioapic_sbdf[IO_APIC_ID(apic)].pin_2_idx )
>>                   {
>>                       printk(XENLOG_ERR "IVHD Error: Out of memory\n");
>>                       return 0;
>>                   }
>> +                memset(ioapic_sbdf[IO_APIC_ID(apic)].pin_2_idx, -1,
>> +                       nr_ioapic_entries[apic]);
>>               }
> 
> Jan,
> 
> Ok.. here is why the (offset >= INTREMAP_ENTRIES) in 
> update_intremap_entry_from_ioapic failed.
> 
> +                memset(ioapic_sbdf[IO_APIC_ID(apic)].pin_2_idx, -1,
> +                       nr_ioapic_entries[apic]);
> 
> should have been
> 
> +                memset(ioapic_sbdf[IO_APIC_ID(apic)].pin_2_idx, -1,
> +                       (nr_ioapic_entries[apic] * sizeof(u16)));
> 
> Since nr_ioapic_entries[apic] = 24, only pin_2_idx[0 to 11] is set to 
> 0xffff.  This causes the pin_2_idx[12-23] to fail the check.

Ah - so a one line fix. Will integrate that into the current patch.
Are you saying that with that change, the patch finally works?

Thanks, Jan




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