Skip to content

Commit

Permalink
samples: matter: Disabled Mbed TLS threading for nRF54L15
Browse files Browse the repository at this point in the history
After introducing mutexes in Mbed TLS, there are sometimes
bus faults visible on the nRF54L15. Temporarily it has to be
disabled as a workaround, to bring back the stable behavior.

By the way pulled connectedhomeip fix increasing the CHIP
stack size for nRF54L15.

Signed-off-by: Kamil Kasperczyk <kamil.kasperczyk@nordicsemi.no>
  • Loading branch information
kkasperczyk-no committed Oct 18, 2024
1 parent 500844c commit 59ddd94
Show file tree
Hide file tree
Showing 9 changed files with 36 additions and 9 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,11 @@ CONFIG_BT_LL_SOFTDEVICE_MULTIROLE=y
# nRF54L15 requires bigger stack sizes than nRF52/nRF53 families
CONFIG_CHIP_MALLOC_SYS_HEAP_SIZE=10240
CONFIG_MPSL_WORK_STACK_SIZE=2048
CONFIG_CHIP_TASK_STACK_SIZE=7168

# Set the ZMS sector count to match the settings partition size that is 40 kB for this application.
CONFIG_SETTINGS_ZMS_SECTOR_COUNT=10

# Workaround: Disable MBEDTLS threading, as in the Mbed TLS 3.6.x the mutexes were added, what sometimes
# leads to bus faults when OpenThread calls psa_get_and_lock_key_slot on SRP service removal.

Check failure on line 18 in samples/matter/light_bulb/boards/nrf54l15dk_nrf54l15_cpuapp.conf

View workflow job for this annotation

GitHub Actions / Run compliance checks on patch series (PR)

TRAILING_WHITESPACE

samples/matter/light_bulb/boards/nrf54l15dk_nrf54l15_cpuapp.conf:18 trailing whitespace
# This brings back the behavior that was proven to work before.
CONFIG_MBEDTLS_THREADING_C=n
Original file line number Diff line number Diff line change
Expand Up @@ -10,10 +10,14 @@ CONFIG_BT_LL_SOFTDEVICE_MULTIROLE=y
# nRF54L15 requires bigger stack sizes than nRF52/nRF53 families
CONFIG_CHIP_MALLOC_SYS_HEAP_SIZE=10240
CONFIG_MPSL_WORK_STACK_SIZE=2048
CONFIG_CHIP_TASK_STACK_SIZE=7168

# Set the ZMS sector count to match the settings partition size that is 40 kB for this application.
CONFIG_SETTINGS_ZMS_SECTOR_COUNT=10

# Low Power mode
CONFIG_POWEROFF=y

# Workaround: Disable MBEDTLS threading, as in the Mbed TLS 3.6.x the mutexes were added, what sometimes
# leads to bus faults when OpenThread calls psa_get_and_lock_key_slot on SRP service removal.

Check failure on line 21 in samples/matter/light_switch/boards/nrf54l15dk_nrf54l15_cpuapp.conf

View workflow job for this annotation

GitHub Actions / Run compliance checks on patch series (PR)

TRAILING_WHITESPACE

samples/matter/light_switch/boards/nrf54l15dk_nrf54l15_cpuapp.conf:21 trailing whitespace
# This brings back the behavior that was proven to work before.
CONFIG_MBEDTLS_THREADING_C=n
6 changes: 5 additions & 1 deletion samples/matter/lock/boards/nrf54l15dk_nrf54l15_cpuapp.conf
Original file line number Diff line number Diff line change
Expand Up @@ -10,10 +10,14 @@ CONFIG_BT_LL_SOFTDEVICE_MULTIROLE=y
# nRF54L15 requires bigger stack sizes than nRF52/nRF53 families
CONFIG_CHIP_MALLOC_SYS_HEAP_SIZE=10240
CONFIG_MPSL_WORK_STACK_SIZE=2048
CONFIG_CHIP_TASK_STACK_SIZE=7168

# Set the ZMS sector count to match the settings partition size that is 40 kB for this application.
CONFIG_SETTINGS_ZMS_SECTOR_COUNT=10

# Low Power mode
CONFIG_POWEROFF=y

# Workaround: Disable MBEDTLS threading, as in the Mbed TLS 3.6.x the mutexes were added, what sometimes
# leads to bus faults when OpenThread calls psa_get_and_lock_key_slot on SRP service removal.

Check failure on line 21 in samples/matter/lock/boards/nrf54l15dk_nrf54l15_cpuapp.conf

View workflow job for this annotation

GitHub Actions / Run compliance checks on patch series (PR)

TRAILING_WHITESPACE

samples/matter/lock/boards/nrf54l15dk_nrf54l15_cpuapp.conf:21 trailing whitespace
# This brings back the behavior that was proven to work before.
CONFIG_MBEDTLS_THREADING_C=n
Original file line number Diff line number Diff line change
Expand Up @@ -10,10 +10,14 @@ CONFIG_BT_LL_SOFTDEVICE_MULTIROLE=y
# nRF54L15 requires bigger stack sizes than nRF52/nRF53 families
CONFIG_CHIP_MALLOC_SYS_HEAP_SIZE=10240
CONFIG_MPSL_WORK_STACK_SIZE=2048
CONFIG_CHIP_TASK_STACK_SIZE=7168

# Set the ZMS sector count to match the settings partition size that is 40 kB for this application.
CONFIG_SETTINGS_ZMS_SECTOR_COUNT=10

# Low Power mode
CONFIG_POWEROFF=y

# Workaround: Disable MBEDTLS threading, as in the Mbed TLS 3.6.x the mutexes were added, what sometimes
# leads to bus faults when OpenThread calls psa_get_and_lock_key_slot on SRP service removal.

Check failure on line 21 in samples/matter/smoke_co_alarm/boards/nrf54l15dk_nrf54l15_cpuapp.conf

View workflow job for this annotation

GitHub Actions / Run compliance checks on patch series (PR)

TRAILING_WHITESPACE

samples/matter/smoke_co_alarm/boards/nrf54l15dk_nrf54l15_cpuapp.conf:21 trailing whitespace
# This brings back the behavior that was proven to work before.
CONFIG_MBEDTLS_THREADING_C=n
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,11 @@ CONFIG_BT_LL_SOFTDEVICE_MULTIROLE=y
# nRF54L15 requires bigger stack sizes than nRF52/nRF53 families
CONFIG_CHIP_MALLOC_SYS_HEAP_SIZE=10240
CONFIG_MPSL_WORK_STACK_SIZE=2048
CONFIG_CHIP_TASK_STACK_SIZE=7168

# Set the ZMS sector count to match the settings partition size that is 40 kB for this application.
CONFIG_SETTINGS_ZMS_SECTOR_COUNT=10

# Workaround: Disable MBEDTLS threading, as in the Mbed TLS 3.6.x the mutexes were added, what sometimes
# leads to bus faults when OpenThread calls psa_get_and_lock_key_slot on SRP service removal.

Check failure on line 18 in samples/matter/template/boards/nrf54l15dk_nrf54l15_cpuapp.conf

View workflow job for this annotation

GitHub Actions / Run compliance checks on patch series (PR)

TRAILING_WHITESPACE

samples/matter/template/boards/nrf54l15dk_nrf54l15_cpuapp.conf:18 trailing whitespace
# This brings back the behavior that was proven to work before.
CONFIG_MBEDTLS_THREADING_C=n
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,6 @@ CONFIG_BT_LL_SOFTDEVICE_MULTIROLE=y
# nRF54L15 requires bigger stack sizes than nRF52/nRF53 families
CONFIG_CHIP_MALLOC_SYS_HEAP_SIZE=10240
CONFIG_MPSL_WORK_STACK_SIZE=2048
CONFIG_CHIP_TASK_STACK_SIZE=7168

# Set the ZMS sector count to match the settings partition size that is 44 kB for this application.
CONFIG_SETTINGS_ZMS_SECTOR_COUNT=11
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,11 @@ CONFIG_BT_LL_SOFTDEVICE_MULTIROLE=y
# nRF54L15 requires bigger stack sizes than nRF52/nRF53 families
CONFIG_CHIP_MALLOC_SYS_HEAP_SIZE=10240
CONFIG_MPSL_WORK_STACK_SIZE=2048
CONFIG_CHIP_TASK_STACK_SIZE=7168

# Set the ZMS sector count to match the settings partition size that is 40 kB for this application.
CONFIG_SETTINGS_ZMS_SECTOR_COUNT=10

# Workaround: Disable MBEDTLS threading, as in the Mbed TLS 3.6.x the mutexes were added, what sometimes
# leads to bus faults when OpenThread calls psa_get_and_lock_key_slot on SRP service removal.

Check failure on line 18 in samples/matter/thermostat/boards/nrf54l15dk_nrf54l15_cpuapp.conf

View workflow job for this annotation

GitHub Actions / Run compliance checks on patch series (PR)

TRAILING_WHITESPACE

samples/matter/thermostat/boards/nrf54l15dk_nrf54l15_cpuapp.conf:18 trailing whitespace
# This brings back the behavior that was proven to work before.
CONFIG_MBEDTLS_THREADING_C=n
Original file line number Diff line number Diff line change
Expand Up @@ -10,10 +10,14 @@ CONFIG_BT_LL_SOFTDEVICE_MULTIROLE=y
# nRF54L15 requires bigger stack sizes than nRF52/nRF53 families
CONFIG_CHIP_MALLOC_SYS_HEAP_SIZE=10240
CONFIG_MPSL_WORK_STACK_SIZE=2048
CONFIG_CHIP_TASK_STACK_SIZE=7168

# Set the ZMS sector count to match the settings partition size that is 40 kB for this application.
CONFIG_SETTINGS_ZMS_SECTOR_COUNT=10

# Low Power mode
CONFIG_POWEROFF=y

# Workaround: Disable MBEDTLS threading, as in the Mbed TLS 3.6.x the mutexes were added, what sometimes
# leads to bus faults when OpenThread calls psa_get_and_lock_key_slot on SRP service removal.

Check failure on line 21 in samples/matter/window_covering/boards/nrf54l15dk_nrf54l15_cpuapp.conf

View workflow job for this annotation

GitHub Actions / Run compliance checks on patch series (PR)

TRAILING_WHITESPACE

samples/matter/window_covering/boards/nrf54l15dk_nrf54l15_cpuapp.conf:21 trailing whitespace
# This brings back the behavior that was proven to work before.
CONFIG_MBEDTLS_THREADING_C=n
2 changes: 1 addition & 1 deletion west.yml
Original file line number Diff line number Diff line change
Expand Up @@ -161,7 +161,7 @@ manifest:
- name: matter
repo-path: sdk-connectedhomeip
path: modules/lib/matter
revision: 171a1d323bf91ac7a655f93c70c18ebb6c1a2253
revision: pull/497/head
west-commands: scripts/west/west-commands.yml
submodules:
- name: nlio
Expand Down

0 comments on commit 59ddd94

Please sign in to comment.