Re: [f2fs-dev] [PATCH v6 0/9] Support negative dentries on case-insensitive ext4 and f2fs

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

To: "Theodore Ts'o" <tytso@xxxxxxx>
Subject: Re: [f2fs-dev] [PATCH v6 0/9] Support negative dentries on case-insensitive ext4 and f2fs
From: Al Viro <viro@xxxxxxxxxxxxxxxxxx>
Date: Wed, 22 Nov 2023 21:04:14 +0000
Cc: Linus Torvalds <torvalds@xxxxxxxxxxxxxxxxxxxx>, Christian Brauner <brauner@xxxxxxxxxx>, Gabriel Krisman Bertazi <krisman@xxxxxxx>, linux-f2fs-devel@xxxxxxxxxxxxxxxxxxxxx, ebiggers@xxxxxxxxxx, linux-fsdevel@xxxxxxxxxxxxxxx, jaegeuk@xxxxxxxxxx, linux-ext4@xxxxxxxxxxxxxxx
In-reply-to: <20231121051215.GA335601@mit.edu>
References: <20230816050803.15660-1-krisman@suse.de> <20231025-selektiert-leibarzt-5d0070d85d93@brauner> <655a9634.630a0220.d50d7.5063SMTPIN_ADDED_BROKEN@mx.google.com> <20231120-nihilismus-verehren-f2b932b799e0@brauner> <CAHk-=whTCWwfmSzv3uVLN286_WZ6coN-GNw=4DWja7NZzp5ytg@mail.gmail.com> <20231121020254.GB291888@mit.edu> <CAHk-=whb80quGmmgVcsq51cXw9dQ9EfNMi9otL9eh34jVZaD2g@mail.gmail.com> <CAHk-=wh+o0Zkzn=mtF6nB1b-EEcod-y4+ZWtAe7=Mi1v7RjUpg@mail.gmail.com> <20231121051215.GA335601@mit.edu>

On Tue, Nov 21, 2023 at 12:12:15AM -0500, Theodore Ts'o wrote:
> Yeah, agreed, that would be a nice optimization.  However, in the
> unfortunate case where (a) it's non-ASCII, and (b) the input string is
> non-normalized and/or differs in case, we end up scanning some portion
> of the two strings twice; once doing the strcmp, and once doing the
> Unicode slow path.

The first pass is cheap and the second one will be running entirely
from the cache, so I doubt that scanning them twice will be a loss...

References:
- [PATCH v6 0/9] Support negative dentries on case-insensitive ext4 and f2fs
  - From: Gabriel Krisman Bertazi
- Re: [PATCH v6 0/9] Support negative dentries on case-insensitive ext4 and f2fs
  - From: Christian Brauner
- Re: [f2fs-dev] [PATCH v6 0/9] Support negative dentries on case-insensitive ext4 and f2fs
  - From: Christian Brauner
- Re: [f2fs-dev] [PATCH v6 0/9] Support negative dentries on case-insensitive ext4 and f2fs
  - From: Linus Torvalds
- Re: [f2fs-dev] [PATCH v6 0/9] Support negative dentries on case-insensitive ext4 and f2fs
  - From: Theodore Ts'o
- Re: [f2fs-dev] [PATCH v6 0/9] Support negative dentries on case-insensitive ext4 and f2fs
  - From: Linus Torvalds
- Re: [f2fs-dev] [PATCH v6 0/9] Support negative dentries on case-insensitive ext4 and f2fs
  - From: Linus Torvalds
- Re: [f2fs-dev] [PATCH v6 0/9] Support negative dentries on case-insensitive ext4 and f2fs
  - From: Theodore Ts'o

Prev by Date: Re: [PATCH v6 3/9] fs: Expose name under lookup to d_revalidate hooks
Next by Date: Re: [f2fs-dev] [PATCH v6 0/9] Support negative dentries on case-insensitive ext4 and f2fs
Previous by thread: Re: [f2fs-dev] [PATCH v6 0/9] Support negative dentries on case-insensitive ext4 and f2fs
Next by thread: Re: [f2fs-dev] [PATCH v6 0/9] Support negative dentries on case-insensitive ext4 and f2fs
Index(es):
- Date
- Thread

[Index of Archives] [Reiser Filesystem Development] [Ceph FS] [Kernel Newbies] [Security] [Netfilter] [Bugtraq] [Linux FS] [Yosemite National Park] [MIPS Linux] [ARM Linux] [Linux Security] [Linux RAID] [Samba] [Device Mapper] [Linux Media]