Re: [PATCH] arm64: dts: rockchip: set TX PBL for rk3328-roc-cc gmac

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 6/14/2019 5:26 AM, Heiko Stuebner wrote:
Hi,

Am Donnerstag, 23. Mai 2019, 02:00:27 CEST schrieb Peter Geis:

On 4/22/2019 9:13 AM, Leonidas P. Papadakos wrote:

Indeed, at least with that first patch users won't have a crappy
experience by default (ssh lags and the like)



I apologize for taking so long at this, but I wanted to make sure I had
it right.

So far I've found a few things that seem to help, and would like y'all
to test them when you get the chance.

Adding snps,aal (configures DMA for address alignment) appears to make a
massive improvement to stability, although the the snps,txpbl = <0x4> is
still necessary for this to work.

Second, I added snps,rxpbl = <0x4> as well, which seems to help with my
RX issues a bit.
The biggest improvement with these two settings is more stable transmit
and receive.
Currently I'm getting about 800mbps on tx and 400 mbps on rx.

Please test when you get the chance and provide feedback.

I have to confess I lost track of the multitude of patches touching
the gmac on rk3328 and their long mail threads in terms of which
to pick up and which are considered superseeded.

So I'd really appreciate if you could resend the patches that matter
and provide relevant Tested-by, Reviewed-by, etc tags from involved
people that actually were able to test patches

Thanks
Heiko



Good Morning Heiko,

Here's a quick catch-up for you:
Currently we use thresh dma mode to allow rgmii to work, but this has performance limitations. Leonidas submitted the txpbl 0x4 patch, which works well for his board but my board has very poor performance with. I submitted a new patch that combined txpbl 0x4 with aal dma mode, which fixes my issue and seems to work well for Leonidas.

That patch is under [PATCH] arm64: dts: rockchip: improve rk3328-roc-cc rgmii performance. I'd like more people to test it prior to submitting it to replace thresh dma mode.

If we get a few more tests without regression, I'll resubmit it with Tested-by tags for inclusion.

Thanks,
Peter



[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux