From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 94AD5E7AD77 for ; Tue, 3 Oct 2023 16:48:36 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S240457AbjJCQsh (ORCPT ); Tue, 3 Oct 2023 12:48:37 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:45324 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231388AbjJCQse (ORCPT ); Tue, 3 Oct 2023 12:48:34 -0400 Received: from bee.birch.relay.mailchannels.net (bee.birch.relay.mailchannels.net [23.83.209.14]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 8F094A1 for ; Tue, 3 Oct 2023 09:48:30 -0700 (PDT) X-Sender-Id: dreamhost|x-authsender|kjlx@templeofstupid.com Received: from relay.mailchannels.net (localhost [127.0.0.1]) by relay.mailchannels.net (Postfix) with ESMTP id ABC318024BD for ; Tue, 3 Oct 2023 16:48:29 +0000 (UTC) Received: from pdx1-sub0-mail-a202.dreamhost.com (unknown [127.0.0.6]) (Authenticated sender: dreamhost) by relay.mailchannels.net (Postfix) with ESMTPA id 4B2D3801364 for ; Tue, 3 Oct 2023 16:48:29 +0000 (UTC) ARC-Seal: i=1; s=arc-2022; d=mailchannels.net; t=1696351709; a=rsa-sha256; cv=none; b=5clOh9oGaT7GQiYD7s5aPblRVYYTvEQwLay19LXKjPL1eWK8MZLn6WA/RMyRREnVtQqe1n 6bnMjNc7bt/GM8xYn6OUQW8oZTqqGqTE9FdxRARcafxSoKiNAUieD3B3gBYBagu9D/ibpY PdfToMge+yGSqt81mj2rfMd33RSdQvgT+6MIHMnnY+BYcziNgexKENYe2xT5NN2J4TpMD1 dGYfUykN/yDiG1g0xCnT0jmRrsU7RTVQ8qnYfpiFEJQ2uW0EhQnab2nX/nqJ6FNPHZJu6f k896Vd3ISLGvTS1DAQ4GnRpHRlTgBqlzInwm+DlCeRfKX6r6mVG5yG5YbU4QTw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=mailchannels.net; s=arc-2022; t=1696351709; 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:dkim-signature; bh=8ZwN54k0AS1FkbT6I9f/xqRXKZcXC1uk6boyOe63Xlo=; b=BnXh4JMzJrnXvTWptuzyGqfkDJpGM7WJUma7/PJE0JSTKWFTWtZxKFGkgDkeeUoaL1yI61 aLZ8reBGPQbpNE58YEDsZiHVfnMUzGZORueUsECSWPt+9lqL9JlOzhKYpta2P1YcXOgh47 oLfcbxj99sCgyWpXJv39azQM0JRNUeOB2BxATAKp7Q5TTzEq+S6xLz1KNRvUPZWkfohoKm rgbF9GYtIH+9Usyfl44sakv7jDIBxUSfqQd317y7odoeSAiXHEcn6VcUQrAG+cu5HpfXcR wXOcH3ABFnkUK4umvKK9lAjlEulp6b9m57pbK+mNQptcDOnrSPuBlyKPe2dJcQ== ARC-Authentication-Results: i=1; rspamd-7c449d4847-9n4sw; auth=pass smtp.auth=dreamhost smtp.mailfrom=kjlx@templeofstupid.com X-Sender-Id: dreamhost|x-authsender|kjlx@templeofstupid.com X-MC-Relay: Neutral X-MailChannels-SenderId: dreamhost|x-authsender|kjlx@templeofstupid.com X-MailChannels-Auth-Id: dreamhost X-Name-Vacuous: 70ea8f876c75d00a_1696351709560_617816540 X-MC-Loop-Signature: 1696351709560:1557507711 X-MC-Ingress-Time: 1696351709560 Received: from pdx1-sub0-mail-a202.dreamhost.com (pop.dreamhost.com [64.90.62.162]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384) by 100.99.159.76 (trex/6.9.1); Tue, 03 Oct 2023 16:48:29 +0000 Received: from kmjvbox (c-73-231-176-24.hsd1.ca.comcast.net [73.231.176.24]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) (Authenticated sender: kjlx@templeofstupid.com) by pdx1-sub0-mail-a202.dreamhost.com (Postfix) with ESMTPSA id 4S0P0F0MJnzGm for ; Tue, 3 Oct 2023 09:48:29 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=templeofstupid.com; s=dreamhost; t=1696351709; bh=8ZwN54k0AS1FkbT6I9f/xqRXKZcXC1uk6boyOe63Xlo=; h=Date:From:To:Cc:Subject:Content-Type; b=KZUXVBMeU78Ql0RidjXDN6sORrm3YqQgFoqhG2qSi67vKhjLcQduX5wsLRzPkBAQx AnBdWErcB2ZF2o5AFRFJkDuq9Zxc64nzdstP9sAHd80JX9CW2kLBM62bEuJvhw2tPd ZreImF4IFaVzRalCiTRHHgq8JhpMm1sZn4megmKX/l/tIvvsk8ai9WHdA3Lew9asb/ AnDZjlw+nj1VMu0vj0smQfZdclqelch7TtJdlDxdnaRsxPbft5SI2A72d7sBFYAtx9 t3VoMVg7OBFERy+qwGopqnypp+9Yyi+L7iCPrltcq3FHDPlFxS4BcNtKZYkM2w7R2x j1nuzj9tcx23Q== Received: from johansen (uid 1000) (envelope-from kjlx@templeofstupid.com) id e00c8 by kmjvbox (DragonFly Mail Agent v0.12); Tue, 03 Oct 2023 09:48:23 -0700 Date: Tue, 3 Oct 2023 09:48:23 -0700 From: Krister Johansen To: Bernd Schubert Cc: Krister Johansen , Miklos Szeredi , linux-fsdevel@vger.kernel.org, Miklos Szeredi , linux-kernel@vger.kernel.org, German Maglione , Greg Kurz , Max Reitz Subject: Re: [resend PATCH v2 0/2] virtiofs submounts that are still in use forgotten by shrinker Message-ID: <20231003164823.GA1995@templeofstupid.com> References: <97163cdf-ab2c-4fb8-abf2-738a4680c47f@fastmail.fm> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <97163cdf-ab2c-4fb8-abf2-738a4680c47f@fastmail.fm> Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Oct 03, 2023 at 12:18:42AM +0200, Bernd Schubert wrote: > > > On 10/2/23 17:24, Krister Johansen wrote: > > Hi, > > I recently ran into a situation where a virtiofs client began > > encountering EBADF after the client / guest system had an OOM. After > > reproducing the issue and debugging, the problem is caused by a > > virtiofsd submount having the nodeid of its root dentry fogotten. This > > occurs because it borrows the reference for this dentry from the parent > > that is passed into the function. > > > Sorry, I didn't forget you, just didn't manage to review the 2nd version > yet. Will definitely do this week. Thanks; I appreciate the feedback you've provided so far. > Please also note that there will be merge conflicts with atomic open patches > from Dharmendra/me. Although probably not too difficult to resolve. Sure. I'm happy to reparent, resolve those conflicts, re-test, and send another revision when we're ready. I suspect there are going to be additional changes requested on the v2. With that in mind, I'll hold off for the moment unless it is going to cause headaches for you. For the atomic-open-revalidate changes: should I be working from what's on the list? This is the most recent patchset I see: https://lore.kernel.org/linux-fsdevel/20230920173445.3943581-1-bschubert@ddn.com/ I found a 6.5 relative tree of yours on GitHub by following the libfuse pull request, but nothing that seemed in sync with fuse/for-next. Thanks, -K