Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1739234
| 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 | Next — Previous in thread | Next in thread | Find similar | Unroll 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