2022-06-22 19:03 (UTC+0200), David Marchand: > On Sat, Jun 18, 2022 at 1:29 PM Dmitry Kozlyuk <dmitry.kozl...@gmail.com> > wrote: > > > > Hi Fidaullah, > > > > Thanks for the fix, > > Acked-by: Dmitry Kozlyuk <dmitry.kozl...@gmail.com> > > This seems backport material. > Dmitry, Anatoly, do you agree? If so I'll mark it when applying. > > As for a Fixes: line, the closer commit touching this part is > 07dcbfe0101f ("malloc: support multiprocess memory hotplug") but I > wonder if this bug predates this commit.
I agree and the offending commit seems the right one. Before that commit the calculation accounted for the header, with 07dcbfe0101f it was lost: align = RTE_MAX(align, MALLOC_ELEM_HEADER_LEN); map_len = RTE_ALIGN_CEIL(align + elt_size + MALLOC_ELEM_TRAILER_LEN, pg_sz);