Re: [PATCH v5 3/3] serial: core: Fix serial core controller port name to show controller id

[Tony Lindgren]

* kernel test robot <oliver.sang@xxxxxxxxx> [230802 08:16]:
> from serial, we observed last print out is:
> 
> [   15.584772][  T954] EDAC MC0: Giving out device to module skx_edac controller Skylake Socket#0 IMC#0: DEV 0000:3a:0a.0 (INTERRUPT)
> [   15.597328][  T954] EDAC MC1: Giving out device to module skx_edac controller Skylake Socket#0 IMC#1: DEV 0000:3a:0c.0 (INTERRUPT)
> [   15.610326][  T954] EDAC MC2: Giving out device to module skx_edac controller Skylake Socket#1 IMC#0: DEV 0000:ae:0a.0 (INTERRUPT)
> [   15.623375][  T954] EDAC MC3: Giving out device to module skx_edac controller Skylake Socket#1 IMC#1: DEV 0000:ae:0c.0 (INTERRUPT)
> [   15.640145][   T19] intel_rapl_common: Found RAPL domain package
> [   15.655890][   T19] intel_rapl_common: Found RAPL domain dram
> [   15.661983][   T19] intel_rapl_common: package-0:package:long_term locked by BIOS
> [   15.678564][   T19] intel_rapl_common: package-0:package:short_term locked by BIOS
> [   15.695259][   T19] intel_rapl_common: package-0:dram:long_term locked by BIOS
> [   15.713068][  T158] intel_rapl_common: Found RAPL domain package
> [   15.728719][  T158] intel_rapl_common: Found RAPL domain dram
> [   15.734743][  T158] intel_rapl_common: package-1:package:long_term locked by BIOS
> [   15.745244][ T1154] raid6: avx512x4 gen() 18153 MB/s
> [   15.761297][  T158] intel_rapl_common: package-1:package:short_term locked by BIOS
> [   15.767244][ T1154] raid6: avx512x2 gen() 18130 MB/s
> [   15.768866][  T158] intel_rapl_common: package-1:dram:long_term locked by BIOS
> [   15.790243][ T1154] raid6: avx512x1 gen() 18155 MB/s
> [   15.812245][ T1154] raid6: avx2x4   gen() 18060 MB/s
> [   15.834244][ T1154] raid6: avx2x2   gen() 18076 MB/s
> [   15.856244][ T1154] raid6: avx2x1   gen() 13836 MB/s
> [   15.861474][ T1154] raid6: using algorithm avx512x1 gen() 18155 MB/s
> [   15.884243][ T1154] raid6: .... xor() 27974 MB/s, rmw enabled
> [   15.890254][ T1154] raid6: using avx512x2 recovery algorithm
> [   15.897891][ T1154] xor: measuring software checksum speed
> [   15.904013][ T1154]    prefetch64-sse  : 31308 MB/sec
> [   15.909878][ T1154]    generic_sse     : 22929 MB/sec
> [   15.915230][ T1154] xor: using function: prefetch64-sse (31308 MB/sec)
> [   16.042623][ T1154] Btrfs loaded, zoned=no, fsverity=no
> [   16.054593][  T930] BTRFS: device fsid e422031c-19be-42f5-ab4f-be5f306aa6e1 devid 1 transid 39725 /dev/sda2 scanned by systemd-udevd (930)
> 
> 
> then the machine is just stuck there. (whole dmesg captured from serial is
> attached), and the issue is 100% reproducible for this commit.
> 
> for parent, we never observed the boot failure.
> 
> it looks quite strange to us why this commit could cause this behavior on our
> machine. could you help check dmesg, config and kernel command line which is
> also captured in dmesg, etc. and guide us if anything need to be updated to be
> compatible with this change? Thanks!

Thanks for the report. With the ctrl and port prefixes dropped, I broke
serial_base_match() looks like. As we attempt to continue anyways, things
still mostly work..

Greg, can you please drop the related commit?

It's the following commit:

1ef2c2df1199 ("serial: core: Fix serial core controller port name to show controller id")

Regards,

Tony