On 6/14/2019 6:45 PM, Marc Gonzalez wrote:
On 12/06/2019 09:15, Vivek Gautam wrote:
This change is inspired by the downstream change from Patrick Daly
to address performance issues with display and camera by handling
this wait-for-safe within separte io-pagetable ops to do TLB
maintenance. So a big thanks to him for the change.
Without this change the UFS reads are pretty slow:
$ time dd if=/dev/sda of=/dev/zero bs=1048576 count=10 conv=sync
10+0 records in
10+0 records out
10485760 bytes (10.0MB) copied, 22.394903 seconds, 457.2KB/s
real 0m 22.39s
user 0m 0.00s
sys 0m 0.01s
With this change they are back to rock!
$ time dd if=/dev/sda of=/dev/zero bs=1048576 count=300 conv=sync
300+0 records in
300+0 records out
314572800 bytes (300.0MB) copied, 1.030541 seconds, 291.1MB/s
real 0m 1.03s
user 0m 0.00s
sys 0m 0.54s
This issue does not affect msm8998, I presume?
Nevertheless, I see much lower performance on msm8998:
# dd if=/dev/sde of=/dev/null bs=1M status=progress
3892314112 bytes (3.9 GB, 3.6 GiB) copied, 50.0042 s, 77.8 MB/s
80 MB/s on msm8998 -- vs -- 300 MB/s on sdm845
Do you have the interconnect patches on sdm845 that allow boosting
the clock/bandwidth for specific HW blocks?
Umm, No. This is the upstream 5.2-rc4 plus 4-6 patches to enable display
and fix splash screen.
Is this the performance for UFS? The numbers i posted were for UFS.
Thanks
Vivek
