feat(perf): features for evaluation#678

Merged

nokosaaan merged 83 commits into

mainfrom

Jul 24, 2026

nokosaaan commented Apr 15, 2026 •

edited

Loading

Contributor

Description

Since period information is required during evaluation, I have provided a feature to enable evaluation outside of the application.
period-index-propagation is normally turned off, which is consistent with the behavior of the traditional awkernel. To enable it, simply add period-index-propagation to the default in the [features] section of kernel/Cargo.toml.

I have added perf as a feature for rv32 and rv64, but this is a temporary measure to pass tests on GitHub and is not intended to permanently add perf functionality to these targets.
The root cause of this issue has not yet been resolved and will be tracked in a separate issue.

I have updated pubsub.rs to allow cycle propagation without altering existing trait boundaries, APIs, or the test application. Please refer to the file attached to the PR test section for the execution results.

Related links

How was this PR tested?

no_feature_compile.log
no_feature_execute.log
feature_compile.log
feature_execute.log

Notes for reviewers

nokosaaan added 3 commits

April 15, 2026 18:20


          fix: add need-get-period feature

3aca1f9

Signed-off-by: nokosaaan <nishimura.r.019@ms.saitama-u.ac.jp>


          fix: delete the portion of impl about macro

87742e5

Signed-off-by: nokosaaan <nishimura.r.019@ms.saitama-u.ac.jp>


          fix: delete unnecessary imports

5d30fc2

Signed-off-by: nokosaaan <nishimura.r.019@ms.saitama-u.ac.jp>

github-actions Bot assigned nokosaaan

nokosaaan added 2 commits

April 17, 2026 12:58


          Merge branch 'main' of https://github.com/tier4/awkernel into eva-fea…

a1f0619

…ture


          fix: attach cargo fmt 1

59b5354

Signed-off-by: nokosaaan <nishimura.r.019@ms.saitama-u.ac.jp>

nokosaaan changed the title ~~add: Features for evaluation~~ add: features for evaluation

nokosaaan added 5 commits

April 17, 2026 16:37


          fix: delete if-else diversion and focus feature diversion

b77385e

Signed-off-by: nokosaaan <nishimura.r.019@ms.saitama-u.ac.jp>


          fix: apply cargo fmt 2

85d9042

Signed-off-by: nokosaaan <nishimura.r.019@ms.saitama-u.ac.jp>


          fix: experimental apply for perf

c88633a

Signed-off-by: nokosaaan <nishimura.r.019@ms.saitama-u.ac.jp>


          fix: experimental apply for perf 2

Signed-off-by: nokosaaan <nishimura.r.019@ms.saitama-u.ac.jp>


          fix: delete .clone() which has Copy trait

8e03ef4

Signed-off-by: nokosaaan <nishimura.r.019@ms.saitama-u.ac.jp>

nokosaaan changed the title ~~add: features for evaluation~~ fix(perf): features for evaluation

nokosaaan added 11 commits

April 17, 2026 17:16


          fix: unused imports and unnecessary casting

a728ff8

Signed-off-by: nokosaaan <nishimura.r.019@ms.saitama-u.ac.jp>


          fix: apply cargo fmt 3

6585af5

Signed-off-by: nokosaaan <nishimura.r.019@ms.saitama-u.ac.jp>


          fix: delete unnecessary import

bc19c69

Signed-off-by: nokosaaan <nishimura.r.019@ms.saitama-u.ac.jp>


          fix: delete unnecessary functions and modify kernel/src/main.rs

adfa4fd

Signed-off-by: nokosaaan <nishimura.r.019@ms.saitama-u.ac.jp>


          fix: add get_period_count for getting periodic period

377d1b1

Signed-off-by: nokosaaan <nishimura.r.019@ms.saitama-u.ac.jp>


          fix: async_lib/Cargo.toml

ddd69c2

Signed-off-by: nokosaaan <nishimura.r.019@ms.saitama-u.ac.jp>


          fix: apply cargo fmt 4

37cae77

Signed-off-by: nokosaaan <nishimura.r.019@ms.saitama-u.ac.jp>


          fix: delete NodeRecord

3ff0804

Signed-off-by: nokosaaan <nishimura.r.019@ms.saitama-u.ac.jp>


          fix: unify start publish_timestamp_at to pubsub.rs

d8e28cd

Signed-off-by: nokosaaan <nishimura.r.019@ms.saitama-u.ac.jp>


          fix: delete unused imports

2de13a9

Signed-off-by: nokosaaan <nishimura.r.019@ms.saitama-u.ac.jp>


          fix: add dependics in kernel/Cargo.toml for print_timestamp_table

9a9d2da

Signed-off-by: nokosaaan <nishimura.r.019@ms.saitama-u.ac.jp>

nokosaaan requested a review from Copilot

April 17, 2026 11:02

Copilot started reviewing on behalf of nokosaaan

April 17, 2026 11:03

View session

This comment was marked as outdated.

Sign in to view

nokosaaan added 3 commits

April 17, 2026 21:31


          fix: clarify the responsibility between send_all and send_all_with_me…

14f92ca

…ta with features

Signed-off-by: nokosaaan <nishimura.r.019@ms.saitama-u.ac.jp>


          fix: delete get_period and

968c058

the logic for passing periodic information has been internalized and solve bootloader error changing const numbers by using Box

Signed-off-by: nokosaaan <nishimura.r.019@ms.saitama-u.ac.jp>


          fix: apply cargo fmt 5

efc54b6

Signed-off-by: nokosaaan <nishimura.r.019@ms.saitama-u.ac.jp>

nokosaaan added 8 commits

May 20, 2026 23:38


          fix: add comment for test code

4da1730

Signed-off-by: nokosaaan <nishimura.r.019@ms.saitama-u.ac.jp>


          fix: assert to log::warn!

19874cd

Signed-off-by: nokosaaan <nishimura.r.019@ms.saitama-u.ac.jp>


          fix: delete if-else handling with dag_info in start_interrupt,start_c…

2824a24

…ontext_switch etc..

Signed-off-by: nokosaaan <nishimura.r.019@ms.saitama-u.ac.jp>


          fix: update_time_and_state_for_dag

Signed-off-by: nokosaaan <nishimura.r.019@ms.saitama-u.ac.jp>


          fix: delete unused import

Signed-off-by: nokosaaan <nishimura.r.019@ms.saitama-u.ac.jp>


          fix: add comment on MAX_LOGS, MAX_PUBSUB, MAX_NODES

bbc863e

Signed-off-by: nokosaaan <nishimura.r.019@ms.saitama-u.ac.jp>


          fix: change value of MAX_NODES

b89e91d

Signed-off-by: nokosaaan <nishimura.r.019@ms.saitama-u.ac.jp>


          fix: assert to log::warn!

2845e6a

Signed-off-by: nokosaaan <nishimura.r.019@ms.saitama-u.ac.jp>

kobayu858 requested changes

View reviewed changes

awkernel_async_lib/src/task.rs Outdated

awkernel_async_lib/src/pubsub.rs

kernel/src/main.rs Outdated

awkernel_async_lib/src/task.rs

nokosaaan added 9 commits

May 22, 2026 11:55


          fix: add comment on MAX_LOGS

c5ea68f

Signed-off-by: nokosaaan <nishimura.r.019@ms.saitama-u.ac.jp>


          fix: change logic of logging recorder

Signed-off-by: nokosaaan <nishimura.r.019@ms.saitama-u.ac.jp>


          fix: add comment to recv_all_with_period_index

3277d25

Signed-off-by: nokosaaan <nishimura.r.019@ms.saitama-u.ac.jp>


          fix: add magic number to valiable

27699d7

Signed-off-by: nokosaaan <nishimura.r.019@ms.saitama-u.ac.jp>


          fix: logging way following ci build test

6ab7887

Signed-off-by: nokosaaan <nishimura.r.019@ms.saitama-u.ac.jp>


          fix: add issue link

4e0ff80

Signed-off-by: nokosaaan <nishimura.r.019@ms.saitama-u.ac.jp>


          Merge branch 'main' of https://github.com/tier4/awkernel into eva-fea…

533d1a3

…ture


          fix: delete todo comment

Signed-off-by: nokosaaan <nishimura.r.019@ms.saitama-u.ac.jp>


          Merge branch 'main' of https://github.com/tier4/awkernel into eva-fea…

54a2601

…ture

kobayu858 approved these changes

View reviewed changes

nokosaaan force-pushed the eva-feature branch from a69a9d6 to 54a2601 Compare

July 15, 2026 03:13

nokosaaan added 7 commits

July 15, 2026 12:15


          Merge branch 'main' of https://github.com/tier4/awkernel into eva-fea…

b22347a

…ture


          fix: cotext_switch(_main) position

ad10def

Signed-off-by: nokosaaan <nishimura.r.019@ms.saitama-u.ac.jp>


          Merge branch 'main' of https://github.com/tier4/awkernel into eva-fea…

3e75e1e

…ture


          fix: rename ctx to context

cdca0d1

Signed-off-by: nokosaaan <nishimura.r.019@ms.saitama-u.ac.jp>


          Merge branch 'main' of https://github.com/tier4/awkernel into eva-fea…

34adf4a

…ture


          fix: Adjusting the starting point of in

de7c27f

Signed-off-by: nokosaaan <nishimura.r.019@ms.saitama-u.ac.jp>


          fix: add description comment for start_context_switch.

Signed-off-by: nokosaaan <nishimura.r.019@ms.saitama-u.ac.jp>

nokosaaan commented Jul 24, 2026 •

edited

Loading

Contributor Author

@kobayu858
I fixed the starting point of context_switch in perf at commit de7c27f.
Since placing it in /awkernel_lib/src/interrupt.rs would cause a circular dependency with the perf module, I moved it to the beginning of preemption(), which is called immediately afterward.

Additionally, the location where I had initially placed it—as discussed earlier—was within an architecture-dependent processing flow, which could prevent the process from running on architectures other than x86; therefore, I determined that the current location is appropriate.

As before, there are discrepancies in processing times between context switches via preemption and those via run_main—due to factors like early returns—which can result in variations in measurement times.
However, I believe this issue can be resolved by comparing the two.

In this PR, I have left a comment explaining why I placed the code here, and our plan is to make adjustments as needed if a more optimal location is found in the future.

nokosaaan merged commit 6c7a629 into main

1 check passed

nokosaaan deleted the eva-feature branch

July 24, 2026 09:12

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet