Received-SPF: pass (google.com: domain of linux-wireless+bounces-7893-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:4601:e00::3 as permitted sender) client-ip=2604:1380:4601:e00::3;
From: Kalle Valo <kvalo@kernel.org>
To: Baochen Qiang <quic_bqiang@quicinc.com>
Cc: <ath12k@lists.infradead.org>,  <linux-wireless@vger.kernel.org>
Subject: Re: [PATCH 0/7] wifi: ath12k: add support for WoW
References: <20240510064748.369452-1-quic_bqiang@quicinc.com>
Date: Tue, 21 May 2024 12:49:54 +0300
In-Reply-To: <20240510064748.369452-1-quic_bqiang@quicinc.com> (Baochen
	Qiang's message of "Fri, 10 May 2024 14:47:41 +0800")
Message-ID: <87h6ercx4t.fsf@kernel.org>
User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/28.2 (gnu/linux)
Precedence: bulk
MIME-Version: 1.0
Content-Type: text/plain

Baochen Qiang <quic_bqiang@quicinc.com> writes:

> Add support for WoW (Wakeup on Wireless) funtionalities, this including
> magic-pattern, net-detect, disconnect and patterns, HW data filter, ARP
> and NS offload, GTK rekey offload.
>
> Also enable keepalive before suspend to avoid unexpected kick out by AP.

There was a simple conflict in ath12k_core_suspend() but I fixed that
(tag ath-pending-202405210818). But I see new warnings:

./include/linux/fortify-string.h:97:33: warning: '__builtin_memcpy' accessing 18446744073709551605 or more bytes at offsets 0 and 0 overlaps 9223372036854775787 bytes at offset -9223372036854775798 [-Wrestrict]
./include/linux/fortify-string.h:97:33: warning: '__builtin_memcpy' accessing 18446744073709551605 or more bytes at offsets 0 and 0 overlaps 9223372036854775787 bytes at offset -9223372036854775798 [-Wrestrict]
drivers/net/wireless/ath/ath12k/core.c:69:5: warning: context imbalance in 'ath12k_core_suspend' - different lock contexts for basic block

And in my suspend tests with WCN7850 the kernel crashes (tried two times
and crashed in both):

[  854.087911] wlan0: associated
[  854.139498] wlan0: Limiting TX power to 23 (23 - 0) dBm as advertised by <censored>
[  854.481975] BUG: sleeping function called from invalid context at include/linux/sched/mm.h:315
[  854.483069] in_atomic(): 0, irqs_disabled(): 0, non_block: 0, pid: 1694, name: wpa_supplicant
[  854.483524] preempt_count: 0, expected: 0
[  854.484010] RCU nest depth: 1, expected: 0
[  854.484416] 2 locks held by wpa_supplicant/1694:
[  854.484914]  #0: ffffffff9c9e2650 (rtnl_mutex){+.+.}-{3:3}, at: rtnetlink_rcv_msg+0x2c8/0xbf0
[  854.485512]  #1: ffffffff9c0cd960 (rcu_read_lock){....}-{1:2}, at: atomic_notifier_call_chain+0x28/0x140
[  854.486290] CPU: 1 PID: 1694 Comm: wpa_supplicant Not tainted 6.9.0-wt-ath+ #1525
[  854.486707] Hardware name: Intel(R) Client Systems NUC8i7HVK/NUC8i7HVB, BIOS HNKBLi70.86A.0067.2021.0528.1339 05/28/2021
[  854.487195] Call Trace:
[  854.487467]  <TASK>
[  854.487734]  dump_stack_lvl+0xa4/0xe0
[  854.488049]  dump_stack+0x10/0x20
[  854.488335]  __might_resched+0x363/0x5a0
[  854.488628]  ? kmalloc_trace+0x4b/0x2d0
[  854.488978]  __might_sleep+0xad/0x160
[  854.489282]  kmalloc_trace+0x264/0x2d0
[  854.489574]  ? ath12k_mac_arvif_get_arp_ns_offload+0x72/0xd0 [ath12k]
[  854.490071]  ath12k_mac_arvif_get_arp_ns_offload+0x72/0xd0 [ath12k]
[  854.490543]  ath12k_mac_op_ipv6_changed+0xc7/0x1470 [ath12k]
[  854.491117]  ? ieee80211_emulate_change_chanctx+0xe0/0xe0 [mac80211]
[  854.491909]  ieee80211_ifa6_changed+0x1c7/0x3a0 [mac80211]
[  854.492663]  notifier_call_chain+0x97/0x310
[  854.492997]  atomic_notifier_call_chain+0x5d/0x140
[  854.493322]  inet6addr_notifier_call_chain+0x16/0x20
[  854.493650]  ipv6_add_addr+0x10ec/0x2160
[  854.493946]  ? reacquire_held_locks+0x4d0/0x4d0
[  854.494294]  ? inet6_dump_ifaddr+0x10/0x10
[  854.494599]  ? ip6_route_add+0x75/0x130
[  854.494910]  ? __kasan_check_write+0x14/0x20
[  854.495214]  ? ip6_route_add+0x8b/0x130
[  854.495525]  addrconf_add_linklocal+0xd6/0x300
[  854.495965]  ? addrconf_add_linklocal+0xd6/0x300
[  854.496297]  ? addrconf_dad_start+0x110/0x110
[  854.499635]  ? addrconf_addr_eui48_base+0x80/0xa0
[  854.503094]  addrconf_addr_gen.constprop.0+0x155/0x260
[  854.506223]  ? mutex_unlock+0xd/0x10
[  854.508461]  ? addrconf_add_linklocal+0x300/0x300
[  854.510681]  addrconf_init_auto_addrs+0x194/0x4e0
[  854.512414]  addrconf_notify+0xabe/0xdd0
[  854.514073]  notifier_call_chain+0x97/0x310
[  854.515733]  ? __unwind_start+0x1f3/0x3d0
[  854.517070]  raw_notifier_call_chain+0x11/0x20
[  854.518407]  call_netdevice_notifiers_info+0x8b/0xf0
[  854.521761]  netdev_state_change+0xe8/0x120
[  854.522882]  ? call_netdevice_notifiers_info+0xf0/0xf0
[  854.524019]  netdev_set_operstate+0xdf/0x130
[  854.525139]  ? rtnl_put_cacheinfo+0x260/0x260
[  854.526216]  set_operstate+0x8a/0x150
[  854.527175]  do_setlink+0xf0e/0x2550
[  854.528130]  ? rtnl_bridge_getlink+0x870/0x870
[  854.529078]  ? check_prev_add+0x2330/0x2330
[  854.530011]  ? mark_lock+0xe6/0x1470
[  854.530919]  ? check_prev_add+0x2330/0x2330
[  854.531710]  ? __lock_acquire+0xd43/0x1dd0
[  854.532491]  ? kasan_print_address_stack_frame+0x1d2/0x230
[  854.533277]  ? nla_get_range_signed+0x340/0x340
[  854.534046]  ? __kasan_check_read+0x11/0x20
[  854.534808]  ? mark_lock+0xe6/0x1470
[  854.535573]  ? __kasan_check_read+0x11/0x20
[  854.536292]  ? __nla_validate_parse+0x1d0/0x400
[  854.536980]  rtnl_setlink+0x217/0x330
[  854.537650]  ? do_setlink+0x2550/0x2550
[  854.538355]  ? rtnetlink_rcv_msg+0x2c8/0xbf0
[  854.539010]  ? mutex_lock_io_nested+0x1270/0x1270
[  854.539657]  ? reacquire_held_locks+0x4d0/0x4d0
[  854.540290]  ? lock_sync+0x1a0/0x1a0
[  854.540981]  ? __this_cpu_preempt_check+0x13/0x20
[  854.541598]  rtnetlink_rcv_msg+0x315/0xbf0
[  854.542278]  ? check_prev_add+0x2330/0x2330
[  854.542894]  ? rtnl_link_fill+0x900/0x900
[  854.543446]  ? __lock_acquire+0xd43/0x1dd0
[  854.544037]  netlink_rcv_skb+0x14c/0x400
[  854.544581]  ? rtnl_link_fill+0x900/0x900
[  854.545132]  ? netlink_ack+0xd70/0xd70
[  854.545851]  ? debug_smp_processor_id+0x17/0x20
[  854.546369]  ? __lock_release.isra.0+0x49c/0xae0
[  854.546939]  ? netlink_deliver_tap+0x143/0x350
[  854.547478]  ? netlink_deliver_tap+0x143/0x350
[  854.548062]  ? __this_cpu_preempt_check+0x13/0x20
[  854.548620]  ? netlink_deliver_tap+0xc8/0x350
[  854.549149]  rtnetlink_rcv+0x10/0x20
[  854.549708]  netlink_unicast+0x45c/0x790
[  854.550211]  ? netlink_attachskb+0x7f0/0x7f0
[  854.550802]  netlink_sendmsg+0x7eb/0xdb0
[  854.551302]  ? netlink_unicast+0x790/0x790
[  854.551847]  ? unix_ioctl+0x358/0x400
[  854.552338]  ? selinux_socket_sendmsg+0x31/0x40
[  854.552873]  ? netlink_unicast+0x790/0x790
[  854.553360]  __sock_sendmsg+0xc9/0x160
[  854.553901]  ? sock_do_ioctl+0xcf/0x1e0
[  854.554387]  __sys_sendto+0x1ca/0x290
[  854.554925]  ? __ia32_sys_getpeername+0xa0/0xa0
[  854.555413]  ? ioctl_has_perm.constprop.0.isra.0+0x276/0x430
[  854.556005]  ? sock_ioctl+0x24d/0x540
[  854.556497]  ? br_ioctl_call+0x90/0x90
[  854.557047]  ? selinux_file_ioctl+0x108/0x250
[  854.557538]  ? debug_smp_processor_id+0x17/0x20
[  854.558076]  __x64_sys_sendto+0xdb/0x1a0
[  854.558588]  ? __this_cpu_preempt_check+0x13/0x20
[  854.559170]  ? lockdep_hardirqs_on+0x7d/0x100
[  854.559716]  x64_sys_call+0x8a2/0x9e0
[  854.560209]  do_syscall_64+0x65/0x130
[  854.560807]  entry_SYSCALL_64_after_hwframe+0x4b/0x53
[  854.561300] RIP: 0033:0x7f0ef17cb750
[  854.561848] Code: ff eb bc 0f 1f 80 00 00 00 00 f3 0f 1e fa 41 89 ca 64 8b 04 25 18 00 00 00 85 c0 75 1d 45 31 c9 45 31 c0 b8 2c 00 00 00 0f 05 <48> 3d 00 f0 ff ff 77 68 c3 0f 1f 80 00 00 00 00 55 48 83 ec 20 48
[  854.562946] RSP: 002b:00007ffc4306af98 EFLAGS: 00000246 ORIG_RAX: 000000000000002c
[  854.563506] RAX: ffffffffffffffda RBX: 0000000000000006 RCX: 00007f0ef17cb750
[  854.564122] RDX: 0000000000000028 RSI: 00007ffc4306afb0 RDI: 0000000000000005
[  854.564738] RBP: 0000556901de3960 R08: 0000000000000000 R09: 0000000000000000
[  854.565298] R10: 0000000000000000 R11: 0000000000000246 R12: 00000000ffffffff
[  854.565945] R13: 00007ffc4306afb0 R14: 0000000000000005 R15: 0000556901e2b1f1
[  854.566522]  </TASK>


-- 
https://patchwork.kernel.org/project/linux-wireless/list/

https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches