Add: seamless boot splash, daemon for disabling keyboard and touchpad, and remove gzip compression on initramfs #215

Maccraft123 · 2020-09-15T22:22:52Z

This needs to be tested on arm64 hardware and dumps of 1280x800, 1366x768 and 2400x1600 of splash have to be taken

gregordinary · 2020-09-16T03:38:56Z

Note: I don't have an arm64 chromebook to test with. I do have a NanoPi m4-v2 (RK3399-based) SBC, but I don't think that board is supported here. Still, wanted to try compiling for ARM-64 just to see the process as last time I compiled PrawnOS, this wasn't an option.

Anyway, tried compiling for arm64, first with @Maccraft123 's branch directly and got the following error:

+ mkdir brcm
+ cp /home/USERNAME/Development/git/ShrimpOS/kernel/resources/arm64/brcmfmac4354-sdio.bin brcm/
cp: cannot stat '/home/USERNAME/Development/git/ShrimpOS/kernel/resources/arm64/brcmfmac4354-sdio.bin': No such file or directory
make[1]: *** [/home/USERNAME/Development/git/ShrimpOS/kernel/makefile:68: /home/USERNAME/Development/git/ShrimpOS/build/arm64/linux/vmlinux.kpart] Error 1
make[1]: Leaving directory '/home/USERNAME/Development/git/ShrimpOS'
make: *** [makefile:89: image] Error 2

Tried again using the main repo from @SolidHal creating a local branch from this Pull Request (probably the right way to do this, I'm learning) and received the following:

1+0 records in
1+0 records out
512 bytes copied, 0.000100901 s, 5.1 MB/s
+ vbutil_kernel --pack vmlinux.kpart --version 1 --vmlinuz vmlinux.uimg --arch aarch64 --keyblock /usr/share/vboot/devkeys/kernel.keyblock --signprivate /usr/share/vboot/devkeys/kernel_data_key.vbprivk --config /home/USERNAME/Development/git/PrawnOS/kernel/resources/arm64/cmdline --bootloader bootloader.bin
+ RESULT=0
+ '[' '!' 0 -eq 0 ']'
++ stat -c %s vmlinux.kpart
+ KERNEL_SIZE=35241984
+ '[' 35241984 -gt 33554432 ']'
+ mv vmlinux.kpart oversized_vmlinux.kpart
+ echo 'kernel larger than max kernel size!'
kernel larger than max kernel size!
+ exit 1
make[1]: *** [/home/USERNAME/Development/git/PrawnOS/kernel/makefile:117: /home/USERNAME/Development/git/PrawnOS/build/arm64/linux-5.7.10/vmlinux.kpart] Error 1
make[1]: Leaving directory '/home/USERNAME/Development/git/PrawnOS'
make: *** [makefile:89: image] Error 2

Might the issue be from removing gzip compression?

Maccraft123 · 2020-09-16T13:52:18Z

Oh it barely doesn't fit.
PrawnOS works only on devices that boot with depthcharge, so it won't work on any SBC that doesn't run coreboot.

Maccraft123 · 2020-09-16T15:05:40Z

I think that to fix this someone with arm64 chromebooks should remove some kernel config options to decrease space requirements and test if it still works fine

Maccraft123 · 2020-09-17T12:51:00Z

Update: blocked until #216 is merged

SolidHal · 2020-09-17T14:05:35Z

I'm not surprised we went over, I made the config quite sparse and it just barely fit.

Maccraft123 · 2020-09-17T14:47:13Z

I guess it's time to do kernel module support

SolidHal · 2020-09-17T15:37:28Z

kernel size bump is merged, pulling this to test

SolidHal · 2020-09-17T16:09:29Z

on an armhf build I get prompted for the following:

scripts/kconfig/conf  --syncconfig Kconfig
*
* Restart config...
*
*
* printk and dmesg options
*
Show timing information on printks (PRINTK_TIME) [Y/n/?] y
Show caller information on printks (PRINTK_CALLER) [N/y/?] n
Default console loglevel (1-15) (CONSOLE_LOGLEVEL_DEFAULT) [7] (NEW)

looks like this PR sets it to 0, which is not a valid value

that said, it looks like we manage the console log level in the initramfs, so is this needed?

SolidHal · 2020-09-17T16:23:57Z

Could I get some insight into why we want these systemd settings? We might need to implement these settings in a different way, as the file will get overwritten anytime the systemd package is updated.

Maccraft123 · 2020-09-17T16:55:51Z

Those settings are needed to silence systemd because any write to tty overwrites splash
And initramfs loglevel sometimes doesn't work

gdallasdye · 2020-09-18T01:00:02Z

Quick questions:

Is disabling selinux necessary? I do have some selinux troubleshooting experience, have seen plymouth boot screens on selinux, and have a preference for keeping selinux enabled as it's mandatory access controls help maintain security when the old fashioned discretionary aces controls fail. I could go on for hours about selinx btw...

And is this some kind of workaround to implement plymouth boot screens in the PrawnOS initrd system? Since as we know, Debian's initrd generator doesn't support depthcharge targets. If so, this could help me use the plymouth-spinner-theme while booting, which has been a goal :)

Taking a look here it would seem like the system.conf file should be place in /etc/systemd/system.conf.d/system.conf You'll want to tell the script to mkdir -p that directory, then change the folder it gets copied to.

Would you also mind creating another branch and post screenshots and camera pictures to visually documenting the changes? A few weeks ago I made a branch called readme-minor-changes, which implies a future branch with bigger changes. But that's another topic for another day.

Maccraft123 · 2020-09-18T05:22:05Z

Selinux displays some error message about some file not found, that overwrites part of splash.
And this is NOT plymouth. I made a youttube video displaying how it looks with gdm, https://youtu.be/Ha1-sOUqaDY
With other display managers it will work same way. I can make some pictures after it gets merged.

This reverts commit 471da0d.

Maccraft123 · 2020-09-18T06:20:09Z

About selinux concerns: I've reverted commit that disables it.

SolidHal · 2020-09-17T16:27:58Z

initramfs/resources/initramfs-init

 #If its an mmcblk device, the kernel partiton will p1. If it is a usb device, the partiton will just be 1
 #Just want everything before the 1
 ROOT_DEV="${BLKID%1:*}"

-echo ${ROOT_DEV} > /dev/tty1
+
+# happens when kernel takes its time looking for devices


I have done many boot tests on veyron-speedy and gru-kevin and have never seen this happen. I'm not convinced it could happen, as we are already booting from the device we are looking for. What situations did you see this happen?

I was testing images on my microSD card and it has happened a few times
It may be due to its old age, or bad quality, either way, it won't hurt i think

SolidHal · 2020-09-17T16:37:21Z

initramfs/resources/initramfs-init

@@ -17,11 +17,9 @@
 # You should have received a copy of the GNU General Public License
 # along with PrawnOS.  If not, see <https://www.gnu.org/licenses/>.



I see some potential issues for debugging with how the splash is implemented, which would be fixed by the following:

some way to disable the splash and instead get all of the console output/debug echos. Doing this without rebuilding the kernel and initramfs would be nice. I'm not sure if this is possible though...

keep the debug logging if we are not presenting the splash screen

the fix for (2) needs to ensure that if we do enter the rescue_shell, we print out the debug messages

Splash can be removed with just simple 'clear' shell command
And inserting device with RESCUESHELL label should be enough, it jumps to shell before splash screen

filesystem/resources/issue

filesystem/packages/font-source-code-pro/font-source-code-pro_1.010-2.dsc

SolidHal · 2020-09-17T17:02:23Z

scripts/BuildScripts/FilesystemScripts/buildFilesystem.sh

+cp $build_resources/system.conf $outmnt/etc/systemd/
+
+#install hwdb file for iio-sensor-proxy to work
+printf 'sensor:modalias:platform:*\n ACCEL_MOUNT_MATRIX=-1, 0, 0; 0, -1, 0; 0, 0, -1\n' > $outmnt/etc/udev/hwdb.d/61-sensor-local.hwdb


I see parts of this repeated in the buildFilesystem and the InstallPackages scripts, are both necessary?

When i was testing, when it was only on buildFilesystem or InstallPackages it was either not working on system before installing, or after installing.

oh and it needs to be tested on kevin and bob

This reverts commit 009d681.

SolidHal · 2020-09-23T23:16:23Z

kernel/resources/armhf/config

@@ -5302,8 +5301,7 @@ CONFIG_INTEGRITY=y
 CONFIG_INTEGRITY_AUDIT=y
 # CONFIG_IMA is not set
 # CONFIG_EVM is not set
-CONFIG_DEFAULT_SECURITY_SELINUX=y


This doesn't actually disable selinux, just changes when it is loaded. Instead of loading selinux then U DAC, it loads U DAC first. If this fixes the issue you are seeing, great, its a win-win.

Ah, I see, you reverted the change that disabled SElinux and this config change stayed?

SolidHal · 2020-09-23T23:36:51Z

hm, I got prompted to set LSM_MMAP_MIN_ADDR on the armhf build

SolidHal · 2020-09-24T00:25:52Z

tested this on my c201 and got dropped right into the initramfs errr shell

Maccraft123 · 2020-09-25T20:28:21Z

oops

SolidHal · 2020-09-28T02:18:42Z

tested the image built here by the PR automation, definitely doesn't work. something goes wrong int the initramfs. @Maccraft123 if you leave this long enough, I will get around to debugging it for you hahaha.

ghost · 2020-09-30T16:29:25Z

Sorry if im on the wrong track here, but i don't think all those suppressed log levels are needed. For FB-Coding you can tell the bootsplash program (whatever is used) to claim the framebuffer as a graphical console and thus silence all console messages like this:

	int tty_fd = open("/dev/tty0", O_RDWR);
	ioctl(tty_fd,KDSETMODE,KD_GRAPHICS);

	//At exit:
	ioctl(tty_fd,KDSETMODE,KD_TEXT);

in ubuntu the splashscreen can be disabled with a simple press on ESC.

I like this solution more because we can handle decrypting the root partition in a similiar nice manner and will give it a try.

SolidHal · 2020-12-11T03:59:25Z

Seems stock plymouth works just fine with depthcharge. Seems reasonable to attempt to implement the logo in plymouth if work picks back up on this.

Maccraft123 added 6 commits September 8, 2020 10:04

When we can't see root device, wait a second and try again

8e846f2

make F A N C Y bootsplash work

4f79bda

code dump for upstream

82e7e72

fix kernels stuff

ce66348

more fix

0653186

disable selinux

471da0d

Maccraft123 added 2 commits September 17, 2020 18:57

fix loglevel

6787f40

add 1280x800 and 1366x768 framebuffer dumps

03d84a2

Revert "disable selinux", it's no longer needed

28b2c85

This reverts commit 471da0d.

Merge remote-tracking branch 'solid/master' into to_upstream

77de970

Maccraft123 mentioned this pull request Sep 19, 2020

/etc/selinux/targeted/policy/policy.31 missing from default install #219

Open

SolidHal reviewed Sep 22, 2020

View reviewed changes

Maccraft123 added 5 commits September 22, 2020 22:08

gitignore and initramfs-init fixes

af1792f

remove font source file

009d681

revert "remove font source file", fix?

96b72a2

This reverts commit 009d681.

i dont know what im doing;

37c4200

hopefully works?

05904a4

Maccraft123 added 2 commits September 22, 2020 22:44

change issue a bit

03df7d2

splash-recover-cursor.issue

40bd269

SolidHal reviewed Sep 23, 2020

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Add: seamless boot splash, daemon for disabling keyboard and touchpad, and remove gzip compression on initramfs #215

Add: seamless boot splash, daemon for disabling keyboard and touchpad, and remove gzip compression on initramfs #215

Maccraft123 commented Sep 15, 2020

gregordinary commented Sep 16, 2020

Maccraft123 commented Sep 16, 2020

Maccraft123 commented Sep 16, 2020

Maccraft123 commented Sep 17, 2020

SolidHal commented Sep 17, 2020

Maccraft123 commented Sep 17, 2020

SolidHal commented Sep 17, 2020

SolidHal commented Sep 17, 2020

SolidHal commented Sep 17, 2020

Maccraft123 commented Sep 17, 2020 •

edited

Loading

gdallasdye commented Sep 18, 2020

Maccraft123 commented Sep 18, 2020

Maccraft123 commented Sep 18, 2020

SolidHal Sep 17, 2020

Maccraft123 Sep 22, 2020

SolidHal Sep 17, 2020

Maccraft123 Sep 22, 2020 •

edited

Loading

SolidHal Sep 17, 2020

Maccraft123 Sep 22, 2020

Maccraft123 Sep 22, 2020

SolidHal Sep 23, 2020

SolidHal Sep 23, 2020

SolidHal commented Sep 23, 2020

SolidHal commented Sep 24, 2020

Maccraft123 commented Sep 25, 2020

SolidHal commented Sep 28, 2020

ghost commented Sep 30, 2020 •

edited by ghost

Loading

SolidHal commented Dec 11, 2020

		@@ -17,11 +17,9 @@
		# You should have received a copy of the GNU General Public License
		# along with PrawnOS. If not, see <https://www.gnu.org/licenses/>.

Add: seamless boot splash, daemon for disabling keyboard and touchpad, and remove gzip compression on initramfs #215

Are you sure you want to change the base?

Add: seamless boot splash, daemon for disabling keyboard and touchpad, and remove gzip compression on initramfs #215

Conversation

Maccraft123 commented Sep 15, 2020

gregordinary commented Sep 16, 2020

Maccraft123 commented Sep 16, 2020

Maccraft123 commented Sep 16, 2020

Maccraft123 commented Sep 17, 2020

SolidHal commented Sep 17, 2020

Maccraft123 commented Sep 17, 2020

SolidHal commented Sep 17, 2020

SolidHal commented Sep 17, 2020

SolidHal commented Sep 17, 2020

Maccraft123 commented Sep 17, 2020 • edited Loading

gdallasdye commented Sep 18, 2020

Maccraft123 commented Sep 18, 2020

Maccraft123 commented Sep 18, 2020

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Maccraft123 Sep 22, 2020 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

SolidHal commented Sep 23, 2020

SolidHal commented Sep 24, 2020

Maccraft123 commented Sep 25, 2020

SolidHal commented Sep 28, 2020

ghost commented Sep 30, 2020 • edited by ghost Loading

SolidHal commented Dec 11, 2020

Maccraft123 commented Sep 17, 2020 •

edited

Loading

Maccraft123 Sep 22, 2020 •

edited

Loading

ghost commented Sep 30, 2020 •

edited by ghost

Loading