mem: move creation of virtual region to sof

creation of virtual region should be done in SOF,
not in Zephyr.

Virtual regions are simply a dynamic memory map, zephyr's
role is to provide the map and ensure no overlapping. Creation
of regions is up to SOF as needed

Signed-off-by: Marcin Szkudlinski <marcin.szkudlinski@intel.com>

Author: marcinszkudlinski

zephyr/Kconfig

@@ -50,6 +50,13 @@ config SOF_ZEPHYR_VIRTUAL_HEAP_SIZE
 	 NOTE: Keep in mind that the heap size should not be greater than the physical
 	 memory size of the system defined in DT (and this includes baseFW text/data).
 
+config SOF_ZEPHYR_VIRTUAL_HEAP_REGION_SIZE
+	hex "Size in bytes of virtual memory region for virtual heap shared for all cores"
+	depends on MM_DRV_INTEL_ADSP_MTL_TLB
+	default 0x100000
+	help
+	  This config defines size of virtual heap region shared between all cores
+
 config ZEPHYR_NATIVE_DRIVERS
 	bool "Use Zephyr native drivers"
 	default n

zephyr/include/sof/lib/regions_mm.h

@@ -17,6 +17,9 @@
 #include <zephyr/init.h>
 #include <zephyr/sys/mem_blocks.h>
 
+/* Attributes for memory regions */
+#define VIRTUAL_REGION_SHARED_HEAP_ATTR 1U	/*< region dedicated for shared virtual heap */
+
 /* Dependency on ipc/topology.h created due to memory capability definitions
  * that are defined there
  */

zephyr/lib/alloc.c

@@ -13,6 +13,7 @@
 #include <sof/schedule/schedule.h>
 #include <sof/lib/notifier.h>
 #include <sof/lib/pm_runtime.h>
+#include <sof/lib/regions_mm.h>
 #include <sof/audio/pipeline.h>
 #include <sof/audio/component_ext.h>
 #include <sof/trace/trace.h>
@@ -290,6 +291,18 @@ static const struct vmh_heap_config static_hp_buffers = {
 
 static int virtual_heap_init(void)
 {
+	int ret;
+
+	if (virtual_buffers_heap)
+		return -EEXIST;
+
+	/* add a virtual memory region */
+	ret = adsp_add_virtual_memory_region(adsp_mm_get_unused_l2_start_aligned(),
+					     CONFIG_SOF_ZEPHYR_VIRTUAL_HEAP_REGION_SIZE,
+					     VIRTUAL_REGION_SHARED_HEAP_ATTR);
+	if (ret)
+		return ret;
+
 	virtual_buffers_heap = vmh_init_heap(&static_hp_buffers, false);
 	if (!virtual_buffers_heap) {
 		tr_err(&zephyr_tr, "Unable to init virtual heap");

zephyr/lib/regions_mm.c

@@ -64,7 +64,7 @@ struct vmh_heap *vmh_init_heap(const struct vmh_heap_config *cfg, bool allocatin
 	const struct sys_mm_drv_region *region;
 
 	SYS_MM_DRV_MEMORY_REGION_FOREACH(virtual_memory_regions, region) {
-		if (region->attr == MEM_REG_ATTR_SHARED_HEAP) {
+		if (region->attr == VIRTUAL_REGION_SHARED_HEAP_ATTR) {
 			new_heap->virtual_region = region;
 			break;
 		}