public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Ingo Molnar <mingo@elte.hu>
To: Jean Delvare <khali@linux-fr.org>
Cc: Stephen Hemminger <shemminger@vyatta.com>, linux-kernel@vger.kernel.org
Subject: Re: [bug] CONFIG_I2C_VIAPRO=y breaks skge
Date: Thu, 29 Jan 2009 15:27:59 +0100	[thread overview]
Message-ID: <20090129142759.GR24391@elte.hu> (raw)
In-Reply-To: <20090129151535.2fb84505@hyperion.delvare>


* Jean Delvare <khali@linux-fr.org> wrote:

> On Thu, 29 Jan 2009 14:49:16 +0100, Ingo Molnar wrote:
> > 
> > * Jean Delvare <khali@linux-fr.org> wrote:
> > > Well my guess is that you have included an I2C chip driver with a weak 
> > > detection routine. More likely this is a typical case of "if it hurts, 
> > > don't do it". But maybe we can improve the help text or default, or even 
> > > disable probing for these devices.
> > > 
> > > At this point I am reasonably certain that the i2c-viapro driver is 
> > > innocent.
> > 
> > i've excluded the VIAPRO driver on that box (and only the VIAPRO driver - 
> > i.e. all those other I2C drivers still run frequently) - and the problem 
> > has not reoccured in about ~1000 bootups.
> > 
> > Combined with the fact that this box has a VIA chipset, does that not 
> > implicate the viapro driver, at least to a certain degree?
> 
> This VIA chipset is the bridge over which the bugs run. Blaming on it 
> for the problem you've hit would be similar to blaming the USB host 
> controller driver for a bug that is in the USB keyboard or mouse driver.
> 
> The "other I2C drivers" don't run frequently at all now that you've 
> disabled i2c-viapro. Without an I2C bus to probe on the machine, these 
> drivers don't do anything.

ah, okay - i see - what i thought to be a surgical workaround has cut off 
the whole leg. Will go back to find the ailing muscle instead.

> So, again, if you are still willing to help me solve the problem, please 
> boot the bad kernel and run:
> 
> $ ls -l /sys/bus/i2c/devices/*/driver
> 
> Then we'll know which drivers attached to I2C devices on your SMBus. 
> Figuring out which one shouldn't have will likely be straightforward.

will take some time - but i'll revisit it.

Could you give me a short list of I2C driver .config options i should 
disable one by one (instead of viapro) ?

	Ingo

  reply	other threads:[~2009-01-29 14:28 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-01-27 15:13 [bug] CONFIG_I2C_VIAPRO=y breaks skge Ingo Molnar
2009-01-27 15:27 ` Jean Delvare
2009-01-27 15:44   ` Ingo Molnar
2009-01-27 17:26     ` Jean Delvare
2009-01-29 13:49       ` Ingo Molnar
2009-01-29 14:15         ` Jean Delvare
2009-01-29 14:27           ` Ingo Molnar [this message]
2009-01-29 15:14             ` Jean Delvare

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20090129142759.GR24391@elte.hu \
    --to=mingo@elte.hu \
    --cc=khali@linux-fr.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=shemminger@vyatta.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox