Merged. On Mon, 26 Feb 2018, Greg D. wrote:
Excellent. Change pushed to my fork; pull request created.
Thanks!
Greg
Michael Balzer wrote:
I second that, the normal stack usage for the Housekeeper is already quite close to 4K:
Number of Tasks = 19 Stack: Now Max Total Heap 32-bit SPIRAM Task 3FFD3718 14 Housekeeping 356 3396 4096 91940 0 0
So I also think 6K should be the default.
Regards, Michael
Am 26.02.2018 um 08:54 schrieb Stephen Casner:
Greg,
Presuming that you are not doing some activity in the Housekeeping task that should really be done in some other task, and presuming that the activity you are doing is not something unreasonable, then indeed you have found a valid need for a larger stack and the change should be committed. The value 6144 is a reasonable step up.
-- Steve
On Sun, 25 Feb 2018, Greg D. wrote:
Ha, yes. Sorry. I bumped it up to 6144 (seems like a popular value), and that fixed it.
So, two questions: 1. What should a proper value be? I used 6144 only because I'd seen it used elsewhere. 2. Shall I push this for permanence?
Thanks,
Greg