On Tue 13-05-25 17:03:26, Max Kellermann wrote: > Allows killing processes that are waiting for the inode lock. > > Signed-off-by: Max Kellermann <max.kellermann@xxxxxxxxx> Looks good. Feel free to add: Reviewed-by: Jan Kara <jack@xxxxxxx> Honza > --- > v2: split into separate patches > > Review here (though nothing about do_truncate()): > https://lore.kernel.org/linux-fsdevel/20250512-unrat-kapital-2122d3777c5d@brauner/ > > Signed-off-by: Max Kellermann <max.kellermann@xxxxxxxxx> > --- > fs/open.c | 5 ++++- > 1 file changed, 4 insertions(+), 1 deletion(-) > > diff --git a/fs/open.c b/fs/open.c > index d2f2df52c458..7828234a7caa 100644 > --- a/fs/open.c > +++ b/fs/open.c > @@ -60,7 +60,10 @@ int do_truncate(struct mnt_idmap *idmap, struct dentry *dentry, > if (ret) > newattrs.ia_valid |= ret | ATTR_FORCE; > > - inode_lock(dentry->d_inode); > + ret = inode_lock_killable(dentry->d_inode); > + if (ret) > + return ret; > + > /* Note any delegations or leases have already been broken: */ > ret = notify_change(idmap, dentry, &newattrs, NULL); > inode_unlock(dentry->d_inode); > -- > 2.47.2 > -- Jan Kara <jack@xxxxxxxx> SUSE Labs, CR
