Re: MSI entry unlinked before arch_teardown_msi_irq().

2007-02-26 Thread Eric W. Biederman
David Miller <[EMAIL PROTECTED]> writes: > From: [EMAIL PROTECTED] (Eric W. Biederman) > Date: Sun, 25 Feb 2007 17:14:43 -0700 > >> David Miller <[EMAIL PROTECTED]> writes: >> >> > From: [EMAIL PROTECTED] (Eric W. Biederman) >> > Date: Sun, 25 Feb 2007 02:08:07 -0700 >> > >> >> If that isn't the

Re: MSI entry unlinked before arch_teardown_msi_irq().

2007-02-26 Thread David Miller
From: [EMAIL PROTECTED] (Eric W. Biederman) Date: Sun, 25 Feb 2007 17:14:43 -0700 > David Miller <[EMAIL PROTECTED]> writes: > > > From: [EMAIL PROTECTED] (Eric W. Biederman) > > Date: Sun, 25 Feb 2007 02:08:07 -0700 > > > >> If that isn't the issue holler and I will look into it a little more.

Re: MSI entry unlinked before arch_teardown_msi_irq().

2007-02-26 Thread David Miller
From: [EMAIL PROTECTED] (Eric W. Biederman) Date: Sun, 25 Feb 2007 17:14:43 -0700 David Miller [EMAIL PROTECTED] writes: From: [EMAIL PROTECTED] (Eric W. Biederman) Date: Sun, 25 Feb 2007 02:08:07 -0700 If that isn't the issue holler and I will look into it a little more. Talking

Re: MSI entry unlinked before arch_teardown_msi_irq().

2007-02-26 Thread Eric W. Biederman
David Miller [EMAIL PROTECTED] writes: From: [EMAIL PROTECTED] (Eric W. Biederman) Date: Sun, 25 Feb 2007 17:14:43 -0700 David Miller [EMAIL PROTECTED] writes: From: [EMAIL PROTECTED] (Eric W. Biederman) Date: Sun, 25 Feb 2007 02:08:07 -0700 If that isn't the issue holler and I will

Re: MSI entry unlinked before arch_teardown_msi_irq().

2007-02-25 Thread Eric W. Biederman
David Miller <[EMAIL PROTECTED]> writes: > Nevermind, something strange is going on here. I think my > patch is bogus and the thing is still crashing, let me get > back to you. Sure. Eric - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to

Re: MSI entry unlinked before arch_teardown_msi_irq().

2007-02-25 Thread David Miller
From: [EMAIL PROTECTED] (Eric W. Biederman) Date: Sun, 25 Feb 2007 17:14:43 -0700 > David Miller <[EMAIL PROTECTED]> writes: > > > From: [EMAIL PROTECTED] (Eric W. Biederman) > > Date: Sun, 25 Feb 2007 02:08:07 -0700 > > > >> If that isn't the issue holler and I will look into it a little more.

Re: MSI entry unlinked before arch_teardown_msi_irq().

2007-02-25 Thread Eric W. Biederman
David Miller <[EMAIL PROTECTED]> writes: > From: [EMAIL PROTECTED] (Eric W. Biederman) > Date: Sun, 25 Feb 2007 02:08:07 -0700 > >> If that isn't the issue holler and I will look into it a little more. > > Talking in code is always more effective :-) > > This patch fixes the OOPS I was talking

Re: MSI entry unlinked before arch_teardown_msi_irq().

2007-02-25 Thread David Miller
From: [EMAIL PROTECTED] (Eric W. Biederman) Date: Sun, 25 Feb 2007 02:08:07 -0700 > If that isn't the issue holler and I will look into it a little more. Talking in code is always more effective :-) This patch fixes the OOPS I was talking about: diff --git a/drivers/pci/msi.c

Re: MSI entry unlinked before arch_teardown_msi_irq().

2007-02-25 Thread Eric W. Biederman
David Miller <[EMAIL PROTECTED]> writes: > ric, you recommended to me that I use: > > struct msi_desc *entry = get_irq_data(irq); > > in my arch_teardown_msi_irq() routine earlier, but the current > code unlinks the entry before the call to arch_teardown_msi_irq() > so I get OOPS's on

Re: MSI entry unlinked before arch_teardown_msi_irq().

2007-02-25 Thread Eric W. Biederman
David Miller [EMAIL PROTECTED] writes: ric, you recommended to me that I use: struct msi_desc *entry = get_irq_data(irq); in my arch_teardown_msi_irq() routine earlier, but the current code unlinks the entry before the call to arch_teardown_msi_irq() so I get OOPS's on shutdown on

Re: MSI entry unlinked before arch_teardown_msi_irq().

2007-02-25 Thread David Miller
From: [EMAIL PROTECTED] (Eric W. Biederman) Date: Sun, 25 Feb 2007 02:08:07 -0700 If that isn't the issue holler and I will look into it a little more. Talking in code is always more effective :-) This patch fixes the OOPS I was talking about: diff --git a/drivers/pci/msi.c

Re: MSI entry unlinked before arch_teardown_msi_irq().

2007-02-25 Thread Eric W. Biederman
David Miller [EMAIL PROTECTED] writes: From: [EMAIL PROTECTED] (Eric W. Biederman) Date: Sun, 25 Feb 2007 02:08:07 -0700 If that isn't the issue holler and I will look into it a little more. Talking in code is always more effective :-) This patch fixes the OOPS I was talking about: Ok.

Re: MSI entry unlinked before arch_teardown_msi_irq().

2007-02-25 Thread David Miller
From: [EMAIL PROTECTED] (Eric W. Biederman) Date: Sun, 25 Feb 2007 17:14:43 -0700 David Miller [EMAIL PROTECTED] writes: From: [EMAIL PROTECTED] (Eric W. Biederman) Date: Sun, 25 Feb 2007 02:08:07 -0700 If that isn't the issue holler and I will look into it a little more. Talking

Re: MSI entry unlinked before arch_teardown_msi_irq().

2007-02-25 Thread Eric W. Biederman
David Miller [EMAIL PROTECTED] writes: Nevermind, something strange is going on here. I think my patch is bogus and the thing is still crashing, let me get back to you. Sure. Eric - To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to [EMAIL

MSI entry unlinked before arch_teardown_msi_irq().

2007-02-24 Thread David Miller
Eric, you recommended to me that I use: struct msi_desc *entry = get_irq_data(irq); in my arch_teardown_msi_irq() routine earlier, but the current code unlinks the entry before the call to arch_teardown_msi_irq() so I get OOPS's on shutdown on sparc64 because of this since

MSI entry unlinked before arch_teardown_msi_irq().

2007-02-24 Thread David Miller
Eric, you recommended to me that I use: struct msi_desc *entry = get_irq_data(irq); in my arch_teardown_msi_irq() routine earlier, but the current code unlinks the entry before the call to arch_teardown_msi_irq() so I get OOPS's on shutdown on sparc64 because of this since