DS での rx_buffer を stream ごとに driver instance から設定可能にする

[meltingrabbit]

概要

DS での rx_buffer を stream ごとに driver instance から設定可能にする

Issue

• DriverSuper のバッファサイズを可変にする #443
• DriverSuperの再設計と再実装 #154
• v3.8.0 リリース準備 #468

詳細

Driverが持っていた DS の rx buffer を DI で生成し，Driver に渡すようにした．
これによって，同一 Driver で複数の DI を持つコンポに対応

検証結果

既存のテストがすべて通る

影響範囲

DI，Driverの初期化がすべて変わる

備考

• general な OK, NG enum の追加（Rust でいう Result） #504 を先にマージする

[meltingrabbit]

@chutaro @ToshiAki64 レビューお願いします．
これで #443 に続く改修がようやく落ち着きます．
v3.8.0 で，Cmd返り値変更に並ぶ重大な update になりそうです．

この PR のあと，細かい DS 周りの修正を予定しています．

[chutaro]

reviewというより、DSに対する質問いくつか書いていってもいいですか？

• stream 0 しか使わない大半のコンポでも、Driver構造体を確保した時点で DS_StreamConfig 構造体は３つ確保されてしまって、2つは使わないからメモリの無駄になってしまうと思うんですが、DriverSuper構造体のメンバーとして DS_StreamConfig もポインターで保持して、使う分だけ DI で確保する、という方針にはならなかったのでしょうか？
• 一方で、DS_StreamConfig 構造体の中で一番メモリを使うであろう DS_StreamRecBuffer については、ポインターで確保していて、stream 0 しか使わない場合は 1,2 は NULL埋めするので、まあ問題ない、という感じですかね？
• tx_frame_ は Driverで確保して、 rx_buffer は DI で確保するのは何故なんでしたっけ？
• これはただのコメントなのですが、rx_bufferを可変にする改修は既に終わってるんですね（あまり意識してませんでしたありがとうございます）。今回はその確保場所を DI に移して色々整理した、という感じですよね？

[meltingrabbit]

Driver構造体を確保した時点で DS_StreamConfig 構造体は３つ確保されてしまって、2つは使わないからメモリの無駄になってしまうと思うんですが、DriverSuper構造体のメンバーとして DS_StreamConfig もポインターで保持して、使う分だけ DI で確保する、という方針にはならなかったのでしょうか？

これが今までメモリ使用量が多いい問題の元凶だったので，一番メモリを使っているバッファ（= stream buffer）だけ，使うstreamごとに外部からわたせるように #503 にした感じです．現在の DS_StreamConfig 構造体は軽いので，許容かなという（C でかつ，malloc 禁止な C2A だと，多分これがメモリと安全性のちょうどいいバランスだと思ってます）

一方で、DS_StreamConfig 構造体の中で一番メモリを使うであろう DS_StreamRecBuffer については、ポインターで確保していて、stream 0 しか使わない場合は 1,2 は NULL埋めするので、まあ問題ない、という感じですかね？

そそ

[meltingrabbit]

tx_frame_ は Driverで確保して、 rx_buffer は DI で確保するのは何故なんでしたっけ？

• tx_frame は送信側なのでframe解析がない
• 送信するときに driver 側で作って，送信後は不要になるので，他の DI で共通化可能（DI ごとにつくるとむしろ無駄）
  • rx は途中まで受信してるときなど，次回の rec 呼び出しに繰り越さないと行けないので，共用不能
• 以下の理由で tx は rx よりそもそもシンプルでこれで十分
  • 受信は，rec呼び出し頻度や物理バッファによって，一度に確保すべきメモリが受信frameよりも大きく，きちんと確保することが大事
  • 送信は，frame 解析が不要で，普通に送るバイナリ列を上から作ってくだけの処理

って感じかな．
あと，tx はコマンドで， rx はテレメであり， tx の frame 長 は一般的に小さい，ってのもある．

[meltingrabbit]

これはただのコメントなのですが、rx_bufferを可変にする改修は既に終わってるんですね（あまり意識してませんでしたありがとうございます）。今回はその確保場所を DI に移して色々整理した、という感じですよね？

そそ．実はね．
ここらへんは @yngyu くんレビューしてくれてた．

• DriverSuperの再設計と再実装 #154

このあたり

[chutaro]

なるほどです。全部納得しましたありがとうございます。

[chutaro]

良さそうです。いくつか些細なコメントいれています

[chutaro]

おつかれさまです！approveしました