Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752339AbdLEHmE (ORCPT ); Tue, 5 Dec 2017 02:42:04 -0500 Received: from hqemgate16.nvidia.com ([216.228.121.65]:18197 "EHLO hqemgate16.nvidia.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750740AbdLEHmB (ORCPT ); Tue, 5 Dec 2017 02:42:01 -0500 X-PGP-Universal: processed; by hqpgpgate102.nvidia.com on Mon, 04 Dec 2017 23:42:44 -0800 Subject: Re: [PATCH v2] mmap.2: MAP_FIXED updated documentation To: Michal Hocko CC: Cyril Hrubis , Michael Kerrisk , linux-man , , Michael Ellerman , , LKML , , Jann Horn , Matthew Wilcox References: <20171204021411.4786-1-jhubbard@nvidia.com> <20171204105549.GA31332@rei> <20171205070510.aojohhvixijk3i27@dhcp22.suse.cz> X-Nvconfidentiality: public From: John Hubbard Message-ID: <2cff594a-b481-269d-dd91-ff2cc2f4100a@nvidia.com> Date: Mon, 4 Dec 2017 23:42:00 -0800 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.5.0 MIME-Version: 1.0 In-Reply-To: <20171205070510.aojohhvixijk3i27@dhcp22.suse.cz> X-Originating-IP: [10.2.167.64] X-ClientProxiedBy: HQMAIL102.nvidia.com (172.18.146.10) To HQMAIL107.nvidia.com (172.20.187.13) Content-Type: text/plain; charset="utf-8" Content-Language: en-US Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2547 Lines: 68 On 12/04/2017 11:05 PM, Michal Hocko wrote: > On Mon 04-12-17 18:14:18, John Hubbard wrote: >> On 12/04/2017 02:55 AM, Cyril Hrubis wrote: >>> Hi! >>> I know that we are not touching the rest of the existing description for >>> MAP_FIXED however the second sentence in the manual page says that "addr >>> must be a multiple of the page size." Which however is misleading as >>> this is not enough on some architectures. Code in the wild seems to >>> (mis)use SHMLBA for aligment purposes but I'm not sure that we should >>> advise something like that in the manpages. >>> >>> So what about something as: >>> >>> "addr must be suitably aligned, for most architectures multiple of page >>> size is sufficient, however some may impose additional restrictions for >>> page mapping addresses." >>> >> >> Hi Cyril, >> >> Right, so I've been looking into this today, and I think we can go a bit >> further than that, even. The kernel, as far back as the *original* git >> commit in 2005, implements mmap on ARM by requiring that the address is >> aligned to SHMLBA: >> >> arch/arm/mm/mmap.c:50: >> >> if (flags & MAP_FIXED) { >> if (aliasing && flags & MAP_SHARED && >> (addr - (pgoff << PAGE_SHIFT)) & (SHMLBA - 1)) >> return -EINVAL; >> return addr; >> } >> >> So, given that this has been the implementation for the last 12+ years (and >> probably the whole time, in fact), I think we can be bold enough to use this >> wording for the second sentence of MAP_FIXED: >> >> "addr must be a multiple of SHMLBA (), which in turn is either >> the system page size (on many architectures) or a multiple of the system >> page size (on some architectures)." >> >> What do you think? > > I am not sure this is a good idea. This is pulling way too many > implementation details into the man page IMHO. Note that your wording is > even incorrect because this applies only to shared mappings and on some > architectures it even requires special memory regions. We do not want > all that in the man page... > Hi Michal, OK, so it sounds like Cyril's original wording would be just about right, after all, like this? "addr must be suitably aligned. For most architectures multiple of page size is sufficient; however, some may impose additional restrictions." (It does seem unfortunate that the man page cannot help the programmer actually write correct code here. He or she is forced to read the kernel implementation, in order to figure out the true alignment rules. I was hoping we could avoid that.) thanks, John Hubbard NVIDIA