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 <[email protected]>; Pablo de Lara > <[email protected]> > Cc: Anoob Joseph <[email protected]>; Jerin Jacob Kollanukkaran > <[email protected]>; Narayana Prasad Raju Athreya > <[email protected]>; Suheil Chandran <[email protected]>; > Ankur Dwivedi <[email protected]>; [email protected] > Subject: [PATCH 1/3] common/cpt: improve structures used in datapath > > Signed-off-by: Ankur Dwivedi <[email protected]> > Signed-off-by: Anoob Joseph <[email protected]> > --- > 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
