> -----Original Message-----
> From: Eelco Chaudron [mailto:echau...@redhat.com]
> Sent: Monday, January 7, 2019 7:42 AM
> To: Stephen Hemminger <step...@networkplumber.org>; Dumitrescu,
> Cristian <cristian.dumitre...@intel.com>
> Cc: dev@dpdk.org
> Subject: Re: [dpdk-dev] [PATCH v4 1/2] lib/librte_meter: add RFC4115 trTCM
> meter support
>
>
>
> On 4 Jan 2019, at 20:42, Stephen Hemminger wrote:
>
> > On Fri, 4 Jan 2019 13:59:42 +0000
> > Eelco Chaudron <echau...@redhat.com> wrote:
> >
> >> This patch adds support for RFC4115 trTCM meters.
> >>
> >> Signed-off-by: Eelco Chaudron <echau...@redhat.com>
> >
> >
> >
> > Fix comment formatting.
>
> Hi Stephen, I left these warnings in on purpose, to match the existing
> comment style in the file.
>
> These specific warnings are from a structure copied from one right above
> which also has the same style and line lengths.
>
> As they are warning I though I should leave it as is, Christian what are
> your thoughts on this?
>
I agree with Eelco, we can later on send a patch to adjust the whole file to
the latest coding standard rather than change just a few lines inconsistently.
> >
> > ### [dpdk-dev] [PATCH v4 1/2] lib/librte_meter: add RFC4115 trTCM
> > meter support
> >
> > WARNING:BLOCK_COMMENT_STYLE: Block comments use * on
> subsequent lines
> > #172: FILE: lib/librte_meter/rte_meter.h:59:
> > +/** trTCM parameters per metered traffic flow. The CIR, EIR, CBS and
> > EBS
> > +parameters only count bytes of IP packets and do not include link
> > specific
> >
> > WARNING:BLOCK_COMMENT_STYLE: Block comments use a trailing */ on a
> > separate line
> > #174: FILE: lib/librte_meter/rte_meter.h:61:
> > +none-zero respectively.*/
> >
> > WARNING:LONG_LINE_COMMENT: line over 80 characters
> > #176: FILE: lib/librte_meter/rte_meter.h:63:
> > + uint64_t cir; /**< Committed Information Rate (CIR). Measured in
> > bytes per second. */
> >
> > WARNING:LONG_LINE_COMMENT: line over 80 characters
> > #177: FILE: lib/librte_meter/rte_meter.h:64:
> > + uint64_t eir; /**< Excess Information Rate (EIR). Measured in bytes
> > per second. */
> >
> > WARNING:LONG_LINE_COMMENT: line over 80 characters
> > #355: FILE: lib/librte_meter/rte_meter.h:406:
> > + /**< Number of bytes currently available in the excess(E) token
> > bucket */
> >
> > total: 0 errors, 5 warnings, 356 lines checked
