How to move serial8250_init out of the critical path to decrease boot time?

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Dear Linux folks,


The goal is to boot a *distribution* Linux kernel as fast as possible. (The goal is currently 500 ms.)

With Linux 4.16-rc7, `serial8250_init()` takes almost 34 ms according to `initcall_debug` on the laptop TUXEDO Book BU1406 with an Intel Kaby Lake processor.

```
[    2.657950] calling  serial8250_init+0x0/0x168 @ 1
[    2.657963] Serial: 8250/16550 driver, 32 ports, IRQ sharing enabled
[ 2.692598] initcall serial8250_init+0x0/0x168 returned 0 after 33820 usecs
```

One problem is, that the distributions do not build this as a module, despite more and more devices do not actually have serial connectors, but the chipset still exposes them.

What options are there?

1. Try to probe it asynchronously as in the attached patch, which is not tested yet.

2.  Where possible try to deactivate this in the firmware?

3. Recommend to build this as a module? But this would only fix future systems?

4.  Avoid the probe, by have an interface to pass the infomr

5. Add an option to disable the driver, which could be specified on the Linux kernel command line? `8250.disable`?


Kind regards,

Paul
>From 536100229e59d9ff7c910d0c26e1c54a56daba81 Mon Sep 17 00:00:00 2001
From: Paul Menzel <pmenzel@xxxxxxxxxxxxx>
Date: Sun, 1 Apr 2018 08:57:53 +0200
Subject: [PATCH] tty/serial/8250: Request driver probe from an async task

Currently, according to `initcall_debug` running `serial8250_init` takes
around 33 ms on a Lenovo X60 and TUXEDO Book BU1406.

As this is in the critical path, and most distributions do *not* build
*8250* as a module, that means `CONFIG_SERIAL_8250=y`, probe the device
asynchronously, so other tasks are not hold up.

Signed-off-by: Paul Menzel <pmenzel@xxxxxxxxxxxxx>
---
 drivers/tty/serial/8250/8250_core.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/tty/serial/8250/8250_core.c b/drivers/tty/serial/8250/8250_core.c
index 9342fc2ee7df..996de9c78001 100644
--- a/drivers/tty/serial/8250/8250_core.c
+++ b/drivers/tty/serial/8250/8250_core.c
@@ -894,6 +894,7 @@ static struct platform_driver serial8250_isa_driver = {
 	.driver		= {
 		.name	= "serial8250",
 	},
+	.driver.probe_type = PROBE_PREFER_ASYNCHRONOUS,
 };
 
 /*
-- 
2.14.1


[Index of Archives]     [Kernel Newbies]     [Security]     [Netfilter]     [Bugtraq]     [Linux PPP]     [Linux FS]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Linmodem]     [Device Mapper]     [Linux Kernel for ARM]

  Powered by Linux