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 lists.sourceforge.net (lists.sourceforge.net [216.105.38.7]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id E235BEB64DA for ; Thu, 20 Jul 2023 07:43:34 +0000 (UTC) Received: from [127.0.0.1] (helo=sfs-ml-4.v29.lw.sourceforge.com) by sfs-ml-4.v29.lw.sourceforge.com with esmtp (Exim 4.95) (envelope-from ) id 1qMOJw-0007Cy-1L; Thu, 20 Jul 2023 07:43:32 +0000 Received: from [172.30.20.202] (helo=mx.sourceforge.net) by sfs-ml-4.v29.lw.sourceforge.com with esmtps (TLS1.2) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.95) (envelope-from ) id 1qMOJu-0007Cs-Bt for linux-f2fs-devel@lists.sourceforge.net; Thu, 20 Jul 2023 07:43:30 +0000 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=sourceforge.net; s=x; h=In-Reply-To:Content-Type:MIME-Version:References: Message-ID:Subject:Cc:To:From:Date:Sender:Reply-To:Content-Transfer-Encoding: Content-ID:Content-Description:Resent-Date:Resent-From:Resent-Sender: Resent-To:Resent-Cc:Resent-Message-ID:List-Id:List-Help:List-Unsubscribe: List-Subscribe:List-Post:List-Owner:List-Archive; bh=vc+6mn4GLjQ4v07KThVfWUEi9QVria8rDdcKrT/ysjw=; b=AeMhUGJLmk1ZjNI+ipsyGfQ89j dCejYsVFg6gUgILEU0LsCERYjACyTzEkPmrCuDTAUdw914CdfQ44ZKpgcoLqhOrxfJoTDwjqYO+Ki 4QarffKO+rTt6XWMXgCqKJKFPfqK2hvs4xjSt4y99yu/ScD3sYDgaaFhDexonN3inL9A=; DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=sf.net; s=x ; h=In-Reply-To:Content-Type:MIME-Version:References:Message-ID:Subject:Cc:To :From:Date:Sender:Reply-To:Content-Transfer-Encoding:Content-ID: Content-Description:Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc :Resent-Message-ID:List-Id:List-Help:List-Unsubscribe:List-Subscribe: List-Post:List-Owner:List-Archive; bh=vc+6mn4GLjQ4v07KThVfWUEi9QVria8rDdcKrT/ysjw=; b=iQMfEX5yJJyf5bL/l+98LUR0VM gcpydTuMN8HQvxZrM/WsOnMzw/H0wMvdpeYbgxCZ0HyUKYzkMjCSU6UzXLt7Gpn3E/hO34AdFCRwZ GAuHSISA+amkMsOKErsmcsNyX7UAljqqkYW5x9dxmiOpA5uAwfGHYNAv81LKmpmPqI9o=; Received: from dfw.source.kernel.org ([139.178.84.217]) by sfi-mx-1.v28.lw.sourceforge.com with esmtps (TLS1.2:ECDHE-RSA-AES256-GCM-SHA384:256) (Exim 4.95) id 1qMOJu-00Dpor-4d for linux-f2fs-devel@lists.sourceforge.net; Thu, 20 Jul 2023 07:43:30 +0000 Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits)) (No client certificate requested) by dfw.source.kernel.org (Postfix) with ESMTPS id 1264D618FA; Thu, 20 Jul 2023 07:43:21 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id F3FBDC433C8; Thu, 20 Jul 2023 07:43:19 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1689839000; bh=wkKDK2+s1pOg2JvqCSmY6SM9Z1YuLDwx2K3C/NKe7xI=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=eizBjjVGnO2vg1Tt2xjCxt2izjubXLk5XoIzez+G7NV6q1Pov2b3Jowbwu6LF+wXa DNbJBORvYQCs0/CSWTrRQCmYn6FsD3maICWoXOwort8kKP5IwPbtFXh0cPUvhaD40s ExnEkERouBZ6IjvHtIe0E+mUHUFZGymwvdhnJCHYdompFyjMBUL9wWaF6UroxpCZqR Dcf1rsRIHmOSHhnxJooVrw450EAg1dAqIDazslQUWsPdBfInFNqjaoEzFkfSmavNZK WkRV1J2XALtQTXEZC0xXWSOWH7LG+fDbExR5IFjiJMxBKUO/yvbkRuAfAebaoIjELj 0C2zvsGLMPibA== Date: Thu, 20 Jul 2023 00:43:18 -0700 From: Eric Biggers To: Gabriel Krisman Bertazi Message-ID: <20230720074318.GA56170@sol.localdomain> References: <20230719221918.8937-1-krisman@suse.de> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20230719221918.8937-1-krisman@suse.de> X-Headers-End: 1qMOJu-00Dpor-4d Subject: Re: [f2fs-dev] [PATCH v3 0/7] Support negative dentries on case-insensitive ext4 and f2fs X-BeenThere: linux-f2fs-devel@lists.sourceforge.net X-Mailman-Version: 2.1.21 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: brauner@kernel.org, tytso@mit.edu, linux-f2fs-devel@lists.sourceforge.net, viro@zeniv.linux.org.uk, linux-fsdevel@vger.kernel.org, jaegeuk@kernel.org, linux-ext4@vger.kernel.org Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: linux-f2fs-devel-bounces@lists.sourceforge.net Sorry, one more thing... On Wed, Jul 19, 2023 at 06:19:11PM -0400, Gabriel Krisman Bertazi wrote: > > Another problem exists when turning a negative dentry to positive. If > the negative dentry has a different case than what is currently being > used for lookup, the dentry cannot be reused without changing its name, > in order to guarantee filename-preserving semantics to userspace. We > need to either change the name or invalidate the dentry. This issue is > currently avoided in mainline, since the negative dentry mechanism is > disabled. Are you sure this problem even needs to be solved? It actually isn't specific to negative dentries. If you have a file "foo" that's not in the dcache, and you open it (or look it up in any other way) as "FOO", then the positive dentry that gets created is named "FOO". As a result, the name that shows up in /proc/$pid/fd/ for anyone who has the file open is "FOO", not the true name "foo". This is true even for processes that open it as "foo", as long as the dentry remains in the dcache. No negative dentries involved at all! Is your thinking that you just don't want to increase the number of ways in which this behavior can occur? Or, it looks like the positive dentry case is solvable using d_add_ci(). So maybe you are planning to do that? It's not clear to me. - Eric _______________________________________________ Linux-f2fs-devel mailing list Linux-f2fs-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel