On Tue, Feb 8, 2022 at 7:42 PM Alexander Kozyrev <akozy...@nvidia.com> wrote: > > > On Tuesday, February 8, 2022 5:57 Jerin Jacob <jerinjac...@gmail.com> wrote: > > On Sun, Feb 6, 2022 at 8:57 AM Alexander Kozyrev <akozy...@nvidia.com> > > wrote: > > > > > +The asynchronous flow rule insertion logic can be broken into two phases. > > > + > > > +1. Initialization stage as shown here: > > > + > > > +.. _figure_rte_flow_q_init: > > > + > > > +.. figure:: img/rte_flow_q_init.* > > > + > > > +2. Main loop as presented on a datapath application example: > > > + > > > +.. _figure_rte_flow_q_usage: > > > + > > > +.. figure:: img/rte_flow_q_usage.* > > > > it is better to add sequence operations as text to understand the flow. > > I prefer keeping the diagram here, it looks more clean and concise. > Block of text gives no new information and harder to follow, IMHO.
I forgot to reply yesterday on this specific item. IMO, The diagram is good. The request is, In the diagram, you can add items like [1] [2] etc and corresponding text can be added either in image or as text See example https://doc.dpdk.org/guides/prog_guide/event_crypto_adapter.html Fig. 49.2.
