Changeset e0a4686 in mainline for uspace/app/taskdump/fibrildump.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/app/taskdump/fibrildump.c (modified) (1 diff)

uspace/app/taskdump/fibrildump.c

@@ -110 +110 @@
                         return EIO;
 
-                pc = fibril.ctx.pc;
+                pc = context_get_pc(&fibril.ctx);
                 fp = context_get_fp(&fibril.ctx);
                 if (0) stacktrace_print_generic(&fibrildump_st_ops, sess,