Commit 460e36ee authored by Al Viro's avatar Al Viro Committed by Greg Kroah-Hartman
Browse files

resctrl: get rid of pointless debugfs_file_{get,put}() 



->write() of file_operations that gets used only via debugfs_create_file()
is called only under debugfs_file_get()

Signed-off-by: default avatarAl Viro <viro@zeniv.linux.org.uk>
Acked-by: default avatarReinette Chatre <reinette.chatre@intel.com>
Link: https://lore.kernel.org/r/20250702211650.GD3406663@ZenIV


Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 8009fb75

fs/resctrl/pseudo_lock.c

+0 −4
Original line number Diff line number Diff line
@@ -764,13 +764,9 @@ static ssize_t pseudo_lock_measure_trigger(struct file *file, 
	if (ret == 0) {

		if (sel != 1 && sel != 2 && sel != 3)

			return -EINVAL;

		ret = debugfs_file_get(file->f_path.dentry);

		if (ret)

			return ret;

		ret = pseudo_lock_measure_cycles(rdtgrp, sel);

		if (ret == 0)

			ret = count;

		debugfs_file_put(file->f_path.dentry);

	}



	return ret;