Skip to content

Conversation

aaaaaaaa-815
Copy link
Contributor

@aviraxp
Copy link
Collaborator

aviraxp commented Oct 7, 2025

Does it work for LKM?

@aaaaaaaa-815
Copy link
Contributor Author

not sure

@hmtheboy154
Copy link
Contributor

Does it work for LKM?

#1785 (comment)

@aaaaaaaa-815
Copy link
Contributor Author

it do work on my devices
photo_2025-10-07_15-16-06

@hmtheboy154
Copy link
Contributor

it do work on my devices

LKM ? Or integrate into the kernel ?

LeCmnGend pushed a commit to LeCmnGend/KernelSU that referenced this pull request Oct 9, 2025
Filtered from tiann/KernelSU tiann#2793

Co-authored-by: libingxuan <[email protected]>
@aaaaaaaa-815
Copy link
Contributor Author

it do work on my devices

LKM ? Or integrate into the kernel ?
into the kernel

@hmtheboy154
Copy link
Contributor

it do work on my devices

LKM ? Or integrate into the kernel ?
into the kernel

ah.... well then it look just like my PR then
#1785

@hmtheboy154
Copy link
Contributor

@backslashxx nvm still no LKM

@aaaaaaaa-815
Copy link
Contributor Author

it do work on my devices

LKM ? Or integrate into the kernel ?
into the kernel

ah.... well then it look just like my PR then #1785

fix another support in group_info

@hmtheboy154
Copy link
Contributor

hmtheboy154 commented Oct 9, 2025

fix another support in group_info

oh I see. But why didn't it cause any error when building for me....

@aaaaaaaa-815
Copy link
Contributor Author

fix another support in group_info

oh I see. But why didn't it cause any error when building for me....

I don't know why but it went wrong when i build kernel

LeCmnGend pushed a commit to LeCmnGend/KernelSU that referenced this pull request Oct 10, 2025
Filtered from tiann/KernelSU tiann#2793

Co-authored-by: libingxuan <[email protected]>
LeCmnGend pushed a commit to LeCmnGend/KernelSU that referenced this pull request Oct 10, 2025
Filtered from tiann/KernelSU tiann#2793

Co-authored-by: libingxuan <[email protected]>
LeCmnGend pushed a commit to LeCmnGend/KernelSU that referenced this pull request Oct 10, 2025
Filtered from tiann/KernelSU tiann#2793

Co-authored-by: libingxuan <[email protected]>
@hmtheboy154
Copy link
Contributor

fix another support in group_info

oh I see. But why didn't it cause any error when building for me....

I don't know why but it went wrong when i build kernel

actually.... can you paste the error when you are building without that group_info part ?

@aaaaaaaa-815
Copy link
Contributor Author

fix another support in group_info

oh I see. But why didn't it cause any error when building for me....

I don't know why but it went wrong when i build kernel

actually.... can you paste the error when you are building without that group_info part ?

drivers/kernelsu/core_hook.c:78:40: error: missing braces around initializer [-Werror=missing-braces]
78 | static struct group_info root_groups = { .usage = ATOMIC_INIT(2) };
| ^
drivers/kernelsu/core_hook.c:78:40: error: missing braces around initializer [-Werror=missing-braces]

@aaaaaaaa-815
Copy link
Contributor Author

fix another support in group_info

oh I see. But why didn't it cause any error when building for me....

I don't know why but it went wrong when i build kernel

actually.... can you paste the error when you are building without that group_info part ?

drivers/kernelsu/core_hook.c:78:40: error: missing braces around initializer [-Werror=missing-braces] 78 | static struct group_info root_groups = { .usage = ATOMIC_INIT(2) }; | ^ drivers/kernelsu/core_hook.c:78:40: error: missing braces around initializer [-Werror=missing-braces]

https://elixir.bootlin.com/linux/v6.7-rc1/source/include/linux/refcount.h#L115

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.

3 participants