Pull request project-chip#2100: Fix deadlock between chip task blocked because 917 nvm3 store can't completeas it also waits on a event to be process by chip task

Merge in WMN_TOOLS/matter from fix/917_deadlock to RC_2.3.1-1.3

Squashed commit of the following:

commit b6d75e6aefd494faca9f1ce4c499087b44521805
Author: Junior Martinez <67972863+jmartinez-silabs@users.noreply.github.com>
Date:   Fri Jan 24 14:02:24 2025 -0500

    Fix deadlock between chip task blocked because  917 nvm3 store can't complete wait  as it also way on event to be process by chip task

Author: jmartinez-silabs

src/platform/silabs/KeyValueStoreManagerImpl.cpp

@@ -164,15 +164,14 @@ void KeyValueStoreManagerImpl::ScheduleKeyMapSave(void)
         During commissioning, the key map will be modified multiples times subsequently.
         Commit the key map in nvm once it as stabilized.
     */
-#if SLI_SI91X_MCU_INTERFACE && CHIP_CONFIG_ENABLE_ICD_SERVER
-    // TODO: Remove this when RTC timer is added MATTER-2705
-    SilabsConfig::WriteConfigValueBin(SilabsConfig::kConfigKey_KvsStringKeyMap, reinterpret_cast<const uint8_t *>(mKvsKeyMap),
-                                      sizeof(mKvsKeyMap));
+#if SLI_SI91X_MCU_INTERFACE
+    // Avoid risk of deadlock by saving the key map immediately
+    ForceKeyMapSave();
 #else
     SystemLayer().StartTimer(
         std::chrono::duration_cast<System::Clock::Timeout>(System::Clock::Seconds32(SL_KVS_SAVE_DELAY_SECONDS)),
         KeyValueStoreManagerImpl::OnScheduledKeyMapSave, NULL);
-#endif // defined(SLI_SI91X_MCU_INTERFACE) && CHIP_CONFIG_ENABLE_ICD_SERVER
+#endif // defined(SLI_SI91X_MCU_INTERFACE)
 }
 
 CHIP_ERROR KeyValueStoreManagerImpl::_Get(const char * key, void * value, size_t value_size, size_t * read_bytes_size,

src/platform/silabs/SiWx917/wifi/wfx_notify.cpp

@@ -155,11 +155,7 @@ void wfx_ipv6_notify(int got_ip)
     /* So the other threads can run and have the connectivity OK */
     if (got_ip)
     {
-        /* Should remember this */
-        vTaskDelay(1);
-        chip::DeviceLayer::PlatformMgr().LockChipStack();
-        chip::app::DnssdServer::Instance().StartServer(/*Dnssd::CommissioningMode::kEnabledBasic*/);
-        chip::DeviceLayer::PlatformMgr().UnlockChipStack();
+        DeviceLayer::SystemLayer().ScheduleLambda([] { app::DnssdServer::Instance().StartServer(); });
     }
 }
 
@@ -184,11 +180,7 @@ void wfx_ip_changed_notify(int got_ip)
     /* So the other threads can run and have the connectivity OK */
     if (got_ip)
     {
-        /* Should remember this */
-        vTaskDelay(1);
-        chip::DeviceLayer::PlatformMgr().LockChipStack();
-        chip::app::DnssdServer::Instance().StartServer(/*Dnssd::CommissioningMode::kEnabledBasic*/);
-        chip::DeviceLayer::PlatformMgr().UnlockChipStack();
+        DeviceLayer::SystemLayer().ScheduleLambda([] { app::DnssdServer::Instance().StartServer(); });
     }
 }