[riot-notifications] [RIOT-OS/RIOT] pkg/nimble/netif: fix and optimize NimBLE buffer sizes (#16503)
notifications at github.com
Thu Jun 10 09:44:53 CEST 2021
@haukepetersen commented on this pull request.
> CFLAGS += -DMYNEWT_VAL_BLE_L2CAP_COC_MAX_NUM=$(NIMBLE_MAX_CONN)
CFLAGS += -DMYNEWT_VAL_BLE_MAX_CONNECTIONS=$(NIMBLE_MAX_CONN)
- # NimBLEs internal buffer need to hold one IPv6 MTU per connection
- # for the internal MTU of 256 byte, we need 10 mbufs per connection...
+ # the maximum fragment size that we can receive. For maximum efficiency this
+ # should be equal to the maximum configured link layer packet size.
+ # WARNING: this value MUST never be larger than MYNEWT_VAL_BLE_LL_MAX_PKT_SIZE
+ CFLAGS += -DMYNEWT_VAL_BLE_L2CAP_COC_MPS=251
+ # in order to fit a 251 byte COC data segment into a single mbuf buffer, the
+ # used block size must be at least 297 byte (251 data + 48 overhead)
+ CFLAGS += -DMYNEWT_VAL_MSYS_1_BLOCK_SIZE=299
Yes, we could do that. Will fix.
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the notifications