tools: sof-kernel-log-check: filter out e1000e network error

[fredoh9]

This error was found in CML and TGL devices.
kernel: e1000e 0000:00:1f.6 enp0s31f6: Hardware Error
kernel: e1000e 0000:00:1f.6 eno0: Hardware Error

Signed-off-by: Fred Oh fred.oh@intel.com

[plbossart]

can we abstract the hardware address with the regexp?

[marc-hb]

@fredoh9 we don't need this since disabling built-in Ethernet in the BIOS, correct?

[marc-hb]

Re-open if needed.

[fredoh9]

This error happens occasionally. Ideally it should not happen as we disable built-in ethernet interface from the BIOS. Number of devices are keep growing and less people to manage all. I will re-open this for the review.

===========================>>
[  156.271599] kernel: e1000e 0000:00:1f.6 eno1: Hardware Error
<<===========================

[fredoh9]

looks it matches fine. current filter is

• e1000e 0000:00:.+..+| en*: Hardware Error

I can slightly change REGEX like this too,

• e1000e [0-9a-fA-F]{4}:[0-9a-fA-F]{2}:[0-9a-fA-F]{2}.[0-9a-fA-F] en.+: Hardware Error
• e1000e 0000:00:[0-9a-fA-F]{2}.[0-9a-fA-F] en.+: Hardware Error

[fredoh9]

Wait a sec, UP Xtreme has two ethernet adapters, looks mostly problem in 1f.6 device. But we need more generic filter like above my suggestion. Will submit a fix shortly.

ubuntu@sh-tglu-up-hda-05:~$ lspci | grep Ethernet
00:1f.6 Ethernet controller: Intel Corporation Ethernet Connection (13) I219-LM (rev 20)
2d:00.0 Ethernet controller: Intel Corporation Ethernet Controller (2) I225-IT (rev 03)

ubuntu@jf-tglu-up-hda-03:~$ lspci | grep Ethernet
00:1f.6 Ethernet controller: Intel Corporation Ethernet Connection (13) I219-LM (rev 20)
02:00.0 Ethernet controller: Intel Corporation Ethernet Controller (2) I225-IT (rev 03)

[marc-hb]

We can blocklist e1000e with Ansible instead, I just did that manually on sh-tglu-up-hda-05, see #564. No need to mess with BIOS anymore. Also, blocklist means anyone can still modprobe e1000e once if needed.

[marc-hb]

blocklist

[fredoh9]

blocklist

you mean blacklist?

[marc-hb]

yes https://www.zdnet.com/article/linux-kernel-developers-unveil-new-blm-coding-style-to-avoid-words-like-blacklist/