On Thu, Apr 15, 2021 at 1:09 AM McDaniel, Timothy <[email protected]> wrote: > > > > > -----Original Message----- > > From: Jerin Jacob <[email protected]> > > Sent: Wednesday, April 14, 2021 2:11 PM > > To: McDaniel, Timothy <[email protected]> > > Cc: dpdk-dev <[email protected]>; Carrillo, Erik G <[email protected]>; > > Gage > > Eads <[email protected]>; Van Haaren, Harry > > <[email protected]>; Jerin Jacob <[email protected]>; Thomas > > Monjalon <[email protected]> > > Subject: Re: [dpdk-dev] [PATCH v3 26/26] event/dlb: move rte config defines > > to > > runtime devargs > > > > On Wed, Apr 14, 2021 at 1:49 AM Timothy McDaniel > > <[email protected]> wrote: > > > > > > The new devarg names and their default values > > > are listed below. The defaults have not changed, and > > > none of these parameters are accessed in the fast path. > > > > > > poll_interval=1000 > > > sw_credit_quantai=32 > > > default_depth_thresh=256 > > > > > > Signed-off-by: Timothy McDaniel <[email protected]> > > > > Please check CI failures. Please make this practice to see the fate of > > the patch by CI after the patch submission to avoid additional delay > > in merging the patch. > > > > http://patches.dpdk.org/project/dpdk/patch/1618344896-2090-27-git-send- > > [email protected]/ > > The failures seems to be > 1) with a reference to dlb2 documentation in the 20.11 release notes
Please update the 20.11 release notes to fix the issue. > 2) an apply failure with the dqueue optimization patch. It requires the DLB > 25 patches > to have been previously installed. I added a depends-on line to its cover > sheet, but that > did not seem to help > 3) there are many false positives reported by check patches, most spelling > related. I do > not see any real issues there > > As for the code cleanup, this was very minor, such as not including mbox.h, > which we do not use. > Not sure it warrants its own patch, and I'm not sure I can even identify the > other minor cleanups, if any. > > Thanks, > Tim >
