Assertion violation introduced by #81

[ghost]

Originally reported by: Anselm Kruis (Bitbucket: akruis, GitHub: akruis)

---

The fix for #81 (1388a20039578) introduced an invalid assertion. I discovered it during tests of 3.4-slp, but all other versions are affected too. Fortunately it is trivial to fix.

The failure message is:
Stackless/core/stacklesseval.c:556: slp_kill_tasks_with_stacks: Assertion `t->cstate == cs' failed

---

• Bitbucket: https://bitbucket.org/stackless-dev/stackless/issue/103

[ghost]

Original comment by Anselm Kruis (Bitbucket: akruis, GitHub: akruis):

---

Test case and fix is in changeset

• 2.7-slp: a2f6602
• 3.3-slp: f2725cf
• 3.4-slp: a5c6bdf
• default-slp: 328bc69

[ghost]

Original comment by Anselm Kruis (Bitbucket: akruis, GitHub: akruis):

---

Unfortunately I have to reopen this issue, because I found another incarnation of this problem Assertion failed: t->cstate == cs, file ..\Stackless\core\stacklesseval.c, line 459.

Unfortunately this assertion violation reveals a serious problem in slp_kill_tasks_with_stacks(). When I reworked this function (issue #81), I didn't take deeply nested tasklets into account.

[ghost]

Original comment by Anselm Kruis (Bitbucket: akruis, GitHub: akruis):

---

I just created pull request #21 for 2.7-slp.

[ghost]

Original comment by Anselm Kruis (Bitbucket: akruis, GitHub: akruis):

---

After two weeks of inactivity I decided to manually merge pull request #21, because it blocked a fix for issue #105.

Changesets:

• 2.7-slp: 130a60b
• 3.3-slp: 667cf50
• 3.4-slp: 510ae88
• default-slp: f5710d9