Fix occasional boot time crash in hns driver

Bug #1833138 reported by dann frazier
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
linux (Ubuntu)
In Progress
Undecided
dann frazier
Bionic
Fix Released
Undecided
dann frazier
Cosmic
Fix Committed
Undecided
dann frazier
Disco
Won't Fix
Undecided
dann frazier
Eoan
In Progress
Undecided
dann frazier

Bug Description

[Impact]
On rare occasions, the hns driver will oops on start-up

[Test Case]
Reboot, watch for oops

[Fix]
c0b0984426814 net: hns: Fix probabilistic memory overwrite when HNS driver initialized

[Regression Risk]
Fix is restricted to the hns driver, which is the driver for the nics on the Hi1616 SoC. Regressions would therefore be limited to platforms using this SoC, such as the HiSilicon D05 board

dann frazier (dannf)
Changed in linux (Ubuntu Bionic):
status: New → In Progress
Changed in linux (Ubuntu Cosmic):
status: New → In Progress
Changed in linux (Ubuntu Disco):
status: New → In Progress
assignee: nobody → dann frazier (dannf)
Changed in linux (Ubuntu Cosmic):
assignee: nobody → dann frazier (dannf)
Changed in linux (Ubuntu Bionic):
assignee: nobody → dann frazier (dannf)
Changed in linux (Ubuntu Cosmic):
status: In Progress → Fix Committed
Changed in linux (Ubuntu Bionic):
status: In Progress → Fix Committed
Revision history for this message
Ubuntu Kernel Bot (ubuntu-kernel-bot) wrote :

This bug is awaiting verification that the kernel in -proposed solves the problem. Please test the kernel and update this bug with the results. If the problem is solved, change the tag 'verification-needed-cosmic' to 'verification-done-cosmic'. If the problem still exists, change the tag 'verification-needed-cosmic' to 'verification-failed-cosmic'.

If verification is not done by 5 working days from today, this fix will be dropped from the source code, and this bug will be closed.

See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Thank you!

tags: added: verification-needed-cosmic
tags: added: verification-needed-bionic
Revision history for this message
Ubuntu Kernel Bot (ubuntu-kernel-bot) wrote :

This bug is awaiting verification that the kernel in -proposed solves the problem. Please test the kernel and update this bug with the results. If the problem is solved, change the tag 'verification-needed-bionic' to 'verification-done-bionic'. If the problem still exists, change the tag 'verification-needed-bionic' to 'verification-failed-bionic'.

If verification is not done by 5 working days from today, this fix will be dropped from the source code, and this bug will be closed.

See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Thank you!

Revision history for this message
dann frazier (dannf) wrote :
Download full text (4.4 KiB)

Verification:

= Disco =
ubuntu@d05-3:~$ cat /proc/version
Linux version 5.0.0-21-generic (buildd@bos02-arm64-046) (gcc version 7.4.0 (Ubuntu/Linaro 7.4.0-1ubuntu1~18.04.1)) #22~18.04.1-Ubuntu SMP Thu Jul 4 17:25:45 UTC 2019
ubuntu@d05-3:~$ dmesg | grep hns
[ 9.183430] hns_dsaf HISI00B2:00: no mc-mac-mask property, set to default value.
[ 9.964208] hns_dsaf HISI00B2:00: no mc-mac-mask property, set to default value.
[ 10.206824] hns_dsaf HISI00B2:00: no mc-mac-mask property, set to default value.
[ 10.228666] hns_dsaf HISI00B2:00: no mc-mac-mask property, set to default value.
[ 10.522229] hns-nic HISI00C2:00 enahisic2i0: renamed from eth0
[ 10.573215] hns-nic HISI00C2:01 enahisic2i1: renamed from eth0
[ 10.629089] hns-nic HISI00C2:02 enahisic2i2: renamed from eth0
[ 10.704777] hns-nic HISI00C2:03 enahisic2i3: renamed from eth1
[ 51.988203] hns-nic HISI00C2:01 enahisic2i1: link up
[ 196.115843] hns-nic HISI00C2:01 enahisic2i1: link
ubuntu@d05-3:~$ dmesg --level=err
[ 1.675454] Couldn't get size: 0x800000000000000e
[ 1.675455] MODSIGN: Couldn't get UEFI db list
[ 1.675465] Couldn't get size: 0x800000000000000e
[ 1.675477] Couldn't get size: 0x800000000000000e
[ 10.195043] cma: cma_alloc: alloc failed, req-size: 32 pages, ret: -12
[ 10.202314] cma: cma_alloc: alloc failed, req-size: 32 pages, ret: -12
[ 10.212589] cma: cma_alloc: alloc failed, req-size: 32 pages, ret: -12
[ 10.230484] cma: cma_alloc: alloc failed, req-size: 32 pages, ret: -12
[ 10.244378] cma: cma_alloc: alloc failed, req-size: 32 pages, ret: -12
[ 10.263688] cma: cma_alloc: alloc failed, req-size: 32 pages, ret: -12
[ 10.270394] cma: cma_alloc: alloc failed, req-size: 32 pages, ret: -12
[ 10.277101] cma: cma_alloc: alloc failed, req-size: 32 pages, ret: -12
[ 26.047067] sd 3:0:0:0: [sdc] No Caching mode page found
[ 26.047069] sd 3:0:0:0: [sdc] Assuming drive cache: write through
[ 41.137188] ipmi_si dmi-ipmi-si.0: Interface detection failed

= Cosmic =
ubuntu@d05-3:~$ cat /proc/version
Linux version 4.18.0-26-generic (buildd@bos02-arm64-042) (gcc version 8.3.0 (Ubuntu 8.3.0-6ubuntu1~18.10.1)) #27-Ubuntu SMP Tue Jul 2 15:36:34 UTC 2019
ubuntu@d05-3:~$ dmesg | grep hns
[ 8.820811] hns_dsaf HISI00B2:00: no mc-mac-mask property, set to default value.
[ 9.501482] hns_dsaf HISI00B2:00: no mc-mac-mask property, set to default value.
[ 9.790848] hns_dsaf HISI00B2:00: no mc-mac-mask property, set to default value.
[ 9.820309] hns_dsaf HISI00B2:00: no mc-mac-mask property, set to default value.
[ 10.228363] hns-nic HISI00C2:00 enahisic2i0: renamed from eth0
[ 10.323629] hns-nic HISI00C2:01 enahisic2i1: renamed from eth0
[ 10.379501] hns-nic HISI00C2:02 enahisic2i2: renamed from eth0
[ 10.447618] hns-nic HISI00C2:03 enahisic2i3: renamed from eth0
[ 43.274888] hns-nic HISI00C2:01 enahisic2i1: link up
ubuntu@d05-3:~$ dmesg --level=err
[ 25.609748] sd 3:0:0:0: [sdc] No Caching mode page found
[ 25.609750] sd 3:0:0:0: [sdc] Assuming drive cache: write through
[ 37.140540] ipmi_si dmi-ipmi-si.0: Interface detection failed

= Bionic =
ubuntu@d05-3:~$ cat /proc/version
Linux version 4.15.0-55-generic (buildd@bo...

Read more...

tags: added: verification-done-bionic verification-done-cosmic
removed: verification-needed-bionic verification-needed-cosmic
Revision history for this message
Launchpad Janitor (janitor) wrote :
Download full text (11.2 KiB)

This bug was fixed in the package linux - 4.15.0-55.60

---------------
linux (4.15.0-55.60) bionic; urgency=medium

  * linux: 4.15.0-55.60 -proposed tracker (LP: #1834954)

  * Request backport of ceph commits into bionic (LP: #1834235)
    - ceph: use atomic_t for ceph_inode_info::i_shared_gen
    - ceph: define argument structure for handle_cap_grant
    - ceph: flush pending works before shutdown super
    - ceph: send cap releases more aggressively
    - ceph: single workqueue for inode related works
    - ceph: avoid dereferencing invalid pointer during cached readdir
    - ceph: quota: add initial infrastructure to support cephfs quotas
    - ceph: quota: support for ceph.quota.max_files
    - ceph: quota: don't allow cross-quota renames
    - ceph: fix root quota realm check
    - ceph: quota: support for ceph.quota.max_bytes
    - ceph: quota: update MDS when max_bytes is approaching
    - ceph: quota: add counter for snaprealms with quota
    - ceph: avoid iput_final() while holding mutex or in dispatch thread

  * QCA9377 isn't being recognized sometimes (LP: #1757218)
    - SAUCE: USB: Disable USB2 LPM at shutdown

  * hns: fix ICMP6 neighbor solicitation messages discard problem (LP: #1833140)
    - net: hns: fix ICMP6 neighbor solicitation messages discard problem
    - net: hns: fix unsigned comparison to less than zero

  * Fix occasional boot time crash in hns driver (LP: #1833138)
    - net: hns: Fix probabilistic memory overwrite when HNS driver initialized

  * use-after-free in hns_nic_net_xmit_hw (LP: #1833136)
    - net: hns: fix KASAN: use-after-free in hns_nic_net_xmit_hw()

  * hns: attempt to restart autoneg when disabled should report error
    (LP: #1833147)
    - net: hns: Restart autoneg need return failed when autoneg off

  * systemd 237-3ubuntu10.14 ADT test failure on Bionic ppc64el (test-seccomp)
    (LP: #1821625)
    - powerpc: sys_pkey_alloc() and sys_pkey_free() system calls
    - powerpc: sys_pkey_mprotect() system call

  * [UBUNTU] pkey: Indicate old mkvp only if old and curr. mkvp are different
    (LP: #1832625)
    - pkey: Indicate old mkvp only if old and current mkvp are different

  * [UBUNTU] kernel: Fix gcm-aes-s390 wrong scatter-gather list processing
    (LP: #1832623)
    - s390/crypto: fix gcm-aes-s390 selftest failures

  * System crashes on hot adding a core with drmgr command (4.15.0-48-generic)
    (LP: #1833716)
    - powerpc/numa: improve control of topology updates
    - powerpc/numa: document topology_updates_enabled, disable by default

  * Kernel modules generated incorrectly when system is localized to a non-
    English language (LP: #1828084)
    - scripts: override locale from environment when running recordmcount.pl

  * [UBUNTU] kernel: Fix wrong dispatching for control domain CPRBs
    (LP: #1832624)
    - s390/zcrypt: Fix wrong dispatching for control domain CPRBs

  * CVE-2019-11815
    - net: rds: force to destroy connection if t_sock is NULL in
      rds_tcp_kill_sock().

  * Sound device not detected after resume from hibernate (LP: #1826868)
    - drm/i915: Force 2*96 MHz cdclk on glk/cnl when audio power is enabled
    - drm/i915: Save the old CDCLK atomic state
...

Changed in linux (Ubuntu Bionic):
status: Fix Committed → Fix Released
Revision history for this message
Ubuntu Kernel Bot (ubuntu-kernel-bot) wrote :

This bug is awaiting verification that the kernel in -proposed solves the problem. Please test the kernel and update this bug with the results. If the problem is solved, change the tag 'verification-needed-xenial' to 'verification-done-xenial'. If the problem still exists, change the tag 'verification-needed-xenial' to 'verification-failed-xenial'.

If verification is not done by 5 working days from today, this fix will be dropped from the source code, and this bug will be closed.

See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Thank you!

tags: added: verification-needed-xenial
Steve Langasek (vorlon)
Changed in linux (Ubuntu Disco):
status: In Progress → Won't Fix
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.