> -----Original Message-----
> From: dev [mailto:[email protected]] On Behalf Of Boris Pismenny
> Sent: Sunday, September 17, 2017 1:07 PM
> To: [email protected]
> Cc: [email protected]; Doherty, Declan <[email protected]>; De
> Lara Guarch, Pablo <[email protected]>;
> [email protected]; Nicolau, Radu <[email protected]>;
> [email protected]; [email protected]; [email protected];
> [email protected]; [email protected];
> [email protected]; [email protected]
> Subject: [dpdk-dev] [PATCH 1/2] doc: add details of rte_flow security
> actions
> 
> Signed-off-by: Boris Pismenny <[email protected]>


Thanks for the docs. Minor comments below.



> +
> +Multiple flows can be configured to use the same security session.
> +
> +- Non-terminating by default.

This is a single bullet point list and it seems disconnected from the previous
line. Maybe convert it to a sentence instead.



> +
> +.. _table_rte_flow_action_security

This is missing a colon at the end and causes a doc build warning.


> +Usage example, configure IPsec inline using INLINE_CRYPTO security
> session:

This doesn't seem to connect to the following paragraph. Maybe something
like:

    The following is an example of configuring IPsec inline using the
    INLINE_CRYPTO security session:

Reviewed-by: John McNamara <[email protected]>

Reply via email to