Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Boot Optimization: Custom Bootanimation zip file #330

Open
wants to merge 539 commits into
base: master
Choose a base branch
from

Conversation

vishalsh94
Copy link

The default bootanimation.zip file contained images & sequence in which they
are displayed to animate till bootcomplete. But it was seen that even after
bootcomplete there were 98 to 157 frames left in animation. At 30fps, this
meant a delay of 3.2 to 5.2sec to display the home screen even after background
threads are completed. The 2sec uncertainty in delay is due to the 60frame
animation loop which runs till bootcomplete. For combating this there were 2
options, either to completely remove animation for achieving best boottime, or
to keep considerable animation without much CPU overhead & ZERO post
bootcomplete delay. As Celadon is a development platform, and developers
expect something on the bootscreen to infer booting, we applied the later
solution, to get close to best results keeping minimal animation.

With this bootanimation, following changes will appear:

  • rotating balls loop will not appear
  • formation of android logo from 4balls will take place pre-bootcomplete
  • transition of android logo to black screen removed
  • after animation is over, android logo will remain on the screen till
    bootcomplete

Following were the scenarios that were tested along with data on APL-NUC:
+--- +-----------------------------------------------------------------------+-------------------+
| # Description | Boot-time
| 1. No Boot Animation (Black screen) | 18.94 s
| 2. Single Frame (ANDROID logo) | 19.30 s
| 3. Above mentioned Animation w/ jerky home transition | 19.36 s <-Applied
| 4. Above Animation with smooth home screen transition| 20.62 s
| 5. Default Animation w/o any change | 25.45 s
+----------------------------------------------------------------------------+-------------------+

The default zip file is present in packages directory, the custom zip file is
kept here and will be copied to product/media after build with a mixins patch.

Tracked-On: OAM-72177
Signed-off-by: Vishal Sharma [email protected]

sgnanase and others added 30 commits July 19, 2018 18:18
HWComposer will check this flag to disable va renderer if
va driver is not ready.

Change-Id: Ice9e791cdc5a55f237a499639e1122a199517f96
Tracked-On: https://jira01.devtools.intel.com/browse/OAM-64421
Signed-off-by: Yuanjun Huang <[email protected]>
Jira: None
Test: None

Signed-off-by: TianmiChen <[email protected]>
indicate video driver projectceladon#36

Tracked-On: None
Signed-off-by: Swaroop Balan <[email protected]>
Set the fastboot_min_battery_level = false in mixins.spec.
The old code set fastboot_min_battery_level=3500, but the Joule and NUC
do not support battery.

Jira: https://01.org/jira/browse/CEL-29
Test: Test it in Joule, can use PFT now.
      Since KBL NUC does not support PFT now, need not to test it.

Signed-off-by: Ming Tan <[email protected]>
Tracked-On: None
Signed-off-by: Rakesh Reddy <[email protected]>
 made permissive

Tracked-On: None
  made permissive

Tracked-On : None
Hostapd is a vendor interface on Android P, hence Hostapd
hal entry should be added into manifests

Tracked-On: None
Signed-off-by: Raveendra, Babu Chennakesavulu <[email protected]>
Signed-off-by: Jeevaka Prabu Badrappan <[email protected]>
Tracked-On: None

Signed-off-by: Raveendra, Babu Chennakesavulu <[email protected]>
Signed-off-by: Jeevaka Prabu Badrappan <[email protected]>
Tracked-On: None
Signed-off-by: rnaidu <[email protected]>
Jira: None
Test: None
Signed-off-by: Huang, Yang <[email protected]>
Jira: None
Test: Device can boot to home screen.
Signed-off-by: swei22 <[email protected]>
Tracked-On: None
Signed-off-by: Swaroop Balan <[email protected]>
Jira: OAM-65293
Test: Done.

Signed-off-by: tianmi.chen <[email protected]>
Jira: None
Test: Device can boot to UI and no obvious regressions were found.

Tracked-On: None
Signed-off-by: ji, zhenlong z <[email protected]>
Tracked-On: None
Signed-off-by: Swaroop Balan <[email protected]>
…ndence of efitools. projectceladon#48

Tracked-On: None
Signed-off-by: Swaroop Balan <[email protected]>
Tracked-On: OAM-67968
Signed-off-by: Swaroop Balan <[email protected]>
Due to missing sepolicies for audio, audio server is getting
restarted and as a result bluetooth service also gets restarted.

Add sepolicies for audio to get audio and bluetooth working.

Tracked-On: OAM-67968

Signed-off-by: Jeevaka Prabu Badrappan <[email protected]>
Signed-off-by: ji, zhenlong z <[email protected]>
Signed-off-by: Ding XinX <[email protected]>
Signed-off-by: Qiming Shi <[email protected]>
buildslave and others added 26 commits February 21, 2019 00:08
iwlwifi: Remove wlan_intel_restore as its not used

Tracked-On: OAM-76216
Mixin-Reviewed-On: projectceladon/device-androidia-mixins#203
Signed-off-by: Jeevaka Prabu Badrappan <[email protected]>
Tracked-On: OAM-76208
Signed-off-by: ji, zhenlong z <[email protected]>
Tracked-On: OAM-76218
Signed-off-by: ji, zhenlong z <[email protected]>
Tracked-On: OAM-76212
Signed-off-by: ji, zhenlong z <[email protected]>
Disable new AVRCP in IVI
As new AVRCP is having issues with IVI package, disable the
new AVRCP by setting property persist.bluetooth.enablenewavrcp
to false.

Tracked-On: OAM-76215
Mixin-Reviewed-On: projectceladon/device-androidia-mixins#207
Signed-off-by: Harshita Goswami <[email protected]>
Tracked-On: OAM-76278
Signed-off-by: swaroopb <[email protected]>
set ro.crypto.volume.filenames_mode=aes-256-cts in system.property

Tracked-On: https://jira.devtools.intel.com/browse/OAM-76267
Mixin-Reviewed-On: projectceladon/device-androidia-mixins#212
Signed-off-by: Zhiwei Li [email protected]
Change-Id: I80d24777b9b9f7733c42c949d9170015dfbe4cdb
Tracked-On: OAM-76314
Signed-off-by: Yan, WalterX <[email protected]>
Tracked-On: OAM-76314
Signed-off-by: Yan, WalterX <[email protected]>
Tracked-On: OAM-76279
Signed-off-by: tianmi.chen <[email protected]>
Tracked-On: OAM-76272
Signed-off-by: Hongfu Ruan <[email protected]>
Tracked-On: OAM-76272
Signed-off-by: Hongcheng Xie <[email protected]>
Signed-off-by: Youhua Zhu <[email protected]>
Tracked-On: OAM-76272
Signed-off-by: Hongcheng Xie <[email protected]>
Signed-off-by: Youhua Zhu <[email protected]>
Add default drm mixin group
Add drm-default sepolicy

Tracked-On: OAM-76272
Mixin-Reviewed-On: projectceladon/device-androidia-mixins#217
Mixin-Reviewed-On: projectceladon#299
Signed-off-by: Hongcheng Xie <[email protected]>
Signed-off-by: Youhua Zhu <[email protected]>
- Added socket permissions for netd, system_server
- Add getattr permissions for dnsmask
- Add dontaudit for some of the reported sepolicy errors.
- Add user_build_only macro

Tracked-On: OAM-76271
Signed-off-by: ji, zhenlong z <[email protected]>
Signed-off-by: Jeevaka Prabu Badrappan <[email protected]>
Add vendor sepolicy directory

Tracked-On: OAM-76271
Mixin-Reviewed-On: projectceladon/device-androidia-mixins#202
Signed-off-by: Jeevaka Prabu Badrappan <[email protected]>
Tracked-On: OAM-76710
Signed-off-by: sgnanase <[email protected]>
Tracked-On: OAM-76710
Signed-off-by: sgnanase <[email protected]>
Call the set_storage only in disk-bus

Tracked-On: OAM-74966
Mixin-Reviewed-On: projectceladon/device-androidia-mixins#222
Signed-off-by: Yang, Kai <[email protected]>
Tracked-On: OAM-76670
Signed-off-by: tianmi.chen <[email protected]>
Required audio congiguration files and mixer_paths should be
present in Vendor partition. Moving same to vendor partition.

Tracked-On: OAM-76659
Signed-off-by: Karan Patidar <[email protected]>
Tracked-On: OAM-76710
Signed-off-by: Jeevaka Prabu Badrappan <[email protected]>
reserve 50m /data usage for system usage.

Tracked-On: https://jira.devtools.intel.com/browse/OAM-76809
Mixin-Reviewed-On: projectceladon/device-androidia-mixins#232
Signed-off-by: Zhiwei Li [email protected]
Set GenCopying as default for celadon IVI
Add a new group dalvik-heap to enable setting of bpssize

Tracked-On: OAM-76016
Mixin-Reviewed-On: projectceladon/device-androidia-mixins#200
Signed-off-by: Amrita H S <[email protected]>
Tracked-On: OAM-76660
Signed-off-by: N, Shyjumon <[email protected]>
Signed-off-by: Salih, MuhammadhX <[email protected]>
The default bootanimation.zip file contained images & sequence in which they
are displayed to animate till bootcomplete. But it was seen that even after
bootcomplete there were 98 to 157 frames left in animation. At 30fps, this
meant a delay of 3.2 to 5.2sec to display the home screen even after background
threads are completed. The 2sec uncertainty in delay is due to the 60frame
animation loop which runs till bootcomplete. For combating this there were 2
options, either to completely remove animation for achieving best boottime, or
to keep considerable animation without much CPU overhead & ZERO post
bootcomplete delay. As Celadon is a development platform, and developers
expect something on the bootscreen to infer booting, we applied the later
solution, to get close to best results keeping minimal animation.

With this bootanimation, following changes will appear:
  - rotating balls loop will not appear
  - formation of android logo from 4balls will take place pre-bootcomplete
  - transition of android logo to black screen removed
  - after animation is over, android logo will remain on the screen till
    bootcomplete

Following were the scenarios that were tested along with data on APL-NUC:
+----+---------------------------------------------------+-------------------+
| #  |           Description                             |     Boot-time     |
| 1  | No Boot Animation (Black screen)                  |  18.94 s          |
| 2  | Single Frame (ANDROID logo)                       |  19.30 s          |
| 3  | Above mentioned Animation w/ jerky home transition|  19.36 s <-Applied|
| 4  | Above Animation with smooth home screen transition|  20.62 s          |
| 5  | Default Animation w/o any change                  |  25.45 s          |
+----+---------------------------------------------------+-------------------+

The default zip file is present in packages directory, the custom zip file is
kept here and will be copied to product/media after build with a mixins patch.

Tracked-On: OAM-72177
Signed-off-by: Vishal Sharma <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Stale Stale label for inactive open prs
Projects
None yet
Development

Successfully merging this pull request may close these issues.