Received: by 2002:a05:7208:20d2:b0:82:bbfa:f723 with SMTP id z18csp58936rbz; Tue, 7 May 2024 10:28:11 -0700 (PDT) X-Forwarded-Encrypted: i=3; AJvYcCWONslMyj0cCsXa2X0+3TCK8u+EMrhlHfry/+zNGdko5acwYP8TPMJ0fm6CdHnLCuDbf/b42KlE3C7WgTCf/dZGXkemc4PFfllmNfnTCw== X-Google-Smtp-Source: AGHT+IFNsFLvQp0oVQCDKC0RGhHvKPHFt9mila6+9FDBQLEBwtYzYViX+RHIqMefN99MLdCeAeoY X-Received: by 2002:a17:902:f786:b0:1e6:622c:7bb4 with SMTP id d9443c01a7336-1eeb0593800mr3370445ad.19.1715102890882; Tue, 07 May 2024 10:28:10 -0700 (PDT) ARC-Seal: i=2; a=rsa-sha256; t=1715102890; cv=pass; d=google.com; s=arc-20160816; b=ve1f+rexJc5O+xXHQaMXebR400wLV/Tb87MWkZBg1sug1ldSFvS4pt6D6yUjAphdtN 3GDPNepWhvPa7LLT3IZZELMmfy63QMA+CF3nSbzLfb2D74qGAIqtDWBJ9a+OutozGw/f KFdBdpRCsuGxtYWCTbLRWGPi8h4bwAQACrJB1ysly0BKGS+ODUIXUWXDIXCRwjtLDeeZ RhgXEY5iEGLmmRa6bxpMQgUO0xc3C6o5iONtiKPofcdEm3hi3z9kGrhznY3QnuTrJfBA XQ4Yh71ZcKNTgfN5FKC1XYYBR4If7rFxmr6ZxKDF4kVV4DHS4TI0e0Boj1vrlxpbc7ke h0RA== ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=ui-outboundreport:content-transfer-encoding:in-reply-to:from :content-language:subject:references:cc:to:user-agent:mime-version :list-unsubscribe:list-subscribe:list-id:precedence:date:message-id :dkim-signature; bh=C+kLGBnJNEx50lH3P5EWi8k9Rwtm4zhAx5svv7hYxmE=; fh=Zg2s24gDO6MEO6MrFVk/2Gv8ZPiBP9Ca8mje4am8K/c=; b=LWQxhB5W1CEpYBghOerecr4Oq3nLAVOm2GeCR0vnP2ZC0Sjsv/zPVrHI/Qof6bnbPz NA4exgd2hc8KJo55lvlBc0GXOrEYvihOQIID3BmZrULQIimyqkc8DfCwu7348756/zTm +8QSJEckmTqaMGr7Qncxyv9ouMrEMPnLyHvSJVyf/TZUFywihrUqmljPHntCJXgAvmvD oDy8jBGFEtKV3FpVAwm15W7UFzs+4RyoZPlFuMR4ZC1NtPvehRd/SzYiOUB3UjIWj36b Klw2/dmX/Ps/CkRd02h5ofVAuVUwJraJOV3lMO3Z8D6+syVQ6JPZOUoaMhA6TKmED5uW +ONA==; dara=google.com ARC-Authentication-Results: i=2; mx.google.com; dkim=pass header.i=@web.de header.s=s29768273 header.b=JTAX6YfG; arc=pass (i=1 spf=pass spfdomain=web.de dkim=pass dkdomain=web.de dmarc=pass fromdomain=web.de); spf=pass (google.com: domain of linux-kernel+bounces-171914-linux.lists.archive=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) smtp.mailfrom="linux-kernel+bounces-171914-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=QUARANTINE sp=QUARANTINE dis=NONE) header.from=web.de Return-Path: Received: from sv.mirrors.kernel.org (sv.mirrors.kernel.org. [139.178.88.99]) by mx.google.com with ESMTPS id lm13-20020a170903298d00b001eb53e073e3si10665879plb.638.2024.05.07.10.28.10 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 07 May 2024 10:28:10 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel+bounces-171914-linux.lists.archive=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) client-ip=139.178.88.99; Authentication-Results: mx.google.com; dkim=pass header.i=@web.de header.s=s29768273 header.b=JTAX6YfG; arc=pass (i=1 spf=pass spfdomain=web.de dkim=pass dkdomain=web.de dmarc=pass fromdomain=web.de); spf=pass (google.com: domain of linux-kernel+bounces-171914-linux.lists.archive=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) smtp.mailfrom="linux-kernel+bounces-171914-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=QUARANTINE sp=QUARANTINE dis=NONE) header.from=web.de Received: from smtp.subspace.kernel.org (wormhole.subspace.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by sv.mirrors.kernel.org (Postfix) with ESMTPS id 4CB6F28390A for ; Tue, 7 May 2024 17:27:43 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 1F54516C84B; Tue, 7 May 2024 17:27:33 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=web.de header.i=markus.elfring@web.de header.b="JTAX6YfG" Received: from mout.web.de (mout.web.de [212.227.17.12]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id D919713C9B9; Tue, 7 May 2024 17:27:29 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=212.227.17.12 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1715102852; cv=none; b=AmDbmRO7hXgkbswFF1pYi1ZJywEPB+pNtq/BDNyQM6CIrL3LzTcBL2JriHnd0g7i/E5WmM7i5fVPxPC0FvkIJY2vzlbAOA/thDm9m91SfkwtL7TwxVSAIoDnMvU6UFzQD40i3N9hdVejpDKwswRzbzLXabDtXloGGYIU4maLpp8= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1715102852; c=relaxed/simple; bh=jD6uYVx8rbc/98HDA4ZpIZc/J2IavGiZtfdo5MM55Z4=; h=Message-ID:Date:MIME-Version:To:Cc:References:Subject:From: In-Reply-To:Content-Type; b=rNKC9S7ReC9Vf0dmpf9WwWivQgmSJGOkUP7LSEc0jj7uf8M7KmUS9v1hY+jBhLLd4H6WlG0sJULHqGN05IHFbwlYPw2TtJLQJccdRUkB8qsaYnIB+2mYYSBOoH4qky/m2fAoSRBNQ5e4LJgZYd1O4w+EofkiK6iqu/79Lvw6SDo= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=web.de; spf=pass smtp.mailfrom=web.de; dkim=pass (2048-bit key) header.d=web.de header.i=markus.elfring@web.de header.b=JTAX6YfG; arc=none smtp.client-ip=212.227.17.12 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=web.de Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=web.de DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=web.de; s=s29768273; t=1715102811; x=1715707611; i=markus.elfring@web.de; bh=C+kLGBnJNEx50lH3P5EWi8k9Rwtm4zhAx5svv7hYxmE=; h=X-UI-Sender-Class:Message-ID:Date:MIME-Version:To:Cc:References: Subject:From:In-Reply-To:Content-Type:Content-Transfer-Encoding: cc:content-transfer-encoding:content-type:date:from:message-id: mime-version:reply-to:subject:to; b=JTAX6YfG4221tvYVuORg9Qks6qzlzqYrIpBkxhL3CoBim/NiacKEeHx/05sShm/H uJ6b5np4XfnbDUmYRFsSxNuQr+OHkvfnCVFA1R5zJTAKXmc3xDrLFlvZ6EiE0yKxT FFwgI2WtPF2slNoMHwoQdi9R+AE8n0SW3M1GLpn5FNBAWe8F2XTtnHTYS8mDmPZ1V yK1ydzeyCN+9YeX6tS5KHddlxkyfuFboltvT2EZ+/uzZXJB37DGfQabr/HD3JShz8 9Jhh9nbDdHM9HGj22eKUkz+tYiDD2XgOwx/sEKJBVDAlRO0YeQDIRyS5L7FY6JZ0W X8+6sMhnRcpaM4WkVA== X-UI-Sender-Class: 814a7b36-bfc1-4dae-8640-3722d8ec6cd6 Received: from [192.168.178.21] ([94.31.89.95]) by smtp.web.de (mrweb105 [213.165.67.124]) with ESMTPSA (Nemesis) id 1M9ISr-1s0pwq1NeJ-004jB4; Tue, 07 May 2024 19:26:51 +0200 Message-ID: Date: Tue, 7 May 2024 19:26:47 +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 To: Jijie Shao , Peiyang Wang , netdev@vger.kernel.org, kernel-janitors@vger.kernel.org, "David S. Miller" , Eric Dumazet , Jakub Kicinski , Jiri Pirko , Paolo Abeni , Salil Mehta , Simon Horman , Yisen Zhuang Cc: LKML , Hao Chen , Jie Wang , Przemek Kitszel , Yonglong Liu References: <20240507134224.2646246-5-shaojijie@huawei.com> Subject: Re: [PATCH V3 net 4/7] net: hns3: release PTP resources if pf initialization failed Content-Language: en-GB From: Markus Elfring In-Reply-To: <20240507134224.2646246-5-shaojijie@huawei.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable X-Provags-ID: V03:K1:0Hw9s5sYlQhOn2cEZWzIg4hsX9x+raV9DEjiYxo/MYl1g1cVGM4 y5YzKwsL6ArzR2UqaipP1soGeCluBn+hLlGseB/yOKxSxz2bKA6CjBZmPdfWD5oEqWJOgaG /+ZEyIq6Su+lO9eBzVahq5/poR26TICcZun4VLY9Rqx2IBu3v+WtgTkRMt7/vDzETmb6c30 1vfQBFj3hy2oAjfJrfIgA== X-Spam-Flag: NO UI-OutboundReport: notjunk:1;M01:P0:TAdli9LW03E=;qK31+OwneJw7czSvxXtU3Pgjjv8 1wIg3KHhaZaq4YPd7ugU1jy5+s1N7q4CZuNNhQXD6lZDNO0PjVyTN7uDQf+vt3SY4Gqz1qyYu XPDxL2nXbMInt4LeqdGgrII0OcRSe/Trm1boQ5xhLdXMz/22+kEY7xhyeSE9JGZC30MicZCTb Nnn1CHvjX2rWE58DkQVv8bslrBitYTNFz/uamvzahJ8TDZbvQzTSPJ3CAxglainfDuf79GFeU cEPzG+5zcc3kF3in3F5P5dZZ2Vy/mD+FUu+Ol+5r2edAa7gORDdpGyDRpqPr/gwRgwBfks8hJ s6jNeTnqLLbIChBZ9/Zbic0LNAUhpnpXbt3hj5HPx2ycuR92uB8NSuCgLbtI/SjW7R/D9ITAx 7YCet0KNFmJ1pCbKcXIWzEgegUVG5iUgaasMlTKvsDbJri1av6fdIh0IJQo8p0SCLLGgDUCY7 rnKzij2syehPZKmYUkas9f5eqXudnS+2gOpJv1/TkEcrhVrUMcLA8iU5I35w8oVcVUNBCMJaS HWPzau6ofEvJiBNlSiw+0Xh/tmNsyfnq8FRriypQQw8qt8ziJ6tESJ9rfyc1wzbnI20rkTN+M TqUY8XRJchtV32I4o6v4W123VQp1MSUx809VJmYQkJI9RPZQZsAa1LN0uxfciJl77iFrsI965 qheH/VImcNxM3NePNmwXAfLjSWLkn2rzywkFwwifOCDGdfNoWivXiqTiTL5CmRp8DVHIqbXR5 VnlgGBF0yFe82xeZK50U+AtKwjtlMzALR3x+/qhwo0PQqzWcyAERyOTPiB7KHkysFIny0cYc5 BahnTBq1NvAjRfffHyrXZ4NFHgrtxdwSZ9QPLA2wJO1lw= =E2=80=A6 > code, hclge_ptp_uninit is called to release the corresponding resources. Wording suggestion: Add a jump target so that the corresponding resources are released with a hclge_ptp_uninit() call. See also: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Do= cumentation/process/submitting-patches.rst?h=3Dv6.9-rc7#n94 Regards, Markus