Changes between Initial Version and Version 8 of Ticket #22633


Ignore:
Timestamp:
03/22/17 08:26:09 (18 months ago)
Author:
jdemeyer
Comment:

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #22633

    • Property Commit changed from to a724b2a0fd86f4894de05f3a0db0b9e078c7d36d
    • Property Report Upstream changed from N/A to Reported upstream. No feedback yet.
    • Property Branch changed from to u/embray/cygwin/pari-mmap
    • Property Authors changed from to Erik Bray
  • Ticket #22633 – Description

    initial v8  
    77This is a big problem in Sage since we set a very large default stack size for Pari.  This was not a problem prior to #21582, since Pari used the MAP_NORESERVE flag which circumvents this issue (only dirty pages need to be copied).  The changes in #21582 make sense for Linux, but for Cygwin the opposite is true.  Different but unfortunate implementation details on both platforms.
    88
    9 The best way forward is to partially revert #21582 for ''Cygwin only''.  That is, keep the current behavior on Linux and other platforms, and restore the old behavior just on Cygwin.
     9The best way forward is to use `MAP_NORESERVE` anyway, which helps Cygwin and won't hurt other platforms.
     10
     11'''Upstream bug''': https://pari.math.u-bordeaux.fr/cgi-bin/bugreport.cgi?bug=1912