Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1459537
| From | Linus Torvalds <torvalds@linux-foundation.org> |
|---|---|
| Newsgroups | linux.kernel |
| Subject | Re: [lkp] [nfsd] b44061d0b9: BUG: Dentry ffff880027d7c540{i=1846f,n=0a} still in use (1) [unmount of btrfs vda] |
| Date | 2016-08-10 21:00 +0200 |
| Message-ID | <s4GTo-n4-17@gated-at.bofh.it> (permalink) |
| References | <s4GTo-n4-19@gated-at.bofh.it> <s4GTo-n4-21@gated-at.bofh.it> <s4GTo-n4-23@gated-at.bofh.it> <s4GTo-n4-25@gated-at.bofh.it> <s4GTo-n4-27@gated-at.bofh.it> |
| Organization | linux.* mail to news gateway |
On Wed, Aug 10, 2016 at 11:46 AM, Josef Bacik <jbacik@fb.com> wrote:
>
> So my naive fix would be something like this
Bruce? Josef's patch looks ObviouslyCorrect(tm) to me now that I look
at it - all the other callers of fh_compose() also seem to just drop
the dentry unconditionally, knowing that fh_compose() took a ref to it
if needed.
In fact, the only thing I'd do differently would be to not even put
the comment there at all, since this call site isn't any different
from any of the others. If anything, it could go on fh_compose() if we
want to add comments.
Linus
Back to linux.kernel | Previous | Next — Next in thread | Find similar | Unroll thread
Re: [lkp] [nfsd] b44061d0b9: BUG: Dentry ffff880027d7c540{i=1846f,n=0a} still in use (1) [unmount of btrfs vda] Linus Torvalds <torvalds@linux-foundation.org> - 2016-08-10 21:00 +0200
Re: [lkp] [nfsd] b44061d0b9: BUG: Dentry ffff880027d7c540{i=1846f,n=0a} still in use (1) [unmount of btrfs vda] "J. Bruce Fields" <bfields@redhat.com> - 2016-08-10 21:10 +0200
Re: [lkp] [nfsd] b44061d0b9: BUG: Dentry ffff880027d7c540{i=1846f,n=0a} still in use (1) [unmount of btrfs vda] Linus Torvalds <torvalds@linux-foundation.org> - 2016-08-10 21:20 +0200
csiph-web