Critical section crash about esp-hal

[MabezDev]

I can't reproduce this. Could you share a full setup, with cargo settings etc in a repo?

[ghismary]

I can't reproduce this. Could you share a full setup, with cargo settings etc in a repo?

Sure! Here it is: https://gitlab.com/ghislainmary/esp32-crash

[MabezDev]

Thanks for the repo, it made it pretty easy to spot!

We recently started switching to runtime binding of interrupts, and to do so we need to put the interrupt table in RAM. Without this patch, the new interrupt handling code will try and write to read only memory (FLASH) which isn't possible, hence the exception.

esp-hal's has the patched pacs, you just need to add the same to your Cargo.toml:

esp-hal/esp-hal/Cargo.toml

Lines 227 to 235 in 1846543

	[patch.crates-io]
	esp32 = { git = "https://github.com/esp-rs/esp-pacs", rev = "963c280621f0b7ec26546a5eff24a5032305437f" }
	esp32s2 = { git = "https://github.com/esp-rs/esp-pacs", rev = "963c280621f0b7ec26546a5eff24a5032305437f" }
	esp32s3 = { git = "https://github.com/esp-rs/esp-pacs", rev = "963c280621f0b7ec26546a5eff24a5032305437f" }
	esp32c2 = { git = "https://github.com/esp-rs/esp-pacs", rev = "963c280621f0b7ec26546a5eff24a5032305437f" }
	esp32c3 = { git = "https://github.com/esp-rs/esp-pacs", rev = "963c280621f0b7ec26546a5eff24a5032305437f" }
	esp32c6 = { git = "https://github.com/esp-rs/esp-pacs", rev = "963c280621f0b7ec26546a5eff24a5032305437f" }
	esp32h2 = { git = "https://github.com/esp-rs/esp-pacs", rev = "963c280621f0b7ec26546a5eff24a5032305437f" }
	esp32p4 = { git = "https://github.com/esp-rs/esp-pacs", rev = "963c280621f0b7ec26546a5eff24a5032305437f" }

.

FYI it's advisable to not use git, unless you really need some feature that isn't release yet :).

[ghismary]

Thanks for the quick reply, I will try that!
I am using the git repo because I'm working on the implementation of transaction for async I²C that I need for a project of mine.

[ghismary]

I confirm this fixes the issue, so I'm closing it.

[MabezDev]

Thanks for the quick reply, I will try that! I am using the git repo because I'm working on the implementation of transaction for async I²C that I need for a project of mine.

Ah, I see! Glad to see someone working on that, let us know if you need any help :).