BCのコピーをFRAMに取ることで不揮発化する #249

chutaro · 2023-10-25T08:59:58Z

Issue

https://github.com/ut-issl/cdh-tasks/issues/223

詳細

詳しい実装方針は issue 参照

検証結果

ビルドチェック (どちらもチェック)

SILSでのビルドチェックに通った(CIで確認)
vMicroでのビルドチェックに通った

動作確認チェック (いずれかをチェック)

SILSでアルゴリズムが想定通りに動いた
実機でアルゴリズムが想定通りに動いた
(テレコマ試験の場合)コマンドファイルを使った試験をパスした

試験結果詳細記述場所 or 詳細ログ保存場所へのリンク

https://wiki.ut-issl.com/65adc1fc50576baaef92fb2b

200km · 2023-10-25T11:14:13Z

これは @conjikidow くん担当の issue #22 と同じものなように見えます。
仕事分担など、どんな状況か教えてもらえると嬉しいです。(ダブルエフォートになってしまっていないかなど)

chutaro · 2023-10-26T06:55:21Z

@conjikidow くんではなく自分が実装するという認識です。11月末まで日本を離れるので、実機検証はその後になると思います。

chutaro · 2023-12-17T16:07:32Z

遅くなりましたが、こちら一旦実装完了しました。ロジックの雰囲気や文法的なところなどレビューいただけるとありがたいです。

ちゃんと動作するかについては、実機でやってみないと分からない部分もあるので、今週のどこかで @conjikidow と実機検証したいと思います。

src/src_user/Applications/UserDefined/Cdh/non_volatile_bc_manager.c

src/src_user/Applications/UserDefined/Cdh/non_volatile_bc_manager.h

200km · 2023-12-18T19:19:40Z

src/src_user/Applications/UserDefined/Cdh/non_volatile_bc_manager.h

+
+AppInfo APP_NVBC_MGR_create_app(void);
+
+CCP_CmdRet Cmd_APP_NVBC_MGR_SET_ENABLE(const CommonCmdPacket* packet);


[NITS] コマンド関数は作られていますが、コマンドDBには追加されていないようです。

忘れていました。追加しました。

CDHコマンドの割り当て数が足りなくなったので、10から12に増やしました。その影響で、以降のコマンドIDがすべてずれています。

わかりました。12だとギリギリなのであれば、もう少し増やして今後のためのマージンを持たせても良いかなと思います。

15に増やしました

src/src_user/Applications/UserDefined/Cdh/non_volatile_bc_manager.c

200km · 2023-12-18T20:46:08Z

src/src_user/Applications/UserDefined/Cdh/non_volatile_bc_manager.c

+    bc_id = begin_bc_id + i;
+    nv_bc_manager_.is_ready_to_restore[bc_id] = 0;
+
+    // 有効化されている BC のみコピーする


[Q] 有効化されていなくてもコピーしても良い気もするのですが、どうでしょうか？

空き番のBCをコピーした場合、意味のないbyte列がFRAMにコピーされることになるが、それを間違えて復元して使おうとしたときに未定義の動作になってしまう可能性がある

無効化しているBCは理由があって無効化しているのであり、それを復元するのは、無効化している理由が引き継がれずそのまま使われる可能性があって危ない？

みたいなことを考えていました。実運用において、無効化したBCをコピーして復元したい場面があまり無いのであれば、このままで良いかなと思うのですが、どうでしょうか？

わかりました。運用していく上で混乱が生じないように仕様やルールが共有されていればどちらの仕組みでも問題無いとは思います。
他でコメントしたものと関連して、これまで通りコマンドでBC追加した時、コマンドでBCをdisable/enableした時に、どうするべきか、何を機にすべきかなど整理してもらえると嬉しいです。

コピーに関して

アプリ有効化中は、8サイクル周期（10個ずつコピーする場合）でBCがFRAMにコピーされる

地上局からBLコマンドでBCを編集した際も、0.8秒後にはFRAMにコピーが取られることになる

コマンドでBCを無効化すると、そのBCのready flagが下がるので、リセットしたときに復元できなくなる。再度BC有効化すると、復元できる

モード遷移やEHで使う重要なBCは通常すべて有効化されているはずなので、問題ないと思われる

復元に関して

AOBCがリセットしてしまったときは、デフォルトでアプリは無効化されている。復元しおえるまでは、アプリを有効化してはいけない

Cmd_APP_NVM_BC_RESTORE_BC_FROM_NVM で一つずつ復元できる

無効化していたBCは復元できない

逆に言うと、復元できたBCはちゃんとしたBCなはずなので、不安なく使えるはず

所望のBC（打ち上げ時から定義変更したモード遷移やEHなど）を復元しおえたら、アプリを再度有効化する

ざっとこんな感じだと思います。実機検証していろいろ分かってから、改めてソースコードやwikiに残したいと思います。

ありがとうございます。理解できました。
使い方をどこかに記載する際、次の点を考慮してもらえると嬉しいです。

コピー、復元はどっち方向か分からない時もあるので、NVMへのコピー、NVMからの復元など方向がわかるように書いてもらえると読みやすい

wikiに書く場合は誰でもwikiにアクセスできる仕組みが必要なので、公開方法も検討してほしい

.h の先頭にコメント整備しました。

200km · 2023-12-18T20:48:25Z

こちらの今回追加したことに関するwarningは対応してもらえると嬉しいです。

200km · 2023-12-18T20:57:31Z

@chutaro 実装ありがとうございます。コメントつけさせていただきました。参考にしてください。

src/src_user/Applications/UserDefined/Cdh/non_volatile_memory_bc_manager.c

chutaro · 2024-01-21T11:59:37Z

AOBC 単体SILSで検証できる部分は想定通り動作した。

chutaro · 2024-01-23T12:32:45Z

@200km @conjikidow 実機検証を行い、想定通りの動作を確認できました！検証メモのリンクは概要欄に貼っています。改めてざっと確認していただき、良さそうであればマージしてしまいたいです。

200km

小さなコメントだけなのでapproveしますが、その部分のコメントを修正してからマージしてください。

src/src_user/Settings/Modes/TaskLists/Elements/tl_elem_cdh_update.c

200km · 2024-01-23T13:29:06Z

@chutaro 動作確認ありがとうございます。レビューしました。

chutaro requested review from sksat and a team as code owners October 25, 2023 08:59

chutaro requested review from 200km, suzuki-toshihir0, t-hosonuma and conjikidow and removed request for a team October 25, 2023 08:59

chutaro self-assigned this Oct 25, 2023

chutaro marked this pull request as draft October 25, 2023 09:01

chutaro removed request for sksat, 200km, suzuki-toshihir0, t-hosonuma and conjikidow October 25, 2023 09:01

seki-hiro added ✈️ priority::medium priority medium 🐬 minor update Minor update labels Oct 25, 2023

200km linked an issue Nov 30, 2023 that may be closed by this pull request

BCT情報を不揮発メモリに移動させる #22

Closed

200km added this to the v8.1.0 Minor update milestone Nov 30, 2023

chutaro force-pushed the feature/non-volatile-bc branch from 13bd593 to 0e78d12 Compare December 15, 2023 09:12

chutaro changed the title ~~Draft: BCのコピーをFRAMに取ることで不揮発化する~~ BCのコピーをFRAMに取ることで不揮発化する Dec 17, 2023

chutaro marked this pull request as ready for review December 17, 2023 16:04

chutaro requested review from 200km and conjikidow December 17, 2023 16:05

200km reviewed Dec 18, 2023

View reviewed changes

200km reviewed Dec 19, 2023

View reviewed changes

src/src_user/Applications/UserDefined/Cdh/non_volatile_memory_bc_manager.c Outdated Show resolved Hide resolved

chutaro added 11 commits January 21, 2024 17:40

err enum を統合

fdb9bba

EL ID を修正

247690b

validationまわりを修正

44fe1c3

file 名を変更

8204ff9

prefixを一斉置換

b51f77e

cmd db に追加

7034274

変数定義の位置を修正

060d592

テレメにdesctiptionを追記

30d08e9

bc_num_to_copy について注意書きを追加

1f724f3

EL_GROUP のenum名を修正

4df37c2

app id の enum 名を修正

8399056

chutaro force-pushed the feature/non-volatile-bc branch from 7266800 to 8399056 Compare January 21, 2024 08:44

テスト用のコマンドファイルを作成

92d8723

chutaro added 9 commits January 21, 2024 21:01

SILS検証を反映

be28abc

vcxproj を修正

15590f5

コマンドファイルを修正

6bf7452

BCの中身のコマンド1つずつ分けてコピーするようにした

9941bb4

BCT_register_cmd を使うように改修

3134103

small fix

8db4ac1

CDH用コマンドを15に増やした

025455c

アドレスを修正

5645775

コメントを微修正

a4876ad

200km approved these changes Jan 23, 2024

View reviewed changes

src/src_user/Settings/Modes/TaskLists/Elements/tl_elem_cdh_update.c Outdated Show resolved Hide resolved

chutaro added 2 commits January 23, 2024 23:22

使い方や注意点を .h のコメントに書いた

06aa994

不要なコメントを消去

e767967

chutaro merged commit f06eeb8 into develop Jan 23, 2024
9 of 11 checks passed

chutaro deleted the feature/non-volatile-bc branch January 23, 2024 14:34

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

BCのコピーをFRAMに取ることで不揮発化する #249

BCのコピーをFRAMに取ることで不揮発化する #249

chutaro commented Oct 25, 2023 •

edited

Loading

200km commented Oct 25, 2023 •

edited

Loading

chutaro commented Oct 26, 2023

chutaro commented Dec 17, 2023

200km Dec 18, 2023

chutaro Dec 20, 2023

chutaro Dec 20, 2023

200km Dec 20, 2023

chutaro Jan 22, 2024

200km Dec 18, 2023

chutaro Dec 19, 2023

200km Dec 19, 2023

chutaro Dec 20, 2023

chutaro Dec 20, 2023

200km Dec 20, 2023

chutaro Jan 23, 2024

200km commented Dec 18, 2023

200km commented Dec 18, 2023

chutaro commented Jan 21, 2024

chutaro commented Jan 23, 2024

200km left a comment

200km commented Jan 23, 2024


		AppInfo APP_NVBC_MGR_create_app(void);

		CCP_CmdRet Cmd_APP_NVBC_MGR_SET_ENABLE(const CommonCmdPacket* packet);

BCのコピーをFRAMに取ることで不揮発化する #249

BCのコピーをFRAMに取ることで不揮発化する #249

Conversation

chutaro commented Oct 25, 2023 • edited Loading

Issue

詳細

検証結果

ビルドチェック (どちらもチェック)

動作確認チェック (いずれかをチェック)

試験結果詳細記述場所 or 詳細ログ保存場所へのリンク

200km commented Oct 25, 2023 • edited Loading

chutaro commented Oct 26, 2023

chutaro commented Dec 17, 2023

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

200km commented Dec 18, 2023

200km commented Dec 18, 2023

chutaro commented Jan 21, 2024

chutaro commented Jan 23, 2024

200km left a comment

Choose a reason for hiding this comment

200km commented Jan 23, 2024

chutaro commented Oct 25, 2023 •

edited

Loading

200km commented Oct 25, 2023 •

edited

Loading