lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Fri, 15 Dec 2017 20:02:04 +1100 From: Michael Ellerman <mpe@...erman.id.au> To: Kees Cook <keescook@...omium.org>, Michal Hocko <mhocko@...nel.org> Cc: Linux API <linux-api@...r.kernel.org>, Khalid Aziz <khalid.aziz@...cle.com>, Andrew Morton <akpm@...ux-foundation.org>, Russell King - ARM Linux <linux@...linux.org.uk>, Andrea Arcangeli <aarcange@...hat.com>, Linux-MM <linux-mm@...ck.org>, LKML <linux-kernel@...r.kernel.org>, linux-arch <linux-arch@...r.kernel.org>, Florian Weimer <fweimer@...hat.com>, John Hubbard <jhubbard@...dia.com>, Matthew Wilcox <willy@...radead.org>, Abdul Haleem <abdhalee@...ux.vnet.ibm.com>, Joel Stanley <joel@....id.au>, Michal Hocko <mhocko@...e.com> Subject: Re: [PATCH v2 0/2] mm: introduce MAP_FIXED_SAFE Kees Cook <keescook@...omium.org> writes: > On Wed, Dec 13, 2017 at 1:25 AM, Michal Hocko <mhocko@...nel.org> wrote: >> >> Hi, >> I am resending with some minor updates based on Michael's review and >> ask for inclusion. There haven't been any fundamental objections for >> the RFC [1] nor the previous version [2]. The biggest discussion >> revolved around the naming. There were many suggestions flowing >> around MAP_REQUIRED, MAP_EXACT, MAP_FIXED_NOCLOBBER, MAP_AT_ADDR, >> MAP_FIXED_NOREPLACE etc... > > With this named MAP_FIXED_NOREPLACE (the best consensus we've got on a > name), please consider this series: > > Acked-by: Kees Cook <keescook@...omium.org> I don't feel like I'm actually qualified to ack the mm and binfmt changes, but everything *looks* correct to me, and you've fixed the flag numbering such that it can go in mman-common.h as I suggested. So if the name was MAP_FIXED_NOREPLACE I would also be happy with it. Acked-by: Michael Ellerman <mpe@...erman.id.au> I can resubmit with the name changed if that's what it takes. cheers
Powered by blists - more mailing lists