You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@nuttx.apache.org by "daniel-p-carvalho (via GitHub)" <gi...@apache.org> on 2026/10/10 13:08:02 UTC

[PR] arch/mips/pic32mz: limit the Wi-Fi frames queued in the library. [nuttx]

daniel-p-carvalho opened a new pull request, #20575:
URL: https://github.com/apache/nuttx/pull/20575

## Summary

`wlan_transmit()` copied the packet to a library buffer and released the network packet at once, so the TX quota of the network device never limited anything. When the application sent faster than the radio, the library queue grew until the packet pool and then the whole heap were exhausted (106 KB of 108 KB in use with UDP from the board to a station).

Keep the network packet until the library frees the buffer (`DRV_PIC32MZW_MemFree`) and signal the TX done event from there. The TX quota now bounds the frames in flight; it is raised from 4 to 16 to keep the radio fed.

## Impact

PIC32MZ-W1 Wi-Fi driver only (`arch/mips/src/pic32mz/pic32mz_w1_wlan.c`). No API, Kconfig or documentation change. Throughput is unchanged; heap use under load drops.

## Testing

EV49N51A (PIC32MZ-W1), `ev49n51a:wlan` built with XC32 v6.00, Soft-AP, UDP from the board to a station (`iperf -c <station> -u`):

- heap peak 106 KB -> 46-51 KB
- throughput unchanged (22.6 Mbit/s)
- 3 station reassociations under load: no leak, IOB 48/48, queued == sent, 0 TX errors

`tools/checkpatch.sh -g upstream/master..HEAD` passes.

--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] arch/mips/pic32mz: limit the Wi-Fi frames queued in the library. [nuttx]

Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.

github-actions[bot] commented on PR #20575:
URL: https://github.com/apache/nuttx/pull/20575#issuecomment-6097885302

   <!-- membrowse-pr-comment -->
   ## <img src="https://membrowse.com/membrowse-logo.svg" height="24" align="top"> MemBrowse Memory Report
   
   *No memory changes detected for:*
   - [arduino-mega2560](https://membrowse.com/public/apache/nuttx/target/arduino-mega2560/compare?head=f5b6f5a07aa0309f11d8aa58cf87e4c8880b927b&base=2dc97a428e232ffe723d3ae69dbc02e01ea319e6)
   - [esp32-devkitc](https://membrowse.com/public/apache/nuttx/target/esp32-devkitc/compare?head=f5b6f5a07aa0309f11d8aa58cf87e4c8880b927b&base=2dc97a428e232ffe723d3ae69dbc02e01ea319e6)
   - [hifive1-revb](https://membrowse.com/public/apache/nuttx/target/hifive1-revb/compare?head=f5b6f5a07aa0309f11d8aa58cf87e4c8880b927b&base=2dc97a428e232ffe723d3ae69dbc02e01ea319e6)
   - [mirtoo](https://membrowse.com/public/apache/nuttx/target/mirtoo/compare?head=f5b6f5a07aa0309f11d8aa58cf87e4c8880b927b&base=2dc97a428e232ffe723d3ae69dbc02e01ea319e6)
   - [qemu-armv8a](https://membrowse.com/public/apache/nuttx/target/qemu-armv8a/compare?head=f5b6f5a07aa0309f11d8aa58cf87e4c8880b927b&base=2dc97a428e232ffe723d3ae69dbc02e01ea319e6)
   - [qemu-intel64](https://membrowse.com/public/apache/nuttx/target/qemu-intel64/compare?head=f5b6f5a07aa0309f11d8aa58cf87e4c8880b927b&base=2dc97a428e232ffe723d3ae69dbc02e01ea319e6)
   - [rx65n-rsk2mb](https://membrowse.com/public/apache/nuttx/target/rx65n-rsk2mb/compare?head=f5b6f5a07aa0309f11d8aa58cf87e4c8880b927b&base=2dc97a428e232ffe723d3ae69dbc02e01ea319e6)
   - [s698pm-dkit](https://membrowse.com/public/apache/nuttx/target/s698pm-dkit/compare?head=f5b6f5a07aa0309f11d8aa58cf87e4c8880b927b&base=2dc97a428e232ffe723d3ae69dbc02e01ea319e6)
   - [stm32-nucleo-f103rb](https://membrowse.com/public/apache/nuttx/target/stm32-nucleo-f103rb/compare?head=f5b6f5a07aa0309f11d8aa58cf87e4c8880b927b&base=2dc97a428e232ffe723d3ae69dbc02e01ea319e6)
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] arch/mips/pic32mz: limit the Wi-Fi frames queued in the library. [nuttx]

Posted by "xiaoxiang781216 (via GitHub)" <gi...@apache.org>.

xiaoxiang781216 merged PR #20575:
URL: https://github.com/apache/nuttx/pull/20575


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org