Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S966765AbbBDRCJ (ORCPT ); Wed, 4 Feb 2015 12:02:09 -0500 Received: from cantor2.suse.de ([195.135.220.15]:59113 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755526AbbBDRCH (ORCPT ); Wed, 4 Feb 2015 12:02:07 -0500 Message-ID: <54D2508A.9030804@suse.cz> Date: Wed, 04 Feb 2015 18:02:02 +0100 From: Vlastimil Babka User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.4.0 MIME-Version: 1.0 To: mtk.manpages@gmail.com CC: "Kirill A. Shutemov" , Dave Hansen , Mel Gorman , "linux-mm@kvack.org" , Minchan Kim , Andrew Morton , lkml , Linux API , linux-man , Hugh Dickins Subject: Re: MADV_DONTNEED semantics? Was: [RFC PATCH] mm: madvise: Ignore repeated MADV_DONTNEED hints References: <20150202165525.GM2395@suse.de> <54CFF8AC.6010102@intel.com> <54D08483.40209@suse.cz> <20150203105301.GC14259@node.dhcp.inet.fi> <54D0B43D.8000209@suse.cz> <54D0F56A.9050003@gmail.com> <54D22298.3040504@suse.cz> In-Reply-To: Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2386 Lines: 67 On 02/04/2015 03:00 PM, Michael Kerrisk (man-pages) wrote: > Hello Vlastimil, > > On 4 February 2015 at 14:46, Vlastimil Babka wrote: >>>> - that covers mlocking ok, not sure if the rest fits the "shared pages" >>>> case >>>> though. I dont see any check for other kinds of shared pages in the code. >>> >>> Agreed. "shared" here seems confused. I've removed it. And I've >>> added mention of "Huge TLB pages" for this error. >> >> Thanks. > > I also added those cases for MADV_REMOVE, BTW. Right. There's also the following for MADV_REMOVE that needs updating: "Currently, only shmfs/tmpfs supports this; other filesystems return with the error ENOSYS." - it's not just shmem/tmpfs anymore. It should be best to refer to fallocate(2) option FALLOC_FL_PUNCH_HOLE which seems to be (more) up to date. - AFAICS it doesn't return ENOSYS but EOPNOTSUPP. Also neither error code is listed in the ERRORS section. >>>>>> - The word "will result" did sound as a guarantee at least to me. So >>>>>> here it >>>>>> could be changed to "may result (unless the advice is ignored)"? >>>>> >>>>> It's too late to fix documentation. Applications already depends on the >>>>> beheviour. >>>> >>>> Right, so as long as they check for EINVAL, it should be safe. It appears >>>> that >>>> jemalloc does. >>> >>> >>> So, first a brief question: in the cases where the call does not error >>> out, >>> are we agreed that in the current implementation, MADV_DONTNEED will >>> always result in zero-filled pages when the region is faulted back in >>> (when we consider pages that are not backed by a file)? >> >> >> I'd agree at this point. > > Thanks for the confirmation. > >> Also we should probably mention anonymously shared pages (shmem). I think >> they behave the same as file here. > > You mean tmpfs here, right? (I don't keep all of the synonyms straight.) shmem is tmpfs (that by itself would fit under "files" just fine), but also sys V segments created by shmget(2) and also mappings created by mmap with MAP_SHARED | MAP_ANONYMOUS. I'm not sure if there's a single manpage to refer to the full list. Thanks, Vlastimil -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/