From: Shweta Choudaha <shweta.choudaha@gmail.com>
To: Ferruh Yigit <ferruh.yigit@intel.com>
Cc: Bruce Richardson <bruce.richardson@intel.com>,
"Lu, Wenzhuo" <wenzhuo.lu@intel.com>,
"dev@dpdk.org" <dev@dpdk.org>,
"shweta.choudaha@att.com" <shweta.choudaha@att.com>,
"Zhang, Helin" <helin.zhang@intel.com>
Subject: Re: [dpdk-dev] [PATCH 1/1] net/ixgbe: Add API to update SBP bit
Date: Tue, 1 May 2018 23:15:31 +0100 [thread overview]
Message-ID: <CAActUdd=Q0-RhvtGMSKgGPG3sERhe+AKvS3BTrfG4wSLiiM60w@mail.gmail.com> (raw)
In-Reply-To: <8866e4a2-3c21-9f94-37d9-947fe0b93cac@intel.com>
Hi Helin, Qi, Wenzhuo,
Any update for this patch.
Anything else needed or can this be merged ?
Thanks,
Shweta
On Thu, Apr 26, 2018 at 11:56 AM, Ferruh Yigit <ferruh.yigit@intel.com>
wrote:
> On 4/26/2018 11:20 AM, Shweta Choudaha wrote:
> > Hi Ferruh/ Helin,
> >
> > Anything else needed for this patch
> >
> > http://dpdk.org/dev/patchwork/patch/37802/
> > Or can this be now acked - Have addressed all comments so far.
>
> Hi Helin, Qi, Wenzhuo,
>
> Any more comment on patch?
> Please let me know if you prefer me getting this directly to next-net?
>
> >
> > Thanks,
> > Shweta
> >
> > On Tue, Apr 10, 2018 at 4:58 PM, Shweta Choudaha <
> shweta.choudaha@gmail.com
> > <mailto:shweta.choudaha@gmail.com>> wrote:
> >
> > Thanks for the review.
> > I have sent an updated patch and will send patch to remove
> experimental from
> > API for next release
> >
> > Thanks,
> > Shweta
> >
> > On Wed, Apr 4, 2018 at 11:40 AM, Bruce Richardson
> > <bruce.richardson@intel.com <mailto:bruce.richardson@intel.com>>
> wrote:
> >
> > On Wed, Apr 04, 2018 at 11:14:35AM +0100, Ferruh Yigit wrote:
> > > On 4/4/2018 3:25 AM, Lu, Wenzhuo wrote:
> > > > Hi Shweta,
> > > >
> > > >
> > > >> -----Original Message-----
> > > >> From: dev [mailto:dev-bounces@dpdk.org
> > <mailto:dev-bounces@dpdk.org>] On Behalf Of Shweta Choudaha
> > > >> Sent: Friday, February 23, 2018 7:59 PM
> > > >> To: dev@dpdk.org <mailto:dev@dpdk.org>
> > > >> Cc: shweta.choudaha@att.com <mailto:shweta.choudaha@att.com
> >
> > > >> Subject: [dpdk-dev] [PATCH 1/1] net/ixgbe: Add API to
> update SBP bit
> > > >>
> > > >> From: Shweta Choudaha <shweta.choudaha@att.com
> > <mailto:shweta.choudaha@att.com>>
> > > >>
> > > >> Add ixgbe API to enable SBP bit in FCTRL register to allow
> > receiving packets
> > > >> that may otherwise be considered length errors by ixgbe and
> dropped
> > > >>
> > > >> Signed-off-by: Shweta Choudaha <shweta.choudaha@att.com
> > <mailto:shweta.choudaha@att.com>>
> > > >> Reviewed-by: Chas Williams <chas3@att.com <mailto:
> chas3@att.com>>
> > > >> Reviewed-by: Luca Boccassi <bluca@debian.org <mailto:
> bluca@debian.org>>
> > > >> ---
> > > >
> > > >
> > > >> --- a/drivers/net/ixgbe/rte_pmd_ixgbe_version.map
> > > >> +++ b/drivers/net/ixgbe/rte_pmd_ixgbe_version.map
> > > >> @@ -52,3 +52,9 @@ DPDK_17.08 {
> > > >> rte_pmd_ixgbe_bypass_wd_timeout_show;
> > > >> rte_pmd_ixgbe_bypass_wd_timeout_store;
> > > >> } DPDK_17.05;
> > > >> +
> > > >> +EXPERIMENTAL {
> > > >> + global:
> > > >> +
> > > >> + rte_pmd_ixgbe_upd_fctrl_sbp;
> > > >> +} DPDK_17.08;
> > > > The patch is fine except we should use 18.05 here.
> > >
> > > We have a rule to have an API as experimental for one release.
> > >
> > > Not sure about this rule covering the PMD specific APIs but by
> default
> > it is, we
> > > can discuss to have an exception for PMD APIs or not.
> > >
> > > After one release developer should send patch to remove
> experimental
> > from API
> > >
> > +1 for following the process.
> > I don't see why pmd-specific APIs should be a general exception.
> Any
> > exceptions should be on an individual case-by-case basis IMHO
> >
> >
> >
>
>
next prev parent reply other threads:[~2018-05-01 22:15 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-02-23 11:59 [dpdk-dev] [PATCH 0/1] " Shweta Choudaha
2018-02-23 11:59 ` [dpdk-dev] [PATCH 1/1] " Shweta Choudaha
2018-04-04 2:25 ` Lu, Wenzhuo
2018-04-04 10:14 ` Ferruh Yigit
2018-04-04 10:40 ` Bruce Richardson
2018-04-10 15:58 ` Shweta Choudaha
2018-04-26 10:20 ` Shweta Choudaha
2018-04-26 10:56 ` Ferruh Yigit
2018-05-01 22:15 ` Shweta Choudaha [this message]
2018-04-10 15:42 ` [dpdk-dev] [PATCH v2] " Shweta Choudaha
2018-05-08 1:47 ` Zhang, Helin
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to='CAActUdd=Q0-RhvtGMSKgGPG3sERhe+AKvS3BTrfG4wSLiiM60w@mail.gmail.com' \
--to=shweta.choudaha@gmail.com \
--cc=bruce.richardson@intel.com \
--cc=dev@dpdk.org \
--cc=ferruh.yigit@intel.com \
--cc=helin.zhang@intel.com \
--cc=shweta.choudaha@att.com \
--cc=wenzhuo.lu@intel.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).