[PATCH 03/19] x86, io_apic: Introduce x86_io_apic_ops.disable()

2012-11-20 Thread Joerg Roedel
This function pointer is used to call a system-specific function for disabling the IO-APIC. Currently this is used for IRQ remapping which has its own disable routine. Also introduce the necessary infrastructure in the interrupt remapping code to overwrite this and other function pointers as neces

[PATCH 03/19] x86, io_apic: Introduce x86_io_apic_ops.disable()

2012-09-26 Thread Joerg Roedel
This function pointer is used to call a system-specific function for disabling the IO-APIC. Currently this is used for IRQ remapping which has its own disable routine. Also introduce the necessary infrastructure in the interrupt remapping code to overwrite this and other function pointers as neces

Re: [PATCH 03/19] x86, io_apic: Introduce x86_io_apic_ops.disable()

2012-08-26 Thread Joerg Roedel
On Sat, Aug 25, 2012 at 07:17:05PM +0200, Sebastian Andrzej Siewior wrote: > On Fri, Aug 24, 2012 at 02:22:26PM +0200, Joerg Roedel wrote: > > Is it worth it to split that out? > > I don't insist to redo the patch because of this. Please redo the patch > description to better describe what you do

Re: [PATCH 03/19] x86, io_apic: Introduce x86_io_apic_ops.disable()

2012-08-25 Thread Sebastian Andrzej Siewior
On Fri, Aug 24, 2012 at 02:22:26PM +0200, Joerg Roedel wrote: > Well, yeah, the confusing thing with this patch is, that it does not only > introduce the x86_io_apic_ops.disable call-back but also adds the > infrastructure to change this op (and others). So the infrastructure > change should probab

Re: [PATCH 03/19] x86, io_apic: Introduce x86_io_apic_ops.disable()

2012-08-24 Thread Joerg Roedel
On Thu, Aug 23, 2012 at 11:11:40PM +0200, Sebastian Andrzej Siewior wrote: > On Mon, Aug 20, 2012 at 03:55:49PM +0200, Joerg Roedel wrote: > What I miss here is the fact that you extract irq-mapping specific bits from > disable_IO_APIC() and put them in a separate function named > irq_remapping_dis

Re: [PATCH 03/19] x86, io_apic: Introduce x86_io_apic_ops.disable()

2012-08-23 Thread Sebastian Andrzej Siewior
On Mon, Aug 20, 2012 at 03:55:49PM +0200, Joerg Roedel wrote: > This function pointer is used to call a system-specific > function for disabling the IO-APIC. Currently this is used > for IRQ remapping which has its own disable routine. What I miss here is the fact that you extract irq-mapping spec

[PATCH 03/19] x86, io_apic: Introduce x86_io_apic_ops.disable()

2012-08-20 Thread Joerg Roedel
This function pointer is used to call a system-specific function for disabling the IO-APIC. Currently this is used for IRQ remapping which has its own disable routine. Signed-off-by: Joerg Roedel --- arch/x86/include/asm/io_apic.h |2 ++ arch/x86/include/asm/x86_init.h |9 + arc

[PATCH 03/19] x86, io_apic: Introduce x86_io_apic_ops.disable()

2012-08-07 Thread Joerg Roedel
This function pointer is used to call a system-specific function for disabling the IO-APIC. Currently this is used for IRQ remapping which has its own disable routine. Signed-off-by: Joerg Roedel --- arch/x86/include/asm/io_apic.h |1 + arch/x86/include/asm/x86_init.h |9 + arch