mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: "Fabio Comolli" <fabio.comolli@gmail.com>
To: "Jean Delvare" <khali@linux-fr.org>
Cc: "Bjorn Helgaas" <bjorn.helgaas@hp.com>,
	"Linux Kernel Mailing List" <linux-kernel@vger.kernel.org>,
	"Rene Herman" <rene.herman@keyaccess.nl>,
	"Andrew Morton" <akpm@osdl.org>,
	"Thomas Renninger" <trenn@suse.de>
Subject: Re: New conflict message in latest GIT
Date: Sat, 9 Aug 2008 18:32:31 +0200	[thread overview]
Message-ID: <b637ec0b0808090932l4d21d973we160c145f007da7c@mail.gmail.com> (raw)
In-Reply-To: <20080809181358.5c9da790@hyperion.delvare>

Hi Jean.

On Sat, Aug 9, 2008 at 6:13 PM, Jean Delvare <khali@linux-fr.org> wrote:
> Hi Fabio,
>
> On Wed, 23 Jul 2008 21:50:24 +0200, Fabio Comolli wrote:
>> Hi.
>>
>> On Wed, Jul 23, 2008 at 8:56 PM, Bjorn Helgaas <bjorn.helgaas@hp.com> wrote:
>> > On Tuesday 22 July 2008 12:56:36 pm Fabio Comolli wrote:
>> >> Linus' GIT tree 2.6.26-05752-g93ded9b shows this message:
>> >>
>> >> i801_smbus 0000:00:1f.3: PCI INT B -> GSI 19 (level, low) -> IRQ 19
>> >> ACPI: I/O resource 0000:00:1f.3 [0x18e0-0x18ff] conflicts with ACPI
>> >> region SMBI [0x18e0-0x18ef]
>> >> ACPI: Device needs an ACPI driver
>> >>
>> >> There is no equivalent in 2.6.26 or previous kernels.
>> >
>> > The "ACPI: I/O resource ... conflicts with ..." message was added by
>> > Thomas:
>> >  http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commitdiff;h=df92e695998e1bc6e426a840eb86d6d1ee87e2a5
>> >
>> > That conflict checking infrastructure was in 2.6.26, but Jean's
>> > change to make the i801_smbus driver use it didn't happen until
>> > about a week ago:
>> >  http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commitdiff;h=54fb4a05af0a4b814e6716cfdf3fa97fc6be7a32
>> >
>> > The message is telling us that the i801_smbus driver thinks it owns
>> > the 0x18e0-0x18ff region, but there's also an ACPI opregion that
>> > references that region.  There's no coordination between ACPI and
>> > the i801_smbus driver, so there may be issues where nearly
>> > simultaneous accesses cause incorrect behavior, e.g,. one may
>> > read the wrong value from a temperature sensor.  That, of course,
>> > can lead to more serious things like unintended machine shutdowns.
>> >
>> > I don't have any ideas about how to address this.  I think Thomas's
>> > intent was to collect better information for unreproducible bugs.
>> > (Maybe this sort of conflict should even set a taint flag?)
>
> Yes, at this point these messages are informative only and displayed as
> a hint when investigating bug reports. In the long run, we might
> decide to grant exclusive access to the shared region to either ACPI or
> the native driver, or to setup a safe concurrent access mechanism.
> That's a long way to go though, due to the diversity of BIOSes out
> there and the fact that many of them declare opregions in bogus ways.
>
>> OK, I actually didn't even know what i801_smbus (i2c_801 I suppose)
>> was. It seems that my laptop has a super-IO chip which is detected by
>> lm-sensors as `Nat. Semi. PC87591 Super IO' which doesn't have a
>> driver and never will.
>>
>> So, if I'm correct, this modules is totally useless for me and I
>> better compile it out. Am I correct?
>
> Yes you are. If you don't need i2c-i801 on this machine, best is to not
> build it or to prevent it from loading. Or you can boot with
> acpi_enforce_resources=strict to prevent the i2c-i801 driver from
> attaching to the device.
>

Ok, thanks for confirming it.

> I am a bit curious though, why ACPI would declare an opregion for a
> device that isn't used. Might be yet another case of BIOS copied from
> another machine and not cleaned up appropriately.
>

Well, you should ask HP for that :-)

Anyway, as usual linux is not supported at all on this laptop, it came
with XP. One thing that puzzles me is why bothering adding a
monitoring chip for which drivers do not exist for any OS?

> --
> Jean Delvare
>

Regards,
Fabio

  reply	other threads:[~2008-08-09 16:32 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-07-22 18:56 Fabio Comolli
2008-07-22 20:00 ` Rene Herman
2008-07-22 20:14   ` Rene Herman
2008-07-23  7:17     ` Fabio Comolli
2008-07-23  8:01       ` Rene Herman
2008-07-23 18:11         ` Fabio Comolli
2008-07-23 18:38           ` Rene Herman
2008-07-23 18:56 ` Bjorn Helgaas
2008-07-23 19:50   ` Fabio Comolli
2008-08-09 16:13     ` Jean Delvare
2008-08-09 16:32       ` Fabio Comolli [this message]
2008-08-09 17:03         ` Jean Delvare
2008-08-09 18:46           ` Fabio Comolli

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=b637ec0b0808090932l4d21d973we160c145f007da7c@mail.gmail.com \
    --to=fabio.comolli@gmail.com \
    --cc=akpm@osdl.org \
    --cc=bjorn.helgaas@hp.com \
    --cc=khali@linux-fr.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=rene.herman@keyaccess.nl \
    --cc=trenn@suse.de \
    /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

Powered by JetHome