linux-kernel - Re: 2.6.19-rc3-mm1 -- missing network adaptors

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Wed, 1 Nov 2006 10:48:53 -0800
From:	Andrew Morton <akpm@...l.org>
To:	Mike Galbraith <efault@....de>
Cc:	Greg KH <gregkh@...e.de>, "Martin J. Bligh" <mbligh@...gle.com>,
	Cornelia Huck <cornelia.huck@...ibm.com>,
	Andy Whitcroft <apw@...dowen.org>,
	linux-kernel@...r.kernel.org, Steve Fox <drfickle@...ibm.com>
Subject: Re: 2.6.19-rc3-mm1 -- missing network adaptors

On Wed, 01 Nov 2006 10:26:24 +0100
Mike Galbraith <efault@....de> wrote:

> On Wed, 2006-11-01 at 07:12 +0100, Mike Galbraith wrote:
> > On Tue, 2006-10-31 at 21:25 -0800, Andrew Morton wrote:
> > > On Wed, 01 Nov 2006 05:43:18 +0100
> > > Mike Galbraith <efault@....de> wrote:
> > > 
> > > > On Tue, 2006-10-31 at 19:14 +0100, Mike Galbraith wrote:
> > > > 
> > > > > Seems it's driver-core-fixes-sysfs_create_link-retval-checks-in.patch
> > > > > 
> > > > > Tomorrow, I'll revert that alone from 2.6.19-rc3-mm1 to confirm...
> > > > 
> > > > Confirmed.  Boots fine with that patch reverted.
> > > 
> > > Could you test with something like this applied?
> > 
> > No output.  I had already enabled debugging, but got nada there either.
> > Bugger.  <scritch scritch>
> 
> Duh!  (what a maroon)  I booted the wrong kernel due to a typo.
> 
> I enabled some other debug options (poke/hope), and it now boots past
> the BUG at arch/i386/mm/pageattr.c:165 point, through the sound NULL
> pointer dereference, and on to the eventual complete hang as NFS is
> being initialized.  The log shows 326 failures at lines 385 and 589.

You mean 326 separate failures?  erp.

So it's failing here:

static int device_add_class_symlinks(struct device *dev)
{
	int error;

	if (!dev->class)
		return 0;
	error = sysfs_create_link(&dev->kobj, &dev->class->subsys.kset.kobj,
				  "subsystem");
	if (error) {
		DB();
		goto out;
	}
	error = sysfs_create_link(&dev->class->subsys.kset.kobj, &dev->kobj,
				  dev->bus_id);
	if (error) {
-->>		DB();
		goto out_subsys;
	}


Now, prior to driver-core-fixes-sysfs_create_link-retval-checks-in.patch we
were simply ignoring the return value of sysfs_create_link().  Now we're
not ignoring it and stuff is failing.

I'm suspecting that the second call to sysfs_create_link() in device_add():


	if (dev->class) {
		sysfs_create_link(&dev->kobj, &dev->class->subsys.kset.kobj,
				  "subsystem");
-->>		sysfs_create_link(&dev->class->subsys.kset.kobj, &dev->kobj,
				  dev->bus_id);

is simply always failing, only we never knew about it.

It would be useful if you could tell us what `error' is in there.  Usually
-EEXIST.

Greg, what is that call actually linking from and to?


-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/