> -----Original Message-----
> From: Robin Jarry <rja...@redhat.com>
> Sent: Wednesday, March 20, 2024 10:41 PM
> To: dev@dpdk.org; Jerin Jacob <jer...@marvell.com>; Kiran Kumar
> Kokkilagadda <kirankum...@marvell.com>; Nithin Kumar Dabilpuram
> <ndabilpu...@marvell.com>; Zhirun Yan <yanzhirun_...@163.com>
> Subject: [EXTERNAL] [PATCH] graph: enhance export to graphviz
>
> Prioritize security for external emails: Confirm sender and content safety
> before clicking links or opening attachments
>
> ----------------------------------------------------------------------
> * Quote graph name to avoid parsing errors when it contains a dash.
> * Use fixed margin and smaller font for a more compact layout.
> * Use sans-serif font, the default is times new roman which is not the
> best choice for a packet processing generated graph.
> * Force bold blue border and arrows for source nodes.
> * Force dark orange borders for sink nodes (nodes without next nodes).
>
> Link: https://urldefense.proofpoint.com/v2/url?u=https-3A__f.jarry.cc_rte-
> 2Dgraph-
> 2Ddot_before.svg&d=DwIDAg&c=nKjWec2b6R0mOyPaz7xtfQ&r=owEKckYY4FT
> mil1Z6oBURwkTThyuRbLAY9LdfiaT6HA&m=pyYDQxlqMplh30E36wiXH6LLReml
> Q19SXfdBaWFn9w76vzq1CZdH-
> MC9xnFF1F73&s=j08NkMjMiHpSAVfucSwMN7c769_JbCqmTlEv-O0LyeQ&e=
> Link: https://urldefense.proofpoint.com/v2/url?u=https-3A__f.jarry.cc_rte-
> 2Dgraph-
> 2Ddot_after.svg&d=DwIDAg&c=nKjWec2b6R0mOyPaz7xtfQ&r=owEKckYY4FTmi
> l1Z6oBURwkTThyuRbLAY9LdfiaT6HA&m=pyYDQxlqMplh30E36wiXH6LLRemlQ1
> 9SXfdBaWFn9w76vzq1CZdH-MC9xnFF1F73&s=dQTT3aUgxo-
> oZVMsw0KJR1UT2Gn3oi9r50TAJdIgJLs&e=
> Signed-off-by: Robin Jarry <rja...@redhat.com>
Acked-by: Kiran Kumar Kokkilagadda <kirankum...@marvell.com>
> ---
> lib/graph/graph.c | 32 +++++++++++++++++++++++++-------
> 1 file changed, 25 insertions(+), 7 deletions(-)
>
> diff --git a/lib/graph/graph.c b/lib/graph/graph.c index
> 26f0968a978d..147bc6c685c5 100644
> --- a/lib/graph/graph.c
> +++ b/lib/graph/graph.c
> @@ -674,25 +674,43 @@ __rte_node_stream_alloc_size(struct rte_graph
> *graph, struct rte_node *node, static int graph_to_dot(FILE *f, struct graph
> *graph) {
> - const char *src_edge_color = " [color=blue]\n";
> - const char *edge_color = "\n";
> struct graph_node *graph_node;
> char *node_name;
> rte_edge_t i;
> int rc;
>
> - rc = fprintf(f, "Digraph %s {\n\trankdir=LR;\n", graph->name);
> + rc = fprintf(f, "digraph \"%s\" {\n\trankdir=LR;\n", graph->name);
> + if (rc < 0)
> + goto end;
> +
> + rc = fprintf(f, "\tnode [margin=0.02 fontsize=11 fontname=sans];\n");
> if (rc < 0)
> goto end;
>
> STAILQ_FOREACH(graph_node, &graph->node_list, next) {
> + const char *attrs = "";
> node_name = graph_node->node->name;
> +
> + rc = fprintf(f, "\t\"%s\"", node_name);
> + if (rc < 0)
> + goto end;
> + if (graph_node->node->flags & RTE_NODE_SOURCE_F) {
> + attrs = " [color=blue style=bold]";
> + rc = fprintf(f, "%s", attrs);
> + if (rc < 0)
> + goto end;
> + } else if (graph_node->node->nb_edges == 0) {
> + rc = fprintf(f, " [color=darkorange]");
> + if (rc < 0)
> + goto end;
> + }
> + rc = fprintf(f, ";\n");
> + if (rc < 0)
> + goto end;
> for (i = 0; i < graph_node->node->nb_edges; i++) {
> - rc = fprintf(f, "\t\"%s\"->\"%s\"%s", node_name,
> + rc = fprintf(f, "\t\"%s\" -> \"%s\"%s;\n", node_name,
> graph_node->adjacency_list[i]->node-
> >name,
> - graph_node->node->flags &
> RTE_NODE_SOURCE_F
> - ? src_edge_color
> - : edge_color);
> + attrs);
> if (rc < 0)
> goto end;
> }
> --
> 2.44.0
