On 11/10/16 20:21, Christophe Milard wrote:
The capability "can_getphy" is introduced and tells whether physical
address queries are available.
The function odpdrv_getphy() is added to query for physical address
(from virtual address)
Signed-off-by: Christophe Milard <christophe.mil...@linaro.org>
---
include/odp/drv/spec/shm.h | 35 +++++++++++++++++++++++++++++++++++
1 file changed, 35 insertions(+)
diff --git a/include/odp/drv/spec/shm.h b/include/odp/drv/spec/shm.h
index ef64f5d..ca3aae2 100644
--- a/include/odp/drv/spec/shm.h
+++ b/include/odp/drv/spec/shm.h
@@ -43,6 +43,12 @@ extern "C" {
#define ODPDRV_SHM_LOCK 0x02 /**< Memory shall be locked (no
swap) */
/**
+ * @typedef odpdrv_phys_addr_t
+ * A physical memory address
+ *
+ */
+
+/**
* Shared memory block info
*/
typedef struct odpdrv_shm_info_t {
@@ -75,6 +81,13 @@ typedef struct odpdrv_shm_capability_t {
* available memory size. */
uint64_t max_align;
+ /** Can retrieve physical addresses
+ *
+ * A true (non-zero) value means that odpdrv_getphy() can be
+ * used to retrieve physical memory addresses, (as well as
+ * the debug function odp_drv_dumpphy() */
+ int can_getphy;
intnphys_addr:1;
+
} odpdrv_shm_capability_t;
/**
@@ -220,6 +233,28 @@ int odpdrv_shm_print_all(const char *title);
uint64_t odpdrv_shm_to_u64(odpdrv_shm_t hdl);
/**
+ * Get physical address from virtual address
+ *
+ * @addr virtual address.
+ * @return Physicall address or NULL
+ *
+ * @note This routine will work only if capability "can_getphy" is true.
+ */
+odpdrv_phys_addr_t odpdrv_getphy(const void *addr);
naming is bad. By 'phy' I used to see physics layer chip, not physical
memory.
Something similar to linux kernel might be:
odpdrv_virt_to_phys()
odpdrv_phys_to_virt()
and (I don't know how writes done from user space), but probably you
need odp version of ioremap()
+
+/**
+ * Print physical address mapping
+ *
print where?
+ * @addr virtual address.
what is len?
+ * @return Physicall address or NULL
+ *
it's void, no return.
+ * @note This routine will work only if capability "can_getphy" is true.
+ * @note This routine is intended to be used for diagnostic purposes,
+ * and uses ODP_DBG printouts
+ */
+void odpdrv_dumpphy(void *addr, uint64_t len);
Maybe odpdrv_memmap_print()? to print current mappings by driver?
should driver handle be passed as argument?
+
+/**
* @}
*/
