On Mon, 31 Aug 2020 09:35:18 +0300
Andrew Rybchenko <arybche...@solarflare.com> wrote:

> >>>>> multisegment packets.    
> >>>>
> >>>> I hope it will be mentioned in the feature documentation in the future, 
> >>>> but
> >>>> I'm not 100% sure that it is required. See below.    
> >>> I suppose there is the hierarchy:
> >>> - applications configures DEV_RX_OFFLOAD_SCATTER on the port and tells in 
> >>> this way:
> >>> "Hey, driver, I'm ready to handle multi-segment packets". Readiness in 
> >>> general.
> >>> - application configures BUFFER_SPLIT and tells PMD _HOW_ it wants to 
> >>> split, in particular way:
> >>> "Hey, driver, please, drop ten bytes here, here and here, and the rest - 
> >>> over there"    
> >>
> >> My idea is to keep SCATTER and BUFFER_SPLIT independent.
> >> SCATTER is a possibility to make multi-segment packets getting
> >> mbufs from main rxq mempool as many as required.
> >> BUFFER_SPLIT is support of many mempools and splitting
> >> received packets as specified.  
> > 
> > No.
> > Once again, drivers should take anything from application and rely on using
> > logic to choose best path. Modern CPU's have good branch predictors, and 
> > making
> > the developer do that work is counter productive.  
> 
> Please, add a bit more details. I simply can see relationship.
> So, right now for me it looks like just misunderstanding.
> 
> Thanks,
> Andrew.

Ok, documenting the existing behaviour is good. I was just concerned that this 
was
going to lead to more per-queue flags.

Reply via email to