Commit 61d5325b authored by Suman Anna's avatar Suman Anna Committed by Hari Kanigeri

SYSLINK: ipc - heapbuf bug fixes

This patch fixes the bugs found during heapbuf sample validation.
The following are the main changes.
	- object creation attributes are assigned conditionally
		based on the create flag passed in _heapbuf_create
	- object lock has been commented out.
	- gt_traces are removed.

These fixes does not resolve the issues seen with heapbuf_destroy
in the sample.
Signed-off-by: default avatarArun M G <arunmg@ti.com>
parent 8438a4e8
This diff is collapsed.
...@@ -82,6 +82,7 @@ static int heapbuf_ioctl_params_init(struct heapbuf_cmd_args *cargs) ...@@ -82,6 +82,7 @@ static int heapbuf_ioctl_params_init(struct heapbuf_cmd_args *cargs)
u32 size; u32 size;
heapbuf_params_init(cargs->args.params_init.handle, &params); heapbuf_params_init(cargs->args.params_init.handle, &params);
cargs->api_status = 0;
size = copy_to_user(cargs->args.params_init.params, &params, size = copy_to_user(cargs->args.params_init.params, &params,
sizeof(struct heapbuf_params)); sizeof(struct heapbuf_params));
if (size) if (size)
...@@ -149,7 +150,7 @@ exit: ...@@ -149,7 +150,7 @@ exit:
*/ */
static int heapbuf_ioctl_delete(struct heapbuf_cmd_args *cargs) static int heapbuf_ioctl_delete(struct heapbuf_cmd_args *cargs)
{ {
cargs->api_status = heapbuf_delete(cargs->args.delete.handle); cargs->api_status = heapbuf_delete(&cargs->args.delete.handle);
return 0; return 0;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment