Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
start-vswitch.sh: update how devices are described.
--devices now include an optional port number, in addition to the PCI location ID. For example, "0000:86:00.1/0" The new handling of device nameing becomes quite extensive, and we still need to fix some of these switch-types to work with the new naming format.
- Loading branch information
c096ce1There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@atheurer - Can you provide some background on this?
c096ce1There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@k-rister We had to do this because there are some NICs which have more than 1 "netdev" per physical port on the card. Our code assumed there was always a 1:1 relationship between a physical-function and a netdev. So, we have to change everything that works with a physical-function and its netdev (like "eth0").
I see I may have inadvertently committed some other changes which I will back out shortly
c096ce1There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I guess I was just wondering if this was something new or something you just have had lingering for awhile.
c096ce1There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have been meaning to commit it for a few weeks but just now got around to it. We may have others who need to use this functionality soon, so we wanted it in the repo. BTW, I am pretty sure some changes will be necessary for VPP.