[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <alpine.LFD.2.00.1102071229050.31804@localhost6.localdomain6>
Date: Mon, 7 Feb 2011 12:30:03 +0100 (CET)
From: Thomas Gleixner <tglx@...utronix.de>
To: Ian Campbell <Ian.Campbell@...rix.com>
cc: LKML <linux-kernel@...r.kernel.org>,
Jeremy Fitzhardinge <jeremy@...p.org>
Subject: Re: [patch 2/4] xen: Switch to new irq_chip functions
On Mon, 7 Feb 2011, Ian Campbell wrote:
> On Sat, 2011-02-05 at 20:08 +0000, Thomas Gleixner wrote:
> >
> > -static void ack_pirq(unsigned int irq)
> > +static void ack_pirq(struct irq_data *data)
> > {
> > - int evtchn = evtchn_from_irq(irq);
> > + int evtchn = evtchn_from_irq(data->irq);
> >
> > - move_native_irq(irq);
> > + irq_move_irq(data);
>
> I tried to test with Linus latest but with this patch:
>
> /local/scratch/ianc/devel/kernels/linux-2.6/drivers/xen/events.c: In function 'ack_pirq':
> /local/scratch/ianc/devel/kernels/linux-2.6/drivers/xen/events.c:568: error: implicit declaration of function 'irq_move_irq'
>
> I can't find irq_move_irq in any of the tip.git branches (incl. master,
> auto-latest and .*irq.*), via google or in any of the other conversion
> patches you posted for other arches. Am I just looking in the wrong
> places or was this a mistake?
>
> For now I went with the following sort-of-reversion to allow me to
> continue:
>
> diff --git a/drivers/xen/events.c b/drivers/xen/events.c
> index 65f05e2..6f5dd38 100644
> --- a/drivers/xen/events.c
> +++ b/drivers/xen/events.c
> @@ -565,7 +565,7 @@ static void ack_pirq(struct irq_data *data)
> {
> int evtchn = evtchn_from_irq(data->irq);
>
> - irq_move_irq(data);
> + move_native_irq(data->irq);
Sorry, had another patch applied before that. Did not recheck before
sending it. Yes, that's the correct resolution.
Thanks,
tglx
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists