Skip to content

Commit

Permalink
Only check IsPort for iobundle types which are Net
Browse files Browse the repository at this point in the history
Skip over HDMI, COM.  IsPort only checks Ifname which can
incorrectly flag non-net devices as a port.

Signed-off-by: Andrew Durbin <andrewd@zededa.com>
  • Loading branch information
andrewd-zededa authored and eriknordmark committed Dec 13, 2024
1 parent 727eef8 commit 7311ddf
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion pkg/pillar/cmd/domainmgr/domainmgr.go
Original file line number Diff line number Diff line change
Expand Up @@ -3154,7 +3154,7 @@ func updatePortAndPciBackIoBundle(ctx *domainContext, ib *types.IoBundle) (chang
// EVE controller doesn't know it
list = aa.ExpandControllers(log, list, hyper.PCISameController)
for _, ib := range list {
if types.IsPort(ctx.deviceNetworkStatus, ib.Ifname) {
if types.IsPort(ctx.deviceNetworkStatus, ib.Ifname) && ib.Type.IsNet() {
isPort = true
keepInHost = true
}
Expand Down

0 comments on commit 7311ddf

Please sign in to comment.