-
Notifications
You must be signed in to change notification settings - Fork 327
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Queue metadata with multiqueue #1258
Comments
Yes, in the simple_switch implementation it seems that the qdepth is measured across all priority queues for a given port: behavioral-model/targets/simple_switch/simple_switch.cpp Lines 411 to 412 in a8e2947
behavioral-model/include/bm/bm_sim/queueing.h Lines 616 to 635 in a8e2947
Notice how the first variant is called by simple_switch |
Thank you @antoninbas. I changed the simple_switch.cpp code to use the second size method variant and recompiled bmv2. It seems that now I can get the depth of only specific priority queues using the enq_qdepth metadata. But it is strange to me that the standard behavior implemented is to return the depth of all priority queues of the egress port together, instead of the specific queue that the packet is crossing. |
Hi! I am working with 3 priority queues on simple_switch, and I am trying to collect InBand Network Telemetry data, about the occupation of these queues. But when I am generating traffic in only one queue (for example queue 0), the enq_qdepth and deq_qdepth values of all three queues are getting high, while the other queues (1 and 2) are supposed to be empty.
I am also checking deq_timedelta, and in this case only in queue 0 I am getting high values (it is the correct behavior I think).
Other thing that I noticed is that when I am generating traffic in 2 queues, the queue depth values are getting higher than 64 (I set the size with set_queue_depth command)
So I have a question: the enq_qdepth and deq_qdepth metadata are about the number of packets adding all the queues in total? Or something is wrong here?
The text was updated successfully, but these errors were encountered: