Opened 4 years ago

Closed 4 years ago

#8858 closed bug (fixed)

Wrong maxStkSize calculation

Reported by: awson Owned by: thoughtpolice
Priority: normal Milestone: 7.8.1
Component: Runtime System Version: 7.8.1-rc2
Keywords: Cc: simonmar
Operating System: Unknown/Multiple Architecture: Unknown/Multiple
Type of failure: None/Unknown Test Case:
Blocked By: Blocking:
Related Tickets: Differential Rev(s):
Wiki Page:

Description

When fighting with #8839, I've found what I think is a bug in maxStkSize calculation: if getPhysicalMemorySize succeeds, we set maxStkSize in bytes, not words.

It did not manifest itself yet, but I've decided to provide a patch.

Attachments (1)

wrong_max_stack_size.patch (875 bytes) - added by awson 4 years ago.

Download all attachments as: .zip

Change History (6)

Changed 4 years ago by awson

Attachment: wrong_max_stack_size.patch added

comment:1 Changed 4 years ago by awson

Status: newpatch

comment:2 Changed 4 years ago by simonmar

Milestone: 7.8.1
Owner: changed from simonmar to thoughtpolice

yep, thanks.

comment:3 Changed 4 years ago by Austin Seipp <austin@…>

In b99ace39cb2484bfc2d648b55a1a43ed78e4b9a0/ghc:

Fix incorrect maxStkSize calculation (#8858)

Signed-off-by: Austin Seipp <austin@well-typed.com>

comment:4 Changed 4 years ago by thoughtpolice

Status: patchmerge

comment:5 Changed 4 years ago by thoughtpolice

Resolution: fixed
Status: mergeclosed

Merged in 7.8.

Note: See TracTickets for help on using tickets.