[Mesa-dev] [PATCH 1/2] r600g/compute: Add documentation to compute_memory_pool
Bruno Jiménez
brunojimen at gmail.com
Thu Jul 10 14:16:03 PDT 2014
---
src/gallium/drivers/r600/compute_memory_pool.c | 54 ++++++++++++++++++++----
src/gallium/drivers/r600/compute_memory_pool.h | 58 ++++++++++++++++----------
2 files changed, 83 insertions(+), 29 deletions(-)
diff --git a/src/gallium/drivers/r600/compute_memory_pool.c b/src/gallium/drivers/r600/compute_memory_pool.c
index fe19d9e..1d0ec85 100644
--- a/src/gallium/drivers/r600/compute_memory_pool.c
+++ b/src/gallium/drivers/r600/compute_memory_pool.c
@@ -44,7 +44,7 @@
#define ITEM_ALIGNMENT 1024
/**
- * Creates a new pool
+ * Creates a new pool.
*/
struct compute_memory_pool* compute_memory_pool_new(
struct r600_screen * rscreen)
@@ -66,6 +66,12 @@ struct compute_memory_pool* compute_memory_pool_new(
return pool;
}
+/**
+ * Initializes the pool with a size of \a initial_size_in_dw.
+ * \param pool The pool to be initialized.
+ * \param initial_size_in_dw The initial size.
+ * \see compute_memory_grow_pool
+ */
static void compute_memory_pool_init(struct compute_memory_pool * pool,
unsigned initial_size_in_dw)
{
@@ -83,7 +89,7 @@ static void compute_memory_pool_init(struct compute_memory_pool * pool,
}
/**
- * Frees all stuff in the pool and the pool struct itself too
+ * Frees all stuff in the pool and the pool struct itself too.
*/
void compute_memory_pool_delete(struct compute_memory_pool* pool)
{
@@ -98,7 +104,10 @@ void compute_memory_pool_delete(struct compute_memory_pool* pool)
/**
* Searches for an empty space in the pool, return with the pointer to the
- * allocatable space in the pool, returns -1 on failure.
+ * allocatable space in the pool.
+ * \param size_in_dw The size of the space we are looking for.
+ * \return -1 on failure
+ * \see compute_memory_promote_item
*/
int64_t compute_memory_prealloc_chunk(
struct compute_memory_pool* pool,
@@ -130,6 +139,9 @@ int64_t compute_memory_prealloc_chunk(
/**
* Search for the chunk where we can link our new chunk after it.
+ * \param start_in_dw The position of the item we want to add to the pool.
+ * \return The item that is just before the passed position
+ * \see compute_memory_promote_item
*/
struct list_head *compute_memory_postalloc_chunk(
struct compute_memory_pool* pool,
@@ -171,7 +183,8 @@ struct list_head *compute_memory_postalloc_chunk(
/**
* Reallocates pool, conserves data.
- * @returns -1 if it fails, 0 otherwise
+ * \returns -1 if it fails, 0 otherwise
+ * \see compute_memory_finalize_pending and compute_memory_promote_item
*/
int compute_memory_grow_pool(struct compute_memory_pool* pool,
struct pipe_context * pipe, int new_size_in_dw)
@@ -212,6 +225,8 @@ int compute_memory_grow_pool(struct compute_memory_pool* pool,
/**
* Copy pool from device to host, or host to device.
+ * \param device_to_host 1 for device->host, 0 for host->device
+ * \see compute_memory_grow_pool
*/
void compute_memory_shadow(struct compute_memory_pool* pool,
struct pipe_context * pipe, int device_to_host)
@@ -229,8 +244,10 @@ void compute_memory_shadow(struct compute_memory_pool* pool,
}
/**
- * Allocates pending allocations in the pool
- * @returns -1 if it fails, 0 otherwise
+ * Moves all the items marked for promotion from the \a unallocated_list
+ * to the \a item_list.
+ * \return -1 if it fails, 0 otherwise
+ * \see evergreen_set_global_binding
*/
int compute_memory_finalize_pending(struct compute_memory_pool* pool,
struct pipe_context * pipe)
@@ -302,6 +319,12 @@ int compute_memory_finalize_pending(struct compute_memory_pool* pool,
return 0;
}
+/**
+ * Moves an item from the \a unallocated_list to the \a item_list.
+ * \param item The item that will be promoted.
+ * \return -1 if it fails, 0 otherwise
+ * \see compute_memory_finalize_pending
+ */
int compute_memory_promote_item(struct compute_memory_pool *pool,
struct compute_memory_item *item, struct pipe_context *pipe,
int64_t allocated)
@@ -373,6 +396,11 @@ int compute_memory_promote_item(struct compute_memory_pool *pool,
return 0;
}
+/**
+ * Moves an item from the \a item_list to the \a unallocated_list.
+ * \param item The item that will be demoted
+ * \see r600_compute_global_transfer_map
+ */
void compute_memory_demote_item(struct compute_memory_pool *pool,
struct compute_memory_item *item, struct pipe_context *pipe)
{
@@ -408,6 +436,10 @@ void compute_memory_demote_item(struct compute_memory_pool *pool,
item->start_in_dw = -1;
}
+/**
+ * Frees the memory asociated to the item with id \a id from the pool.
+ * \param id The id of the item to be freed.
+ */
void compute_memory_free(struct compute_memory_pool* pool, int64_t id)
{
struct compute_memory_item *item, *next;
@@ -457,7 +489,11 @@ void compute_memory_free(struct compute_memory_pool* pool, int64_t id)
}
/**
- * Creates pending allocations
+ * Creates pending allocations for new items, these items are
+ * placed in the unallocated_list.
+ * \param size_in_dw The size, in double words, of the new item.
+ * \return The new item
+ * \see r600_compute_global_buffer_create
*/
struct compute_memory_item* compute_memory_alloc(
struct compute_memory_pool* pool,
@@ -488,7 +524,9 @@ struct compute_memory_item* compute_memory_alloc(
}
/**
- * Transfer data host<->device, offset and size is in bytes
+ * Transfer data host<->device, offset and size is in bytes.
+ * \param device_to_host 1 for device->host, 0 for host->device.
+ * \see compute_memory_shadow
*/
void compute_memory_transfer(
struct compute_memory_pool* pool,
diff --git a/src/gallium/drivers/r600/compute_memory_pool.h b/src/gallium/drivers/r600/compute_memory_pool.h
index 259474a..51cb80e 100644
--- a/src/gallium/drivers/r600/compute_memory_pool.h
+++ b/src/gallium/drivers/r600/compute_memory_pool.h
@@ -36,13 +36,17 @@ struct compute_memory_pool;
struct compute_memory_item
{
- int64_t id; ///ID of the memory chunk
+ int64_t id; /**< ID of the memory chunk */
- uint32_t status; ///Will track the status of the item
+ uint32_t status; /**< Will track the status of the item */
- int64_t start_in_dw; ///Start pointer in dwords relative in the pool bo
- int64_t size_in_dw; ///Size of the chunk in dwords
+ /** Start pointer in dwords relative in the pool bo. If an item
+ * is unallocated, then this value must be -1 to indicate this. */
+ int64_t start_in_dw;
+ int64_t size_in_dw; /**< Size of the chunk in dwords */
+ /** Intermediate buffer asociated with an item. It is used mainly for mapping
+ * items against it. They are listed in the pool's unallocated list */
struct r600_resource *real_buffer;
struct compute_memory_pool* pool;
@@ -52,16 +56,20 @@ struct compute_memory_item
struct compute_memory_pool
{
- int64_t next_id; ///For generating unique IDs for memory chunks
- int64_t size_in_dw; ///Size of the pool in dwords
+ int64_t next_id; /**< For generating unique IDs for memory chunks */
+ int64_t size_in_dw; /**< Size of the pool in dwords */
- struct r600_resource *bo; ///The pool buffer object resource
+ struct r600_resource *bo; /**< The pool buffer object resource */
struct r600_screen *screen;
- uint32_t *shadow; ///host copy of the pool, used for defragmentation
+ uint32_t *shadow; /**< host copy of the pool, used for growing the pool */
- struct list_head *item_list; ///Allocated memory chunks in the buffer,they must be ordered by "start_in_dw"
- struct list_head *unallocated_list; ///Unallocated memory chunks
+ /** Allocated memory items in the pool, they must be ordered by "start_in_dw" */
+ struct list_head *item_list;
+
+ /** Unallocated memory items, this list contains all the items that aren't
+ * yet in the pool */
+ struct list_head *unallocated_list;
};
@@ -70,37 +78,45 @@ static inline int is_item_in_pool(struct compute_memory_item *item)
return item->start_in_dw != -1;
}
-struct compute_memory_pool* compute_memory_pool_new(struct r600_screen *rscreen); ///Creates a new pool
-void compute_memory_pool_delete(struct compute_memory_pool* pool); ///Frees all stuff in the pool and the pool struct itself too
+struct compute_memory_pool* compute_memory_pool_new(struct r600_screen *rscreen);
+
+void compute_memory_pool_delete(struct compute_memory_pool* pool);
-int64_t compute_memory_prealloc_chunk(struct compute_memory_pool* pool, int64_t size_in_dw); ///searches for an empty space in the pool, return with the pointer to the allocatable space in the pool, returns -1 on failure
+int64_t compute_memory_prealloc_chunk(struct compute_memory_pool* pool,
+ int64_t size_in_dw);
-struct list_head *compute_memory_postalloc_chunk(struct compute_memory_pool* pool, int64_t start_in_dw); ///search for the chunk where we can link our new chunk after it
+struct list_head *compute_memory_postalloc_chunk(struct compute_memory_pool* pool,
+ int64_t start_in_dw);
-int compute_memory_grow_pool(struct compute_memory_pool* pool, struct pipe_context * pipe,
- int new_size_in_dw);
+int compute_memory_grow_pool(struct compute_memory_pool* pool,
+ struct pipe_context *pipe, int new_size_in_dw);
void compute_memory_shadow(struct compute_memory_pool* pool,
- struct pipe_context * pipe, int device_to_host);
+ struct pipe_context *pipe, int device_to_host);
int compute_memory_finalize_pending(struct compute_memory_pool* pool,
struct pipe_context * pipe);
int compute_memory_promote_item(struct compute_memory_pool *pool,
- struct compute_memory_item *item, struct pipe_context *pipe,
- int64_t allocated);
+ struct compute_memory_item *item, struct pipe_context *pipe,
+ int64_t allocated);
void compute_memory_demote_item(struct compute_memory_pool *pool,
struct compute_memory_item *item, struct pipe_context *pipe);
void compute_memory_free(struct compute_memory_pool* pool, int64_t id);
-struct compute_memory_item* compute_memory_alloc(struct compute_memory_pool* pool, int64_t size_in_dw); ///Creates pending allocations
+
+struct compute_memory_item* compute_memory_alloc(struct compute_memory_pool* pool,
+ int64_t size_in_dw);
void compute_memory_transfer(struct compute_memory_pool* pool,
struct pipe_context * pipe, int device_to_host,
struct compute_memory_item* chunk, void* data,
int offset_in_chunk, int size);
-void compute_memory_transfer_direct(struct compute_memory_pool* pool, int chunk_to_data, struct compute_memory_item* chunk, struct r600_resource* data, int offset_in_chunk, int offset_in_data, int size); ///Transfer data between chunk<->data, it is for VRAM<->GART transfers
+void compute_memory_transfer_direct(struct compute_memory_pool* pool,
+ int chunk_to_data, struct compute_memory_item* chunk,
+ struct r600_resource* data, int offset_in_chunk,
+ int offset_in_data, int size);
#endif
--
2.0.1
More information about the mesa-dev
mailing list