> From: Jerin Jacob [mailto:jerin.ja...@caviumnetworks.com]
> Sent: Wednesday, December 14, 2016 1:14 PM
> To: Van Haaren, Harry <harry.van.haa...@intel.com>
<snip>

> So incorporating my latest suggestions on moving sub_event_type field around:
> 
> union {
>       uint64_t event;
>       struct {
>               uint32_t flow_id: 20;
>               uint32_t sub_event_type : 8;
>               uint32_t event_type : 4;
> 
>               uint8_t operation  : 2; /* new fwd drop */
>               uint8_t rsvd: 4; /* for future additions */
>               uint8_t sched_type : 2;
> 
>               uint8_t queue_id;
>               uint8_t priority;
>               uint8_t impl_opaque;
>       };
> };

Thanks, looks good to me!

Reply via email to