From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752796AbdKJKpV (ORCPT ); Fri, 10 Nov 2017 05:45:21 -0500 Received: from mail-qk0-f196.google.com ([209.85.220.196]:47594 "EHLO mail-qk0-f196.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751404AbdKJKpS (ORCPT ); Fri, 10 Nov 2017 05:45:18 -0500 X-Google-Smtp-Source: AGs4zMYp4YdS3Pd74qm0oQ6wXC2COpD3ovn9L769HKhlvsxk8fdjfnREMMvcNBkRRa0xCM0IpaeQQur4PyFBxTaftK4= MIME-Version: 1.0 In-Reply-To: References: <1510244046-3256-1-git-send-email-tixxdz@gmail.com> <1510244046-3256-3-git-send-email-tixxdz@gmail.com> From: Djalal Harouni Date: Fri, 10 Nov 2017 11:45:17 +0100 Message-ID: Subject: Re: [PATCH RFC v3 2/7] proc: move /proc/{self|thread-self} dentries to proc_fs_info To: Alexey Dobriyan Cc: Kees Cook , Alexey Gladkov , Andy Lutomirski , Andrew Morton , Linux FS Devel , linux-kernel , kernel-hardening@lists.openwall.com, LSM List , Linux API , Greg Kroah-Hartman , Alexander Viro , Akinobu Mita , "Tobin C. Harding" , Oleg Nesterov , Jeff Layton , Ingo Molnar , "Eric W. Biederman" , Linus Torvalds , Daniel Micay , Jonathan Corbet , "J. Bruce Fields" , Stephen Rothwell , Solar Designer Content-Type: text/plain; charset="UTF-8" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, Nov 10, 2017 at 11:31 AM, Alexey Dobriyan wrote: > On 11/9/17, Djalal Harouni wrote: > >> struct proc_fs_info { >> struct pid_namespace *pid_ns; >> + struct dentry *proc_self; /* For /proc/self/ */ >> + struct dentry *proc_thread_self; /* For /proc/thread-self/ */ > > These are redundant comments. I can remove them, but actually the area is so difficult and uncommented that I won't mind extra comments... Thanks! -- tixxdz From mboxrd@z Thu Jan 1 00:00:00 1970 From: Djalal Harouni Subject: Re: [PATCH RFC v3 2/7] proc: move /proc/{self|thread-self} dentries to proc_fs_info Date: Fri, 10 Nov 2017 11:45:17 +0100 Message-ID: References: <1510244046-3256-1-git-send-email-tixxdz@gmail.com> <1510244046-3256-3-git-send-email-tixxdz@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Return-path: In-Reply-To: Sender: owner-linux-security-module@vger.kernel.org To: Alexey Dobriyan Cc: Kees Cook , Alexey Gladkov , Andy Lutomirski , Andrew Morton , Linux FS Devel , linux-kernel , kernel-hardening@lists.openwall.com, LSM List , Linux API , Greg Kroah-Hartman , Alexander Viro , Akinobu Mita , "Tobin C. Harding" , Oleg Nesterov , Jeff Layton , Ingo Molnar , "Eric W. Biederman" , Linus Torvalds , Daniel Micay List-Id: linux-api@vger.kernel.org On Fri, Nov 10, 2017 at 11:31 AM, Alexey Dobriyan wrote: > On 11/9/17, Djalal Harouni wrote: > >> struct proc_fs_info { >> struct pid_namespace *pid_ns; >> + struct dentry *proc_self; /* For /proc/self/ */ >> + struct dentry *proc_thread_self; /* For /proc/thread-self/ */ > > These are redundant comments. I can remove them, but actually the area is so difficult and uncommented that I won't mind extra comments... Thanks! -- tixxdz From mboxrd@z Thu Jan 1 00:00:00 1970 From: tixxdz@gmail.com (Djalal Harouni) Date: Fri, 10 Nov 2017 11:45:17 +0100 Subject: [PATCH RFC v3 2/7] proc: move /proc/{self|thread-self} dentries to proc_fs_info In-Reply-To: References: <1510244046-3256-1-git-send-email-tixxdz@gmail.com> <1510244046-3256-3-git-send-email-tixxdz@gmail.com> Message-ID: To: linux-security-module@vger.kernel.org List-Id: linux-security-module.vger.kernel.org On Fri, Nov 10, 2017 at 11:31 AM, Alexey Dobriyan wrote: > On 11/9/17, Djalal Harouni wrote: > >> struct proc_fs_info { >> struct pid_namespace *pid_ns; >> + struct dentry *proc_self; /* For /proc/self/ */ >> + struct dentry *proc_thread_self; /* For /proc/thread-self/ */ > > These are redundant comments. I can remove them, but actually the area is so difficult and uncommented that I won't mind extra comments... Thanks! -- tixxdz -- To unsubscribe from this list: send the line "unsubscribe linux-security-module" in the body of a message to majordomo at vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html From mboxrd@z Thu Jan 1 00:00:00 1970 MIME-Version: 1.0 In-Reply-To: References: <1510244046-3256-1-git-send-email-tixxdz@gmail.com> <1510244046-3256-3-git-send-email-tixxdz@gmail.com> From: Djalal Harouni Date: Fri, 10 Nov 2017 11:45:17 +0100 Message-ID: Content-Type: text/plain; charset="UTF-8" Subject: [kernel-hardening] Re: [PATCH RFC v3 2/7] proc: move /proc/{self|thread-self} dentries to proc_fs_info To: Alexey Dobriyan Cc: Kees Cook , Alexey Gladkov , Andy Lutomirski , Andrew Morton , Linux FS Devel , linux-kernel , kernel-hardening@lists.openwall.com, LSM List , Linux API , Greg Kroah-Hartman , Alexander Viro , Akinobu Mita , "Tobin C. Harding" , Oleg Nesterov , Jeff Layton , Ingo Molnar , "Eric W. Biederman" , Linus Torvalds , Daniel Micay , Jonathan Corbet , "J. Bruce Fields" , Stephen Rothwell , Solar Designer List-ID: On Fri, Nov 10, 2017 at 11:31 AM, Alexey Dobriyan wrote: > On 11/9/17, Djalal Harouni wrote: > >> struct proc_fs_info { >> struct pid_namespace *pid_ns; >> + struct dentry *proc_self; /* For /proc/self/ */ >> + struct dentry *proc_thread_self; /* For /proc/thread-self/ */ > > These are redundant comments. I can remove them, but actually the area is so difficult and uncommented that I won't mind extra comments... Thanks! -- tixxdz