Activity since 2025-05-30

• esp-rs/esp-idf-hal repo issues and pull requests

  • Compile error with uart #324 [adrien3d]

    Bug description When trying to compile a freshly generated project for esp32 from esp-idf-template I have the...

    • SCDY362: Which version of esp-idf has the correct implementation?

  • Task Cleanup leaks 160B of Memory #522 [smallbraingames] bug

    Bug description If a FreeRTOS task is created with ThreadSpawnConfig, it leaks 160 bytes in memory when cle Would...

    • benhansen-io: I think I am seeing the same or similar issue with plain std:🧵:spawn Reposting from the...

    • Vollbrecht: you run out of memory in under 1 sec with this approach right? I think before trying to...

    • benhansen-io: Gave that a shot, no real change. Still seeing unbounded memory growth. Code: for batch_num...

    • benhansen-io: Let me know if I can help try anything else. Maybe raw pthread create to see if it leaks?...

    • Vollbrecht: yeah technically that is what the std library is calling through newlib, but of course you can...

    • Vollbrecht: In terms of "where all the code lives" You can have a look at the newlib part of the rust libc...

    • ivmarkov: JFYI this is the implementation of the unix flavor of std::thread that is also used with...

    • Vollbrecht: to close the cycle on the C side, and to answer the question what newlib version is used. It...

    • benhansen-io: pthread_create match with pthread_join does not leak: for batch_num in 0..1000 { let...

    • ivmarkov: Then it must be something that the STD wrapper I referenced above does, which your raw pthread...

    • benhansen-io: Ok. Thanks for the pointers on how to test std code. When I comment out set_current inside...

    • ivmarkov: It should use crate::rt::thread_cleanup but perhaps it doesn't? You can test that by putting a...

    • ivmarkov: BTW @benhansen-io many thanks for your research so far! We absolutely have to fix this memory...

    • benhansen-io: Happy to help where I can. I timed out investigating today but here are my notes so...

    • ivmarkov: Thanks for the investigations so far. Side topic: since you mentioned SPIRAM and entering...

  • Remove Peripheral/PeripheralRef #529 [ivmarkov]

    Submission Checklist 📝 I have updated existing examples or added new ones (if applicable). I have used cargo fmt...

    • Vollbrecht: changing this from i32 to u8, we have to keep in mind that in general gpio numbering on the C...

      • ivmarkov: Yes, of course. Hence pin.pin() as _ everywhere, it is not like the compiler to allow us to use...

created at 2025-06-06 02:00 by dinghy 0.13.1.