<div dir="ltr"><div>Acked-by: Luke Nowakowski-Krijger <<a href="mailto:luke.nowakowskikrijger@canonical.com">luke.nowakowskikrijger@canonical.com</a>><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Jun 10, 2022 at 2:01 AM Aaron Ma <<a href="mailto:aaron.ma@canonical.com">aaron.ma@canonical.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">BugLink: <a href="https://bugs.launchpad.net/bugs/1978297" rel="noreferrer" target="_blank">https://bugs.launchpad.net/bugs/1978297</a><br>
<br>
[Impact]<br>
When bootup Jammy kernel, it shows callrace:<br>
kernel: ================================================================================<br>
kernel: UBSAN: invalid-load in /home/u/aaron/bionic/net/mac80211/status.c:1164:21<br>
kernel: load of value 216 is not a valid value for type '_Bool'<br>
kernel: CPU: 9 PID: 0 Comm: swapper/9 Not tainted 5.15.36+ #12<br>
kernel: Hardware name: LENOVO 21D2SIT061/21D2SIT061, BIOS N3GET30W (1.11 ) 05/27/2022<br>
kernel: Call Trace:<br>
kernel:  <IRQ><br>
kernel:  dump_stack_lvl+0x4a/0x5f<br>
kernel:  dump_stack+0x10/0x12<br>
kernel:  ubsan_epilogue+0x9/0x45<br>
kernel:  __ubsan_handle_load_invalid_value.cold+0x44/0x49<br>
kernel:  ieee80211_tx_status_ext.cold+0x1e/0xb0 [mac80211]<br>
kernel:  ? percpu_ref_kill_and_confirm+0xd0/0xd0<br>
kernel:  ? rht_key_get_hash.isra.0+0x19/0x20 [mac80211]<br>
kernel:  ? sta_info_hash_lookup+0xac/0x100 [mac80211]<br>
kernel:  ieee80211_tx_status+0x72/0xa0 [mac80211]<br>
kernel:  ieee80211_tasklet_handler+0xa2/0xd0 [mac80211]<br>
kernel:  tasklet_action_common.constprop.0+0xc0/0xf0<br>
kernel:  tasklet_action+0x22/0x30<br>
kernel:  __do_softirq+0xd9/0x2e3<br>
kernel:  irq_exit_rcu+0x8c/0xb0<br>
kernel:  common_interrupt+0x8a/0xa0<br>
kernel:  </IRQ><br>
kernel:  <TASK><br>
kernel:  asm_common_interrupt+0x1e/0x40<br>
kernel: RIP: 0010:cpuidle_enter_state+0xd9/0x620<br>
kernel: Code: 3d b4 85 3d 5c e8 57 a7 6b ff 49 89 c7 0f 1f 44 00 00 31 ff e8 a8 b3 6b ff 80 7d d0><br>
kernel: RSP: 0018:ffff9e2dc01f7e38 EFLAGS: 00000246<br>
kernel: RAX: ffff8dfdb6870b00 RBX: ffff8df6c3ab6c00 RCX: 000000023cd33784<br>
kernel: RDX: 000000023d103fa8 RSI: 000000023cd33784 RDI: 0000000000000000<br>
kernel: RBP: ffff9e2dc01f7e88 R08: 000000023cd33d63 R09: 00000000000aae60<br>
kernel: R10: 0000000000000004 R11: 071c71c71c71c71c R12: ffffffffa4ce65a0<br>
kernel: R13: 0000000000000003 R14: 0000000000000003 R15: 000000023cd33d63<br>
kernel:  ? cpuidle_enter_state+0xc8/0x620<br>
kernel:  ? sched_clock_local+0x13/0x90<br>
kernel:  cpuidle_enter+0x2e/0x40<br>
kernel:  do_idle+0x203/0x2a0<br>
kernel:  cpu_startup_entry+0x20/0x30<br>
kernel:  start_secondary+0x12a/0x180<br>
kernel:  secondary_startup_64_no_verify+0xc2/0xcb<br>
kernel:  </TASK><br>
kernel: ================================================================================<br>
<br>
[Fix]<br>
The size of the status_driver_data field was not adjusted<br>
when the is_valid_ack_signal field was added.<br>
Adjust the struct size accordingly.<br>
<br>
[Test]<br>
Verified on hardware, no error.<br>
<br>
[Where problems could occur]<br>
Low risk, It may break wifi driver.<br>
Minor changes in title of "Copyright" when backport.<br>
<br>
Avraham Stern (1):<br>
  mac80211: fix struct ieee80211_tx_info size<br>
<br>
 drivers/net/wireless/ath/ath10k/txrx.c  |  2 +-<br>
 drivers/net/wireless/ath/ath10k/wmi.c   |  2 +-<br>
 drivers/net/wireless/ath/ath11k/dp_tx.c |  5 +++--<br>
 include/net/mac80211.h                  | 19 +++++++++++++++----<br>
 net/mac80211/status.c                   | 11 ++++++++---<br>
 5 files changed, 28 insertions(+), 11 deletions(-)<br>
<br>
-- <br>
2.34.1<br>
<br>
<br>
-- <br>
kernel-team mailing list<br>
<a href="mailto:kernel-team@lists.ubuntu.com" target="_blank">kernel-team@lists.ubuntu.com</a><br>
<a href="https://lists.ubuntu.com/mailman/listinfo/kernel-team" rel="noreferrer" target="_blank">https://lists.ubuntu.com/mailman/listinfo/kernel-team</a><br>
</blockquote></div></div>