Changeset e0a4686 in mainline for uspace/lib/c/generic/fibril_synch.c

Timestamp:

2018-03-08T18:25:31Z (7 years ago)

Author:

Jiří Zárevúcky <zarevucky.jiri@…>

Branches:

lfn, master, serial, ticket/834-toolchain-update, topic/msim-upgrade, topic/simplify-dev-export

Children:

53ad43c, 615e83d

Parents:

fd57cf17

git-author:

Jiří Zárevúcky <zarevucky.jiri@…> (2018-03-06 22:03:11)

git-committer:

Jiří Zárevúcky <zarevucky.jiri@…> (2018-03-08 18:25:31)

Message:

Introduce context_swap() and use it to simplify fibril implementation.

This wraps the use of context_save()/context_restore(), to provide a more
natural (and easier to understand) control flow. The patch also hides the
details of the underlying context, requiring the use of context_get_pc()
to acquire program counter. This might allow context_swap() to only store
one pointer in the context, storing the remaining information in its stack
frame. However, that is not done in this patch.

File:

• uspace/lib/c/generic/fibril_synch.c (modified) (1 diff)

uspace/lib/c/generic/fibril_synch.c

@@ -73 +73 @@
                 if (oi->owned_by == f)
                         break;
-                stacktrace_print_fp_pc(context_get_fp(&oi->owned_by->ctx),
-                    oi->owned_by->ctx.pc);
+                stacktrace_print_fp_pc(
+                    context_get_fp(&oi->owned_by->ctx),
+                    context_get_pc(&oi->owned_by->ctx));
                 printf("Fibril %p waits for primitive %p.\n",
                      oi->owned_by, oi->owned_by->waits_for);