From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <dev-bounces@dpdk.org>
Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124])
	by inbox.dpdk.org (Postfix) with ESMTP id 3635DA0A0C;
	Mon, 28 Jun 2021 16:22:19 +0200 (CEST)
Received: from [217.70.189.124] (localhost [127.0.0.1])
	by mails.dpdk.org (Postfix) with ESMTP id 1BC374069F;
	Mon, 28 Jun 2021 16:22:19 +0200 (CEST)
Received: from mga17.intel.com (mga17.intel.com [192.55.52.151])
 by mails.dpdk.org (Postfix) with ESMTP id 178B540692
 for <dev@dpdk.org>; Mon, 28 Jun 2021 16:22:17 +0200 (CEST)
X-IronPort-AV: E=McAfee;i="6200,9189,10029"; a="188343572"
X-IronPort-AV: E=Sophos;i="5.83,306,1616482800"; d="scan'208";a="188343572"
Received: from orsmga007.jf.intel.com ([10.7.209.58])
 by fmsmga107.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384;
 28 Jun 2021 07:22:17 -0700
X-IronPort-AV: E=Sophos;i="5.83,306,1616482800"; d="scan'208";a="446588075"
Received: from aburakov-mobl.ger.corp.intel.com (HELO [10.213.249.202])
 ([10.213.249.202])
 by orsmga007-auth.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384;
 28 Jun 2021 07:22:15 -0700
To: David Marchand <david.marchand@redhat.com>,
 "Lin, Xueqin" <xueqin.lin@intel.com>
Cc: Jerin Jacob <jerinjacobk@gmail.com>,
 "Peng, ZhihongX" <zhihongx.peng@intel.com>,
 "Ananyev, Konstantin" <konstantin.ananyev@intel.com>,
 Stephen Hemminger <stephen@networkplumber.org>, dpdk-dev <dev@dpdk.org>
References: <20210610051352.48493-1-zhihongx.peng@intel.com>
 <20210615081205.101071-1-zhihongx.peng@intel.com>
 <CALBAE1NpAwTxgeRCa69AimAcqS+C4JEiRB=yiRKhdcQR7KZ8kA@mail.gmail.com>
 <BN7PR11MB2658E9AC54CE77137688DE97940F9@BN7PR11MB2658.namprd11.prod.outlook.com>
 <CALBAE1PdQXOXkbYXF8w73U5z_J7LC6T0aGEhC=WiqE_6gNHXEw@mail.gmail.com>
 <BN7PR11MB26580A732FD5795DC652B768940D9@BN7PR11MB2658.namprd11.prod.outlook.com>
 <CAJFAV8ydiQztQFoELpeebg-JYTjUtW=dLNXqwNfLoZ24WeH4QQ@mail.gmail.com>
From: "Burakov, Anatoly" <anatoly.burakov@intel.com>
Message-ID: <8af553e7-29e1-b46d-dc68-32efaa17c9c0@intel.com>
Date: Mon, 28 Jun 2021 15:22:12 +0100
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:78.0) Gecko/20100101
 Thunderbird/78.11.0
MIME-Version: 1.0
In-Reply-To: <CAJFAV8ydiQztQFoELpeebg-JYTjUtW=dLNXqwNfLoZ24WeH4QQ@mail.gmail.com>
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Language: en-US
Content-Transfer-Encoding: 7bit
Subject: Re: [dpdk-dev] [RFC v2] porting AddressSanitizer feature to DPDK
X-BeenThere: dev@dpdk.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: DPDK patches and discussions <dev.dpdk.org>
List-Unsubscribe: <https://mails.dpdk.org/options/dev>,
 <mailto:dev-request@dpdk.org?subject=unsubscribe>
List-Archive: <http://mails.dpdk.org/archives/dev/>
List-Post: <mailto:dev@dpdk.org>
List-Help: <mailto:dev-request@dpdk.org?subject=help>
List-Subscribe: <https://mails.dpdk.org/listinfo/dev>,
 <mailto:dev-request@dpdk.org?subject=subscribe>
Errors-To: dev-bounces@dpdk.org
Sender: "dev" <dev-bounces@dpdk.org>

On 18-Jun-21 10:04 AM, David Marchand wrote:
> On Fri, Jun 18, 2021 at 9:49 AM Lin, Xueqin <xueqin.lin@intel.com> wrote:
>>>> Suggest listing demo code and tool capture information for user to try if
>>> tool works, also add this part into doc.
>>>>
>>>>>
>>>>> # Also, Please update the release note for this feature.
>>>> Sure, we can update the release note if code merge.
>>>
>>> Probably you can send v1 version next i.e change the RFC status to get
>>> merged.
>>
>> Sure, we will send v1 patch if no obvious objection for that, hope patch could receive some ACKs and could success to merge, thanks.
> 
> How did you test this work?
> 
> UNH recently started testing with ASAN and it reveals leaks just in
> the unit test.
> 
> Merging these annotations will help catch more issues.
> But users will hit the current issues that we must fix first.
> 

As far as i can tell, the regular build is not affected by this patch, 
so no issues will be hit until someone actually runs the test. IMO it's 
better to merge it early to catch more issues than to gate the feature 
on the condition that we fix all bugs unrelated to this feature first.

-- 
Thanks,
Anatoly