diff options
author | Iliyan Malchev <malchev@google.com> | 2011-08-09 14:42:08 -0700 |
---|---|---|
committer | Dan Willemsen <dwillemsen@nvidia.com> | 2011-11-30 21:38:58 -0800 |
commit | 69b246de276c58e1de0bd8d0095adb83ca12cbc8 (patch) | |
tree | 05b96e198de2f6c0cf2b194ee9ae07912883cca8 /include/linux/ion.h | |
parent | 2b2f237feabf4f162525c77e597e088bdd41c632 (diff) |
ion: minor clean up
-- init rb nodes in ion_handle_create
-- in ion_handle_destroy, check that a node belongs to a tree before removing
it (safety check, does not happen right now)
-- mark as static functions used only inside ion.c
-- update comments to ion_share() with a relevant blurb from the implementation
-- other minor updates/typo fixes to comments
Signed-off-by: Iliyan Malchev <malchev@google.com>
Diffstat (limited to 'include/linux/ion.h')
-rw-r--r-- | include/linux/ion.h | 9 |
1 files changed, 7 insertions, 2 deletions
diff --git a/include/linux/ion.h b/include/linux/ion.h index 111982f48dc5..aed8349279ed 100644 --- a/include/linux/ion.h +++ b/include/linux/ion.h @@ -186,9 +186,14 @@ void ion_unmap_dma(struct ion_client *client, struct ion_handle *handle); * @client: the client * @handle: the handle to share * - * Given a handle, return a buffer which exists in a global name - * space and can be passed to other clients. Should be passed into ion_import + * Given a handle, return a buffer, which exists in a global name + * space, and can be passed to other clients. Should be passed into ion_import * to obtain a new handle for this buffer. + * + * NOTE: This function does do not an extra reference. The burden is on the + * caller to make sure the buffer doesn't go away while it's being passed to + * another client. That is, ion_free should not be called on this handle until + * the buffer has been imported into the other client. */ struct ion_buffer *ion_share(struct ion_client *client, struct ion_handle *handle); |