khenaidoo | b920354 | 2018-09-17 22:56:37 -0400 | [diff] [blame] | 1 | /* |
| 2 | * Copyright 2018-present Open Networking Foundation |
| 3 | |
| 4 | * Licensed under the Apache License, Version 2.0 (the "License"); |
| 5 | * you may not use this file except in compliance with the License. |
| 6 | * You may obtain a copy of the License at |
| 7 | |
| 8 | * http://www.apache.org/licenses/LICENSE-2.0 |
| 9 | |
| 10 | * Unless required by applicable law or agreed to in writing, software |
| 11 | * distributed under the License is distributed on an "AS IS" BASIS, |
| 12 | * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. |
| 13 | * See the License for the specific language governing permissions and |
| 14 | * limitations under the License. |
| 15 | */ |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 16 | |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 17 | package device |
khenaidoo | b920354 | 2018-09-17 22:56:37 -0400 | [diff] [blame] | 18 | |
| 19 | import ( |
| 20 | "context" |
Matteo Scandolo | 360605d | 2019-11-05 18:29:17 -0800 | [diff] [blame] | 21 | "encoding/hex" |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 22 | "errors" |
| 23 | "fmt" |
David Bainbridge | d1afd66 | 2020-03-26 18:27:41 -0700 | [diff] [blame] | 24 | "sync" |
| 25 | "time" |
| 26 | |
khenaidoo | b920354 | 2018-09-17 22:56:37 -0400 | [diff] [blame] | 27 | "github.com/gogo/protobuf/proto" |
sbarbari | 17d7e22 | 2019-11-05 10:02:29 -0500 | [diff] [blame] | 28 | "github.com/opencord/voltha-go/db/model" |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 29 | fd "github.com/opencord/voltha-go/rw_core/flowdecomposition" |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 30 | "github.com/opencord/voltha-go/rw_core/route" |
Scott Baker | b671a86 | 2019-10-24 10:53:40 -0700 | [diff] [blame] | 31 | coreutils "github.com/opencord/voltha-go/rw_core/utils" |
serkant.uluderya | 2ae470f | 2020-01-21 11:13:09 -0800 | [diff] [blame] | 32 | fu "github.com/opencord/voltha-lib-go/v3/pkg/flows" |
| 33 | "github.com/opencord/voltha-lib-go/v3/pkg/log" |
| 34 | ic "github.com/opencord/voltha-protos/v3/go/inter_container" |
| 35 | ofp "github.com/opencord/voltha-protos/v3/go/openflow_13" |
| 36 | "github.com/opencord/voltha-protos/v3/go/voltha" |
khenaidoo | b920354 | 2018-09-17 22:56:37 -0400 | [diff] [blame] | 37 | "google.golang.org/grpc/codes" |
| 38 | "google.golang.org/grpc/status" |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 39 | ) |
| 40 | |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 41 | // LogicalAgent represent attributes of logical device agent |
| 42 | type LogicalAgent struct { |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 43 | logicalDeviceID string |
David Bainbridge | d1afd66 | 2020-03-26 18:27:41 -0700 | [diff] [blame] | 44 | serialNumber string |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 45 | rootDeviceID string |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 46 | deviceMgr *Manager |
| 47 | ldeviceMgr *LogicalManager |
khenaidoo | 3306c99 | 2019-05-24 16:57:35 -0400 | [diff] [blame] | 48 | clusterDataProxy *model.Proxy |
Kent Hagerman | 4f355f5 | 2020-03-30 16:01:33 -0400 | [diff] [blame] | 49 | stopped bool |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 50 | deviceRoutes *route.DeviceRoutes |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 51 | lockDeviceRoutes sync.RWMutex |
khenaidoo | 3306c99 | 2019-05-24 16:57:35 -0400 | [diff] [blame] | 52 | logicalPortsNo map[uint32]bool //value is true for NNI port |
| 53 | lockLogicalPortsNo sync.RWMutex |
| 54 | flowDecomposer *fd.FlowDecomposer |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 55 | defaultTimeout time.Duration |
khenaidoo | 6e55d9e | 2019-12-12 18:26:26 -0500 | [diff] [blame] | 56 | logicalDevice *voltha.LogicalDevice |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 57 | requestQueue *coreutils.RequestQueue |
| 58 | startOnce sync.Once |
| 59 | stopOnce sync.Once |
khenaidoo | b920354 | 2018-09-17 22:56:37 -0400 | [diff] [blame] | 60 | } |
| 61 | |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 62 | func newLogicalDeviceAgent(id string, sn string, deviceID string, ldeviceMgr *LogicalManager, |
| 63 | deviceMgr *Manager, cdProxy *model.Proxy, timeout time.Duration) *LogicalAgent { |
| 64 | var agent LogicalAgent |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 65 | agent.logicalDeviceID = id |
David Bainbridge | d1afd66 | 2020-03-26 18:27:41 -0700 | [diff] [blame] | 66 | agent.serialNumber = sn |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 67 | agent.rootDeviceID = deviceID |
khenaidoo | b920354 | 2018-09-17 22:56:37 -0400 | [diff] [blame] | 68 | agent.deviceMgr = deviceMgr |
khenaidoo | 9a46896 | 2018-09-19 15:33:13 -0400 | [diff] [blame] | 69 | agent.clusterDataProxy = cdProxy |
khenaidoo | b920354 | 2018-09-17 22:56:37 -0400 | [diff] [blame] | 70 | agent.ldeviceMgr = ldeviceMgr |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 71 | agent.flowDecomposer = fd.NewFlowDecomposer(agent.deviceMgr) |
khenaidoo | 2c6a099 | 2019-04-29 13:46:56 -0400 | [diff] [blame] | 72 | agent.logicalPortsNo = make(map[uint32]bool) |
khenaidoo | 2c6a099 | 2019-04-29 13:46:56 -0400 | [diff] [blame] | 73 | agent.defaultTimeout = timeout |
Kent Hagerman | 730cbdf | 2020-03-31 12:22:08 -0400 | [diff] [blame] | 74 | agent.requestQueue = coreutils.NewRequestQueue() |
khenaidoo | b920354 | 2018-09-17 22:56:37 -0400 | [diff] [blame] | 75 | return &agent |
| 76 | } |
| 77 | |
khenaidoo | 4d4802d | 2018-10-04 21:59:49 -0400 | [diff] [blame] | 78 | // start creates the logical device and add it to the data model |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 79 | func (agent *LogicalAgent) start(ctx context.Context, loadFromDB bool) error { |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 80 | needToStart := false |
| 81 | if agent.startOnce.Do(func() { needToStart = true }); !needToStart { |
| 82 | return nil |
| 83 | } |
| 84 | |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 85 | logger.Infow("starting-logical_device-agent", log.Fields{"logical-device-id": agent.logicalDeviceID, "load-from-db": loadFromDB}) |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 86 | |
| 87 | var startSucceeded bool |
| 88 | defer func() { |
| 89 | if !startSucceeded { |
| 90 | if err := agent.stop(ctx); err != nil { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 91 | logger.Errorw("failed-to-cleanup-after-unsuccessful-start", log.Fields{"logical-device-id": agent.logicalDeviceID, "error": err}) |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 92 | } |
| 93 | } |
| 94 | }() |
| 95 | |
khenaidoo | 297cd25 | 2019-02-07 22:10:23 -0500 | [diff] [blame] | 96 | var ld *voltha.LogicalDevice |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 97 | if !loadFromDB { |
khenaidoo | 7e3d8f1 | 2019-08-02 16:06:30 -0400 | [diff] [blame] | 98 | //Build the logical device based on information retrieved from the device adapter |
| 99 | var switchCap *ic.SwitchCapability |
khenaidoo | 297cd25 | 2019-02-07 22:10:23 -0500 | [diff] [blame] | 100 | var err error |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 101 | if switchCap, err = agent.deviceMgr.getSwitchCapability(ctx, agent.rootDeviceID); err != nil { |
khenaidoo | 7e3d8f1 | 2019-08-02 16:06:30 -0400 | [diff] [blame] | 102 | return err |
| 103 | } |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 104 | ld = &voltha.LogicalDevice{Id: agent.logicalDeviceID, RootDeviceId: agent.rootDeviceID} |
khenaidoo | 297cd25 | 2019-02-07 22:10:23 -0500 | [diff] [blame] | 105 | |
| 106 | // Create the datapath ID (uint64) using the logical device ID (based on the MAC Address) |
| 107 | var datapathID uint64 |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 108 | if datapathID, err = coreutils.CreateDataPathID(agent.serialNumber); err != nil { |
khenaidoo | 297cd25 | 2019-02-07 22:10:23 -0500 | [diff] [blame] | 109 | return err |
| 110 | } |
| 111 | ld.DatapathId = datapathID |
khenaidoo | 7e3d8f1 | 2019-08-02 16:06:30 -0400 | [diff] [blame] | 112 | ld.Desc = (proto.Clone(switchCap.Desc)).(*ofp.OfpDesc) |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 113 | logger.Debugw("Switch-capability", log.Fields{"Desc": ld.Desc, "fromAd": switchCap.Desc}) |
khenaidoo | 7e3d8f1 | 2019-08-02 16:06:30 -0400 | [diff] [blame] | 114 | ld.SwitchFeatures = (proto.Clone(switchCap.SwitchFeatures)).(*ofp.OfpSwitchFeatures) |
khenaidoo | 297cd25 | 2019-02-07 22:10:23 -0500 | [diff] [blame] | 115 | ld.Flows = &ofp.Flows{Items: nil} |
| 116 | ld.FlowGroups = &ofp.FlowGroups{Items: nil} |
khenaidoo | 4908535 | 2020-01-13 19:15:43 -0500 | [diff] [blame] | 117 | ld.Ports = []*voltha.LogicalPort{} |
khenaidoo | 297cd25 | 2019-02-07 22:10:23 -0500 | [diff] [blame] | 118 | |
khenaidoo | 297cd25 | 2019-02-07 22:10:23 -0500 | [diff] [blame] | 119 | // Save the logical device |
Kent Hagerman | 4f355f5 | 2020-03-30 16:01:33 -0400 | [diff] [blame] | 120 | if err := agent.clusterDataProxy.AddWithID(ctx, "logical_devices", ld.Id, ld); err != nil { |
| 121 | logger.Errorw("failed-to-add-logical-device", log.Fields{"logical-device-id": agent.logicalDeviceID}) |
Thomas Lee S | e5a4401 | 2019-11-07 20:32:24 +0530 | [diff] [blame] | 122 | return err |
| 123 | } |
Kent Hagerman | 4f355f5 | 2020-03-30 16:01:33 -0400 | [diff] [blame] | 124 | logger.Debugw("logicaldevice-created", log.Fields{"logical-device-id": agent.logicalDeviceID, "root-id": ld.RootDeviceId}) |
khenaidoo | 6e55d9e | 2019-12-12 18:26:26 -0500 | [diff] [blame] | 125 | |
| 126 | agent.logicalDevice = proto.Clone(ld).(*voltha.LogicalDevice) |
khenaidoo | fc1314d | 2019-03-14 09:34:21 -0400 | [diff] [blame] | 127 | |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 128 | // Setup the logicalports - internal processing, no need to propagate the client context |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 129 | go func() { |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 130 | err := agent.setupLogicalPorts(context.Background()) |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 131 | if err != nil { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 132 | logger.Errorw("unable-to-setup-logical-ports", log.Fields{"error": err}) |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 133 | } |
| 134 | }() |
khenaidoo | 297cd25 | 2019-02-07 22:10:23 -0500 | [diff] [blame] | 135 | } else { |
| 136 | // load from dB - the logical may not exist at this time. On error, just return and the calling function |
| 137 | // will destroy this agent. |
Kent Hagerman | 4f355f5 | 2020-03-30 16:01:33 -0400 | [diff] [blame] | 138 | ld := &voltha.LogicalDevice{} |
| 139 | have, err := agent.clusterDataProxy.Get(ctx, "logical_devices/"+agent.logicalDeviceID, ld) |
Thomas Lee S | e5a4401 | 2019-11-07 20:32:24 +0530 | [diff] [blame] | 140 | if err != nil { |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 141 | return err |
Kent Hagerman | 4f355f5 | 2020-03-30 16:01:33 -0400 | [diff] [blame] | 142 | } else if !have { |
khenaidoo | 6e55d9e | 2019-12-12 18:26:26 -0500 | [diff] [blame] | 143 | return status.Errorf(codes.NotFound, "logical_device-%s", agent.logicalDeviceID) |
khenaidoo | 297cd25 | 2019-02-07 22:10:23 -0500 | [diff] [blame] | 144 | } |
Kent Hagerman | 4f355f5 | 2020-03-30 16:01:33 -0400 | [diff] [blame] | 145 | |
khenaidoo | 8c3303d | 2019-02-13 14:59:39 -0500 | [diff] [blame] | 146 | // Update the root device Id |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 147 | agent.rootDeviceID = ld.RootDeviceId |
khenaidoo | 3d3b8c2 | 2019-05-22 18:10:39 -0400 | [diff] [blame] | 148 | |
khenaidoo | 6e55d9e | 2019-12-12 18:26:26 -0500 | [diff] [blame] | 149 | // Update the last data |
| 150 | agent.logicalDevice = proto.Clone(ld).(*voltha.LogicalDevice) |
| 151 | |
khenaidoo | 3d3b8c2 | 2019-05-22 18:10:39 -0400 | [diff] [blame] | 152 | // Setup the local list of logical ports |
| 153 | agent.addLogicalPortsToMap(ld.Ports) |
khenaidoo | b920354 | 2018-09-17 22:56:37 -0400 | [diff] [blame] | 154 | } |
khenaidoo | fc1314d | 2019-03-14 09:34:21 -0400 | [diff] [blame] | 155 | |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 156 | // Setup the device routes. Building routes may fail if the pre-conditions are not satisfied (e.g. no PON ports present) |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 157 | if loadFromDB { |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 158 | go func() { |
| 159 | if err := agent.buildRoutes(context.Background()); err != nil { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 160 | logger.Warn("routes-not-ready", log.Fields{"logical-device-id": agent.logicalDeviceID, "error": err}) |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 161 | } |
| 162 | }() |
khenaidoo | 4c9e559 | 2019-09-09 16:20:41 -0400 | [diff] [blame] | 163 | } |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 164 | startSucceeded = true |
| 165 | |
khenaidoo | b920354 | 2018-09-17 22:56:37 -0400 | [diff] [blame] | 166 | return nil |
| 167 | } |
| 168 | |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 169 | // stop stops the logical device agent. This removes the logical device from the data model. |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 170 | func (agent *LogicalAgent) stop(ctx context.Context) error { |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 171 | var returnErr error |
| 172 | agent.stopOnce.Do(func() { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 173 | logger.Info("stopping-logical_device-agent") |
khenaidoo | 8c3303d | 2019-02-13 14:59:39 -0500 | [diff] [blame] | 174 | |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 175 | if err := agent.requestQueue.WaitForGreenLight(ctx); err != nil { |
| 176 | // This should never happen - an error is returned only if the agent is stopped and an agent is only stopped once. |
| 177 | returnErr = err |
| 178 | return |
| 179 | } |
| 180 | defer agent.requestQueue.RequestComplete() |
| 181 | |
| 182 | //Remove the logical device from the model |
Kent Hagerman | 4f355f5 | 2020-03-30 16:01:33 -0400 | [diff] [blame] | 183 | if err := agent.clusterDataProxy.Remove(ctx, "logical_devices/"+agent.logicalDeviceID); err != nil { |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 184 | returnErr = err |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 185 | } else { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 186 | logger.Debugw("logicaldevice-removed", log.Fields{"logicaldeviceId": agent.logicalDeviceID}) |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 187 | } |
| 188 | |
Kent Hagerman | 4f355f5 | 2020-03-30 16:01:33 -0400 | [diff] [blame] | 189 | agent.stopped = true |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 190 | |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 191 | logger.Info("logical_device-agent-stopped") |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 192 | }) |
| 193 | return returnErr |
khenaidoo | 4d4802d | 2018-10-04 21:59:49 -0400 | [diff] [blame] | 194 | } |
| 195 | |
khenaidoo | 6e55d9e | 2019-12-12 18:26:26 -0500 | [diff] [blame] | 196 | // GetLogicalDevice returns the latest logical device data |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 197 | func (agent *LogicalAgent) GetLogicalDevice(ctx context.Context) (*voltha.LogicalDevice, error) { |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 198 | if err := agent.requestQueue.WaitForGreenLight(ctx); err != nil { |
| 199 | return nil, err |
| 200 | } |
| 201 | defer agent.requestQueue.RequestComplete() |
| 202 | return proto.Clone(agent.logicalDevice).(*voltha.LogicalDevice), nil |
khenaidoo | 92e62c5 | 2018-10-03 14:02:54 -0400 | [diff] [blame] | 203 | } |
| 204 | |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 205 | // ListLogicalDeviceFlows returns logical device flows |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 206 | func (agent *LogicalAgent) ListLogicalDeviceFlows(ctx context.Context) (*ofp.Flows, error) { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 207 | logger.Debug("ListLogicalDeviceFlows") |
khenaidoo | 6e55d9e | 2019-12-12 18:26:26 -0500 | [diff] [blame] | 208 | |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 209 | logicalDevice, err := agent.GetLogicalDevice(ctx) |
| 210 | if err != nil { |
| 211 | return nil, err |
khenaidoo | dd23717 | 2019-05-27 16:37:17 -0400 | [diff] [blame] | 212 | } |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 213 | if logicalDevice.Flows == nil { |
| 214 | return &ofp.Flows{}, nil |
| 215 | } |
| 216 | return (proto.Clone(logicalDevice.Flows)).(*ofp.Flows), nil |
khenaidoo | dd23717 | 2019-05-27 16:37:17 -0400 | [diff] [blame] | 217 | } |
| 218 | |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 219 | // ListLogicalDeviceMeters returns logical device meters |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 220 | func (agent *LogicalAgent) ListLogicalDeviceMeters(ctx context.Context) (*ofp.Meters, error) { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 221 | logger.Debug("ListLogicalDeviceMeters") |
khenaidoo | 6e55d9e | 2019-12-12 18:26:26 -0500 | [diff] [blame] | 222 | |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 223 | logicalDevice, err := agent.GetLogicalDevice(ctx) |
| 224 | if err != nil { |
| 225 | return nil, err |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 226 | } |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 227 | if logicalDevice.Meters == nil { |
| 228 | return &ofp.Meters{}, nil |
| 229 | } |
| 230 | return (proto.Clone(logicalDevice.Meters)).(*ofp.Meters), nil |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 231 | } |
| 232 | |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 233 | // ListLogicalDeviceFlowGroups returns logical device flow groups |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 234 | func (agent *LogicalAgent) ListLogicalDeviceFlowGroups(ctx context.Context) (*ofp.FlowGroups, error) { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 235 | logger.Debug("ListLogicalDeviceFlowGroups") |
khenaidoo | 6e55d9e | 2019-12-12 18:26:26 -0500 | [diff] [blame] | 236 | |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 237 | logicalDevice, err := agent.GetLogicalDevice(ctx) |
| 238 | if err != nil { |
| 239 | return nil, err |
khenaidoo | dd23717 | 2019-05-27 16:37:17 -0400 | [diff] [blame] | 240 | } |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 241 | if logicalDevice.FlowGroups == nil { |
| 242 | return &ofp.FlowGroups{}, nil |
| 243 | } |
| 244 | return (proto.Clone(logicalDevice.FlowGroups)).(*ofp.FlowGroups), nil |
khenaidoo | dd23717 | 2019-05-27 16:37:17 -0400 | [diff] [blame] | 245 | } |
| 246 | |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 247 | // ListLogicalDevicePorts returns logical device ports |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 248 | func (agent *LogicalAgent) ListLogicalDevicePorts(ctx context.Context) (*voltha.LogicalPorts, error) { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 249 | logger.Debug("ListLogicalDevicePorts") |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 250 | logicalDevice, err := agent.GetLogicalDevice(ctx) |
| 251 | if err != nil { |
| 252 | return nil, err |
| 253 | } |
| 254 | if logicalDevice == nil { |
| 255 | return &voltha.LogicalPorts{}, nil |
| 256 | } |
khenaidoo | 6e55d9e | 2019-12-12 18:26:26 -0500 | [diff] [blame] | 257 | lPorts := make([]*voltha.LogicalPort, 0) |
| 258 | lPorts = append(lPorts, logicalDevice.Ports...) |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 259 | return &voltha.LogicalPorts{Items: lPorts}, nil |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 260 | } |
| 261 | |
khenaidoo | 4c9e559 | 2019-09-09 16:20:41 -0400 | [diff] [blame] | 262 | //updateLogicalDeviceFlowsWithoutLock updates the logical device with the latest flows in the model. |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 263 | func (agent *LogicalAgent) updateLogicalDeviceFlowsWithoutLock(ctx context.Context, flows *ofp.Flows) error { |
khenaidoo | 6e55d9e | 2019-12-12 18:26:26 -0500 | [diff] [blame] | 264 | ld := agent.getLogicalDeviceWithoutLock() |
khenaidoo | 4c9e559 | 2019-09-09 16:20:41 -0400 | [diff] [blame] | 265 | |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 266 | logger.Debugw("logical-device-before", log.Fields{"lports": len(ld.Ports)}) |
khenaidoo | 6e55d9e | 2019-12-12 18:26:26 -0500 | [diff] [blame] | 267 | ld.Flows = flows |
| 268 | |
npujar | 467fe75 | 2020-01-16 20:17:45 +0530 | [diff] [blame] | 269 | if err := agent.updateLogicalDeviceWithoutLock(ctx, ld); err != nil { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 270 | logger.Errorw("error-updating-logical-device-with-flows", log.Fields{"error": err}) |
khenaidoo | 4c9e559 | 2019-09-09 16:20:41 -0400 | [diff] [blame] | 271 | return err |
khenaidoo | 43c8212 | 2018-11-22 18:38:28 -0500 | [diff] [blame] | 272 | } |
khenaidoo | 43c8212 | 2018-11-22 18:38:28 -0500 | [diff] [blame] | 273 | return nil |
| 274 | } |
| 275 | |
khenaidoo | 4c9e559 | 2019-09-09 16:20:41 -0400 | [diff] [blame] | 276 | //updateLogicalDeviceMetersWithoutLock updates the logical device with the meters info |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 277 | func (agent *LogicalAgent) updateLogicalDeviceMetersWithoutLock(ctx context.Context, meters *ofp.Meters) error { |
khenaidoo | 6e55d9e | 2019-12-12 18:26:26 -0500 | [diff] [blame] | 278 | ld := agent.getLogicalDeviceWithoutLock() |
khenaidoo | 4c9e559 | 2019-09-09 16:20:41 -0400 | [diff] [blame] | 279 | |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 280 | logger.Debugw("logical-device-before", log.Fields{"lports": len(ld.Ports)}) |
khenaidoo | 6e55d9e | 2019-12-12 18:26:26 -0500 | [diff] [blame] | 281 | ld.Meters = meters |
| 282 | |
npujar | 467fe75 | 2020-01-16 20:17:45 +0530 | [diff] [blame] | 283 | if err := agent.updateLogicalDeviceWithoutLock(ctx, ld); err != nil { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 284 | logger.Errorw("error-updating-logical-device-with-meters", log.Fields{"error": err}) |
khenaidoo | 4c9e559 | 2019-09-09 16:20:41 -0400 | [diff] [blame] | 285 | return err |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 286 | } |
| 287 | return nil |
| 288 | } |
| 289 | |
khenaidoo | 4c9e559 | 2019-09-09 16:20:41 -0400 | [diff] [blame] | 290 | //updateLogicalDeviceFlowGroupsWithoutLock updates the logical device with the flow groups |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 291 | func (agent *LogicalAgent) updateLogicalDeviceFlowGroupsWithoutLock(ctx context.Context, flowGroups *ofp.FlowGroups) error { |
khenaidoo | 6e55d9e | 2019-12-12 18:26:26 -0500 | [diff] [blame] | 292 | ld := agent.getLogicalDeviceWithoutLock() |
khenaidoo | 4c9e559 | 2019-09-09 16:20:41 -0400 | [diff] [blame] | 293 | |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 294 | logger.Debugw("logical-device-before", log.Fields{"lports": len(ld.Ports)}) |
khenaidoo | 6e55d9e | 2019-12-12 18:26:26 -0500 | [diff] [blame] | 295 | ld.FlowGroups = flowGroups |
| 296 | |
npujar | 467fe75 | 2020-01-16 20:17:45 +0530 | [diff] [blame] | 297 | if err := agent.updateLogicalDeviceWithoutLock(ctx, ld); err != nil { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 298 | logger.Errorw("error-updating-logical-device-with-flowgroups", log.Fields{"error": err}) |
khenaidoo | 4c9e559 | 2019-09-09 16:20:41 -0400 | [diff] [blame] | 299 | return err |
khenaidoo | 43c8212 | 2018-11-22 18:38:28 -0500 | [diff] [blame] | 300 | } |
khenaidoo | 43c8212 | 2018-11-22 18:38:28 -0500 | [diff] [blame] | 301 | return nil |
| 302 | } |
| 303 | |
khenaidoo | 6e55d9e | 2019-12-12 18:26:26 -0500 | [diff] [blame] | 304 | // getLogicalDeviceWithoutLock returns a cloned logical device to a function that already holds the agent lock. |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 305 | func (agent *LogicalAgent) getLogicalDeviceWithoutLock() *voltha.LogicalDevice { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 306 | logger.Debug("getLogicalDeviceWithoutLock") |
khenaidoo | 6e55d9e | 2019-12-12 18:26:26 -0500 | [diff] [blame] | 307 | return proto.Clone(agent.logicalDevice).(*voltha.LogicalDevice) |
khenaidoo | 92e62c5 | 2018-10-03 14:02:54 -0400 | [diff] [blame] | 308 | } |
| 309 | |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 310 | func (agent *LogicalAgent) updateLogicalPort(ctx context.Context, device *voltha.Device, port *voltha.Port) error { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 311 | logger.Debugw("updateLogicalPort", log.Fields{"deviceId": device.Id, "port": port}) |
khenaidoo | 2c6a099 | 2019-04-29 13:46:56 -0400 | [diff] [blame] | 312 | var err error |
| 313 | if port.Type == voltha.Port_ETHERNET_NNI { |
npujar | 467fe75 | 2020-01-16 20:17:45 +0530 | [diff] [blame] | 314 | if _, err = agent.addNNILogicalPort(ctx, device, port); err != nil { |
khenaidoo | 2c6a099 | 2019-04-29 13:46:56 -0400 | [diff] [blame] | 315 | return err |
| 316 | } |
| 317 | agent.addLogicalPortToMap(port.PortNo, true) |
| 318 | } else if port.Type == voltha.Port_ETHERNET_UNI { |
npujar | 467fe75 | 2020-01-16 20:17:45 +0530 | [diff] [blame] | 319 | if _, err = agent.addUNILogicalPort(ctx, device, port); err != nil { |
khenaidoo | 2c6a099 | 2019-04-29 13:46:56 -0400 | [diff] [blame] | 320 | return err |
| 321 | } |
| 322 | agent.addLogicalPortToMap(port.PortNo, false) |
| 323 | } else { |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 324 | // Update the device routes to ensure all routes on the logical device have been calculated |
| 325 | if err = agent.buildRoutes(ctx); err != nil { |
| 326 | // Not an error - temporary state |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 327 | logger.Warnw("failed-to-update-routes", log.Fields{"device-id": device.Id, "port": port, "error": err}) |
khenaidoo | 2c6a099 | 2019-04-29 13:46:56 -0400 | [diff] [blame] | 328 | } |
| 329 | } |
| 330 | return nil |
| 331 | } |
| 332 | |
khenaidoo | 3d3b8c2 | 2019-05-22 18:10:39 -0400 | [diff] [blame] | 333 | // setupLogicalPorts is invoked once the logical device has been created and is ready to get ports |
| 334 | // added to it. While the logical device was being created we could have received requests to add |
| 335 | // NNI and UNI ports which were discarded. Now is the time to add them if needed |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 336 | func (agent *LogicalAgent) setupLogicalPorts(ctx context.Context) error { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 337 | logger.Infow("setupLogicalPorts", log.Fields{"logicalDeviceId": agent.logicalDeviceID}) |
khenaidoo | 3d3b8c2 | 2019-05-22 18:10:39 -0400 | [diff] [blame] | 338 | // First add any NNI ports which could have been missing |
npujar | 467fe75 | 2020-01-16 20:17:45 +0530 | [diff] [blame] | 339 | if err := agent.setupNNILogicalPorts(ctx, agent.rootDeviceID); err != nil { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 340 | logger.Errorw("error-setting-up-NNI-ports", log.Fields{"error": err, "deviceId": agent.rootDeviceID}) |
khenaidoo | 3d3b8c2 | 2019-05-22 18:10:39 -0400 | [diff] [blame] | 341 | return err |
| 342 | } |
| 343 | |
| 344 | // Now, set up the UNI ports if needed. |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 345 | children, err := agent.deviceMgr.GetAllChildDevices(ctx, agent.rootDeviceID) |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 346 | if err != nil { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 347 | logger.Errorw("error-getting-child-devices", log.Fields{"error": err, "deviceId": agent.rootDeviceID}) |
khenaidoo | 3d3b8c2 | 2019-05-22 18:10:39 -0400 | [diff] [blame] | 348 | return err |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 349 | } |
| 350 | responses := make([]coreutils.Response, 0) |
| 351 | for _, child := range children.Items { |
| 352 | response := coreutils.NewResponse() |
| 353 | responses = append(responses, response) |
| 354 | go func(child *voltha.Device) { |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 355 | if err = agent.setupUNILogicalPorts(context.Background(), child); err != nil { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 356 | logger.Error("setting-up-UNI-ports-failed", log.Fields{"deviceID": child.Id}) |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 357 | response.Error(status.Errorf(codes.Internal, "UNI-ports-setup-failed: %s", child.Id)) |
| 358 | } |
| 359 | response.Done() |
| 360 | }(child) |
| 361 | } |
| 362 | // Wait for completion |
| 363 | if res := coreutils.WaitForNilOrErrorResponses(agent.defaultTimeout, responses...); res != nil { |
| 364 | return status.Errorf(codes.Aborted, "errors-%s", res) |
khenaidoo | 3d3b8c2 | 2019-05-22 18:10:39 -0400 | [diff] [blame] | 365 | } |
| 366 | return nil |
| 367 | } |
| 368 | |
khenaidoo | fc1314d | 2019-03-14 09:34:21 -0400 | [diff] [blame] | 369 | // setupNNILogicalPorts creates an NNI port on the logical device that represents an NNI interface on a root device |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 370 | func (agent *LogicalAgent) setupNNILogicalPorts(ctx context.Context, deviceID string) error { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 371 | logger.Infow("setupNNILogicalPorts-start", log.Fields{"logicalDeviceId": agent.logicalDeviceID}) |
khenaidoo | b920354 | 2018-09-17 22:56:37 -0400 | [diff] [blame] | 372 | // Build the logical device based on information retrieved from the device adapter |
khenaidoo | fc1314d | 2019-03-14 09:34:21 -0400 | [diff] [blame] | 373 | var err error |
| 374 | |
| 375 | var device *voltha.Device |
npujar | 467fe75 | 2020-01-16 20:17:45 +0530 | [diff] [blame] | 376 | if device, err = agent.deviceMgr.GetDevice(ctx, deviceID); err != nil { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 377 | logger.Errorw("error-retrieving-device", log.Fields{"error": err, "deviceId": deviceID}) |
khenaidoo | fc1314d | 2019-03-14 09:34:21 -0400 | [diff] [blame] | 378 | return err |
| 379 | } |
| 380 | |
| 381 | //Get UNI port number |
khenaidoo | fc1314d | 2019-03-14 09:34:21 -0400 | [diff] [blame] | 382 | for _, port := range device.Ports { |
| 383 | if port.Type == voltha.Port_ETHERNET_NNI { |
npujar | 467fe75 | 2020-01-16 20:17:45 +0530 | [diff] [blame] | 384 | if _, err = agent.addNNILogicalPort(ctx, device, port); err != nil { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 385 | logger.Errorw("error-adding-UNI-port", log.Fields{"error": err}) |
khenaidoo | fc1314d | 2019-03-14 09:34:21 -0400 | [diff] [blame] | 386 | } |
khenaidoo | 2c6a099 | 2019-04-29 13:46:56 -0400 | [diff] [blame] | 387 | agent.addLogicalPortToMap(port.PortNo, true) |
khenaidoo | fc1314d | 2019-03-14 09:34:21 -0400 | [diff] [blame] | 388 | } |
| 389 | } |
khenaidoo | fc1314d | 2019-03-14 09:34:21 -0400 | [diff] [blame] | 390 | return err |
| 391 | } |
| 392 | |
khenaidoo | 171b98e | 2019-10-31 11:48:15 -0400 | [diff] [blame] | 393 | // updatePortState updates the port state of the device |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 394 | func (agent *LogicalAgent) updatePortState(ctx context.Context, deviceID string, portNo uint32, operStatus voltha.OperStatus_Types) error { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 395 | logger.Infow("updatePortState-start", log.Fields{"logicalDeviceId": agent.logicalDeviceID, "portNo": portNo, "state": operStatus}) |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 396 | if err := agent.requestQueue.WaitForGreenLight(ctx); err != nil { |
| 397 | return err |
| 398 | } |
| 399 | defer agent.requestQueue.RequestComplete() |
khenaidoo | 171b98e | 2019-10-31 11:48:15 -0400 | [diff] [blame] | 400 | // Get the latest logical device info |
Kent Hagerman | d9cc2e9 | 2019-11-04 13:28:15 -0500 | [diff] [blame] | 401 | original := agent.getLogicalDeviceWithoutLock() |
| 402 | updatedPorts := clonePorts(original.Ports) |
| 403 | for _, port := range updatedPorts { |
| 404 | if port.DeviceId == deviceID && port.DevicePortNo == portNo { |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 405 | if operStatus == voltha.OperStatus_ACTIVE { |
Kent Hagerman | d9cc2e9 | 2019-11-04 13:28:15 -0500 | [diff] [blame] | 406 | port.OfpPort.Config = port.OfpPort.Config & ^uint32(ofp.OfpPortConfig_OFPPC_PORT_DOWN) |
| 407 | port.OfpPort.State = uint32(ofp.OfpPortState_OFPPS_LIVE) |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 408 | } else { |
Kent Hagerman | d9cc2e9 | 2019-11-04 13:28:15 -0500 | [diff] [blame] | 409 | port.OfpPort.Config = port.OfpPort.Config | uint32(ofp.OfpPortConfig_OFPPC_PORT_DOWN) |
| 410 | port.OfpPort.State = uint32(ofp.OfpPortState_OFPPS_LINK_DOWN) |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 411 | } |
| 412 | // Update the logical device |
Kent Hagerman | d9cc2e9 | 2019-11-04 13:28:15 -0500 | [diff] [blame] | 413 | if err := agent.updateLogicalDevicePortsWithoutLock(ctx, original, updatedPorts); err != nil { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 414 | logger.Errorw("error-updating-logical-device", log.Fields{"error": err}) |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 415 | return err |
| 416 | } |
| 417 | return nil |
| 418 | } |
| 419 | } |
| 420 | return status.Errorf(codes.NotFound, "port-%d-not-exist", portNo) |
khenaidoo | 171b98e | 2019-10-31 11:48:15 -0400 | [diff] [blame] | 421 | } |
| 422 | |
khenaidoo | 3ab3488 | 2019-05-02 21:33:30 -0400 | [diff] [blame] | 423 | // updatePortsState updates the ports state related to the device |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 424 | func (agent *LogicalAgent) updatePortsState(ctx context.Context, device *voltha.Device, state voltha.OperStatus_Types) error { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 425 | logger.Infow("updatePortsState-start", log.Fields{"logicalDeviceId": agent.logicalDeviceID}) |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 426 | if err := agent.requestQueue.WaitForGreenLight(ctx); err != nil { |
| 427 | return err |
| 428 | } |
| 429 | defer agent.requestQueue.RequestComplete() |
khenaidoo | 3ab3488 | 2019-05-02 21:33:30 -0400 | [diff] [blame] | 430 | // Get the latest logical device info |
Kent Hagerman | d9cc2e9 | 2019-11-04 13:28:15 -0500 | [diff] [blame] | 431 | original := agent.getLogicalDeviceWithoutLock() |
| 432 | updatedPorts := clonePorts(original.Ports) |
| 433 | for _, port := range updatedPorts { |
| 434 | if port.DeviceId == device.Id { |
kesavand | bc2d162 | 2020-01-21 00:42:01 -0500 | [diff] [blame] | 435 | if state == voltha.OperStatus_ACTIVE { |
Kent Hagerman | d9cc2e9 | 2019-11-04 13:28:15 -0500 | [diff] [blame] | 436 | port.OfpPort.Config = port.OfpPort.Config & ^uint32(ofp.OfpPortConfig_OFPPC_PORT_DOWN) |
| 437 | port.OfpPort.State = uint32(ofp.OfpPortState_OFPPS_LIVE) |
kesavand | bc2d162 | 2020-01-21 00:42:01 -0500 | [diff] [blame] | 438 | } else { |
Kent Hagerman | d9cc2e9 | 2019-11-04 13:28:15 -0500 | [diff] [blame] | 439 | port.OfpPort.Config = port.OfpPort.Config | uint32(ofp.OfpPortConfig_OFPPC_PORT_DOWN) |
| 440 | port.OfpPort.State = uint32(ofp.OfpPortState_OFPPS_LINK_DOWN) |
khenaidoo | 3ab3488 | 2019-05-02 21:33:30 -0400 | [diff] [blame] | 441 | } |
| 442 | } |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 443 | } |
| 444 | // Updating the logical device will trigger the poprt change events to be populated to the controller |
Kent Hagerman | d9cc2e9 | 2019-11-04 13:28:15 -0500 | [diff] [blame] | 445 | if err := agent.updateLogicalDevicePortsWithoutLock(ctx, original, updatedPorts); err != nil { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 446 | logger.Warnw("logical-device-update-failed", log.Fields{"ldeviceId": agent.logicalDeviceID, "error": err}) |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 447 | return err |
khenaidoo | 3ab3488 | 2019-05-02 21:33:30 -0400 | [diff] [blame] | 448 | } |
| 449 | return nil |
| 450 | } |
| 451 | |
khenaidoo | fc1314d | 2019-03-14 09:34:21 -0400 | [diff] [blame] | 452 | // setupUNILogicalPorts creates a UNI port on the logical device that represents a child UNI interface |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 453 | func (agent *LogicalAgent) setupUNILogicalPorts(ctx context.Context, childDevice *voltha.Device) error { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 454 | logger.Infow("setupUNILogicalPort", log.Fields{"logicalDeviceId": agent.logicalDeviceID}) |
khenaidoo | fc1314d | 2019-03-14 09:34:21 -0400 | [diff] [blame] | 455 | // Build the logical device based on information retrieved from the device adapter |
khenaidoo | b920354 | 2018-09-17 22:56:37 -0400 | [diff] [blame] | 456 | var err error |
khenaidoo | 59ef7be | 2019-06-21 12:40:28 -0400 | [diff] [blame] | 457 | var added bool |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 458 | //Get UNI port number |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 459 | for _, port := range childDevice.Ports { |
| 460 | if port.Type == voltha.Port_ETHERNET_UNI { |
npujar | 467fe75 | 2020-01-16 20:17:45 +0530 | [diff] [blame] | 461 | if added, err = agent.addUNILogicalPort(ctx, childDevice, port); err != nil { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 462 | logger.Errorw("error-adding-UNI-port", log.Fields{"error": err}) |
khenaidoo | fc1314d | 2019-03-14 09:34:21 -0400 | [diff] [blame] | 463 | } |
khenaidoo | 59ef7be | 2019-06-21 12:40:28 -0400 | [diff] [blame] | 464 | if added { |
| 465 | agent.addLogicalPortToMap(port.PortNo, false) |
| 466 | } |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 467 | } |
| 468 | } |
khenaidoo | fc1314d | 2019-03-14 09:34:21 -0400 | [diff] [blame] | 469 | return err |
khenaidoo | 92e62c5 | 2018-10-03 14:02:54 -0400 | [diff] [blame] | 470 | } |
| 471 | |
Girish Gowdra | 408cd96 | 2020-03-11 14:31:31 -0700 | [diff] [blame] | 472 | // deleteAllLogicalPorts deletes all logical ports associated with this logical device |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 473 | func (agent *LogicalAgent) deleteAllLogicalPorts(ctx context.Context) error { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 474 | logger.Infow("updatePortsState-start", log.Fields{"logicalDeviceId": agent.logicalDeviceID}) |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 475 | if err := agent.requestQueue.WaitForGreenLight(ctx); err != nil { |
| 476 | return err |
| 477 | } |
| 478 | defer agent.requestQueue.RequestComplete() |
khenaidoo | 0a822f9 | 2019-05-08 15:15:57 -0400 | [diff] [blame] | 479 | // Get the latest logical device info |
Kent Hagerman | d9cc2e9 | 2019-11-04 13:28:15 -0500 | [diff] [blame] | 480 | cloned := agent.getLogicalDeviceWithoutLock() |
khenaidoo | 6e55d9e | 2019-12-12 18:26:26 -0500 | [diff] [blame] | 481 | |
Kent Hagerman | d9cc2e9 | 2019-11-04 13:28:15 -0500 | [diff] [blame] | 482 | if err := agent.updateLogicalDevicePortsWithoutLock(ctx, cloned, []*voltha.LogicalPort{}); err != nil { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 483 | logger.Warnw("logical-device-update-failed", log.Fields{"ldeviceId": agent.logicalDeviceID, "error": err}) |
Girish Gowdra | 408cd96 | 2020-03-11 14:31:31 -0700 | [diff] [blame] | 484 | return err |
khenaidoo | 0a822f9 | 2019-05-08 15:15:57 -0400 | [diff] [blame] | 485 | } |
| 486 | return nil |
| 487 | } |
| 488 | |
Hardik Windlass | c704def | 2020-02-26 18:23:19 +0000 | [diff] [blame] | 489 | // deleteAllUNILogicalPorts deletes all UNI logical ports associated with this parent device |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 490 | func (agent *LogicalAgent) deleteAllUNILogicalPorts(ctx context.Context, parentDevice *voltha.Device) error { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 491 | logger.Debugw("delete-all-uni-logical-ports", log.Fields{"logical-device-id": agent.logicalDeviceID}) |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 492 | if err := agent.requestQueue.WaitForGreenLight(ctx); err != nil { |
| 493 | return err |
| 494 | } |
| 495 | defer agent.requestQueue.RequestComplete() |
Hardik Windlass | c704def | 2020-02-26 18:23:19 +0000 | [diff] [blame] | 496 | // Get the latest logical device info |
| 497 | ld := agent.getLogicalDeviceWithoutLock() |
| 498 | |
Hardik Windlass | c704def | 2020-02-26 18:23:19 +0000 | [diff] [blame] | 499 | updateLogicalPorts := []*voltha.LogicalPort{} |
Kent Hagerman | d9cc2e9 | 2019-11-04 13:28:15 -0500 | [diff] [blame] | 500 | for _, lport := range ld.Ports { |
Hardik Windlass | c704def | 2020-02-26 18:23:19 +0000 | [diff] [blame] | 501 | // Save NNI ports only |
| 502 | if agent.isNNIPort(lport.DevicePortNo) { |
| 503 | updateLogicalPorts = append(updateLogicalPorts, lport) |
| 504 | } |
| 505 | } |
Kent Hagerman | d9cc2e9 | 2019-11-04 13:28:15 -0500 | [diff] [blame] | 506 | if len(updateLogicalPorts) < len(ld.Ports) { |
Hardik Windlass | c704def | 2020-02-26 18:23:19 +0000 | [diff] [blame] | 507 | // Updating the logical device will trigger the port change events to be populated to the controller |
Kent Hagerman | d9cc2e9 | 2019-11-04 13:28:15 -0500 | [diff] [blame] | 508 | if err := agent.updateLogicalDevicePortsWithoutLock(ctx, ld, updateLogicalPorts); err != nil { |
Hardik Windlass | c704def | 2020-02-26 18:23:19 +0000 | [diff] [blame] | 509 | return err |
| 510 | } |
| 511 | } else { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 512 | logger.Debugw("no-change-required", log.Fields{"logical-device-id": agent.logicalDeviceID}) |
Hardik Windlass | c704def | 2020-02-26 18:23:19 +0000 | [diff] [blame] | 513 | } |
| 514 | return nil |
| 515 | } |
| 516 | |
Kent Hagerman | d9cc2e9 | 2019-11-04 13:28:15 -0500 | [diff] [blame] | 517 | func clonePorts(ports []*voltha.LogicalPort) []*voltha.LogicalPort { |
| 518 | return proto.Clone(&voltha.LogicalPorts{Items: ports}).(*voltha.LogicalPorts).Items |
| 519 | } |
| 520 | |
| 521 | //updateLogicalDevicePortsWithoutLock updates the |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 522 | func (agent *LogicalAgent) updateLogicalDevicePortsWithoutLock(ctx context.Context, device *voltha.LogicalDevice, newPorts []*voltha.LogicalPort) error { |
Kent Hagerman | d9cc2e9 | 2019-11-04 13:28:15 -0500 | [diff] [blame] | 523 | oldPorts := device.Ports |
| 524 | device.Ports = newPorts |
| 525 | if err := agent.updateLogicalDeviceWithoutLock(ctx, device); err != nil { |
| 526 | return err |
| 527 | } |
| 528 | agent.portUpdated(oldPorts, newPorts) |
| 529 | return nil |
| 530 | } |
| 531 | |
khenaidoo | 92e62c5 | 2018-10-03 14:02:54 -0400 | [diff] [blame] | 532 | //updateLogicalDeviceWithoutLock updates the model with the logical device. It clones the logicaldevice before saving it |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 533 | func (agent *LogicalAgent) updateLogicalDeviceWithoutLock(ctx context.Context, logicalDevice *voltha.LogicalDevice) error { |
Kent Hagerman | 4f355f5 | 2020-03-30 16:01:33 -0400 | [diff] [blame] | 534 | if agent.stopped { |
| 535 | return errors.New("logical device agent stopped") |
| 536 | } |
| 537 | |
npujar | 467fe75 | 2020-01-16 20:17:45 +0530 | [diff] [blame] | 538 | updateCtx := context.WithValue(ctx, model.RequestTimestamp, time.Now().UnixNano()) |
Kent Hagerman | 4f355f5 | 2020-03-30 16:01:33 -0400 | [diff] [blame] | 539 | if err := agent.clusterDataProxy.Update(updateCtx, "logical_devices/"+agent.logicalDeviceID, logicalDevice); err != nil { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 540 | logger.Errorw("failed-to-update-logical-devices-to-cluster-proxy", log.Fields{"error": err}) |
Thomas Lee S | e5a4401 | 2019-11-07 20:32:24 +0530 | [diff] [blame] | 541 | return err |
| 542 | } |
Kent Hagerman | 4f355f5 | 2020-03-30 16:01:33 -0400 | [diff] [blame] | 543 | |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 544 | agent.logicalDevice = logicalDevice |
| 545 | |
khenaidoo | 92e62c5 | 2018-10-03 14:02:54 -0400 | [diff] [blame] | 546 | return nil |
| 547 | } |
| 548 | |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 549 | //generateDeviceRoutesIfNeeded generates the device routes if the logical device has been updated since the last time |
khenaidoo | 4c9e559 | 2019-09-09 16:20:41 -0400 | [diff] [blame] | 550 | //that device graph was generated. |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 551 | func (agent *LogicalAgent) generateDeviceRoutesIfNeeded(ctx context.Context) error { |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 552 | agent.lockDeviceRoutes.Lock() |
| 553 | defer agent.lockDeviceRoutes.Unlock() |
| 554 | |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 555 | ld, err := agent.GetLogicalDevice(ctx) |
| 556 | if err != nil { |
| 557 | return err |
| 558 | } |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 559 | |
| 560 | if agent.deviceRoutes != nil && agent.deviceRoutes.IsUpToDate(ld) { |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 561 | return nil |
| 562 | } |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 563 | logger.Debug("Generation of device route required") |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 564 | if err := agent.buildRoutes(ctx); err != nil { |
khenaidoo | 787224a | 2020-04-16 18:08:47 -0400 | [diff] [blame] | 565 | // No Route is not an error |
| 566 | if !errors.Is(err, route.ErrNoRoute) { |
| 567 | return err |
| 568 | } |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 569 | } |
khenaidoo | 4c9e559 | 2019-09-09 16:20:41 -0400 | [diff] [blame] | 570 | return nil |
| 571 | } |
| 572 | |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 573 | //updateFlowTable updates the flow table of that logical device |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 574 | func (agent *LogicalAgent) updateFlowTable(ctx context.Context, flow *ofp.OfpFlowMod) error { |
| 575 | logger.Debug("UpdateFlowTable") |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 576 | if flow == nil { |
| 577 | return nil |
| 578 | } |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 579 | if err := agent.generateDeviceRoutesIfNeeded(ctx); err != nil { |
khenaidoo | 4c9e559 | 2019-09-09 16:20:41 -0400 | [diff] [blame] | 580 | return err |
| 581 | } |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 582 | switch flow.GetCommand() { |
| 583 | case ofp.OfpFlowModCommand_OFPFC_ADD: |
npujar | 467fe75 | 2020-01-16 20:17:45 +0530 | [diff] [blame] | 584 | return agent.flowAdd(ctx, flow) |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 585 | case ofp.OfpFlowModCommand_OFPFC_DELETE: |
npujar | 467fe75 | 2020-01-16 20:17:45 +0530 | [diff] [blame] | 586 | return agent.flowDelete(ctx, flow) |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 587 | case ofp.OfpFlowModCommand_OFPFC_DELETE_STRICT: |
npujar | 467fe75 | 2020-01-16 20:17:45 +0530 | [diff] [blame] | 588 | return agent.flowDeleteStrict(ctx, flow) |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 589 | case ofp.OfpFlowModCommand_OFPFC_MODIFY: |
| 590 | return agent.flowModify(flow) |
| 591 | case ofp.OfpFlowModCommand_OFPFC_MODIFY_STRICT: |
| 592 | return agent.flowModifyStrict(flow) |
| 593 | } |
| 594 | return status.Errorf(codes.Internal, |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 595 | "unhandled-command: lDeviceId:%s, command:%s", agent.logicalDeviceID, flow.GetCommand()) |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 596 | } |
| 597 | |
| 598 | //updateGroupTable updates the group table of that logical device |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 599 | func (agent *LogicalAgent) updateGroupTable(ctx context.Context, groupMod *ofp.OfpGroupMod) error { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 600 | logger.Debug("updateGroupTable") |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 601 | if groupMod == nil { |
| 602 | return nil |
| 603 | } |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 604 | if err := agent.generateDeviceRoutesIfNeeded(ctx); err != nil { |
khenaidoo | 4c9e559 | 2019-09-09 16:20:41 -0400 | [diff] [blame] | 605 | return err |
| 606 | } |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 607 | |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 608 | switch groupMod.GetCommand() { |
| 609 | case ofp.OfpGroupModCommand_OFPGC_ADD: |
npujar | 467fe75 | 2020-01-16 20:17:45 +0530 | [diff] [blame] | 610 | return agent.groupAdd(ctx, groupMod) |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 611 | case ofp.OfpGroupModCommand_OFPGC_DELETE: |
npujar | 467fe75 | 2020-01-16 20:17:45 +0530 | [diff] [blame] | 612 | return agent.groupDelete(ctx, groupMod) |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 613 | case ofp.OfpGroupModCommand_OFPGC_MODIFY: |
npujar | 467fe75 | 2020-01-16 20:17:45 +0530 | [diff] [blame] | 614 | return agent.groupModify(ctx, groupMod) |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 615 | } |
| 616 | return status.Errorf(codes.Internal, |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 617 | "unhandled-command: lDeviceId:%s, command:%s", agent.logicalDeviceID, groupMod.GetCommand()) |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 618 | } |
| 619 | |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 620 | // updateMeterTable updates the meter table of that logical device |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 621 | func (agent *LogicalAgent) updateMeterTable(ctx context.Context, meterMod *ofp.OfpMeterMod) error { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 622 | logger.Debug("updateMeterTable") |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 623 | if meterMod == nil { |
| 624 | return nil |
| 625 | } |
| 626 | switch meterMod.GetCommand() { |
| 627 | case ofp.OfpMeterModCommand_OFPMC_ADD: |
npujar | 467fe75 | 2020-01-16 20:17:45 +0530 | [diff] [blame] | 628 | return agent.meterAdd(ctx, meterMod) |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 629 | case ofp.OfpMeterModCommand_OFPMC_DELETE: |
npujar | 467fe75 | 2020-01-16 20:17:45 +0530 | [diff] [blame] | 630 | return agent.meterDelete(ctx, meterMod) |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 631 | case ofp.OfpMeterModCommand_OFPMC_MODIFY: |
npujar | 467fe75 | 2020-01-16 20:17:45 +0530 | [diff] [blame] | 632 | return agent.meterModify(ctx, meterMod) |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 633 | } |
| 634 | return status.Errorf(codes.Internal, |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 635 | "unhandled-command: lDeviceId:%s, command:%s", agent.logicalDeviceID, meterMod.GetCommand()) |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 636 | |
| 637 | } |
| 638 | |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 639 | func (agent *LogicalAgent) meterAdd(ctx context.Context, meterMod *ofp.OfpMeterMod) error { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 640 | logger.Debugw("meterAdd", log.Fields{"metermod": *meterMod}) |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 641 | if meterMod == nil { |
| 642 | return nil |
| 643 | } |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 644 | if err := agent.requestQueue.WaitForGreenLight(ctx); err != nil { |
| 645 | return err |
| 646 | } |
| 647 | defer agent.requestQueue.RequestComplete() |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 648 | logger.Debug("Acquired logical device lock") |
khenaidoo | 6e55d9e | 2019-12-12 18:26:26 -0500 | [diff] [blame] | 649 | lDevice := agent.getLogicalDeviceWithoutLock() |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 650 | |
| 651 | var meters []*ofp.OfpMeterEntry |
| 652 | if lDevice.Meters != nil && lDevice.Meters.Items != nil { |
| 653 | meters = lDevice.Meters.Items |
| 654 | } |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 655 | logger.Debugw("Available meters", log.Fields{"meters": meters}) |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 656 | |
| 657 | for _, meter := range meters { |
| 658 | if meterMod.MeterId == meter.Config.MeterId { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 659 | logger.Infow("Meter-already-exists", log.Fields{"meter": *meterMod}) |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 660 | return nil |
| 661 | } |
| 662 | } |
| 663 | |
| 664 | meterEntry := fu.MeterEntryFromMeterMod(meterMod) |
| 665 | meters = append(meters, meterEntry) |
| 666 | //Update model |
npujar | 467fe75 | 2020-01-16 20:17:45 +0530 | [diff] [blame] | 667 | if err := agent.updateLogicalDeviceMetersWithoutLock(ctx, &ofp.Meters{Items: meters}); err != nil { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 668 | logger.Errorw("db-meter-update-failed", log.Fields{"logicalDeviceId": agent.logicalDeviceID}) |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 669 | return err |
| 670 | } |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 671 | logger.Debugw("Meter-added-successfully", log.Fields{"Added-meter": meterEntry, "updated-meters": lDevice.Meters}) |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 672 | return nil |
| 673 | } |
| 674 | |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 675 | func (agent *LogicalAgent) meterDelete(ctx context.Context, meterMod *ofp.OfpMeterMod) error { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 676 | logger.Debug("meterDelete", log.Fields{"meterMod": *meterMod}) |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 677 | if meterMod == nil { |
| 678 | return nil |
| 679 | } |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 680 | if err := agent.requestQueue.WaitForGreenLight(ctx); err != nil { |
| 681 | return err |
| 682 | } |
| 683 | defer agent.requestQueue.RequestComplete() |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 684 | |
khenaidoo | 6e55d9e | 2019-12-12 18:26:26 -0500 | [diff] [blame] | 685 | lDevice := agent.getLogicalDeviceWithoutLock() |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 686 | |
| 687 | var meters []*ofp.OfpMeterEntry |
| 688 | var flows []*ofp.OfpFlowStats |
| 689 | updatedFlows := make([]*ofp.OfpFlowStats, 0) |
| 690 | if lDevice.Meters != nil && lDevice.Meters.Items != nil { |
| 691 | meters = lDevice.Meters.Items |
| 692 | } |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 693 | |
| 694 | changedMeter := false |
| 695 | changedFow := false |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 696 | logger.Debugw("Available meters", log.Fields{"meters": meters}) |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 697 | for index, meter := range meters { |
| 698 | if meterMod.MeterId == meter.Config.MeterId { |
| 699 | flows = lDevice.Flows.Items |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 700 | changedFow, updatedFlows = agent.getUpdatedFlowsAfterDeletebyMeterID(flows, meterMod.MeterId) |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 701 | meters = append(meters[:index], meters[index+1:]...) |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 702 | logger.Debugw("Meter has been deleted", log.Fields{"meter": meter, "index": index}) |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 703 | changedMeter = true |
| 704 | break |
| 705 | } |
| 706 | } |
| 707 | if changedMeter { |
| 708 | //Update model |
| 709 | metersToUpdate := &ofp.Meters{} |
| 710 | if lDevice.Meters != nil { |
| 711 | metersToUpdate = &ofp.Meters{Items: meters} |
| 712 | } |
npujar | 467fe75 | 2020-01-16 20:17:45 +0530 | [diff] [blame] | 713 | if err := agent.updateLogicalDeviceMetersWithoutLock(ctx, metersToUpdate); err != nil { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 714 | logger.Errorw("db-meter-update-failed", log.Fields{"logicalDeviceId": agent.logicalDeviceID}) |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 715 | return err |
| 716 | } |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 717 | logger.Debug("Meter-deleted-from-DB-successfully", log.Fields{"updatedMeters": metersToUpdate, "no-of-meter": len(metersToUpdate.Items)}) |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 718 | |
| 719 | } |
| 720 | if changedFow { |
| 721 | //Update model |
npujar | 467fe75 | 2020-01-16 20:17:45 +0530 | [diff] [blame] | 722 | if err := agent.updateLogicalDeviceFlowsWithoutLock(ctx, &ofp.Flows{Items: updatedFlows}); err != nil { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 723 | logger.Errorw("db-flow-update-failed", log.Fields{"logicalDeviceId": agent.logicalDeviceID}) |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 724 | return err |
| 725 | } |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 726 | logger.Debug("Flows-associated-with-meter-deleted-from-DB-successfully", |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 727 | log.Fields{"updated-no-of-flows": len(updatedFlows), "meter": meterMod.MeterId}) |
| 728 | } |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 729 | logger.Debugw("meterDelete success", log.Fields{"meterID": meterMod.MeterId}) |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 730 | return nil |
| 731 | } |
| 732 | |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 733 | func (agent *LogicalAgent) meterModify(ctx context.Context, meterMod *ofp.OfpMeterMod) error { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 734 | logger.Debug("meterModify") |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 735 | if meterMod == nil { |
| 736 | return nil |
| 737 | } |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 738 | if err := agent.requestQueue.WaitForGreenLight(ctx); err != nil { |
| 739 | return err |
| 740 | } |
| 741 | defer agent.requestQueue.RequestComplete() |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 742 | |
khenaidoo | 6e55d9e | 2019-12-12 18:26:26 -0500 | [diff] [blame] | 743 | lDevice := agent.getLogicalDeviceWithoutLock() |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 744 | |
| 745 | var meters []*ofp.OfpMeterEntry |
| 746 | if lDevice.Meters != nil && lDevice.Meters.Items != nil { |
| 747 | meters = lDevice.Meters.Items |
| 748 | } |
| 749 | changedMeter := false |
| 750 | for index, meter := range meters { |
| 751 | if meterMod.MeterId == meter.Config.MeterId { |
| 752 | newmeterEntry := fu.MeterEntryFromMeterMod(meterMod) |
| 753 | newmeterEntry.Stats.FlowCount = meter.Stats.FlowCount |
| 754 | meters[index] = newmeterEntry |
| 755 | changedMeter = true |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 756 | logger.Debugw("Found meter, replaced with new meter", log.Fields{"old meter": meter, "new meter": newmeterEntry}) |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 757 | break |
| 758 | } |
| 759 | } |
| 760 | if changedMeter { |
| 761 | //Update model |
| 762 | metersToUpdate := &ofp.Meters{} |
| 763 | if lDevice.Meters != nil { |
| 764 | metersToUpdate = &ofp.Meters{Items: meters} |
| 765 | } |
npujar | 467fe75 | 2020-01-16 20:17:45 +0530 | [diff] [blame] | 766 | if err := agent.updateLogicalDeviceMetersWithoutLock(ctx, metersToUpdate); err != nil { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 767 | logger.Errorw("db-meter-update-failed", log.Fields{"logicalDeviceId": agent.logicalDeviceID}) |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 768 | return err |
| 769 | } |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 770 | logger.Debugw("meter-updated-in-DB-successfully", log.Fields{"updated_meters": meters}) |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 771 | return nil |
| 772 | } |
| 773 | |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 774 | logger.Errorw("Meter not found ", log.Fields{"meter": meterMod}) |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 775 | return fmt.Errorf("no-logical-device-present:%d", meterMod.MeterId) |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 776 | |
| 777 | } |
| 778 | |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 779 | func (agent *LogicalAgent) getUpdatedFlowsAfterDeletebyMeterID(flows []*ofp.OfpFlowStats, meterID uint32) (bool, []*ofp.OfpFlowStats) { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 780 | logger.Infow("Delete flows matching meter", log.Fields{"meter": meterID}) |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 781 | changed := false |
| 782 | //updatedFlows := make([]*ofp.OfpFlowStats, 0) |
| 783 | for index := len(flows) - 1; index >= 0; index-- { |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 784 | if mID := fu.GetMeterIdFromFlow(flows[index]); mID != 0 && mID == meterID { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 785 | logger.Debugw("Flow to be deleted", log.Fields{"flow": flows[index], "index": index}) |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 786 | flows = append(flows[:index], flows[index+1:]...) |
| 787 | changed = true |
| 788 | } |
| 789 | } |
| 790 | return changed, flows |
| 791 | } |
| 792 | |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 793 | func (agent *LogicalAgent) updateFlowCountOfMeterStats(modCommand *ofp.OfpFlowMod, meters []*ofp.OfpMeterEntry, flow *ofp.OfpFlowStats) bool { |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 794 | |
| 795 | flowCommand := modCommand.GetCommand() |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 796 | meterID := fu.GetMeterIdFromFlow(flow) |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 797 | logger.Debugw("Meter-id-in-flow-mod", log.Fields{"meterId": meterID}) |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 798 | if meterID == 0 { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 799 | logger.Debugw("No meter present in the flow", log.Fields{"flow": *flow}) |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 800 | return false |
| 801 | } |
| 802 | if meters == nil { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 803 | logger.Debug("No meters present in logical device") |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 804 | return false |
| 805 | } |
| 806 | changedMeter := false |
| 807 | for _, meter := range meters { |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 808 | if meterID == meter.Config.MeterId { // Found meter in Logicaldevice |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 809 | if flowCommand == ofp.OfpFlowModCommand_OFPFC_ADD { |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 810 | meter.Stats.FlowCount++ |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 811 | changedMeter = true |
| 812 | } else if flowCommand == ofp.OfpFlowModCommand_OFPFC_DELETE_STRICT { |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 813 | meter.Stats.FlowCount-- |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 814 | changedMeter = true |
| 815 | } |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 816 | logger.Debugw("Found meter, updated meter flow stats", log.Fields{" meterId": meterID}) |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 817 | break |
| 818 | } |
| 819 | } |
| 820 | return changedMeter |
| 821 | } |
| 822 | |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 823 | //flowAdd adds a flow to the flow table of that logical device |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 824 | func (agent *LogicalAgent) flowAdd(ctx context.Context, mod *ofp.OfpFlowMod) error { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 825 | logger.Debugw("flowAdd", log.Fields{"flow": mod}) |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 826 | if mod == nil { |
| 827 | return nil |
| 828 | } |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 829 | if err := agent.requestQueue.WaitForGreenLight(ctx); err != nil { |
| 830 | return err |
| 831 | } |
| 832 | defer agent.requestQueue.RequestComplete() |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 833 | |
khenaidoo | 6e55d9e | 2019-12-12 18:26:26 -0500 | [diff] [blame] | 834 | lDevice := agent.getLogicalDeviceWithoutLock() |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 835 | |
| 836 | var flows []*ofp.OfpFlowStats |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 837 | var meters []*ofp.OfpMeterEntry |
| 838 | var flow *ofp.OfpFlowStats |
Scott Baker | fdea1e3 | 2020-02-21 15:35:41 -0800 | [diff] [blame] | 839 | var err error |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 840 | |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 841 | if lDevice.Flows != nil && lDevice.Flows.Items != nil { |
| 842 | flows = lDevice.Flows.Items |
| 843 | } |
| 844 | |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 845 | if lDevice.Meters != nil && lDevice.Meters.Items != nil { |
| 846 | meters = lDevice.Meters.Items |
| 847 | } |
khenaidoo | 2c6a099 | 2019-04-29 13:46:56 -0400 | [diff] [blame] | 848 | updatedFlows := make([]*ofp.OfpFlowStats, 0) |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 849 | changed := false |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 850 | updated := false |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 851 | checkOverlap := (mod.Flags & uint32(ofp.OfpFlowModFlags_OFPFF_CHECK_OVERLAP)) != 0 |
| 852 | if checkOverlap { |
| 853 | if overlapped := fu.FindOverlappingFlows(flows, mod); len(overlapped) != 0 { |
| 854 | // TODO: should this error be notified other than being logged? |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 855 | logger.Warnw("overlapped-flows", log.Fields{"logicaldeviceId": agent.logicalDeviceID}) |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 856 | } else { |
Matt Jeanneret | 518b5a4 | 2019-10-29 10:30:46 -0400 | [diff] [blame] | 857 | // Add flow |
Scott Baker | fdea1e3 | 2020-02-21 15:35:41 -0800 | [diff] [blame] | 858 | flow, err = fu.FlowStatsEntryFromFlowModMessage(mod) |
| 859 | if err != nil { |
| 860 | return err |
| 861 | } |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 862 | flows = append(flows, flow) |
khenaidoo | 2c6a099 | 2019-04-29 13:46:56 -0400 | [diff] [blame] | 863 | updatedFlows = append(updatedFlows, flow) |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 864 | changed = true |
| 865 | } |
| 866 | } else { |
Scott Baker | fdea1e3 | 2020-02-21 15:35:41 -0800 | [diff] [blame] | 867 | flow, err = fu.FlowStatsEntryFromFlowModMessage(mod) |
| 868 | if err != nil { |
| 869 | return err |
| 870 | } |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 871 | idx := fu.FindFlows(flows, flow) |
| 872 | if idx >= 0 { |
| 873 | oldFlow := flows[idx] |
| 874 | if (mod.Flags & uint32(ofp.OfpFlowModFlags_OFPFF_RESET_COUNTS)) != 0 { |
| 875 | flow.ByteCount = oldFlow.ByteCount |
| 876 | flow.PacketCount = oldFlow.PacketCount |
| 877 | } |
Mahir Gunyel | 5e98c78 | 2020-04-13 15:08:29 -0700 | [diff] [blame^] | 878 | if !proto.Equal(oldFlow, flow) { |
Matt Jeanneret | 518b5a4 | 2019-10-29 10:30:46 -0400 | [diff] [blame] | 879 | flows[idx] = flow |
| 880 | updatedFlows = append(updatedFlows, flow) |
| 881 | changed = true |
| 882 | updated = true |
| 883 | } |
| 884 | } else { |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 885 | flows = append(flows, flow) |
khenaidoo | 2c6a099 | 2019-04-29 13:46:56 -0400 | [diff] [blame] | 886 | updatedFlows = append(updatedFlows, flow) |
| 887 | changed = true |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 888 | } |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 889 | } |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 890 | logger.Debugw("flowAdd-changed", log.Fields{"changed": changed}) |
khenaidoo | 4c9e559 | 2019-09-09 16:20:41 -0400 | [diff] [blame] | 891 | |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 892 | if changed { |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 893 | var flowMetadata voltha.FlowMetadata |
| 894 | if err := agent.GetMeterConfig(updatedFlows, meters, &flowMetadata); err != nil { // This should never happen,meters should be installed before flow arrives |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 895 | logger.Error("Meter-referred-in-flows-not-present") |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 896 | return err |
| 897 | } |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 898 | deviceRules, err := agent.flowDecomposer.DecomposeRules(ctx, agent, ofp.Flows{Items: updatedFlows}, *lDevice.FlowGroups) |
| 899 | if err != nil { |
| 900 | return err |
| 901 | } |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 902 | logger.Debugw("rules", log.Fields{"rules": deviceRules.String()}) |
khenaidoo | 0458db6 | 2019-06-20 08:50:36 -0400 | [diff] [blame] | 903 | |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 904 | // Update model |
npujar | 467fe75 | 2020-01-16 20:17:45 +0530 | [diff] [blame] | 905 | if err := agent.updateLogicalDeviceFlowsWithoutLock(ctx, &ofp.Flows{Items: flows}); err != nil { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 906 | logger.Errorw("db-flow-update-failed", log.Fields{"logicalDeviceId": agent.logicalDeviceID}) |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 907 | return err |
| 908 | } |
Matt Jeanneret | 518b5a4 | 2019-10-29 10:30:46 -0400 | [diff] [blame] | 909 | if !updated { |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 910 | changedMeterStats := agent.updateFlowCountOfMeterStats(mod, meters, flow) |
| 911 | metersToUpdate := &ofp.Meters{} |
| 912 | if lDevice.Meters != nil { |
| 913 | metersToUpdate = &ofp.Meters{Items: meters} |
| 914 | } |
| 915 | if changedMeterStats { |
| 916 | //Update model |
npujar | 467fe75 | 2020-01-16 20:17:45 +0530 | [diff] [blame] | 917 | if err := agent.updateLogicalDeviceMetersWithoutLock(ctx, metersToUpdate); err != nil { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 918 | logger.Errorw("db-meter-update-failed", log.Fields{"logicalDeviceId": agent.logicalDeviceID}) |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 919 | return err |
| 920 | } |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 921 | logger.Debugw("meter-stats-updated-in-DB-successfully", log.Fields{"updated_meters": meters}) |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 922 | |
| 923 | } |
| 924 | } |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 925 | // Send the flows to the devices |
| 926 | respChannels := agent.addFlowsAndGroupsToDevices(ctx, deviceRules, &flowMetadata) |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 927 | |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 928 | // Create the go routines to wait |
| 929 | go func() { |
| 930 | // Wait for completion |
| 931 | if res := coreutils.WaitForNilOrErrorResponses(agent.defaultTimeout, respChannels...); res != nil { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 932 | logger.Warnw("failure-to-add-flows", log.Fields{"errors": res, "logical-device-id": agent.logicalDeviceID}) |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 933 | // TODO : revert added flow |
| 934 | } |
| 935 | }() |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 936 | } |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 937 | return nil |
| 938 | } |
| 939 | |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 940 | // GetMeterConfig returns meter config |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 941 | func (agent *LogicalAgent) GetMeterConfig(flows []*ofp.OfpFlowStats, meters []*ofp.OfpMeterEntry, metadata *voltha.FlowMetadata) error { |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 942 | m := make(map[uint32]bool) |
| 943 | for _, flow := range flows { |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 944 | if flowMeterID := fu.GetMeterIdFromFlow(flow); flowMeterID != 0 && !m[flowMeterID] { |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 945 | foundMeter := false |
| 946 | // Meter is present in the flow , Get from logical device |
| 947 | for _, meter := range meters { |
| 948 | if flowMeterID == meter.Config.MeterId { |
| 949 | metadata.Meters = append(metadata.Meters, meter.Config) |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 950 | logger.Debugw("Found meter in logical device", |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 951 | log.Fields{"meterID": flowMeterID, "meter-band": meter.Config}) |
| 952 | m[flowMeterID] = true |
| 953 | foundMeter = true |
| 954 | break |
| 955 | } |
| 956 | } |
| 957 | if !foundMeter { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 958 | logger.Errorw("Meter-referred-by-flow-is-not-found-in-logicaldevice", |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 959 | log.Fields{"meterID": flowMeterID, "Available-meters": meters, "flow": *flow}) |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 960 | return errors.New("Meter-referred-by-flow-is-not-found-in-logicaldevice") |
| 961 | } |
| 962 | } |
| 963 | } |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 964 | logger.Debugw("meter-bands-for-flows", log.Fields{"flows": len(flows), "metadata": metadata}) |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 965 | return nil |
| 966 | |
| 967 | } |
| 968 | |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 969 | //flowDelete deletes a flow from the flow table of that logical device |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 970 | func (agent *LogicalAgent) flowDelete(ctx context.Context, mod *ofp.OfpFlowMod) error { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 971 | logger.Debug("flowDelete") |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 972 | if mod == nil { |
| 973 | return nil |
| 974 | } |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 975 | if err := agent.requestQueue.WaitForGreenLight(ctx); err != nil { |
| 976 | return err |
| 977 | } |
| 978 | defer agent.requestQueue.RequestComplete() |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 979 | |
khenaidoo | 6e55d9e | 2019-12-12 18:26:26 -0500 | [diff] [blame] | 980 | lDevice := agent.getLogicalDeviceWithoutLock() |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 981 | |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 982 | var meters []*ofp.OfpMeterEntry |
| 983 | var flows []*ofp.OfpFlowStats |
Esin Karaman | fc3b1af | 2020-01-29 16:52:00 +0000 | [diff] [blame] | 984 | var flowGroups []*ofp.OfpGroupEntry |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 985 | |
| 986 | if lDevice.Flows != nil && lDevice.Flows.Items != nil { |
| 987 | flows = lDevice.Flows.Items |
| 988 | } |
| 989 | |
| 990 | if lDevice.Meters != nil && lDevice.Meters.Items != nil { |
| 991 | meters = lDevice.Meters.Items |
| 992 | } |
Esin Karaman | fc3b1af | 2020-01-29 16:52:00 +0000 | [diff] [blame] | 993 | |
| 994 | if lDevice.FlowGroups != nil && lDevice.FlowGroups.Items != nil { |
| 995 | flowGroups = lDevice.FlowGroups.Items |
| 996 | } |
| 997 | |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 998 | //build a list of what to keep vs what to delete |
| 999 | toKeep := make([]*ofp.OfpFlowStats, 0) |
khenaidoo | 0458db6 | 2019-06-20 08:50:36 -0400 | [diff] [blame] | 1000 | toDelete := make([]*ofp.OfpFlowStats, 0) |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1001 | for _, f := range flows { |
khenaidoo | 0458db6 | 2019-06-20 08:50:36 -0400 | [diff] [blame] | 1002 | // Check whether the flow and the flowmod matches |
Scott Baker | fdea1e3 | 2020-02-21 15:35:41 -0800 | [diff] [blame] | 1003 | fs, err := fu.FlowStatsEntryFromFlowModMessage(mod) |
| 1004 | if err != nil { |
| 1005 | return err |
| 1006 | } |
| 1007 | if fu.FlowMatch(f, fs) { |
khenaidoo | 0458db6 | 2019-06-20 08:50:36 -0400 | [diff] [blame] | 1008 | toDelete = append(toDelete, f) |
| 1009 | continue |
| 1010 | } |
| 1011 | // Check wild card match |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1012 | if !fu.FlowMatchesMod(f, mod) { |
| 1013 | toKeep = append(toKeep, f) |
khenaidoo | 0458db6 | 2019-06-20 08:50:36 -0400 | [diff] [blame] | 1014 | } else { |
| 1015 | toDelete = append(toDelete, f) |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1016 | } |
| 1017 | } |
| 1018 | |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1019 | logger.Debugw("flowDelete", log.Fields{"logicalDeviceId": agent.logicalDeviceID, "toKeep": len(toKeep), "toDelete": toDelete}) |
khenaidoo | 0458db6 | 2019-06-20 08:50:36 -0400 | [diff] [blame] | 1020 | |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1021 | //Update flows |
khenaidoo | 0458db6 | 2019-06-20 08:50:36 -0400 | [diff] [blame] | 1022 | if len(toDelete) > 0 { |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 1023 | var flowMetadata voltha.FlowMetadata |
| 1024 | if err := agent.GetMeterConfig(toDelete, meters, &flowMetadata); err != nil { // This should never happen |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1025 | logger.Error("Meter-referred-in-flows-not-present") |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 1026 | return errors.New("Meter-referred-in-flows-not-present") |
| 1027 | } |
khenaidoo | 787224a | 2020-04-16 18:08:47 -0400 | [diff] [blame] | 1028 | |
| 1029 | var respChnls []coreutils.Response |
| 1030 | var partialRoute bool |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 1031 | deviceRules, err := agent.flowDecomposer.DecomposeRules(ctx, agent, ofp.Flows{Items: toDelete}, ofp.FlowGroups{Items: flowGroups}) |
| 1032 | if err != nil { |
khenaidoo | 787224a | 2020-04-16 18:08:47 -0400 | [diff] [blame] | 1033 | // A no route error means no route exists between the ports specified in the flow. This can happen when the |
| 1034 | // child device is deleted and a request to delete flows from the parent device is received |
| 1035 | if !errors.Is(err, route.ErrNoRoute) { |
| 1036 | logger.Errorw("unexpected-error-received", log.Fields{"flows-to-delete": toDelete, "error": err}) |
| 1037 | return err |
| 1038 | } |
| 1039 | partialRoute = true |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 1040 | } |
khenaidoo | 0458db6 | 2019-06-20 08:50:36 -0400 | [diff] [blame] | 1041 | |
khenaidoo | 787224a | 2020-04-16 18:08:47 -0400 | [diff] [blame] | 1042 | // Update the dB |
npujar | 467fe75 | 2020-01-16 20:17:45 +0530 | [diff] [blame] | 1043 | if err := agent.updateLogicalDeviceFlowsWithoutLock(ctx, &ofp.Flows{Items: toKeep}); err != nil { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1044 | logger.Errorw("cannot-update-flows", log.Fields{"logicalDeviceId": agent.logicalDeviceID}) |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1045 | return err |
| 1046 | } |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 1047 | |
| 1048 | // Update the devices |
khenaidoo | 787224a | 2020-04-16 18:08:47 -0400 | [diff] [blame] | 1049 | if partialRoute { |
| 1050 | respChnls = agent.deleteFlowsFromParentDevice(ctx, ofp.Flows{Items: toDelete}, &flowMetadata) |
| 1051 | } else { |
| 1052 | respChnls = agent.deleteFlowsAndGroupsFromDevices(ctx, deviceRules, &flowMetadata) |
| 1053 | } |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 1054 | |
| 1055 | // Wait for the responses |
| 1056 | go func() { |
| 1057 | // Wait for completion |
| 1058 | if res := coreutils.WaitForNilOrErrorResponses(agent.defaultTimeout, respChnls...); res != nil { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1059 | logger.Errorw("failure-updating-device-flows", log.Fields{"logicalDeviceId": agent.logicalDeviceID, "errors": res}) |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 1060 | // TODO: Revert the flow deletion |
| 1061 | } |
| 1062 | }() |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1063 | } |
| 1064 | |
| 1065 | //TODO: send announcement on delete |
| 1066 | return nil |
| 1067 | } |
| 1068 | |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 1069 | func (agent *LogicalAgent) addFlowsAndGroupsToDevices(ctx context.Context, deviceRules *fu.DeviceRules, flowMetadata *voltha.FlowMetadata) []coreutils.Response { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1070 | logger.Debugw("send-add-flows-to-device-manager", log.Fields{"logicalDeviceID": agent.logicalDeviceID, "deviceRules": deviceRules, "flowMetadata": flowMetadata}) |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1071 | |
Kent Hagerman | 8da2f1e | 2019-11-25 17:28:09 -0500 | [diff] [blame] | 1072 | responses := make([]coreutils.Response, 0) |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 1073 | for deviceID, value := range deviceRules.GetRules() { |
Kent Hagerman | 8da2f1e | 2019-11-25 17:28:09 -0500 | [diff] [blame] | 1074 | response := coreutils.NewResponse() |
| 1075 | responses = append(responses, response) |
| 1076 | go func(deviceId string, value *fu.FlowsAndGroups) { |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 1077 | ctx, cancel := context.WithTimeout(context.Background(), agent.defaultTimeout) |
| 1078 | defer cancel() |
npujar | 467fe75 | 2020-01-16 20:17:45 +0530 | [diff] [blame] | 1079 | if err := agent.deviceMgr.addFlowsAndGroups(ctx, deviceId, value.ListFlows(), value.ListGroups(), flowMetadata); err != nil { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1080 | logger.Errorw("flow-add-failed", log.Fields{"deviceID": deviceId, "error": err}) |
Kent Hagerman | 8da2f1e | 2019-11-25 17:28:09 -0500 | [diff] [blame] | 1081 | response.Error(status.Errorf(codes.Internal, "flow-add-failed: %s", deviceId)) |
khenaidoo | 0458db6 | 2019-06-20 08:50:36 -0400 | [diff] [blame] | 1082 | } |
Kent Hagerman | 8da2f1e | 2019-11-25 17:28:09 -0500 | [diff] [blame] | 1083 | response.Done() |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 1084 | }(deviceID, value) |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1085 | } |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 1086 | // Return responses (an array of channels) for the caller to wait for a response from the far end. |
| 1087 | return responses |
khenaidoo | 0458db6 | 2019-06-20 08:50:36 -0400 | [diff] [blame] | 1088 | } |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1089 | |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 1090 | func (agent *LogicalAgent) deleteFlowsAndGroupsFromDevices(ctx context.Context, deviceRules *fu.DeviceRules, flowMetadata *voltha.FlowMetadata) []coreutils.Response { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1091 | logger.Debugw("send-delete-flows-to-device-manager", log.Fields{"logicalDeviceID": agent.logicalDeviceID}) |
khenaidoo | 0458db6 | 2019-06-20 08:50:36 -0400 | [diff] [blame] | 1092 | |
Kent Hagerman | 8da2f1e | 2019-11-25 17:28:09 -0500 | [diff] [blame] | 1093 | responses := make([]coreutils.Response, 0) |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 1094 | for deviceID, value := range deviceRules.GetRules() { |
Kent Hagerman | 8da2f1e | 2019-11-25 17:28:09 -0500 | [diff] [blame] | 1095 | response := coreutils.NewResponse() |
| 1096 | responses = append(responses, response) |
| 1097 | go func(deviceId string, value *fu.FlowsAndGroups) { |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 1098 | ctx, cancel := context.WithTimeout(context.Background(), agent.defaultTimeout) |
| 1099 | defer cancel() |
npujar | 467fe75 | 2020-01-16 20:17:45 +0530 | [diff] [blame] | 1100 | if err := agent.deviceMgr.deleteFlowsAndGroups(ctx, deviceId, value.ListFlows(), value.ListGroups(), flowMetadata); err != nil { |
khenaidoo | 787224a | 2020-04-16 18:08:47 -0400 | [diff] [blame] | 1101 | logger.Errorw("flow-delete-failed", log.Fields{"deviceID": deviceId, "error": err}) |
Kent Hagerman | 8da2f1e | 2019-11-25 17:28:09 -0500 | [diff] [blame] | 1102 | response.Error(status.Errorf(codes.Internal, "flow-delete-failed: %s", deviceId)) |
khenaidoo | 0458db6 | 2019-06-20 08:50:36 -0400 | [diff] [blame] | 1103 | } |
Kent Hagerman | 8da2f1e | 2019-11-25 17:28:09 -0500 | [diff] [blame] | 1104 | response.Done() |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 1105 | }(deviceID, value) |
khenaidoo | 0458db6 | 2019-06-20 08:50:36 -0400 | [diff] [blame] | 1106 | } |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 1107 | return responses |
khenaidoo | 0458db6 | 2019-06-20 08:50:36 -0400 | [diff] [blame] | 1108 | } |
| 1109 | |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 1110 | func (agent *LogicalAgent) updateFlowsAndGroupsOfDevice(ctx context.Context, deviceRules *fu.DeviceRules, flowMetadata *voltha.FlowMetadata) []coreutils.Response { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1111 | logger.Debugw("send-update-flows-to-device-manager", log.Fields{"logicalDeviceID": agent.logicalDeviceID}) |
khenaidoo | 0458db6 | 2019-06-20 08:50:36 -0400 | [diff] [blame] | 1112 | |
Kent Hagerman | 8da2f1e | 2019-11-25 17:28:09 -0500 | [diff] [blame] | 1113 | responses := make([]coreutils.Response, 0) |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 1114 | for deviceID, value := range deviceRules.GetRules() { |
Kent Hagerman | 8da2f1e | 2019-11-25 17:28:09 -0500 | [diff] [blame] | 1115 | response := coreutils.NewResponse() |
| 1116 | responses = append(responses, response) |
| 1117 | go func(deviceId string, value *fu.FlowsAndGroups) { |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 1118 | ctx, cancel := context.WithTimeout(context.Background(), agent.defaultTimeout) |
| 1119 | defer cancel() |
npujar | 467fe75 | 2020-01-16 20:17:45 +0530 | [diff] [blame] | 1120 | if err := agent.deviceMgr.updateFlowsAndGroups(ctx, deviceId, value.ListFlows(), value.ListGroups(), flowMetadata); err != nil { |
khenaidoo | 787224a | 2020-04-16 18:08:47 -0400 | [diff] [blame] | 1121 | logger.Errorw("flow-update-failed", log.Fields{"deviceID": deviceId, "error": err}) |
Kent Hagerman | 8da2f1e | 2019-11-25 17:28:09 -0500 | [diff] [blame] | 1122 | response.Error(status.Errorf(codes.Internal, "flow-update-failed: %s", deviceId)) |
khenaidoo | 0458db6 | 2019-06-20 08:50:36 -0400 | [diff] [blame] | 1123 | } |
Kent Hagerman | 8da2f1e | 2019-11-25 17:28:09 -0500 | [diff] [blame] | 1124 | response.Done() |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 1125 | }(deviceID, value) |
khenaidoo | 0458db6 | 2019-06-20 08:50:36 -0400 | [diff] [blame] | 1126 | } |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 1127 | return responses |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1128 | } |
| 1129 | |
khenaidoo | 787224a | 2020-04-16 18:08:47 -0400 | [diff] [blame] | 1130 | // getUNILogicalPortNo returns the UNI logical port number specified in the flow |
| 1131 | func (agent *LogicalAgent) getUNILogicalPortNo(flow *ofp.OfpFlowStats) (uint32, error) { |
| 1132 | var uniPort uint32 |
| 1133 | inPortNo := fu.GetInPort(flow) |
| 1134 | outPortNo := fu.GetOutPort(flow) |
| 1135 | if agent.isNNIPort(inPortNo) { |
| 1136 | uniPort = outPortNo |
| 1137 | } else if agent.isNNIPort(outPortNo) { |
| 1138 | uniPort = inPortNo |
| 1139 | } |
| 1140 | if uniPort != 0 { |
| 1141 | return uniPort, nil |
| 1142 | } |
| 1143 | return 0, status.Errorf(codes.NotFound, "no-uni-port: %v", flow) |
| 1144 | } |
| 1145 | |
| 1146 | func (agent *LogicalAgent) deleteFlowsFromParentDevice(ctx context.Context, flows ofp.Flows, metadata *voltha.FlowMetadata) []coreutils.Response { |
| 1147 | logger.Debugw("deleting-flows-from-parent-device", log.Fields{"logical-device-id": agent.logicalDeviceID, "flows": flows}) |
| 1148 | responses := make([]coreutils.Response, 0) |
| 1149 | for _, flow := range flows.Items { |
| 1150 | response := coreutils.NewResponse() |
| 1151 | responses = append(responses, response) |
| 1152 | uniPort, err := agent.getUNILogicalPortNo(flow) |
| 1153 | if err != nil { |
| 1154 | logger.Error("no-uni-port-in-flow", log.Fields{"deviceID": agent.rootDeviceID, "flow": flow, "error": err}) |
| 1155 | response.Error(err) |
| 1156 | response.Done() |
| 1157 | continue |
| 1158 | } |
| 1159 | logger.Debugw("uni-port", log.Fields{"flows": flows, "uni-port": uniPort}) |
| 1160 | go func(uniPort uint32, metadata *voltha.FlowMetadata) { |
| 1161 | ctx, cancel := context.WithTimeout(context.Background(), agent.defaultTimeout) |
| 1162 | defer cancel() |
| 1163 | if err := agent.deviceMgr.deleteParentFlows(ctx, agent.rootDeviceID, uniPort, metadata); err != nil { |
| 1164 | logger.Error("flow-delete-failed", log.Fields{"device-id": agent.rootDeviceID, "error": err}) |
| 1165 | response.Error(status.Errorf(codes.Internal, "flow-delete-failed: %s %v", agent.rootDeviceID, err)) |
| 1166 | } |
| 1167 | response.Done() |
| 1168 | }(uniPort, metadata) |
| 1169 | } |
| 1170 | return responses |
| 1171 | } |
| 1172 | |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1173 | //flowDeleteStrict deletes a flow from the flow table of that logical device |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 1174 | func (agent *LogicalAgent) flowDeleteStrict(ctx context.Context, mod *ofp.OfpFlowMod) error { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1175 | logger.Debug("flowDeleteStrict") |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1176 | if mod == nil { |
| 1177 | return nil |
| 1178 | } |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 1179 | if err := agent.requestQueue.WaitForGreenLight(ctx); err != nil { |
| 1180 | return err |
| 1181 | } |
| 1182 | defer agent.requestQueue.RequestComplete() |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1183 | |
khenaidoo | 6e55d9e | 2019-12-12 18:26:26 -0500 | [diff] [blame] | 1184 | lDevice := agent.getLogicalDeviceWithoutLock() |
| 1185 | |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 1186 | var meters []*ofp.OfpMeterEntry |
| 1187 | var flows []*ofp.OfpFlowStats |
Esin Karaman | fc3b1af | 2020-01-29 16:52:00 +0000 | [diff] [blame] | 1188 | var flowGroups []*ofp.OfpGroupEntry |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 1189 | if lDevice.Meters != nil && lDevice.Meters.Items != nil { |
| 1190 | meters = lDevice.Meters.Items |
| 1191 | } |
| 1192 | if lDevice.Flows != nil && lDevice.Flows.Items != nil { |
| 1193 | flows = lDevice.Flows.Items |
| 1194 | } |
Esin Karaman | fc3b1af | 2020-01-29 16:52:00 +0000 | [diff] [blame] | 1195 | if lDevice.FlowGroups != nil && lDevice.FlowGroups.Items != nil { |
| 1196 | flowGroups = lDevice.FlowGroups.Items |
| 1197 | } |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 1198 | |
| 1199 | changedFlow := false |
| 1200 | changedMeter := false |
Scott Baker | fdea1e3 | 2020-02-21 15:35:41 -0800 | [diff] [blame] | 1201 | flow, err := fu.FlowStatsEntryFromFlowModMessage(mod) |
| 1202 | if err != nil { |
| 1203 | return err |
| 1204 | } |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 1205 | flowsToDelete := make([]*ofp.OfpFlowStats, 0) |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1206 | idx := fu.FindFlows(flows, flow) |
| 1207 | if idx >= 0 { |
Gamze Abaka | 6e4ac16 | 2019-10-21 11:10:10 +0000 | [diff] [blame] | 1208 | changedMeter = agent.updateFlowCountOfMeterStats(mod, meters, flows[idx]) |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 1209 | flowsToDelete = append(flowsToDelete, flows[idx]) |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1210 | flows = append(flows[:idx], flows[idx+1:]...) |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 1211 | changedFlow = true |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1212 | } else { |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 1213 | return fmt.Errorf("Cannot delete flow - %s", flow) |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1214 | } |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 1215 | if changedMeter { |
| 1216 | //Update model |
| 1217 | metersToUpdate := &ofp.Meters{} |
| 1218 | if lDevice.Meters != nil { |
| 1219 | metersToUpdate = &ofp.Meters{Items: meters} |
| 1220 | } |
npujar | 467fe75 | 2020-01-16 20:17:45 +0530 | [diff] [blame] | 1221 | if err := agent.updateLogicalDeviceMetersWithoutLock(ctx, metersToUpdate); err != nil { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1222 | logger.Errorw("db-meter-update-failed", log.Fields{"logicalDeviceId": agent.logicalDeviceID}) |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 1223 | return err |
| 1224 | } |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1225 | |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 1226 | } |
| 1227 | if changedFlow { |
| 1228 | var flowMetadata voltha.FlowMetadata |
| 1229 | if err := agent.GetMeterConfig(flowsToDelete, meters, &flowMetadata); err != nil { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1230 | logger.Error("meter-referred-in-flows-not-present") |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 1231 | return err |
| 1232 | } |
khenaidoo | 787224a | 2020-04-16 18:08:47 -0400 | [diff] [blame] | 1233 | var respChnls []coreutils.Response |
| 1234 | var partialRoute bool |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 1235 | deviceRules, err := agent.flowDecomposer.DecomposeRules(ctx, agent, ofp.Flows{Items: flowsToDelete}, ofp.FlowGroups{Items: flowGroups}) |
| 1236 | if err != nil { |
khenaidoo | 787224a | 2020-04-16 18:08:47 -0400 | [diff] [blame] | 1237 | // A no route error means no route exists between the ports specified in the flow. This can happen when the |
| 1238 | // child device is deleted and a request to delete flows from the parent device is received |
| 1239 | if !errors.Is(err, route.ErrNoRoute) { |
| 1240 | logger.Errorw("unexpected-error-received", log.Fields{"flows-to-delete": flowsToDelete, "error": err}) |
| 1241 | return err |
| 1242 | } |
| 1243 | partialRoute = true |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 1244 | } |
khenaidoo | 0458db6 | 2019-06-20 08:50:36 -0400 | [diff] [blame] | 1245 | |
khenaidoo | 787224a | 2020-04-16 18:08:47 -0400 | [diff] [blame] | 1246 | // Update the dB |
npujar | 467fe75 | 2020-01-16 20:17:45 +0530 | [diff] [blame] | 1247 | if err := agent.updateLogicalDeviceFlowsWithoutLock(ctx, &ofp.Flows{Items: flows}); err != nil { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1248 | logger.Errorw("cannot-update-flows", log.Fields{"logicalDeviceId": agent.logicalDeviceID}) |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1249 | return err |
| 1250 | } |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 1251 | |
| 1252 | // Update the devices |
khenaidoo | 787224a | 2020-04-16 18:08:47 -0400 | [diff] [blame] | 1253 | if partialRoute { |
| 1254 | respChnls = agent.deleteFlowsFromParentDevice(ctx, ofp.Flows{Items: flowsToDelete}, &flowMetadata) |
| 1255 | } else { |
| 1256 | respChnls = agent.deleteFlowsAndGroupsFromDevices(ctx, deviceRules, &flowMetadata) |
| 1257 | } |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 1258 | |
| 1259 | // Wait for completion |
| 1260 | go func() { |
| 1261 | if res := coreutils.WaitForNilOrErrorResponses(agent.defaultTimeout, respChnls...); res != nil { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1262 | logger.Warnw("failure-deleting-device-flows", log.Fields{"logicalDeviceId": agent.logicalDeviceID, "errors": res}) |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 1263 | //TODO: Revert flow changes |
| 1264 | } |
| 1265 | }() |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1266 | } |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1267 | return nil |
| 1268 | } |
| 1269 | |
| 1270 | //flowModify modifies a flow from the flow table of that logical device |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 1271 | func (agent *LogicalAgent) flowModify(mod *ofp.OfpFlowMod) error { |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1272 | return errors.New("flowModify not implemented") |
| 1273 | } |
| 1274 | |
| 1275 | //flowModifyStrict deletes a flow from the flow table of that logical device |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 1276 | func (agent *LogicalAgent) flowModifyStrict(mod *ofp.OfpFlowMod) error { |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1277 | return errors.New("flowModifyStrict not implemented") |
| 1278 | } |
| 1279 | |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 1280 | func (agent *LogicalAgent) groupAdd(ctx context.Context, groupMod *ofp.OfpGroupMod) error { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1281 | logger.Debug("groupAdd") |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1282 | if groupMod == nil { |
| 1283 | return nil |
| 1284 | } |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 1285 | if err := agent.requestQueue.WaitForGreenLight(ctx); err != nil { |
| 1286 | return err |
| 1287 | } |
| 1288 | defer agent.requestQueue.RequestComplete() |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1289 | |
khenaidoo | 6e55d9e | 2019-12-12 18:26:26 -0500 | [diff] [blame] | 1290 | lDevice := agent.getLogicalDeviceWithoutLock() |
| 1291 | |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1292 | groups := lDevice.FlowGroups.Items |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1293 | if fu.FindGroup(groups, groupMod.GroupId) == -1 { |
khenaidoo | 68c930b | 2019-05-13 11:46:51 -0400 | [diff] [blame] | 1294 | groups = append(groups, fu.GroupEntryFromGroupMod(groupMod)) |
khenaidoo | 0458db6 | 2019-06-20 08:50:36 -0400 | [diff] [blame] | 1295 | |
Esin Karaman | 2ea5921 | 2019-12-06 11:41:58 +0000 | [diff] [blame] | 1296 | deviceRules := fu.NewDeviceRules() |
| 1297 | deviceRules.CreateEntryIfNotExist(agent.rootDeviceID) |
| 1298 | fg := fu.NewFlowsAndGroups() |
| 1299 | fg.AddGroup(fu.GroupEntryFromGroupMod(groupMod)) |
| 1300 | deviceRules.AddFlowsAndGroup(agent.rootDeviceID, fg) |
| 1301 | |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1302 | logger.Debugw("rules", log.Fields{"rules for group-add": deviceRules.String()}) |
khenaidoo | 0458db6 | 2019-06-20 08:50:36 -0400 | [diff] [blame] | 1303 | |
npujar | 467fe75 | 2020-01-16 20:17:45 +0530 | [diff] [blame] | 1304 | if err := agent.updateLogicalDeviceFlowGroupsWithoutLock(ctx, &ofp.FlowGroups{Items: groups}); err != nil { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1305 | logger.Errorw("cannot-update-group", log.Fields{"logicalDeviceId": agent.logicalDeviceID}) |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1306 | return err |
| 1307 | } |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 1308 | |
| 1309 | // Update the devices |
| 1310 | respChnls := agent.addFlowsAndGroupsToDevices(ctx, deviceRules, &voltha.FlowMetadata{}) |
| 1311 | |
| 1312 | // Wait for completion |
| 1313 | go func() { |
| 1314 | if res := coreutils.WaitForNilOrErrorResponses(agent.defaultTimeout, respChnls...); res != nil { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1315 | logger.Warnw("failure-updating-device-flows-groups", log.Fields{"logicalDeviceId": agent.logicalDeviceID, "errors": res}) |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 1316 | //TODO: Revert flow changes |
| 1317 | } |
| 1318 | }() |
| 1319 | return nil |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1320 | } |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 1321 | return fmt.Errorf("Groups %d already present", groupMod.GroupId) |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1322 | } |
| 1323 | |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 1324 | func (agent *LogicalAgent) groupDelete(ctx context.Context, groupMod *ofp.OfpGroupMod) error { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1325 | logger.Debug("groupDelete") |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1326 | if groupMod == nil { |
| 1327 | return nil |
| 1328 | } |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 1329 | if err := agent.requestQueue.WaitForGreenLight(ctx); err != nil { |
| 1330 | return err |
| 1331 | } |
| 1332 | defer agent.requestQueue.RequestComplete() |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1333 | |
khenaidoo | 6e55d9e | 2019-12-12 18:26:26 -0500 | [diff] [blame] | 1334 | lDevice := agent.getLogicalDeviceWithoutLock() |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1335 | groups := lDevice.FlowGroups.Items |
| 1336 | flows := lDevice.Flows.Items |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 1337 | var groupsChanged bool |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1338 | flowsChanged := false |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 1339 | groupID := groupMod.GroupId |
| 1340 | if groupID == uint32(ofp.OfpGroup_OFPG_ALL) { |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1341 | //TODO we must delete all flows that point to this group and |
| 1342 | //signal controller as requested by flow's flag |
| 1343 | groups = []*ofp.OfpGroupEntry{} |
| 1344 | groupsChanged = true |
| 1345 | } else { |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 1346 | idx := fu.FindGroup(groups, groupID) |
| 1347 | if idx == -1 { |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1348 | return nil // Valid case |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1349 | } |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 1350 | flowsChanged, flows = fu.FlowsDeleteByGroupId(flows, groupID) |
| 1351 | groups = append(groups[:idx], groups[idx+1:]...) |
| 1352 | groupsChanged = true |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1353 | } |
khenaidoo | 0458db6 | 2019-06-20 08:50:36 -0400 | [diff] [blame] | 1354 | if flowsChanged || groupsChanged { |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 1355 | deviceRules, err := agent.flowDecomposer.DecomposeRules(ctx, agent, ofp.Flows{Items: flows}, ofp.FlowGroups{Items: groups}) |
| 1356 | if err != nil { |
| 1357 | return err |
| 1358 | } |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1359 | logger.Debugw("rules", log.Fields{"rules": deviceRules.String()}) |
khenaidoo | 0458db6 | 2019-06-20 08:50:36 -0400 | [diff] [blame] | 1360 | |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 1361 | if groupsChanged { |
| 1362 | if err := agent.updateLogicalDeviceFlowGroupsWithoutLock(ctx, &ofp.FlowGroups{Items: groups}); err != nil { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1363 | logger.Errorw("cannot-update-group", log.Fields{"logicalDeviceId": agent.logicalDeviceID}) |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 1364 | return err |
| 1365 | } |
khenaidoo | 0458db6 | 2019-06-20 08:50:36 -0400 | [diff] [blame] | 1366 | } |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 1367 | if flowsChanged { |
| 1368 | if err := agent.updateLogicalDeviceFlowsWithoutLock(ctx, &ofp.Flows{Items: flows}); err != nil { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1369 | logger.Errorw("cannot-update-flow", log.Fields{"logicalDeviceId": agent.logicalDeviceID}) |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 1370 | return err |
| 1371 | } |
| 1372 | } |
khenaidoo | 0458db6 | 2019-06-20 08:50:36 -0400 | [diff] [blame] | 1373 | |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 1374 | // Update the devices |
| 1375 | respChnls := agent.updateFlowsAndGroupsOfDevice(ctx, deviceRules, nil) |
| 1376 | |
| 1377 | // Wait for completion |
| 1378 | go func() { |
| 1379 | if res := coreutils.WaitForNilOrErrorResponses(agent.defaultTimeout, respChnls...); res != nil { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1380 | logger.Warnw("failure-updating-device-flows-groups", log.Fields{"logicalDeviceId": agent.logicalDeviceID, "errors": res}) |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 1381 | //TODO: Revert flow changes |
| 1382 | } |
| 1383 | }() |
khenaidoo | 43c8212 | 2018-11-22 18:38:28 -0500 | [diff] [blame] | 1384 | } |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1385 | return nil |
| 1386 | } |
| 1387 | |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 1388 | func (agent *LogicalAgent) groupModify(ctx context.Context, groupMod *ofp.OfpGroupMod) error { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1389 | logger.Debug("groupModify") |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1390 | if groupMod == nil { |
| 1391 | return nil |
| 1392 | } |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 1393 | if err := agent.requestQueue.WaitForGreenLight(ctx); err != nil { |
| 1394 | return err |
| 1395 | } |
| 1396 | defer agent.requestQueue.RequestComplete() |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1397 | |
khenaidoo | 6e55d9e | 2019-12-12 18:26:26 -0500 | [diff] [blame] | 1398 | lDevice := agent.getLogicalDeviceWithoutLock() |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1399 | groups := lDevice.FlowGroups.Items |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 1400 | var groupsChanged bool |
| 1401 | groupID := groupMod.GroupId |
| 1402 | idx := fu.FindGroup(groups, groupID) |
| 1403 | if idx == -1 { |
| 1404 | return fmt.Errorf("group-absent:%d", groupID) |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1405 | } |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 1406 | //replace existing group entry with new group definition |
| 1407 | groupEntry := fu.GroupEntryFromGroupMod(groupMod) |
| 1408 | groups[idx] = groupEntry |
| 1409 | groupsChanged = true |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1410 | if groupsChanged { |
Esin Karaman | 2ea5921 | 2019-12-06 11:41:58 +0000 | [diff] [blame] | 1411 | deviceRules := fu.NewDeviceRules() |
| 1412 | deviceRules.CreateEntryIfNotExist(agent.rootDeviceID) |
| 1413 | fg := fu.NewFlowsAndGroups() |
| 1414 | fg.AddGroup(fu.GroupEntryFromGroupMod(groupMod)) |
| 1415 | deviceRules.AddFlowsAndGroup(agent.rootDeviceID, fg) |
khenaidoo | 0458db6 | 2019-06-20 08:50:36 -0400 | [diff] [blame] | 1416 | |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1417 | logger.Debugw("rules", log.Fields{"rules for group-modify": deviceRules.String()}) |
khenaidoo | 0458db6 | 2019-06-20 08:50:36 -0400 | [diff] [blame] | 1418 | |
npujar | 467fe75 | 2020-01-16 20:17:45 +0530 | [diff] [blame] | 1419 | if err := agent.updateLogicalDeviceFlowGroupsWithoutLock(ctx, &ofp.FlowGroups{Items: groups}); err != nil { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1420 | logger.Errorw("Cannot-update-logical-group", log.Fields{"logicalDeviceId": agent.logicalDeviceID}) |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1421 | return err |
| 1422 | } |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 1423 | |
| 1424 | // Update the devices |
| 1425 | respChnls := agent.updateFlowsAndGroupsOfDevice(ctx, deviceRules, &voltha.FlowMetadata{}) |
| 1426 | |
| 1427 | // Wait for completion |
| 1428 | go func() { |
| 1429 | if res := coreutils.WaitForNilOrErrorResponses(agent.defaultTimeout, respChnls...); res != nil { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1430 | logger.Warnw("failure-updating-device-flows-groups", log.Fields{"logicalDeviceId": agent.logicalDeviceID, "errors": res}) |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 1431 | //TODO: Revert flow changes |
| 1432 | } |
| 1433 | }() |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1434 | } |
| 1435 | return nil |
| 1436 | } |
| 1437 | |
| 1438 | // deleteLogicalPort removes the logical port |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 1439 | func (agent *LogicalAgent) deleteLogicalPort(ctx context.Context, lPort *voltha.LogicalPort) error { |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 1440 | if err := agent.requestQueue.WaitForGreenLight(ctx); err != nil { |
| 1441 | return err |
| 1442 | } |
| 1443 | defer agent.requestQueue.RequestComplete() |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1444 | |
khenaidoo | 6e55d9e | 2019-12-12 18:26:26 -0500 | [diff] [blame] | 1445 | logicalDevice := agent.getLogicalDeviceWithoutLock() |
| 1446 | |
khenaidoo | 92e62c5 | 2018-10-03 14:02:54 -0400 | [diff] [blame] | 1447 | index := -1 |
khenaidoo | 6e55d9e | 2019-12-12 18:26:26 -0500 | [diff] [blame] | 1448 | for i, logicalPort := range logicalDevice.Ports { |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1449 | if logicalPort.Id == lPort.Id { |
khenaidoo | 92e62c5 | 2018-10-03 14:02:54 -0400 | [diff] [blame] | 1450 | index = i |
| 1451 | break |
| 1452 | } |
| 1453 | } |
| 1454 | if index >= 0 { |
Kent Hagerman | d9cc2e9 | 2019-11-04 13:28:15 -0500 | [diff] [blame] | 1455 | clonedPorts := clonePorts(logicalDevice.Ports) |
| 1456 | if index < len(clonedPorts)-1 { |
| 1457 | copy(clonedPorts[index:], clonedPorts[index+1:]) |
| 1458 | } |
| 1459 | clonedPorts[len(clonedPorts)-1] = nil |
| 1460 | clonedPorts = clonedPorts[:len(clonedPorts)-1] |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1461 | logger.Debugw("logical-port-deleted", log.Fields{"logicalDeviceId": agent.logicalDeviceID}) |
Kent Hagerman | d9cc2e9 | 2019-11-04 13:28:15 -0500 | [diff] [blame] | 1462 | if err := agent.updateLogicalDevicePortsWithoutLock(ctx, logicalDevice, clonedPorts); err != nil { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1463 | logger.Errorw("logical-device-update-failed", log.Fields{"logicalDeviceId": agent.logicalDeviceID}) |
khenaidoo | 0a822f9 | 2019-05-08 15:15:57 -0400 | [diff] [blame] | 1464 | return err |
| 1465 | } |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 1466 | |
| 1467 | // Remove the logical port from cache |
| 1468 | agent.deleteLogicalPortsFromMap([]uint32{lPort.DevicePortNo}) |
| 1469 | |
| 1470 | // Reset the logical device routes |
| 1471 | go func() { |
| 1472 | if err := agent.buildRoutes(context.Background()); err != nil { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1473 | logger.Warnw("device-routes-not-ready", log.Fields{"logicalDeviceId": agent.logicalDeviceID, "error": err}) |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 1474 | } |
| 1475 | }() |
khenaidoo | 92e62c5 | 2018-10-03 14:02:54 -0400 | [diff] [blame] | 1476 | } |
| 1477 | return nil |
khenaidoo | b920354 | 2018-09-17 22:56:37 -0400 | [diff] [blame] | 1478 | } |
| 1479 | |
khenaidoo | 0a822f9 | 2019-05-08 15:15:57 -0400 | [diff] [blame] | 1480 | // deleteLogicalPorts removes the logical ports associated with that deviceId |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 1481 | func (agent *LogicalAgent) deleteLogicalPorts(ctx context.Context, deviceID string) error { |
Andrea Campanella | 09400bd | 2020-04-02 11:58:04 +0200 | [diff] [blame] | 1482 | logger.Debugw("deleting-logical-ports", log.Fields{"device-id": deviceID}) |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 1483 | if err := agent.requestQueue.WaitForGreenLight(ctx); err != nil { |
| 1484 | return err |
| 1485 | } |
| 1486 | defer agent.requestQueue.RequestComplete() |
khenaidoo | 0a822f9 | 2019-05-08 15:15:57 -0400 | [diff] [blame] | 1487 | |
khenaidoo | 6e55d9e | 2019-12-12 18:26:26 -0500 | [diff] [blame] | 1488 | logicalDevice := agent.getLogicalDeviceWithoutLock() |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 1489 | lPortstoKeep := []*voltha.LogicalPort{} |
| 1490 | lPortsNoToDelete := []uint32{} |
khenaidoo | 6e55d9e | 2019-12-12 18:26:26 -0500 | [diff] [blame] | 1491 | for _, logicalPort := range logicalDevice.Ports { |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 1492 | if logicalPort.DeviceId != deviceID { |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 1493 | lPortstoKeep = append(lPortstoKeep, logicalPort) |
| 1494 | } else { |
| 1495 | lPortsNoToDelete = append(lPortsNoToDelete, logicalPort.DevicePortNo) |
khenaidoo | 0a822f9 | 2019-05-08 15:15:57 -0400 | [diff] [blame] | 1496 | } |
| 1497 | } |
Andrea Campanella | 09400bd | 2020-04-02 11:58:04 +0200 | [diff] [blame] | 1498 | logger.Debugw("deleted-logical-ports", log.Fields{"ports": lPortstoKeep}) |
Kent Hagerman | d9cc2e9 | 2019-11-04 13:28:15 -0500 | [diff] [blame] | 1499 | if err := agent.updateLogicalDevicePortsWithoutLock(ctx, logicalDevice, lPortstoKeep); err != nil { |
Andrea Campanella | 09400bd | 2020-04-02 11:58:04 +0200 | [diff] [blame] | 1500 | logger.Errorw("logical-device-update-failed", log.Fields{"logical-device-id": agent.logicalDeviceID}) |
khenaidoo | 0a822f9 | 2019-05-08 15:15:57 -0400 | [diff] [blame] | 1501 | return err |
| 1502 | } |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 1503 | // Remove the port from the cached logical ports set |
| 1504 | agent.deleteLogicalPortsFromMap(lPortsNoToDelete) |
| 1505 | |
| 1506 | // Reset the logical device routes |
| 1507 | go func() { |
| 1508 | if err := agent.buildRoutes(context.Background()); err != nil { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1509 | logger.Warnw("routes-not-ready", log.Fields{"logical-device-id": agent.logicalDeviceID, "error": err}) |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 1510 | } |
| 1511 | }() |
khenaidoo | 0a822f9 | 2019-05-08 15:15:57 -0400 | [diff] [blame] | 1512 | |
| 1513 | return nil |
| 1514 | } |
| 1515 | |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1516 | // enableLogicalPort enables the logical port |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 1517 | func (agent *LogicalAgent) enableLogicalPort(ctx context.Context, lPortID string) error { |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 1518 | if err := agent.requestQueue.WaitForGreenLight(ctx); err != nil { |
| 1519 | return err |
| 1520 | } |
| 1521 | defer agent.requestQueue.RequestComplete() |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1522 | |
khenaidoo | 6e55d9e | 2019-12-12 18:26:26 -0500 | [diff] [blame] | 1523 | logicalDevice := agent.getLogicalDeviceWithoutLock() |
| 1524 | |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1525 | index := -1 |
khenaidoo | 6e55d9e | 2019-12-12 18:26:26 -0500 | [diff] [blame] | 1526 | for i, logicalPort := range logicalDevice.Ports { |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 1527 | if logicalPort.Id == lPortID { |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1528 | index = i |
| 1529 | break |
| 1530 | } |
| 1531 | } |
| 1532 | if index >= 0 { |
Kent Hagerman | d9cc2e9 | 2019-11-04 13:28:15 -0500 | [diff] [blame] | 1533 | clonedPorts := clonePorts(logicalDevice.Ports) |
| 1534 | clonedPorts[index].OfpPort.Config = clonedPorts[index].OfpPort.Config & ^uint32(ofp.OfpPortConfig_OFPPC_PORT_DOWN) |
| 1535 | return agent.updateLogicalDevicePortsWithoutLock(ctx, logicalDevice, clonedPorts) |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1536 | } |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 1537 | return status.Errorf(codes.NotFound, "Port %s on Logical Device %s", lPortID, agent.logicalDeviceID) |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1538 | } |
| 1539 | |
| 1540 | // disableLogicalPort disabled the logical port |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 1541 | func (agent *LogicalAgent) disableLogicalPort(ctx context.Context, lPortID string) error { |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 1542 | if err := agent.requestQueue.WaitForGreenLight(ctx); err != nil { |
| 1543 | return err |
| 1544 | } |
| 1545 | defer agent.requestQueue.RequestComplete() |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1546 | |
| 1547 | // Get the most up to date logical device |
khenaidoo | 6e55d9e | 2019-12-12 18:26:26 -0500 | [diff] [blame] | 1548 | logicalDevice := agent.getLogicalDeviceWithoutLock() |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1549 | index := -1 |
khenaidoo | 6e55d9e | 2019-12-12 18:26:26 -0500 | [diff] [blame] | 1550 | for i, logicalPort := range logicalDevice.Ports { |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 1551 | if logicalPort.Id == lPortID { |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1552 | index = i |
| 1553 | break |
| 1554 | } |
| 1555 | } |
| 1556 | if index >= 0 { |
Kent Hagerman | d9cc2e9 | 2019-11-04 13:28:15 -0500 | [diff] [blame] | 1557 | clonedPorts := clonePorts(logicalDevice.Ports) |
| 1558 | clonedPorts[index].OfpPort.Config = (clonedPorts[index].OfpPort.Config & ^uint32(ofp.OfpPortConfig_OFPPC_PORT_DOWN)) | uint32(ofp.OfpPortConfig_OFPPC_PORT_DOWN) |
| 1559 | return agent.updateLogicalDevicePortsWithoutLock(ctx, logicalDevice, clonedPorts) |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1560 | } |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 1561 | return status.Errorf(codes.NotFound, "Port %s on Logical Device %s", lPortID, agent.logicalDeviceID) |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1562 | } |
| 1563 | |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 1564 | func (agent *LogicalAgent) getPreCalculatedRoute(ingress, egress uint32) ([]route.Hop, error) { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1565 | logger.Debugw("ROUTE", log.Fields{"len": len(agent.deviceRoutes.Routes)}) |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 1566 | for routeLink, route := range agent.deviceRoutes.Routes { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1567 | logger.Debugw("ROUTELINKS", log.Fields{"ingress": ingress, "egress": egress, "routelink": routeLink}) |
khenaidoo | 89b0e94 | 2018-10-21 21:11:33 -0400 | [diff] [blame] | 1568 | if ingress == routeLink.Ingress && egress == routeLink.Egress { |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 1569 | return route, nil |
khenaidoo | 89b0e94 | 2018-10-21 21:11:33 -0400 | [diff] [blame] | 1570 | } |
| 1571 | } |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 1572 | return nil, status.Errorf(codes.FailedPrecondition, "no route from:%d to:%d", ingress, egress) |
khenaidoo | 89b0e94 | 2018-10-21 21:11:33 -0400 | [diff] [blame] | 1573 | } |
| 1574 | |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 1575 | // GetRoute returns route |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 1576 | func (agent *LogicalAgent) GetRoute(ctx context.Context, ingressPortNo uint32, egressPortNo uint32) ([]route.Hop, error) { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1577 | logger.Debugw("getting-route", log.Fields{"ingress-port": ingressPortNo, "egress-port": egressPortNo}) |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 1578 | routes := make([]route.Hop, 0) |
khenaidoo | 2c6a099 | 2019-04-29 13:46:56 -0400 | [diff] [blame] | 1579 | |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1580 | // Note: A port value of 0 is equivalent to a nil port |
| 1581 | |
khenaidoo | 89b0e94 | 2018-10-21 21:11:33 -0400 | [diff] [blame] | 1582 | // Consider different possibilities |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1583 | if egressPortNo != 0 && ((egressPortNo & 0x7fffffff) == uint32(ofp.OfpPortNo_OFPP_CONTROLLER)) { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1584 | logger.Debugw("controller-flow", log.Fields{"ingressPortNo": ingressPortNo, "egressPortNo": egressPortNo, "logicalPortsNo": agent.logicalPortsNo}) |
khenaidoo | 2c6a099 | 2019-04-29 13:46:56 -0400 | [diff] [blame] | 1585 | if agent.isNNIPort(ingressPortNo) { |
khenaidoo | 89b0e94 | 2018-10-21 21:11:33 -0400 | [diff] [blame] | 1586 | //This is a trap on the NNI Port |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 1587 | if len(agent.deviceRoutes.Routes) == 0 { |
khenaidoo | 8f47419 | 2019-04-03 17:20:44 -0400 | [diff] [blame] | 1588 | // If there are no routes set (usually when the logical device has only NNI port(s), then just return an |
Humera Kouser | 4ff8901 | 2019-08-25 19:01:51 -0400 | [diff] [blame] | 1589 | // route with same IngressHop and EgressHop |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 1590 | hop := route.Hop{DeviceID: agent.rootDeviceID, Ingress: ingressPortNo, Egress: ingressPortNo} |
khenaidoo | 8f47419 | 2019-04-03 17:20:44 -0400 | [diff] [blame] | 1591 | routes = append(routes, hop) |
| 1592 | routes = append(routes, hop) |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 1593 | return routes, nil |
khenaidoo | 8f47419 | 2019-04-03 17:20:44 -0400 | [diff] [blame] | 1594 | } |
khenaidoo | 89b0e94 | 2018-10-21 21:11:33 -0400 | [diff] [blame] | 1595 | //Return a 'half' route to make the flow decomposer logic happy |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 1596 | for routeLink, path := range agent.deviceRoutes.Routes { |
khenaidoo | 2c6a099 | 2019-04-29 13:46:56 -0400 | [diff] [blame] | 1597 | if agent.isNNIPort(routeLink.Egress) { |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 1598 | routes = append(routes, route.Hop{}) // first hop is set to empty |
| 1599 | routes = append(routes, path[1]) |
| 1600 | return routes, nil |
khenaidoo | 89b0e94 | 2018-10-21 21:11:33 -0400 | [diff] [blame] | 1601 | } |
| 1602 | } |
khenaidoo | 787224a | 2020-04-16 18:08:47 -0400 | [diff] [blame] | 1603 | return nil, fmt.Errorf("no upstream route from:%d to:%d :%w", ingressPortNo, egressPortNo, route.ErrNoRoute) |
khenaidoo | 89b0e94 | 2018-10-21 21:11:33 -0400 | [diff] [blame] | 1604 | } |
| 1605 | //treat it as if the output port is the first NNI of the OLT |
khenaidoo | 2c6a099 | 2019-04-29 13:46:56 -0400 | [diff] [blame] | 1606 | var err error |
| 1607 | if egressPortNo, err = agent.getFirstNNIPort(); err != nil { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1608 | logger.Warnw("no-nni-port", log.Fields{"error": err}) |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 1609 | return nil, err |
khenaidoo | 2c6a099 | 2019-04-29 13:46:56 -0400 | [diff] [blame] | 1610 | } |
khenaidoo | 89b0e94 | 2018-10-21 21:11:33 -0400 | [diff] [blame] | 1611 | } |
| 1612 | //If ingress port is not specified (nil), it may be a wildcarded |
| 1613 | //route if egress port is OFPP_CONTROLLER or a nni logical port, |
| 1614 | //in which case we need to create a half-route where only the egress |
| 1615 | //hop is filled, the first hop is nil |
khenaidoo | 2c6a099 | 2019-04-29 13:46:56 -0400 | [diff] [blame] | 1616 | if ingressPortNo == 0 && agent.isNNIPort(egressPortNo) { |
khenaidoo | 89b0e94 | 2018-10-21 21:11:33 -0400 | [diff] [blame] | 1617 | // We can use the 2nd hop of any upstream route, so just find the first upstream: |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 1618 | for routeLink, path := range agent.deviceRoutes.Routes { |
khenaidoo | 2c6a099 | 2019-04-29 13:46:56 -0400 | [diff] [blame] | 1619 | if agent.isNNIPort(routeLink.Egress) { |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 1620 | routes = append(routes, route.Hop{}) // first hop is set to empty |
| 1621 | routes = append(routes, path[1]) |
| 1622 | return routes, nil |
khenaidoo | 89b0e94 | 2018-10-21 21:11:33 -0400 | [diff] [blame] | 1623 | } |
| 1624 | } |
khenaidoo | 787224a | 2020-04-16 18:08:47 -0400 | [diff] [blame] | 1625 | return nil, fmt.Errorf("no upstream route from:%d to:%d :%w", ingressPortNo, egressPortNo, route.ErrNoRoute) |
khenaidoo | 89b0e94 | 2018-10-21 21:11:33 -0400 | [diff] [blame] | 1626 | } |
| 1627 | //If egress port is not specified (nil), we can also can return a "half" route |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1628 | if egressPortNo == 0 { |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 1629 | for routeLink, path := range agent.deviceRoutes.Routes { |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1630 | if routeLink.Ingress == ingressPortNo { |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 1631 | routes = append(routes, path[0]) |
| 1632 | routes = append(routes, route.Hop{}) |
| 1633 | return routes, nil |
khenaidoo | 89b0e94 | 2018-10-21 21:11:33 -0400 | [diff] [blame] | 1634 | } |
| 1635 | } |
khenaidoo | 787224a | 2020-04-16 18:08:47 -0400 | [diff] [blame] | 1636 | return nil, fmt.Errorf("no downstream route from:%d to:%d :%w", ingressPortNo, egressPortNo, route.ErrNoRoute) |
khenaidoo | 89b0e94 | 2018-10-21 21:11:33 -0400 | [diff] [blame] | 1637 | } |
khenaidoo | 89b0e94 | 2018-10-21 21:11:33 -0400 | [diff] [blame] | 1638 | // Return the pre-calculated route |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1639 | return agent.getPreCalculatedRoute(ingressPortNo, egressPortNo) |
khenaidoo | 89b0e94 | 2018-10-21 21:11:33 -0400 | [diff] [blame] | 1640 | } |
| 1641 | |
khenaidoo | 3d3b8c2 | 2019-05-22 18:10:39 -0400 | [diff] [blame] | 1642 | //GetWildcardInputPorts filters out the logical port number from the set of logical ports on the device and |
| 1643 | //returns their port numbers. This function is invoked only during flow decomposition where the lock on the logical |
| 1644 | //device is already held. Therefore it is safe to retrieve the logical device without lock. |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 1645 | func (agent *LogicalAgent) GetWildcardInputPorts(excludePort ...uint32) []uint32 { |
khenaidoo | 89b0e94 | 2018-10-21 21:11:33 -0400 | [diff] [blame] | 1646 | lPorts := make([]uint32, 0) |
| 1647 | var exclPort uint32 |
| 1648 | if len(excludePort) == 1 { |
| 1649 | exclPort = excludePort[0] |
| 1650 | } |
khenaidoo | 6e55d9e | 2019-12-12 18:26:26 -0500 | [diff] [blame] | 1651 | lDevice := agent.getLogicalDeviceWithoutLock() |
| 1652 | for _, port := range lDevice.Ports { |
| 1653 | if port.OfpPort.PortNo != exclPort { |
| 1654 | lPorts = append(lPorts, port.OfpPort.PortNo) |
khenaidoo | 89b0e94 | 2018-10-21 21:11:33 -0400 | [diff] [blame] | 1655 | } |
| 1656 | } |
| 1657 | return lPorts |
| 1658 | } |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1659 | |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 1660 | // GetDeviceRoutes returns device graph |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 1661 | func (agent *LogicalAgent) GetDeviceRoutes() *route.DeviceRoutes { |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 1662 | return agent.deviceRoutes |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1663 | } |
| 1664 | |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 1665 | //rebuildRoutes rebuilds the device routes |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 1666 | func (agent *LogicalAgent) buildRoutes(ctx context.Context) error { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1667 | logger.Debugf("building-routes", log.Fields{"logical-device-id": agent.logicalDeviceID}) |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 1668 | if err := agent.requestQueue.WaitForGreenLight(ctx); err != nil { |
| 1669 | return err |
| 1670 | } |
| 1671 | defer agent.requestQueue.RequestComplete() |
| 1672 | |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 1673 | if agent.deviceRoutes == nil { |
| 1674 | agent.deviceRoutes = route.NewDeviceRoutes(agent.logicalDeviceID, agent.deviceMgr.GetDevice) |
khenaidoo | 2c6a099 | 2019-04-29 13:46:56 -0400 | [diff] [blame] | 1675 | } |
| 1676 | // Get all the logical ports on that logical device |
khenaidoo | 6e55d9e | 2019-12-12 18:26:26 -0500 | [diff] [blame] | 1677 | lDevice := agent.getLogicalDeviceWithoutLock() |
| 1678 | |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 1679 | if err := agent.deviceRoutes.ComputeRoutes(ctx, lDevice.Ports); err != nil { |
| 1680 | return err |
| 1681 | } |
| 1682 | if err := agent.deviceRoutes.Print(); err != nil { |
| 1683 | return err |
| 1684 | } |
| 1685 | |
khenaidoo | 2c6a099 | 2019-04-29 13:46:56 -0400 | [diff] [blame] | 1686 | return nil |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1687 | } |
| 1688 | |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 1689 | //updateRoutes updates the device routes |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 1690 | func (agent *LogicalAgent) updateRoutes(ctx context.Context, lp *voltha.LogicalPort) error { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1691 | logger.Debugw("updateRoutes", log.Fields{"logicalDeviceId": agent.logicalDeviceID}) |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 1692 | if err := agent.requestQueue.WaitForGreenLight(ctx); err != nil { |
| 1693 | return err |
| 1694 | } |
| 1695 | defer agent.requestQueue.RequestComplete() |
| 1696 | |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 1697 | if agent.deviceRoutes == nil { |
| 1698 | agent.deviceRoutes = route.NewDeviceRoutes(agent.logicalDeviceID, agent.deviceMgr.GetDevice) |
khenaidoo | 910204f | 2019-04-08 17:56:40 -0400 | [diff] [blame] | 1699 | } |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 1700 | if err := agent.deviceRoutes.AddPort(ctx, lp, agent.logicalDevice.Ports); err != nil { |
| 1701 | return err |
khenaidoo | 0a822f9 | 2019-05-08 15:15:57 -0400 | [diff] [blame] | 1702 | } |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 1703 | if err := agent.deviceRoutes.Print(); err != nil { |
| 1704 | return err |
| 1705 | } |
| 1706 | return nil |
khenaidoo | 0a822f9 | 2019-05-08 15:15:57 -0400 | [diff] [blame] | 1707 | } |
| 1708 | |
khenaidoo | fc1314d | 2019-03-14 09:34:21 -0400 | [diff] [blame] | 1709 | // diff go over two lists of logical ports and return what's new, what's changed and what's removed. |
khenaidoo | 910204f | 2019-04-08 17:56:40 -0400 | [diff] [blame] | 1710 | func diff(oldList, newList []*voltha.LogicalPort) (newPorts, changedPorts, deletedPorts []*voltha.LogicalPort) { |
khenaidoo | fc1314d | 2019-03-14 09:34:21 -0400 | [diff] [blame] | 1711 | newPorts = make([]*voltha.LogicalPort, 0) |
| 1712 | changedPorts = make([]*voltha.LogicalPort, 0) |
| 1713 | deletedPorts = make([]*voltha.LogicalPort, 0) |
| 1714 | for _, o := range oldList { |
| 1715 | found := false |
khenaidoo | fc1314d | 2019-03-14 09:34:21 -0400 | [diff] [blame] | 1716 | for _, n := range newList { |
| 1717 | if o.Id == n.Id { |
khenaidoo | fc1314d | 2019-03-14 09:34:21 -0400 | [diff] [blame] | 1718 | found = true |
| 1719 | break |
| 1720 | } |
| 1721 | } |
| 1722 | if !found { |
| 1723 | deletedPorts = append(deletedPorts, o) |
| 1724 | } |
khenaidoo | fc1314d | 2019-03-14 09:34:21 -0400 | [diff] [blame] | 1725 | } |
| 1726 | for _, n := range newList { |
| 1727 | found := false |
khenaidoo | 2bc4828 | 2019-07-16 18:13:46 -0400 | [diff] [blame] | 1728 | changed := false |
khenaidoo | fc1314d | 2019-03-14 09:34:21 -0400 | [diff] [blame] | 1729 | for _, o := range oldList { |
| 1730 | if o.Id == n.Id { |
Mahir Gunyel | 5e98c78 | 2020-04-13 15:08:29 -0700 | [diff] [blame^] | 1731 | changed = !proto.Equal(o, n) |
khenaidoo | fc1314d | 2019-03-14 09:34:21 -0400 | [diff] [blame] | 1732 | found = true |
| 1733 | break |
| 1734 | } |
| 1735 | } |
| 1736 | if !found { |
| 1737 | newPorts = append(newPorts, n) |
| 1738 | } |
khenaidoo | 2bc4828 | 2019-07-16 18:13:46 -0400 | [diff] [blame] | 1739 | if changed { |
| 1740 | changedPorts = append(changedPorts, n) |
| 1741 | } |
khenaidoo | fc1314d | 2019-03-14 09:34:21 -0400 | [diff] [blame] | 1742 | } |
| 1743 | return |
| 1744 | } |
| 1745 | |
| 1746 | // portUpdated is invoked when a port is updated on the logical device. Until |
| 1747 | // the POST_ADD notification is fixed, we will use the logical device to |
| 1748 | // update that data. |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 1749 | func (agent *LogicalAgent) portUpdated(oldPorts, newPorts []*voltha.LogicalPort) interface{} { |
khenaidoo | fc1314d | 2019-03-14 09:34:21 -0400 | [diff] [blame] | 1750 | // Get the difference between the two list |
Kent Hagerman | d9cc2e9 | 2019-11-04 13:28:15 -0500 | [diff] [blame] | 1751 | newPorts, changedPorts, deletedPorts := diff(oldPorts, newPorts) |
khenaidoo | fc1314d | 2019-03-14 09:34:21 -0400 | [diff] [blame] | 1752 | // Send the port change events to the OF controller |
khenaidoo | 2c6a099 | 2019-04-29 13:46:56 -0400 | [diff] [blame] | 1753 | for _, newP := range newPorts { |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 1754 | go agent.ldeviceMgr.eventCallbacks.SendChangeEvent(agent.logicalDeviceID, |
khenaidoo | 2c6a099 | 2019-04-29 13:46:56 -0400 | [diff] [blame] | 1755 | &ofp.OfpPortStatus{Reason: ofp.OfpPortReason_OFPPR_ADD, Desc: newP.OfpPort}) |
khenaidoo | fc1314d | 2019-03-14 09:34:21 -0400 | [diff] [blame] | 1756 | } |
| 1757 | for _, change := range changedPorts { |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 1758 | go agent.ldeviceMgr.eventCallbacks.SendChangeEvent(agent.logicalDeviceID, |
khenaidoo | 910204f | 2019-04-08 17:56:40 -0400 | [diff] [blame] | 1759 | &ofp.OfpPortStatus{Reason: ofp.OfpPortReason_OFPPR_MODIFY, Desc: change.OfpPort}) |
khenaidoo | fc1314d | 2019-03-14 09:34:21 -0400 | [diff] [blame] | 1760 | } |
| 1761 | for _, del := range deletedPorts { |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 1762 | go agent.ldeviceMgr.eventCallbacks.SendChangeEvent(agent.logicalDeviceID, |
khenaidoo | 910204f | 2019-04-08 17:56:40 -0400 | [diff] [blame] | 1763 | &ofp.OfpPortStatus{Reason: ofp.OfpPortReason_OFPPR_DELETE, Desc: del.OfpPort}) |
khenaidoo | fc1314d | 2019-03-14 09:34:21 -0400 | [diff] [blame] | 1764 | } |
| 1765 | |
| 1766 | return nil |
| 1767 | } |
| 1768 | |
khenaidoo | 8f47419 | 2019-04-03 17:20:44 -0400 | [diff] [blame] | 1769 | // addNNILogicalPort adds an NNI port to the logical device. It returns a bool representing whether a port has been |
| 1770 | // added and an eror in case a valid error is encountered. If the port was successfully added it will return |
| 1771 | // (true, nil). If the device is not in the correct state it will return (false, nil) as this is a valid |
| 1772 | // scenario. This also applies to the case where the port was already added. |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 1773 | func (agent *LogicalAgent) addNNILogicalPort(ctx context.Context, device *voltha.Device, port *voltha.Port) (bool, error) { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1774 | logger.Debugw("addNNILogicalPort", log.Fields{"NNI": port}) |
Chaitrashree G S | 7849b32 | 2020-03-29 19:25:49 -0400 | [diff] [blame] | 1775 | |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 1776 | if err := agent.requestQueue.WaitForGreenLight(ctx); err != nil { |
| 1777 | return false, err |
| 1778 | } |
khenaidoo | 1ce37ad | 2019-03-24 22:07:24 -0400 | [diff] [blame] | 1779 | if agent.portExist(device, port) { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1780 | logger.Debugw("port-already-exist", log.Fields{"port": port}) |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 1781 | agent.requestQueue.RequestComplete() |
khenaidoo | 8f47419 | 2019-04-03 17:20:44 -0400 | [diff] [blame] | 1782 | return false, nil |
khenaidoo | 1ce37ad | 2019-03-24 22:07:24 -0400 | [diff] [blame] | 1783 | } |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 1784 | agent.requestQueue.RequestComplete() |
khenaidoo | 1ce37ad | 2019-03-24 22:07:24 -0400 | [diff] [blame] | 1785 | |
khenaidoo | fc1314d | 2019-03-14 09:34:21 -0400 | [diff] [blame] | 1786 | var portCap *ic.PortCapability |
| 1787 | var err error |
| 1788 | // First get the port capability |
npujar | 467fe75 | 2020-01-16 20:17:45 +0530 | [diff] [blame] | 1789 | if portCap, err = agent.deviceMgr.getPortCapability(ctx, device.Id, port.PortNo); err != nil { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1790 | logger.Errorw("error-retrieving-port-capabilities", log.Fields{"error": err}) |
khenaidoo | 8f47419 | 2019-04-03 17:20:44 -0400 | [diff] [blame] | 1791 | return false, err |
khenaidoo | fc1314d | 2019-03-14 09:34:21 -0400 | [diff] [blame] | 1792 | } |
khenaidoo | 1ce37ad | 2019-03-24 22:07:24 -0400 | [diff] [blame] | 1793 | |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 1794 | if err := agent.requestQueue.WaitForGreenLight(ctx); err != nil { |
| 1795 | return false, err |
| 1796 | } |
| 1797 | |
| 1798 | defer agent.requestQueue.RequestComplete() |
khenaidoo | 1ce37ad | 2019-03-24 22:07:24 -0400 | [diff] [blame] | 1799 | // Double check again if this port has been already added since the getPortCapability could have taken a long time |
| 1800 | if agent.portExist(device, port) { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1801 | logger.Debugw("port-already-exist", log.Fields{"port": port}) |
khenaidoo | 8f47419 | 2019-04-03 17:20:44 -0400 | [diff] [blame] | 1802 | return false, nil |
khenaidoo | 1ce37ad | 2019-03-24 22:07:24 -0400 | [diff] [blame] | 1803 | } |
| 1804 | |
khenaidoo | fc1314d | 2019-03-14 09:34:21 -0400 | [diff] [blame] | 1805 | portCap.Port.RootPort = true |
| 1806 | lp := (proto.Clone(portCap.Port)).(*voltha.LogicalPort) |
| 1807 | lp.DeviceId = device.Id |
| 1808 | lp.Id = fmt.Sprintf("nni-%d", port.PortNo) |
| 1809 | lp.OfpPort.PortNo = port.PortNo |
| 1810 | lp.OfpPort.Name = lp.Id |
| 1811 | lp.DevicePortNo = port.PortNo |
| 1812 | |
khenaidoo | 6e55d9e | 2019-12-12 18:26:26 -0500 | [diff] [blame] | 1813 | ld := agent.getLogicalDeviceWithoutLock() |
| 1814 | |
Kent Hagerman | d9cc2e9 | 2019-11-04 13:28:15 -0500 | [diff] [blame] | 1815 | clonedPorts := clonePorts(ld.Ports) |
| 1816 | if clonedPorts == nil { |
| 1817 | clonedPorts = make([]*voltha.LogicalPort, 0) |
khenaidoo | fc1314d | 2019-03-14 09:34:21 -0400 | [diff] [blame] | 1818 | } |
Kent Hagerman | d9cc2e9 | 2019-11-04 13:28:15 -0500 | [diff] [blame] | 1819 | clonedPorts = append(clonedPorts, lp) |
khenaidoo | fc1314d | 2019-03-14 09:34:21 -0400 | [diff] [blame] | 1820 | |
Kent Hagerman | d9cc2e9 | 2019-11-04 13:28:15 -0500 | [diff] [blame] | 1821 | if err = agent.updateLogicalDevicePortsWithoutLock(ctx, ld, clonedPorts); err != nil { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1822 | logger.Errorw("error-updating-logical-device", log.Fields{"error": err}) |
khenaidoo | 8f47419 | 2019-04-03 17:20:44 -0400 | [diff] [blame] | 1823 | return false, err |
khenaidoo | fc1314d | 2019-03-14 09:34:21 -0400 | [diff] [blame] | 1824 | } |
khenaidoo | 910204f | 2019-04-08 17:56:40 -0400 | [diff] [blame] | 1825 | |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 1826 | // Update the device routes with this new logical port |
khenaidoo | 910204f | 2019-04-08 17:56:40 -0400 | [diff] [blame] | 1827 | clonedLP := (proto.Clone(lp)).(*voltha.LogicalPort) |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 1828 | go func() { |
| 1829 | if err := agent.updateRoutes(context.Background(), clonedLP); err != nil { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1830 | logger.Warnw("routes-not-ready", log.Fields{"logical-device-id": agent.logicalDeviceID, "logical-port": lp.OfpPort.PortNo, "error": err}) |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 1831 | } |
| 1832 | }() |
khenaidoo | 910204f | 2019-04-08 17:56:40 -0400 | [diff] [blame] | 1833 | |
khenaidoo | 8f47419 | 2019-04-03 17:20:44 -0400 | [diff] [blame] | 1834 | return true, nil |
khenaidoo | fc1314d | 2019-03-14 09:34:21 -0400 | [diff] [blame] | 1835 | } |
| 1836 | |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 1837 | func (agent *LogicalAgent) portExist(device *voltha.Device, port *voltha.Port) bool { |
khenaidoo | 6e55d9e | 2019-12-12 18:26:26 -0500 | [diff] [blame] | 1838 | ldevice := agent.getLogicalDeviceWithoutLock() |
| 1839 | for _, lPort := range ldevice.Ports { |
| 1840 | if lPort.DeviceId == device.Id && lPort.DevicePortNo == port.PortNo && lPort.Id == port.Label { |
| 1841 | return true |
khenaidoo | fc1314d | 2019-03-14 09:34:21 -0400 | [diff] [blame] | 1842 | } |
| 1843 | } |
| 1844 | return false |
| 1845 | } |
| 1846 | |
khenaidoo | 8f47419 | 2019-04-03 17:20:44 -0400 | [diff] [blame] | 1847 | // addUNILogicalPort adds an UNI port to the logical device. It returns a bool representing whether a port has been |
| 1848 | // added and an eror in case a valid error is encountered. If the port was successfully added it will return |
| 1849 | // (true, nil). If the device is not in the correct state it will return (false, nil) as this is a valid |
| 1850 | // scenario. This also applies to the case where the port was already added. |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 1851 | func (agent *LogicalAgent) addUNILogicalPort(ctx context.Context, childDevice *voltha.Device, port *voltha.Port) (bool, error) { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1852 | logger.Debugw("addUNILogicalPort", log.Fields{"port": port}) |
khenaidoo | 8f47419 | 2019-04-03 17:20:44 -0400 | [diff] [blame] | 1853 | if childDevice.AdminState != voltha.AdminState_ENABLED || childDevice.OperStatus != voltha.OperStatus_ACTIVE { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1854 | logger.Infow("device-not-ready", log.Fields{"deviceId": childDevice.Id, "admin": childDevice.AdminState, "oper": childDevice.OperStatus}) |
khenaidoo | 8f47419 | 2019-04-03 17:20:44 -0400 | [diff] [blame] | 1855 | return false, nil |
khenaidoo | fc1314d | 2019-03-14 09:34:21 -0400 | [diff] [blame] | 1856 | } |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 1857 | if err := agent.requestQueue.WaitForGreenLight(ctx); err != nil { |
| 1858 | return false, err |
| 1859 | } |
| 1860 | |
khenaidoo | 1ce37ad | 2019-03-24 22:07:24 -0400 | [diff] [blame] | 1861 | if agent.portExist(childDevice, port) { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1862 | logger.Debugw("port-already-exist", log.Fields{"port": port}) |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 1863 | agent.requestQueue.RequestComplete() |
khenaidoo | 8f47419 | 2019-04-03 17:20:44 -0400 | [diff] [blame] | 1864 | return false, nil |
khenaidoo | 1ce37ad | 2019-03-24 22:07:24 -0400 | [diff] [blame] | 1865 | } |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 1866 | agent.requestQueue.RequestComplete() |
khenaidoo | fc1314d | 2019-03-14 09:34:21 -0400 | [diff] [blame] | 1867 | var portCap *ic.PortCapability |
| 1868 | var err error |
| 1869 | // First get the port capability |
npujar | 467fe75 | 2020-01-16 20:17:45 +0530 | [diff] [blame] | 1870 | if portCap, err = agent.deviceMgr.getPortCapability(ctx, childDevice.Id, port.PortNo); err != nil { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1871 | logger.Errorw("error-retrieving-port-capabilities", log.Fields{"error": err}) |
khenaidoo | 8f47419 | 2019-04-03 17:20:44 -0400 | [diff] [blame] | 1872 | return false, err |
khenaidoo | fc1314d | 2019-03-14 09:34:21 -0400 | [diff] [blame] | 1873 | } |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 1874 | if err := agent.requestQueue.WaitForGreenLight(ctx); err != nil { |
| 1875 | return false, err |
| 1876 | } |
| 1877 | defer agent.requestQueue.RequestComplete() |
khenaidoo | 1ce37ad | 2019-03-24 22:07:24 -0400 | [diff] [blame] | 1878 | // Double check again if this port has been already added since the getPortCapability could have taken a long time |
| 1879 | if agent.portExist(childDevice, port) { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1880 | logger.Debugw("port-already-exist", log.Fields{"port": port}) |
khenaidoo | 8f47419 | 2019-04-03 17:20:44 -0400 | [diff] [blame] | 1881 | return false, nil |
khenaidoo | 1ce37ad | 2019-03-24 22:07:24 -0400 | [diff] [blame] | 1882 | } |
khenaidoo | fc1314d | 2019-03-14 09:34:21 -0400 | [diff] [blame] | 1883 | // Get stored logical device |
khenaidoo | 6e55d9e | 2019-12-12 18:26:26 -0500 | [diff] [blame] | 1884 | ldevice := agent.getLogicalDeviceWithoutLock() |
| 1885 | |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1886 | logger.Debugw("adding-uni", log.Fields{"deviceId": childDevice.Id}) |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 1887 | portCap.Port.RootPort = false |
| 1888 | portCap.Port.Id = port.Label |
| 1889 | portCap.Port.OfpPort.PortNo = port.PortNo |
| 1890 | portCap.Port.DeviceId = childDevice.Id |
| 1891 | portCap.Port.DevicePortNo = port.PortNo |
Kent Hagerman | d9cc2e9 | 2019-11-04 13:28:15 -0500 | [diff] [blame] | 1892 | clonedPorts := clonePorts(ldevice.Ports) |
| 1893 | if clonedPorts == nil { |
| 1894 | clonedPorts = make([]*voltha.LogicalPort, 0) |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 1895 | } |
Kent Hagerman | d9cc2e9 | 2019-11-04 13:28:15 -0500 | [diff] [blame] | 1896 | clonedPorts = append(clonedPorts, portCap.Port) |
| 1897 | if err := agent.updateLogicalDevicePortsWithoutLock(ctx, ldevice, clonedPorts); err != nil { |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 1898 | return false, err |
| 1899 | } |
| 1900 | // Update the device graph with this new logical port |
| 1901 | clonedLP := (proto.Clone(portCap.Port)).(*voltha.LogicalPort) |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 1902 | |
| 1903 | go func() { |
| 1904 | if err := agent.updateRoutes(context.Background(), clonedLP); err != nil { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1905 | logger.Warn("routes-not-ready", log.Fields{"logical-device-id": agent.logicalDeviceID, "error": err}) |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 1906 | } |
| 1907 | }() |
| 1908 | |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 1909 | return true, nil |
khenaidoo | fc1314d | 2019-03-14 09:34:21 -0400 | [diff] [blame] | 1910 | } |
| 1911 | |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 1912 | func (agent *LogicalAgent) packetOut(ctx context.Context, packet *ofp.OfpPacketOut) { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1913 | logger.Debugw("packet-out", log.Fields{ |
Matteo Scandolo | 360605d | 2019-11-05 18:29:17 -0800 | [diff] [blame] | 1914 | "packet": hex.EncodeToString(packet.Data), |
| 1915 | "inPort": packet.GetInPort(), |
| 1916 | }) |
khenaidoo | 68c930b | 2019-05-13 11:46:51 -0400 | [diff] [blame] | 1917 | outPort := fu.GetPacketOutPort(packet) |
khenaidoo | fdbad6e | 2018-11-06 22:26:38 -0500 | [diff] [blame] | 1918 | //frame := packet.GetData() |
| 1919 | //TODO: Use a channel between the logical agent and the device agent |
npujar | 467fe75 | 2020-01-16 20:17:45 +0530 | [diff] [blame] | 1920 | if err := agent.deviceMgr.packetOut(ctx, agent.rootDeviceID, outPort, packet); err != nil { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1921 | logger.Error("packetout-failed", log.Fields{"logicalDeviceID": agent.rootDeviceID}) |
khenaidoo | ca30132 | 2019-01-09 23:06:32 -0500 | [diff] [blame] | 1922 | } |
khenaidoo | fdbad6e | 2018-11-06 22:26:38 -0500 | [diff] [blame] | 1923 | } |
| 1924 | |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 1925 | func (agent *LogicalAgent) packetIn(port uint32, transactionID string, packet []byte) { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1926 | logger.Debugw("packet-in", log.Fields{ |
Matteo Scandolo | 360605d | 2019-11-05 18:29:17 -0800 | [diff] [blame] | 1927 | "port": port, |
| 1928 | "packet": hex.EncodeToString(packet), |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 1929 | "transactionId": transactionID, |
Matteo Scandolo | 360605d | 2019-11-05 18:29:17 -0800 | [diff] [blame] | 1930 | }) |
khenaidoo | 68c930b | 2019-05-13 11:46:51 -0400 | [diff] [blame] | 1931 | packetIn := fu.MkPacketIn(port, packet) |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 1932 | agent.ldeviceMgr.eventCallbacks.SendPacketIn(agent.logicalDeviceID, transactionID, packetIn) |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1933 | logger.Debugw("sending-packet-in", log.Fields{"packet": hex.EncodeToString(packetIn.Data)}) |
khenaidoo | fdbad6e | 2018-11-06 22:26:38 -0500 | [diff] [blame] | 1934 | } |
khenaidoo | 2c6a099 | 2019-04-29 13:46:56 -0400 | [diff] [blame] | 1935 | |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 1936 | func (agent *LogicalAgent) addLogicalPortToMap(portNo uint32, nniPort bool) { |
khenaidoo | 2c6a099 | 2019-04-29 13:46:56 -0400 | [diff] [blame] | 1937 | agent.lockLogicalPortsNo.Lock() |
| 1938 | defer agent.lockLogicalPortsNo.Unlock() |
| 1939 | if exist := agent.logicalPortsNo[portNo]; !exist { |
| 1940 | agent.logicalPortsNo[portNo] = nniPort |
| 1941 | } |
| 1942 | } |
| 1943 | |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 1944 | func (agent *LogicalAgent) deleteLogicalPortsFromMap(portsNo []uint32) { |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 1945 | agent.lockLogicalPortsNo.Lock() |
| 1946 | defer agent.lockLogicalPortsNo.Unlock() |
| 1947 | for _, pNo := range portsNo { |
| 1948 | delete(agent.logicalPortsNo, pNo) |
| 1949 | } |
| 1950 | } |
| 1951 | |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 1952 | func (agent *LogicalAgent) addLogicalPortsToMap(lps []*voltha.LogicalPort) { |
khenaidoo | 3d3b8c2 | 2019-05-22 18:10:39 -0400 | [diff] [blame] | 1953 | agent.lockLogicalPortsNo.Lock() |
| 1954 | defer agent.lockLogicalPortsNo.Unlock() |
| 1955 | for _, lp := range lps { |
| 1956 | if exist := agent.logicalPortsNo[lp.DevicePortNo]; !exist { |
| 1957 | agent.logicalPortsNo[lp.DevicePortNo] = lp.RootPort |
| 1958 | } |
| 1959 | } |
| 1960 | } |
| 1961 | |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 1962 | func (agent *LogicalAgent) isNNIPort(portNo uint32) bool { |
khenaidoo | 2c6a099 | 2019-04-29 13:46:56 -0400 | [diff] [blame] | 1963 | agent.lockLogicalPortsNo.RLock() |
| 1964 | defer agent.lockLogicalPortsNo.RUnlock() |
| 1965 | if exist := agent.logicalPortsNo[portNo]; exist { |
| 1966 | return agent.logicalPortsNo[portNo] |
| 1967 | } |
| 1968 | return false |
| 1969 | } |
| 1970 | |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 1971 | func (agent *LogicalAgent) getFirstNNIPort() (uint32, error) { |
khenaidoo | 2c6a099 | 2019-04-29 13:46:56 -0400 | [diff] [blame] | 1972 | agent.lockLogicalPortsNo.RLock() |
| 1973 | defer agent.lockLogicalPortsNo.RUnlock() |
| 1974 | for portNo, nni := range agent.logicalPortsNo { |
| 1975 | if nni { |
| 1976 | return portNo, nil |
| 1977 | } |
| 1978 | } |
| 1979 | return 0, status.Error(codes.NotFound, "No NNI port found") |
| 1980 | } |
Esin Karaman | 09959ae | 2019-11-29 13:59:58 +0000 | [diff] [blame] | 1981 | |
| 1982 | //GetNNIPorts returns NNI ports. |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 1983 | func (agent *LogicalAgent) GetNNIPorts() []uint32 { |
Esin Karaman | 09959ae | 2019-11-29 13:59:58 +0000 | [diff] [blame] | 1984 | agent.lockLogicalPortsNo.RLock() |
| 1985 | defer agent.lockLogicalPortsNo.RUnlock() |
| 1986 | nniPorts := make([]uint32, 0) |
| 1987 | for portNo, nni := range agent.logicalPortsNo { |
| 1988 | if nni { |
| 1989 | nniPorts = append(nniPorts, portNo) |
| 1990 | } |
| 1991 | } |
| 1992 | return nniPorts |
| 1993 | } |