[Devel] [PATCH RHEL9 COMMIT] Revert "mm/memflags/procfs: Allow userspace to set kernel memory allocation flags"

[Konstantin Khorenko]

The commit is pushed to "branch-rh9-5.14.0-162.18.1.vz9.19.x-ovz" and will appear at https://src.openvz.org/scm/ovz/vzkernel.git
after rh9-5.14.0-162.18.1.vz9.19.6
------>
commit c3d4818f7955e20ba582e07f1e8c43b888f963fc
Author: Konstantin Khorenko <khorenko at virtuozzo.com>
Date:   Thu Apr 20 16:16:37 2023 +0300

    Revert "mm/memflags/procfs: Allow userspace to set kernel memory allocation flags"
    
    This reverts commit 55d2e8ecb5d7b26b5bb8056f60c1ee99ae68a654.
    
    We will add same functionality but via prctl interface.
    
    https://jira.vzint.dev/browse/PSBM-141577
    
    Signed-off-by: Konstantin Khorenko <khorenko at virtuozzo.com>
---
 fs/proc/base.c | 77 ----------------------------------------------------------
 1 file changed, 77 deletions(-)

diff --git a/fs/proc/base.c b/fs/proc/base.c
index a2b4f09aaa56..0b900343aef6 100644
--- a/fs/proc/base.c
+++ b/fs/proc/base.c
@@ -1360,81 +1360,6 @@ static const struct file_operations proc_oom_score_adj_operations = {
 	.llseek		= default_llseek,
 };
 
-#define MEMALLOC_FLAGS_MASK (PF_MEMALLOC | PF_MEMALLOC_NOFS | \
-			     PF_MEMALLOC_NOIO | PF_MEMALLOC_PIN)
-
-static ssize_t memalloc_flags_read(struct file *file, char __user *buf,
-					size_t count, loff_t *ppos)
-{
-	struct task_struct *task = get_proc_task(file_inode(file));
-	char buffer[PROC_NUMBUF];
-	unsigned int pflags;
-	size_t len;
-
-	if (!task)
-		return -ESRCH;
-
-	pflags = READ_ONCE(task->flags) & MEMALLOC_FLAGS_MASK;
-	put_task_struct(task);
-	len = snprintf(buffer, sizeof(buffer), "%u\n", pflags);
-	return simple_read_from_buffer(buf, count, ppos, buffer, len);
-}
-
-static ssize_t memalloc_flags_write(struct file *file, const char __user *buf,
-					size_t count, loff_t *ppos)
-{
-	struct task_struct *task = get_proc_task(file_inode(file));
-	char buffer[PROC_NUMBUF] = {0};
-	int memalloc_flags;
-	int err = -ESRCH;
-	unsigned int pflags;
-
-	if (!task)
-		goto out;
-
-	if (!ve_is_super(get_exec_env())) {
-		err = -EPERM;
-		goto out;
-	}
-	/*
-	 * Potential issue here if task != current
-	 * concurrent setting of flags need synchronization
-	 * but currently flags are expected to change only from
-	 * current process so there is none.
-	 */
-	if (task != current) {
-		err = -EINVAL;
-		goto out;
-	}
-
-	count = min(count, sizeof(buffer) -1);
-	if (copy_from_user(buffer, buf, count)) {
-		err = -EFAULT;
-		goto out;
-	}
-
-	err = kstrtoint(strstrip(buffer), 0, &memalloc_flags);
-	if (err)
-		goto out;
-	if (memalloc_flags & ~MEMALLOC_FLAGS_MASK) {
-		err = -EINVAL;
-		goto out;
-	}
-
-	pflags = READ_ONCE(task->flags) & ~MEMALLOC_FLAGS_MASK;
-	WRITE_ONCE(task->flags, pflags | memalloc_flags);
-out:
-	if (task)
-		put_task_struct(task);
-	return err < 0 ? err : count;
-}
-
-static const struct file_operations proc_memalloc_flags_operations = {
-	.read		= memalloc_flags_read,
-	.write		= memalloc_flags_write,
-	.llseek		= default_llseek,
-};
-
 #ifdef CONFIG_AUDIT
 #define TMPBUFLEN 11
 static ssize_t proc_loginuid_read(struct file * file, char __user * buf,
@@ -3475,7 +3400,6 @@ static const struct pid_entry tgid_base_stuff[] = {
 	ONE("oom_score",  S_IRUGO, proc_oom_score),
 	REG("oom_adj",    S_IRUGO|S_IWUSR, proc_oom_adj_operations),
 	REG("oom_score_adj", S_IRUGO|S_IWUSR, proc_oom_score_adj_operations),
-	REG("memalloc_flags", S_IRUGO|S_IWUSR, proc_memalloc_flags_operations),
 #ifdef CONFIG_AUDIT
 	REG("loginuid",   S_IWUSR|S_IRUGO, proc_loginuid_operations),
 	REG("sessionid",  S_IRUGO, proc_sessionid_operations),
@@ -3825,7 +3749,6 @@ static const struct pid_entry tid_base_stuff[] = {
 	ONE("oom_score", S_IRUGO, proc_oom_score),
 	REG("oom_adj",   S_IRUGO|S_IWUSR, proc_oom_adj_operations),
 	REG("oom_score_adj", S_IRUGO|S_IWUSR, proc_oom_score_adj_operations),
-	REG("memalloc_flags", S_IRUGO|S_IWUSR, proc_memalloc_flags_operations),
 #ifdef CONFIG_AUDIT
 	REG("loginuid",  S_IWUSR|S_IRUGO, proc_loginuid_operations),
 	REG("sessionid",  S_IRUGO, proc_sessionid_operations),