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

Downport the netlink patch to libteam1.26. Increase netlink buffers #2822

Merged
merged 1 commit into from
Apr 25, 2019

Conversation

pavel-shirshov
Copy link
Contributor

@pavel-shirshov pavel-shirshov commented Apr 25, 2019

- What I did

  1. Downport the netlink patch:
    jpirko/libteam@046fb6b#diff-2534a34d63e1982340c4691ab57c193f
  2. Increase default buffer size from 98304 to 983040

I did that because I found: PortChannel115: Failed to send options change via netlink (err -105) In kernel.log. I checked kernel sources and found that error 105 means: ENOBUFS.
From netlink man page

   However, reliable transmissions from kernel to user are impossible in
   any case.  The kernel can't send a  netlink  message  if  the  socket
   buffer  is  full:  the message will be dropped and the kernel and the
   user-space process will no longer have the same view of kernel state.
   It  is  up  to  the  application to detect when this happens (via the
   ENOBUFS error returned by recvmsg(2)) and resynchronize.

So I applied a patch witch increases size of the netlink buffers.

- How I did it

- How to verify it
Build an image and run on DUT with 32 LACP interfaces

- Description for the changelog

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

@pavel-shirshov pavel-shirshov merged commit 452e75c into master Apr 25, 2019
@pavel-shirshov pavel-shirshov deleted the pavelsh/libteam_buf branch April 25, 2019 23:16
MichelMoriniaux pushed a commit to criteo-forks/sonic-buildimage that referenced this pull request May 28, 2019
mssonicbld added a commit that referenced this pull request Jun 16, 2023
…lly (#15468)

#### Why I did it
src/sonic-swss
```
* 87e0b08 - (HEAD -> master, origin/master, origin/HEAD) [portsorch]: Enhancing SWSS OA logs to capture host_tx_ready change events (#2822) (11 hours ago) [mihirpat1]
* c7e52a0 - [subinterface]: Fix admin state handling. (#2806) (34 hours ago) [Nazarii Hnydyn]
* ebfda13 - [aclorch] Fix TODO: use SAI object API to query capabilities (#2743) (2 days ago) [Stepan Blyshchak]
```
#### How I did it
#### How to verify it
#### Description for the changelog
yxieca pushed a commit that referenced this pull request Jun 16, 2023
…lly (#15508)

src/sonic-swss

* 444e3be - (HEAD -> 202205, origin/202205) [subinterface]: Fix admin state handling. (#2806) (10 hours ago) [Nazarii Hnydyn]
* 9690c5a - [portsorch]: Enhancing SWSS OA logs to capture host_tx_ready change events (#2822) (10 hours ago) [mihirpat1]
sonic-otn pushed a commit to sonic-otn/sonic-buildimage that referenced this pull request Sep 20, 2023
…lly (sonic-net#15468)

#### Why I did it
src/sonic-swss
```
* 87e0b08 - (HEAD -> master, origin/master, origin/HEAD) [portsorch]: Enhancing SWSS OA logs to capture host_tx_ready change events (sonic-net#2822) (11 hours ago) [mihirpat1]
* c7e52a0 - [subinterface]: Fix admin state handling. (sonic-net#2806) (34 hours ago) [Nazarii Hnydyn]
* ebfda13 - [aclorch] Fix TODO: use SAI object API to query capabilities (sonic-net#2743) (2 days ago) [Stepan Blyshchak]
```
#### How I did it
#### How to verify it
#### Description for the changelog
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.

3 participants