Page MenuHomeVyOS Platform

Add Google Virtual NIC (gVNIC) support
Closed, ResolvedPublicFEATURE REQUEST

Description

This NIC type is now one of the native types for Google Cloud and can be enabled with a simple kernel config:

CONFIG_NET_VENDOR_GOOGLE=y
CONFIG_GVE=m

Details

Difficulty level
Easy (less than an hour)
Version
-
Why the issue appeared?
Will be filled on close
Is it a breaking change?
Perfectly compatible
Issue type
Improvement (missing useful functionality)

Event Timeline

zsdc updated the task description. (Show Details)
c-po changed the task status from Open to In progress.Aug 25 2022, 5:14 PM
c-po triaged this task as High priority.
c-po moved this task from Need Triage to Finished on the VyOS 1.4 Sagitta board.
c-po edited projects, added VyOS 1.3 Equuleus (1.3.2); removed VyOS 1.3 Equuleus.
c-po changed Difficulty level from Unknown (require assessment) to Easy (less than an hour).
c-po moved this task from Need Triage to Finished on the VyOS 1.3 Equuleus (1.3.2) board.
dmbaturin subscribed.

As @zsdc says, it's not enough to just have the driver, the problem is that it doesn't work with MTUs over 1460, and VyOS now tries to force it to 1500 if it's not specified. We need to adjust that logic so that MTU setting error doesn't cause a commit error.

Whether we should add a special case or try to "adaptively" set the MTU is up to debate.

I am suggesting marking this task as "Resolved" because the driver works by himself and NIC can be used with a proper configuration.

The MTU problem is far more complex and should be reviewed independently. I already tried to raise it once in https://phabricator.vyos.net/T4572. We should create a specific task for it again.