Bug 32797 - dkms-anbox need updating for kernel 6.6
Summary: dkms-anbox need updating for kernel 6.6
Status: RESOLVED WONTFIX
Alias: None
Product: Mageia
Classification: Unclassified
Component: RPM Packages (show other bugs)
Version: 9
Hardware: All Linux
Priority: High major
Target Milestone: ---
Assignee: Chris Denice
QA Contact:
URL:
Whiteboard: IN_ERRATA9
Keywords:
Depends on:
Blocks: 32467
  Show dependency treegraph
 
Reported: 2024-01-31 21:38 CET by Morgan Leijström
Modified: 2024-02-15 06:54 CET (History)
6 users (show)

See Also:
Source RPM: dkms-anbox
CVE:
Status comment:


Attachments
Changes from current spec in mageia 9 (525 bytes, patch)
2024-01-31 22:28 CET, katnatek
Details | Diff

Description Morgan Leijström 2024-01-31 21:38:52 CET
Description of problem:
dkms-anbox need updating for kernel 6.6

Also probably good for waydroid
Bug 32467 - On mga9, we cannot start waydroid due to various errors

Also see Bug 29611
Morgan Leijström 2024-01-31 21:41:40 CET

Blocks: (none) => 32786

Morgan Leijström 2024-01-31 21:43:19 CET

Blocks: (none) => 32467

Morgan Leijström 2024-01-31 21:44:05 CET

Priority: Normal => High

katnatek 2024-01-31 22:23:11 CET

CC: (none) => eatdirt, ngompa13

Comment 1 katnatek 2024-01-31 22:28:20 CET
Created attachment 14322 [details]
Changes from current spec in mageia 9

@Chris Denice, I test the recent commit and works for kernel 6.6 server and linus and at less with current kernel 6.5 desktop

With kernel 6.6 desktop in my hardware I get kernel panic if I add the psi=1 option
Comment 2 katnatek 2024-01-31 23:36:43 CET
(In reply to Morgan Leijström from comment #0)
> Description of problem:
> dkms-anbox need updating for kernel 6.6
> 
> Also probably good for waydroid
> Bug 32467 - On mga9, we cannot start waydroid due to various errors
> 
> Also see Bug 29611

Bug 29611 was solved with the updates of Bug 32496
katnatek 2024-02-01 00:03:02 CET

Whiteboard: (none) => MGA9TOO
Version: 9 => Cauldron

Comment 3 katnatek 2024-02-01 00:31:36 CET
(In reply to katnatek from comment #1)
> Created attachment 14322 [details]
> Changes from current spec in mageia 9
> 
> @Chris Denice, I test the recent commit and works for kernel 6.6 server and
> linus and at less with current kernel 6.5 desktop
> 
> With kernel 6.6 desktop in my hardware I get kernel panic if I add the psi=1
> option

Not need to add psi=1, in the first reboot after install dkms-anbox the modules are build and if reboot again I get the panic
Comment 4 Chris Denice 2024-02-01 09:36:25 CET
I'll have a look if upstream provides new patches!

But if not, it is a lost fight. May be we should drop the whole anbox packages and waydroid. If we're longer to push dkms-anbox updates than we update kernels that break them, there is no point of working on that :-/

Maybe we should also think about our kernel update policy...
Comment 5 katnatek 2024-02-01 19:08:27 CET
(In reply to Chris Denice from comment #4)
> I'll have a look if upstream provides new patches!
> 
> But if not, it is a lost fight. May be we should drop the whole anbox
> packages and waydroid. If we're longer to push dkms-anbox updates than we
> update kernels that break them, there is no point of working on that :-/
> 
> Maybe we should also think about our kernel update policy...

We follow lts versions 6.5 is EOL is not sure keep it
I hope you find some fix, but if not we most recommend use just kernel server or linus flavors
Comment 6 Guillaume Bedot 2024-02-03 08:06:28 CET
Which kernels need dkms-anbox and which are not ?
It seems to me that's already built-in for some time...

$ zgrep ANDR /boot/config-6.6.14-desktop-2.mga10
CONFIG_X86_ANDROID_TABLETS=m
CONFIG_ANDROID_BINDER_IPC=y
CONFIG_ANDROID_BINDERFS=y
CONFIG_ANDROID_BINDER_DEVICES="binder,hwbinder,vndbinder"
# CONFIG_ANDROID_BINDER_IPC_SELFTEST is not set

CC: (none) => geex+mageia

Comment 7 katnatek 2024-02-03 18:49:58 CET
(In reply to Guillaume Bedot from comment #6)
> Which kernels need dkms-anbox and which are not ?
> It seems to me that's already built-in for some time...
> 
> $ zgrep ANDR /boot/config-6.6.14-desktop-2.mga10
> CONFIG_X86_ANDROID_TABLETS=m
> CONFIG_ANDROID_BINDER_IPC=y
> CONFIG_ANDROID_BINDERFS=y
> CONFIG_ANDROID_BINDER_DEVICES="binder,hwbinder,vndbinder"
> # CONFIG_ANDROID_BINDER_IPC_SELFTEST is not set

I got kernel panics with kernel 6.6 after install dkms-anbox and reboot twice, build the recent commit mitigate the situation, but works just with server and linus flavors, if you want to make the test in your system be sure to have a kernel 6.5 as alternative
Comment 8 Chris Denice 2024-02-03 21:38:52 CET
All right,

dkms-anbox-0.0.3-1.3.mga9.src.rpm

will soon be available on updates_testing!
Comment 9 katnatek 2024-02-03 23:58:12 CET
Package in 9/Core/Updates_testing:
=======================
dkms-anbox-0.0.3-1.3.mga9.noarch.rpm

From SRPM:
dkms-anbox-0.0.3-1.3.mga9.src.rpm

Assignee: bgmilne => qa-bugs

Comment 10 katnatek 2024-02-04 00:13:36 CET
(In reply to Chris Denice from comment #8)
> All right,
> 
> dkms-anbox-0.0.3-1.3.mga9.src.rpm
> 
> will soon be available on updates_testing!

Maybe due dracut and microcode testing packages but now just kernel server don't panic :(

This need other test if is just my system then we can add a Errata just in case other users have similar hardware

Need to test with kernel 6.6 desktop and linus https://bugs.mageia.org/show_bug.cgi?id=32786#c75, and have server as fallback option

Install current version of dkms-anbox
Reboot 
Select Advanced options
Select one of the 6.6 kernels 
Wait until dm screen or desktop load if you have autologin
Reboot
If the panic is produced 
Press Alt + Print Screen + R E I S U B 
Select Advanced options
Select other kernel

Update to testing version and repeat the test
If the panic is produced after Press Alt + Print Screen + R E I S U B 
Select Advanced options
Select other kernel
Uninstall dkms-anbox

Please confirm if you get a kernel panic or not

I will test on my i586 hardware
Comment 11 katnatek 2024-02-04 02:53:19 CET
Tested on real hardware mageia 9 i586

Tested
-kernel-desktop-6.6.14-2.mga9
-kernel-desktop586-6.6.14-2.mga9
-kernel-server-6.6.14-2.mga9
-kernel-linus-6.6.14-1.mga9

Current and testing versions of dkms-anbox, not issues , so my current x86_64 system have some issues with the combination of kernels 6.6 != server and dkms-anbox
Comment 12 Marja Van Waes 2024-02-05 21:51:19 CET
advisory was uploaded by tarazed

CC: (none) => marja11
Keywords: (none) => advisory

Marja Van Waes 2024-02-07 22:11:39 CET

Whiteboard: MGA9TOO => (none)
Version: Cauldron => 9

Thomas Andrews 2024-02-08 22:15:57 CET

CC: (none) => andrewsfarm
Blocks: 32786 => (none)

Comment 13 Thomas Andrews 2024-02-08 22:25:23 CET
Sending this back until the Waydroid issues can be resolved. As it is, it just isn't ready for QA yet.

Assignee: qa-bugs => eatdirt

Comment 14 Morgan Leijström 2024-02-08 23:36:54 CET
https://wiki.mageia.org/en/Mageia_9_Errata#Various_software

Whiteboard: (none) => IN_ERRATA9

Comment 15 Morgan Leijström 2024-02-09 17:55:14 CET
Is dkms-anbox really needed?

From https://ml.mageia.org/l/arc/qa-discuss/2024-02/msg00053.html :


> Why aren't we just turning on binderfs in the kernel? We don't need
> dkms-anbox if we have the features turned on in the kernel itself.

CONFIG_ANDROID_BINDERFS=y

is enabled in kernel-desktop (not in kernel-server, of course), so according to that it should work already without dkms-anbox.
Comment 16 Chris Denice 2024-02-09 18:09:22 CET
As far as I see

1) binderfs is not anbox, so you're not going to replace anbox by binderfs.

2) anbox was supposed to support binderfs:

https://github.com/anbox/anbox/pull/1309

3) Good spot that kernel server are not built with binderfs! That may explain the kernel panic observed with all but kernel-servers, possibly that support is actually broken.

4) If there is a way to make waydroid working directly with binderfs, that would allow us to drop dkms-anbox, but I did not find any.

Finally, I cannot tutor QA testers at their home to make waydroid working, and parts of waydroid are out-of-mageia and cannot be fixed within mageia. But I'll have another full test to see if this is still working for me.
Comment 17 Neal Gompa 2024-02-09 18:10:54 CET
Waydroid already does in Fedora, that's why Waydroid is packaged there.
Comment 18 Chris Denice 2024-02-09 18:25:08 CET
Ah good, I'll have a look to their packaging then!!
Comment 19 Marja Van Waes 2024-02-11 20:55:54 CET
Removing the advisory keyword, as it will likely be needed to change or drop the uploaded one.

Keywords: advisory => (none)

Comment 21 Chris Denice 2024-02-12 23:49:07 CET
I am on it. All that mess comes from kernel-6.6 because it actually has binderfs built in, indeed, but not our kernels <=6.5

So, waydroid should be easier to run on >= 6.6, but people having kernel <=6.5 will still need dkms-anbox.

I giving a last push to everything , remove the Requires:dkms-anbox in waydroid, but we should let that package in mageia 9 for people running kernels <=6.5.
Comment 22 Chris Denice 2024-02-13 00:23:46 CET
First, let's close this bug as dkms-anbox is not needed anymore for waydroid.

Resolution: (none) => WONTFIX
Status: NEW => RESOLVED

Morgan Leijström 2024-02-13 10:44:39 CET

See Also: (none) => https://bugs.mageia.org/show_bug.cgi?id=32467

david Cossé 2024-02-15 06:54:01 CET

CC: (none) => saveurlinux


Note You need to log in before you can comment on or make changes to this bug.