Re: [serial: 8250_lpss] a13e19cf3d: BUG: kernel hang in boot stage

[Fengguang Wu]

Hi Andy,

On Sat, Oct 22, 2016 at 07:40:14PM +0300, Andy Shevchenko wrote:
> On Mon, 2016-09-05 at 18:15 +0300, Andy Shevchenko wrote:
> > +Heikki
> >
> > On Sat, 2016-09-03 at 16:25 +0800, kernel test robot wrote:
> > > Greetings,
> > >
> > > 0day kernel testing robot got the below dmesg and the first bad
> > > commit
> > > is
> >
> > Briefly tested I have hang issues *with* or *without* the mentioned
> > commit if under hang you meant no getty available on serial console.
> > Otherwise I see output from kernel tillÂ
> >
> > [ÂÂÂ61.625900] raid6test: complete (2429 tests, 0 failures)
> > [ÂÂÂ61.671165] BIOS EDD facility v0.16 2004-Jun-25, 0 devices found
> > [ÂÂÂ61.678004] EDD information not available.
> > [ÂÂÂ61.714036] ALSA device list:
> > [ÂÂÂ61.717459]ÂÂÂ#0: Dummy 1
> > [ÂÂÂ61.720480]ÂÂÂ#1: MTPAV on parallel port at 0x378
> >
> > Attached defconfigs per commit I have tested (also diff between mine
> > and
> > yours when I did make savedefconfig).
> >
> > So, may I ask what exact hardware you are using to test this one?
>
> May I ask beloved LKP team what the status of the issue is? Shall we
> close it or there is more information coming?

Please close the issue.

I tried rebuild the kernel and redo the tests, and find the new kernel
no longer has early boot hang. Sorry for the noise!

Cheers,
Fengguang

> > >
> > > https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/tty.git tty-
> > > next
> > >
> > > commit a13e19cf3dc1080cf8a3a174cefd9199610faed7
> > > Author:ÂÂÂÂÂAndy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx>
> > > AuthorDate: Wed Aug 17 19:20:27 2016 +0300
> > > Commit:ÂÂÂÂÂGreg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx>
> > > CommitDate: Wed Aug 31 16:13:30 2016 +0200
> > >
> > > ÂÂÂÂserial: 8250_lpss: split LPSS driver to separate module
> > > ÂÂÂÂ
> > > ÂÂÂÂThe SoCs, such as Intel Braswell, have DesignWare UART IP. Split
> > > out the
> > > ÂÂÂÂsupport of such chips to a separate module which also will be
> > > used
> > > for Intel
> > > ÂÂÂÂQuark later.
> > > ÂÂÂÂ
> > > ÂÂÂÂThe rationale to have the separate driver to be existing:
> > > ÂÂÂÂ- Do not contaminate 8250_pci.c anymore with LPSS related quirks
> > > ÂÂÂÂ- All of them are using same DMA engine and they are Designware
> > > IP
> > > which means
> > > ÂÂÂÂÂÂthat in the future we might share the code between 8250_dw.c
> > > and
> > > 8250_lpss.c
> > > ÂÂÂÂ- It reduces the kernel memory footprint on non-X86 machines
> > > where
> > > 8250_pci.c
> > > ÂÂÂÂÂÂis in use
> > > ÂÂÂÂ
> > > ÂÂÂÂBesides the split the driver also has been refactored, in
> > > particular a) the DMA
> > > ÂÂÂÂand port setup are separate functions, b) the two new structures
> > > lpss8250 and
> > > ÂÂÂÂlpss8250_board are introduced to keep necessary data instead of
> > > ÂÂÂÂpciserial_board, c) DMA parameters are passed to the DMA setup
> > > via
> > > mentioned
> > > ÂÂÂÂcustom structure. Most of the changes are done due to the future
> > > support of
> > > ÂÂÂÂUART DMA on Intel Quark.
> > > ÂÂÂÂ
> > > ÂÂÂÂThe Intel Quark UART DMA support is based on bits taking from
> > > BSP
> > > code
> > > ÂÂÂÂpublished by Intel earlier.
> > > ÂÂÂÂ
> > > ÂÂÂÂThe driver does not use any specific power management. PCI core
> > > takes care of
> > > ÂÂÂÂthe default behaviour during suspend and resume.
> > > ÂÂÂÂ
> > > ÂÂÂÂSigned-off-by: Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxx
> > > m>
> > > ÂÂÂÂTested-by: Bryan O'Donoghue <pure.logic@xxxxxxxxxxxxxxxxx>
> > > ÂÂÂÂSigned-off-by: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx>
> > >
> > > +-------------------------------+------------+------------+---------
> > > --
> > > -+
> > > >
> > > > ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ| 68af490b03 | a13e19cf3d |
> > > > 6fe4da5401
> > > > >
> > >
> > > +-------------------------------+------------+------------+---------
> > > --
> > > -+
> > > >
> > > > boot_successesÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ| 63ÂÂÂÂÂÂÂÂÂ| 0ÂÂÂÂÂÂÂÂÂÂ|
> > > > 0ÂÂÂÂÂÂÂÂÂÂ|
> > > > boot_failuresÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ| 0ÂÂÂÂÂÂÂÂÂÂ| 26ÂÂÂÂÂÂÂÂÂ|
> > > > 27ÂÂÂÂÂÂÂÂÂ|
> > > > BUG:kernel_hang_in_boot_stage | 0ÂÂÂÂÂÂÂÂÂÂ| 26ÂÂÂÂÂÂÂÂÂ|
> > > > 27ÂÂÂÂÂÂÂÂÂ|
> > >
> > > +-------------------------------+------------+------------+---------
> > > --
> > > -+
> > >
> > > Booting the kernel.
> > >
> > > Elapsed time: 500
> > > BUG: kernel hang in boot stage
> > > Linux version 4.8.0-rc2-00028-ga13e19c #1
> > > Command line:ÂÂroot=/dev/ram0 hung_task_panic=1 debug apic=debug
> > > sysrq_always_enabled rcupdate.rcu_cpu_stall_timeout=100
> > > net.ifnames=0
> > > printk.devkmsg=on panic=-1 softlockup_panic=1 nmi_watchdog=panic
> > > oops=panic load_ramdisk=2 prompt_ramdisk=0 systemd.log_level=err
> > > ignore_loglevel earlyprintk=ttyS0,115200 console=ttyS0,115200
> > > console=tty0 vga=normal rw link=/kbuild-tests/run-queue/kvm/x86_64-
> > > randconfig-r0-09031101/linux-devel:devel-spot-
> > > 201609030826:a13e19cf3dc1080cf8a3a174cefd9199610faed7:bisect-linux-
> > > 0/.vmlinuz-a13e19cf3dc1080cf8a3a174cefd9199610faed7-20160903152912-
> > > 8-
> > > vp branch=linux-devel/devel-spot-201609030826
> > > BOOT_IMAGE=/pkg/linux/x86_64-randconfig-r0-09031101/gcc-
> > > 6/a13e19cf3dc1080cf8a3a174cefd9199610faed7/vmlinuz-4.8.0-rc2-00028-
> > > ga13e19c drbd.minor_count=8
> > >
> > >
> > > git bisect start 6fe4da54019254083959652e61fab5d7b6d8b16b
> > > 3eab887a55424fc2c27553b7bfe32330df83f7b8 --
> > > git bisect good 86374892625f368ff0a7d0045bbd10f5005f5095ÂÂ#
> > > 13:00ÂÂÂÂÂ20+ÂÂÂÂÂÂ0ÂÂMerge 'linux-review/Paul-Burton/Partial-MIPS-
> > > Malta-DT-conversion/20160903-000153' into devel-spot-201609030826
> > > git bisectÂÂbad 28b0a3136795a9a1d1abb0331790bbae75e71e03ÂÂ#
> > > 13:14ÂÂÂÂÂÂ0-ÂÂÂÂÂ11ÂÂMerge 'renesas/topic/sdr104-v6' into devel-
> > > spot-
> > > 201609030826
> > > git bisect good 2e5ad9bf16a597bcf52ffee14dc5a3dc80e60ae9ÂÂ#
> > > 13:24ÂÂÂÂÂ22+ÂÂÂÂÂÂ0ÂÂMerge 'linux-review/Wang-Nan/perf-report-
> > > Recognize-hugetlb-mapping-as-anon-mapping/20160902-220432' into
> > > devel-
> > > spot-201609030826
> > > git bisectÂÂbad bedfbf8942db947f2a9be3e3a4e01d8ddf3bdf4eÂÂ#
> > > 13:40ÂÂÂÂÂÂ0-ÂÂÂÂÂÂ8ÂÂMerge 'linux-review/Luca-Ceresoli/rtl8723au-
> > > remove-declaration-of-unimplemented-functions/20160902-211231' into
> > > devel-spot-201609030826
> > > git bisectÂÂbad 687a5fb1c4382d1e37d444fdc24f9a0ecb4034a0ÂÂ#
> > > 14:02ÂÂÂÂÂÂ0-ÂÂÂÂÂ10ÂÂMerge 'tty/tty-testing' into devel-spot-
> > > 201609030826
> > > git bisect good 259bbca0cd7b626ab0fc5eb63dfc24b1408e29b4ÂÂ#
> > > 14:13ÂÂÂÂÂ21+ÂÂÂÂÂÂ0ÂÂMerge 'linux-review/Toke-H-iland-J-
> > > rgensen/mac80211-Move-reorder-sensitive-TX-handlers-to-after-TXQ-
> > > dequeue/20160902-215155' into devel-spot-201609030826
> > > git bisect good bf7a791d2c931375ae966adfb0a93eae426c6d7eÂÂ#
> > > 14:26ÂÂÂÂÂ22+ÂÂÂÂÂÂ0ÂÂMerge 'linux-review/Hariprasad-Shenai/cxgb4-
> > > Add-
> > > support-for-ndo_get_vf_config/20160902-214923' into devel-spot-
> > > 201609030826
> > > git bisect good 5887ad43ee02a00f17a6132b7fb256dc6865474cÂÂ#
> > > 14:41ÂÂÂÂÂ22+ÂÂÂÂÂÂ0ÂÂtty: serial: fsl_lpuart: Use cyclic DMA for Rx
> > > git bisect good 9217a5bf66061c392e461255c5d9d41aac136e58ÂÂ#
> > > 14:50ÂÂÂÂÂ22+ÂÂÂÂÂÂ0ÂÂdmaengine: dw: keep copy of custom slave
> > > config
> > > in dwc
> > > git bisect good 8d17047207d52182ccd55b9529e1c8cc062c07c2ÂÂ#
> > > 15:06ÂÂÂÂÂ22+ÂÂÂÂÂÂ0ÂÂserial: 8250_dma: switch to new
> > > dmaengine_terminate_* API
> > > git bisect good 68af490b03b40cc31b2cd47899661d87ab9f127dÂÂ#
> > > 15:22ÂÂÂÂÂ22+ÂÂÂÂÂÂ0ÂÂserial: 8250: enable AFE on ports where FIFO
> > > is
> > > 16 bytes
> > > git bisectÂÂbad a13e19cf3dc1080cf8a3a174cefd9199610faed7ÂÂ#
> > > 15:36ÂÂÂÂÂÂ0-ÂÂÂÂÂÂ4ÂÂserial: 8250_lpss: split LPSS driver to
> > > separate
> > > module
> > > # first bad commit: [a13e19cf3dc1080cf8a3a174cefd9199610faed7]
> > > serial:
> > > 8250_lpss: split LPSS driver to separate module
> > > git bisect good 68af490b03b40cc31b2cd47899661d87ab9f127dÂÂ#
> > > 15:40ÂÂÂÂÂ63+ÂÂÂÂÂÂ0ÂÂserial: 8250: enable AFE on ports where FIFO
> > > is
> > > 16 bytes
> > > # extra tests on HEAD of linux-devel/devel-spot-201609030826
> > > git bisectÂÂbad 6fe4da54019254083959652e61fab5d7b6d8b16bÂÂ#
> > > 15:40ÂÂÂÂÂÂ0-ÂÂÂÂÂ27ÂÂ0day head guard for 'devel-spot-201609030826'
> > > # extra tests on tree/branch tty/tty-next
> > > git bisectÂÂbad d2f5a7311bcaed681a41cb3419b8fe92a7b68bf5ÂÂ#
> > > 16:00ÂÂÂÂÂÂ0-ÂÂÂÂÂ17ÂÂdmaengine: hsu: refactor hsu_dma_do_irq() to
> > > return int
> > > # extra tests on tree/branch linus/master
> > > git bisect good 0141af184adc7f6ea9bd2f9631a15704d0806f8bÂÂ#
> > > 16:10ÂÂÂÂÂ66+ÂÂÂÂÂÂ1ÂÂMerge branch 'for-linus' of
> > > git://git.kernel.dk/linux-block
> > > # extra tests on tree/branch linux-next/master
> > > git bisect good 0f98f121e1670eaa2a2fbb675e07d6ba7f0e146fÂÂ#
> > > 16:24ÂÂÂÂÂ65+ÂÂÂÂÂÂ1ÂÂAdd linux-next specific files for 20160825
> > >
> > >
> > > ---
> > > 0-DAY kernel test infrastructureÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂOpen Source
> > > Technology
> > > Center
> > > https://lists.01.org/pipermail/lkpÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂIntel
> > > Corporation
>
> --
> Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx>
> Intel Finland Oy