public inbox for cygwin-cvs@sourceware.org
help / color / mirror / Atom feed
From: Corinna Vinschen <corinna@sourceware.org>
To: cygwin-cvs@sourceware.org
Subject: [newlib-cygwin] Cygwin: drop wincap::mmap_storage_high
Date: Sun,  4 Dec 2022 13:27:52 +0000 (GMT)	[thread overview]
Message-ID: <20221204132752.84E76386102D@sourceware.org> (raw)

https://sourceware.org/git/gitweb.cgi?p=newlib-cygwin.git;h=ae2829a2269010230506e08a550b80e43d2083cc

commit ae2829a2269010230506e08a550b80e43d2083cc
Author:     Corinna Vinschen <corinna@vinschen.de>
AuthorDate: Tue Nov 15 17:46:52 2022 +0100
Commit:     Corinna Vinschen <corinna@vinschen.de>
CommitDate: Sun Dec 4 14:01:41 2022 +0100

    Cygwin: drop wincap::mmap_storage_high
    
    Only required for Windows 7 and Windows 8.
    
    Signed-off-by: Corinna Vinschen <corinna@vinschen.de>

Diff:
---
 winsup/cygwin/local_includes/memory_layout.h |  9 ++-------
 winsup/cygwin/local_includes/wincap.h        |  2 --
 winsup/cygwin/wincap.cc                      | 12 ------------
 3 files changed, 2 insertions(+), 21 deletions(-)

diff --git a/winsup/cygwin/local_includes/memory_layout.h b/winsup/cygwin/local_includes/memory_layout.h
index 83ccf8f65817..6a4b7a28a11c 100644
--- a/winsup/cygwin/local_includes/memory_layout.h
+++ b/winsup/cygwin/local_includes/memory_layout.h
@@ -53,11 +53,6 @@ details. */
 #define USERHEAP_START			0xa00000000UL
 
 /* The memory region used for memory maps.  Mmaps grow downwards.
-   Set the lowest address to leave ~32 Gigs for heap.
-
-   Up to Win 8 we only have a 44 bit address space, 48 bit address space
-   starting with 8.1, so the max value is a system-specific variable. */
+   Set the lowest address to leave ~32 Gigs for heap. */
 #define MMAP_STORAGE_LOW		0x001000000000UL
-#define __MMAP_STORAGE_HIGH_LEGACY	0x070000000000UL
-#define __MMAP_STORAGE_HIGH		0x700000000000UL
-#define MMAP_STORAGE_HIGH		wincap.mmap_storage_high ()
+#define MMAP_STORAGE_HIGH		0x700000000000UL
diff --git a/winsup/cygwin/local_includes/wincap.h b/winsup/cygwin/local_includes/wincap.h
index 96bd68a93f95..d5579e7ea71a 100644
--- a/winsup/cygwin/local_includes/wincap.h
+++ b/winsup/cygwin/local_includes/wincap.h
@@ -12,7 +12,6 @@ details. */
 struct wincaps
 {
   DWORD def_guard_pages;
-  int64_t mmap_storage_high;
   /* The bitfields must be 8 byte aligned on x86_64, otherwise the bitfield
      ops generated by gcc are off by 4 bytes. */
   struct  __attribute__ ((aligned (8))) {
@@ -67,7 +66,6 @@ public:
   {
     return ((wincaps *) this->caps)->def_guard_pages * page_size ();
   }
-  intptr_t IMPLEMENT (mmap_storage_high)
   bool  IMPLEMENT (is_server)
   bool	IMPLEMENT (has_new_pebteb_region)
   bool	IMPLEMENT (has_unprivileged_createsymlink)
diff --git a/winsup/cygwin/wincap.cc b/winsup/cygwin/wincap.cc
index f83eed8a7e1a..7aefde48cb49 100644
--- a/winsup/cygwin/wincap.cc
+++ b/winsup/cygwin/wincap.cc
@@ -21,7 +21,6 @@ details. */
 
 wincaps wincap_7 __attribute__((section (".cygwin_dll_common"), shared)) = {
   def_guard_pages:2,
-  mmap_storage_high:__MMAP_STORAGE_HIGH_LEGACY,
   {
     is_server:false,
     has_new_pebteb_region:false,
@@ -45,7 +44,6 @@ wincaps wincap_7 __attribute__((section (".cygwin_dll_common"), shared)) = {
 
 wincaps wincap_8 __attribute__((section (".cygwin_dll_common"), shared)) = {
   def_guard_pages:3,
-  mmap_storage_high:__MMAP_STORAGE_HIGH_LEGACY,
   {
     is_server:false,
     has_new_pebteb_region:false,
@@ -69,7 +67,6 @@ wincaps wincap_8 __attribute__((section (".cygwin_dll_common"), shared)) = {
 
 wincaps wincap_8_1 __attribute__((section (".cygwin_dll_common"), shared)) = {
   def_guard_pages:3,
-  mmap_storage_high:__MMAP_STORAGE_HIGH,
   {
     is_server:false,
     has_new_pebteb_region:false,
@@ -93,7 +90,6 @@ wincaps wincap_8_1 __attribute__((section (".cygwin_dll_common"), shared)) = {
 
 wincaps  wincap_10_1507 __attribute__((section (".cygwin_dll_common"), shared)) = {
   def_guard_pages:3,
-  mmap_storage_high:__MMAP_STORAGE_HIGH,
   {
     is_server:false,
     has_new_pebteb_region:false,
@@ -117,7 +113,6 @@ wincaps  wincap_10_1507 __attribute__((section (".cygwin_dll_common"), shared))
 
 wincaps  wincap_10_1607 __attribute__((section (".cygwin_dll_common"), shared)) = {
   def_guard_pages:3,
-  mmap_storage_high:__MMAP_STORAGE_HIGH,
   {
     is_server:false,
     has_new_pebteb_region:false,
@@ -141,7 +136,6 @@ wincaps  wincap_10_1607 __attribute__((section (".cygwin_dll_common"), shared))
 
 wincaps wincap_10_1703 __attribute__((section (".cygwin_dll_common"), shared)) = {
   def_guard_pages:3,
-  mmap_storage_high:__MMAP_STORAGE_HIGH,
   {
     is_server:false,
     has_new_pebteb_region:true,
@@ -165,7 +159,6 @@ wincaps wincap_10_1703 __attribute__((section (".cygwin_dll_common"), shared)) =
 
 wincaps wincap_10_1709 __attribute__((section (".cygwin_dll_common"), shared)) = {
   def_guard_pages:3,
-  mmap_storage_high:__MMAP_STORAGE_HIGH,
   {
     is_server:false,
     has_new_pebteb_region:true,
@@ -189,7 +182,6 @@ wincaps wincap_10_1709 __attribute__((section (".cygwin_dll_common"), shared)) =
 
 wincaps wincap_10_1803 __attribute__((section (".cygwin_dll_common"), shared)) = {
   def_guard_pages:3,
-  mmap_storage_high:__MMAP_STORAGE_HIGH,
   {
     is_server:false,
     has_new_pebteb_region:true,
@@ -213,7 +205,6 @@ wincaps wincap_10_1803 __attribute__((section (".cygwin_dll_common"), shared)) =
 
 wincaps wincap_10_1809 __attribute__((section (".cygwin_dll_common"), shared)) = {
   def_guard_pages:3,
-  mmap_storage_high:__MMAP_STORAGE_HIGH,
   {
     is_server:false,
     has_new_pebteb_region:true,
@@ -237,7 +228,6 @@ wincaps wincap_10_1809 __attribute__((section (".cygwin_dll_common"), shared)) =
 
 wincaps wincap_10_1903 __attribute__((section (".cygwin_dll_common"), shared)) = {
   def_guard_pages:3,
-  mmap_storage_high:__MMAP_STORAGE_HIGH,
   {
     is_server:false,
     has_new_pebteb_region:true,
@@ -261,7 +251,6 @@ wincaps wincap_10_1903 __attribute__((section (".cygwin_dll_common"), shared)) =
 
 wincaps wincap_10_2004 __attribute__((section (".cygwin_dll_common"), shared)) = {
   def_guard_pages:3,
-  mmap_storage_high:__MMAP_STORAGE_HIGH,
   {
     is_server:false,
     has_new_pebteb_region:true,
@@ -285,7 +274,6 @@ wincaps wincap_10_2004 __attribute__((section (".cygwin_dll_common"), shared)) =
 
 wincaps wincap_11 __attribute__((section (".cygwin_dll_common"), shared)) = {
   def_guard_pages:3,
-  mmap_storage_high:__MMAP_STORAGE_HIGH,
   {
     is_server:false,
     has_new_pebteb_region:true,

                 reply	other threads:[~2022-12-04 13:27 UTC|newest]

Thread overview: [no followups] expand[flat|nested]  mbox.gz  Atom feed

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20221204132752.84E76386102D@sourceware.org \
    --to=corinna@sourceware.org \
    --cc=cygwin-cvs@sourceware.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).