On Fri, Jan 24, 2020 at 10:03:23AM -0800, Jaegeuk Kim wrote: > On 01/24, Linus Torvalds wrote: > > On Thu, Jan 23, 2020 at 8:29 PM Eric Biggers <ebiggers@xxxxxxxxxx> wrote: > > > > > > Thanks Al. I sent out fixes for this: > > > > How did that f2fs_d_compare() function ever work? It was doing the > > memcmp on completely the wrong thing. > > Urg.. my bad. I didn't do enough stress test on casefolding case which > is only activated given "mkfs -C utf8:strict". And Android hasn't enabled > it yet. > It also didn't cause *really* obvious breakage because in practice it only caused ->d_compare() to incorrectly return false, and that just caused new dentries to be created rather than the existing ones reused. So most things continued to work. It can be noticed by way of deleting files not actually freeing up space... Or the way I noticed it is that my reproducer for the ->d_hash() crash worked on ext4 but not f2fs. - Eric
