Hi Akhil,
Can you review this series and share your thoughts?
Thanks,
Anoob
> -----Original Message-----
> From: Anoob Joseph
> Sent: Saturday, March 2, 2019 12:12 AM
> To: Akhil Goyal <akhil.go...@nxp.com>; Pablo de Lara
> <pablo.de.lara.gua...@intel.com>
> Cc: Anoob Joseph <ano...@marvell.com>; Jerin Jacob Kollanukkaran
> <jer...@marvell.com>; Narayana Prasad Raju Athreya
> <pathr...@marvell.com>; Suheil Chandran <schand...@marvell.com>;
> Ankur Dwivedi <adwiv...@marvell.com>; dev@dpdk.org
> Subject: [PATCH 1/3] common/cpt: improve structures used in datapath
>
> Signed-off-by: Ankur Dwivedi <adwiv...@marvell.com>
> Signed-off-by: Anoob Joseph <ano...@marvell.com>
> ---
> drivers/common/cpt/cpt_common.h | 10 +++++-----
> 1 file changed, 5 insertions(+), 5 deletions(-)
>
> diff --git a/drivers/common/cpt/cpt_common.h
> b/drivers/common/cpt/cpt_common.h index 8461cd6..8568d5b 100644
> --- a/drivers/common/cpt/cpt_common.h
> +++ b/drivers/common/cpt/cpt_common.h
> @@ -60,14 +60,14 @@ struct rid {
> *
> */
> struct pending_queue {
> + /** Pending requests count */
> + uint64_t pending_count;
> + /** Array of pending requests */
> + struct rid *rid_queue;
> /** Tail of queue to be used for enqueue */
> uint16_t enq_tail;
> /** Head of queue to be used for dequeue */
> uint16_t deq_head;
> - /** Array of pending requests */
> - struct rid *rid_queue;
> - /** Pending requests count */
> - uint64_t pending_count;
> };
>
> struct cpt_request_info {
> @@ -86,6 +86,6 @@ struct cpt_request_info {
> /** Control path fields */
> uint64_t time_out;
> uint8_t extra_time;
> -};
> +} __rte_cache_aligned;
>
> #endif /* _CPT_COMMON_H_ */
> --
> 2.7.4
