Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1739234

Re: [patch v2] mremap.2: Add description of old_size == 0 functionality

Path csiph.com!eternal-september.org!feeder.eternal-september.org!news.unit0.net!news.panservice.it!bofh.it!news.nic.it!robomod
From "Michael Kerrisk (man-pages)" <mtk.manpages@gmail.com>
Newsgroups linux.kernel
Subject Re: [patch v2] mremap.2: Add description of old_size == 0 functionality
Date Mon, 25 Sep 2017 21:30:01 +0200
Message-ID <utHeN-2eS-13@gated-at.bofh.it> (permalink)
References <urpFo-5m4-17@gated-at.bofh.it> <uryz0-2kz-7@gated-at.bofh.it> <urHCi-8iH-25@gated-at.bofh.it> <utEAh-q7-7@gated-at.bofh.it>
X-Original-To Mike Kravetz <mike.kravetz@oracle.com>
Dkim-Signature v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=mime-version:reply-to:in-reply-to:references:from:date:message-id :subject:to:cc; bh=fI7gyRBSzeqSGX4P4KwiomT1maORzogrfHTXVvnBIWg=; b=BvrSZNjaY9mU7Wmwy7kNB8WKoRVTejCr++eIFtqwF+04igJBqY1PzDm0UcwOVE+iiT gdUMh0CaP/uzN/fij3FLDhJj/Yrpg/xpA7UuhUPpgDIPDAFrWjML28KyMmDtpvj1/Fe0 KAujpg86uMLI2L3ZNQOUuDDayaUMKjoGVRVYVuuLRy6t2VJlNKHX1dPcJvxYhgc9o6Y+ SGj8uj/0z9a+FbOq6EKWJgTGtRrQY/OUzpzi2MGvHkJ3+mNSm5MVx3VVWliDkRCttLNz mFQmxoNfjZ/304Z37tOf8FBq3u1VnSP6jbv8XDaJw9+/XcOsEJ/QepE7f4Qq0cXZ/X/A FM/g==
X-Google-Dkim-Signature v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:reply-to:in-reply-to:references :from:date:message-id:subject:to:cc; bh=fI7gyRBSzeqSGX4P4KwiomT1maORzogrfHTXVvnBIWg=; b=kNh2a0t8knBaM4VmeR3mv68CB2o9UBjjBQxBe4Xt/XSTnD84vBE0n7R1pAg363rbJq 5t2aCzjpSVXtrFSadgfEQdZ+/ZJEkS0WrZRQwh8xeIuEVHf7t0rhTrMUTjma7a+IK356 yLqvvYdv9XFg2GQTSCRU/oh/wAvnGa1E8Qj8+1EHN+qgrSBvDqtAXTcHoz2d4aNgckNU uDXzGIRQA96ksOUprQmtEuXW6Pq1t77kxw1FJMKrphD6Yp463WaQ1wG+dVhMpaog6xYR 9c7mbWdqb+lg460i1+MNDSP2Dz7B/z6QL2ajAVjWZxNY5kqgrMPtETKmMoo1ObPf+eSr 0aOw==
X-Gm-Message-State AHPjjUhkGtPqXFfOparglSxG6fTi2161KIJExQIImkVQsiYAuihrsTHl 6e4grD3eidPzZCnqnDL+RTAwodwJtJma0aTuspQ=
X-Google-SMTP-Source AOwi7QCT3+xrb7z082hxa79iDhCgqla+3+E/u+VCM/ZEendI0F9ZE05CaeAuOIm3czf8MGr5uKoN+ipmR+6LUPiJPWw=
X-Received by 10.223.160.68 with SMTP id l4mr6701393wrl.79.1506367622632; Mon, 25 Sep 2017 12:27:02 -0700 (PDT)
MIME-Version 1.0
Reply-To mtk.manpages@gmail.com
Content-Type text/plain; charset="UTF-8"
Sender robomod@news.nic.it
List-ID <linux-kernel.vger.kernel.org>
X-Mailing-List linux-kernel@vger.kernel.org
Approved robomod@news.nic.it
Lines 36
Organization linux.* mail to news gateway
X-Original-Cc linux-man <linux-man@vger.kernel.org>, "linux-mm@kvack.org" <linux-mm@kvack.org>, lkml <linux-kernel@vger.kernel.org>, Linux API <linux-api@vger.kernel.org>, Jann Horn <jannh@google.com>, Florian Weimer <fweimer@redhat.com>, Michal Hocko <mhocko@suse.com>, Andrea Arcangeli <aarcange@redhat.com>, "Kirill A . Shutemov" <kirill.shutemov@linux.intel.com>, Vlastimil Babka <vbabka@suse.cz>, Anshuman Khandual <khandual@linux.vnet.ibm.com>
X-Original-Date Mon, 25 Sep 2017 21:26:42 +0200
X-Original-Message-ID <CAKgNAkhJEyK=LhFj9W-RgSv+ET64d+MaEAQ41y5eximfQmYPDw@mail.gmail.com>
X-Original-References <a5d279cb-a015-f74c-2e40-a231aa7f7a8c@redhat.com> <20170919214224.19561-1-mike.kravetz@oracle.com> <6fafdae8-4fea-c967-f5cd-d22c205608fa@gmail.com> <83b023da-e9f5-2957-981e-5b0e71e9bf1b@oracle.com>
X-Original-Sender linux-kernel-owner@vger.kernel.org
Xref csiph.com linux.kernel:1739234

Show key headers only | View raw


Hi Mike,

On 25 September 2017 at 18:33, Mike Kravetz <mike.kravetz@oracle.com> wrote:
> On 09/20/2017 12:25 AM, Michael Kerrisk (man-pages) wrote:

[...]

>> I've applied this, and added Reviewed-by tags for Florian and Jann.
>> But, I think it's also worth noting the older, now disallowed, behavior,
>> and why the behavior was changed. So I added a note in BUGS:
>>
>>     BUGS
>>        Before Linux 4.14, if old_size was zero and the  mapping  referred
>>        to  by  old_address  was  a private mapping (mmap(2) MAP_PRIVATE),
>>        mremap() created a new private mapping unrelated to  the  original
>>        mapping.   This behavior was unintended and probably unexpected in
>>        user-space applications (since the intention  of  mremap()  is  to
>>        create  a new mapping based on the original mapping).  Since Linux
>>        4.14, mremap() fails with the error EINVAL in this scenario.
>>
>> Does that seem okay?
>
> Sorry for the late reply Michael,  I've been away for a few days.
>
> Yes, the above seems okay.  Thanks for your help with this.

You're welcome. Thanks for checking it over!

Cheers,

Michael

-- 
Michael Kerrisk
Linux man-pages maintainer; http://www.kernel.org/doc/man-pages/
Linux/UNIX System Programming Training: http://man7.org/training/

Back to linux.kernel | Previous | NextPrevious in thread | Next in thread | Find similar | Unroll thread


Thread

[patch] mremap.2: Add description of old_size == 0 functionality Mike Kravetz <mike.kravetz@oracle.com> - 2017-09-15 23:40 +0200
  Re: [patch] mremap.2: Add description of old_size == 0 functionality Mike Kravetz <mike.kravetz@oracle.com> - 2017-09-16 00:00 +0200
    Re: [patch] mremap.2: Add description of old_size == 0 functionality Florian Weimer <fweimer@redhat.com> - 2017-09-18 15:50 +0200
      Re: [patch] mremap.2: Add description of old_size == 0 functionality Mike Kravetz <mike.kravetz@oracle.com> - 2017-09-18 19:20 +0200
        Re: [patch] mremap.2: Add description of old_size == 0 functionality Florian Weimer <fweimer@redhat.com> - 2017-09-19 14:20 +0200
          [patch v2] mremap.2: Add description of old_size == 0 functionality Mike Kravetz <mike.kravetz@oracle.com> - 2017-09-19 23:50 +0200
            Re: [patch v2] mremap.2: Add description of old_size == 0  functionality "Michael Kerrisk (man-pages)" <mtk.manpages@gmail.com> - 2017-09-20 09:30 +0200
              Re: [patch v2] mremap.2: Add description of old_size == 0  functionality Michal Hocko <mhocko@kernel.org> - 2017-09-25 14:40 +0200
                Re: [patch v2] mremap.2: Add description of old_size == 0 functionality "Michael Kerrisk (man-pages)" <mtk.manpages@gmail.com> - 2017-09-25 14:50 +0200
              Re: [patch v2] mremap.2: Add description of old_size == 0  functionality Mike Kravetz <mike.kravetz@oracle.com> - 2017-09-25 18:40 +0200
                Re: [patch v2] mremap.2: Add description of old_size == 0 functionality "Michael Kerrisk (man-pages)" <mtk.manpages@gmail.com> - 2017-09-25 21:30 +0200
          Re: [patch] mremap.2: Add description of old_size == 0 functionality Michal Hocko <mhocko@kernel.org> - 2017-09-25 14:40 +0200
            Re: [patch] mremap.2: Add description of old_size == 0 functionality Florian Weimer <fweimer@redhat.com> - 2017-09-25 14:50 +0200
              Re: [patch] mremap.2: Add description of old_size == 0 functionality Michal Hocko <mhocko@kernel.org> - 2017-09-25 15:00 +0200
                Re: [patch] mremap.2: Add description of old_size == 0 functionality Florian Weimer <fweimer@redhat.com> - 2017-09-25 15:20 +0200
                Re: [patch] mremap.2: Add description of old_size == 0 functionality Michal Hocko <mhocko@kernel.org> - 2017-09-25 17:00 +0200
                Re: [patch] mremap.2: Add description of old_size == 0 functionality Florian Weimer <fweimer@redhat.com> - 2017-09-25 17:00 +0200
  Re: [patch] mremap.2: Add description of old_size == 0 functionality Jann Horn <jannh@google.com> - 2017-09-18 04:00 +0200
    Re: [patch] mremap.2: Add description of old_size == 0 functionality Mike Kravetz <mike.kravetz@oracle.com> - 2017-09-18 19:30 +0200

csiph-web