Mellanox / libxlio

Other
41 stars 19 forks source link

issue: 3781322 Fix for 100% CPU load #96

Closed iftahl closed 7 months ago

iftahl commented 8 months ago

Split poll_sn to poll_sn_rx and poll_sn_tx. Required after splitting cq to cq_rx and cq_tx.

Change type

What kind of change does this PR introduce?

Check list

pasis commented 8 months ago

Do we need these poll_sn? In most cases we have to create fake variables on stack just to satisfy such API. If it's necessary in some specific cases, probably we can provide poll_sn only in those cases?

iftahl commented 8 months ago

Do we need these poll_sn? In most cases we have to create fake variables on stack just to satisfy such API. If it's necessary in some specific cases, probably we can provide poll_sn only in those cases?

maybe we can do something to optimize it, but im not really familiar with all the flows. but when we split cq_mgr into 2 different classes, we needed to split also the poll_sn.

galnoam commented 7 months ago

@AlexanderGrissik, please give it priority. Thanks!