> -----Original Message----- > From: David Marchand <david.march...@redhat.com> > Sent: Thursday, July 1, 2021 3:41 PM > To: Lin, Xueqin <xueqin....@intel.com>; Burakov, Anatoly > <anatoly.bura...@intel.com> > Cc: Jerin Jacob <jerinjac...@gmail.com>; Peng, ZhihongX > <zhihongx.p...@intel.com>; Ananyev, Konstantin > <konstantin.anan...@intel.com>; Stephen Hemminger > <step...@networkplumber.org>; dpdk-dev <dev@dpdk.org> > Subject: Re: [dpdk-dev] [RFC v2] porting AddressSanitizer feature to DPDK > > On Thu, Jul 1, 2021 at 8:48 AM Lin, Xueqin <xueqin....@intel.com> wrote: > > We also captured this type issues and selected one, reported in internal > system JARA and bugzilla in May. > > Pls check https://bugs.dpdk.org/show_bug.cgi?id=722 for detail, some > developers are analysis this issue in JIRA but not resolved so far, Anatoly > should also know this. > > Thanks for sharing this. > > > > It is DPDK issue, not this patch issue, it doesn't mean we not do much test. > > We put all of thousands of regression cases as case list for this patch > > test, > some issues still are tracking. > > Sorry, can you rephrase? > I sure hope there are not thousands of issues with ASAN. Relax, don't have so many issues. I mean we have thousands of automation dpdk cases, run the cases for DPDK version build with Asan tool, check dpdk code if have memory issue. Some issues are still WIP to check. > > > Are all identified issues reported in dpdk.org bugzilla? > It would help to have a hint in the bzs title, like "caught with ASAN". No, we reported common issue to Bugzilla. Good idea to add title.
> > I can't find a report about the infinite loop on invalid fds, making a dpdk > app > unable to exit gracefully with ASAN enabled. > But at least, I sent a fix for this, waiting for Anatoly to review. > https://patchwork.dpdk.org/project/dpdk/patch/20210614091213.3953-1- > david.march...@redhat.com/ > > > > > > > > Anyway, the memory allocator is your stuff, so your call. > > > > > > Prefix for the title of such a patch should be mem: > > Anatoly and Jerin like your patch, but it is still a RFC and there were > comments on a missing release note update and unit test. > Please address those comments and submit a regular patch. Sure, we are preparing the v1 patch according to comment. > > > Thanks. > > -- > David Marchand
