Re: PCI: Fix boot-time hang on G31/G33 PC
Vitaliy Gusev wrote: > On the 28 September 2007 03:13 Greg KH, wrote: >> On Thu, Sep 27, 2007 at 11:36:32AM -0700, Kok, Auke wrote: >>> Ivan Kokshaysky wrote: On Wed, Sep 26, 2007 at 03:20:40PM -0700, Jesse Barnes wrote: > Ivan, your concern is about disabling things like interrupt > controllers and power management chips during probe right? You're > right that doing that could cause problems if we get and interrupt or > PMU event at just the wrong time, but that could just as easily happen > if decode was still enabled but the BAR had a bogus address programmed > (as it would during probing). Yes, nobody is arguing that moving the BAR around is unsafe, but generally it's the less of two evils. The major problem here is that with IO and MEM bits cleared in the command register you disable *all* address decoders on the device, not just ranges that have respective BARs. At least this behaviour is required by PCI spec. Examples: - legacy VGA IO and memory (no corresponding BARs); - base/limit registers of P2P bridge; - PMU and SMBus registers (sort of normal BARs, but hidden elsewhere in the config space); - IDE legacy mode registers; - IO-APIC registers (typically sort of read-only BAR). For all of these address ranges our current BAR probe is effectively a no-op, but disable/re-enable clearly isn't. > Ultimately, I don't care much one way or another as long as we can get > the desktop platforms fixed somehow. I think disabling decode is the > most correct way of doing this, but I'm open to other solutions (this > is the only patch I've seen though that's been tested to solve the > problem). There are two other solutions: one is to disable decode selectively, only on devices or systems where it's necessary and known to be safe. I've posted a patch which introduces "disable_while_probe" pci_dev field for that purpose. Another one is to delay mmconfig probe until after the PCI probe is done, as Matthew suggested, and Robert confirmed that it's feasible. >>> for everyone who's using this quirk or has the same boot issue: I just >>> confirmed that the new dg33tl bios update v0287 (released 9/20) fixes the >>> boot issue for my systems. I encourage everyone to update their BIOS >>> image and see if this works. > > No, it still doesn't work even with a latest BIOS verstion. I have a computer > with Intel DQ35MP motherboard. I upgraded BIOS to rev. 0696, released > Oct 1. But kernel (2.6.22.5) still hangs up. Kernel with this fix patch boots > and works fine. please note that your BIOS is completely different than the one posted above. This may be a clue. interestingly enough I can attest (somewhat) to this. After the BIOS upgrade 2.6.22 booted just fine without pci=nommconf, but I've had several boot lockups with 2.6.23-rc8. So, the issue is still open and Matthew/Jesse's suggested fix becomes much more attractive to me now. Greg, I suggest putting this patch back in the "grey" zone Auke - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: PCI: Fix boot-time hang on G31/G33 PC
On the 28 September 2007 03:13 Greg KH, wrote: > On Thu, Sep 27, 2007 at 11:36:32AM -0700, Kok, Auke wrote: > > Ivan Kokshaysky wrote: > > > On Wed, Sep 26, 2007 at 03:20:40PM -0700, Jesse Barnes wrote: > > >> Ivan, your concern is about disabling things like interrupt > > >> controllers and power management chips during probe right? You're > > >> right that doing that could cause problems if we get and interrupt or > > >> PMU event at just the wrong time, but that could just as easily happen > > >> if decode was still enabled but the BAR had a bogus address programmed > > >> (as it would during probing). > > > > > > Yes, nobody is arguing that moving the BAR around is unsafe, but > > > generally it's the less of two evils. > > > > > > The major problem here is that with IO and MEM bits cleared in the > > > command register you disable *all* address decoders on the device, not > > > just ranges that have respective BARs. At least this behaviour is > > > required by PCI spec. Examples: > > > - legacy VGA IO and memory (no corresponding BARs); > > > - base/limit registers of P2P bridge; > > > - PMU and SMBus registers (sort of normal BARs, but hidden elsewhere > > > in the config space); > > > - IDE legacy mode registers; > > > - IO-APIC registers (typically sort of read-only BAR). > > > > > > For all of these address ranges our current BAR probe is effectively > > > a no-op, but disable/re-enable clearly isn't. > > > > > >> Ultimately, I don't care much one way or another as long as we can get > > >> the desktop platforms fixed somehow. I think disabling decode is the > > >> most correct way of doing this, but I'm open to other solutions (this > > >> is the only patch I've seen though that's been tested to solve the > > >> problem). > > > > > > There are two other solutions: one is to disable decode selectively, > > > only on devices or systems where it's necessary and known to be safe. > > > I've posted a patch which introduces "disable_while_probe" pci_dev > > > field for that purpose. > > > Another one is to delay mmconfig probe until after the PCI probe is > > > done, as Matthew suggested, and Robert confirmed that it's feasible. > > > > for everyone who's using this quirk or has the same boot issue: I just > > confirmed that the new dg33tl bios update v0287 (released 9/20) fixes the > > boot issue for my systems. I encourage everyone to update their BIOS > > image and see if this works. > No, it still doesn't work even with a latest BIOS verstion. I have a computer with Intel DQ35MP motherboard. I upgraded BIOS to rev. 0696, released Oct 1. But kernel (2.6.22.5) still hangs up. Kernel with this fix patch boots and works fine. > Thanks for letting us know. So, another reason to drop this patch :) > > thanks, > > greg k-h > - > To unsubscribe from this list: send the line "unsubscribe linux-kernel" in > the body of a message to [EMAIL PROTECTED] > More majordomo info at http://vger.kernel.org/majordomo-info.html > Please read the FAQ at http://www.tux.org/lkml/ -- Thanks, Vitaliy Gusev - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: PCI: Fix boot-time hang on G31/G33 PC
Vitaliy Gusev wrote: On the 28 September 2007 03:13 Greg KH, wrote: On Thu, Sep 27, 2007 at 11:36:32AM -0700, Kok, Auke wrote: Ivan Kokshaysky wrote: On Wed, Sep 26, 2007 at 03:20:40PM -0700, Jesse Barnes wrote: Ivan, your concern is about disabling things like interrupt controllers and power management chips during probe right? You're right that doing that could cause problems if we get and interrupt or PMU event at just the wrong time, but that could just as easily happen if decode was still enabled but the BAR had a bogus address programmed (as it would during probing). Yes, nobody is arguing that moving the BAR around is unsafe, but generally it's the less of two evils. The major problem here is that with IO and MEM bits cleared in the command register you disable *all* address decoders on the device, not just ranges that have respective BARs. At least this behaviour is required by PCI spec. Examples: - legacy VGA IO and memory (no corresponding BARs); - base/limit registers of P2P bridge; - PMU and SMBus registers (sort of normal BARs, but hidden elsewhere in the config space); - IDE legacy mode registers; - IO-APIC registers (typically sort of read-only BAR). For all of these address ranges our current BAR probe is effectively a no-op, but disable/re-enable clearly isn't. Ultimately, I don't care much one way or another as long as we can get the desktop platforms fixed somehow. I think disabling decode is the most correct way of doing this, but I'm open to other solutions (this is the only patch I've seen though that's been tested to solve the problem). There are two other solutions: one is to disable decode selectively, only on devices or systems where it's necessary and known to be safe. I've posted a patch which introduces disable_while_probe pci_dev field for that purpose. Another one is to delay mmconfig probe until after the PCI probe is done, as Matthew suggested, and Robert confirmed that it's feasible. for everyone who's using this quirk or has the same boot issue: I just confirmed that the new dg33tl bios update v0287 (released 9/20) fixes the boot issue for my systems. I encourage everyone to update their BIOS image and see if this works. No, it still doesn't work even with a latest BIOS verstion. I have a computer with Intel DQ35MP motherboard. I upgraded BIOS to rev. 0696, released Oct 1. But kernel (2.6.22.5) still hangs up. Kernel with this fix patch boots and works fine. please note that your BIOS is completely different than the one posted above. This may be a clue. interestingly enough I can attest (somewhat) to this. After the BIOS upgrade 2.6.22 booted just fine without pci=nommconf, but I've had several boot lockups with 2.6.23-rc8. So, the issue is still open and Matthew/Jesse's suggested fix becomes much more attractive to me now. Greg, I suggest putting this patch back in the grey zone Auke - To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: PCI: Fix boot-time hang on G31/G33 PC
On the 28 September 2007 03:13 Greg KH, wrote: On Thu, Sep 27, 2007 at 11:36:32AM -0700, Kok, Auke wrote: Ivan Kokshaysky wrote: On Wed, Sep 26, 2007 at 03:20:40PM -0700, Jesse Barnes wrote: Ivan, your concern is about disabling things like interrupt controllers and power management chips during probe right? You're right that doing that could cause problems if we get and interrupt or PMU event at just the wrong time, but that could just as easily happen if decode was still enabled but the BAR had a bogus address programmed (as it would during probing). Yes, nobody is arguing that moving the BAR around is unsafe, but generally it's the less of two evils. The major problem here is that with IO and MEM bits cleared in the command register you disable *all* address decoders on the device, not just ranges that have respective BARs. At least this behaviour is required by PCI spec. Examples: - legacy VGA IO and memory (no corresponding BARs); - base/limit registers of P2P bridge; - PMU and SMBus registers (sort of normal BARs, but hidden elsewhere in the config space); - IDE legacy mode registers; - IO-APIC registers (typically sort of read-only BAR). For all of these address ranges our current BAR probe is effectively a no-op, but disable/re-enable clearly isn't. Ultimately, I don't care much one way or another as long as we can get the desktop platforms fixed somehow. I think disabling decode is the most correct way of doing this, but I'm open to other solutions (this is the only patch I've seen though that's been tested to solve the problem). There are two other solutions: one is to disable decode selectively, only on devices or systems where it's necessary and known to be safe. I've posted a patch which introduces disable_while_probe pci_dev field for that purpose. Another one is to delay mmconfig probe until after the PCI probe is done, as Matthew suggested, and Robert confirmed that it's feasible. for everyone who's using this quirk or has the same boot issue: I just confirmed that the new dg33tl bios update v0287 (released 9/20) fixes the boot issue for my systems. I encourage everyone to update their BIOS image and see if this works. No, it still doesn't work even with a latest BIOS verstion. I have a computer with Intel DQ35MP motherboard. I upgraded BIOS to rev. 0696, released Oct 1. But kernel (2.6.22.5) still hangs up. Kernel with this fix patch boots and works fine. Thanks for letting us know. So, another reason to drop this patch :) thanks, greg k-h - To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/ -- Thanks, Vitaliy Gusev - To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: PCI: Fix boot-time hang on G31/G33 PC
On Thu, Sep 27, 2007 at 11:36:32AM -0700, Kok, Auke wrote: > Ivan Kokshaysky wrote: > > On Wed, Sep 26, 2007 at 03:20:40PM -0700, Jesse Barnes wrote: > >> Ivan, your concern is about disabling things like interrupt controllers > >> and power management chips during probe right? You're right that doing > >> that could cause problems if we get and interrupt or PMU event at just > >> the wrong time, but that could just as easily happen if decode was > >> still enabled but the BAR had a bogus address programmed (as it would > >> during probing). > > > > Yes, nobody is arguing that moving the BAR around is unsafe, but generally > > it's the less of two evils. > > > > The major problem here is that with IO and MEM bits cleared in the command > > register you disable *all* address decoders on the device, not just ranges > > that have respective BARs. At least this behaviour is required by PCI spec. > > Examples: > > - legacy VGA IO and memory (no corresponding BARs); > > - base/limit registers of P2P bridge; > > - PMU and SMBus registers (sort of normal BARs, but hidden elsewhere > > in the config space); > > - IDE legacy mode registers; > > - IO-APIC registers (typically sort of read-only BAR). > > > > For all of these address ranges our current BAR probe is effectively > > a no-op, but disable/re-enable clearly isn't. > > > >> Ultimately, I don't care much one way or another as long as we can get > >> the desktop platforms fixed somehow. I think disabling decode is the > >> most correct way of doing this, but I'm open to other solutions (this > >> is the only patch I've seen though that's been tested to solve the > >> problem). > > > > There are two other solutions: one is to disable decode selectively, > > only on devices or systems where it's necessary and known to be safe. > > I've posted a patch which introduces "disable_while_probe" pci_dev field > > for that purpose. > > Another one is to delay mmconfig probe until after the PCI probe is done, > > as Matthew suggested, and Robert confirmed that it's feasible. > > > for everyone who's using this quirk or has the same boot issue: I just > confirmed > that the new dg33tl bios update v0287 (released 9/20) fixes the boot issue > for my > systems. I encourage everyone to update their BIOS image and see if this > works. Thanks for letting us know. So, another reason to drop this patch :) thanks, greg k-h - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: PCI: Fix boot-time hang on G31/G33 PC
On Thu, Sep 27, 2007 at 11:36:32AM -0700, Kok, Auke wrote: Ivan Kokshaysky wrote: On Wed, Sep 26, 2007 at 03:20:40PM -0700, Jesse Barnes wrote: Ivan, your concern is about disabling things like interrupt controllers and power management chips during probe right? You're right that doing that could cause problems if we get and interrupt or PMU event at just the wrong time, but that could just as easily happen if decode was still enabled but the BAR had a bogus address programmed (as it would during probing). Yes, nobody is arguing that moving the BAR around is unsafe, but generally it's the less of two evils. The major problem here is that with IO and MEM bits cleared in the command register you disable *all* address decoders on the device, not just ranges that have respective BARs. At least this behaviour is required by PCI spec. Examples: - legacy VGA IO and memory (no corresponding BARs); - base/limit registers of P2P bridge; - PMU and SMBus registers (sort of normal BARs, but hidden elsewhere in the config space); - IDE legacy mode registers; - IO-APIC registers (typically sort of read-only BAR). For all of these address ranges our current BAR probe is effectively a no-op, but disable/re-enable clearly isn't. Ultimately, I don't care much one way or another as long as we can get the desktop platforms fixed somehow. I think disabling decode is the most correct way of doing this, but I'm open to other solutions (this is the only patch I've seen though that's been tested to solve the problem). There are two other solutions: one is to disable decode selectively, only on devices or systems where it's necessary and known to be safe. I've posted a patch which introduces disable_while_probe pci_dev field for that purpose. Another one is to delay mmconfig probe until after the PCI probe is done, as Matthew suggested, and Robert confirmed that it's feasible. for everyone who's using this quirk or has the same boot issue: I just confirmed that the new dg33tl bios update v0287 (released 9/20) fixes the boot issue for my systems. I encourage everyone to update their BIOS image and see if this works. Thanks for letting us know. So, another reason to drop this patch :) thanks, greg k-h - To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: PCI: Fix boot-time hang on G31/G33 PC
Ivan Kokshaysky wrote: > On Wed, Sep 26, 2007 at 03:20:40PM -0700, Jesse Barnes wrote: >> Ivan, your concern is about disabling things like interrupt controllers >> and power management chips during probe right? You're right that doing >> that could cause problems if we get and interrupt or PMU event at just >> the wrong time, but that could just as easily happen if decode was >> still enabled but the BAR had a bogus address programmed (as it would >> during probing). > > Yes, nobody is arguing that moving the BAR around is unsafe, but generally > it's the less of two evils. > > The major problem here is that with IO and MEM bits cleared in the command > register you disable *all* address decoders on the device, not just ranges > that have respective BARs. At least this behaviour is required by PCI spec. > Examples: > - legacy VGA IO and memory (no corresponding BARs); > - base/limit registers of P2P bridge; > - PMU and SMBus registers (sort of normal BARs, but hidden elsewhere > in the config space); > - IDE legacy mode registers; > - IO-APIC registers (typically sort of read-only BAR). > > For all of these address ranges our current BAR probe is effectively > a no-op, but disable/re-enable clearly isn't. > >> Ultimately, I don't care much one way or another as long as we can get >> the desktop platforms fixed somehow. I think disabling decode is the >> most correct way of doing this, but I'm open to other solutions (this >> is the only patch I've seen though that's been tested to solve the >> problem). > > There are two other solutions: one is to disable decode selectively, > only on devices or systems where it's necessary and known to be safe. > I've posted a patch which introduces "disable_while_probe" pci_dev field > for that purpose. > Another one is to delay mmconfig probe until after the PCI probe is done, > as Matthew suggested, and Robert confirmed that it's feasible. for everyone who's using this quirk or has the same boot issue: I just confirmed that the new dg33tl bios update v0287 (released 9/20) fixes the boot issue for my systems. I encourage everyone to update their BIOS image and see if this works. Cheers, Auke - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: PCI: Fix boot-time hang on G31/G33 PC
On Wed, Sep 26, 2007 at 03:20:40PM -0700, Jesse Barnes wrote: > Ivan, your concern is about disabling things like interrupt controllers > and power management chips during probe right? You're right that doing > that could cause problems if we get and interrupt or PMU event at just > the wrong time, but that could just as easily happen if decode was > still enabled but the BAR had a bogus address programmed (as it would > during probing). Yes, nobody is arguing that moving the BAR around is unsafe, but generally it's the less of two evils. The major problem here is that with IO and MEM bits cleared in the command register you disable *all* address decoders on the device, not just ranges that have respective BARs. At least this behaviour is required by PCI spec. Examples: - legacy VGA IO and memory (no corresponding BARs); - base/limit registers of P2P bridge; - PMU and SMBus registers (sort of normal BARs, but hidden elsewhere in the config space); - IDE legacy mode registers; - IO-APIC registers (typically sort of read-only BAR). For all of these address ranges our current BAR probe is effectively a no-op, but disable/re-enable clearly isn't. > Ultimately, I don't care much one way or another as long as we can get > the desktop platforms fixed somehow. I think disabling decode is the > most correct way of doing this, but I'm open to other solutions (this > is the only patch I've seen though that's been tested to solve the > problem). There are two other solutions: one is to disable decode selectively, only on devices or systems where it's necessary and known to be safe. I've posted a patch which introduces "disable_while_probe" pci_dev field for that purpose. Another one is to delay mmconfig probe until after the PCI probe is done, as Matthew suggested, and Robert confirmed that it's feasible. Ivan. - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: PCI: Fix boot-time hang on G31/G33 PC
On Wed, Sep 26, 2007 at 03:20:40PM -0700, Jesse Barnes wrote: Ivan, your concern is about disabling things like interrupt controllers and power management chips during probe right? You're right that doing that could cause problems if we get and interrupt or PMU event at just the wrong time, but that could just as easily happen if decode was still enabled but the BAR had a bogus address programmed (as it would during probing). Yes, nobody is arguing that moving the BAR around is unsafe, but generally it's the less of two evils. The major problem here is that with IO and MEM bits cleared in the command register you disable *all* address decoders on the device, not just ranges that have respective BARs. At least this behaviour is required by PCI spec. Examples: - legacy VGA IO and memory (no corresponding BARs); - base/limit registers of P2P bridge; - PMU and SMBus registers (sort of normal BARs, but hidden elsewhere in the config space); - IDE legacy mode registers; - IO-APIC registers (typically sort of read-only BAR). For all of these address ranges our current BAR probe is effectively a no-op, but disable/re-enable clearly isn't. Ultimately, I don't care much one way or another as long as we can get the desktop platforms fixed somehow. I think disabling decode is the most correct way of doing this, but I'm open to other solutions (this is the only patch I've seen though that's been tested to solve the problem). There are two other solutions: one is to disable decode selectively, only on devices or systems where it's necessary and known to be safe. I've posted a patch which introduces disable_while_probe pci_dev field for that purpose. Another one is to delay mmconfig probe until after the PCI probe is done, as Matthew suggested, and Robert confirmed that it's feasible. Ivan. - To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: PCI: Fix boot-time hang on G31/G33 PC
Ivan Kokshaysky wrote: On Wed, Sep 26, 2007 at 03:20:40PM -0700, Jesse Barnes wrote: Ivan, your concern is about disabling things like interrupt controllers and power management chips during probe right? You're right that doing that could cause problems if we get and interrupt or PMU event at just the wrong time, but that could just as easily happen if decode was still enabled but the BAR had a bogus address programmed (as it would during probing). Yes, nobody is arguing that moving the BAR around is unsafe, but generally it's the less of two evils. The major problem here is that with IO and MEM bits cleared in the command register you disable *all* address decoders on the device, not just ranges that have respective BARs. At least this behaviour is required by PCI spec. Examples: - legacy VGA IO and memory (no corresponding BARs); - base/limit registers of P2P bridge; - PMU and SMBus registers (sort of normal BARs, but hidden elsewhere in the config space); - IDE legacy mode registers; - IO-APIC registers (typically sort of read-only BAR). For all of these address ranges our current BAR probe is effectively a no-op, but disable/re-enable clearly isn't. Ultimately, I don't care much one way or another as long as we can get the desktop platforms fixed somehow. I think disabling decode is the most correct way of doing this, but I'm open to other solutions (this is the only patch I've seen though that's been tested to solve the problem). There are two other solutions: one is to disable decode selectively, only on devices or systems where it's necessary and known to be safe. I've posted a patch which introduces disable_while_probe pci_dev field for that purpose. Another one is to delay mmconfig probe until after the PCI probe is done, as Matthew suggested, and Robert confirmed that it's feasible. for everyone who's using this quirk or has the same boot issue: I just confirmed that the new dg33tl bios update v0287 (released 9/20) fixes the boot issue for my systems. I encourage everyone to update their BIOS image and see if this works. Cheers, Auke - To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: PCI: Fix boot-time hang on G31/G33 PC
Jesse Barnes wrote: On Wednesday, September 26, 2007 2:56 pm Greg KH wrote: On Wed, Sep 26, 2007 at 02:55:55PM -0700, Jesse Barnes wrote: On Wednesday, September 26, 2007 2:18 pm Greg KH wrote: Due to the issues surrounding this patch, I'm dropping it from my repo. What issues? Is it causing problems for people? I thought this was the patch that Ivan objected to. Yeah, Ivan objected to this, but incorrectly I think. Ivan, your concern is about disabling things like interrupt controllers and power management chips during probe right? You're right that doing that could cause problems if we get and interrupt or PMU event at just the wrong time, but that could just as easily happen if decode was still enabled but the BAR had a bogus address programmed (as it would during probing). Ultimately, I don't care much one way or another as long as we can get the desktop platforms fixed somehow. I think disabling decode is the most correct way of doing this, but I'm open to other solutions (this is the only patch I've seen though that's been tested to solve the problem). I agree, I've yet to see a single report of an actual problem that disabling the decode causes, nor even a theoretical problem which couldn't already happen due to the possibility of the device being accessed during BAR sizing, which just shouldn't be allowed since it can't work with or without this patch. Until and unless we have something better that fixes the real and serious boot-time problems that we need this patch to fix, I would say it should stay in. -- Robert Hancock Saskatoon, SK, Canada To email, remove "nospam" from [EMAIL PROTECTED] Home Page: http://www.roberthancock.com/ - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: PCI: Fix boot-time hang on G31/G33 PC
On Wednesday, September 26, 2007 2:56 pm Greg KH wrote: > On Wed, Sep 26, 2007 at 02:55:55PM -0700, Jesse Barnes wrote: > > On Wednesday, September 26, 2007 2:18 pm Greg KH wrote: > > > Due to the issues surrounding this patch, I'm dropping it from my > > > repo. > > > > What issues? Is it causing problems for people? > > I thought this was the patch that Ivan objected to. Yeah, Ivan objected to this, but incorrectly I think. Ivan, your concern is about disabling things like interrupt controllers and power management chips during probe right? You're right that doing that could cause problems if we get and interrupt or PMU event at just the wrong time, but that could just as easily happen if decode was still enabled but the BAR had a bogus address programmed (as it would during probing). Ultimately, I don't care much one way or another as long as we can get the desktop platforms fixed somehow. I think disabling decode is the most correct way of doing this, but I'm open to other solutions (this is the only patch I've seen though that's been tested to solve the problem). Jesse - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: PCI: Fix boot-time hang on G31/G33 PC
On Wed, Sep 26, 2007 at 02:55:55PM -0700, Jesse Barnes wrote: > On Wednesday, September 26, 2007 2:18 pm Greg KH wrote: > > Due to the issues surrounding this patch, I'm dropping it from my > > repo. > > What issues? Is it causing problems for people? I thought this was the patch that Ivan objected to. thanks, greg k-h - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: PCI: Fix boot-time hang on G31/G33 PC
On Wednesday, September 26, 2007 2:18 pm Greg KH wrote: > Due to the issues surrounding this patch, I'm dropping it from my > repo. What issues? Is it causing problems for people? Jesse - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: PCI: Fix boot-time hang on G31/G33 PC
Due to the issues surrounding this patch, I'm dropping it from my repo. thanks, greg k-h On Sat, Aug 25, 2007 at 07:55:56PM -0600, Matthew Wilcox wrote: > > > This patch, loosely based on a patch from Robert Hancock, which was in > turn based on a patch from Jesse Barnes, fixes a boot-time hang on my > shiny new PC. The 'conflict' mentioned in the patch in my case happens > to be between mmconfig and the graphics card, but it could easily be > between any pair of devices if they are left enabled by the BIOS and > mappen in the wrong place. > > Signed-off-by: Matthew Wilcox <[EMAIL PROTECTED]> > Acked-by: Robert Hancock <[EMAIL PROTECTED]> > Acked-by: Jesse Barnes <[EMAIL PROTECTED]> > Signed-off-by: Greg Kroah-Hartman <[EMAIL PROTECTED]> > > --- > drivers/pci/probe.c | 18 ++ > 1 file changed, 18 insertions(+) > > --- a/drivers/pci/probe.c > +++ b/drivers/pci/probe.c > @@ -180,11 +180,26 @@ static inline int is_64bit_memory(u32 ma > return 0; > } > > +/* > + * Sizing PCI BARs requires us to disable decoding, otherwise we may run > + * into conflicts with other devices while trying to size the BAR. Normally > + * this isn't a problem, but it happens on some machines normally, and can > + * happen on others during PCI device hotplug. Don't disable BARs for host > + * bridges, though. Some of them do silly things like disable accesses to > + * RAM from the CPU > + */ > static void pci_read_bases(struct pci_dev *dev, unsigned int howmany, int > rom) > { > unsigned int pos, reg, next; > u32 l, sz; > struct resource *res; > + u16 orig_cmd; > + > + if ((dev->class >> 8) != PCI_CLASS_BRIDGE_HOST) { > + pci_read_config_word(dev, PCI_COMMAND, _cmd); > + pci_write_config_word(dev, PCI_COMMAND, > + orig_cmd & ~(PCI_COMMAND_MEMORY | PCI_COMMAND_IO)); > + } > > for(pos=0; pos u64 l64; > @@ -283,6 +298,9 @@ static void pci_read_bases(struct pci_de > } > } > } > + > + if ((dev->class >> 8) != PCI_CLASS_BRIDGE_HOST) > + pci_write_config_word(dev, PCI_COMMAND, orig_cmd); > } > > void pci_read_bridge_bases(struct pci_bus *child) - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: PCI: Fix boot-time hang on G31/G33 PC
Due to the issues surrounding this patch, I'm dropping it from my repo. thanks, greg k-h On Sat, Aug 25, 2007 at 07:55:56PM -0600, Matthew Wilcox wrote: This patch, loosely based on a patch from Robert Hancock, which was in turn based on a patch from Jesse Barnes, fixes a boot-time hang on my shiny new PC. The 'conflict' mentioned in the patch in my case happens to be between mmconfig and the graphics card, but it could easily be between any pair of devices if they are left enabled by the BIOS and mappen in the wrong place. Signed-off-by: Matthew Wilcox [EMAIL PROTECTED] Acked-by: Robert Hancock [EMAIL PROTECTED] Acked-by: Jesse Barnes [EMAIL PROTECTED] Signed-off-by: Greg Kroah-Hartman [EMAIL PROTECTED] --- drivers/pci/probe.c | 18 ++ 1 file changed, 18 insertions(+) --- a/drivers/pci/probe.c +++ b/drivers/pci/probe.c @@ -180,11 +180,26 @@ static inline int is_64bit_memory(u32 ma return 0; } +/* + * Sizing PCI BARs requires us to disable decoding, otherwise we may run + * into conflicts with other devices while trying to size the BAR. Normally + * this isn't a problem, but it happens on some machines normally, and can + * happen on others during PCI device hotplug. Don't disable BARs for host + * bridges, though. Some of them do silly things like disable accesses to + * RAM from the CPU + */ static void pci_read_bases(struct pci_dev *dev, unsigned int howmany, int rom) { unsigned int pos, reg, next; u32 l, sz; struct resource *res; + u16 orig_cmd; + + if ((dev-class 8) != PCI_CLASS_BRIDGE_HOST) { + pci_read_config_word(dev, PCI_COMMAND, orig_cmd); + pci_write_config_word(dev, PCI_COMMAND, + orig_cmd ~(PCI_COMMAND_MEMORY | PCI_COMMAND_IO)); + } for(pos=0; poshowmany; pos = next) { u64 l64; @@ -283,6 +298,9 @@ static void pci_read_bases(struct pci_de } } } + + if ((dev-class 8) != PCI_CLASS_BRIDGE_HOST) + pci_write_config_word(dev, PCI_COMMAND, orig_cmd); } void pci_read_bridge_bases(struct pci_bus *child) - To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: PCI: Fix boot-time hang on G31/G33 PC
On Wednesday, September 26, 2007 2:18 pm Greg KH wrote: Due to the issues surrounding this patch, I'm dropping it from my repo. What issues? Is it causing problems for people? Jesse - To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: PCI: Fix boot-time hang on G31/G33 PC
On Wed, Sep 26, 2007 at 02:55:55PM -0700, Jesse Barnes wrote: On Wednesday, September 26, 2007 2:18 pm Greg KH wrote: Due to the issues surrounding this patch, I'm dropping it from my repo. What issues? Is it causing problems for people? I thought this was the patch that Ivan objected to. thanks, greg k-h - To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: PCI: Fix boot-time hang on G31/G33 PC
On Wednesday, September 26, 2007 2:56 pm Greg KH wrote: On Wed, Sep 26, 2007 at 02:55:55PM -0700, Jesse Barnes wrote: On Wednesday, September 26, 2007 2:18 pm Greg KH wrote: Due to the issues surrounding this patch, I'm dropping it from my repo. What issues? Is it causing problems for people? I thought this was the patch that Ivan objected to. Yeah, Ivan objected to this, but incorrectly I think. Ivan, your concern is about disabling things like interrupt controllers and power management chips during probe right? You're right that doing that could cause problems if we get and interrupt or PMU event at just the wrong time, but that could just as easily happen if decode was still enabled but the BAR had a bogus address programmed (as it would during probing). Ultimately, I don't care much one way or another as long as we can get the desktop platforms fixed somehow. I think disabling decode is the most correct way of doing this, but I'm open to other solutions (this is the only patch I've seen though that's been tested to solve the problem). Jesse - To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: PCI: Fix boot-time hang on G31/G33 PC
Jesse Barnes wrote: On Wednesday, September 26, 2007 2:56 pm Greg KH wrote: On Wed, Sep 26, 2007 at 02:55:55PM -0700, Jesse Barnes wrote: On Wednesday, September 26, 2007 2:18 pm Greg KH wrote: Due to the issues surrounding this patch, I'm dropping it from my repo. What issues? Is it causing problems for people? I thought this was the patch that Ivan objected to. Yeah, Ivan objected to this, but incorrectly I think. Ivan, your concern is about disabling things like interrupt controllers and power management chips during probe right? You're right that doing that could cause problems if we get and interrupt or PMU event at just the wrong time, but that could just as easily happen if decode was still enabled but the BAR had a bogus address programmed (as it would during probing). Ultimately, I don't care much one way or another as long as we can get the desktop platforms fixed somehow. I think disabling decode is the most correct way of doing this, but I'm open to other solutions (this is the only patch I've seen though that's been tested to solve the problem). I agree, I've yet to see a single report of an actual problem that disabling the decode causes, nor even a theoretical problem which couldn't already happen due to the possibility of the device being accessed during BAR sizing, which just shouldn't be allowed since it can't work with or without this patch. Until and unless we have something better that fixes the real and serious boot-time problems that we need this patch to fix, I would say it should stay in. -- Robert Hancock Saskatoon, SK, Canada To email, remove nospam from [EMAIL PROTECTED] Home Page: http://www.roberthancock.com/ - To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/