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

radvd out of range MTU error #3161

Closed
rajendra-dendukuri opened this issue Jul 16, 2019 · 4 comments · Fixed by #5242
Closed

radvd out of range MTU error #3161

rajendra-dendukuri opened this issue Jul 16, 2019 · 4 comments · Fixed by #5242
Assignees

Comments

@rajendra-dendukuri
Copy link
Contributor

rajendra-dendukuri commented Jul 16, 2019

Description

radvd out of range MTU error

As part of "PR#2795 Build radvd from source; Patch so as not to treat out-of-range MTU as an error", the error message was reduced to warning
and failure was bypassed.

This issue is created to track a final fix. Following fixes have been commited to 4.18 kernel to support setting of bridge MTU to a value > 9100. When SONiC kernrel is upgraded to a kernel > 4.18, the patch done in radvd needs to be removed and Bridge MTU needs to be set to 9100.

torvalds/linux@3341d91#diff-221ed728b95e6fa6475069eed594287c
torvalds/linux@804b854#diff-221ed728b95e6fa6475069eed594287c
torvalds/linux@f40aa23#diff-221ed728b95e6fa6475069eed594287c
torvalds/linux@419d14a#diff-221ed728b95e6fa6475069eed594287c

Steps to reproduce the issue:

  1. Make sure that DEVICE_METADATA|localhost|type="ToRRouter"

  2. Create a VLAN and add an active link to it
    config vlan add 100
    config vlan member add 100 Ethernet0

  3. Assing IPv6 address to the Vlan interface
    config interface ip add Vlan100 2002::1/64

  4. Restart radv container
    systemctl restart radv

  5. Check syslog

Describe the results you received:

Following error message is seen
Jul 16 00:41:19.823049 sonic WARNING radv#radvd[60]: AdvLinkMTU for Vlan1 (9100) must be zero or between 1280 and 1500

Describe the results you expected:
No error/warnings need to be observed

@jleveque
Copy link
Contributor

Thanks for creating this issue to track this.

@jleveque
Copy link
Contributor

Master branch is now running 4.19 kernel, so the patches in the description should be present.

@prsunny: Do you know if there is a plan to set the bridge MTU to 9100?

@tahmed-dev
Copy link
Contributor

@jleveque: @prsunny and I verified that we can set the bridge MTU to 9100. PR:1393 should close this issue.

@jleveque
Copy link
Contributor

jleveque commented Aug 20, 2020

@tahmed-dev: Thanks for the info! We should then also remove the patch from radvd and ensure we no longer see the errors before closing this issue.

mssonicbld added a commit that referenced this issue May 26, 2024
…lly (#19064)

#### Why I did it
src/sonic-swss
```
* 3ee73618 - (HEAD -> master, origin/master, origin/HEAD) [orchagent] Add support for Path Tracing Midpoint (#2903) (34 hours ago) [Carmine Scarpitta]
* 5ffb0b50 - [Bridge] Disable MAC learning for Link Local Mulitcast Frames on the kernel bridge (#3162) (2 days ago) [Gord Chen]
* de9baac1 - [Marvell] PFC Detect lua script (#3160) (2 days ago) [Rajkumar-Marvell]
* 31d9d6c1 - Add fabric counter rate mbps command handling. (#3129) (2 days ago) [jfeng-arista]
* 425a8526 - Parse subport value added to config_db (#3161) (2 days ago) [Bobby McGonigle]
```
#### How I did it
#### How to verify it
#### Description for the changelog
mssonicbld added a commit that referenced this issue May 28, 2024
…lly (#19095)

#### Why I did it
src/sonic-swss
```
* 5075e660 - (HEAD -> 202311, origin/202311) Parse subport value added to config_db (#3161) (8 seconds ago) [Bobby McGonigle]
* 92fd1b88 - revert requests from 2.32.0 to 2.31.0 (#3154) (#3169) (8 hours ago) [Guohan Lu]
```
#### 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
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants