linux-i2c.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Martin Mokrejs <mmokrejs-08dBlVkRsZWoiTQjSSYKZesEoJ4y9sgM@public.gmane.org>
To: Jason Baron <jbaron-JqFfY2XvxFXQT0dZR+AlfA@public.gmane.org>,
	linux-i2c-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
Cc: asierra-AQeFf1F/bRxBDgjK7y7TUQ@public.gmane.org
Subject: Re: i2c_i801: status of use hardware semaphore patch?
Date: Fri, 22 Mar 2013 00:09:12 +0100	[thread overview]
Message-ID: <514B9318.4050902@fold.natur.cuni.cz> (raw)
In-Reply-To: <514B84AC.6040703-JqFfY2XvxFXQT0dZR+AlfA@public.gmane.org>

Jason Baron wrote:
> Hi,
> 
> I've also hit the i2c_i801 driver conflicts with ACPI region:
> 
> kernel: [73904.044722] i801_smbus 0000:00:1f.3: PCI INT C -> GSI 18
> (level, low) -> IRQ 18
> kernel: [73904.045573] ACPI: resource 0000:00:1f.3 [io  0xf000-0xf01f]
> conflicts with ACPI region SMBI [io 0xf000-0xf00f]
> kernel: [73904.046461] ACPI: If an ACPI driver is available for this
> device, you should use it instead of the native driver
> 
> I tested this patch: http://marc.info/?l=linux-i2c&m=135188429220187&w=2
> 
> And it avoids the warning (as we no longer check for acpi conflicts).
> However, I don't see the patch in any trees. Are there
> any reservations in applying it?


Hi Jason,
  I am getting the resource conflict on my laptop. Would you tell me
if this kmemleak reported by kernel could be fixed by the patch?
You say it will shut down the check and warning ... but is that
meaningful?


unreferenced object 0xffff88040ac7d690 (size 256):
  comm "swapper/0", pid 1, jiffies 4294937576 (age 30752.970s)
  hex dump (first 32 bytes):
    00 00 00 00 ad 4e ad de ff ff ff ff 00 00 00 00  .....N..........
    ff ff ff ff ff ff ff ff c8 6f 5e 82 ff ff ff ff  .........o^.....
  backtrace:
    [<ffffffff815bad38>] kmemleak_alloc+0x21/0x3e
    [<ffffffff811105d2>] slab_post_alloc_hook+0x28/0x2a
    [<ffffffff81112b0a>] __kmalloc+0xf2/0x104
    [<ffffffff8130542d>] kzalloc.constprop.14+0xe/0x10
    [<ffffffff8130588e>] device_private_init+0x14/0x63
    [<ffffffff81307974>] dev_set_drvdata+0x19/0x2f
    [<ffffffff815cb1c4>] i801_probe+0x5e/0x451
    [<ffffffff812810c8>] local_pci_probe+0x39/0x61
    [<ffffffff812821db>] pci_device_probe+0xc6/0xf3
    [<ffffffff81307f3d>] driver_probe_device+0xa9/0x1c1
    [<ffffffff813080af>] __driver_attach+0x5a/0x7e
    [<ffffffff813067d7>] bus_for_each_dev+0x5c/0x88
    [<ffffffff81307ada>] driver_attach+0x19/0x1b
    [<ffffffff813076aa>] bus_add_driver+0xa8/0x1fa
    [<ffffffff81308521>] driver_register+0x8c/0x106
    [<ffffffff81281dd6>] __pci_register_driver+0x59/0x5d




Thanks,
Martin

  parent reply	other threads:[~2013-03-21 23:09 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-03-21 22:07 i2c_i801: status of use hardware semaphore patch? Jason Baron
     [not found] ` <514B84AC.6040703-JqFfY2XvxFXQT0dZR+AlfA@public.gmane.org>
2013-03-21 23:09   ` Martin Mokrejs [this message]
     [not found]     ` <514B9318.4050902-08dBlVkRsZWoiTQjSSYKZesEoJ4y9sgM@public.gmane.org>
2013-03-25 21:30       ` Aaron Sierra

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=514B9318.4050902@fold.natur.cuni.cz \
    --to=mmokrejs-08dblvkrszwoitqjssykzeseoj4y9sgm@public.gmane.org \
    --cc=asierra-AQeFf1F/bRxBDgjK7y7TUQ@public.gmane.org \
    --cc=jbaron-JqFfY2XvxFXQT0dZR+AlfA@public.gmane.org \
    --cc=linux-i2c-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    /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;
as well as URLs for NNTP newsgroup(s).