[PATCH v8 1/6] ip_frag: use a dynamic logtype
David Marchand
david.marchand at redhat.com
Fri Dec 1 14:11:06 CET 2023
Hello Konstantin,
On Fri, Dec 1, 2023 at 1:17 PM Konstantin Ananyev
<konstantin.ananyev at huawei.com> wrote:
>
> Hi David,
>
> > On Mon, Aug 14, 2023 at 6:31 PM Stephen Hemminger
> > <stephen at networkplumber.org> wrote:
> > > @@ -52,20 +54,20 @@ rte_ip_frag_table_create(uint32_t bucket_num, uint32_t bucket_entries,
> > > if (rte_is_power_of_2(bucket_entries) == 0 ||
> > > nb_entries > UINT32_MAX || nb_entries == 0 ||
> > > nb_entries < max_entries) {
> > > - RTE_LOG(ERR, USER1, "%s: invalid input parameter\n", __func__);
> > > + RTE_LOG(ERR, IPFRAG, "%s: invalid input parameter\n", __func__);
> > > return NULL;
> > > }
> > >
> > > sz = sizeof (*tbl) + nb_entries * sizeof (tbl->pkt[0]);
> > > if ((tbl = rte_zmalloc_socket(__func__, sz, RTE_CACHE_LINE_SIZE,
> > > socket_id)) == NULL) {
> > > - RTE_LOG(ERR, USER1,
> > > + RTE_LOG(ERR, IPFRAG,
> > > "%s: allocation of %zu bytes at socket %d failed do\n",
> > > __func__, sz, socket_id);
> > > return NULL;
> > > }
> > >
> > > - RTE_LOG(INFO, USER1, "%s: allocated of %zu bytes at socket %d\n",
> > > + RTE_LOG(INFO, IPFRAG, "%s: allocated of %zu bytes at socket %d\n",
> > > __func__, sz, socket_id);
> > >
> > > tbl->max_cycles = max_cycles;
> > > --
> > > 2.39.2
> > >
> >
> > Any reason not to use the IP_FRAG_LOG macro?
> > This is easy for me to fix when applying if you have no objection.
>
> As I remember, IP_FRAG_LOG is disabled by default.
> To enable it, user has to build it with -DRTE_LIBRTE_IP_FRAG_DEBUG or so.
Indeed, so nothing to do here, thanks.
--
David Marchand
More information about the dev
mailing list