kernel: rename h -> heap
Avoid single characker variables that renders code unreadable and might
cause conflicts in maing, similar to t for both timeout and thread in
some places.
Signed-off-by: Anas Nashif <anas.nashif@intel.com>
diff --git a/kernel/kheap.c b/kernel/kheap.c
index 71a3da5..fd29df2 100644
--- a/kernel/kheap.c
+++ b/kernel/kheap.c
@@ -12,17 +12,17 @@
#include <ksched.h>
#include <wait_q.h>
-void k_heap_init(struct k_heap *h, void *mem, size_t bytes)
+void k_heap_init(struct k_heap *heap, void *mem, size_t bytes)
{
- z_waitq_init(&h->wait_q);
- sys_heap_init(&h->heap, mem, bytes);
+ z_waitq_init(&heap->wait_q);
+ sys_heap_init(&heap->heap, mem, bytes);
- SYS_PORT_TRACING_OBJ_INIT(k_heap, h);
+ SYS_PORT_TRACING_OBJ_INIT(k_heap, heap);
}
static int statics_init(void)
{
- STRUCT_SECTION_FOREACH(k_heap, h) {
+ STRUCT_SECTION_FOREACH(k_heap, heap) {
#if defined(CONFIG_DEMAND_PAGING) && !defined(CONFIG_LINKER_GENERIC_SECTIONS_PRESENT_AT_BOOT)
/* Some heaps may not present at boot, so we need to wait for
* paging mechanism to be initialized before we can initialize
@@ -37,17 +37,17 @@
* pinned region as they have already been initialized and
* possibly already in use. Otherwise initialize.
*/
- if (lnkr_is_pinned((uint8_t *)h) &&
- lnkr_is_pinned((uint8_t *)&h->wait_q) &&
- lnkr_is_region_pinned((uint8_t *)h->heap.init_mem,
- h->heap.init_bytes)) {
+ if (lnkr_is_pinned((uint8_t *)heap) &&
+ lnkr_is_pinned((uint8_t *)&heap->wait_q) &&
+ lnkr_is_region_pinned((uint8_t *)heap->heap.init_mem,
+ heap->heap.init_bytes)) {
do_clear = !do_clear;
}
if (do_clear)
#endif /* CONFIG_DEMAND_PAGING && !CONFIG_LINKER_GENERIC_SECTIONS_PRESENT_AT_BOOT */
{
- k_heap_init(h, h->heap.init_mem, h->heap.init_bytes);
+ k_heap_init(heap, heap->heap.init_mem, heap->heap.init_bytes);
}
}
return 0;
@@ -62,22 +62,22 @@
SYS_INIT_NAMED(statics_init_post, statics_init, POST_KERNEL, 0);
#endif /* CONFIG_DEMAND_PAGING && !CONFIG_LINKER_GENERIC_SECTIONS_PRESENT_AT_BOOT */
-void *k_heap_aligned_alloc(struct k_heap *h, size_t align, size_t bytes,
+void *k_heap_aligned_alloc(struct k_heap *heap, size_t align, size_t bytes,
k_timeout_t timeout)
{
k_timepoint_t end = sys_timepoint_calc(timeout);
void *ret = NULL;
- k_spinlock_key_t key = k_spin_lock(&h->lock);
+ k_spinlock_key_t key = k_spin_lock(&heap->lock);
- SYS_PORT_TRACING_OBJ_FUNC_ENTER(k_heap, aligned_alloc, h, timeout);
+ SYS_PORT_TRACING_OBJ_FUNC_ENTER(k_heap, aligned_alloc, heap, timeout);
__ASSERT(!arch_is_in_isr() || K_TIMEOUT_EQ(timeout, K_NO_WAIT), "");
bool blocked_alloc = false;
while (ret == NULL) {
- ret = sys_heap_aligned_alloc(&h->heap, align, bytes);
+ ret = sys_heap_aligned_alloc(&heap->heap, align, bytes);
if (!IS_ENABLED(CONFIG_MULTITHREADING) ||
(ret != NULL) || K_TIMEOUT_EQ(timeout, K_NO_WAIT)) {
@@ -87,7 +87,7 @@
if (!blocked_alloc) {
blocked_alloc = true;
- SYS_PORT_TRACING_OBJ_FUNC_BLOCKING(k_heap, aligned_alloc, h, timeout);
+ SYS_PORT_TRACING_OBJ_FUNC_BLOCKING(k_heap, aligned_alloc, heap, timeout);
} else {
/**
* @todo Trace attempt to avoid empty trace segments
@@ -95,37 +95,37 @@
}
timeout = sys_timepoint_timeout(end);
- (void) z_pend_curr(&h->lock, key, &h->wait_q, timeout);
- key = k_spin_lock(&h->lock);
+ (void) z_pend_curr(&heap->lock, key, &heap->wait_q, timeout);
+ key = k_spin_lock(&heap->lock);
}
- SYS_PORT_TRACING_OBJ_FUNC_EXIT(k_heap, aligned_alloc, h, timeout, ret);
+ SYS_PORT_TRACING_OBJ_FUNC_EXIT(k_heap, aligned_alloc, heap, timeout, ret);
- k_spin_unlock(&h->lock, key);
+ k_spin_unlock(&heap->lock, key);
return ret;
}
-void *k_heap_alloc(struct k_heap *h, size_t bytes, k_timeout_t timeout)
+void *k_heap_alloc(struct k_heap *heap, size_t bytes, k_timeout_t timeout)
{
- SYS_PORT_TRACING_OBJ_FUNC_ENTER(k_heap, alloc, h, timeout);
+ SYS_PORT_TRACING_OBJ_FUNC_ENTER(k_heap, alloc, heap, timeout);
- void *ret = k_heap_aligned_alloc(h, sizeof(void *), bytes, timeout);
+ void *ret = k_heap_aligned_alloc(heap, sizeof(void *), bytes, timeout);
- SYS_PORT_TRACING_OBJ_FUNC_EXIT(k_heap, alloc, h, timeout, ret);
+ SYS_PORT_TRACING_OBJ_FUNC_EXIT(k_heap, alloc, heap, timeout, ret);
return ret;
}
-void k_heap_free(struct k_heap *h, void *mem)
+void k_heap_free(struct k_heap *heap, void *mem)
{
- k_spinlock_key_t key = k_spin_lock(&h->lock);
+ k_spinlock_key_t key = k_spin_lock(&heap->lock);
- sys_heap_free(&h->heap, mem);
+ sys_heap_free(&heap->heap, mem);
- SYS_PORT_TRACING_OBJ_FUNC(k_heap, free, h);
- if (IS_ENABLED(CONFIG_MULTITHREADING) && z_unpend_all(&h->wait_q) != 0) {
- z_reschedule(&h->lock, key);
+ SYS_PORT_TRACING_OBJ_FUNC(k_heap, free, heap);
+ if (IS_ENABLED(CONFIG_MULTITHREADING) && z_unpend_all(&heap->wait_q) != 0) {
+ z_reschedule(&heap->lock, key);
} else {
- k_spin_unlock(&h->lock, key);
+ k_spin_unlock(&heap->lock, key);
}
}
diff --git a/kernel/sched.c b/kernel/sched.c
index 9e50c18..898d8fa 100644
--- a/kernel/sched.c
+++ b/kernel/sched.c
@@ -452,9 +452,9 @@
return ret;
}
-static void slice_timeout(struct _timeout *t)
+static void slice_timeout(struct _timeout *timeout)
{
- int cpu = ARRAY_INDEX(slice_timeouts, t);
+ int cpu = ARRAY_INDEX(slice_timeouts, timeout);
slice_expired[cpu] = true;
@@ -1225,20 +1225,20 @@
return need_sched;
}
-void init_ready_q(struct _ready_q *rq)
+void init_ready_q(struct _ready_q *ready_q)
{
#if defined(CONFIG_SCHED_SCALABLE)
- rq->runq = (struct _priq_rb) {
+ ready_q->runq = (struct _priq_rb) {
.tree = {
.lessthan_fn = z_priq_rb_lessthan,
}
};
#elif defined(CONFIG_SCHED_MULTIQ)
for (int i = 0; i < ARRAY_SIZE(_kernel.ready_q.runq.queues); i++) {
- sys_dlist_init(&rq->runq.queues[i]);
+ sys_dlist_init(&ready_q->runq.queues[i]);
}
#else
- sys_dlist_init(&rq->runq);
+ sys_dlist_init(&ready_q->runq);
#endif
}