patches for DPDK stable branches
 help / color / mirror / Atom feed
* RE: [PATCH 1/2] examples/ipsec-secgw: fix width of variables
       [not found] ` <>
@ 2024-01-09 17:38   ` Akhil Goyal
  0 siblings, 0 replies; only message in thread
From: Akhil Goyal @ 2024-01-09 17:38 UTC (permalink / raw)
  To: Konstantin Ananyev, Anoob Joseph, Radu Nicolau
  Cc: Konstantin Ananyev, Jerin Jacob Kollanukkaran, dev, stable

> > 'rte_eth_rx_burst' returns uint16_t. The same value need to be passed
> > to 'process_packets' functions which performs further processing. Having
> > this function use 'uint8_t' can result in issues when MAX_PKT_BURST is
> > larger.
> >
> > The route functions (route4_pkts & route6_pkts) take uint8_t as the
> > argument. The caller can pass larger values as the field that is passed
> > is of type uint32_t. And the function can work with uint32_t as it loops
> > through the packets and sends it out. Using uint8_t can result in silent
> > packet drops.
> >
> > Fixes: 4fbfa6c7c921 ("examples/ipsec-secgw: update eth header during route
> lookup")


> >
> > Signed-off-by: Anoob Joseph <>
Acked-by: Akhil Goyal <>

Applied to dpdk-next-crypto
> Acked-by: Konstantin Ananyev <>
Email Id shows warning in mailmap
Contributor name/email mismatch with .mailmap:
        Konstantin Ananyev <> is not the primary email address

I believe Konstantin is using both email ids, but mailmap is taking the first one as primary one.
Is it possible for you to use a single email ID and update mailmap accordingly?

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2024-01-09 17:38 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <>
     [not found] ` <>
2024-01-09 17:38   ` [PATCH 1/2] examples/ipsec-secgw: fix width of variables Akhil Goyal

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).