Skip to content

Commit

Permalink
Merge r229669 - Improve error message when Gigacage cannot allocate v…
Browse files Browse the repository at this point in the history
…irtual memory

https://bugs.webkit.org/show_bug.cgi?id=183329

Reviewed by Filip Pizlo.

We've discovered that Deja Dup monitor sets a virtual memory limit, breaking Gigacage. Since
it runs in the background on a fresh out-of-the-box install of Ubuntu, this is not good.
That will have to be fixed by Deja Dup, but there is concern that other applications might
try this, or that users will set a virtual memory limit for the entire desktop session. Of
particular concern is the possibility that users might have copypasted a ulimit line into
a session startup script without understanding it. Let's try to make it slightly easier to
understand what's going wrong.

* bmalloc/Gigacage.cpp:
(Gigacage::ensureGigacage):
  • Loading branch information
mcatanzaro authored and carlosgcampos committed Apr 9, 2018
1 parent 73764a7 commit a524b69
Show file tree
Hide file tree
Showing 2 changed files with 19 additions and 0 deletions.
18 changes: 18 additions & 0 deletions Source/bmalloc/ChangeLog
@@ -1,3 +1,21 @@
2018-03-16 Michael Catanzaro <mcatanzaro@igalia.com>

Improve error message when Gigacage cannot allocate virtual memory
https://bugs.webkit.org/show_bug.cgi?id=183329

Reviewed by Filip Pizlo.

We've discovered that Deja Dup monitor sets a virtual memory limit, breaking Gigacage. Since
it runs in the background on a fresh out-of-the-box install of Ubuntu, this is not good.
That will have to be fixed by Deja Dup, but there is concern that other applications might
try this, or that users will set a virtual memory limit for the entire desktop session. Of
particular concern is the possibility that users might have copypasted a ulimit line into
a session startup script without understanding it. Let's try to make it slightly easier to
understand what's going wrong.

* bmalloc/Gigacage.cpp:
(Gigacage::ensureGigacage):

2018-02-05 JF Bastien <jfbastien@apple.com>

Gigacage: enable only for WebContent process and token executables
Expand Down
1 change: 1 addition & 0 deletions Source/bmalloc/bmalloc/Gigacage.cpp
Expand Up @@ -151,6 +151,7 @@ void ensureGigacage()
if (GIGACAGE_ALLOCATION_CAN_FAIL)
return;
fprintf(stderr, "FATAL: Could not allocate gigacage memory with maxAlignment = %lu, totalSize = %lu.\n", maxAlignment, totalSize);
fprintf(stderr, "(Make sure you have not set a virtual memory limit.)\n");
BCRASH();
}

Expand Down

0 comments on commit a524b69

Please sign in to comment.