RE: [xen-devel][PATCH][VTD] Fix apic pin to interrupt remapping table index

["Cui, Dexuan" <dexuan.cui@xxxxxxxxx>]

Yes, I'm organizing the order. I can further change the code. :-)

Thanks,
-- Dexuan

-----Original Message-----
From: Han, Weidong 
Sent: 2009?6?4? 9:16
To: 'Keir Fraser'
Cc: 'xen-devel'; 'Jan Beulich'; 'Isaku Yamahata'; Cui, Dexuan
Subject: RE: [xen-devel][PATCH][VTD] Fix apic pin to interrupt remapping table 
index

Keir,

Yes, your modified patch saves memory. I wanted to do like it. But Dexuan is 
working on a x2apic patch, which will move interrupt remapping enabling before 
IOAPIC setup. So I'm wondering ioapic stuffs (nr_ioapic_registers[], 
nr_ioapics, etc.) aren't ready when enable interrupt remapping after that 
moving. Dexuan, can you have a look at it?

Regards,
Weidong

Keir Fraser wrote:
> Wasteful of memory, so I checked in a modified version as c/s 19707,
> which dynamically sizes the array. Please take a look and check it's
> okay. 
> 
> It probably breaks ia64 build due to undefined nr_ioapics and
> nr_ioapic_registers[], but I think yours broke ia64 too so we're
> even. :-) 
> 
> Isaku: can you suggest ia64 equivalents for nr_ioapics and
> nr_ioapic_registers[]? We can do some ifdef magic at the top of
> intremap.c, including defining a nr_ioapic_registers() macro, if that
> helps. 
> 
>  Thanks,
>  Keir
> 
> On 03/06/2009 10:28, "Han, Weidong" <weidong.han@xxxxxxxxx> wrote:
> 
>> Originally, it calls xmalloc to set index in
>> ioapic_rte_to_remap_entry(). When make with debug=y, it may trigger
>> spinlock BUG_ON because allocate memory with interrupt disabled. 
>> 
>> This patch doesn't allocate list_head entry in
>> ioapic_rte_to_remap_entry(), instead allocate the array in
>> enable_intremap() to avoid allocating memory with interrupt
>> disabled.  
>> 
>> 
>> Signed-off-by: Weidong Han <weidong.han@xxxxxxxxx>


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel