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

[quagga]: Increase netlink receive buffer for zebra #4228

Merged
merged 1 commit into from
Mar 6, 2020

Conversation

pavel-shirshov
Copy link
Contributor

@pavel-shirshov pavel-shirshov commented Mar 5, 2020

Otherwise we see following messages from zebra sometimes
zebra[60]: netlink-listen recvmsg overrun: No buffer space available

- What I did
Increased netlink socket receive buffer size for zebra. Otherwise we receive following messages sometimes:
zebra[60]: netlink-listen recvmsg overrun: No buffer space available

- How I did it
Used -s command line parameter of zebra

- How to verify it
Build an image and run it. Currently I didn't find how we can check receive buffer size from the command line. I'll update the pr if I find the way.
- Description for the changelog

- A picture of a cute animal (not mandatory but encouraged)

Otherwise we see following messages from zebra sometimes
zebra[60]: netlink-listen recvmsg overrun: No buffer space available
@lguohan
Copy link
Collaborator

lguohan commented Mar 5, 2020

201911 branch is using frr, you need to generate a new pr.

@jleveque
Copy link
Contributor

jleveque commented Mar 5, 2020

@pavel-shirshov: Can you briefly explain how you settled on this size?

@yxieca
Copy link
Contributor

yxieca commented Mar 5, 2020

@jleveque expand Prince's discussion :-)

Basically Pavel set it to max value sysctrl allows.

@pavel-shirshov
Copy link
Contributor Author

@jleveque By default all sockets are created with default receive buffer equal to 212992
net.core.rmem_default = 212992.
This value definitely is not enough for quagga, because we're keep seeing issues with it here and there. I don't know what is the optimal value for the buffer.
I made it 2Mb here #767
So in this patch I increased quagga value to the maximum.

Probably we should increase default receive buffer for higher values.

@lguohan lguohan merged commit b3ce209 into sonic-net:201811 Mar 6, 2020
@pavel-shirshov pavel-shirshov deleted the pavelsh/nl_rcv_2018 branch March 6, 2020 22:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants