Messages in this thread | | | Date | Tue, 10 Dec 2013 13:43:40 -0500 (EST) | Subject | Re: [PATCH] net-tg3: Initialize REG_BASE_ADDR at PCI config offset 120 to 0 | From | David Miller <> |
| |
From: Michael Chan <mchan@broadcom.com> Date: Mon, 9 Dec 2013 22:58:44 -0800
> On Mon, 2013-12-09 at 13:07 -0800, Michael Chan wrote: >> On Tue, 2013-12-10 at 00:18 +0300, Sergei Shtylyov wrote: >> > > We had crashes when the PCI config space got scanned via >> > > /sys/devices/pcixxxx/....../config. >> > >> > > I agree that this fix will not help if the scan happens before the tg3 >> > > driver gets loaded. >> > >> > Then perhaps a better place for such fixup would be a PCI quirk? >> > >> Yes, I agree. Thanks. >> > > On second thought, I think your original patch should be sufficient and > we don't need to add the PCI quirk to cover so many devices. The reason > is that indirect access needs to be explicitly enabled in the > MISC_HOST_CTRL (0x68) register. The default value for register 0x68 > should have indirect access disabled. > > Nat, does this match what you're seeing? Did you ever see any system > crash before tg3 loads?
What if the kernel is booted via kexec, and the driver in the kernel we are kexec'ing from left indirect access enabled in MISC_HOST_CTRL?
| |