> 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!