Re: [PATCH v2 1/5] exec: Only compute current once in flush_old_exec

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

To: "Eric W. Biederman" <ebiederm@xxxxxxxxxxxx>
Subject: Re: [PATCH v2 1/5] exec: Only compute current once in flush_old_exec
From: Christian Brauner <christian.brauner@xxxxxxxxxx>
Date: Tue, 10 Mar 2020 22:12:40 +0100
Cc: Bernd Edlinger <bernd.edlinger@xxxxxxxxxx>, Kees Cook <keescook@xxxxxxxxxxxx>, Jann Horn <jannh@xxxxxxxxxx>, Jonathan Corbet <corbet@xxxxxxx>, Alexander Viro <viro@xxxxxxxxxxxxxxxxxx>, Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx>, Alexey Dobriyan <adobriyan@xxxxxxxxx>, Thomas Gleixner <tglx@xxxxxxxxxxxxx>, Oleg Nesterov <oleg@xxxxxxxxxx>, Frederic Weisbecker <frederic@xxxxxxxxxx>, Andrei Vagin <avagin@xxxxxxxxx>, Ingo Molnar <mingo@xxxxxxxxxx>, "Peter Zijlstra (Intel)" <peterz@xxxxxxxxxxxxx>, Yuyang Du <duyuyang@xxxxxxxxx>, David Hildenbrand <david@xxxxxxxxxx>, Sebastian Andrzej Siewior <bigeasy@xxxxxxxxxxxxx>, Anshuman Khandual <anshuman.khandual@xxxxxxx>, David Howells <dhowells@xxxxxxxxxx>, James Morris <jamorris@xxxxxxxxxxxxxxxxxxx>, Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx>, Shakeel Butt <shakeelb@xxxxxxxxxx>, Jason Gunthorpe <jgg@xxxxxxxx>, Christian Kellner <christian@xxxxxxxxxx>, Andrea Arcangeli <aarcange@xxxxxxxxxx>, Aleksa Sarai <cyphar@xxxxxxxxxx>, "Dmitry V. Levin" <ldv@xxxxxxxxxxxx>, "linux-doc@xxxxxxxxxxxxxxx" <linux-doc@xxxxxxxxxxxxxxx>, "linux-kernel@xxxxxxxxxxxxxxx" <linux-kernel@xxxxxxxxxxxxxxx>, "linux-fsdevel@xxxxxxxxxxxxxxx" <linux-fsdevel@xxxxxxxxxxxxxxx>, "linux-mm@xxxxxxxxx" <linux-mm@xxxxxxxxx>, "stable@xxxxxxxxxxxxxxx" <stable@xxxxxxxxxxxxxxx>, "linux-api@xxxxxxxxxxxxxxx" <linux-api@xxxxxxxxxxxxxxx>
In-reply-to: <87pndm5y3l.fsf_-_@x220.int.ebiederm.org>
References: <87v9nlii0b.fsf@x220.int.ebiederm.org> <AM6PR03MB5170609D44967E044FD1BE40E4E40@AM6PR03MB5170.eurprd03.prod.outlook.com> <87a74xi4kz.fsf@x220.int.ebiederm.org> <AM6PR03MB51705AA3009B4986BB6EF92FE4E50@AM6PR03MB5170.eurprd03.prod.outlook.com> <87r1y8dqqz.fsf@x220.int.ebiederm.org> <AM6PR03MB517053AED7DC89F7C0704B7DE4E50@AM6PR03MB5170.eurprd03.prod.outlook.com> <AM6PR03MB51703B44170EAB4626C9B2CAE4E20@AM6PR03MB5170.eurprd03.prod.outlook.com> <87tv32cxmf.fsf_-_@x220.int.ebiederm.org> <87v9ne5y4y.fsf_-_@x220.int.ebiederm.org> <87pndm5y3l.fsf_-_@x220.int.ebiederm.org>

On Sun, Mar 08, 2020 at 04:35:26PM -0500, Eric W. Biederman wrote:
> 
> Make it clear that current only needs to be computed once in
> flush_old_exec.  This may have some efficiency improvements and it
> makes the code easier to change.
> 
> Signed-off-by: "Eric W. Biederman" <ebiederm@xxxxxxxxxxxx>

Acked-by: Christian Brauner <christian.brauner@xxxxxxxxxx>

References:
- Re: [PATCHv5] exec: Fix a deadlock in ptrace
  - From: Eric W. Biederman
- Re: [PATCHv5] exec: Fix a deadlock in ptrace
  - From: Bernd Edlinger
- Re: [PATCHv5] exec: Fix a deadlock in ptrace
  - From: Eric W. Biederman
- Re: [PATCHv5] exec: Fix a deadlock in ptrace
  - From: Bernd Edlinger
- Re: [PATCHv5] exec: Fix a deadlock in ptrace
  - From: Eric W. Biederman
- [PATCHv6] exec: Fix a deadlock in ptrace
  - From: Bernd Edlinger
- Re: [PATCHv6] exec: Fix a deadlock in ptrace
  - From: Bernd Edlinger
- [PATCH 0/2] Infrastructure to allow fixing exec deadlocks
  - From: Eric W. Biederman
- [PATCH 0/5] Infrastructure to allow fixing exec deadlocks
  - From: Eric W. Biederman
- [PATCH v2 1/5] exec: Only compute current once in flush_old_exec
  - From: Eric W. Biederman

Prev by Date: Re: [PATCH v2 4/5] exec: Move exec_mmap right after de_thread in flush_old_exec
Next by Date: Re: [PATCH v2 5/5] exec: Add a exec_update_mutex to replace cred_guard_mutex
Previous by thread: Re: [PATCH v2 1/5] exec: Only compute current once in flush_old_exec
Next by thread: [PATCH v2 2/5] exec: Factor unshare_sighand out of de_thread and call it separately
Index(es):
- Date
- Thread

[Index of Archives] [Kernel Newbies] [Security] [Netfilter] [Bugtraq] [Linux FS] [Yosemite Forum] [MIPS Linux] [ARM Linux] [Linux Security] [Linux RAID] [Samba] [Video 4 Linux] [Device Mapper] [Linux Resources]