From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wr1-f43.google.com (mail-wr1-f43.google.com [209.85.221.43]) (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 5039944606B for ; Tue, 11 Aug 2026 13:56:12 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.221.43 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1786456574; cv=none; b=l3vlLLVJNcua/lUrKjDlRiduuEbiPAPs1/J91YnDNoooJtJv1bmXH6xQQ+4WGYnJb79/IkPLTqvMs6BnyhaYINq9XBbO88q6TfuuTmMw5ariS8OlQLfIR7fDkfIdGCLf+5nmD6QenSb80hDxhGaPKDLuS0o0zmP2Kgoef6raNOM= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1786456574; c=relaxed/simple; bh=mPX092SAxdnLkN/6z9ICmIXNghJissgSgQYrcMNDQ88=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=WwG4PRhNN/mpnGw5J547FksA1MuvLWptA6DLT6PHV6+YsGOz71TofUTpXVVpEp0M1vAN1wLKHGzwcuhxDfWuMWXGohGPre8NXY4TyBzB53v3jnC6GXpPx6c2KyRmfOPXuZAdm30kgZmGhjKc1KDhw/HOk430Vk02kB64tIRMq+0= 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=hC4Agfhz; arc=none smtp.client-ip=209.85.221.43 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="hC4Agfhz" Received: by mail-wr1-f43.google.com with SMTP id ffacd0b85a97d-4731f5ffa74so254938f8f.1 for ; Tue, 11 Aug 2026 06:56:12 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1786456570; x=1787061370; darn=vger.kernel.org; h=content-transfer-encoding:content-type:in-reply-to:autocrypt:from :content-language:references:cc:to:subject:user-agent:mime-version :date:message-id:from:to:cc:subject:date:message-id:reply-to :content-type; bh=QKydGfzalUMU8QK5vpmVMtplqbkcNF1gUk8nxhRQcp8=; b=hC4Agfhz5Fswem8ScJFdVM4gJwnPSHTmKttvqDNjs17JbasXU6MgP5FMLKtikGQcTQ W00rejijq+UNJHL9I5ljJfNUQMoNztMne6obQ864gevr09FEbkwIbH/gpmLKTbizDFPs f0YgQVwE84lqlJYd2U+E7f/yxFKtBJEsO0d91CvCFqKuRE91WLgOa6rfmuxAb0uP0J2F cpd2RjApUAnbv99iyJnjZ/bDXH9n7ab1UkTs/EAcCDgq5w4aMy69Psm/KoPBinvnlle5 juqil1UXck2zieKh/vfEYGQwo7XORBvS9oewHO+CXXTND0BC3puzu3aR8Fu+0VBSdaqW QIdQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1786456570; x=1787061370; h=content-transfer-encoding:content-type:in-reply-to:autocrypt:from :content-language:references:cc:to:subject:user-agent:mime-version :date:message-id:x-gm-gg:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to:content-type; bh=QKydGfzalUMU8QK5vpmVMtplqbkcNF1gUk8nxhRQcp8=; b=kMq6Ik35AGW3AmwssWGm2Tt3NryMXPHUD60k9trUElbr3vruEO8R8hkLy1pH9K+23o b3zimJOpC8gtxk2fKHUx6I8aQW/VTemLVY3Opl4bIsxzwvE1IHrT1GBiABB/eken7Yxe uW0Q9Yr6RvCal1F3V7ib7Zw2O5hYVGf353aQ0YSLKR64H3QGOdClavzr6c6FXNjqaeU3 0DqktPhcrw+wv/g9LsdAdz/5yfGtosBotxaINfjcDIvDzOcS6bEq9c/Hci5fBe1JbUZX AXs0Lyh2kXkoqv6IEhOvVjKO6U+1boIXGIWL+jtHXh2vMcq30vjt0PaSvRoc1/MaD/qc mQWQ== X-Forwarded-Encrypted: i=1; AHgh+RqFKwXvZGy7MvJs7w9T56tGfwMLBQl/BwmfpWXBK+L6ZW5R1GQfMBlkUy1o9b8VLLmPD/XoJ5oNaZDLKl4=@vger.kernel.org X-Gm-Message-State: AOJu0Yxao/24FRLgiHBN6JjhaiPWl5Dx3vcGB7aia2WmYYoDoK9my1Hv ShbFosezn0wMzGkW7x3RMad1meBRNeWTDL5Un+vX1k+7HAM0xTQbJGqi X-Gm-Gg: AR+sD10oEVAGgRZpxMWxSLFLFAZceLYwaAvGmXUzOTYktJ4yqEbTSuIf8X3me66pHyo 0MWEyQheTwkJG0OsKAk/7loB09tGgHq/Ik10KyQ8Skggw0T/fEmMmBcFCTXCzJyETn8R1fAM7fV ukVqCYyaYkztxmnWflJfUwiVanlv5/LQ+X8CLZCnwb6HWityXLPaqmFAVnp34fS7kJB8STWcz0m VmB4wMy3oEzQMEpEUS9KvxDdDt2JkZ24nAdHTuMD7CYLzxBD+OqG4vc9r4Nh2AtCMtCWDWK+gu0 adRQ0TfNJHh/lLZ7jSstMgHziYyOjVSsZO6jNaABGBn9jvWqV8FW4B6dZzUEbfVOIoEKVVVJT30 j/j0DKKjkGotb+jLnYzqxU1DmbssaSOXTDbEj5ZgKjf0pf4qXU2RystD40yTQCSEj58niU1t8cZ 0dQ3DFvJwo2gVCgeqOzVWqSYLd/9iPSfFUoKGWZd6aJl/eVLWIbO05FreL+zcpuTa2xGjxhiEJ8 ipuXn5GgQXayB34CE5D6jb6cjLbi5TRgSDyjsC529vb3f+ppt0k71m7jguAoB4COUtjnpgc/OzB /4TnodgjrRgd X-Received: by 2002:a05:600c:35cb:b0:499:73ca:c419 with SMTP id 5b1f17b1804b1-4997846a357mr36107395e9.4.1786456570151; Tue, 11 Aug 2026 06:56:10 -0700 (PDT) Received: from [192.168.2.69] (dynamic-077-179-054-188.77.179.pool.telefonica.de. [77.179.54.188]) by smtp.gmail.com with ESMTPSA id 5b1f17b1804b1-499793ce27bsm27603255e9.3.2026.08.11.06.56.09 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Tue, 11 Aug 2026 06:56:09 -0700 (PDT) Message-ID: <063eb98e-95d2-4d95-9720-08b351fb6e6e@gmail.com> Date: Tue, 11 Aug 2026 15:56:08 +0200 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH net] net: hsr: free learned nodes on device setup failure To: Hangbin Liu Cc: netdev@vger.kernel.org, davem@davemloft.net, edumazet@google.com, kuba@kernel.org, pabeni@redhat.com, horms@kernel.org, fmaurer@redhat.com, luka.gejak@linux.dev, kexinsun@smail.nju.edu.cn, arvid.brodin@alten.se, linux-kernel@vger.kernel.org References: <20260808110814.1637-1-xiexinet@gmail.com> Content-Language: en-GB From: Xin Xie Autocrypt: addr=xiexinet@gmail.com; keydata= xsDNBGpiHugBDADJ5KuJaOYfUx7TGbRw0UE1km4dLnJomCSNtgq2T9BqgZ6TRPEy37soAmW3 XxCPvhfL6CPLldzy6O6EkUD7/8zlbQP1DM6gUIQUyKxxkb8uR+oCogDKjsxt0QPycP/dR9fk SL4qKReprnkOOcGeLahLHzofVdn46mc5igCBsAA/PPSpo1GqsfGk+zYV97rSVa8zhnSXsqZ/ 9DbUfQynZKVqYN4tm1B30JE5SEWDCko95Qw9fAORX2W356MOQJGr05z9qMSy7CkF7bAUkziG d6ZSPsGEsel5jYByP8ZvwQ2A8kdKpRy8+zQCaC+vEotTH7Fs7SIzE/Z5mEVIi1xQqEzN1rqG BW7KgzRDWjw8lKsoBbxv20nfhUzuZLan2HzUJ+LxZ18Os4zkWg9sxHgMi+AGqn/+0ppbDcOH 9Jm072/xA7O1MCXLgr4FOP+4PJH1vcqU+0EZVz7KCjrwkdz+l5Ojso2FLXq2MGoqWGShIsDV bF1JYpgm4spdPbnISkidUC0AEQEAAc0cWGluIFhpZSA8eGlleGluZXRAZ21haWwuY29tPsLB DQQTAQgANxYhBAHOahKZFnMu2n/CxrTrSufyPi3LBQJqYh7oBQkFo5qAAhsDBAsJCAcFFQgJ CgsFFgIDAQAACgkQtOtK5/I+LctmngwAlmrGPWnWxDW6PSJeR9SM0faqwuY33TWAE35nshm/ EEkBgpVVhG4z4Cdy7L+6TG6NHVDnvl+IHLyyOZlL3LQPQIpiKVgo6jHmm9TF6mK+Vo3nsXAA uSJYu9iY9Aywmy2JVQ15ttQ0NfcoZ0sV14bT7pSr9zXB6D8p3XA4AQI6IBeLvj993Z1+tPyC 6BJ+2aWWLBx7xGpFo3X8dhjS/Lm8QoietKbI+ACefKKCDNiM2KG53P76wdXJp52u4dqP6eFc +Qv35QefnALoPkOGQEdd1qvUycyZhuaCj8h0AxDu7bL3G/IM8G2K8qsbSJuui8zCSZ9b/QZf H7u1nevPOgxyWC1Pep/TbRZ62ktn8lrmWA8PZjuaJJQGjahuK1GgTQZNG+7DB7NwjLr6aXIm JirtT4FeR+IUovvL/ll+VHzpGpFbgXz4GTsWsikQkT1huFSB8MRLlWMXdAKblL96L754j8Z5 p/YV26YTppXI56uJ3XyOwZSrfrK1pfqNIp2Yr9vUzsDNBGpiHukBDACePM1YZ1FvAiFay/2v KVDNpuazmpVb1CBFy+rqM9HcuHe+5CuxLd4RI4hb0qmjlm5Vi5M8+AuNB/wi/f8oDuhJwlMn v9L0lfgpRRdShnB26hnt2wWwWFgOU3BwymSruxhEYq+eIjjxCAo6yW7Qm+ArZ+riETMI67sy ZyLx6o4yRiWxOrh1nhwV/f6PIOl+Iv4yLAG3eFXpU7/EZpim610bKMwcXEpiRkf3NjiflIha adoKHV0LCiVNrU7r8TICjjnugpl1tXAR2RlaRcSxmvOzrXY2O/xBLpwPr5erbdk8RCaxUzS0 kK0hw+/On3Sr4Z4gtLl2xtDkjRh3KQ9ZqMPrFD/5XOs842ueNivwIyAf2kQJjKfJGWc771EM 0ZWo1r1AthbuM+IeN5QM74rMJxaf5WyDX3ZeqBA55fCJGJ3zrp7Tqeoq8DYufiNkppE9S71z Cvxva2f8mUWTlTUf8s33sRD7C87XpT0whKpn1I9/nJB9Kb0dKyRCdcsT4o1ynxUAEQEAAcLA /AQYAQgAJhYhBAHOahKZFnMu2n/CxrTrSufyPi3LBQJqYh7pBQkFo5qAAhsMAAoJELTrSufy Pi3L/iwL/3aoCeq19Pog3fqisRyEZeX8pXw2GYlrJWYoM0mQ1USaTpdsMwCXaoFMoSBqid0F VPrqlH4bSIEEAzVmFPX0NjGhREDMrQ9eR5S9yqSpBvX+hz+xgpt4NOPhbVQmV+9f4lX9TJT7 GOO3EX2eeokg6ZAISycCAx7srXuc8oyRTJtnUQziH9GXrEeon88LHf1QHg0Z0sL/c7DfDqvW 9924M4ngQAZ8RgCBGZocoWuXCemSPkfQVLt5DzycSgSQODhbNSEmr+jb1V9Svf9D9k0CQfEY cB3uwOHWb6Z/yaEARbGZPSPn89YjXBncQTpgSvUM578b5x3pKi6PjZB/hwtUsRgrWGWoIwRi +ZmcHnCzsuo6uuNYX8VvmWqYsVajykjnpjdCGmU89QUsVmrtnDPBrfr8VVJcZPnCJhE595Pk LO29Oj0au9kr7kMW5Q+fBeHj5zz6VggAaRRU74/sgtwLODBWESPZ9LqUUdB5L/XbrSq2VDlD sNySbH3veZ/55EtlXA== In-Reply-To: Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit On 10/08/2026 03:31, Hangbin Liu wrote: > Should we use this fix tag? The proxy_node_db is added in > 5055cccfc2d1 ("net: hsr: Provide RedBox support (HSR-SAN)"). > > Thanks > Hangbin Thanks for checking. I believe 81ba6afd6e64 is the right tag. The demonstrated leak is node_db, and the window that creates it was born in 81ba6afd6e64: that commit moved per-device RX handler registration into hsr_dev_finalize(), ahead of steps that could still fail (the second handler registration, self-node allocation, register_netdevice()), while the failure unwind never released nodes learned through the already-live handler. Before it, reception used the module-global dev_add_pack() handler, which could not reach an instance until register_hsr_master() at the successful end of finalize, so there was nothing to leak. proxy_node_db cannot hold entries on any current finalize error exit: it is fed only by interlink-port RX, and the interlink add is the last failable step in finalize. On this path, the second hsr_del_nodes() call is a harmless no-op on an empty list, keeping the unwind symmetric with hsr_dellink(). Using 5055cccfc2d1 would instead keep the fix away from older stable trees, where the node_db leak does exist. -- Xin