Received: by 2002:a05:6358:1087:b0:cb:c9d3:cd90 with SMTP id j7csp5651631rwi; Tue, 18 Oct 2022 02:05:35 -0700 (PDT) X-Google-Smtp-Source: AMsMyM5FRVK5oXxU7BBk0xosH/m9ZKiscSs9Bj0+pvijJKtI/TH6eXvJY0A2hBITTEQtJgN20bBd X-Received: by 2002:a17:906:db0e:b0:77b:82cf:54a6 with SMTP id xj14-20020a170906db0e00b0077b82cf54a6mr1511144ejb.691.1666083935735; Tue, 18 Oct 2022 02:05:35 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1666083935; cv=none; d=google.com; s=arc-20160816; b=QA2+S8OIf02F0XkmfsfAV3nMV0qRx+lNSVrfWGZZISrj47L/b1XY5vR+DMS/lbK3xg +ydKqrzz8pmvH+rNhv5bvFJvAY1yaLXmZDzx8zHItLs8UhDSFpTsxfb2YBAKhEaSvSWt 8cfPKfQbsfx1MNXhuK1VyII/u2Ji9S3SUylNrNkttChyT9kaP/7zmiOC0UrS4BhidyG3 kPpLGzgpvkJWFEnHvdFxmR9h/zvVa3w7wt6fOhQGYaVhqHOpKk8v2Yh0zaPIVtBv4tfP 5X7wSn93ZYOT2lZu+nHINzJX9KI7JgfRmfyzb27Jf8gCgZeEE9xG8fOM/Rp2TAOcas0/ GtEQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:mime-version:user-agent :content-transfer-encoding:references:in-reply-to:date:cc:to:from :subject:message-id:dkim-signature; bh=mZMYrdJe0ak55MfkV9WDpQoDddT502UmBNyt8pPDbA4=; b=sjV7VSuGFMAXrVG3NUHzT1TyNc9L4od4Q+LDs7IP1XdfUQ4dzQZ3PUBtmsv6TpFSSh Itf/J7HstPm0QXxlbRXrUqvcHgitjKpVZJA2BtL0fQIwIoLczDpd3wJLuKKBQQI/PTbp 7skW8TVUp6aHb4wXARKslDb4dwU4XJ9QzcpQ2Mn2PCQaeOXbTeoe2YzZZ/vKplCcXdzs mZtSHz8NKb/MK/fZTh240kNtqsFDPMFGr/g+p74Swq2yWp0ySnDKMv2PY4DUTZWhmqFM myEvZxgQBFjHoCeV4nMpZGawQO1HhjetnStxXo4ISzG4tWGNBP4C0F87/+QUo9ivu9Li tskg== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@sipsolutions.net header.s=mail header.b=hfbxZfJV; spf=pass (google.com: domain of linux-wireless-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-wireless-owner@vger.kernel.org; dmarc=pass (p=NONE sp=REJECT dis=NONE) header.from=sipsolutions.net Return-Path: Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id c7-20020a170906528700b00783df65ae5esi10625085ejm.683.2022.10.18.02.05.20; Tue, 18 Oct 2022 02:05:35 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-wireless-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@sipsolutions.net header.s=mail header.b=hfbxZfJV; spf=pass (google.com: domain of linux-wireless-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-wireless-owner@vger.kernel.org; dmarc=pass (p=NONE sp=REJECT dis=NONE) header.from=sipsolutions.net Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229939AbiJRIxk (ORCPT + 63 others); Tue, 18 Oct 2022 04:53:40 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:36756 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229489AbiJRIxj (ORCPT ); Tue, 18 Oct 2022 04:53:39 -0400 Received: from sipsolutions.net (s3.sipsolutions.net [IPv6:2a01:4f8:191:4433::2]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 2F65D1D318 for ; Tue, 18 Oct 2022 01:53:38 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=sipsolutions.net; s=mail; h=MIME-Version:Content-Transfer-Encoding: Content-Type:References:In-Reply-To:Date:Cc:To:From:Subject:Message-ID:Sender :Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From:Resent-To: Resent-Cc:Resent-Message-ID; bh=mZMYrdJe0ak55MfkV9WDpQoDddT502UmBNyt8pPDbA4=; t=1666083218; x=1667292818; b=hfbxZfJVfMVj56T2a+9FNJe9O5eAINFbrtZa6VeDZic7UgQ aAqT1XllHdAwzlTnipgyunsuap5ckDYuJLcAi8Fii8QwJT2zqsBi5SzSZ0l6danMbHszmNxkZzlza 6ctsEeug6YJUb4KC8ZLwAfnCcQU4rYgZ+3By5MyFod91o9waUNBdR4XdTnlFzL1rI0BQHV05cpoTM NURDxpphtOUteUsmVDUIMhLO8Dxz8tDEmb8/n0hf2Nb3EPVMdPij6tX2unHgxfCyo4oxHwdRJOvzs HH2VKOS6wyNRNIYeNTu/VAYn6micMvAKb9asuMUdWZwq3UUw8dD8t/WON3jOYa9A==; Received: by sipsolutions.net with esmtpsa (TLS1.3:ECDHE_X25519__RSA_PSS_RSAE_SHA256__AES_256_GCM:256) (Exim 4.96) (envelope-from ) id 1okiLv-00A6G2-2b; Tue, 18 Oct 2022 10:53:36 +0200 Message-ID: Subject: Re: [PATCH 01/15] mac80211: split bss_info_changed method From: Johannes Berg To: Wen Gong , linux-wireless@vger.kernel.org Cc: ath11k@lists.infradead.org, ath12k@lists.infradead.org Date: Tue, 18 Oct 2022 10:53:34 +0200 In-Reply-To: References: <20220601073958.8345-1-johannes@sipsolutions.net> <20220601093922.347d47c279fe.I15937cfe8405999084f164ddf57390f8b8d2bc61@changeid> <3483d05c1d5a39b9243b54d9f28450344a897655.camel@sipsolutions.net> Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable User-Agent: Evolution 3.44.4 (3.44.4-2.fc36) MIME-Version: 1.0 X-malware-bazaar: not-scanned X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,SPF_HELO_PASS,SPF_PASS autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-wireless@vger.kernel.org On Tue, 2022-10-18 at 16:52 +0800, Wen Gong wrote: > On 10/18/2022 4:50 PM, Johannes Berg wrote: > > On Tue, 2022-10-18 at 16:47 +0800, Wen Gong wrote: > > > > + if (changed & ~BSS_CHANGED_VIF_CFG_FLAGS) { > > > > + /* FIXME: should be for each link */ > > > > + trace_drv_link_info_changed(local, sdata, 0, changed); > > > > + if (local->ops->link_info_changed) > > > > + local->ops->link_info_changed(&local->hw, &sdata->vif, > > > > + 0, changed); > > > I think you/someone will change here later for the "/* FIXME: should = be > > > for each link */", right? > > Maybe. I'm not actually sure it's really needed, it depends how we use > > this in the future. > >=20 > > > It lead error/kernel crash as below while reconfig single MLO link wh= ich > > > link id is 2. > > > When test with single MLO link which link id is 0, not found > > > error/kernel crash. > > I'm not surprised, I just worked on fixing reconfig in the last few > > days, will post it after some more review/testing. > Thanks. > > So I think that might not need changes *here*, but rather a proper FW > > restart. Which I haven't tested in AP mode though -- was the crash in A= P > > mode? > It is crash in station mode. try this? https://p.sipsolutions.net/0652bbbbe350b126.txt johannes