Port to Oneplus 2

UniSuperBox

That means that the rootfs is never starting. It's possible that the kernel is panicking. Take a look at the cmdline you're passing to the kernel, ensure that there's no "console=" directive or anything similar. The cmdline is specified in device/oneplus/oneplus2/BoardConfig.mk

SGCMarkus

The complete command line (maybe something missing which i overread?)
BOARD_KERNEL_CMDLINE := androidboot.hardware=qcom user_debug=31 msm_rtb.filter=0x37 ehci-hcd.park=3 lpm_levels.sleep_disabled=1 boot_cpus=0-5

Also noticed that the recovery.img that was created doesnt boot either so i guess a bit more is going wrong (i tried to boot it with fastboot boot recovery.img and it loaded a bit, rebooted and booted my normal installed recovery)

UniSuperBox

@SGCMarkus Interesting.
Finding why the recovery won't boot would be the number 1 priority... Try to remove as much of the cmdline as possible. Just leave, say, "androidboot.hardware=qcom boot_cpus=0-5".

SGCMarkus

@UniSuperBox
I suppose i used a wrong kernel then? I think i have currently the CM13 kernel source, which is CAF as far as i know.

Will try what you said, and some other kernels tomorrow, hopefully that will work

Edit: appearantly "TARGET_USES_UNCOMPRESSED_KERNEL" was set to true... and the new firmware needs a compressed kernel... so i guess thats the main reason it failed, building now again, this time compressed and with an AOSP kernel

SGCMarkus

So... I feel stupid now... forgot completly about the kernel patches that are described in the the official porting guide ...

With the script thats provided there, i set the variables needed, but now i get an error while its compiling the kernel

  CC      kernel/irq/resend.o
In file included from /home/markus/ubport_6/kernel/oneplus/msm8994/include/net/inet_hashtables.h:32:0,
                 from /home/markus/ubport_6/kernel/oneplus/msm8994/include/net/tcp.h:37,
                 from /home/markus/ubport_6/kernel/oneplus/msm8994/security/selinux/hooks.c:55:
/home/markus/ubport_6/kernel/oneplus/msm8994/include/net/route.h: In function 'ip_route_output_ports':
/home/markus/ubport_6/kernel/oneplus/msm8994/include/net/route.h:146:55: error: type mismatch in conditional expression
       daddr, saddr, dport, sport, sk ? sock_i_uid(sk) : 0);
                                                       ^
/home/markus/ubport_6/kernel/oneplus/msm8994/include/net/route.h: In function 'ip_route_connect_init':
/home/markus/ubport_6/kernel/oneplus/msm8994/include/net/route.h:256:2: error: incompatible type for argument 12 of 'flowi4_init_output'
  flowi4_init_output(fl4, oif, sk->sk_mark, tos, RT_SCOPE_UNIVERSE,
  ^
In file included from /home/markus/ubport_6/kernel/oneplus/msm8994/include/linux/netfilter.h:275:0,
                 from /home/markus/ubport_6/kernel/oneplus/msm8994/include/uapi/linux/netfilter_ipv4.h:8,
                 from /home/markus/ubport_6/kernel/oneplus/msm8994/include/linux/netfilter_ipv4.h:7,
                 from /home/markus/ubport_6/kernel/oneplus/msm8994/security/selinux/hooks.c:49:
/home/markus/ubport_6/kernel/oneplus/msm8994/include/net/flow.h:87:20: note: expected 'uid_t' but argument is of type 'kuid_t'
 static inline void flowi4_init_output(struct flowi4 *fl4, int oif,
                    ^
/home/markus/ubport_6/kernel/oneplus/msm8994/scripts/Makefile.build:308: recipe for target 'security/selinux/hooks.o' failed
make[4]: *** [security/selinux/hooks.o] Error 1
/home/markus/ubport_6/kernel/oneplus/msm8994/scripts/Makefile.build:455: recipe for target 'security/selinux' failed
make[3]: *** [security/selinux] Error 2
/home/markus/ubport_6/kernel/oneplus/msm8994/Makefile:843: recipe for target 'security' failed
  CC      crypto/twofish_common.o
make[2]: *** [security] Error 2
make[2]: *** Waiting for unfinished jobs....

/home/markus/ubport_6/kernel/oneplus/msm8994/kernel/cgroup.c: In function 'subsys_cgroup_allow_attach':
/home/markus/ubport_6/kernel/oneplus/msm8994/kernel/cgroup.c:2138:37: error: invalid operands to binary != (have 'kuid_t' and 'kuid_t')
   if (current != task && cred->euid != tcred->uid &&
                                     ^
/home/markus/ubport_6/kernel/oneplus/msm8994/kernel/cgroup.c:2139:18: error: invalid operands to binary != (have 'kuid_t' and 'kuid_t')
       cred->euid != tcred->suid)
                  ^
/home/markus/ubport_6/kernel/oneplus/msm8994/scripts/Makefile.build:308: recipe for target 'kernel/cgroup.o' failed
make[3]: *** [kernel/cgroup.o] Error 1
make[3]: *** Waiting for unfinished jobs....

/home/markus/ubport_6/kernel/oneplus/msm8994/fs/exfat/exfat_super.c: In function 'exfat_allow_set_time':
/home/markus/ubport_6/kernel/oneplus/msm8994/fs/exfat/exfat_super.c:1145:22: error: invalid operands to binary != (have 'kuid_t' and 'kuid_t')
  if (current_fsuid() != inode->i_uid) {
                      ^
/home/markus/ubport_6/kernel/oneplus/msm8994/fs/exfat/exfat_super.c: In function 'exfat_setattr':
/home/markus/ubport_6/kernel/oneplus/msm8994/fs/exfat/exfat_super.c:1218:18: error: invalid operands to binary != (have 'kuid_t' and 'uid_t')
    (attr->ia_uid != sbi->options.fs_uid)) ||
                  ^
/home/markus/ubport_6/kernel/oneplus/msm8994/fs/exfat/exfat_super.c:1220:18: error: invalid operands to binary != (have 'kgid_t' and 'gid_t')
    (attr->ia_gid != sbi->options.fs_gid)) ||
                  ^
/home/markus/ubport_6/kernel/oneplus/msm8994/fs/exfat/exfat_super.c: In function 'exfat_fill_inode':
/home/markus/ubport_6/kernel/oneplus/msm8994/fs/exfat/exfat_super.c:1677:15: error: incompatible types when assigning to type 'kuid_t' from type 'uid_t'
  inode->i_uid = sbi->options.fs_uid;
               ^
/home/markus/ubport_6/kernel/oneplus/msm8994/fs/exfat/exfat_super.c:1678:15: error: incompatible types when assigning to type 'kgid_t' from type 'gid_t'
  inode->i_gid = sbi->options.fs_gid;
               ^
/home/markus/ubport_6/kernel/oneplus/msm8994/fs/exfat/exfat_super.c: In function 'parse_options':
/home/markus/ubport_6/kernel/oneplus/msm8994/fs/exfat/exfat_super.c:2047:15: error: incompatible types when assigning to type 'uid_t' from type 'kuid_t'
  opts->fs_uid = current_uid();
               ^
/home/markus/ubport_6/kernel/oneplus/msm8994/fs/exfat/exfat_super.c:2048:15: error: incompatible types when assigning to type 'gid_t' from type 'kgid_t'
  opts->fs_gid = current_gid();
               ^
/home/markus/ubport_6/kernel/oneplus/msm8994/fs/exfat/exfat_super.c: In function 'exfat_read_root':
/home/markus/ubport_6/kernel/oneplus/msm8994/fs/exfat/exfat_super.c:2182:15: error: incompatible types when assigning to type 'kuid_t' from type 'uid_t'
  inode->i_uid = sbi->options.fs_uid;
               ^
/home/markus/ubport_6/kernel/oneplus/msm8994/fs/exfat/exfat_super.c:2183:15: error: incompatible types when assigning to type 'kgid_t' from type 'gid_t'
  inode->i_gid = sbi->options.fs_gid;
               ^
/home/markus/ubport_6/kernel/oneplus/msm8994/scripts/Makefile.build:308: recipe for target 'fs/exfat/exfat_super.o' failed
make[4]: *** [fs/exfat/exfat_super.o] Error 1
/home/markus/ubport_6/kernel/oneplus/msm8994/scripts/Makefile.build:455: recipe for target 'fs/exfat' failed
make[3]: *** [fs/exfat] Error 2
make[3]: *** Waiting for unfinished jobs....

and should CONFIG_DEFAULT_SECURITY be "apparmor" or "selinux" ? the script didnt set it to "apparmor" on first run

Or is even something else making problems now suddenly

EDIT: a few more errors that appeared at the same time which i didnt see

UniSuperBox

@SGCMarkus

It should be apparmor. These config changes will probably cause more build errors, so don't be alarmed.

Since the device uses a 3.10 kernel, you're going to need the commits from other 3.10 kernels, like I used on my tree for the Nexus 5X: https://github.com/usb-bullhead-ubuntu-touch/kernel_msm/commits/android-msm-bullhead-3.10-marshmallow-dr1.6-ut

SGCMarkus

@UniSuperBox

What is this bull**** now lol

(!gid_eq(attr->ia_gid, sbi->options.fs_gid))
results in ->
/home/markus/ubport_6/kernel/oneplus/msm8994/fs/exfat/exfat_super.c:1220:6: error: incompatible type for argument 2 of 'gid_eq'
(!gid_eq(attr->ia_gid, sbi->options.fs_gid ))) ||
note: expected 'kgid_t' but argument is of type 'gid_t'

and if i have

(!gid_eq(make_kgid(NULL, attr->ia_gid), make_kgid(NULL, sbi->options.fs_gid)))
it results in ->
error: incompatible type for argument 2 of 'make_kgid'
note: expected 'gid_t' but argument is of type 'kgid_t'

so if i trust the second one, and its already kgid_t, why does it say its not in the first one...? and it has no .val value, so it has to be of type gid_t (what the first one says) so why doesnt the second one not work... what the fuck is going on there O.o

UniSuperBox

@SGCMarkus

You'll need to take a look at the commit log that I sent in my last post. These are known bugs in the 3.10 kernel that you will need to fix.

SGCMarkus

@UniSuperBox applied all that already, and some more that produced similar errors afterwards, atm trying to apply that to everything else that produces errors that have todo with uid_t/kuid_t (gid) ... but that from my last post confuses me, because it says its the other, even though it isnt and should work with the first O.o

UniSuperBox

@SGCMarkus

What file is that in? It looks like something with the filesystem, did you disable CONFIG_EXT4_FS_POSIX_ACL in the defconfig? You'll need to comment out or remove the line, not set it to No.

SGCMarkus

@UniSuperBox

that error is in fs/exfat/exfat_super.c line ~1217 (may be a bit different from versio nto version what i found) in function "exfat_setattr".

Another one:

/home/markus/ubport_6/kernel/oneplus/msm8994/drivers/platform/msm/pft.c: In function 'pft_is_current_process_registered':
/home/markus/ubport_6/kernel/oneplus/msm8994/include/linux/cred.h:332:24: error: incompatible types when initializing type 'u32' using type 'kuid_t'
 #define current_uid()  (current_cred_xxx(uid))
                        ^
/home/markus/ubport_6/kernel/oneplus/msm8994/drivers/platform/msm/pft.c:369:12: note: in expansion of macro 'current_uid'
  u32 uid = current_uid();
            ^
In file included from /home/markus/ubport_6/kernel/oneplus/msm8994/include/linux/kernel.h:14:0,
                 from /home/markus/ubport_6/kernel/oneplus/msm8994/drivers/platform/msm/pft.c:48:
/home/markus/ubport_6/kernel/oneplus/msm8994/drivers/platform/msm/pft.c: In function 'pft_inode_create':
/home/markus/ubport_6/kernel/oneplus/msm8994/include/linux/dynamic_debug.h:64:16: warning: format '%u' expects argument of type 'unsigned int', but argument 4 has type 'kuid_t' [-Wformat=]
error, forbidden warning: dynamic_debug.h:64
  CC      drivers/rtc/rtc-lib.o
/home/markus/ubport_6/kernel/oneplus/msm8994/scripts/Makefile.build:308: recipe for target 'drivers/platform/msm/pft.o' failed
make[5]: *** [drivers/platform/msm/pft.o] Error 1
make[5]: *** Waiting for unfinished jobs....

Currently rebuilding without any CONFIG_EXT4_FS_POSIX_ACL in the defconfig, will update this post with the result.

UniSuperBox

@SGCMarkus One more thing, it doesn't look like I have any exfat enabled in my kernel. You might consider disabling it for yours, too.

SGCMarkus

@UniSuperBox disabled exfat now... couldnt get it compiled, at all
Otherwise i had... atleast it felt like 1000 other errors in mainly the ipv4 and ipv6 files... with the uid and kuid stuff, got that fixed now and managed a successful build with building and building after error and fixing. Making a clean build now to make sure, and then lets see what happens, hopefully it will atleast do "a bit" more than earlier xD

SGCMarkus

So.. everything compiled, no error, flashed everything, now i just see the "wonderful" logo which comes before the boot animation for like... idk 10 min now. Phone gets kinda warm though, but i dont have access via USB.
Is it normal that the first boot takes a while (even this long)?

Aborted it though now... saw that it created a few folders in /data, user-data, system-data, android-data and i tried to push the modified adbd from the official guide which appearantly should start the adbd process before GUI is able to run. Didnt work

so... is it doing something during this process, or (which i kinda believe) did it fuck up somewhere... but atleast it doesnt instantly reboot anymore, which is good i guess xD

The recovery created "boots", but without any GUI, just a black screen, but adb is running there

UniSuperBox

@SGCMarkus Awesome! You're exactly where I was with the Nexus 5X. There's something wrong with initializing the Android layer, and it seems that dbus is failing as well. Maybe @Flohack would be able to answer what to do from here?

SGCMarkus

Also noticed that the recovery cant mount any of the partitions, as it searches for /dev/block/bootdevice/by-name/ but thats not existing in the recovery it built. Dont know if thats on purpose or not?

But its "good to know" that im not the only one having this problem xD

flohack

@SGCMarkus welcome to porting hell. We all came this way, sometimes there are solutions, sometimes not. Im on a business trip, reply to you soon. You need to boot, wait fir your warm devuce, boot recovery and try to get /proc/last_kmsg somehow. And plz use pastebin for it, not the forum

SGCMarkus

Cant get a /proc/last_kmsg doesnt exist ... i guess i gotta find a way to enable it?

otherwise, i have the kmsg from the recovery, if helpful: link

have fun/good luck on your business trip though ^^ i gotta survive 5 exams next week xD

UniSuperBox

@SGCMarkus

You might want to try the information in this wiki article: https://wiki.ubports.com/wiki/How-to-work-around-missing-ADB-functionality

SGCMarkus

@UniSuperBox thanks, will try that too
had no time the past few days, exams ^^ 2 left, but those are rather small so i have more time now

have a few other ideas too that i wanna check, like https://developer.ubuntu.com/en/phone/devices/porting-new-device/ at debugging, appearantly i can set break points, and it should start adb then, if it "breaks" (hopefully) and some other things a friend of mine (hes good with linux in general) told me

will update here when im done with testing ^^