DPDK patches and discussions
 help / color / mirror / Atom feed
* [dpdk-dev] [PATCH] mbuf: document how to set length when attaching ext buffer
@ 2019-12-26 12:54 Olivier Matz
  2019-12-28  9:40 ` Jörg Thalheim
  0 siblings, 1 reply; 3+ messages in thread
From: Olivier Matz @ 2019-12-26 12:54 UTC (permalink / raw)
  To: dev; +Cc: Jörg Thalheim

From: Jörg Thalheim <joerg@thalheim.io>

Enhance API documentation of rte_pktmbuf_attach_extbuf() to
explain that the attached mbuf is initialized with length = 0.

Link: https://bugs.dpdk.org/show_bug.cgi?id=362

Signed-off-by: Jörg Thalheim <joerg@thalheim.io>
Signed-off-by: Olivier Matz <olivier.matz@6wind.com>
---

Hi,

This patch is a slight reword of a patch submitted in bug id 362 by Jörg
Thalheim.

@Jörg, feel free to comment if I missed something.

Thanks
Olivier


 lib/librte_mbuf/rte_mbuf.h | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/lib/librte_mbuf/rte_mbuf.h b/lib/librte_mbuf/rte_mbuf.h
index 219b110b7..2d4bda251 100644
--- a/lib/librte_mbuf/rte_mbuf.h
+++ b/lib/librte_mbuf/rte_mbuf.h
@@ -924,10 +924,14 @@ rte_pktmbuf_ext_shinfo_init_helper(void *buf_addr, uint16_t *buf_len,
  * provided via shinfo. This callback function will be called once all the
  * mbufs are detached from the buffer (refcnt becomes zero).
  *
- * The headroom for the attaching mbuf will be set to zero and this can be
- * properly adjusted after attachment. For example, ``rte_pktmbuf_adj()``
+ * The headroom length of the attaching mbuf will be set to zero and this
+ * can be properly adjusted after attachment. For example, ``rte_pktmbuf_adj()``
  * or ``rte_pktmbuf_reset_headroom()`` might be used.
  *
+ * Similarly, the packet length is initialized to 0. If the buffer contains
+ * data, the user has to adjust ``data_len`` and the ``pkt_len`` field of
+ * the mbuf accordingly.
+ *
  * More mbufs can be attached to the same external buffer by
  * ``rte_pktmbuf_attach()`` once the external buffer has been attached by
  * this API.
-- 
2.20.1


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [dpdk-dev] [PATCH] mbuf: document how to set length when attaching ext buffer
  2019-12-26 12:54 [dpdk-dev] [PATCH] mbuf: document how to set length when attaching ext buffer Olivier Matz
@ 2019-12-28  9:40 ` Jörg Thalheim
  2020-01-20 12:33   ` Thomas Monjalon
  0 siblings, 1 reply; 3+ messages in thread
From: Jörg Thalheim @ 2019-12-28  9:40 UTC (permalink / raw)
  To: Olivier Matz; +Cc: dev


Looks good to me!

Olivier Matz <olivier.matz@6wind.com> writes:

> From: Jörg Thalheim <joerg@thalheim.io>
>
> Enhance API documentation of rte_pktmbuf_attach_extbuf() to
> explain that the attached mbuf is initialized with length = 0.
>
> Link: https://bugs.dpdk.org/show_bug.cgi?id=362
>
> Signed-off-by: Jörg Thalheim <joerg@thalheim.io>
> Signed-off-by: Olivier Matz <olivier.matz@6wind.com>
> ---
>
> Hi,
>
> This patch is a slight reword of a patch submitted in bug id 362 by Jörg
> Thalheim.
>
> @Jörg, feel free to comment if I missed something.
>
> Thanks
> Olivier
>
>
>  lib/librte_mbuf/rte_mbuf.h | 8 ++++++--
>  1 file changed, 6 insertions(+), 2 deletions(-)
>
> diff --git a/lib/librte_mbuf/rte_mbuf.h b/lib/librte_mbuf/rte_mbuf.h
> index 219b110b7..2d4bda251 100644
> --- a/lib/librte_mbuf/rte_mbuf.h
> +++ b/lib/librte_mbuf/rte_mbuf.h
> @@ -924,10 +924,14 @@ rte_pktmbuf_ext_shinfo_init_helper(void *buf_addr, uint16_t *buf_len,
>   * provided via shinfo. This callback function will be called once all the
>   * mbufs are detached from the buffer (refcnt becomes zero).
>   *
> - * The headroom for the attaching mbuf will be set to zero and this can be
> - * properly adjusted after attachment. For example, ``rte_pktmbuf_adj()``
> + * The headroom length of the attaching mbuf will be set to zero and this
> + * can be properly adjusted after attachment. For example, ``rte_pktmbuf_adj()``
>   * or ``rte_pktmbuf_reset_headroom()`` might be used.
>   *
> + * Similarly, the packet length is initialized to 0. If the buffer contains
> + * data, the user has to adjust ``data_len`` and the ``pkt_len`` field of
> + * the mbuf accordingly.
> + *
>   * More mbufs can be attached to the same external buffer by
>   * ``rte_pktmbuf_attach()`` once the external buffer has been attached by
>   * this API.


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [dpdk-dev] [PATCH] mbuf: document how to set length when attaching ext buffer
  2019-12-28  9:40 ` Jörg Thalheim
@ 2020-01-20 12:33   ` Thomas Monjalon
  0 siblings, 0 replies; 3+ messages in thread
From: Thomas Monjalon @ 2020-01-20 12:33 UTC (permalink / raw)
  To: Olivier Matz; +Cc: dev, Jörg Thalheim

28/12/2019 10:40, Jörg Thalheim:
> 
> Looks good to me!
> 
> Olivier Matz <olivier.matz@6wind.com> writes:
> 
> > From: Jörg Thalheim <joerg@thalheim.io>
> >
> > Enhance API documentation of rte_pktmbuf_attach_extbuf() to
> > explain that the attached mbuf is initialized with length = 0.
> >
> > Link: https://bugs.dpdk.org/show_bug.cgi?id=362
> >
> > Signed-off-by: Jörg Thalheim <joerg@thalheim.io>
> > Signed-off-by: Olivier Matz <olivier.matz@6wind.com>
> > ---
> >
> > Hi,
> >
> > This patch is a slight reword of a patch submitted in bug id 362 by Jörg
> > Thalheim.
> >
> > @Jörg, feel free to comment if I missed something.
> >
> > Thanks
> > Olivier

Applied, thanks



^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2020-01-20 12:33 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-12-26 12:54 [dpdk-dev] [PATCH] mbuf: document how to set length when attaching ext buffer Olivier Matz
2019-12-28  9:40 ` Jörg Thalheim
2020-01-20 12:33   ` Thomas Monjalon

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