Extend support for FlooDMA traffic generation by gbellocchi · Pull Request #182 · pulp-platform/FlooNoC

gbellocchi · 2026-05-12T10:00:06Z

SUMMARY OF CONTRIBUTIONS

Add FlooDMA traffic generation from YAML configuration files
I have extended the FlooDMA job generator (gen_jobs.py) to support FlooDMA job generation from traffic configuration files (YAML).
This feature simplifies the generation of custom traffic patterns, reusing FlooGen to retrieve the target FlooNoC topology information.

Below are the main contributions:

Parse the NoC configuration file (typically under: floogen/examples/) using FlooGen and produce a NoC model;
Parse the traffic configuration file (typically under: hw/test/traffic_cfg/);
Use the NoC model to resolve initiator and endpoint addresses for each traffic flow;
Create a traffic model;
Use the traffic model to generate DMA jobs;
Extend miscellaneous components (Makefile targets, etc.).

Then, I have verified the new feature using the supported testbenches, making use of FlooDMA:

Examples of traffic configuration files (traffic_cfg) have been added under hw/test/traffic_cfg;
Launch tests using tb_floo_axi_mesh.sv and tb_floo_nw_mesh.sv;
Use axi_bw_monitor to compare traffic generated directly in Python against traffic generated using traffic_cfg.

Extend the FlooDMA control plane
I have modified the control plane of floo_dma_test_node to launch DMA jobs after triggering a start signal.
This feature enables running FlooDMA multiple times during one simulation run, which might prove useful during design space exploration.

Below are the main contributions:

Add start_of_sim_i signal to the floo_dma_test_node interface;
Extend the control plane of the floo_dma_test_node;
Extend the control plane of tb_floo_axi_mesh.sv and tb_floo_nw_mesh.sv;
Verify functionality in RTL simulation.

* Floogen is used to populate the traffic model with information concerning the FlooNoC instance.

* The class content match the traffic cfg information, similarly to what floogen also does.

The generator currently loads and compiles floonoc and traffic models.

* Add start, done, and ready control signals to control the execution of floo dma. * Loop execution of floo dma based on control interface. * Access queues with array indexing to preserve dma jobs between different execution runs.

* Handle execution corner case where floo dma is activated without jobs to execute.

* hw: Add floogen makefile recipe. * hw: Add `nw_mesh` to bender compilation. * util: Add makefile arguments for traffic generation. * util: Change default `traffic_type` to `dma_mesh` as `random` type is not supported.

I have removed the ready signal as it was redundant and was creating problems if initialized to 1'b1 at reset-time, which would however make sense from a logical pov. Thus, end_of_sim_o is directly used in place of ready.

* Update interfaces of floo_dma_test_node and axi_bw_monitor. * Add procedural start-wait controls. * Tested in rtl simulation.

Clean the tb procedural block.

* Add support in Makefile. * Add two examples of `traffic_cfg` files for xy-based mesh testbenches (narrow, narrow-wide).

The floonoc protocol type is used to populate the traffic model and distinguish between narrow, wide, or other floonoc channel types. If this is not specified in the floogen configuration, then the job generator will assert warnings.

fischeti

Thanks for the PR @gbellocchi! I finally got some time to review it🤓 I was thinking whether it makes sense to directly integrate the gen_jobs.py script into floogen. Because some of the things are actually duplicate now.

Also, it would be nice to add some examples that can be used as reference.

There are also some trivial linting issues that need to be fixed.

fischeti · 2026-05-12T18:10:49Z


 protocols:
  - name: "axi_in"
+    type: "narrow"


is this necessary for the axi network type?

fischeti · 2026-05-12T18:14:58Z

+  always_ff @(posedge clk_i or negedge rst_ni) begin : proc_start
+    if(!rst_ni) begin
+      start <= 1'b0;
+    end else begin
+      start <= start_of_sim_i;
+    end
+  end
+
+  always_ff @(posedge clk_i or negedge rst_ni) begin : proc_ready
+    if(!rst_ni) begin
+      end_of_sim_o <= 1'b0;
+    end else if(done == 1'b1) begin // job completion, floo dma is ready to accept new jobs
+      end_of_sim_o <= 1'b1;
+    end else if(start == 1'b1) begin // start, with jobs to run
+      end_of_sim_o <= 1'b0;
+    end
+  end
+


could use FF macros from common_cells/registers.svh if you want

fischeti · 2026-05-12T18:21:18Z

+try:
+    from floogen.config_parser import parse_config
+    from floogen.model.network import Network
+    FLOOGEN_AVAILABLE = True
+except ImportError:
+    FLOOGEN_AVAILABLE = False


In theory, you could also integrate this directly into floogen as a new target. And then call it something like this:

floogen traffic -c config.yml --traffic traffic.yml

What do you think? Then we could get rid of this separate script

fischeti · 2026-06-01T07:43:36Z

        job_file.close()

-
 def gen_chimney2chimney_traffic(


I think gen_[nw]_chimney2chimney_traffic can ge removed entirely. This is legacy stuff that is not used anymore.

fischeti · 2026-06-01T07:44:39Z

    if args.tb == "chimney2chimney":
        gen_chimney2chimney_traffic(**kwargs)
    elif args.tb == "nw_chimney2chimney":
        gen_nw_chimney2chimney_traffic(**kwargs)


Can be removed, see above

fischeti · 2026-06-01T07:46:23Z

 clean: clean-vsim clean-spyglass clean-jobs clean-sources clean-vcs
 build: compile-vsim
-run: run-vsim
+run: run-vsim


nit: missing newline

fischeti · 2026-06-01T07:48:36Z

+  always_ff @(posedge clk_i or negedge rst_ni) begin : proc_ready
+    if(!rst_ni) begin
+      end_of_sim_o <= 1'b0;
+    end else if(done == 1'b1) begin // job completion, floo dma is ready to accept new jobs


Suggested change

end else if(done == 1'b1) begin // job completion, floo dma is ready to accept new jobs

end else if(done) begin // job completion, floo dma is ready to accept new jobs

fischeti · 2026-06-01T07:48:48Z

+      end_of_sim_o <= 1'b0;
+    end else if(done == 1'b1) begin // job completion, floo dma is ready to accept new jobs
+      end_of_sim_o <= 1'b1;
+    end else if(start == 1'b1) begin // start, with jobs to run


Suggested change

end else if(start == 1'b1) begin // start, with jobs to run

end else if(start) begin // start, with jobs to run

Gianluca Bellocchi added 30 commits May 7, 2026 15:38

util: Import packages to create traffic model

653569c

* Floogen is used to populate the traffic model with information concerning the FlooNoC instance.

util: Add traffic and floonoc cfg to parser arguments

6979f14

util: Add generator call case when importing traffic cfg

841a57d

util: Define traffic classes

1211f4c

* The class content match the traffic cfg information, similarly to what floogen also does.

util: Compile floonoc model using floogen

386f395

util: Parse traffic configuration

4b36a6a

util: Create traffic model based on traffic cfg and class

9a0957f

util: Resolve initiator and endpoint addresses for each traffic flow

a538518

util: Add print summary for the traffic model

00b3be2

util: Add dma job generator from traffic and floonoc cfg

8f96864

The generator currently loads and compiles floonoc and traffic models.

util: Generate dma traffic jobs from custom traffic cfg

a074e50

util: Move rw stream type into traffic cfg

55f68ba

util: Add burst data width from floonoc cfg

5c0ca5d

util: Get tile xy coordinates to compute job index

ace0c06

sim: Add conditional execution of floo dma debug prints

34914dc

sim: Handle corner case with start without jobs

8d559df

* Handle execution corner case where floo dma is activated without jobs to execute.

util: Add xy notation and improve code readability

34fff15

util: Add xy-to-sam conversion using floonoc model

434751b

Add python var in makefile

1f6743f

Add makefile support for traffic generation

cb95d74

* hw: Add floogen makefile recipe. * hw: Add `nw_mesh` to bender compilation. * util: Add makefile arguments for traffic generation. * util: Change default `traffic_type` to `dma_mesh` as `random` type is not supported.

sim: Fix job completion notificatio to tb

cf6f05b

I have removed the ready signal as it was redundant and was creating problems if initialized to 1'b1 at reset-time, which would however make sense from a logical pov. Thus, end_of_sim_o is directly used in place of ready.

sim: Update mesh tb with new version of floo_dma_test_node

85b616b

* Update interfaces of floo_dma_test_node and axi_bw_monitor. * Add procedural start-wait controls. * Tested in rtl simulation.

sim: Clean noc mesh tb

5ddafbd

Clean the tb procedural block.

util: Parametrize job file names using traffic_name

0d688e3

util: Use one variable to specify the job output directory

9ab0773

util: Fix hbm channel base address generation

a649889

util: Add examples of traffic configuration files

9dad3cd

* Add support in Makefile. * Add two examples of `traffic_cfg` files for xy-based mesh testbenches (narrow, narrow-wide).

sim: Match narrow job indices in tb_floo_axi_mesh

5e3e384

gbellocchi requested review from Lore0599 and fischeti May 12, 2026 10:00

gbellocchi self-assigned this May 12, 2026

fischeti reviewed Jun 1, 2026

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Extend support for FlooDMA traffic generation#182

Extend support for FlooDMA traffic generation#182
gbellocchi wants to merge 30 commits into
mainfrom
gb/gen-jobs-from-cfg-v0.8.1

gbellocchi commented May 12, 2026

Uh oh!

fischeti left a comment •

edited

Loading

Uh oh!

fischeti May 12, 2026

Uh oh!

fischeti May 12, 2026

Uh oh!

fischeti May 12, 2026

Uh oh!

fischeti Jun 1, 2026

Uh oh!

fischeti Jun 1, 2026

Uh oh!

fischeti Jun 1, 2026

Uh oh!

fischeti Jun 1, 2026

Uh oh!

fischeti Jun 1, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

	end else if(done == 1'b1) begin // job completion, floo dma is ready to accept new jobs
	end else if(done) begin // job completion, floo dma is ready to accept new jobs

	end else if(start == 1'b1) begin // start, with jobs to run
	end else if(start) begin // start, with jobs to run

Conversation

gbellocchi commented May 12, 2026

Uh oh!

fischeti left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

fischeti left a comment •

edited

Loading