> > > You should also set a default value of this flag.
> >
> > Default functionality is disable SDAP. Application is defining the cperf 
> > options
> > with all values 0s in its main(): "struct cperf_options opts = {0};". Is 
> > n't it
> > sufficient?
> It would be good to initialize that with other options. It will help in 
> debugging
> and
> Improve code readability.
> Please add it in cperf_options_default() as it is done for other PDCP options.
> 
Please send v4 of this patch as standalone patch. The rest of the patches have 
been
Applied to dpdk-next-crypto

Thanks.

Reply via email to