ciniml / rust-dap

CMSIS-DAP Rust implementation
Apache License 2.0
88 stars 10 forks source link

`defmt` をデフォルトにする #58

Open ciniml opened 1 year ago

ciniml commented 1 year ago
          `defmt` をデフォルトにしといてもいいんじゃないかと思いました。

_Originally posted by @ciniml in https://github.com/ciniml/rust-dap/pull/56#discussion_r1399547978_

tnishinaga commented 1 year ago

コードサイズ増えたりメモリレイアウト変わっちゃうので、RAM上にコードを収めるとかの障害になるかなと思ってましたが、どうなんでしょう?

ciniml commented 1 year ago

あー、なるほど。RAMに入らなくなる…か?

tnishinaga commented 1 year ago

(試してないのに言うのはあまり良くないと思いつつ)defmt-rtt使うとRAM上にログを書き出すので、少なくともその分のRAM使用量は増えちゃうはずです。

elfmimi commented 1 year ago

だいぶ前になってしまいますが手元では ram-exec と defmt を合わせて使ってました。 たしか probe-run も改造しないと使いづらいか、使えないかという部分があったはず。

ciniml commented 1 year ago

なるほど。ちなみにprobe-runはarchiveされてprobe-rs使えになりましたが、そっちだと問題なかったりします?

elfmimi commented 1 year ago

興味あるので調べたい所なんですがまだ手を付けてないです。

ciniml commented 1 year ago

了解です。まあ急ぎじゃないのでどこかで試してみます。