On Fri, May 12, 2023 at 05:40:11PM +0200, Piotr Raczynski wrote: > > @@ -1162,6 +1162,9 @@ int ice_clean_rx_irq(struct ice_rx_ring *rx_ring, int budget) > > bool failure; > > u32 first; > > > > + if (ntc != rx_ring->first_desc) > > + cached_ntc = rx_ring->first_desc; > > + > Would it make sense to apply likely/unlikely here since we check it per > packet? This was done before running Rx processing loop so this was not per packet, only a single check, but in the end we don't need this, see Simon's reply. > > Piotr > > /* Frame size depend on rx_ring setup when PAGE_SIZE=4K */ > > #if (PAGE_SIZE < 8192) > > >