DPDK patches and discussions
 help / color / mirror / Atom feed
* [Bug 1132] net/i40e changes recommended Rx ring_size after device start
@ 2022-11-23  8:57 bugzilla
  2022-11-30  7:35 ` bugzilla
  0 siblings, 1 reply; 2+ messages in thread
From: bugzilla @ 2022-11-23  8:57 UTC (permalink / raw)
  To: dev

https://bugs.dpdk.org/show_bug.cgi?id=1132

            Bug ID: 1132
           Summary: net/i40e changes recommended Rx ring_size after device
                    start
           Product: DPDK
           Version: 22.11
          Hardware: All
                OS: All
            Status: UNCONFIRMED
          Severity: normal
          Priority: Normal
         Component: ethdev
          Assignee: dev@dpdk.org
          Reporter: andrew.rybchenko@oktetlabs.ru
  Target Milestone: ---

net/i40e changes recommended Rx ring_size after device start

NIC is Intel X710

It does not make sense since all Rx queues are typically already configured and
started.

It happens because of link_speeds condition above:
790 »·······} else {
 3791 »·······»·······/* For X710 */
 3792 »·······»·······dev_info->speed_capa = RTE_ETH_LINK_SPEED_1G |
RTE_ETH_LINK_SPEED_10G;
 3793 »·······»·······dev_info->default_rxportconf.nb_queues = 1;
 3794 »·······»·······dev_info->default_txportconf.nb_queues = 1;
 3795 »·······»·······if (dev->data->dev_conf.link_speeds &
RTE_ETH_LINK_SPEED_10G) {
 3796 »·······»·······»·······dev_info->default_rxportconf.ring_size = 512;
 3797 »·······»·······»·······dev_info->default_txportconf.ring_size = 256;
 3798 »·······»·······} else {
 3799 »·······»·······»·······dev_info->default_rxportconf.ring_size = 256;
 3800 »·······»·······»·······dev_info->default_txportconf.ring_size = 256;
 3801 »·······»·······}
 3802 »·······}

which are updated on device start in i40e_apply_link_speed().

Before start default_rxportconf.ring_size is 256, after start it is 512.

Found using [1]

[1]
https://ts-factory.io/doc/dpdk-ethdev-ts/generated/rst/group_usecases-dev_info_persistence.html#doxid-group-usecases-dev-info-persistence

-- 
You are receiving this mail because:
You are the assignee for the bug.

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2022-11-30  7:35 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-11-23  8:57 [Bug 1132] net/i40e changes recommended Rx ring_size after device start bugzilla
2022-11-30  7:35 ` bugzilla

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).