From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 30DE9A0C47 for ; Wed, 18 Aug 2021 09:08:14 +0200 (CEST) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 237384118E; Wed, 18 Aug 2021 09:08:14 +0200 (CEST) Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.133.124]) by mails.dpdk.org (Postfix) with ESMTP id 23E1F41153 for ; Wed, 18 Aug 2021 09:08:13 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1629270492; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=p1ia+L2DYiWUa+ZL/80JtPiYHRO0pLB5h1a4ajk2fyI=; b=C4J/3cRx7GwuaZRsYX4c65l5KXuRrgRmRirCBdbGqpYSmkfggRRIA2ACbBCwHhjGTygX4f vJSB2jovz44SoErFvQL15YHbhO4LFPIKqQ9BMG7aGp7Bzau+4iK4iPpecN0b+pf/zzciBv ATshOaxMwVEkx79eiJW8ClRK7APgUOo= Received: from mail-ua1-f69.google.com (mail-ua1-f69.google.com [209.85.222.69]) (Using TLS) by relay.mimecast.com with ESMTP id us-mta-486-9yecJRcaOh25gt_SC_er3Q-1; Wed, 18 Aug 2021 03:08:09 -0400 X-MC-Unique: 9yecJRcaOh25gt_SC_er3Q-1 Received: by mail-ua1-f69.google.com with SMTP id u9-20020ab07209000000b002ab978d1714so406450uao.23 for ; Wed, 18 Aug 2021 00:08:09 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=p1ia+L2DYiWUa+ZL/80JtPiYHRO0pLB5h1a4ajk2fyI=; b=YHb37eVsctcgye3M1lQoSoh2vKjLSTw7rK5FBtaBhIoI47laCgO/5pAJUxet4URpkk B5V4PW+yvpP4G/Tcj02J7W8W+VHN/H0kUaVImUoOsdKaQcXB3LK1WgMRqb/y+30eCK+R t1pEuIWQV4x83bZrSSbkbEIdR4FYnikFvAoCKZ2cUJTt7RJxBj1+VdFUHB4fNaRP3/xl 2ZBgTH9MoqIGxNaz1zjgpTlliQkDtcBOUgJJpk3isUId39HqaZVdCaIPCIPsJoaJpfUC /pLRZJRLeza1rLwK1S5IL9DfkhdPcCofZIBUzgkKtWHMRSw5Rh3KyDbGos2jXKCl3VJO 5V1Q== X-Gm-Message-State: AOAM533qmecqpmVjhNa0uxblIqFXJizekVFoRMHzx1Fl1toIC/8/wGFa L1aJrcPnB4vDq450oVoYAshC3tScn/KggMdOJ49S/gmyMnsSSlS6Fgykjk6i8yvrJlWoIZzOX8e d+eB1i8Yptl7FQYgnegop4is= X-Received: by 2002:ab0:608e:: with SMTP id i14mr5499426ual.126.1629270488765; Wed, 18 Aug 2021 00:08:08 -0700 (PDT) X-Google-Smtp-Source: ABdhPJzHrdf/Oc3DPnSM5W0JFvBmTWVDgfuJzOrFpwPWtZAiXQgKjX6n9/hUM7g+nJlqByVI2YKYnrx2fFtUJcq74Yo= X-Received: by 2002:ab0:608e:: with SMTP id i14mr5499415ual.126.1629270488559; Wed, 18 Aug 2021 00:08:08 -0700 (PDT) MIME-Version: 1.0 References: <20210813062154.62985-1-haiyue.wang@intel.com> In-Reply-To: From: David Marchand Date: Wed, 18 Aug 2021 09:07:57 +0200 Message-ID: To: "Wang, Haiyue" Cc: dev , "Zhang, Qi Z" , dpdk stable , "Yang, Qiming" , Xiaolong Ye , "Xing, Beilei" , "Wang, Ying A" Authentication-Results: relay.mimecast.com; auth=pass smtp.auth=CUSA124A263 smtp.mailfrom=dmarchan@redhat.com X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Type: text/plain; charset="UTF-8" Subject: Re: [dpdk-stable] [PATCH v1] net/ice: fix memzone leak when device init failed X-BeenThere: stable@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: patches for DPDK stable branches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: stable-bounces@dpdk.org Sender: "stable" On Wed, Aug 18, 2021 at 2:46 AM Wang, Haiyue wrote: > > > -----Original Message----- > > From: David Marchand > > Sent: Tuesday, August 17, 2021 17:19 > > To: Wang, Haiyue > > Cc: dev ; Zhang, Qi Z ; dpdk stable ; Yang, > > Qiming ; Xiaolong Ye ; Xing, Beilei > > ; Wang, Ying A > > Subject: Re: [PATCH v1] net/ice: fix memzone leak when device init failed > > > > On Fri, Aug 13, 2021 at 8:45 AM Haiyue Wang wrote: > > > > > > When flow engine initialization or FXP resource reset failed, it needs > > > to free the memory zone and unregister the interrupt callback. > > > > > > Bugzilla ID: 752 > > > Fixes: 84dc7a95a2d3 ("net/ice: enable flow director engine") > > > Fixes: 7615a6895009 ("net/ice: rework for generic flow enabling") > > > Fixes: 7edc7158d771 ("net/ice: cleanup RSS/FDIR profile on device init") > > > Cc: stable@dpdk.org > > > > > > Reported-by: David Marchand > > > Signed-off-by: Haiyue Wang > > > --- > > > drivers/net/ice/ice_ethdev.c | 10 ++++++++-- > > > drivers/net/ice/ice_fdir_filter.c | 2 ++ > > > 2 files changed, 10 insertions(+), 2 deletions(-) > > > > > > diff --git a/drivers/net/ice/ice_ethdev.c b/drivers/net/ice/ice_ethdev.c > > > index 64ee569525..8d62b84805 100644 > > > --- a/drivers/net/ice/ice_ethdev.c > > > +++ b/drivers/net/ice/ice_ethdev.c > > > @@ -2139,20 +2139,26 @@ ice_dev_init(struct rte_eth_dev *dev) > > > ret = ice_flow_init(ad); > > > if (ret) { > > > PMD_INIT_LOG(ERR, "Failed to initialize flow"); > > > - return ret; > > > + goto err_flow_init; > > > > Is it safe to call flow engine uninit callbacks when ice_flow_init() fails? > > If each engine->init/uninit handles its internal setting correctly, yes, > it's safe, if not, this single engine has BUG, let's fix it. ;-) That was my understanding, but I preferred to ask. Then, patch lgtm, thanks Haiyue. -- David Marchand