Skip to content

llext: (cosmetic) rename a function #9856

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
lgirdwood merged 1 commit into from
Feb 25, 2025
+4 −4
Merged

llext: (cosmetic) rename a function #9856

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 4 additions & 4 deletions src/library_manager/llext_manager.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -483,7 +483,7 @@ static int llext_lib_find(const struct llext *llext, struct lib_manager_module *
return -ENOENT;
}

static void llext_depend_unlink(struct lib_manager_module *dep_ctx[], int n)
static void llext_manager_depend_unlink_rollback(struct lib_manager_module *dep_ctx[], int n)
{
for (; n >= 0; n--)
if (dep_ctx[n] && dep_ctx[n]->llext->use_count == 1)
Expand Down Expand Up @@ -565,7 +565,7 @@ uintptr_t llext_manager_allocate_module(const struct comp_ipc_config *ipc_config

ret = llext_manager_load_module(dep_ctx[i]);
if (ret < 0) {
llext_depend_unlink(dep_ctx, i - 1);
llext_manager_depend_unlink_rollback(dep_ctx, i - 1);
return 0;
}
}
Expand Down Expand Up @@ -617,15 +617,15 @@ int llext_manager_free_module(const uint32_t component_id)
}

struct lib_manager_module *dep_ctx[LLEXT_MAX_DEPENDENCIES] = {};
int i; /* signed to match llext_depend_unlink() */
int i; /* signed to match llext_manager_depend_unlink_rollback() */

for (i = 0; i < ARRAY_SIZE(mctx->llext->dependency); i++)
if (llext_lib_find(mctx->llext->dependency[i], &dep_ctx[i]) < 0)
break;

/* Last user cleaning up, put dependencies */
if (i)
llext_depend_unlink(dep_ctx, i - 1);
llext_manager_depend_unlink_rollback(dep_ctx, i - 1);

/*
* The last instance of the module has been destroyed and it can now be
Expand Down
Loading