ignition rx running disparity update is sub-optimal
まだ誰も着手していません。
評価
- 難易度
- 4/5
- 見積もり時間
- 3〜5日
- 初心者へのやさしさ
- 35/100
- issue の種類
- バグ
- 明瞭さ
- おおむね明確
- 活発さ
- 停滞
調査の方向性
hdl/ip/bsv/ignition/IgnitionReceiver.bsv の239行目および344-352行目付近を読み、次にreceiverが8b10b decoderからrunning disparityをどのように取得し、IDLE1とIDLE2をどのように処理するかを確認してください。disparity trackingとpolarity inversionの動作を検証するため、issueで要求されているsimulationを実行するか追加してください。完了の条件は、receiverがrunning disparityについてtransmitterが選択したIDLEパターンに依存しなくなることです。
索引モデルが issue の本文から書いたものです。
説明
I believe this to be a fundamentally incorrect way of determining a receiver's running disparity as it relies on the transmitter transmitting specific patterns on specific disparities, which isn't something you'd typically see in a link. Normally, you'd take the output of your actual 8b10b decoder. It does "work" in our case here because the transmitter is doing extra work to pop between IDLE1 and IDLE2 sequences based on it's running disparity. I believe this was done to facilitate polarity inversion detection but it isn't required and while flipping between IDLE1 and IDLE2 sequences is fine, it really should have no bearing on the rx side's running disparity.
I need to get in and do some actual simulation on this, but all links should generally capture the current running disparity from the 8b10b decoder but i think the resetting of the rd on IDLE sequences can and should just be removed, leaving this state to keep the running disp up-to-date:
^ The above provides a self-correcting mechanism. Upon link start up you assume some running disparity (say negative) and if you're wrong you get a disparity error (don't care b/c you had a 50-50 chance of getting it right) but you always get your K28.5 comma regardless of polarity inversion. After the first byte, you have a correct running disparity from the decoder, and the next byte can be decoded. Note that you'll never get the wrong decoded byte just due to disparity mismatch, all the byte patterns are uniquely coded.
Longer-term, I think while we can accept IDLE1 or IDLE2 sequences and may continue bouncing between them there's no reason to tie them into a specific running disparity. And regardless of which IDLE sequence you get, the bit-inverse of it is always detectable for polarity inversion, regardless of the running disparity.
- 主要言語
- VHDL
- スター
- 24
- フォーク
- 2
- 平均マージ
- 31分
- マージ済み PR（30日）
- 8
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
oxidecomputer/quartz のほかの issue
-
難易度 4/5 3〜5日 初心者へのやさしさ 35/100
oxidecomputer/quartz#542 ·
-
難易度 5/5 1週間以上 初心者へのやさしさ 25/100
oxidecomputer/quartz#535 ·
-
難易度 3/5 1〜2日 初心者へのやさしさ 45/100
oxidecomputer/quartz#534 ·
-
難易度 5/5 1週間以上 初心者へのやさしさ 20/100
oxidecomputer/quartz#533 ·
-
FPGA-FPGA link再び着手できるかも @nathanaelhuffman が 49 日前に担当しましたが、オープン中のプルリクエストはありません。 オープン
oxidecomputer/quartz#532 · 担当者 1 名 ·
oxidecomputer/quartz の issue をすべて見る
似ている issue
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
prestonbrown/helixscreen#2023 ·
メンテナーはふだん 2 日以内に返信
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
jniebuhr/gaggimate#1009 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 77/100
メンテナーはふだん 1 日以内に返信
-
難易度 1/5 1時間未満 初心者へのやさしさ 64/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
arendst/Tasmota#25232 · コメント 3 件 ·
メンテナーはふだん 1 日以内に返信