From: Neil Brown <neilb@suse.de>
To: Greg KH <greg@kroah.com>
Date: Wed, 14 Feb 2007 09:39:10 +1100
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Transfer-Encoding: 7bit
Message-ID: <17874.15886.523387.629797@notabene.brown>
Cc: Andreas Gruenbacher <agruen@suse.de>, torvalds@linux-foundation.org,
       viro@zeniv.linux.org.uk, hch@lst.de, akpm@linux-foundation.org,
       linux-kernel@vger.kernel.org
Subject: Re: Bug in current -git tree causing dbus and gnome to chew up cpu time
In-Reply-To: message from Greg KH on Tuesday February 13
References: <20070213195116.GA5181@kroah.com>
	<200702131158.02809.agruen@suse.de>
	<20070213213737.GA14383@kroah.com>
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1041
Lines: 35

On Tuesday February 13, greg@kroah.com wrote:
> 
> This causes the following to just loop over and over, incrementing the
> pid:
> 
> gnome-vfs-daemo(8914): 

That's odd.  getcwd returning an empty string.
I can only see that happening if you do the equivalent of
   chdir("/");
   chroot("/somewhere");
   getcwd(buf,sizeof(buf));

That would return an empty string with the new code but not with the
old.
But I cannot imagine gnome-vfs-daemon doing a chroot, as it does not
run as root.

You could try removing the
		if (*buffer == '/') {
			buffer++;
			buflen++;
		}
from the new __d_path.  But that still wouldn't really explain what
was happening.

Maybe also put a WARN_ON(! *buffer) just before the return to find out
exactly what is going on when the empty string is returned.

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