Skip to content
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

num_slots on RX rings are zero #950

Open
cygnus2048 opened this issue Mar 5, 2024 · 2 comments
Open

num_slots on RX rings are zero #950

cygnus2048 opened this issue Mar 5, 2024 · 2 comments
Assignees

Comments

@cygnus2048
Copy link

I was using the ixgbevf driver @ commit 67901e0 and all was functioning well.
I then went to commit 1bf288e to also get ICE driver support, however, I now do not receive any packets at all and adding some debug I show that the num_slots on the RX rings are set to zero. Note I am still using the ixgbevf driver.

I did verify TX is functioning and the far-end device is indeed receiving the TX packets.

NIOCGINFO shows the following:

	if (ioctl(dev_fd, NIOCGINFO, &nm_req))
	log_info("    Version: %d, hw tx/rx rings: %d/%d, tx/rx slots: %d/%d", nm_req.nr_version, nm_req.nr_tx_rings, nm_req.nr_rx_rings,
																		   nm_req.nr_tx_slots, nm_req.nr_rx_slots);

INFO (inst: 0): Version: 14, hw tx/rx rings: 2/2, tx/rx slots: 1024/1600

		auto rxring = NETMAP_RXRING(pNetmapdev->nifp, pNetmapdev->first_rx_ring);
		log_info("Thread %d, Queue %d: rxring %d (max %d) buffer size %d slots %d", uInstance, uQueueIdx, (int)pNetmapdev->first_rx_ring,
				 (int)pNetmapdev->last_rx_ring, (int)rxring->nr_buf_size, (int)rxring->num_slots);

INFO (inst: 0): Thread 0, Queue 0: rxring 0 (max 0) buffer size 0 slots 0
INFO (inst: 0): Thread 0, Queue 1: rxring 1 (max 1) buffer size 0 slots 0

Thanks for any help.

@giuseppelettieri
Copy link
Collaborator

I just tried with the latest master and it appears to work. May you switch to the most recent commit, please? Note, by the way, that the initial support for ICE was completely broken, so you may want to use master anyway.

@giuseppelettieri giuseppelettieri self-assigned this Mar 13, 2024
@cygnus2048
Copy link
Author

Ok, I will give that a try. However, when I tried the latest about a year ago, I saw a regression in performance on ixgbevf compared to the commit 67901e0 so that is why I was trying to stay as close to that as possible.
That said, my primary goal is to build the ICE driver, so I can look into any other issues later. For now, I opened another ticket for building the ice driver as I am getting many errors when trying to build that @master. Thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants