lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Message-ID: <20221007174914.GA1032@fastly.com> Date: Fri, 7 Oct 2022 10:49:14 -0700 From: Joe Damato <jdamato@...tly.com> To: Maciej Fijalkowski <maciej.fijalkowski@...el.com> Cc: intel-wired-lan@...ts.osuosl.org, netdev@...r.kernel.org, kuba@...nel.org, davem@...emloft.net, anthony.l.nguyen@...el.com, jesse.brandeburg@...el.com Subject: Re: [next-queue v3 4/4] i40e: Add i40e_napi_poll tracepoint On Fri, Oct 07, 2022 at 10:18:41AM +0200, Maciej Fijalkowski wrote: > On Thu, Oct 06, 2022 at 04:43:58PM -0700, Joe Damato wrote: > > Add a tracepoint for i40e_napi_poll that allows users to get detailed > > information about the amount of work done. This information can help users > > better tune the correct NAPI parameters (like weight and budget), as well > > as debug NIC settings like rx-usecs and tx-usecs, etc. > > > > An example of the output from this tracepoint: > > > > $ sudo perf trace -e i40e:i40e_napi_poll -a --call-graph=fp --libtraceevent_print > > > > [..snip..] > > > > 388.258 :0/0 i40e:i40e_napi_poll(i40e_napi_poll on dev eth2 q > > i40e-eth2-TxRx-9 irq 346 irq_mask > > 00000000,00000000,00000000,00000000,00000000,00800000 curr_cpu 23 budget > > 64 bpr 64 rx_cleaned 28 tx_cleaned 0 rx_clean_complete 1 > > tx_clean_complete 1) > > So from AF_XDP POV I won't be using it as I would need some other > information. > > As I said, we don't work on NAPI budget but rather with the > free ring space and I don't get it here. tx_cleaned is also quite > incorrect name to me for count of produced descriptors to Tx ring. I feel > like it would be better to stub it for AF_XDP. > > As Jesse said previously we probably can followup with AF_XDP specific > tracepoint with tx cleaned/tx transmitted/NAPI budget/AF_XDP budget (free > ring space) if we find the need for it. > > That's my 0.02$, I'm not going to hold this set or whatever, I'll leave the > decision to Sridhar & Jesse. I'll send the other patchset I've written and tested as an RFC which doesn't touch anything in XDP at all and only adds the tracepoint in i40e_napi_poll only if xdp is not enabled. The code in that branch for i40e_napi_poll looks like this: + if (!i40e_enabled_xdp_vsi(vsi)) + trace_i40e_napi_poll(napi, q_vector, budget, budget_per_ring, rx_cleaned, + tx_cleaned, rx_clean_complete, tx_clean_complete); The XDP functions are not modified to take any out parameters in that branch. In that case, anyone who cares about XDP (when I think about XDP, I feel nothing, so that probably won't be me) can go back through and add their own XDP related tracepoint as the 'else' branch and modify all the xdp related functions and add (very precisely named) out parameters. As far as I'm concerned: I submit this code simply because it's been very useful for me to debug i40e performance and to tune settings and I thought it might be helpful for others.
Powered by blists - more mailing lists