Received: by 2002:a25:1506:0:0:0:0:0 with SMTP id 6csp2619066ybv; Mon, 24 Feb 2020 08:28:48 -0800 (PST) X-Google-Smtp-Source: APXvYqxN8jH+HwcC9yAezrHMn1eqfCDYXhDYFbWnu4yI0C9j98bWRtCfNgjJBHvsXBhVi7BPpA6E X-Received: by 2002:a05:6830:134c:: with SMTP id r12mr18399136otq.126.1582561728067; Mon, 24 Feb 2020 08:28:48 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1582561728; cv=none; d=google.com; s=arc-20160816; b=X1fz5yVpy0SDiqhht2Sto0tRBYJcrvQYMshXnW79nSSoQvLjHT51UoDnmw2bM+dQmz vGN2PKOwWv54nnpxZEJuzCuJ3Rg0cXDLua0ov7bosXPIxJHwNVbBVXxs55iHJbJ7vjUb 0gia5o7p83cQsI4CkzDgE/PFK5i5xXLgh1NndBjGM7PdZ6jtI7Yb199bXf2vwVg/xxnr CbARwTDmsqrAE6Yn+WES3A/h5/SF2Lyw4RWUAZWuwcOpD60igF5NMQ7KbalMuAxWoY+A 6aCRR/5G8FkTUP2IUMWo0PB0jhs7y8a6lk6EaEWseHs+3uKt8grxkZcS1uw++1PD6JbT 4rMA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:subject:mime-version:user-agent :message-id:in-reply-to:date:references:cc:to:from; bh=JTWarDhdErYLOTI5MmwWDNBukeZW31kkvg6szTtqkSk=; b=RlSb0/NOKwyjHm1Jt3jL264yO5j2RLD55C/m2avfSHsWlt71TLR7t+zI/o1TyTFwt8 YKJqGOnQxUOQnHvrkFkiKkpPtT5StBSnW6GU5RERcdFLjxZLmrRsCDkLvk5M4VaKkWxH RGmLRjrj7ufdYJnvV7KzwyCr5cH1zWf1Lwausz32jPnJV/dVPlGg9acnL23NJRM5nIt5 Du8f89ZWMIvT9tUR94MtZ0c3w5Cd9naS8ea67X7/b9o3sb9s3c9jce7oD3MHE7H+Ke0S yQLMQGLSAW4kRKka6nwP9FfjokQnQXTopmkPcKQz/BiF0g0z0yUidTtaBlTLryCSw4UU PTHg== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=xmission.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id r4si6299405otq.188.2020.02.24.08.28.34; Mon, 24 Feb 2020 08:28:48 -0800 (PST) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=xmission.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727854AbgBXQ1Y (ORCPT + 99 others); Mon, 24 Feb 2020 11:27:24 -0500 Received: from out02.mta.xmission.com ([166.70.13.232]:48850 "EHLO out02.mta.xmission.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1727160AbgBXQ1X (ORCPT ); Mon, 24 Feb 2020 11:27:23 -0500 Received: from in01.mta.xmission.com ([166.70.13.51]) by out02.mta.xmission.com with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1j6GZk-0004Dy-PN; Mon, 24 Feb 2020 09:27:20 -0700 Received: from ip68-227-160-95.om.om.cox.net ([68.227.160.95] helo=x220.xmission.com) by in01.mta.xmission.com with esmtpsa (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.87) (envelope-from ) id 1j6GZk-0007ND-2p; Mon, 24 Feb 2020 09:27:20 -0700 From: ebiederm@xmission.com (Eric W. Biederman) To: Linus Torvalds Cc: Al Viro , LKML , Kernel Hardening , Linux API , Linux FS Devel , Linux Security Module , Akinobu Mita , Alexey Dobriyan , Andrew Morton , Andy Lutomirski , Daniel Micay , Djalal Harouni , "Dmitry V . Levin" , Greg Kroah-Hartman , Ingo Molnar , "J . Bruce Fields" , Jeff Layton , Jonathan Corbet , Kees Cook , Oleg Nesterov , Solar Designer , Alexey Gladkov References: <20200210150519.538333-8-gladkov.alexey@gmail.com> <87v9odlxbr.fsf@x220.int.ebiederm.org> <20200212144921.sykucj4mekcziicz@comp-core-i7-2640m-0182e6> <87tv3vkg1a.fsf@x220.int.ebiederm.org> <87v9obipk9.fsf@x220.int.ebiederm.org> <20200212200335.GO23230@ZenIV.linux.org.uk> <20200212203833.GQ23230@ZenIV.linux.org.uk> <20200212204124.GR23230@ZenIV.linux.org.uk> <87lfp7h422.fsf@x220.int.ebiederm.org> <87pnejf6fz.fsf@x220.int.ebiederm.org> <871rqpaswu.fsf_-_@x220.int.ebiederm.org> Date: Mon, 24 Feb 2020 10:25:16 -0600 In-Reply-To: <871rqpaswu.fsf_-_@x220.int.ebiederm.org> (Eric W. Biederman's message of "Thu, 20 Feb 2020 14:46:25 -0600") Message-ID: <871rqk2brn.fsf_-_@x220.int.ebiederm.org> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/26.1 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain X-XM-SPF: eid=1j6GZk-0007ND-2p;;;mid=<871rqk2brn.fsf_-_@x220.int.ebiederm.org>;;;hst=in01.mta.xmission.com;;;ip=68.227.160.95;;;frm=ebiederm@xmission.com;;;spf=neutral X-XM-AID: U2FsdGVkX18r19JNQj7E6kX0G2mdhpQOxD8W+HrcR10= X-SA-Exim-Connect-IP: 68.227.160.95 X-SA-Exim-Mail-From: ebiederm@xmission.com X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on sa03.xmission.com X-Spam-Level: X-Spam-Status: No, score=-1.0 required=8.0 tests=ALL_TRUSTED,BAYES_40, DCC_CHECK_NEGATIVE,T_TooManySym_01 autolearn=disabled version=3.4.2 X-Spam-Virus: No X-Spam-Report: * -1.0 ALL_TRUSTED Passed through trusted hosts only via SMTP * -0.0 BAYES_40 BODY: Bayes spam probability is 20 to 40% * [score: 0.3245] * -0.0 DCC_CHECK_NEGATIVE Not listed in DCC * [sa03 1397; Body=1 Fuz1=1 Fuz2=1] * 0.0 T_TooManySym_01 4+ unique symbols in subject X-Spam-DCC: XMission; sa03 1397; Body=1 Fuz1=1 Fuz2=1 X-Spam-Combo: ;Linus Torvalds X-Spam-Relay-Country: X-Spam-Timing: total 312 ms - load_scoreonly_sql: 0.04 (0.0%), signal_user_changed: 2.4 (0.8%), b_tie_ro: 1.64 (0.5%), parse: 1.06 (0.3%), extract_message_metadata: 3.2 (1.0%), get_uri_detail_list: 1.25 (0.4%), tests_pri_-1000: 6 (1.9%), tests_pri_-950: 1.43 (0.5%), tests_pri_-900: 1.15 (0.4%), tests_pri_-90: 29 (9.1%), check_bayes: 27 (8.6%), b_tokenize: 10 (3.3%), b_tok_get_all: 8 (2.6%), b_comp_prob: 2.8 (0.9%), b_tok_touch_all: 3.3 (1.0%), b_finish: 0.60 (0.2%), tests_pri_0: 251 (80.4%), check_dkim_signature: 0.59 (0.2%), check_dkim_adsp: 2.7 (0.8%), poll_dns_idle: 0.91 (0.3%), tests_pri_10: 2.5 (0.8%), tests_pri_500: 6 (1.9%), rewrite_mail: 0.00 (0.0%) Subject: [PATCH v2 0/6] proc: Dentry flushing without proc_mnt X-Spam-Flag: No X-SA-Exim-Version: 4.2.1 (built Thu, 05 May 2016 13:38:54 -0600) X-SA-Exim-Scanned: Yes (on in01.mta.xmission.com) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org I have addressed all of the review comments as I understand them, and fixed the small oversight the kernel test robot was able to find. (I had failed to initialize the new field pid->inodes). I did not hear any concerns from the 10,000 foot level last time so I am assuming this set of changes (baring bugs) is good to go. Unless some new issues appear my plan is to put this in my tree and get this into linux-next. Which will give Alexey something to build his changes on. I tested this set of changes by running: (while ls -1 -f /proc > /dev/null ; do :; done ) & And monitoring the amount of free memory. With the flushing disabled I saw the used memory in the system grow by 20M before the shrinker would bring it back down to where it started. With the patch applied I saw the memory usage stay essentially fixed. So flushing definitely keeps things working better. If anyone sees any problems with this code please let me know. Thank you, Eric W. Biederman (6): proc: Rename in proc_inode rename sysctl_inodes sibling_inodes proc: Generalize proc_sys_prune_dcache into proc_prune_siblings_dcache proc: In proc_prune_siblings_dcache cache an aquired super block proc: Use d_invalidate in proc_prune_siblings_dcache proc: Clear the pieces of proc_inode that proc_evict_inode cares about proc: Use a list of inodes to flush from proc fs/proc/base.c | 111 ++++++++++++++++-------------------------------- fs/proc/inode.c | 73 ++++++++++++++++++++++++++++--- fs/proc/internal.h | 4 +- fs/proc/proc_sysctl.c | 45 +++----------------- include/linux/pid.h | 1 + include/linux/proc_fs.h | 4 +- kernel/exit.c | 4 +- kernel/pid.c | 1 + 8 files changed, 120 insertions(+), 123 deletions(-)