Skip to content

Further critical section issues #6427

Closed
@devyte

Description

@devyte

Basic Infos

  • This issue complies with the issue POLICY doc.
  • I have read the documentation at readthedocs and the issue is not addressed there.
  • I have tested that the issue is present in current master branch (aka latest git).
  • I have searched the issue tracker for a similar issue.
  • If there is a stack dump, I have decoded it.
  • I have filled out all fields below.

Platform

All

Settings in IDE

All

Problem Description

With #6274, and related PRs, several major flaws in interrupt locking were addressed in our core code, especially in umm malloc (configured defines for critical sections). In essence, ets_intr_{un}lock() were found to not allow nesting of interrupt critical sections, which would lead to instability.
The core code was fixed, but the SDK and ROM code is still pending.

The SDK closed source libs must be searched for calls to ets_intr_{un}lock() as well as the asm instruction(s) that dis/enable interrupts. Libs that contain such calls must be patched to call a function in our core instead.

The ROM must be reverse engineered enough to build a calling tree of every ROM function that calls ets_intr{un}lock(), and work upwards through all functions that call that function, and so on. Then, the SDK closed libs (and possibly our core) must be patched by adding fixed versions of the ROM functions, and then have the SDK libs (and possibly our core) call those functions instead of the ROM ones.

The above is, of course, a top level view of what must be done. It's entirely possible that some linker magic could make things somewhat easier.

Metadata

Metadata

Assignees

Type

No type

Projects

No projects

Milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions