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:I thought this was the patch that Ivan objected to.Due to the issues surrounding this patch, I'm dropping it from myWhat issues? Is it causing problems for people?
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).