(Since Shawn didn't quite answer this piece) On Wed, May 24, 2017 at 09:04:33AM +0800, Shawn Lin wrote: > ? 2017/5/24 3:44, Bjorn Helgaas ??: > >On Fri, May 19, 2017 at 02:58:27PM +0800, Shawn Lin wrote: > >>This patch checks the link status before reading and > >>writing configure space of devices attached to the RC. > >>If the link status is down, we shouldn't try to access > >>the devices. > > > >What bad things happen without this patch? On this SoC, I've seen this sort of behavior (reading the config space when the device isn't responding) yield aborts, which panic the system. I can't speak exactly for which scenarios Shawn is addressing though. As I mentioned, this patch doesn't do anything for me so far. Brian
