| 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: <56D55DAD.7060002@axis.com> Date: Tue, 1 Mar 2016 10:15:25 +0100 From: Lars Persson <lars.persson@...s.com> To: Giuseppe CAVALLARO <peppe.cavallaro@...com>, <netdev@...r.kernel.org>, <maxime.coquelin@...com>, <rabinv@...s.com>, <larper@...s.com>, Rayagond K <rayagond@...avyalabs.com> CC: Alexandre TORGUE <alexandre.torgue@...com>, <fabrice.gasnier@...com>, Joachim Eastwood <manabian@...il.com> Subject: Re: [PATCH v3 00/17] stmmac: enhance driver performances and update the version Hi Peppe We look forward to seeing the patches for 4.10a support. I hope that we can work together to have one driver supporting all chips with the 4.x GMACs. - Lars On 02/29/2016 05:50 PM, Giuseppe CAVALLARO wrote: > Gents > > on top of these patches, there is a new train to enhance the stmmac to > support the DWMAC_4.x chips. They will be proposed very soon and on > top of this update (as soon as reviewed and merged). > > In our context, it has been very useful working with the same driver > that runs fine on several (x86, arm, sh4) boxes with different SYNP > MAC/GMAC IPs (starting from MAC10/100 Database Release 1.5 to databook > 3.70a and .... 4.00a and 4.10a). We got the benefit to have all the > features already supported by stmmac plus the good performances > available with TSO on gmac4. > > I can image no big issues to enhance the stmmac on supporting the > new 4.00a and 4.10a although there is some other work made by > Rabin and Larper. > > I guess, stmmac users will continue to be happy to continue to have the > same d.d. working on their platforms with new gmac. > But! it also makes sense to avoid to have two drivers that aim > to do the same job. Or to get more synergy on the same code as > done in the past with Rayagond for PTP and EEE. > > If you have some concern or advice, please do not hesitate to ask. > We will try to send the patches soon to show the code in case of > people are interested in. > > Kind Regards > Peppe > > On 2/29/2016 2:27 PM, Alexandre TORGUE wrote: >> According to Giuseppe, I send the v3 series. >> >> This is a subset of patches to rework the driver in order to improve its >> performances and make it more robust under stress conditions. >> >> All patches have been ported on STi mainstream kernel branch and >> tested on ARM STiH4xx platforms and newer ones. >> >> This series also updates the driver version and prepares it >> to include further development to support new chips. >> >> In detail, these patches are: >> >> o to rework and improve the internal DMA bus settings >> >> Fine tuning is mandatory on some platforms for both >> performance and stability issues. >> >> o to rework and optimize the descriptor management. >> >> This will help a lot on performance side and preparing >> the inclusion on the GMAC4.x. >> >> o to add a set of optimizations for both xmit and rx functions. >> >> These will help a lot on performance side and making the driver >> more robust in case of low memory conditions and under some >> stress test, performed for example on IP-STB. >> >> Below some throughput figures obtained on some boxes before and after >> the patches. >> >> nuttcp (mbps) iperf (Mbps) >> ------------------------------------------------------------------ >> tcp udp tcp udp >> tx rx tx rx tx rx tx rx >> ------------------------------------------ >> old 680 800 480 506 760 800 600 700 >> new 830 880 540 630 840 880 700 800 >> >> ============================================================================== >> >> >> V2: - rx_copybreak is now managed by using ethtool. >> V3: - improve comments on PCIe detailing that there are no regressions >> - rework some APIs to properly define some params as bool as >> expected >> - rework the formula to get the element inside the ring. >> Comparing V2, >> patches 4 and 13 have been merged because the same formula have been >> used. After this rework, no evident benefit has been noticed in terms >> of performances so the table above is still valid. Disassembling the >> code for SH4 and ARM, with the new formula just an instr is saved >> (depending on compiler flags) and this gives us not so relevanti >> gain, >> for example, on SH4 where some instr are executed in the same >> pipeline >> stage. >> Ring sizes are now fixed and maybe they can be reworked to be tuned >> w/o using stmmaceth= cmdline option. Indeed, nobody change these >> sizes >> and indeed the numbers selected by default respect the budget and >> avoid to pass invalid setup. These are the best driver default sizes >> for ring and chain. >> >> ============================================================================== >> >> Fabrice Gasnier (3): >> stmmac: merge get_rx_owner into rx_status routine. >> stmmac: optimize tx clean function >> stmmac: fix phy init when attached to a phy >> >> Giuseppe Cavallaro (14): >> stmmac: share reset function between dwmac100 and dwmac1000 >> stmmac: rework DMA bus setting and introduce new platform AXI >> structure >> stmmac: change descriptor layout >> stmmac: review RX/TX ring management >> stmmac: add length field to dma data >> stmmac: add last_segment field to dma data >> stmmac: add is_jumbo field to dma data >> stmmac: optimize tx desc management >> stmmac: set dirty index out of the loop >> stmmac: first frame prep at the end of xmit routine >> stmmac: do not poll phy handler when attach a switch >> stmmac: do not perform zero-copy for rx frames >> stmmac: tune rx copy via threshold. >> stmmac: update version to Oct_2015 >> >> Documentation/devicetree/bindings/net/stmmac.txt | 54 ++- >> drivers/net/ethernet/stmicro/stmmac/chain_mode.c | 37 +- >> drivers/net/ethernet/stmicro/stmmac/common.h | 39 +- >> drivers/net/ethernet/stmicro/stmmac/descs.h | 330 +++++++------- >> drivers/net/ethernet/stmicro/stmmac/descs_com.h | 77 ++-- >> drivers/net/ethernet/stmicro/stmmac/dwmac100.h | 1 - >> drivers/net/ethernet/stmicro/stmmac/dwmac1000.h | 3 +- >> .../net/ethernet/stmicro/stmmac/dwmac1000_dma.c | 111 +++-- >> drivers/net/ethernet/stmicro/stmmac/dwmac100_dma.c | 22 +- >> drivers/net/ethernet/stmicro/stmmac/dwmac_dma.h | 39 +- >> drivers/net/ethernet/stmicro/stmmac/dwmac_lib.c | 21 + >> drivers/net/ethernet/stmicro/stmmac/enh_desc.c | 226 +++++----- >> drivers/net/ethernet/stmicro/stmmac/norm_desc.c | 150 ++++--- >> drivers/net/ethernet/stmicro/stmmac/ring_mode.c | 32 +- >> drivers/net/ethernet/stmicro/stmmac/stmmac.h | 9 +- >> .../net/ethernet/stmicro/stmmac/stmmac_ethtool.c | 41 +- >> drivers/net/ethernet/stmicro/stmmac/stmmac_main.c | 473 >> ++++++++++++--------- >> drivers/net/ethernet/stmicro/stmmac/stmmac_pci.c | 4 +- >> .../net/ethernet/stmicro/stmmac/stmmac_platform.c | 42 +- >> include/linux/stmmac.h | 17 +- >> 20 files changed, 1016 insertions(+), 712 deletions(-) >> >
Powered by blists - more mailing lists