<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sun, Nov 5, 2023 at 5:56 PM Avi Kivity <<a href="mailto:avi@scylladb.com">avi@scylladb.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div class="msg8901303231479849399"><div><div>Thanks, it makes sense. I'll get around to it "eventually".</div><div><br></div><div>On Thu, 2023-11-02 at 11:04 +0100, Thomas Monjalon wrote:</div><blockquote type="cite" style="margin:0px 0px 0px 0.8ex;border-left:2px solid rgb(114,159,207);padding-left:1ex"><div>Hello,<br></div><div><br></div><div>While looking at Seastar, I see it uses this patch on top of DPDK:<br></div><div><br></div><div>        build: add meson options of max_memseg_lists<br></div><div><br></div><div>        RTE_MAX_MEMSEG_LISTS = 128 is not enough for high-memory machines,<br></div><div>        in our case, we need to increase it to 8192.<br></div><div>        so add an option so user can override it.<br></div><div><br></div><div>        <a href="https://github.com/scylladb/dpdk/commit/cafaa3cf457584de" target="_blank">https://github.com/scylladb/dpdk/commit/cafaa3cf457584de</a><br></div><div><br></div><div>I think we could allow to configure this at runtime,<br></div><div>as we did already for RTE_MAX_MEMZONE:<br></div><div>we've added rte_memzone_max_set() / rte_memzone_max_get().<br></div><div><br></div><div>Opinions, comments, volunteers?<br></div></blockquote></div></div></blockquote><div><br></div><div>Hi Thomas, </div><div><br></div><div>Thank you for looking into it. I sent this patch[0] to DPDK 2 years ago. but i failed to find a solid proof to prove that we need such a massive number, and failed to follow-up on the suggestion[1] on calculating his number based on the lcores / numa node as I was trying to port the newer dpdk to seastar at that moment, so dropped the ball on my end, sorry for that. just revisited the places where we use RTE_MAX_MEMSEG_LISTS. it seems it would be a bigger effort to make it a run-time configurable option instead of a compile-time one.<br></div></div><div><br></div><div>--<br></div><div>[0] <a href="https://inbox.dpdk.org/dev/20211013205417.84119-3-tchaikov@gmail.com/">https://inbox.dpdk.org/dev/20211013205417.84119-3-tchaikov@gmail.com/</a></div><div>[1] <a href="https://inbox.dpdk.org/dev/2642296.XfZ1dg20Xv@thomas/">https://inbox.dpdk.org/dev/2642296.XfZ1dg20Xv@thomas/</a></div><div><br></div><span class="gmail_signature_prefix">-- </span><br><div dir="ltr" class="gmail_signature">Regards<br>Kefu Chai</div></div>