lkml.org 
[lkml]   [2020]   [Jan]   [24]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 4.19 376/639] afs: Dont invalidate callback if AFS_VNODE_DIR_VALID not set
    Date
    From: David Howells <dhowells@redhat.com>

    [ Upstream commit d9052dda8a39069312218f913d22d99c48d90004 ]

    Don't invalidate the callback promise on a directory if the
    AFS_VNODE_DIR_VALID flag is not set (which indicates that the directory
    contents are invalid, due to edit failure, callback break, page reclaim).

    The directory will be reloaded next time the directory is accessed, so
    clearing the callback flag at this point may race with a reload of the
    directory and cancel it's recorded callback promise.

    Fixes: f3ddee8dc4e2 ("afs: Fix directory handling")
    Signed-off-by: David Howells <dhowells@redhat.com>
    Signed-off-by: Sasha Levin <sashal@kernel.org>
    ---
    fs/afs/inode.c | 7 ++-----
    1 file changed, 2 insertions(+), 5 deletions(-)

    diff --git a/fs/afs/inode.c b/fs/afs/inode.c
    index 718fab2f151a1..e6f11da5461be 100644
    --- a/fs/afs/inode.c
    +++ b/fs/afs/inode.c
    @@ -398,12 +398,9 @@ int afs_validate(struct afs_vnode *vnode, struct key *key)
    vnode->cb_s_break = vnode->cb_interest->server->cb_s_break;
    vnode->cb_v_break = vnode->volume->cb_v_break;
    valid = false;
    - } else if (vnode->status.type == AFS_FTYPE_DIR &&
    - (!test_bit(AFS_VNODE_DIR_VALID, &vnode->flags) ||
    - vnode->cb_expires_at - 10 <= now)) {
    + } else if (test_bit(AFS_VNODE_ZAP_DATA, &vnode->flags)) {
    valid = false;
    - } else if (test_bit(AFS_VNODE_ZAP_DATA, &vnode->flags) ||
    - vnode->cb_expires_at - 10 <= now) {
    + } else if (vnode->cb_expires_at - 10 <= now) {
    valid = false;
    } else {
    valid = true;
    --
    2.20.1


    \
     
     \ /
      Last update: 2020-01-24 12:22    [W:3.073 / U:0.496 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site