Shreeya Patel <shreeya.patel@xxxxxxxxxxxxx> writes: > When d_add_ci is called from the fs layer, we face a soft hang which is > caused by the deadlock in d_alloc_parallel. First patch in the series > tries to resolve it by doing a case-exact match instead of the > case-inexact match done by d_same_name function. Hi Shreeya, I understand what you are trying to solve here, but this could use some clarification. There is no such deadlock in the upstream code base, since d_add_ci is never called by a file system with a d_compare hook that causes the issue. Patch 02/02 will be the first to include such path, to address the /proc/self/cwd leakage, therefore, Patch 01/02 is done in preparation of that patch. That needs to be clearly stated here. Originally, the 'native', per-directory case-insensitive implementation merged in ext4/f2fs stores the case of the first lookup on the dcache, regardless of the disk exact file name case. This is intended as an internal implementation detail, that shouldn't be leaked to userspace. Whenever the kernel returns a name to userspace it should be the exact name, as written on disk. But, on /proc/self/cwd, the internal name is leaked to userspace. The goal of the series is *solely* to fix the leakage of this implementation detail to userspace. I think the solution is in the right direction, but I see some issues on the implementation I'm discussing inline. > The second patch resolves the inconsistent name that is exposed by >/proc/self/cwd in case of a case-insensitive filesystem. >/proc/self/cwd uses the dentry name stored in dcache. Since the dcache >is populated only on the first lookup, with the string used in that >lookup, cwd will have an unexpected case, depending on how the data was >first looked-up in a case-insesitive filesystem. > > > Shreeya Patel (2): > fs: dcache: Handle case-exact lookup in d_alloc_parallel > fs: ext4: Fix the inconsistent name exposed by /proc/self/cwd > > fs/dcache.c | 20 ++++++++++++++++++-- > fs/ext4/namei.c | 13 +++++++++++++ > 2 files changed, 31 insertions(+), 2 deletions(-) -- Gabriel Krisman Bertazi
