llext: debug lmdk failure

Commit bd9a877 ("lib-manager: store segment sizes locally") breaks system service-based modules, check why. Signed-off-by: Guennadi Liakhovetski <guennadi.liakhovetski@linux.intel.com>
thesofproject · Jan 23, 2024 · 1fbc349 · 1fbc349
1 parent 03e8bd2
commit 1fbc349
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 2 deletions.
diff --git a/src/audio/module_adapter/module/modules.c b/src/audio/module_adapter/module/modules.c
@@ -78,7 +78,6 @@ static int modules_init(struct processing_module *mod)
 		return -EINVAL;
 	}
 	md->module_entry_point = module_entry_point;
-	comp_info(mod->dev, "modules_init() start");
 
 	uint32_t module_id = IPC4_MOD_ID(mod->dev->ipc_config.id);
 	uint32_t instance_id = IPC4_INST_ID(mod->dev->ipc_config.id);
@@ -101,6 +100,11 @@ static int modules_init(struct processing_module *mod)
 
 	void *mod_adp;
 
+	comp_info(mod->dev, "modules_init() start %#x desc %p mod-entry %p buildinfo %p",
+		  module_id, desc, module_entry, mod_buildinfo);
+	comp_info(mod->dev, "modules_init() fmt %#x version %#x", mod_buildinfo->format,
+		  mod_buildinfo->api_version_number.full);
+
 	/* Check if module is FDK */
 	if (mod_buildinfo->format == IADK_MODULE_API_BUILD_INFO_FORMAT &&
 	    mod_buildinfo->api_version_number.full == IADK_MODULE_API_CURRENT_VERSION) {

diff --git a/src/library_manager/lib_manager.c b/src/library_manager/lib_manager.c
@@ -208,6 +208,9 @@ static int lib_manager_load_module(uint32_t module_id, struct sof_man_module *mo
 	/* Copy Code */
 	ret = lib_manager_load_data_from_storage(va_base_text, src_txt, st_text_size,
 						 SYS_MM_MEM_PERM_RW | SYS_MM_MEM_PERM_EXEC);
+	tr_info(&lib_manager_tr, "copy %#x of .text from %p to %p: %#x",
+		st_text_size, src_txt, va_base_text,
+		((unsigned int *)va_base_text)[0]/*, ((unsigned int *)va_base_text)[1]*/);
 	if (ret < 0)
 		return ret;
 
@@ -797,7 +800,7 @@ int lib_manager_load_library(uint32_t dma_id, uint32_t lib_id, uint32_t type)
 	_ext_lib->runtime_data = NULL;
 
 	if (!ret)
-		tr_info(&ipc_tr, "loaded library id: %u", lib_id);
+		tr_info(&lib_manager_tr, "loaded library id: %u", lib_id);
 
 	return ret;
 }