<div dir="ltr"><div>Hi Konstantin,<br></div><br><div class="gmail_quote gmail_quote_container"><div dir="ltr" class="gmail_attr">On Mon, Jan 20, 2025 at 12:44 AM Konstantin Ananyev <<a href="mailto:konstantin.ananyev@huawei.com">konstantin.ananyev@huawei.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"><br>
<br>
> Some functions did not set rte_errno; for avoiding that, they are turned<br>
> into single-exit ones.<br>
> <br>
> Bugzilla ID: 1559<br>
> Signed-off-by: Ariel Otilibili <<a href="mailto:ariel.otilibili@6wind.com" target="_blank">ariel.otilibili@6wind.com</a>><br>
<br>
But reading through public API comments none of these functions are <br>
expected to set rte_errno value.<br>
If rte_mempool_create_empty() forgets to set rte_errno, why it is not<br>
enough just to add missing one in rte_mempool_create_empty()?<br></blockquote><div>Thanks for your feedback. Indeed, only  rte_mempool_create_empty() returns a rte_errno</div><div><a href="https://doc.dpdk.org/api/rte__mempool_8h.html#a82e301ee33ed7a263ceb4582655dc3ea">https://doc.dpdk.org/api/rte__mempool_8h.html#a82e301ee33ed7a263ceb4582655dc3ea</a></div><div><br></div><div>I'll push a new series with this change.</div><div><br></div><div>Regards,</div><div>Ariel<br></div></div></div>