Opened 16 months ago
Last modified 12 months ago
#6699 new defect
[macOS] Memory issue with 27412 — at Version 7
Reported by: | Langbart | Owned by: | |
---|---|---|---|
Priority: | Must Have | Milestone: | Backlog |
Component: | Core engine | Keywords: | |
Cc: | Patch: |
Description (last modified by )
to consistently reproduce (1)
- add this debug message in
chatHelper.js
-
binaries/data/mods/public/simulation/ai/petra/chatHelper.js
a b PETRA.chatNewDiplomacy = function(gameState, player, newDiplomaticStance) 220 220 221 221 PETRA.chatAnswerRequestDiplomacy = function(gameState, player, requestType, response, requiredTribute) 222 222 { 223 warn(uneval(['debug', gameState, player, requestType, response, requiredTribute])) 223 224 Engine.PostCommand(PlayerID, { 224 225 "type": "aichat", 225 226 "message": "/msg " + gameState.sharedScript.playersData[player].name + " " +
- start the attached replay from ticket #6654 (~2mins long)
- the game freezes at the 97th turn
- forcefully quit the
pyrogenesis
app - check activity monitor for
spindump
- For the case described above, the process is always called
spindump
to allocate large amounts of memory, but the process can also be calledsandboxed
orReportCrash
in other situations.
- For the case described above, the process is always called
to consistently reproduce (2)
- pass this to the terminal and run
pyrogenesis -replay-visual="/Users/paria\ crash"
- the game crashes with the following message
- check activity monitor for
ReportCrash
- same as above with
spindump
, if I don`t stop the process manually, the PC becomes unusable with time
- same as above with
bisect
- conflicting test results, could still have been due to spidermonkey
- reproducing the steps with the released A26 version did not show any problems, so the problem must be with macOS or somewhere in the development cycle of A27
with [27411] the game would also freeze, but once I would forcefully closepyrogenesis
no extra process would be eating up my memoryscreenshot below is from the terminal after I forcefully closedpyrogenesis
on [27411]
related ticket
related articles
Change History (10)
by , 16 months ago
Attachment: | terminal.png added |
---|
follow-up: 2 comment:1 by , 16 months ago
comment:2 by , 16 months ago
Replying to Stan‘:
In your case it's printing the game state thousands of times that crashes the game. Is it just a way to make it faster? Can't be vulkan if it happened before. So it's Spidermonley most likely but it's probably cause we're doing something it does not like
the issue is not about the lldb
(ignore that), it about an external processing spinning up called spindump
.
follow-up: 5 comment:3 by , 16 months ago
@langbart -> Did this happen to you in regular play? Or just when adding some debugs?
comment:5 by , 16 months ago
Replying to wraitii:
@langbart -> Did this happen to you in regular play? Or just when adding some debugs?
- I have also noticed it in a regular game but have not found consistently reproducible steps.
- I described some details here: Comment 1 in Ticket 6697
comment:6 by , 16 months ago
Description: | modified (diff) |
---|
The description was extended by another possibility to reproduce the problem consistently.
by , 16 months ago
Attachment: | term_report.png added |
---|
by , 16 months ago
Attachment: | ReportCrash.png added |
---|
comment:7 by , 16 months ago
Description: | modified (diff) |
---|
- Remove bisect number
- conflicting test results, could still have been due to spidermonkey
- reproducing the steps with the released A26 version did not show any problems, so the problem must be with macOS or somewhere in the development cycle of A27
- lldb has been removed as it is not needed for the problem.
- Reproduction steps remain valid
In your case it's printing the game state thousands of times that crashes the game. Is it just a way to make it faster? Can't be vulkan if it happened before. So it's Spidermonley most likely but it's probably cause we're doing something it does not like