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

ksud: Skip patch init_boot.img on kmi android12-xxx devices #1744

Merged
merged 4 commits into from
May 25, 2024

Conversation

HDTianRu
Copy link
Contributor

有一些设备(如ace2v),带了initboot,但ramdisk却在boot里,导致无法使用ota安装与直接安装,所以添加了个开关
因为本人开发环境有点简陋(mt管理器),而且对项目代码不太熟,合并前最好review一下(

@tiann
Copy link
Owner

tiann commented May 19, 2024

你的设备 ace2v 内核具体版本号是多少?

@HDTianRu
Copy link
Contributor Author

你的设备 ace2v 内核具体版本号是多少?

一加ACE 2V
内核版本5.10.198-android12-9
coloros14.0,安卓版本14
系统有init_boot.img
但需要修补boot.img
如直接在管理器进行直接安装或ota安装,会自动选择修补init_boot,导致修补失败,报错退出
管理器界面
直接安装

@tiann
Copy link
Owner

tiann commented May 20, 2024

我认为这种实现方法更好:如果存在 init_boot 分区,但内核版本为 android12-xxxx 那么依然选择 boot 分区,不需要用户去干预。目前遇到的几个设备都是这样的。

@HDTianRu
Copy link
Contributor Author

我认为这种实现方法更好:如果存在 init_boot 分区,但内核版本为 android12-xxxx 那么依然选择 boot 分区,不需要用户去干预。目前遇到的几个设备都是这样的。

我感觉有init_boot分区但需要修补boot分区的抽象设备只是少数,可能还有部分用户是 android12-xxxx 内核且需要正常修补 init_boot 分区,单纯以内核版本判断可能出现新的问题,毕竟因为在正常设备上,管理器可以正常工作,所以他们并不会反馈问题

@HDTianRu
Copy link
Contributor Author

或者我觉得可以先尝试修补 init_boot 分区,解包失败发现没有 ramdisk 且为 android12-xxxx 内核后尝试修补 boot 分区

@tiann
Copy link
Owner

tiann commented May 21, 2024

android12-xxx 绝大部分都是修补 boot 分区,有 init_boot 分区的是极少数,这个逻辑并不会影响到它们。

@HDTianRu HDTianRu reopened this May 21, 2024
@HDTianRu HDTianRu marked this pull request as ready for review May 21, 2024 15:06
@HDTianRu HDTianRu changed the title manager: 添加禁止默认修补init_boot的开关 ksud: Skip patch init_boot.img on kmi android12-xxx devices May 21, 2024
@HDTianRu
Copy link
Contributor Author

好了,希望不会伤害到任何人()

@tiann tiann self-requested a review May 23, 2024 03:09
Copy link
Owner

@tiann tiann left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

请修复一下格式问题

@HDTianRu HDTianRu requested a review from tiann May 23, 2024 14:06
@tiann tiann merged commit 0674841 into tiann:main May 25, 2024
2 checks passed
WenHao2130 pushed a commit to WenHao2130/KernelSU that referenced this pull request Jun 1, 2024
有一些设备(如ace2v),带了initboot,但ramdisk却在boot里,导致无法使用ota安装与直接安装,所以添加了个开关
因为本人开发环境有点简陋(mt管理器),而且对项目代码不太熟,合并前最好review一下(
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants