Thanks, I will apply this.

I also saw earlier last week you fixed a bug on au_unpin.
I haven't applied that - you think I should to that too?

cheers!

On Thu, 24 Jul 2014 01:15:36 +0900
sf...@users.sourceforge.net wrote:

> 
> James B:
> > Okay now I confirm that it is zero. I had to reboot the box as it gave me 
> > rcu sched stall anyway ... so this time around I will run with the default 
> > printk, it seems to be giving me some output from "dmesg -w" (I've got 
> > something like "d-1" or "i-1 aufs do_rename" followed by the name of the 
> > file being modified ... will that be useful?
> 
> Just before the crash is most meaningful.
> 
> I've just found a bug in aufs_rename(). But I am not sure this is your
> case. I'd just ask you to try this patch.
> 
> 
> J. R. Okajima
> 
> diff --git a/fs/aufs/i_op_ren.c b/fs/aufs/i_op_ren.c
> index 61a6516..9c95deb 100644
> --- a/fs/aufs/i_op_ren.c
> +++ b/fs/aufs/i_op_ren.c
> @@ -245,7 +245,8 @@ static int au_ren_diropq(struct au_ren_args *a)
>       au_hn_imtx_unlock(a->src_hinode);
>       if (IS_ERR(diropq))
>               err = PTR_ERR(diropq);
> -     dput(diropq);
> +     else
> +             dput(diropq);
>  
>       return err;
>  }


-- 
James B <jamesbond3...@gmail.com>

------------------------------------------------------------------------------
Want fast and easy access to all the code in your enterprise? Index and
search up to 200,000 lines of code with a free copy of Black Duck
Code Sight - the same software that powers the world's largest code
search on Ohloh, the Black Duck Open Hub! Try it now.
http://p.sf.net/sfu/bds

Reply via email to