Re: [dpdk-dev] [PATCH v13 1/4] enable ASan AddressSanitizer

2021-11-03 Thread David Marchand
On Wed, Nov 3, 2021 at 10:36 AM Lin, Xueqin wrote: > > > Are they DPDK bugs or this tool code bugs? > > > > DPDK bugs that I started reporting some weeks ago. > > I mentionned it during CI call. > > bzs can be queried via: https://bugs.dpdk.org/buglist.cgi?quicksearch=[asan] > > Some fixes have be

Re: [dpdk-dev] [PATCH v13 1/4] enable ASan AddressSanitizer

2021-11-03 Thread Lin, Xueqin
t; ; Richardson, Bruce > ; dev ; Chen, Zhaoyan > ; Tu, Lijuan > Subject: Re: [dpdk-dev] [PATCH v13 1/4] enable ASan AddressSanitizer > > On Mon, Nov 1, 2021 at 4:37 AM Lin, Xueqin wrote: > > > ASan is a really interesting tool, but there are still issues in some > component

Re: [dpdk-dev] [PATCH v13 1/4] enable ASan AddressSanitizer

2021-11-02 Thread David Marchand
On Mon, Nov 1, 2021 at 4:37 AM Lin, Xueqin wrote: > > ASan is a really interesting tool, but there are still issues in some > > components. > > I caught a few and opened bugs. > > Are they DPDK bugs or this tool code bugs? DPDK bugs that I started reporting some weeks ago. I mentionned it during

Re: [dpdk-dev] [PATCH v13 1/4] enable ASan AddressSanitizer

2021-10-31 Thread Lin, Xueqin
> -Original Message- > From: David Marchand > Sent: Friday, October 29, 2021 10:51 PM > To: Peng, ZhihongX > Cc: Thomas Monjalon ; Burakov, Anatoly > ; Ananyev, Konstantin > ; Stephen Hemminger > ; Dumitrescu, Cristian > ; Mcnamara, John > ; Richardson, Bruce > ; dev ; Lin, Xueqin > > Su

Re: [dpdk-dev] [PATCH v13 1/4] enable ASan AddressSanitizer

2021-10-29 Thread David Marchand
On Fri, Oct 29, 2021 at 11:21 AM David Marchand wrote: > > Replying on this patch since there is no cover letter. > > This series looks acceptable to me for rc2. > > Patch 3 and 4 will be merged first, since they fix issues that would > be hit with ASan enabled. > > I have comments mainly on rewor

Re: [dpdk-dev] [PATCH v13 1/4] enable ASan AddressSanitizer

2021-10-29 Thread Peng, ZhihongX
> -Original Message- > From: David Marchand > Sent: Friday, October 29, 2021 5:22 PM > To: Peng, ZhihongX > Cc: Thomas Monjalon ; Burakov, Anatoly > ; Ananyev, Konstantin > ; Stephen Hemminger > ; Dumitrescu, Cristian > ; Mcnamara, John > ; Richardson, Bruce > ; dev ; Lin, Xueqin > > Sub

Re: [dpdk-dev] [PATCH v13 1/4] enable ASan AddressSanitizer

2021-10-29 Thread David Marchand
Replying on this patch since there is no cover letter. This series looks acceptable to me for rc2. Patch 3 and 4 will be merged first, since they fix issues that would be hit with ASan enabled. I have comments mainly on rewording in commitlogs and documentation. If you are fine with those comme

Re: [dpdk-dev] [PATCH v13 1/4] enable ASan AddressSanitizer

2021-10-24 Thread Peng, ZhihongX
> -Original Message- > From: Peng, ZhihongX > Sent: Wednesday, October 20, 2021 3:47 PM > To: david.march...@redhat.com; tho...@monjalon.net; Burakov, Anatoly > ; Ananyev, Konstantin > ; step...@networkplumber.org; > Dumitrescu, Cristian ; Mcnamara, John > ; Richardson, Bruce > > Cc: dev@

[dpdk-dev] [PATCH v13 1/4] enable ASan AddressSanitizer

2021-10-20 Thread zhihongx . peng
From: Zhihong Peng `AddressSanitizer `_ (ASan) is a widely-used debugging tool to detect memory access errors. It helps to detect issues like use-after-free, various kinds of buffer overruns in C/C++ programs, and other similar errors, a