[dpdk-dev] [PATCH v9] net/i40e: determine number of queues per VF during run time
Dai, Wei
wei.dai at intel.com
Tue Jan 9 10:09:41 CET 2018
Thanks to Helin and Ferruh,
I will submit v10 patch following your guide.
> -----Original Message-----
> From: Zhang, Helin
> Sent: Monday, January 8, 2018 2:01 PM
> To: Dai, Wei <wei.dai at intel.com>; Wu, Jingjing <jingjing.wu at intel.com>;
> Xing, Beilei <beilei.xing at intel.com>
> Cc: dev at dpdk.org
> Subject: RE: [PATCH v9] net/i40e: determine number of queues per VF during
> run time
>
>
>
> > -----Original Message-----
> > From: Dai, Wei
> > Sent: Tuesday, December 26, 2017 11:25 PM
> > To: Wu, Jingjing; Xing, Beilei; Zhang, Helin
> > Cc: dev at dpdk.org; Dai, Wei
> > Subject: [PATCH v9] net/i40e: determine number of queues per VF during
> > run time
> >
> > Without this patch, the number of queues per i40e VF is defined as 4
> > by
> > CONFIG_RTE_LIBRTE_I40E_QUEUE_NUM_PER_VF=4 in
> config/common_base.
> > It is fixed value determined in building time and can't be changed
> > during run time.
> > With this patch, the number of queues per i40e VF can be determinated
> > during run time. For example, if the PCI address of an i40e PF is
> > aaaa:bb.cc, with the EAL parameter -w aaaa:bb.cc,queue-num-per-vf=8 ,
> > the number of queues per VF created from this PF is 8.
> > If there is no "queue-num-per-vf" setting in EAL parameters, it is 4
> > by default as before. And if the value after the "queue-num-per-vf"
> > is invalid, it is set as 4 forcibly. The valid values include 1, 2, 4, 8, 16 .
> >
> > Signed-off-by: Wei Dai <wei.dai at intel.com>
> > Acked-by: Konstantin Ananyev <konstantin.ananyev at intel.com>
> > ---
> > v9:
> > v9 = v8+v7, is a complete version for maintainer's convenience.
> > v8:
> > As v7 patch has been accepted into dpdk-next-net-intel, this patch
> > is based on v7 patch.
> > add description in i40e document
> > fix the last member of valid_keys[] for rte_kvargs_parse( )
> > add RTE_PMD_REGISTER_PARAM_STRING for this feature
> > v7:
> > use the macro instead of natural number
> > correct git log message as the EAL parameter is only valid for PF
> > v6:
> > fix a small bug when detecting end character of strtoul
> > v5:
> > fix git log message and WARNING of coding stype
> > v4:
> > use rte_kvargs instead of pervious parsing function;
> > use malloc/free instead of rte_zmalloc/rte_free.
> > v3:
> > fix WARNING of coding style issues from checkpatch at dpdk.org
> > v2:
> > fix WARNING of coding style issues from checkpatch at dpdk.org
> > ---
> > config/common_base | 1 -
> > doc/guides/nics/i40e.rst | 12 ++++---
> > drivers/net/i40e/i40e_ethdev.c | 75
> > +++++++++++++++++++++++++++++++++++++++---
> > 3 files changed, 78 insertions(+), 10 deletions(-)
>
> BTW, I think release notes should be updated for your modifications.
More information about the dev
mailing list