Re: [Xen-devel] [PATCH 10/11] __setup_vector_irq: handle NULL chip_data

[Konrad Rzeszutek Wilk <konrad.wilk@xxxxxxxxxx>]

On Mon, May 10, 2010 at 03:20:46PM +0100, Stefano Stabellini wrote:
> Signed-off-by: Stefano Stabellini <stefano.stabellini@xxxxxxxxxxxxx>
> ---
>  arch/x86/kernel/apic/io_apic.c |    2 ++
>  1 files changed, 2 insertions(+), 0 deletions(-)
> 
> diff --git a/arch/x86/kernel/apic/io_apic.c b/arch/x86/kernel/apic/io_apic.c
> index 96f6bba..976646f 100644
> --- a/arch/x86/kernel/apic/io_apic.c
> +++ b/arch/x86/kernel/apic/io_apic.c
> @@ -1287,6 +1287,8 @@ void __setup_vector_irq(int cpu)
>       /* Mark the inuse vectors */
>       for_each_irq_desc(irq, desc) {
>               cfg = desc->chip_data;
> +             if (!cfg)
> +                     continue;
>               if (!cpumask_test_cpu(cpu, cfg->domain))
>                       continue;
>               vector = cfg->vector;

Most curious. This looks be fixed upstream:

http://lkml.org/lkml/2010/4/30/458

Do you know why the chip_data isn't set?

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