lib/ip: alias type for memory block name and cache addres space IDs in IP

fpga/include/villas/fpga/ip.hpp

@@ -127,8 +127,11 @@ public:
 	virtual void dump();
 
 protected:
+	/// Key-type for accessing maps addressTranslations and slaveAddressSpaces
+	using MemoryBlockName = std::string;
+
 	/// Each IP can declare via this function which memory blocks it requires
-	virtual std::list<std::string>
+	virtual std::list<MemoryBlockName>
 	getMemoryBlocks() const
 	{ return {}; }
 
@@ -177,11 +180,15 @@ public:
 
 protected:
 	uintptr_t
-	getBaseAddr(const std::string& block) const
+	getBaseAddr(const MemoryBlockName& block) const
 	{ return getLocalAddr(block, 0); }
 
 	uintptr_t
-	getLocalAddr(const std::string& block, uintptr_t address) const;
+	getLocalAddr(const MemoryBlockName& block, uintptr_t address) const;
+
+	MemoryManager::AddressSpaceId
+	getAddressSpaceId(const MemoryBlockName& block) const
+	{ return slaveAddressSpaces.at(block); }
 
 	InterruptController*
 	getInterruptController(const std::string& interruptName) const;
@@ -206,7 +213,10 @@ protected:
 	std::map<std::string, IrqPort> irqs;
 
 	/// Cached translations from the process address space to each memory block
-	std::map<std::string, MemoryTranslation> addressTranslations;
+	std::map<MemoryBlockName, MemoryTranslation> addressTranslations;
+
+	/// Lookup for IP's slave address spaces (= memory blocks)
+	std::map<MemoryBlockName, MemoryManager::AddressSpaceId> slaveAddressSpaces;
 
 	/// AXI bus master interfaces to access memory somewhere
 	std::map<std::string, MemoryManager::AddressSpaceId> busMasterInterfaces;

fpga/include/villas/fpga/ips/dma.hpp

@@ -84,7 +84,7 @@ private:
 	// optional Scatter-Gather interface to access descriptors
 	static constexpr char sgInterface[] = "M_AXI_SG";
 
-	std::list<std::string> getMemoryBlocks() const
+	std::list<MemoryBlockName> getMemoryBlocks() const
 	{ return { registerMemory }; }
 
 	XAxiDma xDma;

fpga/include/villas/fpga/ips/fifo.hpp

@@ -54,7 +54,7 @@ private:
 	static constexpr char axi4Memory[] = "Mem1";
 	static constexpr char irqName[] = "interrupt";
 
-	std::list<std::string> getMemoryBlocks() const
+	std::list<MemoryBlockName> getMemoryBlocks() const
 	{ return { registerMemory, axi4Memory }; }
 
 	XLlFifo xFifo;

fpga/include/villas/fpga/ips/intc.hpp

@@ -64,7 +64,7 @@ private:
 
 	static constexpr char registerMemory[] = "Reg";
 
-	std::list<std::string> getMemoryBlocks() const
+	std::list<MemoryBlockName> getMemoryBlocks() const
 	{ return { registerMemory }; }
 
 

fpga/include/villas/fpga/ips/switch.hpp

@@ -58,7 +58,7 @@ private:
 
 	static constexpr char registerMemory[] = "Reg";
 
-	std::list<std::string> getMemoryBlocks() const
+	std::list<MemoryBlockName> getMemoryBlocks() const
 	{ return { registerMemory }; }
 
 	struct Path {

fpga/include/villas/fpga/ips/timer.hpp

@@ -63,7 +63,7 @@ public:
 
 private:
 
-	std::list<std::string> getMemoryBlocks() const
+	std::list<MemoryBlockName> getMemoryBlocks() const
 	{ return { registerMemory }; }
 
 	static constexpr char irqName[] = "generateout0";

fpga/lib/ip.cpp

@@ -281,6 +281,9 @@ IpCoreFactory::make(PCIeCard* card, json_t *json_ips)
 			const auto addrSpaceId =
 			        MemoryManager::get().findAddressSpace(addrSpaceName);
 
+			// ... and save it in IP
+			ip->slaveAddressSpaces.emplace(memoryBlock, addrSpaceId);
+
 			// get the translation to the address space
 			const auto& translation =
 			        MemoryManager::get().getTranslationFromProcess(addrSpaceId);
@@ -341,7 +344,7 @@ IpCoreFactory::lookup(const Vlnv &vlnv)
 
 
 uintptr_t
-IpCore::getLocalAddr(const std::string& block, uintptr_t address) const
+IpCore::getLocalAddr(const MemoryBlockName& block, uintptr_t address) const
 {
 	// throws exception if block not present
 	auto& translation = addressTranslations.at(block);