linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
From: linux@arm.linux.org.uk (Russell King - ARM Linux)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 3/7] s3c-hsudc: add a remove function
Date: Sun, 18 Dec 2011 20:39:53 +0000	[thread overview]
Message-ID: <20111218203953.GY14542@n2100.arm.linux.org.uk> (raw)
In-Reply-To: <201112182124.13313.heiko@sntech.de>

On Sun, Dec 18, 2011 at 09:24:12PM +0100, Heiko St?bner wrote:
> Am Sonntag 18 Dezember 2011, 20:45:18 schrieb Russell King - ARM Linux:
> > On Sun, Dec 18, 2011 at 08:33:32PM +0100, Heiko St?bner wrote:
> > > Am Sonntag 18 Dezember 2011, 20:01:02 schrieben Sie:
> > > > On Sun, Dec 18, 2011 at 07:50:37PM +0100, Heiko St?bner wrote:
> > > > > I didn't get this far. With your patch the Oopses already happen
> > > > > during the startup of the system / the loading of the modules.
> > > > 
> > > > > A bit of the message spew I got during testing with linux-
> next-20111216:
> > > > In some way, this is a good thing because it's showing that there's
> > > > problems with kobject lifetime rules.
> > > > 
> > > > The #2 and further oops dumps are a result of corrupting the work
> > > > queues as a result of #1, so #2 onwards should be ignored.
> > > > 
> > > > I suspect if you avoid loading the s3c_hsudc module these will go away.
> > > 
> > > nope :-), same faults happen even if s3c-hsudc is not present at all.
> > > So it seems, this delayed cleanup poses problems for other drivers as
> > > well.
> > 
> > Okay, let's try to find out which one it is.  Please use the attached
> > patch - it'll be a little more noisy, reporting which kobjects are
> > being released at the point when they're added to the workqueue.
> The cuplrit seems to be a kobject named "holders" and from what I
> gathered is from kernel/module.c and handling module sysfs entries.
> 
> 
> Partial log below:
> 
> kobject: 'bq24022' (c78a9a80): kobject_release
> [...]
> kobject: 'gpio-vbus' (c78a9cc0): kobject_release
> [...]
> kobject: 'bq24022' (c78a9a80): kobject_cleanup
> kobject: 'gpio-vbus' (c78a9cc0): kobject_cleanup
> [...]
> Found /sbin/init, booting ...
> 
> INIT: version 2.88 booting
> 
> Starting the hotplug events dispatcher: udevdudevd[367]: starting version 172
> .
> Synthesizing the initial hotplug events...done.
> Waiting for /dev to be fully populated...
> [...]
> Cleaning up ifupdown....
> Loading kernel modules...
> kobject: 'holders' (c7addc80): kobject_release
> kobject: 'notes' (c7add080): kobject_release
> done.
> Activating lvm and md swap...done.
> Checking file systems...fsck from util-linux 2.19.1
> done.
> [...]
> kobject: 'holders' (c7addc80): kobject_cleanup
> Unable to handle kernel paging request at virtual address bf055504
> pgd = c0004000
> [bf055504] *pgd=371f9811, *pte=00000000, *ppte=00000000
> Internal error: Oops: 7 [#1]

Please post the entire first oops dump for the above run - it may contain
useful information to properly track this down.

Thanks.

  reply	other threads:[~2011-12-18 20:39 UTC|newest]

Thread overview: 24+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-12-17 19:23 [PATCH v2 0/7] s3c-hsudc: regulator handling and a lot of fixes Heiko Stübner
2011-12-17 19:24 ` [PATCH 1/7] s3c-hsudc: move platform_data struct to global header Heiko Stübner
2011-12-17 19:25 ` [PATCH 2/7] s3c-hsudc: add __devinit to probe function Heiko Stübner
2011-12-17 19:26 ` [PATCH 3/7] s3c-hsudc: add a remove function Heiko Stübner
2011-12-18  8:03   ` Russell King - ARM Linux
2011-12-18  8:10   ` Russell King - ARM Linux
2011-12-18  9:42     ` Heiko Stübner
2011-12-18 13:44     ` Heiko Stübner
2011-12-18 14:43       ` Russell King - ARM Linux
2011-12-18 18:50         ` Heiko Stübner
2011-12-18 19:01           ` Russell King - ARM Linux
2011-12-18 19:33             ` Heiko Stübner
2011-12-18 19:45               ` Russell King - ARM Linux
2011-12-18 20:24                 ` Heiko Stübner
2011-12-18 20:39                   ` Russell King - ARM Linux [this message]
2011-12-18 20:46                     ` Heiko Stübner
2011-12-18 21:37                       ` Russell King - ARM Linux
2011-12-20  6:08                         ` Greg KH
2011-12-20  6:07           ` Greg KH
2011-12-17 19:27 ` [PATCH 4/7] s3c-hsudc: add missing otg_put_transceiver in probe Heiko Stübner
2011-12-17 19:28 ` [PATCH 5/7] s3c-hsudc: move device registration to probe and remove Heiko Stübner
2011-12-18  8:09   ` Russell King - ARM Linux
2011-12-17 19:29 ` [PATCH 6/7] s3c-hsudc: use udc_start and udc_stop functions Heiko Stübner
2011-12-17 19:30 ` [PATCH 7/7] s3c-hsudc: Add regulator handling Heiko Stübner

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=20111218203953.GY14542@n2100.arm.linux.org.uk \
    --to=linux@arm.linux.org.uk \
    --cc=linux-arm-kernel@lists.infradead.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).