Received: by 2002:a25:ca44:0:0:0:0:0 with SMTP id a65csp2202924ybg; Thu, 30 Jul 2020 13:06:26 -0700 (PDT) X-Google-Smtp-Source: ABdhPJzW261HbCcpso3KicesHaIpcCADWsPgadZ1szXRHj8k7XflgEnNEU3KSVMO6oUyui1cJ7Qs X-Received: by 2002:a05:6402:31b5:: with SMTP id dj21mr701739edb.154.1596139586082; Thu, 30 Jul 2020 13:06:26 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1596139586; cv=none; d=google.com; s=arc-20160816; b=SxPdxZQrRp3atrzvhwzlnectmwzrPEVn20OzpA/iZpJj4Lw8L5iyXsIia5hRInvJyd IPecWI6YOxPfA7KZi7O6zI5vwYc7dyF8CrzhUkDy5XlsHIl+skgObEE9pJJSJI00plqC kCJsGZlByjcvgEdzBfCNMfrbYYE2TFDg9USLmf9TcUIbz9Iy0wyl66RDG5jj3HHaPDqG pZz9LjQbZXKVmrf+JNEJwD1LipL6p9w5smcOYq9h9sT4xUO8e7N+i369ms1BkJREUEUa punuUqQZuR/s8MqrREFBDJ6m961wUh1LNdIxeqFlxYJp8v8tgpOTDSFXW2atovH8Fy7a bNzA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:message-id:date:content-id:mime-version :subject:cc:to:references:in-reply-to:from:organization :dkim-signature; bh=BKn7BpC5Yl43Dk6OZEWBG/yR01/4YmaH6m9WnDrhG3w=; b=Fcd/QKZdjGZfl/mzawcxX9+VTUa7FMcPbZefIanJg+Wn8GL9779Ay3D9iOVIe3EK3p 8EKzrrCcVzGt9R+7dUoZf8LoqZVLTfRmQv06bpwZCk+REyMJ50/cvgwSoW5docTqJ4/Z Kosjc/juPLhrd2S5kbPEg/CZaoRBbwQFV5olkpUFODJomejUiiquosR86J7ki8089Nit AAjinlUBmyD4yPRMtUIdIRCMMmrVJPa0kSbzFE7N3iTQrp38R5kZwzF74wt5+4rXuID1 n2ODCbIMU3wAEwIvFrWdpMQ9PfZGiHyjxt4IIV6pitXmHS0oabzLS8mnJbcZmENumT79 NcFA== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@redhat.com header.s=mimecast20190719 header.b=QG1W4nhe; spf=pass (google.com: domain of linux-nfs-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-nfs-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=redhat.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id u8si4362117eda.396.2020.07.30.13.06.02; Thu, 30 Jul 2020 13:06:26 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-nfs-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@redhat.com header.s=mimecast20190719 header.b=QG1W4nhe; spf=pass (google.com: domain of linux-nfs-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-nfs-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=redhat.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726957AbgG3UFP (ORCPT + 99 others); Thu, 30 Jul 2020 16:05:15 -0400 Received: from us-smtp-1.mimecast.com ([205.139.110.61]:24182 "EHLO us-smtp-delivery-1.mimecast.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1726939AbgG3UFP (ORCPT ); Thu, 30 Jul 2020 16:05:15 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1596139514; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=BKn7BpC5Yl43Dk6OZEWBG/yR01/4YmaH6m9WnDrhG3w=; b=QG1W4nhelwefq28epF9B5pOLbYsmnpMEoC2FLLy9e47ARnDaLzI3r2W+P4R/Rh1/n2JGyv 5c96ArKLgc1B7jH78EXWc73Ly93zeDLbViH/SsVhaZdmf1ZvBf2V9oXt9fHDov6a20VRMp JRyGIhvZvy7jPyoom+X7vjQ3A76saNE= Received: from mimecast-mx01.redhat.com (mimecast-mx01.redhat.com [209.132.183.4]) (Using TLS) by relay.mimecast.com with ESMTP id us-mta-328-kKQeCJp-O7OXk9OZWW4V7w-1; Thu, 30 Jul 2020 16:05:11 -0400 X-MC-Unique: kKQeCJp-O7OXk9OZWW4V7w-1 Received: from smtp.corp.redhat.com (int-mx08.intmail.prod.int.phx2.redhat.com [10.5.11.23]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx01.redhat.com (Postfix) with ESMTPS id 6EBDE18839EB; Thu, 30 Jul 2020 20:03:56 +0000 (UTC) Received: from warthog.procyon.org.uk (ovpn-112-32.rdu2.redhat.com [10.10.112.32]) by smtp.corp.redhat.com (Postfix) with ESMTP id 5282219D7B; Thu, 30 Jul 2020 20:03:52 +0000 (UTC) Organization: Red Hat UK Ltd. Registered Address: Red Hat UK Ltd, Amberley Place, 107-111 Peascod Street, Windsor, Berkshire, SI4 1TE, United Kingdom. Registered in England and Wales under Company Registration No. 3798903 From: David Howells In-Reply-To: References: <1596031949-26793-1-git-send-email-dwysocha@redhat.com> <1596031949-26793-14-git-send-email-dwysocha@redhat.com> <43e8a8ff1ea015bb7bd335d5616268d36155327a.camel@redhat.com> To: David Wysochanski Cc: dhowells@redhat.com, Jeff Layton , Trond Myklebust , Anna Schumaker , linux-nfs , linux-cachefs@redhat.com Subject: Re: [Linux-cachefs] [RFC PATCH v2 13/14] NFS: Call fscache_resize_cookie() when inode size changes due to setattr MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-ID: <538845.1596139431.1@warthog.procyon.org.uk> Date: Thu, 30 Jul 2020 21:03:51 +0100 Message-ID: <538846.1596139431@warthog.procyon.org.uk> X-Scanned-By: MIMEDefang 2.84 on 10.5.11.23 Sender: linux-nfs-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-nfs@vger.kernel.org David Wysochanski wrote: > To be honest I'm not sure about needing a call to fscache_use/unuse_cookie() > around the call to fscache_resize_cookie(). If the cookie has a > refcount of 1 when it is created, and a file is never opened, so > we never call fscache_use_cookie(), what might happen inside > fscache_resize_cookie()? The header on use_cookie() says I've have afs_setattr() doing use/unuse on the cookie around resize. David