On Wed, Nov 05, 2008 at 01:20:00PM +1100, Stephen Rothwell wrote:
> Hi Dave,
>
> On Thu, 30 Oct 2008 20:08:32 +1100 Dave Chinner <david@xxxxxxxxxxxxx> wrote:
> >
> > On Thu, Oct 30, 2008 at 01:23:24PM +1100, Stephen Rothwell wrote:
> > >
> > > Today's linux-next build (powerpc ppc64_defconfig) failed like this:
> > >
> > > ERROR: "inode_wait" [fs/xfs/xfs.ko] undefined!
> > >
> > > Caused by commit 676a7711ee281c3bf41db18e29e296584de82793 ("[XFS] Fix
> > > race when looking up reclaimable inodes") which introduced a use of
> > > wait_on_inode() which is an inline that calls inode_wait().
> >
> > Putting a non-exported symbol in an inline function in a header file
> > that is full of exported functions. That's pretty anti-social. ;)
> >
> > > I applied the following patch (probably wrong :-)).
> >
> > I think it's probably the only solution - we need to wait until
> > the I_LOCK bit is cleared from the inode and AFAICT wait_on_inode()
> > is the only way to do it.
>
> Any resolution?
I was waiting on a comment from Christoph. I'll cc him directly this
time ;)
Cheers,
Dave.
--
Dave Chinner
david@xxxxxxxxxxxxx
--
To unsubscribe from this list: send the line "unsubscribe linux-next" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
