)]}'
{"id":"LineageOS%2Fandroid_kernel_asus_sm8150~251943","triplet_id":"LineageOS%2Fandroid_kernel_asus_sm8150~lineage-16.0~I44fc7adc155a126ea61f6956a15ca1a02fdc4692","project":"LineageOS/android_kernel_asus_sm8150","branch":"lineage-16.0","hashtags":[],"change_id":"I44fc7adc155a126ea61f6956a15ca1a02fdc4692","subject":"ANDROID: cpufreq/schedutil: add up/down frequency transition rate limits","status":"MERGED","created":"2019-07-29 09:29:36.000000000","updated":"2019-07-29 09:32:53.000000000","submitted":"2019-07-29 09:32:53.000000000","submitter":{"_account_id":11883,"name":"Luca Stefani","email":"luca.stefani.ge1@gmail.com","username":"luca020400","avatars":[{"url":"https://www.gravatar.com/avatar/e0d7fe321be897bf85cf3d240cbd3be9.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d32","height":32},{"url":"https://www.gravatar.com/avatar/e0d7fe321be897bf85cf3d240cbd3be9.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d56","height":56},{"url":"https://www.gravatar.com/avatar/e0d7fe321be897bf85cf3d240cbd3be9.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d100","height":100},{"url":"https://www.gravatar.com/avatar/e0d7fe321be897bf85cf3d240cbd3be9.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d120","height":120}]},"total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"266bee179bf61b0d29620baa28612392ea7bc403","_number":251943,"virtual_id_number":251943,"owner":{"_account_id":11883,"name":"Luca Stefani","email":"luca.stefani.ge1@gmail.com","username":"luca020400","avatars":[{"url":"https://www.gravatar.com/avatar/e0d7fe321be897bf85cf3d240cbd3be9.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d32","height":32},{"url":"https://www.gravatar.com/avatar/e0d7fe321be897bf85cf3d240cbd3be9.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d56","height":56},{"url":"https://www.gravatar.com/avatar/e0d7fe321be897bf85cf3d240cbd3be9.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d100","height":100},{"url":"https://www.gravatar.com/avatar/e0d7fe321be897bf85cf3d240cbd3be9.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d120","height":120}]},"actions":{},"labels":{"Verified":{"all":[{"value":0,"_account_id":11883,"name":"Luca Stefani","email":"luca.stefani.ge1@gmail.com","username":"luca020400","avatars":[{"url":"https://www.gravatar.com/avatar/e0d7fe321be897bf85cf3d240cbd3be9.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d32","height":32},{"url":"https://www.gravatar.com/avatar/e0d7fe321be897bf85cf3d240cbd3be9.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d56","height":56},{"url":"https://www.gravatar.com/avatar/e0d7fe321be897bf85cf3d240cbd3be9.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d100","height":100},{"url":"https://www.gravatar.com/avatar/e0d7fe321be897bf85cf3d240cbd3be9.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d120","height":120}]}],"values":{"-1":"Fails"," 0":"No score","+1":"Verified"},"description":"","default_value":0},"Code-Review":{"all":[{"value":0,"_account_id":11883,"name":"Luca Stefani","email":"luca.stefani.ge1@gmail.com","username":"luca020400","avatars":[{"url":"https://www.gravatar.com/avatar/e0d7fe321be897bf85cf3d240cbd3be9.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d32","height":32},{"url":"https://www.gravatar.com/avatar/e0d7fe321be897bf85cf3d240cbd3be9.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d56","height":56},{"url":"https://www.gravatar.com/avatar/e0d7fe321be897bf85cf3d240cbd3be9.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d100","height":100},{"url":"https://www.gravatar.com/avatar/e0d7fe321be897bf85cf3d240cbd3be9.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d120","height":120}]}],"values":{"-2":"Do not submit","-1":"I would prefer that you didn\u0027t submit this"," 0":"No score","+1":"Looks good to me, but someone else must approve","+2":"Looks good to me, approved"},"description":"","default_value":0},"CI":{"all":[{"value":0,"_account_id":11883,"name":"Luca Stefani","email":"luca.stefani.ge1@gmail.com","username":"luca020400","avatars":[{"url":"https://www.gravatar.com/avatar/e0d7fe321be897bf85cf3d240cbd3be9.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d32","height":32},{"url":"https://www.gravatar.com/avatar/e0d7fe321be897bf85cf3d240cbd3be9.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d56","height":56},{"url":"https://www.gravatar.com/avatar/e0d7fe321be897bf85cf3d240cbd3be9.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d100","height":100},{"url":"https://www.gravatar.com/avatar/e0d7fe321be897bf85cf3d240cbd3be9.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d120","height":120}]}],"values":{"-1":"Fail"," 0":"No score","+1":"Pass"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{},"pending_reviewers":{},"reviewer_updates":[],"messages":[{"id":"b6879258a5530b09c89815a7bfed2471a306629e","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":11883,"name":"Luca Stefani","email":"luca.stefani.ge1@gmail.com","username":"luca020400","avatars":[{"url":"https://www.gravatar.com/avatar/e0d7fe321be897bf85cf3d240cbd3be9.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d32","height":32},{"url":"https://www.gravatar.com/avatar/e0d7fe321be897bf85cf3d240cbd3be9.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d56","height":56},{"url":"https://www.gravatar.com/avatar/e0d7fe321be897bf85cf3d240cbd3be9.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d100","height":100},{"url":"https://www.gravatar.com/avatar/e0d7fe321be897bf85cf3d240cbd3be9.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d120","height":120}]},"date":"2019-07-29 09:29:36.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"266bee179bf61b0d29620baa28612392ea7bc403","tag":"autogenerated:gerrit:merged","author":{"_account_id":11883,"name":"Luca Stefani","email":"luca.stefani.ge1@gmail.com","username":"luca020400","avatars":[{"url":"https://www.gravatar.com/avatar/e0d7fe321be897bf85cf3d240cbd3be9.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d32","height":32},{"url":"https://www.gravatar.com/avatar/e0d7fe321be897bf85cf3d240cbd3be9.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d56","height":56},{"url":"https://www.gravatar.com/avatar/e0d7fe321be897bf85cf3d240cbd3be9.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d100","height":100},{"url":"https://www.gravatar.com/avatar/e0d7fe321be897bf85cf3d240cbd3be9.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d120","height":120}]},"date":"2019-07-29 09:32:53.000000000","message":"Change has been successfully pushed.","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"57a879b004b129dbf54beaa074132aeee838fcd8","revisions":{"57a879b004b129dbf54beaa074132aeee838fcd8":{"kind":"REWORK","_number":1,"created":"2019-07-29 09:29:36.000000000","uploader":{"_account_id":11883,"name":"Luca Stefani","email":"luca.stefani.ge1@gmail.com","username":"luca020400","avatars":[{"url":"https://www.gravatar.com/avatar/e0d7fe321be897bf85cf3d240cbd3be9.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d32","height":32},{"url":"https://www.gravatar.com/avatar/e0d7fe321be897bf85cf3d240cbd3be9.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d56","height":56},{"url":"https://www.gravatar.com/avatar/e0d7fe321be897bf85cf3d240cbd3be9.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d100","height":100},{"url":"https://www.gravatar.com/avatar/e0d7fe321be897bf85cf3d240cbd3be9.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d120","height":120}]},"ref":"refs/changes/43/251943/1","fetch":{"anonymous http":{"url":"https://github.com/LineageOS/android_kernel_asus_sm8150","ref":"refs/changes/43/251943/1","commands":{"Branch":"git fetch https://github.com/LineageOS/android_kernel_asus_sm8150 refs/changes/43/251943/1 \u0026\u0026 git checkout -b change-251943 FETCH_HEAD","Checkout":"git fetch https://github.com/LineageOS/android_kernel_asus_sm8150 refs/changes/43/251943/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://github.com/LineageOS/android_kernel_asus_sm8150 refs/changes/43/251943/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://github.com/LineageOS/android_kernel_asus_sm8150 refs/changes/43/251943/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://github.com/LineageOS/android_kernel_asus_sm8150 refs/changes/43/251943/1","Reset To":"git fetch https://github.com/LineageOS/android_kernel_asus_sm8150 refs/changes/43/251943/1 \u0026\u0026 git reset --hard FETCH_HEAD"}}},"commit":{"parents":[{"commit":"c933e9dc8fcd0d3d6bf7d4047df2cc6e98d59353","subject":"sched: Improve the scheduler","web_links":[{"name":"GitHub","tooltip":"Open in GitWeb","url":"https://github.com/LineageOS/android_kernel_asus_sm8150/commit/c933e9dc8fcd0d3d6bf7d4047df2cc6e98d59353"}]}],"author":{"name":"Steve Muckle","email":"smuckle.linux@gmail.com","date":"2016-11-17 05:18:45.000000000","tz":330},"committer":{"name":"Luca Stefani","email":"luca.stefani.ge1@gmail.com","date":"2019-07-29 09:24:38.000000000","tz":120},"subject":"ANDROID: cpufreq/schedutil: add up/down frequency transition rate limits","message":"ANDROID: cpufreq/schedutil: add up/down frequency transition rate limits\n\nThe rate-limit tunable in the schedutil governor applies to transitions\nto both lower and higher frequencies. On several platforms it is not the\nideal tunable though, as it is difficult to get best power/performance\nfigures using the same limit in both directions.\n\nIt is common on mobile platforms with demanding user interfaces to want\nto increase frequency rapidly for example but decrease slowly.\n\nOne of the example can be a case where we have short busy periods\nfollowed by similar or longer idle periods. If we keep the rate-limit\nhigh enough, we will not go to higher frequencies soon enough. On the\nother hand, if we keep it too low, we will have too many frequency\ntransitions, as we will always reduce the frequency after the busy\nperiod.\n\nIt would be very useful if we can set low rate-limit while increasing\nthe frequency (so that we can respond to the short busy periods quickly)\nand high rate-limit while decreasing frequency (so that we don\u0027t reduce\nthe frequency immediately after the short busy period and that may avoid\nfrequency transitions before the next busy period).\n\nImplement separate up/down transition rate limits. Note that the\ngovernor avoids frequency recalculations for a period equal to minimum\nof up and down rate-limit. A global mutex is also defined to protect\nupdates to min_rate_limit_us via two separate sysfs files.\n\nNote that this wouldn\u0027t change behavior of the schedutil governor for\nthe platforms which wish to keep same values for both up and down rate\nlimits.\n\nThis is tested with the rt-app [1] on ARM Exynos, dual A15 processor\nplatform.\n\nTestcase: Run a SCHED_OTHER thread on CPU0 which will emulate work-load\nfor X ms of busy period out of the total period of Y ms, i.e. Y - X ms\nof idle period. The values of X/Y taken were: 20/40, 20/50, 20/70, i.e\nidle periods of 20, 30 and 50 ms respectively. These were tested against\nvalues of up/down rate limits as: 10/10 ms and 10/40 ms.\n\nFor every test we noticed a performance increase of 5-10% with the\nschedutil governor, which was very much expected.\n\n[Viresh]: Simplified user interface and introduced min_rate_limit_us +\n\t      mutex, rewrote commit log and included test results.\n\n[1] https://github.com/scheduler-tools/rt-app/\n\nChange-Id: I44fc7adc155a126ea61f6956a15ca1a02fdc4692\nSigned-off-by: Steve Muckle \u003csmuckle.linux@gmail.com\u003e\nSigned-off-by: Viresh Kumar \u003cviresh.kumar@linaro.org\u003e\n(applied from https://marc.info/?l\u003dlinux-kernel\u0026m\u003d147936011103832\u0026w\u003d2)\n[trivial adaptations]\nSigned-off-by: Juri Lelli \u003cjuri.lelli@arm.com\u003e\n[updated rate limiting \u0026 fixed conflicts]\nSigned-off-by: Chris Redpath \u003cchris.redpath@arm.com\u003e\n(cherry picked from commit 50c26fdb74563ec0cb4a83373d42667f4e83a23e)\n[Trivial cherry-pick conflicts]\nSigned-off-by: Quentin Perret \u003cquentin.perret@arm.com\u003e\nGit-commit: 50c26fdb74563ec0cb4a83373d42667f4e83a23e\nGit-repo: https://android.googlesource.com/kernel/common/\n[gsantosh@codeaurora.org: refactored variable to fix compilation errors\n  also fixed checkpatch errors]\nSigned-off-by: Santosh Mardi \u003cgsantosh@codeaurora.org\u003e\n","web_links":[{"name":"GitHub","tooltip":"Open in GitWeb","url":"https://github.com/LineageOS/android_kernel_asus_sm8150/commit/57a879b004b129dbf54beaa074132aeee838fcd8"}],"resolve_conflicts_web_links":[{"name":"GitHub","tooltip":"Open in GitWeb","url":"https://github.com/LineageOS/android_kernel_asus_sm8150/commit/57a879b004b129dbf54beaa074132aeee838fcd8"}]},"branch":"refs/heads/lineage-16.0"}},"requirements":[],"submit_records":[],"submit_requirements":[]}