From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pf1-f170.google.com (mail-pf1-f170.google.com [209.85.210.170]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 034073B0AE4 for ; Thu, 23 Jul 2026 20:22:04 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.210.170 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1784838126; cv=none; b=H6dseBVJlXaMJOBODQDl5UsvVBTWbBo++Bk9VH1EmVxQz9MbnJN7Bh7Ki9ZgWGe9zTO3/O9CUAShMIwsiXDACJ2ozQtu/0Qwcek+WjKSILdpR94rVInHxzzn8d3JXm2mBISTkv6E+U3K+alZVChitscA00BkW8lN3oHhLF4QXps= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1784838126; c=relaxed/simple; bh=UvNFjVJt3A7v6MKSV0fCcl/Kvk133EkSapsUZJ16PEM=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=E0C/UF+nmQyaanXP4TyHhBt490/gb4+nuxOvXEW//20Exy/MYCD2AFJQNDHPOAqBQFOIhEVYn5h3C4wXKAX4QldKmvO6EB6oQ5PL1ttBiNcgvvon5jptcetErQFzSFlbg2etHKtonqVU8XtrYMNTA2ZHGrQth31RYQQaIMuzvhU= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=cZNEcbvT; arc=none smtp.client-ip=209.85.210.170 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="cZNEcbvT" Received: by mail-pf1-f170.google.com with SMTP id d2e1a72fcca58-84a2dcede83so1518077b3a.3 for ; Thu, 23 Jul 2026 13:22:04 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1784838124; x=1785442924; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to:content-type; bh=Ivr3zf0J1qyWYjxVIvGBofjs0Q7zbDQA3/AdjyP35eM=; b=cZNEcbvTiD4aovB5VIqSTKHHIx7tU1nu++hX7evMp6p725WB/zi+Fl71gh9iTzIYl2 vFnXpSXFZxXcZuoqQxiJPOgL62U7cGz3u6QevfoP29dEuuQLiCrnhxHaTFq/w4OQW94U m0V5Xoi1p7rwIa7xmHIZaWmNmUVHmFl3ZQQPL4Xa7nGf+1IpEOn50BkgMprwpY2OTYib jVBA+Upy1CqT8HLJlfKEE2Os6W6F3AIXC3k0kRqIgB3X8LjS0b0fHjcRxVx8Urb3XEGR z75qrRj4hmjkYjndtOmZtezzT6LHYu3XwWvLFlJY2tjKdJFULCAIz1QUG0vh+U+jV5ik sq0w== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1784838124; x=1785442924; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-gg:x-gm-message-state:from :to:cc:subject:date:message-id:reply-to:content-type; bh=Ivr3zf0J1qyWYjxVIvGBofjs0Q7zbDQA3/AdjyP35eM=; b=BrtL6QqDdvnOocHzj0MxPKOPZhTM9iF6AxBtihv6+LsM6JdFjCi9w6osgJooht7tGa zNajEJXMxnjLQeQCsR08LAg7mpPpxlPa5c+Mi+FzKxnkVQe7D6GD9g/FpUnu9ISVkaFV Mu5ThMI6cILps4IzJS1KVdh07lPH/cVZVsI7uSN0D6oGuZJPH0TtPMDkRgyq/w+PvTXC flMGguWlQnY3YQRahUEydWli3Ihy0Vk6rr0fTrPQRxd5NlPG6316cQZLI9AHvCoRnNFf ftPpK7ZcdHwIn0GjvagmfI9cQ7zFh806V6ZpkX0SCmP0+vERItuCxZPWJNL3um5x8Whu PRig== X-Forwarded-Encrypted: i=1; AHgh+RrQ+7I9AS0dHn+znzhaVw9+h7884WV11UrTfhFgMdqgJ13Svh1T/BhK8xCwUGSjNthR5i7WI0PrTZR5OJNIkQ==@vger.kernel.org X-Gm-Message-State: AOJu0YyiMY7NVGkde54rq6Z1jZW6re5XAqP9qk7tyaXvkmkJDtPWAJ1f SNNTVRAoohWtI5ZL5YtRUEl4QVW12OHJuMRtaZ4WPHngQRMgeUCxHdhQ X-Gm-Gg: AR+sD12UuDwj4REbKcUm0FPGZ6GNEqyryrv3IrMTx9FNx1LgB9bBH24uHx512RHzf9f XzjaPFecuiArGAH4DZ6bxlxqVuFQVttBE98A7204IPnC8zHP6ZMFIB03LFnoFpeuh+ndR5G9w2H 7Al0tPuKiiijsrUwWA4DZ93TaGjMPlNqRMyHfjXVnfPn2xvFDnt80zG+M20OyV3moNpOIbevOu5 LL1PmdQ4KF4IA9Vlu4qq+Zt/raw/BsaEzKlB362nRzr1qNUjGELvnCGDg31qh99moDIjZ4VxyKL jSv91Ar5OOgDTfHTBVbWgqCXglRNS45ILdN4RlJeqw85Upu1cypNCTIAaEk+k/N9ouXUMaQgMmw 9cRklCNzjaM7xyMRPBTzxTulXDQMbpC3OoB9EidUtodqJyS232cFZbFMMVslTtoR1YA3J5leZQ0 hz4Wa8eq3K8twCCnrvwg3e3U9UMU19cTtWLx1iD/Ga8tGOUBNY1bY= X-Received: by 2002:a05:6a00:bb84:b0:848:4faa:480b with SMTP id d2e1a72fcca58-84e2bbed051mr5675121b3a.12.1784838124207; Thu, 23 Jul 2026 13:22:04 -0700 (PDT) Received: from KRHW1CJW23.bytedance.net ([61.141.241.115]) by smtp.gmail.com with ESMTPSA id d2e1a72fcca58-84e17604a46sm3670338b3a.60.2026.07.23.13.21.59 (version=TLS1_3 cipher=TLS_CHACHA20_POLY1305_SHA256 bits=256/256); Thu, 23 Jul 2026 13:22:03 -0700 (PDT) From: Zhao Li To: Mark Brown Cc: Johannes Berg , linux-wireless@vger.kernel.org, Johannes Berg , linux-kernel@vger.kernel.org, linux-next@vger.kernel.org, Miri Korenblit , Pagadala Yesu Anjaneyulu Subject: Re: linux-next: manual merge of the wireless-next tree with the origin tree Date: Fri, 24 Jul 2026 04:21:53 +0800 Message-ID: <20260723202153.97352-1-enderaoelyther@gmail.com> X-Mailer: git-send-email 2.50.1 In-Reply-To: References: Precedence: bulk X-Mailing-List: linux-wireless@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit On Thu, Jul 23, 2026 at 7:51 PM Mark Brown wrote: > On Wed, Jul 22, 2026 at 03:53:59PM -0500, Enderaoe Lyther wrote: > > > I noticed one semantic conflict later in ieee80211_rx_mgmt_assoc_resp(). > > I can't tell which bit of code you are talking about here. I see there > is a block at line 7274 of -next: > > if (elems->aid_resp) > aid = le16_to_cpu(elems->aid_resp->aid); > else if (!assoc_data->s1g) > aid = le16_to_cpu(mgmt->u.assoc_resp.aid); > else if (status_code == WLAN_STATUS_SUCCESS) > goto notify_driver; Sorry about the formatting in my earlier message. I should have quoted the code inline instead of top-posting. That last branch was introduced by 035ed430ce6a ("wifi: mac80211: avoid non-S1G AID fallback for S1G assoc") as: else if (status_code == WLAN_STATUS_SUCCESS) goto abandon_assoc; f13e573ab3f12 ("wifi: mac80211: notify driver before destroying assoc link") consolidated terminal association cleanup at destroy_assoc_data and removed abandon_assoc. The conflict resolution retargeted this branch to notify_driver, but notify_driver only calls drv_mgd_complete_tx() without destroying the association data. Since assoc_status is initialized to ASSOC_ABANDON at function entry, the equivalent target is destroy_assoc_data. A successful S1G association response with no AID Response element otherwise leaves assoc_data live instead of abandoning it. > but that is immediately after another goto notify_driver, there's > further notify_driver error handling afterwards and all the earlier > error handling is return statements so it looks at least unclear what's > supposed to be going on. Other goto notify_driver targets are intentional: - "if (!elems) goto notify_driver" is a pre-existing allocation failure bail-out; so association timeout handles cleanup. - The comeback path WLAN_STATUS_ASSOC_REJECTED_TEMPORARILY keeps assoc_data live deliberately for retry. A successful S1G response without an AID Response element is terminal. It needs to abandon association immediately, so the target should be destroy_assoc_data rather than notify_driver. > Please don't top post, reply in line with needed context. Understood. Sorry for the unclear top-posted reply. Zhao