Review Request 129058: [corona] Sync config immidiately when it's dirty

Anthony Fieroni bvbfan at abv.bg
Wed Sep 28 10:39:23 UTC 2016



> On Sept. 28, 2016, 1:12 p.m., Marco Martin wrote:
> > all thisdoes is causing more disk access, making things slower
> 
> Anthony Fieroni wrote:
>     More disk access, maybe, but slower, how can be?
> 
> Aleix Pol Gonzalez wrote:
>     Why more disk access?
>     At least this won't be emitting configChanged if it was dirty and was a no-op sync.
> 
> David Edmundson wrote:
>     because there's more disk access.
>     
>     Turning this question around: how can yours be faster? - nothing is blocked waiting on the file to be written - so making that happen sooner won't speed anything up

Yes it is. Waiting 10 sec is never be faster. Even i wait more than 20 sec because timer is scheduled twice. On SSD, i have boot time 3 sec, KDE splash > 20 sec.


- Anthony


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/129058/#review99612
-----------------------------------------------------------


On Sept. 28, 2016, 10:42 a.m., Anthony Fieroni wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/129058/
> -----------------------------------------------------------
> 
> (Updated Sept. 28, 2016, 10:42 a.m.)
> 
> 
> Review request for Plasma, Aleix Pol Gonzalez and David Edmundson.
> 
> 
> Repository: plasma-framework
> 
> 
> Description
> -------
> 
> This patch trys to minimize loading time of plasmashell, especially annoying waiting at splash screen.
> 
> 
> Diffs
> -----
> 
>   src/plasma/corona.cpp 7ff64cd 
> 
> Diff: https://git.reviewboard.kde.org/r/129058/diff/
> 
> 
> Testing
> -------
> 
> Seems good but it may have downsides?
> 
> 
> Thanks,
> 
> Anthony Fieroni
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20160928/05e5f184/attachment.html>


More information about the Plasma-devel mailing list