-
Notifications
You must be signed in to change notification settings - Fork 48
updated description of UPV components in tristan-isolde-unified-access-page #88
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -1027,7 +1027,7 @@ <h3 id="2.2"> | |
| Wormhole NoC | ||
| </td> | ||
| <td> | ||
| Not ready yet | ||
| <a href="https://gitlab.com/selene-riscv-platform/euros2pronoc/-/tree/AXI-MM-Network-Interface?ref_type=heads">NoC</a> | ||
| </td> | ||
| <td> | ||
| ISOLDE | ||
|
|
@@ -1039,10 +1039,10 @@ <h3 id="2.2"> | |
| UPV | ||
| </td> | ||
| <td> | ||
| | ||
| Work in progress | ||
| </td> | ||
| <td> | ||
| TBD | ||
| In this architecture, all the communication among tiles is carried out through the network. A tile is an abstraction entity that encapsulates one or more IPs of a SoC and provides an AXI Memory-Mapped interface to them. The main components of every tile are the on-chip network switch, the Network Interface (NI), and a placeholder (a.k.a. UNIT), which is used to implement the desired Intellectual Property (IP) or Processing Element (PE). | ||
| </td> | ||
| <td> | ||
| MIT | ||
|
|
@@ -1056,7 +1056,7 @@ <h3 id="2.2"> | |
| AXI Sniffer | ||
| </td> | ||
| <td> | ||
| Not ready yet | ||
| <a href="https://gitlab.com/selene-riscv-platform/selene-hardware/-/tree/master/interconnect/libnoc/mem_monitor">AXI Sniffer</a> | ||
| </td> | ||
| <td> | ||
| ISOLDE | ||
|
|
@@ -1068,10 +1068,10 @@ <h3 id="2.2"> | |
| UPV | ||
| </td> | ||
| <td> | ||
| | ||
| Completed | ||
| </td> | ||
| <td> | ||
| TBD | ||
| This module monitors axi4 transactions and determines contention between the different initiators of transactions. The initiators of transactions are given by a world ID encoded in the QoS field of the AXI channels. Supports following contention tracking features: read contention monitoring backpressure channels, read contention blaming the head of the queue for all contention caused in the queue, cross read-write backpressure, write backpressure monitoring, cross write-read backpressure. | ||
|
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I would change "axi4"to "AXI4", but that is ok.
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. uppercase notation "AXI4" is better |
||
| </td> | ||
| <td> | ||
| MIT | ||
|
|
@@ -3916,7 +3916,7 @@ <h3 id="4.3"> | |
| Bit-accurate FPGA fault injection (BAFFI) tool | ||
| </td> | ||
| <td> | ||
| | ||
| <a href="https://gitlab.com/selene-riscv-platform/DAVOS">BAFFI</a> | ||
|
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Here I would either the name or label of the URL to "DAVOS", or change the URL to "https://gitlab.com/selene-riscv-platform/DAVOS#1-bit-accurate-fpga-fault-injection-tool-baffi", as I understand BAFFI is one of the features of the DAVOS project
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I would keep the label "BAFFI", and change the URL to "https://gitlab.com/selene-riscv-platform/DAVOS#1-bit-accurate-fpga-fault-injection-tool-baffi". |
||
| </td> | ||
| <td> | ||
| ISOLDE | ||
|
|
@@ -3928,10 +3928,10 @@ <h3 id="4.3"> | |
| UPV | ||
| </td> | ||
| <td> | ||
| | ||
| Work in progress | ||
| </td> | ||
| <td> | ||
| TBD | ||
| BAFFI supports bit-accurate fault injection into Ultrascale+ and 7-series AMD FPGAs. Supports transient fault models representative for FPGA and ASIC designs. Adapted to the SELENE (NOEL-V) SoC in VCU118 evaluation board. | ||
| </td> | ||
| <td> | ||
| MIT | ||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For a name or label of the URL "NoC" is too generic, I would simply keep Wormhole NoC or change it to "euros2pronoc" or "osNoC: Open-Source Network-on-Chip" as indicated when you visit the repo
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I agree, we can keep "Wormhole NoC".