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 |
Kent Hagerman | 45a13e4 | 2020-04-13 12:23:50 -0400 | [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 | |
Kent Hagerman | d9cc2e9 | 2019-11-04 13:28:15 -0500 | [diff] [blame] | 489 | func clonePorts(ports []*voltha.LogicalPort) []*voltha.LogicalPort { |
| 490 | return proto.Clone(&voltha.LogicalPorts{Items: ports}).(*voltha.LogicalPorts).Items |
| 491 | } |
| 492 | |
khenaidoo | 8b4abbf | 2020-04-24 17:04:30 -0400 | [diff] [blame] | 493 | func cloneFlows(flows []*ofp.OfpFlowStats) []*ofp.OfpFlowStats { |
| 494 | return proto.Clone(&ofp.Flows{Items: flows}).(*ofp.Flows).Items |
| 495 | } |
| 496 | |
| 497 | func cloneMeters(meters []*ofp.OfpMeterEntry) []*ofp.OfpMeterEntry { |
| 498 | return proto.Clone(&ofp.Meters{Items: meters}).(*ofp.Meters).Items |
| 499 | } |
| 500 | |
Kent Hagerman | d9cc2e9 | 2019-11-04 13:28:15 -0500 | [diff] [blame] | 501 | //updateLogicalDevicePortsWithoutLock updates the |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 502 | 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] | 503 | oldPorts := device.Ports |
| 504 | device.Ports = newPorts |
| 505 | if err := agent.updateLogicalDeviceWithoutLock(ctx, device); err != nil { |
| 506 | return err |
| 507 | } |
| 508 | agent.portUpdated(oldPorts, newPorts) |
| 509 | return nil |
| 510 | } |
| 511 | |
khenaidoo | 92e62c5 | 2018-10-03 14:02:54 -0400 | [diff] [blame] | 512 | //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] | 513 | func (agent *LogicalAgent) updateLogicalDeviceWithoutLock(ctx context.Context, logicalDevice *voltha.LogicalDevice) error { |
Kent Hagerman | 4f355f5 | 2020-03-30 16:01:33 -0400 | [diff] [blame] | 514 | if agent.stopped { |
| 515 | return errors.New("logical device agent stopped") |
| 516 | } |
| 517 | |
npujar | 467fe75 | 2020-01-16 20:17:45 +0530 | [diff] [blame] | 518 | updateCtx := context.WithValue(ctx, model.RequestTimestamp, time.Now().UnixNano()) |
Kent Hagerman | 4f355f5 | 2020-03-30 16:01:33 -0400 | [diff] [blame] | 519 | 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] | 520 | 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] | 521 | return err |
| 522 | } |
Kent Hagerman | 4f355f5 | 2020-03-30 16:01:33 -0400 | [diff] [blame] | 523 | |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 524 | agent.logicalDevice = logicalDevice |
| 525 | |
khenaidoo | 92e62c5 | 2018-10-03 14:02:54 -0400 | [diff] [blame] | 526 | return nil |
| 527 | } |
| 528 | |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 529 | //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] | 530 | //that device graph was generated. |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 531 | func (agent *LogicalAgent) generateDeviceRoutesIfNeeded(ctx context.Context) error { |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 532 | agent.lockDeviceRoutes.Lock() |
| 533 | defer agent.lockDeviceRoutes.Unlock() |
| 534 | |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 535 | ld, err := agent.GetLogicalDevice(ctx) |
| 536 | if err != nil { |
| 537 | return err |
| 538 | } |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 539 | |
| 540 | if agent.deviceRoutes != nil && agent.deviceRoutes.IsUpToDate(ld) { |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 541 | return nil |
| 542 | } |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 543 | logger.Debug("Generation of device route required") |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 544 | if err := agent.buildRoutes(ctx); err != nil { |
khenaidoo | 787224a | 2020-04-16 18:08:47 -0400 | [diff] [blame] | 545 | // No Route is not an error |
| 546 | if !errors.Is(err, route.ErrNoRoute) { |
| 547 | return err |
| 548 | } |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 549 | } |
khenaidoo | 4c9e559 | 2019-09-09 16:20:41 -0400 | [diff] [blame] | 550 | return nil |
| 551 | } |
| 552 | |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 553 | //updateFlowTable updates the flow table of that logical device |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 554 | func (agent *LogicalAgent) updateFlowTable(ctx context.Context, flow *ofp.OfpFlowMod) error { |
| 555 | logger.Debug("UpdateFlowTable") |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 556 | if flow == nil { |
| 557 | return nil |
| 558 | } |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 559 | if err := agent.generateDeviceRoutesIfNeeded(ctx); err != nil { |
khenaidoo | 4c9e559 | 2019-09-09 16:20:41 -0400 | [diff] [blame] | 560 | return err |
| 561 | } |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 562 | switch flow.GetCommand() { |
| 563 | case ofp.OfpFlowModCommand_OFPFC_ADD: |
npujar | 467fe75 | 2020-01-16 20:17:45 +0530 | [diff] [blame] | 564 | return agent.flowAdd(ctx, flow) |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 565 | case ofp.OfpFlowModCommand_OFPFC_DELETE: |
npujar | 467fe75 | 2020-01-16 20:17:45 +0530 | [diff] [blame] | 566 | return agent.flowDelete(ctx, flow) |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 567 | case ofp.OfpFlowModCommand_OFPFC_DELETE_STRICT: |
npujar | 467fe75 | 2020-01-16 20:17:45 +0530 | [diff] [blame] | 568 | return agent.flowDeleteStrict(ctx, flow) |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 569 | case ofp.OfpFlowModCommand_OFPFC_MODIFY: |
| 570 | return agent.flowModify(flow) |
| 571 | case ofp.OfpFlowModCommand_OFPFC_MODIFY_STRICT: |
| 572 | return agent.flowModifyStrict(flow) |
| 573 | } |
| 574 | return status.Errorf(codes.Internal, |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 575 | "unhandled-command: lDeviceId:%s, command:%s", agent.logicalDeviceID, flow.GetCommand()) |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 576 | } |
| 577 | |
| 578 | //updateGroupTable updates the group table of that logical device |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 579 | func (agent *LogicalAgent) updateGroupTable(ctx context.Context, groupMod *ofp.OfpGroupMod) error { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 580 | logger.Debug("updateGroupTable") |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 581 | if groupMod == nil { |
| 582 | return nil |
| 583 | } |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 584 | if err := agent.generateDeviceRoutesIfNeeded(ctx); err != nil { |
khenaidoo | 4c9e559 | 2019-09-09 16:20:41 -0400 | [diff] [blame] | 585 | return err |
| 586 | } |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 587 | |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 588 | switch groupMod.GetCommand() { |
| 589 | case ofp.OfpGroupModCommand_OFPGC_ADD: |
npujar | 467fe75 | 2020-01-16 20:17:45 +0530 | [diff] [blame] | 590 | return agent.groupAdd(ctx, groupMod) |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 591 | case ofp.OfpGroupModCommand_OFPGC_DELETE: |
npujar | 467fe75 | 2020-01-16 20:17:45 +0530 | [diff] [blame] | 592 | return agent.groupDelete(ctx, groupMod) |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 593 | case ofp.OfpGroupModCommand_OFPGC_MODIFY: |
npujar | 467fe75 | 2020-01-16 20:17:45 +0530 | [diff] [blame] | 594 | return agent.groupModify(ctx, groupMod) |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 595 | } |
| 596 | return status.Errorf(codes.Internal, |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 597 | "unhandled-command: lDeviceId:%s, command:%s", agent.logicalDeviceID, groupMod.GetCommand()) |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 598 | } |
| 599 | |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 600 | // updateMeterTable updates the meter table of that logical device |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 601 | func (agent *LogicalAgent) updateMeterTable(ctx context.Context, meterMod *ofp.OfpMeterMod) error { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 602 | logger.Debug("updateMeterTable") |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 603 | if meterMod == nil { |
| 604 | return nil |
| 605 | } |
| 606 | switch meterMod.GetCommand() { |
| 607 | case ofp.OfpMeterModCommand_OFPMC_ADD: |
npujar | 467fe75 | 2020-01-16 20:17:45 +0530 | [diff] [blame] | 608 | return agent.meterAdd(ctx, meterMod) |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 609 | case ofp.OfpMeterModCommand_OFPMC_DELETE: |
npujar | 467fe75 | 2020-01-16 20:17:45 +0530 | [diff] [blame] | 610 | return agent.meterDelete(ctx, meterMod) |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 611 | case ofp.OfpMeterModCommand_OFPMC_MODIFY: |
npujar | 467fe75 | 2020-01-16 20:17:45 +0530 | [diff] [blame] | 612 | return agent.meterModify(ctx, meterMod) |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 613 | } |
| 614 | return status.Errorf(codes.Internal, |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 615 | "unhandled-command: lDeviceId:%s, command:%s", agent.logicalDeviceID, meterMod.GetCommand()) |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 616 | |
| 617 | } |
| 618 | |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 619 | func (agent *LogicalAgent) meterAdd(ctx context.Context, meterMod *ofp.OfpMeterMod) error { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 620 | logger.Debugw("meterAdd", log.Fields{"metermod": *meterMod}) |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 621 | if meterMod == nil { |
| 622 | return nil |
| 623 | } |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 624 | if err := agent.requestQueue.WaitForGreenLight(ctx); err != nil { |
| 625 | return err |
| 626 | } |
| 627 | defer agent.requestQueue.RequestComplete() |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 628 | logger.Debug("Acquired logical device lock") |
khenaidoo | 6e55d9e | 2019-12-12 18:26:26 -0500 | [diff] [blame] | 629 | lDevice := agent.getLogicalDeviceWithoutLock() |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 630 | |
| 631 | var meters []*ofp.OfpMeterEntry |
| 632 | if lDevice.Meters != nil && lDevice.Meters.Items != nil { |
| 633 | meters = lDevice.Meters.Items |
| 634 | } |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 635 | logger.Debugw("Available meters", log.Fields{"meters": meters}) |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 636 | |
| 637 | for _, meter := range meters { |
| 638 | if meterMod.MeterId == meter.Config.MeterId { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 639 | logger.Infow("Meter-already-exists", log.Fields{"meter": *meterMod}) |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 640 | return nil |
| 641 | } |
| 642 | } |
| 643 | |
| 644 | meterEntry := fu.MeterEntryFromMeterMod(meterMod) |
| 645 | meters = append(meters, meterEntry) |
| 646 | //Update model |
npujar | 467fe75 | 2020-01-16 20:17:45 +0530 | [diff] [blame] | 647 | if err := agent.updateLogicalDeviceMetersWithoutLock(ctx, &ofp.Meters{Items: meters}); err != nil { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 648 | logger.Errorw("db-meter-update-failed", log.Fields{"logicalDeviceId": agent.logicalDeviceID}) |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 649 | return err |
| 650 | } |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 651 | 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] | 652 | return nil |
| 653 | } |
| 654 | |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 655 | func (agent *LogicalAgent) meterDelete(ctx context.Context, meterMod *ofp.OfpMeterMod) error { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 656 | logger.Debug("meterDelete", log.Fields{"meterMod": *meterMod}) |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 657 | if meterMod == nil { |
| 658 | return nil |
| 659 | } |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 660 | if err := agent.requestQueue.WaitForGreenLight(ctx); err != nil { |
| 661 | return err |
| 662 | } |
| 663 | defer agent.requestQueue.RequestComplete() |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 664 | |
khenaidoo | 6e55d9e | 2019-12-12 18:26:26 -0500 | [diff] [blame] | 665 | lDevice := agent.getLogicalDeviceWithoutLock() |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 666 | |
| 667 | var meters []*ofp.OfpMeterEntry |
| 668 | var flows []*ofp.OfpFlowStats |
| 669 | updatedFlows := make([]*ofp.OfpFlowStats, 0) |
| 670 | if lDevice.Meters != nil && lDevice.Meters.Items != nil { |
| 671 | meters = lDevice.Meters.Items |
| 672 | } |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 673 | |
| 674 | changedMeter := false |
| 675 | changedFow := false |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 676 | logger.Debugw("Available meters", log.Fields{"meters": meters}) |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 677 | for index, meter := range meters { |
| 678 | if meterMod.MeterId == meter.Config.MeterId { |
| 679 | flows = lDevice.Flows.Items |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 680 | changedFow, updatedFlows = agent.getUpdatedFlowsAfterDeletebyMeterID(flows, meterMod.MeterId) |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 681 | meters = append(meters[:index], meters[index+1:]...) |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 682 | logger.Debugw("Meter has been deleted", log.Fields{"meter": meter, "index": index}) |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 683 | changedMeter = true |
| 684 | break |
| 685 | } |
| 686 | } |
| 687 | if changedMeter { |
| 688 | //Update model |
| 689 | metersToUpdate := &ofp.Meters{} |
| 690 | if lDevice.Meters != nil { |
| 691 | metersToUpdate = &ofp.Meters{Items: meters} |
| 692 | } |
npujar | 467fe75 | 2020-01-16 20:17:45 +0530 | [diff] [blame] | 693 | if err := agent.updateLogicalDeviceMetersWithoutLock(ctx, metersToUpdate); err != nil { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 694 | logger.Errorw("db-meter-update-failed", log.Fields{"logicalDeviceId": agent.logicalDeviceID}) |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 695 | return err |
| 696 | } |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 697 | 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] | 698 | |
| 699 | } |
| 700 | if changedFow { |
| 701 | //Update model |
npujar | 467fe75 | 2020-01-16 20:17:45 +0530 | [diff] [blame] | 702 | if err := agent.updateLogicalDeviceFlowsWithoutLock(ctx, &ofp.Flows{Items: updatedFlows}); err != nil { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 703 | logger.Errorw("db-flow-update-failed", log.Fields{"logicalDeviceId": agent.logicalDeviceID}) |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 704 | return err |
| 705 | } |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 706 | logger.Debug("Flows-associated-with-meter-deleted-from-DB-successfully", |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 707 | log.Fields{"updated-no-of-flows": len(updatedFlows), "meter": meterMod.MeterId}) |
| 708 | } |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 709 | logger.Debugw("meterDelete success", log.Fields{"meterID": meterMod.MeterId}) |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 710 | return nil |
| 711 | } |
| 712 | |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 713 | func (agent *LogicalAgent) meterModify(ctx context.Context, meterMod *ofp.OfpMeterMod) error { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 714 | logger.Debug("meterModify") |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 715 | if meterMod == nil { |
| 716 | return nil |
| 717 | } |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 718 | if err := agent.requestQueue.WaitForGreenLight(ctx); err != nil { |
| 719 | return err |
| 720 | } |
| 721 | defer agent.requestQueue.RequestComplete() |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 722 | |
khenaidoo | 6e55d9e | 2019-12-12 18:26:26 -0500 | [diff] [blame] | 723 | lDevice := agent.getLogicalDeviceWithoutLock() |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 724 | |
| 725 | var meters []*ofp.OfpMeterEntry |
| 726 | if lDevice.Meters != nil && lDevice.Meters.Items != nil { |
| 727 | meters = lDevice.Meters.Items |
| 728 | } |
| 729 | changedMeter := false |
| 730 | for index, meter := range meters { |
| 731 | if meterMod.MeterId == meter.Config.MeterId { |
| 732 | newmeterEntry := fu.MeterEntryFromMeterMod(meterMod) |
| 733 | newmeterEntry.Stats.FlowCount = meter.Stats.FlowCount |
| 734 | meters[index] = newmeterEntry |
| 735 | changedMeter = true |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 736 | 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] | 737 | break |
| 738 | } |
| 739 | } |
| 740 | if changedMeter { |
| 741 | //Update model |
| 742 | metersToUpdate := &ofp.Meters{} |
| 743 | if lDevice.Meters != nil { |
| 744 | metersToUpdate = &ofp.Meters{Items: meters} |
| 745 | } |
npujar | 467fe75 | 2020-01-16 20:17:45 +0530 | [diff] [blame] | 746 | if err := agent.updateLogicalDeviceMetersWithoutLock(ctx, metersToUpdate); err != nil { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 747 | logger.Errorw("db-meter-update-failed", log.Fields{"logicalDeviceId": agent.logicalDeviceID}) |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 748 | return err |
| 749 | } |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 750 | logger.Debugw("meter-updated-in-DB-successfully", log.Fields{"updated_meters": meters}) |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 751 | return nil |
| 752 | } |
| 753 | |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 754 | logger.Errorw("Meter not found ", log.Fields{"meter": meterMod}) |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 755 | return fmt.Errorf("no-logical-device-present:%d", meterMod.MeterId) |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 756 | |
| 757 | } |
| 758 | |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 759 | func (agent *LogicalAgent) getUpdatedFlowsAfterDeletebyMeterID(flows []*ofp.OfpFlowStats, meterID uint32) (bool, []*ofp.OfpFlowStats) { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 760 | logger.Infow("Delete flows matching meter", log.Fields{"meter": meterID}) |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 761 | changed := false |
| 762 | //updatedFlows := make([]*ofp.OfpFlowStats, 0) |
| 763 | for index := len(flows) - 1; index >= 0; index-- { |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 764 | if mID := fu.GetMeterIdFromFlow(flows[index]); mID != 0 && mID == meterID { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 765 | 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] | 766 | flows = append(flows[:index], flows[index+1:]...) |
| 767 | changed = true |
| 768 | } |
| 769 | } |
| 770 | return changed, flows |
| 771 | } |
| 772 | |
khenaidoo | 8b4abbf | 2020-04-24 17:04:30 -0400 | [diff] [blame] | 773 | func (agent *LogicalAgent) updateFlowCountOfMeterStats(modCommand *ofp.OfpFlowMod, meters []*ofp.OfpMeterEntry, flow *ofp.OfpFlowStats, revertUpdate bool) bool { |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 774 | |
| 775 | flowCommand := modCommand.GetCommand() |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 776 | meterID := fu.GetMeterIdFromFlow(flow) |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 777 | logger.Debugw("Meter-id-in-flow-mod", log.Fields{"meterId": meterID}) |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 778 | if meterID == 0 { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 779 | logger.Debugw("No meter present in the flow", log.Fields{"flow": *flow}) |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 780 | return false |
| 781 | } |
| 782 | if meters == nil { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 783 | logger.Debug("No meters present in logical device") |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 784 | return false |
| 785 | } |
| 786 | changedMeter := false |
| 787 | for _, meter := range meters { |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 788 | if meterID == meter.Config.MeterId { // Found meter in Logicaldevice |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 789 | if flowCommand == ofp.OfpFlowModCommand_OFPFC_ADD { |
khenaidoo | 8b4abbf | 2020-04-24 17:04:30 -0400 | [diff] [blame] | 790 | if revertUpdate { |
| 791 | meter.Stats.FlowCount-- |
| 792 | } else { |
| 793 | meter.Stats.FlowCount++ |
| 794 | } |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 795 | changedMeter = true |
| 796 | } else if flowCommand == ofp.OfpFlowModCommand_OFPFC_DELETE_STRICT { |
khenaidoo | 8b4abbf | 2020-04-24 17:04:30 -0400 | [diff] [blame] | 797 | if revertUpdate { |
| 798 | meter.Stats.FlowCount++ |
| 799 | } else { |
| 800 | meter.Stats.FlowCount-- |
| 801 | } |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 802 | changedMeter = true |
| 803 | } |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 804 | logger.Debugw("Found meter, updated meter flow stats", log.Fields{" meterId": meterID}) |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 805 | break |
| 806 | } |
| 807 | } |
| 808 | return changedMeter |
| 809 | } |
| 810 | |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 811 | //flowAdd adds a flow to the flow table of that logical device |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 812 | func (agent *LogicalAgent) flowAdd(ctx context.Context, mod *ofp.OfpFlowMod) error { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 813 | logger.Debugw("flowAdd", log.Fields{"flow": mod}) |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 814 | if mod == nil { |
| 815 | return nil |
| 816 | } |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 817 | if err := agent.requestQueue.WaitForGreenLight(ctx); err != nil { |
| 818 | return err |
| 819 | } |
| 820 | defer agent.requestQueue.RequestComplete() |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 821 | |
khenaidoo | 6e55d9e | 2019-12-12 18:26:26 -0500 | [diff] [blame] | 822 | lDevice := agent.getLogicalDeviceWithoutLock() |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 823 | |
| 824 | var flows []*ofp.OfpFlowStats |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 825 | var meters []*ofp.OfpMeterEntry |
khenaidoo | 8b4abbf | 2020-04-24 17:04:30 -0400 | [diff] [blame] | 826 | var flowToReplace *ofp.OfpFlowStats |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 827 | var flow *ofp.OfpFlowStats |
Scott Baker | fdea1e3 | 2020-02-21 15:35:41 -0800 | [diff] [blame] | 828 | var err error |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 829 | |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 830 | if lDevice.Flows != nil && lDevice.Flows.Items != nil { |
| 831 | flows = lDevice.Flows.Items |
| 832 | } |
| 833 | |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 834 | if lDevice.Meters != nil && lDevice.Meters.Items != nil { |
| 835 | meters = lDevice.Meters.Items |
| 836 | } |
khenaidoo | 2c6a099 | 2019-04-29 13:46:56 -0400 | [diff] [blame] | 837 | updatedFlows := make([]*ofp.OfpFlowStats, 0) |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 838 | changed := false |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 839 | updated := false |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 840 | checkOverlap := (mod.Flags & uint32(ofp.OfpFlowModFlags_OFPFF_CHECK_OVERLAP)) != 0 |
| 841 | if checkOverlap { |
| 842 | if overlapped := fu.FindOverlappingFlows(flows, mod); len(overlapped) != 0 { |
| 843 | // TODO: should this error be notified other than being logged? |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 844 | logger.Warnw("overlapped-flows", log.Fields{"logicaldeviceId": agent.logicalDeviceID}) |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 845 | } else { |
Matt Jeanneret | 518b5a4 | 2019-10-29 10:30:46 -0400 | [diff] [blame] | 846 | // Add flow |
Scott Baker | fdea1e3 | 2020-02-21 15:35:41 -0800 | [diff] [blame] | 847 | flow, err = fu.FlowStatsEntryFromFlowModMessage(mod) |
| 848 | if err != nil { |
| 849 | return err |
| 850 | } |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 851 | flows = append(flows, flow) |
khenaidoo | 2c6a099 | 2019-04-29 13:46:56 -0400 | [diff] [blame] | 852 | updatedFlows = append(updatedFlows, flow) |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 853 | changed = true |
| 854 | } |
| 855 | } else { |
Scott Baker | fdea1e3 | 2020-02-21 15:35:41 -0800 | [diff] [blame] | 856 | flow, err = fu.FlowStatsEntryFromFlowModMessage(mod) |
| 857 | if err != nil { |
| 858 | return err |
| 859 | } |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 860 | idx := fu.FindFlows(flows, flow) |
| 861 | if idx >= 0 { |
khenaidoo | 8b4abbf | 2020-04-24 17:04:30 -0400 | [diff] [blame] | 862 | flowToReplace = flows[idx] |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 863 | if (mod.Flags & uint32(ofp.OfpFlowModFlags_OFPFF_RESET_COUNTS)) != 0 { |
khenaidoo | 8b4abbf | 2020-04-24 17:04:30 -0400 | [diff] [blame] | 864 | flow.ByteCount = flowToReplace.ByteCount |
| 865 | flow.PacketCount = flowToReplace.PacketCount |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 866 | } |
khenaidoo | 8b4abbf | 2020-04-24 17:04:30 -0400 | [diff] [blame] | 867 | if !proto.Equal(flowToReplace, flow) { |
Matt Jeanneret | 518b5a4 | 2019-10-29 10:30:46 -0400 | [diff] [blame] | 868 | flows[idx] = flow |
| 869 | updatedFlows = append(updatedFlows, flow) |
| 870 | changed = true |
| 871 | updated = true |
| 872 | } |
| 873 | } else { |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 874 | flows = append(flows, flow) |
khenaidoo | 2c6a099 | 2019-04-29 13:46:56 -0400 | [diff] [blame] | 875 | updatedFlows = append(updatedFlows, flow) |
| 876 | changed = true |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 877 | } |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 878 | } |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 879 | logger.Debugw("flowAdd-changed", log.Fields{"changed": changed}) |
khenaidoo | 4c9e559 | 2019-09-09 16:20:41 -0400 | [diff] [blame] | 880 | |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 881 | if changed { |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 882 | var flowMetadata voltha.FlowMetadata |
| 883 | 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] | 884 | logger.Error("Meter-referred-in-flows-not-present") |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 885 | return err |
| 886 | } |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 887 | deviceRules, err := agent.flowDecomposer.DecomposeRules(ctx, agent, ofp.Flows{Items: updatedFlows}, *lDevice.FlowGroups) |
| 888 | if err != nil { |
| 889 | return err |
| 890 | } |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 891 | logger.Debugw("rules", log.Fields{"rules": deviceRules.String()}) |
khenaidoo | 0458db6 | 2019-06-20 08:50:36 -0400 | [diff] [blame] | 892 | |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 893 | // Update model |
npujar | 467fe75 | 2020-01-16 20:17:45 +0530 | [diff] [blame] | 894 | if err := agent.updateLogicalDeviceFlowsWithoutLock(ctx, &ofp.Flows{Items: flows}); err != nil { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 895 | logger.Errorw("db-flow-update-failed", log.Fields{"logicalDeviceId": agent.logicalDeviceID}) |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 896 | return err |
| 897 | } |
khenaidoo | 8b4abbf | 2020-04-24 17:04:30 -0400 | [diff] [blame] | 898 | |
Matt Jeanneret | 518b5a4 | 2019-10-29 10:30:46 -0400 | [diff] [blame] | 899 | if !updated { |
khenaidoo | 8b4abbf | 2020-04-24 17:04:30 -0400 | [diff] [blame] | 900 | changedMeterStats := agent.updateFlowCountOfMeterStats(mod, meters, flow, false) |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 901 | metersToUpdate := &ofp.Meters{} |
| 902 | if lDevice.Meters != nil { |
| 903 | metersToUpdate = &ofp.Meters{Items: meters} |
| 904 | } |
| 905 | if changedMeterStats { |
| 906 | //Update model |
npujar | 467fe75 | 2020-01-16 20:17:45 +0530 | [diff] [blame] | 907 | if err := agent.updateLogicalDeviceMetersWithoutLock(ctx, metersToUpdate); err != nil { |
khenaidoo | 8b4abbf | 2020-04-24 17:04:30 -0400 | [diff] [blame] | 908 | logger.Errorw("db-meter-update-failed", log.Fields{"logicalDeviceId": agent.logicalDeviceID, "error": err}) |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 909 | return err |
| 910 | } |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 911 | 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] | 912 | |
| 913 | } |
| 914 | } |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 915 | // Send the flows to the devices |
| 916 | respChannels := agent.addFlowsAndGroupsToDevices(ctx, deviceRules, &flowMetadata) |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 917 | |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 918 | // Create the go routines to wait |
| 919 | go func() { |
| 920 | // Wait for completion |
| 921 | if res := coreutils.WaitForNilOrErrorResponses(agent.defaultTimeout, respChannels...); res != nil { |
khenaidoo | 8b4abbf | 2020-04-24 17:04:30 -0400 | [diff] [blame] | 922 | logger.Infow("failed-to-add-flows-will-attempt-deletion", log.Fields{"errors": res, "logical-device-id": agent.logicalDeviceID}) |
| 923 | // Revert added flows |
| 924 | if err := agent.revertAddedFlows(context.Background(), mod, flow, flowToReplace, deviceRules, &flowMetadata); err != nil { |
| 925 | logger.Errorw("failure-to-delete-flows-after-failed-addition", log.Fields{"logical-device-id": agent.logicalDeviceID, "error": err}) |
| 926 | } |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 927 | } |
| 928 | }() |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 929 | } |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 930 | return nil |
| 931 | } |
| 932 | |
khenaidoo | 8b4abbf | 2020-04-24 17:04:30 -0400 | [diff] [blame] | 933 | // revertAddedFlows reverts flows after the flowAdd request has failed. All flows corresponding to that flowAdd request |
| 934 | // will be reverted, both from the logical devices and the devices. |
| 935 | func (agent *LogicalAgent) revertAddedFlows(ctx context.Context, mod *ofp.OfpFlowMod, addedFlow *ofp.OfpFlowStats, replacedFlow *ofp.OfpFlowStats, deviceRules *fu.DeviceRules, metadata *voltha.FlowMetadata) error { |
| 936 | logger.Debugw("revertFlowAdd", log.Fields{"added-flow": addedFlow, "replaced-flow": replacedFlow, "device-rules": deviceRules, "metadata": metadata}) |
| 937 | if err := agent.requestQueue.WaitForGreenLight(context.Background()); err != nil { |
| 938 | return err |
| 939 | } |
| 940 | defer agent.requestQueue.RequestComplete() |
| 941 | |
| 942 | lDevice := agent.getLogicalDeviceWithoutLock() |
| 943 | |
| 944 | // Revert flows |
| 945 | clonedFlows := cloneFlows(lDevice.Flows.Items) |
| 946 | idx := fu.FindFlows(clonedFlows, addedFlow) |
| 947 | if idx < 0 { |
| 948 | // Not found - do nothing |
| 949 | log.Debugw("flow-not-found", log.Fields{"added-flow": addedFlow}) |
| 950 | return nil |
| 951 | } |
| 952 | if replacedFlow != nil { |
| 953 | clonedFlows[idx] = replacedFlow |
| 954 | } else { |
| 955 | clonedFlows = deleteFlowWithoutPreservingOrder(clonedFlows, idx) |
| 956 | } |
| 957 | lDevice.Flows = &ofp.Flows{Items: clonedFlows} |
| 958 | |
khenaidoo | a29a471 | 2020-05-05 10:17:17 -0400 | [diff] [blame] | 959 | // Revert meters, if necessary |
| 960 | if lDevice.Meters != nil && len(lDevice.Meters.Items) > 0 { |
| 961 | meters := cloneMeters(lDevice.Meters.Items) |
| 962 | changedMeterStats := agent.updateFlowCountOfMeterStats(mod, meters, addedFlow, true) |
| 963 | if changedMeterStats { |
| 964 | lDevice.Meters = &ofp.Meters{Items: meters} |
| 965 | } |
khenaidoo | 8b4abbf | 2020-04-24 17:04:30 -0400 | [diff] [blame] | 966 | } |
| 967 | |
| 968 | // Update the model |
| 969 | if err := agent.updateLogicalDeviceWithoutLock(ctx, lDevice); err != nil { |
| 970 | logger.Errorw("db-flow-update-failed", log.Fields{"logicalDeviceId": agent.logicalDeviceID, "error": err}) |
| 971 | return err |
| 972 | } |
| 973 | |
| 974 | // Update the devices |
| 975 | respChnls := agent.deleteFlowsAndGroupsFromDevices(ctx, deviceRules, metadata) |
| 976 | |
| 977 | // Wait for the responses |
| 978 | go func() { |
| 979 | // Since this action is taken following an add failure, we may also receive a failure for the revert |
| 980 | if res := coreutils.WaitForNilOrErrorResponses(agent.defaultTimeout, respChnls...); res != nil { |
| 981 | logger.Warnw("failure-reverting-added-flows", log.Fields{"logicalDeviceId": agent.logicalDeviceID, "errors": res}) |
| 982 | } |
| 983 | }() |
| 984 | |
| 985 | return nil |
| 986 | } |
| 987 | |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 988 | // GetMeterConfig returns meter config |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 989 | 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] | 990 | m := make(map[uint32]bool) |
| 991 | for _, flow := range flows { |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 992 | if flowMeterID := fu.GetMeterIdFromFlow(flow); flowMeterID != 0 && !m[flowMeterID] { |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 993 | foundMeter := false |
| 994 | // Meter is present in the flow , Get from logical device |
| 995 | for _, meter := range meters { |
| 996 | if flowMeterID == meter.Config.MeterId { |
| 997 | metadata.Meters = append(metadata.Meters, meter.Config) |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 998 | logger.Debugw("Found meter in logical device", |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 999 | log.Fields{"meterID": flowMeterID, "meter-band": meter.Config}) |
| 1000 | m[flowMeterID] = true |
| 1001 | foundMeter = true |
| 1002 | break |
| 1003 | } |
| 1004 | } |
| 1005 | if !foundMeter { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1006 | logger.Errorw("Meter-referred-by-flow-is-not-found-in-logicaldevice", |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 1007 | log.Fields{"meterID": flowMeterID, "Available-meters": meters, "flow": *flow}) |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 1008 | return errors.New("Meter-referred-by-flow-is-not-found-in-logicaldevice") |
| 1009 | } |
| 1010 | } |
| 1011 | } |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1012 | 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] | 1013 | return nil |
| 1014 | |
| 1015 | } |
| 1016 | |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1017 | //flowDelete deletes a flow from the flow table of that logical device |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 1018 | func (agent *LogicalAgent) flowDelete(ctx context.Context, mod *ofp.OfpFlowMod) error { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1019 | logger.Debug("flowDelete") |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1020 | if mod == nil { |
| 1021 | return nil |
| 1022 | } |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 1023 | if err := agent.requestQueue.WaitForGreenLight(ctx); err != nil { |
| 1024 | return err |
| 1025 | } |
| 1026 | defer agent.requestQueue.RequestComplete() |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1027 | |
khenaidoo | 6e55d9e | 2019-12-12 18:26:26 -0500 | [diff] [blame] | 1028 | lDevice := agent.getLogicalDeviceWithoutLock() |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1029 | |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 1030 | var meters []*ofp.OfpMeterEntry |
| 1031 | var flows []*ofp.OfpFlowStats |
Esin Karaman | fc3b1af | 2020-01-29 16:52:00 +0000 | [diff] [blame] | 1032 | var flowGroups []*ofp.OfpGroupEntry |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 1033 | |
| 1034 | if lDevice.Flows != nil && lDevice.Flows.Items != nil { |
| 1035 | flows = lDevice.Flows.Items |
| 1036 | } |
| 1037 | |
| 1038 | if lDevice.Meters != nil && lDevice.Meters.Items != nil { |
| 1039 | meters = lDevice.Meters.Items |
| 1040 | } |
Esin Karaman | fc3b1af | 2020-01-29 16:52:00 +0000 | [diff] [blame] | 1041 | |
| 1042 | if lDevice.FlowGroups != nil && lDevice.FlowGroups.Items != nil { |
| 1043 | flowGroups = lDevice.FlowGroups.Items |
| 1044 | } |
| 1045 | |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1046 | //build a list of what to keep vs what to delete |
| 1047 | toKeep := make([]*ofp.OfpFlowStats, 0) |
khenaidoo | 0458db6 | 2019-06-20 08:50:36 -0400 | [diff] [blame] | 1048 | toDelete := make([]*ofp.OfpFlowStats, 0) |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1049 | for _, f := range flows { |
khenaidoo | 0458db6 | 2019-06-20 08:50:36 -0400 | [diff] [blame] | 1050 | // Check whether the flow and the flowmod matches |
Scott Baker | fdea1e3 | 2020-02-21 15:35:41 -0800 | [diff] [blame] | 1051 | fs, err := fu.FlowStatsEntryFromFlowModMessage(mod) |
| 1052 | if err != nil { |
| 1053 | return err |
| 1054 | } |
| 1055 | if fu.FlowMatch(f, fs) { |
khenaidoo | 0458db6 | 2019-06-20 08:50:36 -0400 | [diff] [blame] | 1056 | toDelete = append(toDelete, f) |
| 1057 | continue |
| 1058 | } |
| 1059 | // Check wild card match |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1060 | if !fu.FlowMatchesMod(f, mod) { |
| 1061 | toKeep = append(toKeep, f) |
khenaidoo | 0458db6 | 2019-06-20 08:50:36 -0400 | [diff] [blame] | 1062 | } else { |
| 1063 | toDelete = append(toDelete, f) |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1064 | } |
| 1065 | } |
| 1066 | |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1067 | logger.Debugw("flowDelete", log.Fields{"logicalDeviceId": agent.logicalDeviceID, "toKeep": len(toKeep), "toDelete": toDelete}) |
khenaidoo | 0458db6 | 2019-06-20 08:50:36 -0400 | [diff] [blame] | 1068 | |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1069 | //Update flows |
khenaidoo | 0458db6 | 2019-06-20 08:50:36 -0400 | [diff] [blame] | 1070 | if len(toDelete) > 0 { |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 1071 | var flowMetadata voltha.FlowMetadata |
| 1072 | 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] | 1073 | logger.Error("Meter-referred-in-flows-not-present") |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 1074 | return errors.New("Meter-referred-in-flows-not-present") |
| 1075 | } |
khenaidoo | 787224a | 2020-04-16 18:08:47 -0400 | [diff] [blame] | 1076 | |
| 1077 | var respChnls []coreutils.Response |
| 1078 | var partialRoute bool |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 1079 | deviceRules, err := agent.flowDecomposer.DecomposeRules(ctx, agent, ofp.Flows{Items: toDelete}, ofp.FlowGroups{Items: flowGroups}) |
| 1080 | if err != nil { |
khenaidoo | 787224a | 2020-04-16 18:08:47 -0400 | [diff] [blame] | 1081 | // A no route error means no route exists between the ports specified in the flow. This can happen when the |
| 1082 | // child device is deleted and a request to delete flows from the parent device is received |
| 1083 | if !errors.Is(err, route.ErrNoRoute) { |
| 1084 | logger.Errorw("unexpected-error-received", log.Fields{"flows-to-delete": toDelete, "error": err}) |
| 1085 | return err |
| 1086 | } |
| 1087 | partialRoute = true |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 1088 | } |
khenaidoo | 0458db6 | 2019-06-20 08:50:36 -0400 | [diff] [blame] | 1089 | |
khenaidoo | 787224a | 2020-04-16 18:08:47 -0400 | [diff] [blame] | 1090 | // Update the dB |
npujar | 467fe75 | 2020-01-16 20:17:45 +0530 | [diff] [blame] | 1091 | if err := agent.updateLogicalDeviceFlowsWithoutLock(ctx, &ofp.Flows{Items: toKeep}); err != nil { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1092 | logger.Errorw("cannot-update-flows", log.Fields{"logicalDeviceId": agent.logicalDeviceID}) |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1093 | return err |
| 1094 | } |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 1095 | |
| 1096 | // Update the devices |
khenaidoo | 787224a | 2020-04-16 18:08:47 -0400 | [diff] [blame] | 1097 | if partialRoute { |
| 1098 | respChnls = agent.deleteFlowsFromParentDevice(ctx, ofp.Flows{Items: toDelete}, &flowMetadata) |
| 1099 | } else { |
| 1100 | respChnls = agent.deleteFlowsAndGroupsFromDevices(ctx, deviceRules, &flowMetadata) |
| 1101 | } |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 1102 | |
| 1103 | // Wait for the responses |
| 1104 | go func() { |
| 1105 | // Wait for completion |
| 1106 | if res := coreutils.WaitForNilOrErrorResponses(agent.defaultTimeout, respChnls...); res != nil { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1107 | logger.Errorw("failure-updating-device-flows", log.Fields{"logicalDeviceId": agent.logicalDeviceID, "errors": res}) |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 1108 | // TODO: Revert the flow deletion |
| 1109 | } |
| 1110 | }() |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1111 | } |
| 1112 | |
| 1113 | //TODO: send announcement on delete |
| 1114 | return nil |
| 1115 | } |
| 1116 | |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 1117 | 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] | 1118 | 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] | 1119 | |
Kent Hagerman | 8da2f1e | 2019-11-25 17:28:09 -0500 | [diff] [blame] | 1120 | responses := make([]coreutils.Response, 0) |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 1121 | for deviceID, value := range deviceRules.GetRules() { |
Kent Hagerman | 8da2f1e | 2019-11-25 17:28:09 -0500 | [diff] [blame] | 1122 | response := coreutils.NewResponse() |
| 1123 | responses = append(responses, response) |
| 1124 | go func(deviceId string, value *fu.FlowsAndGroups) { |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 1125 | ctx, cancel := context.WithTimeout(context.Background(), agent.defaultTimeout) |
| 1126 | defer cancel() |
npujar | 467fe75 | 2020-01-16 20:17:45 +0530 | [diff] [blame] | 1127 | 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] | 1128 | logger.Errorw("flow-add-failed", log.Fields{"deviceID": deviceId, "error": err}) |
Kent Hagerman | 8da2f1e | 2019-11-25 17:28:09 -0500 | [diff] [blame] | 1129 | response.Error(status.Errorf(codes.Internal, "flow-add-failed: %s", deviceId)) |
khenaidoo | 0458db6 | 2019-06-20 08:50:36 -0400 | [diff] [blame] | 1130 | } |
Kent Hagerman | 8da2f1e | 2019-11-25 17:28:09 -0500 | [diff] [blame] | 1131 | response.Done() |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 1132 | }(deviceID, value) |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1133 | } |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 1134 | // Return responses (an array of channels) for the caller to wait for a response from the far end. |
| 1135 | return responses |
khenaidoo | 0458db6 | 2019-06-20 08:50:36 -0400 | [diff] [blame] | 1136 | } |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1137 | |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 1138 | 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] | 1139 | logger.Debugw("send-delete-flows-to-device-manager", log.Fields{"logicalDeviceID": agent.logicalDeviceID}) |
khenaidoo | 0458db6 | 2019-06-20 08:50:36 -0400 | [diff] [blame] | 1140 | |
Kent Hagerman | 8da2f1e | 2019-11-25 17:28:09 -0500 | [diff] [blame] | 1141 | responses := make([]coreutils.Response, 0) |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 1142 | for deviceID, value := range deviceRules.GetRules() { |
Kent Hagerman | 8da2f1e | 2019-11-25 17:28:09 -0500 | [diff] [blame] | 1143 | response := coreutils.NewResponse() |
| 1144 | responses = append(responses, response) |
| 1145 | go func(deviceId string, value *fu.FlowsAndGroups) { |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 1146 | ctx, cancel := context.WithTimeout(context.Background(), agent.defaultTimeout) |
| 1147 | defer cancel() |
npujar | 467fe75 | 2020-01-16 20:17:45 +0530 | [diff] [blame] | 1148 | 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] | 1149 | logger.Errorw("flow-delete-failed", log.Fields{"deviceID": deviceId, "error": err}) |
Kent Hagerman | 8da2f1e | 2019-11-25 17:28:09 -0500 | [diff] [blame] | 1150 | response.Error(status.Errorf(codes.Internal, "flow-delete-failed: %s", deviceId)) |
khenaidoo | 0458db6 | 2019-06-20 08:50:36 -0400 | [diff] [blame] | 1151 | } |
Kent Hagerman | 8da2f1e | 2019-11-25 17:28:09 -0500 | [diff] [blame] | 1152 | response.Done() |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 1153 | }(deviceID, value) |
khenaidoo | 0458db6 | 2019-06-20 08:50:36 -0400 | [diff] [blame] | 1154 | } |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 1155 | return responses |
khenaidoo | 0458db6 | 2019-06-20 08:50:36 -0400 | [diff] [blame] | 1156 | } |
| 1157 | |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 1158 | 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] | 1159 | logger.Debugw("send-update-flows-to-device-manager", log.Fields{"logicalDeviceID": agent.logicalDeviceID}) |
khenaidoo | 0458db6 | 2019-06-20 08:50:36 -0400 | [diff] [blame] | 1160 | |
Kent Hagerman | 8da2f1e | 2019-11-25 17:28:09 -0500 | [diff] [blame] | 1161 | responses := make([]coreutils.Response, 0) |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 1162 | for deviceID, value := range deviceRules.GetRules() { |
Kent Hagerman | 8da2f1e | 2019-11-25 17:28:09 -0500 | [diff] [blame] | 1163 | response := coreutils.NewResponse() |
| 1164 | responses = append(responses, response) |
| 1165 | go func(deviceId string, value *fu.FlowsAndGroups) { |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 1166 | ctx, cancel := context.WithTimeout(context.Background(), agent.defaultTimeout) |
| 1167 | defer cancel() |
npujar | 467fe75 | 2020-01-16 20:17:45 +0530 | [diff] [blame] | 1168 | 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] | 1169 | logger.Errorw("flow-update-failed", log.Fields{"deviceID": deviceId, "error": err}) |
Kent Hagerman | 8da2f1e | 2019-11-25 17:28:09 -0500 | [diff] [blame] | 1170 | response.Error(status.Errorf(codes.Internal, "flow-update-failed: %s", deviceId)) |
khenaidoo | 0458db6 | 2019-06-20 08:50:36 -0400 | [diff] [blame] | 1171 | } |
Kent Hagerman | 8da2f1e | 2019-11-25 17:28:09 -0500 | [diff] [blame] | 1172 | response.Done() |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 1173 | }(deviceID, value) |
khenaidoo | 0458db6 | 2019-06-20 08:50:36 -0400 | [diff] [blame] | 1174 | } |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 1175 | return responses |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1176 | } |
| 1177 | |
khenaidoo | 787224a | 2020-04-16 18:08:47 -0400 | [diff] [blame] | 1178 | // getUNILogicalPortNo returns the UNI logical port number specified in the flow |
| 1179 | func (agent *LogicalAgent) getUNILogicalPortNo(flow *ofp.OfpFlowStats) (uint32, error) { |
| 1180 | var uniPort uint32 |
| 1181 | inPortNo := fu.GetInPort(flow) |
| 1182 | outPortNo := fu.GetOutPort(flow) |
| 1183 | if agent.isNNIPort(inPortNo) { |
| 1184 | uniPort = outPortNo |
| 1185 | } else if agent.isNNIPort(outPortNo) { |
| 1186 | uniPort = inPortNo |
| 1187 | } |
| 1188 | if uniPort != 0 { |
| 1189 | return uniPort, nil |
| 1190 | } |
| 1191 | return 0, status.Errorf(codes.NotFound, "no-uni-port: %v", flow) |
| 1192 | } |
| 1193 | |
| 1194 | func (agent *LogicalAgent) deleteFlowsFromParentDevice(ctx context.Context, flows ofp.Flows, metadata *voltha.FlowMetadata) []coreutils.Response { |
| 1195 | logger.Debugw("deleting-flows-from-parent-device", log.Fields{"logical-device-id": agent.logicalDeviceID, "flows": flows}) |
| 1196 | responses := make([]coreutils.Response, 0) |
| 1197 | for _, flow := range flows.Items { |
| 1198 | response := coreutils.NewResponse() |
| 1199 | responses = append(responses, response) |
| 1200 | uniPort, err := agent.getUNILogicalPortNo(flow) |
| 1201 | if err != nil { |
| 1202 | logger.Error("no-uni-port-in-flow", log.Fields{"deviceID": agent.rootDeviceID, "flow": flow, "error": err}) |
| 1203 | response.Error(err) |
| 1204 | response.Done() |
| 1205 | continue |
| 1206 | } |
| 1207 | logger.Debugw("uni-port", log.Fields{"flows": flows, "uni-port": uniPort}) |
| 1208 | go func(uniPort uint32, metadata *voltha.FlowMetadata) { |
| 1209 | ctx, cancel := context.WithTimeout(context.Background(), agent.defaultTimeout) |
| 1210 | defer cancel() |
| 1211 | if err := agent.deviceMgr.deleteParentFlows(ctx, agent.rootDeviceID, uniPort, metadata); err != nil { |
| 1212 | logger.Error("flow-delete-failed", log.Fields{"device-id": agent.rootDeviceID, "error": err}) |
| 1213 | response.Error(status.Errorf(codes.Internal, "flow-delete-failed: %s %v", agent.rootDeviceID, err)) |
| 1214 | } |
| 1215 | response.Done() |
| 1216 | }(uniPort, metadata) |
| 1217 | } |
| 1218 | return responses |
| 1219 | } |
| 1220 | |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1221 | //flowDeleteStrict deletes a flow from the flow table of that logical device |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 1222 | func (agent *LogicalAgent) flowDeleteStrict(ctx context.Context, mod *ofp.OfpFlowMod) error { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1223 | logger.Debug("flowDeleteStrict") |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1224 | if mod == nil { |
| 1225 | return nil |
| 1226 | } |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 1227 | if err := agent.requestQueue.WaitForGreenLight(ctx); err != nil { |
| 1228 | return err |
| 1229 | } |
| 1230 | defer agent.requestQueue.RequestComplete() |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1231 | |
khenaidoo | 6e55d9e | 2019-12-12 18:26:26 -0500 | [diff] [blame] | 1232 | lDevice := agent.getLogicalDeviceWithoutLock() |
| 1233 | |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 1234 | var meters []*ofp.OfpMeterEntry |
| 1235 | var flows []*ofp.OfpFlowStats |
Esin Karaman | fc3b1af | 2020-01-29 16:52:00 +0000 | [diff] [blame] | 1236 | var flowGroups []*ofp.OfpGroupEntry |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 1237 | if lDevice.Meters != nil && lDevice.Meters.Items != nil { |
| 1238 | meters = lDevice.Meters.Items |
| 1239 | } |
| 1240 | if lDevice.Flows != nil && lDevice.Flows.Items != nil { |
| 1241 | flows = lDevice.Flows.Items |
| 1242 | } |
Esin Karaman | fc3b1af | 2020-01-29 16:52:00 +0000 | [diff] [blame] | 1243 | if lDevice.FlowGroups != nil && lDevice.FlowGroups.Items != nil { |
| 1244 | flowGroups = lDevice.FlowGroups.Items |
| 1245 | } |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 1246 | |
| 1247 | changedFlow := false |
| 1248 | changedMeter := false |
Scott Baker | fdea1e3 | 2020-02-21 15:35:41 -0800 | [diff] [blame] | 1249 | flow, err := fu.FlowStatsEntryFromFlowModMessage(mod) |
| 1250 | if err != nil { |
| 1251 | return err |
| 1252 | } |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 1253 | flowsToDelete := make([]*ofp.OfpFlowStats, 0) |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1254 | idx := fu.FindFlows(flows, flow) |
| 1255 | if idx >= 0 { |
khenaidoo | 8b4abbf | 2020-04-24 17:04:30 -0400 | [diff] [blame] | 1256 | changedMeter = agent.updateFlowCountOfMeterStats(mod, meters, flows[idx], false) |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 1257 | flowsToDelete = append(flowsToDelete, flows[idx]) |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1258 | flows = append(flows[:idx], flows[idx+1:]...) |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 1259 | changedFlow = true |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1260 | } else { |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 1261 | return fmt.Errorf("Cannot delete flow - %s", flow) |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1262 | } |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 1263 | if changedMeter { |
| 1264 | //Update model |
| 1265 | metersToUpdate := &ofp.Meters{} |
| 1266 | if lDevice.Meters != nil { |
| 1267 | metersToUpdate = &ofp.Meters{Items: meters} |
| 1268 | } |
npujar | 467fe75 | 2020-01-16 20:17:45 +0530 | [diff] [blame] | 1269 | if err := agent.updateLogicalDeviceMetersWithoutLock(ctx, metersToUpdate); err != nil { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1270 | logger.Errorw("db-meter-update-failed", log.Fields{"logicalDeviceId": agent.logicalDeviceID}) |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 1271 | return err |
| 1272 | } |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1273 | |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 1274 | } |
| 1275 | if changedFlow { |
| 1276 | var flowMetadata voltha.FlowMetadata |
| 1277 | if err := agent.GetMeterConfig(flowsToDelete, meters, &flowMetadata); err != nil { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1278 | logger.Error("meter-referred-in-flows-not-present") |
Manikkaraj k | b1a1092 | 2019-07-29 12:10:34 -0400 | [diff] [blame] | 1279 | return err |
| 1280 | } |
khenaidoo | 787224a | 2020-04-16 18:08:47 -0400 | [diff] [blame] | 1281 | var respChnls []coreutils.Response |
| 1282 | var partialRoute bool |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 1283 | deviceRules, err := agent.flowDecomposer.DecomposeRules(ctx, agent, ofp.Flows{Items: flowsToDelete}, ofp.FlowGroups{Items: flowGroups}) |
| 1284 | if err != nil { |
khenaidoo | 787224a | 2020-04-16 18:08:47 -0400 | [diff] [blame] | 1285 | // A no route error means no route exists between the ports specified in the flow. This can happen when the |
| 1286 | // child device is deleted and a request to delete flows from the parent device is received |
| 1287 | if !errors.Is(err, route.ErrNoRoute) { |
| 1288 | logger.Errorw("unexpected-error-received", log.Fields{"flows-to-delete": flowsToDelete, "error": err}) |
| 1289 | return err |
| 1290 | } |
| 1291 | partialRoute = true |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 1292 | } |
khenaidoo | 0458db6 | 2019-06-20 08:50:36 -0400 | [diff] [blame] | 1293 | |
khenaidoo | 787224a | 2020-04-16 18:08:47 -0400 | [diff] [blame] | 1294 | // Update the dB |
npujar | 467fe75 | 2020-01-16 20:17:45 +0530 | [diff] [blame] | 1295 | if err := agent.updateLogicalDeviceFlowsWithoutLock(ctx, &ofp.Flows{Items: flows}); err != nil { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1296 | logger.Errorw("cannot-update-flows", log.Fields{"logicalDeviceId": agent.logicalDeviceID}) |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1297 | return err |
| 1298 | } |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 1299 | |
| 1300 | // Update the devices |
khenaidoo | 787224a | 2020-04-16 18:08:47 -0400 | [diff] [blame] | 1301 | if partialRoute { |
| 1302 | respChnls = agent.deleteFlowsFromParentDevice(ctx, ofp.Flows{Items: flowsToDelete}, &flowMetadata) |
| 1303 | } else { |
| 1304 | respChnls = agent.deleteFlowsAndGroupsFromDevices(ctx, deviceRules, &flowMetadata) |
| 1305 | } |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 1306 | |
| 1307 | // Wait for completion |
| 1308 | go func() { |
| 1309 | if res := coreutils.WaitForNilOrErrorResponses(agent.defaultTimeout, respChnls...); res != nil { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1310 | logger.Warnw("failure-deleting-device-flows", log.Fields{"logicalDeviceId": agent.logicalDeviceID, "errors": res}) |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 1311 | //TODO: Revert flow changes |
| 1312 | } |
| 1313 | }() |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1314 | } |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1315 | return nil |
| 1316 | } |
| 1317 | |
| 1318 | //flowModify modifies a flow from the flow table of that logical device |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 1319 | func (agent *LogicalAgent) flowModify(mod *ofp.OfpFlowMod) error { |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1320 | return errors.New("flowModify not implemented") |
| 1321 | } |
| 1322 | |
| 1323 | //flowModifyStrict deletes a flow from the flow table of that logical device |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 1324 | func (agent *LogicalAgent) flowModifyStrict(mod *ofp.OfpFlowMod) error { |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1325 | return errors.New("flowModifyStrict not implemented") |
| 1326 | } |
| 1327 | |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 1328 | func (agent *LogicalAgent) groupAdd(ctx context.Context, groupMod *ofp.OfpGroupMod) error { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1329 | logger.Debug("groupAdd") |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1330 | if groupMod == nil { |
| 1331 | return nil |
| 1332 | } |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 1333 | if err := agent.requestQueue.WaitForGreenLight(ctx); err != nil { |
| 1334 | return err |
| 1335 | } |
| 1336 | defer agent.requestQueue.RequestComplete() |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1337 | |
khenaidoo | 6e55d9e | 2019-12-12 18:26:26 -0500 | [diff] [blame] | 1338 | lDevice := agent.getLogicalDeviceWithoutLock() |
| 1339 | |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1340 | groups := lDevice.FlowGroups.Items |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1341 | if fu.FindGroup(groups, groupMod.GroupId) == -1 { |
khenaidoo | 68c930b | 2019-05-13 11:46:51 -0400 | [diff] [blame] | 1342 | groups = append(groups, fu.GroupEntryFromGroupMod(groupMod)) |
khenaidoo | 0458db6 | 2019-06-20 08:50:36 -0400 | [diff] [blame] | 1343 | |
Esin Karaman | 2ea5921 | 2019-12-06 11:41:58 +0000 | [diff] [blame] | 1344 | deviceRules := fu.NewDeviceRules() |
| 1345 | deviceRules.CreateEntryIfNotExist(agent.rootDeviceID) |
| 1346 | fg := fu.NewFlowsAndGroups() |
| 1347 | fg.AddGroup(fu.GroupEntryFromGroupMod(groupMod)) |
| 1348 | deviceRules.AddFlowsAndGroup(agent.rootDeviceID, fg) |
| 1349 | |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1350 | logger.Debugw("rules", log.Fields{"rules for group-add": deviceRules.String()}) |
khenaidoo | 0458db6 | 2019-06-20 08:50:36 -0400 | [diff] [blame] | 1351 | |
npujar | 467fe75 | 2020-01-16 20:17:45 +0530 | [diff] [blame] | 1352 | if err := agent.updateLogicalDeviceFlowGroupsWithoutLock(ctx, &ofp.FlowGroups{Items: groups}); err != nil { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1353 | logger.Errorw("cannot-update-group", log.Fields{"logicalDeviceId": agent.logicalDeviceID}) |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1354 | return err |
| 1355 | } |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 1356 | |
| 1357 | // Update the devices |
| 1358 | respChnls := agent.addFlowsAndGroupsToDevices(ctx, deviceRules, &voltha.FlowMetadata{}) |
| 1359 | |
| 1360 | // Wait for completion |
| 1361 | go func() { |
| 1362 | if res := coreutils.WaitForNilOrErrorResponses(agent.defaultTimeout, respChnls...); res != nil { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1363 | 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] | 1364 | //TODO: Revert flow changes |
| 1365 | } |
| 1366 | }() |
| 1367 | return nil |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1368 | } |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 1369 | return fmt.Errorf("Groups %d already present", groupMod.GroupId) |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1370 | } |
| 1371 | |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 1372 | func (agent *LogicalAgent) groupDelete(ctx context.Context, groupMod *ofp.OfpGroupMod) error { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1373 | logger.Debug("groupDelete") |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1374 | if groupMod == nil { |
| 1375 | return nil |
| 1376 | } |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 1377 | if err := agent.requestQueue.WaitForGreenLight(ctx); err != nil { |
| 1378 | return err |
| 1379 | } |
| 1380 | defer agent.requestQueue.RequestComplete() |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1381 | |
khenaidoo | 6e55d9e | 2019-12-12 18:26:26 -0500 | [diff] [blame] | 1382 | lDevice := agent.getLogicalDeviceWithoutLock() |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1383 | groups := lDevice.FlowGroups.Items |
| 1384 | flows := lDevice.Flows.Items |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 1385 | var groupsChanged bool |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1386 | flowsChanged := false |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 1387 | groupID := groupMod.GroupId |
| 1388 | if groupID == uint32(ofp.OfpGroup_OFPG_ALL) { |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1389 | //TODO we must delete all flows that point to this group and |
| 1390 | //signal controller as requested by flow's flag |
| 1391 | groups = []*ofp.OfpGroupEntry{} |
| 1392 | groupsChanged = true |
| 1393 | } else { |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 1394 | idx := fu.FindGroup(groups, groupID) |
| 1395 | if idx == -1 { |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1396 | return nil // Valid case |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1397 | } |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 1398 | flowsChanged, flows = fu.FlowsDeleteByGroupId(flows, groupID) |
| 1399 | groups = append(groups[:idx], groups[idx+1:]...) |
| 1400 | groupsChanged = true |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1401 | } |
khenaidoo | 0458db6 | 2019-06-20 08:50:36 -0400 | [diff] [blame] | 1402 | if flowsChanged || groupsChanged { |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 1403 | deviceRules, err := agent.flowDecomposer.DecomposeRules(ctx, agent, ofp.Flows{Items: flows}, ofp.FlowGroups{Items: groups}) |
| 1404 | if err != nil { |
| 1405 | return err |
| 1406 | } |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1407 | logger.Debugw("rules", log.Fields{"rules": deviceRules.String()}) |
khenaidoo | 0458db6 | 2019-06-20 08:50:36 -0400 | [diff] [blame] | 1408 | |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 1409 | if groupsChanged { |
| 1410 | if err := agent.updateLogicalDeviceFlowGroupsWithoutLock(ctx, &ofp.FlowGroups{Items: groups}); err != nil { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1411 | logger.Errorw("cannot-update-group", log.Fields{"logicalDeviceId": agent.logicalDeviceID}) |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 1412 | return err |
| 1413 | } |
khenaidoo | 0458db6 | 2019-06-20 08:50:36 -0400 | [diff] [blame] | 1414 | } |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 1415 | if flowsChanged { |
| 1416 | if err := agent.updateLogicalDeviceFlowsWithoutLock(ctx, &ofp.Flows{Items: flows}); err != nil { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1417 | logger.Errorw("cannot-update-flow", log.Fields{"logicalDeviceId": agent.logicalDeviceID}) |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 1418 | return err |
| 1419 | } |
| 1420 | } |
khenaidoo | 0458db6 | 2019-06-20 08:50:36 -0400 | [diff] [blame] | 1421 | |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 1422 | // Update the devices |
| 1423 | respChnls := agent.updateFlowsAndGroupsOfDevice(ctx, deviceRules, nil) |
| 1424 | |
| 1425 | // Wait for completion |
| 1426 | go func() { |
| 1427 | if res := coreutils.WaitForNilOrErrorResponses(agent.defaultTimeout, respChnls...); res != nil { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1428 | 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] | 1429 | //TODO: Revert flow changes |
| 1430 | } |
| 1431 | }() |
khenaidoo | 43c8212 | 2018-11-22 18:38:28 -0500 | [diff] [blame] | 1432 | } |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1433 | return nil |
| 1434 | } |
| 1435 | |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 1436 | func (agent *LogicalAgent) groupModify(ctx context.Context, groupMod *ofp.OfpGroupMod) error { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1437 | logger.Debug("groupModify") |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1438 | if groupMod == nil { |
| 1439 | return nil |
| 1440 | } |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 1441 | if err := agent.requestQueue.WaitForGreenLight(ctx); err != nil { |
| 1442 | return err |
| 1443 | } |
| 1444 | defer agent.requestQueue.RequestComplete() |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1445 | |
khenaidoo | 6e55d9e | 2019-12-12 18:26:26 -0500 | [diff] [blame] | 1446 | lDevice := agent.getLogicalDeviceWithoutLock() |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1447 | groups := lDevice.FlowGroups.Items |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 1448 | var groupsChanged bool |
| 1449 | groupID := groupMod.GroupId |
| 1450 | idx := fu.FindGroup(groups, groupID) |
| 1451 | if idx == -1 { |
| 1452 | return fmt.Errorf("group-absent:%d", groupID) |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1453 | } |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 1454 | //replace existing group entry with new group definition |
| 1455 | groupEntry := fu.GroupEntryFromGroupMod(groupMod) |
| 1456 | groups[idx] = groupEntry |
| 1457 | groupsChanged = true |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1458 | if groupsChanged { |
Esin Karaman | 2ea5921 | 2019-12-06 11:41:58 +0000 | [diff] [blame] | 1459 | deviceRules := fu.NewDeviceRules() |
| 1460 | deviceRules.CreateEntryIfNotExist(agent.rootDeviceID) |
| 1461 | fg := fu.NewFlowsAndGroups() |
| 1462 | fg.AddGroup(fu.GroupEntryFromGroupMod(groupMod)) |
| 1463 | deviceRules.AddFlowsAndGroup(agent.rootDeviceID, fg) |
khenaidoo | 0458db6 | 2019-06-20 08:50:36 -0400 | [diff] [blame] | 1464 | |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1465 | logger.Debugw("rules", log.Fields{"rules for group-modify": deviceRules.String()}) |
khenaidoo | 0458db6 | 2019-06-20 08:50:36 -0400 | [diff] [blame] | 1466 | |
npujar | 467fe75 | 2020-01-16 20:17:45 +0530 | [diff] [blame] | 1467 | if err := agent.updateLogicalDeviceFlowGroupsWithoutLock(ctx, &ofp.FlowGroups{Items: groups}); err != nil { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1468 | logger.Errorw("Cannot-update-logical-group", log.Fields{"logicalDeviceId": agent.logicalDeviceID}) |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1469 | return err |
| 1470 | } |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 1471 | |
| 1472 | // Update the devices |
| 1473 | respChnls := agent.updateFlowsAndGroupsOfDevice(ctx, deviceRules, &voltha.FlowMetadata{}) |
| 1474 | |
| 1475 | // Wait for completion |
| 1476 | go func() { |
| 1477 | if res := coreutils.WaitForNilOrErrorResponses(agent.defaultTimeout, respChnls...); res != nil { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1478 | 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] | 1479 | //TODO: Revert flow changes |
| 1480 | } |
| 1481 | }() |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1482 | } |
| 1483 | return nil |
| 1484 | } |
| 1485 | |
| 1486 | // deleteLogicalPort removes the logical port |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 1487 | func (agent *LogicalAgent) deleteLogicalPort(ctx context.Context, lPort *voltha.LogicalPort) error { |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 1488 | if err := agent.requestQueue.WaitForGreenLight(ctx); err != nil { |
| 1489 | return err |
| 1490 | } |
| 1491 | defer agent.requestQueue.RequestComplete() |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1492 | |
khenaidoo | 6e55d9e | 2019-12-12 18:26:26 -0500 | [diff] [blame] | 1493 | logicalDevice := agent.getLogicalDeviceWithoutLock() |
| 1494 | |
khenaidoo | 92e62c5 | 2018-10-03 14:02:54 -0400 | [diff] [blame] | 1495 | index := -1 |
khenaidoo | 6e55d9e | 2019-12-12 18:26:26 -0500 | [diff] [blame] | 1496 | for i, logicalPort := range logicalDevice.Ports { |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1497 | if logicalPort.Id == lPort.Id { |
khenaidoo | 92e62c5 | 2018-10-03 14:02:54 -0400 | [diff] [blame] | 1498 | index = i |
| 1499 | break |
| 1500 | } |
| 1501 | } |
| 1502 | if index >= 0 { |
Kent Hagerman | d9cc2e9 | 2019-11-04 13:28:15 -0500 | [diff] [blame] | 1503 | clonedPorts := clonePorts(logicalDevice.Ports) |
| 1504 | if index < len(clonedPorts)-1 { |
| 1505 | copy(clonedPorts[index:], clonedPorts[index+1:]) |
| 1506 | } |
| 1507 | clonedPorts[len(clonedPorts)-1] = nil |
| 1508 | clonedPorts = clonedPorts[:len(clonedPorts)-1] |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1509 | logger.Debugw("logical-port-deleted", log.Fields{"logicalDeviceId": agent.logicalDeviceID}) |
Kent Hagerman | d9cc2e9 | 2019-11-04 13:28:15 -0500 | [diff] [blame] | 1510 | if err := agent.updateLogicalDevicePortsWithoutLock(ctx, logicalDevice, clonedPorts); err != nil { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1511 | logger.Errorw("logical-device-update-failed", log.Fields{"logicalDeviceId": agent.logicalDeviceID}) |
khenaidoo | 0a822f9 | 2019-05-08 15:15:57 -0400 | [diff] [blame] | 1512 | return err |
| 1513 | } |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 1514 | |
| 1515 | // Remove the logical port from cache |
| 1516 | agent.deleteLogicalPortsFromMap([]uint32{lPort.DevicePortNo}) |
| 1517 | |
| 1518 | // Reset the logical device routes |
| 1519 | go func() { |
| 1520 | if err := agent.buildRoutes(context.Background()); err != nil { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1521 | logger.Warnw("device-routes-not-ready", log.Fields{"logicalDeviceId": agent.logicalDeviceID, "error": err}) |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 1522 | } |
| 1523 | }() |
khenaidoo | 92e62c5 | 2018-10-03 14:02:54 -0400 | [diff] [blame] | 1524 | } |
| 1525 | return nil |
khenaidoo | b920354 | 2018-09-17 22:56:37 -0400 | [diff] [blame] | 1526 | } |
| 1527 | |
khenaidoo | 0a822f9 | 2019-05-08 15:15:57 -0400 | [diff] [blame] | 1528 | // deleteLogicalPorts removes the logical ports associated with that deviceId |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 1529 | func (agent *LogicalAgent) deleteLogicalPorts(ctx context.Context, deviceID string) error { |
Andrea Campanella | 09400bd | 2020-04-02 11:58:04 +0200 | [diff] [blame] | 1530 | logger.Debugw("deleting-logical-ports", log.Fields{"device-id": deviceID}) |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 1531 | if err := agent.requestQueue.WaitForGreenLight(ctx); err != nil { |
| 1532 | return err |
| 1533 | } |
| 1534 | defer agent.requestQueue.RequestComplete() |
khenaidoo | 0a822f9 | 2019-05-08 15:15:57 -0400 | [diff] [blame] | 1535 | |
khenaidoo | 6e55d9e | 2019-12-12 18:26:26 -0500 | [diff] [blame] | 1536 | logicalDevice := agent.getLogicalDeviceWithoutLock() |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 1537 | lPortstoKeep := []*voltha.LogicalPort{} |
| 1538 | lPortsNoToDelete := []uint32{} |
khenaidoo | 6e55d9e | 2019-12-12 18:26:26 -0500 | [diff] [blame] | 1539 | for _, logicalPort := range logicalDevice.Ports { |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 1540 | if logicalPort.DeviceId != deviceID { |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 1541 | lPortstoKeep = append(lPortstoKeep, logicalPort) |
| 1542 | } else { |
| 1543 | lPortsNoToDelete = append(lPortsNoToDelete, logicalPort.DevicePortNo) |
khenaidoo | 0a822f9 | 2019-05-08 15:15:57 -0400 | [diff] [blame] | 1544 | } |
| 1545 | } |
Andrea Campanella | 09400bd | 2020-04-02 11:58:04 +0200 | [diff] [blame] | 1546 | logger.Debugw("deleted-logical-ports", log.Fields{"ports": lPortstoKeep}) |
Kent Hagerman | d9cc2e9 | 2019-11-04 13:28:15 -0500 | [diff] [blame] | 1547 | if err := agent.updateLogicalDevicePortsWithoutLock(ctx, logicalDevice, lPortstoKeep); err != nil { |
Andrea Campanella | 09400bd | 2020-04-02 11:58:04 +0200 | [diff] [blame] | 1548 | logger.Errorw("logical-device-update-failed", log.Fields{"logical-device-id": agent.logicalDeviceID}) |
khenaidoo | 0a822f9 | 2019-05-08 15:15:57 -0400 | [diff] [blame] | 1549 | return err |
| 1550 | } |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 1551 | // Remove the port from the cached logical ports set |
| 1552 | agent.deleteLogicalPortsFromMap(lPortsNoToDelete) |
| 1553 | |
| 1554 | // Reset the logical device routes |
| 1555 | go func() { |
| 1556 | if err := agent.buildRoutes(context.Background()); err != nil { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1557 | 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] | 1558 | } |
| 1559 | }() |
khenaidoo | 0a822f9 | 2019-05-08 15:15:57 -0400 | [diff] [blame] | 1560 | |
| 1561 | return nil |
| 1562 | } |
| 1563 | |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1564 | // enableLogicalPort enables the logical port |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 1565 | func (agent *LogicalAgent) enableLogicalPort(ctx context.Context, lPortID string) error { |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 1566 | if err := agent.requestQueue.WaitForGreenLight(ctx); err != nil { |
| 1567 | return err |
| 1568 | } |
| 1569 | defer agent.requestQueue.RequestComplete() |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1570 | |
khenaidoo | 6e55d9e | 2019-12-12 18:26:26 -0500 | [diff] [blame] | 1571 | logicalDevice := agent.getLogicalDeviceWithoutLock() |
| 1572 | |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1573 | index := -1 |
khenaidoo | 6e55d9e | 2019-12-12 18:26:26 -0500 | [diff] [blame] | 1574 | for i, logicalPort := range logicalDevice.Ports { |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 1575 | if logicalPort.Id == lPortID { |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1576 | index = i |
| 1577 | break |
| 1578 | } |
| 1579 | } |
| 1580 | if index >= 0 { |
Kent Hagerman | d9cc2e9 | 2019-11-04 13:28:15 -0500 | [diff] [blame] | 1581 | clonedPorts := clonePorts(logicalDevice.Ports) |
| 1582 | clonedPorts[index].OfpPort.Config = clonedPorts[index].OfpPort.Config & ^uint32(ofp.OfpPortConfig_OFPPC_PORT_DOWN) |
| 1583 | return agent.updateLogicalDevicePortsWithoutLock(ctx, logicalDevice, clonedPorts) |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1584 | } |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 1585 | 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] | 1586 | } |
| 1587 | |
| 1588 | // disableLogicalPort disabled the logical port |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 1589 | func (agent *LogicalAgent) disableLogicalPort(ctx context.Context, lPortID string) error { |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 1590 | if err := agent.requestQueue.WaitForGreenLight(ctx); err != nil { |
| 1591 | return err |
| 1592 | } |
| 1593 | defer agent.requestQueue.RequestComplete() |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1594 | |
| 1595 | // Get the most up to date logical device |
khenaidoo | 6e55d9e | 2019-12-12 18:26:26 -0500 | [diff] [blame] | 1596 | logicalDevice := agent.getLogicalDeviceWithoutLock() |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1597 | index := -1 |
khenaidoo | 6e55d9e | 2019-12-12 18:26:26 -0500 | [diff] [blame] | 1598 | for i, logicalPort := range logicalDevice.Ports { |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 1599 | if logicalPort.Id == lPortID { |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1600 | index = i |
| 1601 | break |
| 1602 | } |
| 1603 | } |
| 1604 | if index >= 0 { |
Kent Hagerman | d9cc2e9 | 2019-11-04 13:28:15 -0500 | [diff] [blame] | 1605 | clonedPorts := clonePorts(logicalDevice.Ports) |
| 1606 | clonedPorts[index].OfpPort.Config = (clonedPorts[index].OfpPort.Config & ^uint32(ofp.OfpPortConfig_OFPPC_PORT_DOWN)) | uint32(ofp.OfpPortConfig_OFPPC_PORT_DOWN) |
| 1607 | return agent.updateLogicalDevicePortsWithoutLock(ctx, logicalDevice, clonedPorts) |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1608 | } |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 1609 | 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] | 1610 | } |
| 1611 | |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 1612 | func (agent *LogicalAgent) getPreCalculatedRoute(ingress, egress uint32) ([]route.Hop, error) { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1613 | logger.Debugw("ROUTE", log.Fields{"len": len(agent.deviceRoutes.Routes)}) |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 1614 | for routeLink, route := range agent.deviceRoutes.Routes { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1615 | logger.Debugw("ROUTELINKS", log.Fields{"ingress": ingress, "egress": egress, "routelink": routeLink}) |
khenaidoo | 89b0e94 | 2018-10-21 21:11:33 -0400 | [diff] [blame] | 1616 | if ingress == routeLink.Ingress && egress == routeLink.Egress { |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 1617 | return route, nil |
khenaidoo | 89b0e94 | 2018-10-21 21:11:33 -0400 | [diff] [blame] | 1618 | } |
| 1619 | } |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 1620 | 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] | 1621 | } |
| 1622 | |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 1623 | // GetRoute returns route |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 1624 | 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] | 1625 | logger.Debugw("getting-route", log.Fields{"ingress-port": ingressPortNo, "egress-port": egressPortNo}) |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 1626 | routes := make([]route.Hop, 0) |
khenaidoo | 2c6a099 | 2019-04-29 13:46:56 -0400 | [diff] [blame] | 1627 | |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1628 | // Note: A port value of 0 is equivalent to a nil port |
| 1629 | |
khenaidoo | 89b0e94 | 2018-10-21 21:11:33 -0400 | [diff] [blame] | 1630 | // Consider different possibilities |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1631 | if egressPortNo != 0 && ((egressPortNo & 0x7fffffff) == uint32(ofp.OfpPortNo_OFPP_CONTROLLER)) { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1632 | logger.Debugw("controller-flow", log.Fields{"ingressPortNo": ingressPortNo, "egressPortNo": egressPortNo, "logicalPortsNo": agent.logicalPortsNo}) |
khenaidoo | 2c6a099 | 2019-04-29 13:46:56 -0400 | [diff] [blame] | 1633 | if agent.isNNIPort(ingressPortNo) { |
khenaidoo | 89b0e94 | 2018-10-21 21:11:33 -0400 | [diff] [blame] | 1634 | //This is a trap on the NNI Port |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 1635 | if len(agent.deviceRoutes.Routes) == 0 { |
khenaidoo | 8f47419 | 2019-04-03 17:20:44 -0400 | [diff] [blame] | 1636 | // 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] | 1637 | // route with same IngressHop and EgressHop |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 1638 | hop := route.Hop{DeviceID: agent.rootDeviceID, Ingress: ingressPortNo, Egress: ingressPortNo} |
khenaidoo | 8f47419 | 2019-04-03 17:20:44 -0400 | [diff] [blame] | 1639 | routes = append(routes, hop) |
| 1640 | routes = append(routes, hop) |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 1641 | return routes, nil |
khenaidoo | 8f47419 | 2019-04-03 17:20:44 -0400 | [diff] [blame] | 1642 | } |
khenaidoo | 89b0e94 | 2018-10-21 21:11:33 -0400 | [diff] [blame] | 1643 | //Return a 'half' route to make the flow decomposer logic happy |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 1644 | for routeLink, path := range agent.deviceRoutes.Routes { |
khenaidoo | 2c6a099 | 2019-04-29 13:46:56 -0400 | [diff] [blame] | 1645 | if agent.isNNIPort(routeLink.Egress) { |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 1646 | routes = append(routes, route.Hop{}) // first hop is set to empty |
| 1647 | routes = append(routes, path[1]) |
| 1648 | return routes, nil |
khenaidoo | 89b0e94 | 2018-10-21 21:11:33 -0400 | [diff] [blame] | 1649 | } |
| 1650 | } |
khenaidoo | 787224a | 2020-04-16 18:08:47 -0400 | [diff] [blame] | 1651 | 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] | 1652 | } |
| 1653 | //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] | 1654 | var err error |
| 1655 | if egressPortNo, err = agent.getFirstNNIPort(); err != nil { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1656 | logger.Warnw("no-nni-port", log.Fields{"error": err}) |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 1657 | return nil, err |
khenaidoo | 2c6a099 | 2019-04-29 13:46:56 -0400 | [diff] [blame] | 1658 | } |
khenaidoo | 89b0e94 | 2018-10-21 21:11:33 -0400 | [diff] [blame] | 1659 | } |
| 1660 | //If ingress port is not specified (nil), it may be a wildcarded |
| 1661 | //route if egress port is OFPP_CONTROLLER or a nni logical port, |
| 1662 | //in which case we need to create a half-route where only the egress |
| 1663 | //hop is filled, the first hop is nil |
khenaidoo | 2c6a099 | 2019-04-29 13:46:56 -0400 | [diff] [blame] | 1664 | if ingressPortNo == 0 && agent.isNNIPort(egressPortNo) { |
khenaidoo | 89b0e94 | 2018-10-21 21:11:33 -0400 | [diff] [blame] | 1665 | // 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] | 1666 | for routeLink, path := range agent.deviceRoutes.Routes { |
khenaidoo | 2c6a099 | 2019-04-29 13:46:56 -0400 | [diff] [blame] | 1667 | if agent.isNNIPort(routeLink.Egress) { |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 1668 | routes = append(routes, route.Hop{}) // first hop is set to empty |
| 1669 | routes = append(routes, path[1]) |
| 1670 | return routes, nil |
khenaidoo | 89b0e94 | 2018-10-21 21:11:33 -0400 | [diff] [blame] | 1671 | } |
| 1672 | } |
khenaidoo | 787224a | 2020-04-16 18:08:47 -0400 | [diff] [blame] | 1673 | 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] | 1674 | } |
| 1675 | //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] | 1676 | if egressPortNo == 0 { |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 1677 | for routeLink, path := range agent.deviceRoutes.Routes { |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1678 | if routeLink.Ingress == ingressPortNo { |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 1679 | routes = append(routes, path[0]) |
| 1680 | routes = append(routes, route.Hop{}) |
| 1681 | return routes, nil |
khenaidoo | 89b0e94 | 2018-10-21 21:11:33 -0400 | [diff] [blame] | 1682 | } |
| 1683 | } |
khenaidoo | 787224a | 2020-04-16 18:08:47 -0400 | [diff] [blame] | 1684 | 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] | 1685 | } |
khenaidoo | 89b0e94 | 2018-10-21 21:11:33 -0400 | [diff] [blame] | 1686 | // Return the pre-calculated route |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1687 | return agent.getPreCalculatedRoute(ingressPortNo, egressPortNo) |
khenaidoo | 89b0e94 | 2018-10-21 21:11:33 -0400 | [diff] [blame] | 1688 | } |
| 1689 | |
khenaidoo | 3d3b8c2 | 2019-05-22 18:10:39 -0400 | [diff] [blame] | 1690 | //GetWildcardInputPorts filters out the logical port number from the set of logical ports on the device and |
| 1691 | //returns their port numbers. This function is invoked only during flow decomposition where the lock on the logical |
| 1692 | //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] | 1693 | func (agent *LogicalAgent) GetWildcardInputPorts(excludePort ...uint32) []uint32 { |
khenaidoo | 89b0e94 | 2018-10-21 21:11:33 -0400 | [diff] [blame] | 1694 | lPorts := make([]uint32, 0) |
| 1695 | var exclPort uint32 |
| 1696 | if len(excludePort) == 1 { |
| 1697 | exclPort = excludePort[0] |
| 1698 | } |
khenaidoo | 6e55d9e | 2019-12-12 18:26:26 -0500 | [diff] [blame] | 1699 | lDevice := agent.getLogicalDeviceWithoutLock() |
| 1700 | for _, port := range lDevice.Ports { |
| 1701 | if port.OfpPort.PortNo != exclPort { |
| 1702 | lPorts = append(lPorts, port.OfpPort.PortNo) |
khenaidoo | 89b0e94 | 2018-10-21 21:11:33 -0400 | [diff] [blame] | 1703 | } |
| 1704 | } |
| 1705 | return lPorts |
| 1706 | } |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1707 | |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 1708 | // GetDeviceRoutes returns device graph |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 1709 | func (agent *LogicalAgent) GetDeviceRoutes() *route.DeviceRoutes { |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 1710 | return agent.deviceRoutes |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1711 | } |
| 1712 | |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 1713 | //rebuildRoutes rebuilds the device routes |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 1714 | func (agent *LogicalAgent) buildRoutes(ctx context.Context) error { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1715 | logger.Debugf("building-routes", log.Fields{"logical-device-id": agent.logicalDeviceID}) |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 1716 | if err := agent.requestQueue.WaitForGreenLight(ctx); err != nil { |
| 1717 | return err |
| 1718 | } |
| 1719 | defer agent.requestQueue.RequestComplete() |
| 1720 | |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 1721 | if agent.deviceRoutes == nil { |
Kent Hagerman | 45a13e4 | 2020-04-13 12:23:50 -0400 | [diff] [blame] | 1722 | agent.deviceRoutes = route.NewDeviceRoutes(agent.logicalDeviceID, agent.deviceMgr.getDevice) |
khenaidoo | 2c6a099 | 2019-04-29 13:46:56 -0400 | [diff] [blame] | 1723 | } |
| 1724 | // Get all the logical ports on that logical device |
khenaidoo | 6e55d9e | 2019-12-12 18:26:26 -0500 | [diff] [blame] | 1725 | lDevice := agent.getLogicalDeviceWithoutLock() |
| 1726 | |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 1727 | if err := agent.deviceRoutes.ComputeRoutes(ctx, lDevice.Ports); err != nil { |
| 1728 | return err |
| 1729 | } |
| 1730 | if err := agent.deviceRoutes.Print(); err != nil { |
| 1731 | return err |
| 1732 | } |
| 1733 | |
khenaidoo | 2c6a099 | 2019-04-29 13:46:56 -0400 | [diff] [blame] | 1734 | return nil |
khenaidoo | 19d7b63 | 2018-10-30 10:49:50 -0400 | [diff] [blame] | 1735 | } |
| 1736 | |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 1737 | //updateRoutes updates the device routes |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 1738 | func (agent *LogicalAgent) updateRoutes(ctx context.Context, lp *voltha.LogicalPort) error { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1739 | logger.Debugw("updateRoutes", log.Fields{"logicalDeviceId": agent.logicalDeviceID}) |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 1740 | if err := agent.requestQueue.WaitForGreenLight(ctx); err != nil { |
| 1741 | return err |
| 1742 | } |
| 1743 | defer agent.requestQueue.RequestComplete() |
| 1744 | |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 1745 | if agent.deviceRoutes == nil { |
Kent Hagerman | 45a13e4 | 2020-04-13 12:23:50 -0400 | [diff] [blame] | 1746 | agent.deviceRoutes = route.NewDeviceRoutes(agent.logicalDeviceID, agent.deviceMgr.getDevice) |
khenaidoo | 910204f | 2019-04-08 17:56:40 -0400 | [diff] [blame] | 1747 | } |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 1748 | if err := agent.deviceRoutes.AddPort(ctx, lp, agent.logicalDevice.Ports); err != nil { |
| 1749 | return err |
khenaidoo | 0a822f9 | 2019-05-08 15:15:57 -0400 | [diff] [blame] | 1750 | } |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 1751 | if err := agent.deviceRoutes.Print(); err != nil { |
| 1752 | return err |
| 1753 | } |
| 1754 | return nil |
khenaidoo | 0a822f9 | 2019-05-08 15:15:57 -0400 | [diff] [blame] | 1755 | } |
| 1756 | |
khenaidoo | fc1314d | 2019-03-14 09:34:21 -0400 | [diff] [blame] | 1757 | // diff go over two lists of logical ports and return what's new, what's changed and what's removed. |
Kent Hagerman | 8ad2995 | 2020-04-21 11:48:02 -0400 | [diff] [blame] | 1758 | func diff(oldList, newList []*voltha.LogicalPort) (newPorts, changedPorts, deletedPorts map[string]*voltha.LogicalPort) { |
| 1759 | newPorts = make(map[string]*voltha.LogicalPort, len(newList)) |
| 1760 | changedPorts = make(map[string]*voltha.LogicalPort, len(oldList)) |
| 1761 | deletedPorts = make(map[string]*voltha.LogicalPort, len(oldList)) |
| 1762 | |
khenaidoo | fc1314d | 2019-03-14 09:34:21 -0400 | [diff] [blame] | 1763 | for _, n := range newList { |
Kent Hagerman | 8ad2995 | 2020-04-21 11:48:02 -0400 | [diff] [blame] | 1764 | newPorts[n.Id] = n |
| 1765 | } |
| 1766 | |
| 1767 | for _, o := range oldList { |
| 1768 | if n, have := newPorts[o.Id]; have { |
| 1769 | delete(newPorts, o.Id) // not new |
| 1770 | if !proto.Equal(n, o) { |
| 1771 | changedPorts[n.Id] = n // changed |
khenaidoo | fc1314d | 2019-03-14 09:34:21 -0400 | [diff] [blame] | 1772 | } |
Kent Hagerman | 8ad2995 | 2020-04-21 11:48:02 -0400 | [diff] [blame] | 1773 | } else { |
| 1774 | deletedPorts[o.Id] = o // deleted |
khenaidoo | 2bc4828 | 2019-07-16 18:13:46 -0400 | [diff] [blame] | 1775 | } |
khenaidoo | fc1314d | 2019-03-14 09:34:21 -0400 | [diff] [blame] | 1776 | } |
Kent Hagerman | 8ad2995 | 2020-04-21 11:48:02 -0400 | [diff] [blame] | 1777 | |
| 1778 | return newPorts, changedPorts, deletedPorts |
khenaidoo | fc1314d | 2019-03-14 09:34:21 -0400 | [diff] [blame] | 1779 | } |
| 1780 | |
Kent Hagerman | 8ad2995 | 2020-04-21 11:48:02 -0400 | [diff] [blame] | 1781 | // portUpdated is invoked when a port is updated on the logical device |
| 1782 | func (agent *LogicalAgent) portUpdated(prevPorts, currPorts []*voltha.LogicalPort) interface{} { |
khenaidoo | fc1314d | 2019-03-14 09:34:21 -0400 | [diff] [blame] | 1783 | // Get the difference between the two list |
Kent Hagerman | 8ad2995 | 2020-04-21 11:48:02 -0400 | [diff] [blame] | 1784 | newPorts, changedPorts, deletedPorts := diff(prevPorts, currPorts) |
| 1785 | |
khenaidoo | fc1314d | 2019-03-14 09:34:21 -0400 | [diff] [blame] | 1786 | // Send the port change events to the OF controller |
khenaidoo | 2c6a099 | 2019-04-29 13:46:56 -0400 | [diff] [blame] | 1787 | for _, newP := range newPorts { |
Kent Hagerman | 45a13e4 | 2020-04-13 12:23:50 -0400 | [diff] [blame] | 1788 | go agent.ldeviceMgr.SendChangeEvent(agent.logicalDeviceID, |
khenaidoo | 2c6a099 | 2019-04-29 13:46:56 -0400 | [diff] [blame] | 1789 | &ofp.OfpPortStatus{Reason: ofp.OfpPortReason_OFPPR_ADD, Desc: newP.OfpPort}) |
khenaidoo | fc1314d | 2019-03-14 09:34:21 -0400 | [diff] [blame] | 1790 | } |
| 1791 | for _, change := range changedPorts { |
Kent Hagerman | 45a13e4 | 2020-04-13 12:23:50 -0400 | [diff] [blame] | 1792 | go agent.ldeviceMgr.SendChangeEvent(agent.logicalDeviceID, |
khenaidoo | 910204f | 2019-04-08 17:56:40 -0400 | [diff] [blame] | 1793 | &ofp.OfpPortStatus{Reason: ofp.OfpPortReason_OFPPR_MODIFY, Desc: change.OfpPort}) |
khenaidoo | fc1314d | 2019-03-14 09:34:21 -0400 | [diff] [blame] | 1794 | } |
| 1795 | for _, del := range deletedPorts { |
Kent Hagerman | 45a13e4 | 2020-04-13 12:23:50 -0400 | [diff] [blame] | 1796 | go agent.ldeviceMgr.SendChangeEvent(agent.logicalDeviceID, |
khenaidoo | 910204f | 2019-04-08 17:56:40 -0400 | [diff] [blame] | 1797 | &ofp.OfpPortStatus{Reason: ofp.OfpPortReason_OFPPR_DELETE, Desc: del.OfpPort}) |
khenaidoo | fc1314d | 2019-03-14 09:34:21 -0400 | [diff] [blame] | 1798 | } |
| 1799 | |
| 1800 | return nil |
| 1801 | } |
| 1802 | |
khenaidoo | 8f47419 | 2019-04-03 17:20:44 -0400 | [diff] [blame] | 1803 | // addNNILogicalPort adds an NNI port to the logical device. It returns a bool representing whether a port has been |
| 1804 | // added and an eror in case a valid error is encountered. If the port was successfully added it will return |
| 1805 | // (true, nil). If the device is not in the correct state it will return (false, nil) as this is a valid |
| 1806 | // 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] | 1807 | 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] | 1808 | logger.Debugw("addNNILogicalPort", log.Fields{"NNI": port}) |
Chaitrashree G S | 7849b32 | 2020-03-29 19:25:49 -0400 | [diff] [blame] | 1809 | |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 1810 | if err := agent.requestQueue.WaitForGreenLight(ctx); err != nil { |
| 1811 | return false, err |
| 1812 | } |
khenaidoo | 1ce37ad | 2019-03-24 22:07:24 -0400 | [diff] [blame] | 1813 | if agent.portExist(device, port) { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1814 | logger.Debugw("port-already-exist", log.Fields{"port": port}) |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 1815 | agent.requestQueue.RequestComplete() |
khenaidoo | 8f47419 | 2019-04-03 17:20:44 -0400 | [diff] [blame] | 1816 | return false, nil |
khenaidoo | 1ce37ad | 2019-03-24 22:07:24 -0400 | [diff] [blame] | 1817 | } |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 1818 | agent.requestQueue.RequestComplete() |
khenaidoo | 1ce37ad | 2019-03-24 22:07:24 -0400 | [diff] [blame] | 1819 | |
khenaidoo | fc1314d | 2019-03-14 09:34:21 -0400 | [diff] [blame] | 1820 | var portCap *ic.PortCapability |
| 1821 | var err error |
| 1822 | // First get the port capability |
npujar | 467fe75 | 2020-01-16 20:17:45 +0530 | [diff] [blame] | 1823 | 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] | 1824 | logger.Errorw("error-retrieving-port-capabilities", log.Fields{"error": err}) |
khenaidoo | 8f47419 | 2019-04-03 17:20:44 -0400 | [diff] [blame] | 1825 | return false, err |
khenaidoo | fc1314d | 2019-03-14 09:34:21 -0400 | [diff] [blame] | 1826 | } |
khenaidoo | 1ce37ad | 2019-03-24 22:07:24 -0400 | [diff] [blame] | 1827 | |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 1828 | if err := agent.requestQueue.WaitForGreenLight(ctx); err != nil { |
| 1829 | return false, err |
| 1830 | } |
| 1831 | |
| 1832 | defer agent.requestQueue.RequestComplete() |
khenaidoo | 1ce37ad | 2019-03-24 22:07:24 -0400 | [diff] [blame] | 1833 | // Double check again if this port has been already added since the getPortCapability could have taken a long time |
| 1834 | if agent.portExist(device, port) { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1835 | logger.Debugw("port-already-exist", log.Fields{"port": port}) |
khenaidoo | 8f47419 | 2019-04-03 17:20:44 -0400 | [diff] [blame] | 1836 | return false, nil |
khenaidoo | 1ce37ad | 2019-03-24 22:07:24 -0400 | [diff] [blame] | 1837 | } |
| 1838 | |
khenaidoo | fc1314d | 2019-03-14 09:34:21 -0400 | [diff] [blame] | 1839 | portCap.Port.RootPort = true |
| 1840 | lp := (proto.Clone(portCap.Port)).(*voltha.LogicalPort) |
| 1841 | lp.DeviceId = device.Id |
| 1842 | lp.Id = fmt.Sprintf("nni-%d", port.PortNo) |
| 1843 | lp.OfpPort.PortNo = port.PortNo |
| 1844 | lp.OfpPort.Name = lp.Id |
| 1845 | lp.DevicePortNo = port.PortNo |
| 1846 | |
khenaidoo | 6e55d9e | 2019-12-12 18:26:26 -0500 | [diff] [blame] | 1847 | ld := agent.getLogicalDeviceWithoutLock() |
| 1848 | |
Kent Hagerman | d9cc2e9 | 2019-11-04 13:28:15 -0500 | [diff] [blame] | 1849 | clonedPorts := clonePorts(ld.Ports) |
| 1850 | if clonedPorts == nil { |
| 1851 | clonedPorts = make([]*voltha.LogicalPort, 0) |
khenaidoo | fc1314d | 2019-03-14 09:34:21 -0400 | [diff] [blame] | 1852 | } |
Kent Hagerman | d9cc2e9 | 2019-11-04 13:28:15 -0500 | [diff] [blame] | 1853 | clonedPorts = append(clonedPorts, lp) |
khenaidoo | fc1314d | 2019-03-14 09:34:21 -0400 | [diff] [blame] | 1854 | |
Kent Hagerman | d9cc2e9 | 2019-11-04 13:28:15 -0500 | [diff] [blame] | 1855 | if err = agent.updateLogicalDevicePortsWithoutLock(ctx, ld, clonedPorts); err != nil { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1856 | logger.Errorw("error-updating-logical-device", log.Fields{"error": err}) |
khenaidoo | 8f47419 | 2019-04-03 17:20:44 -0400 | [diff] [blame] | 1857 | return false, err |
khenaidoo | fc1314d | 2019-03-14 09:34:21 -0400 | [diff] [blame] | 1858 | } |
khenaidoo | 910204f | 2019-04-08 17:56:40 -0400 | [diff] [blame] | 1859 | |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 1860 | // Update the device routes with this new logical port |
khenaidoo | 910204f | 2019-04-08 17:56:40 -0400 | [diff] [blame] | 1861 | clonedLP := (proto.Clone(lp)).(*voltha.LogicalPort) |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 1862 | go func() { |
| 1863 | if err := agent.updateRoutes(context.Background(), clonedLP); err != nil { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1864 | 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] | 1865 | } |
| 1866 | }() |
khenaidoo | 910204f | 2019-04-08 17:56:40 -0400 | [diff] [blame] | 1867 | |
khenaidoo | 8f47419 | 2019-04-03 17:20:44 -0400 | [diff] [blame] | 1868 | return true, nil |
khenaidoo | fc1314d | 2019-03-14 09:34:21 -0400 | [diff] [blame] | 1869 | } |
| 1870 | |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 1871 | func (agent *LogicalAgent) portExist(device *voltha.Device, port *voltha.Port) bool { |
khenaidoo | 6e55d9e | 2019-12-12 18:26:26 -0500 | [diff] [blame] | 1872 | ldevice := agent.getLogicalDeviceWithoutLock() |
| 1873 | for _, lPort := range ldevice.Ports { |
| 1874 | if lPort.DeviceId == device.Id && lPort.DevicePortNo == port.PortNo && lPort.Id == port.Label { |
| 1875 | return true |
khenaidoo | fc1314d | 2019-03-14 09:34:21 -0400 | [diff] [blame] | 1876 | } |
| 1877 | } |
| 1878 | return false |
| 1879 | } |
| 1880 | |
khenaidoo | 8f47419 | 2019-04-03 17:20:44 -0400 | [diff] [blame] | 1881 | // addUNILogicalPort adds an UNI port to the logical device. It returns a bool representing whether a port has been |
| 1882 | // added and an eror in case a valid error is encountered. If the port was successfully added it will return |
| 1883 | // (true, nil). If the device is not in the correct state it will return (false, nil) as this is a valid |
| 1884 | // 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] | 1885 | 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] | 1886 | logger.Debugw("addUNILogicalPort", log.Fields{"port": port}) |
khenaidoo | 8f47419 | 2019-04-03 17:20:44 -0400 | [diff] [blame] | 1887 | if childDevice.AdminState != voltha.AdminState_ENABLED || childDevice.OperStatus != voltha.OperStatus_ACTIVE { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1888 | 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] | 1889 | return false, nil |
khenaidoo | fc1314d | 2019-03-14 09:34:21 -0400 | [diff] [blame] | 1890 | } |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 1891 | if err := agent.requestQueue.WaitForGreenLight(ctx); err != nil { |
| 1892 | return false, err |
| 1893 | } |
| 1894 | |
khenaidoo | 1ce37ad | 2019-03-24 22:07:24 -0400 | [diff] [blame] | 1895 | if agent.portExist(childDevice, port) { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1896 | logger.Debugw("port-already-exist", log.Fields{"port": port}) |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 1897 | agent.requestQueue.RequestComplete() |
khenaidoo | 8f47419 | 2019-04-03 17:20:44 -0400 | [diff] [blame] | 1898 | return false, nil |
khenaidoo | 1ce37ad | 2019-03-24 22:07:24 -0400 | [diff] [blame] | 1899 | } |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 1900 | agent.requestQueue.RequestComplete() |
khenaidoo | fc1314d | 2019-03-14 09:34:21 -0400 | [diff] [blame] | 1901 | var portCap *ic.PortCapability |
| 1902 | var err error |
| 1903 | // First get the port capability |
npujar | 467fe75 | 2020-01-16 20:17:45 +0530 | [diff] [blame] | 1904 | 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] | 1905 | logger.Errorw("error-retrieving-port-capabilities", log.Fields{"error": err}) |
khenaidoo | 8f47419 | 2019-04-03 17:20:44 -0400 | [diff] [blame] | 1906 | return false, err |
khenaidoo | fc1314d | 2019-03-14 09:34:21 -0400 | [diff] [blame] | 1907 | } |
khenaidoo | 442e7c7 | 2020-03-10 16:13:48 -0400 | [diff] [blame] | 1908 | if err := agent.requestQueue.WaitForGreenLight(ctx); err != nil { |
| 1909 | return false, err |
| 1910 | } |
| 1911 | defer agent.requestQueue.RequestComplete() |
khenaidoo | 1ce37ad | 2019-03-24 22:07:24 -0400 | [diff] [blame] | 1912 | // Double check again if this port has been already added since the getPortCapability could have taken a long time |
| 1913 | if agent.portExist(childDevice, port) { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1914 | logger.Debugw("port-already-exist", log.Fields{"port": port}) |
khenaidoo | 8f47419 | 2019-04-03 17:20:44 -0400 | [diff] [blame] | 1915 | return false, nil |
khenaidoo | 1ce37ad | 2019-03-24 22:07:24 -0400 | [diff] [blame] | 1916 | } |
khenaidoo | fc1314d | 2019-03-14 09:34:21 -0400 | [diff] [blame] | 1917 | // Get stored logical device |
khenaidoo | 6e55d9e | 2019-12-12 18:26:26 -0500 | [diff] [blame] | 1918 | ldevice := agent.getLogicalDeviceWithoutLock() |
| 1919 | |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1920 | logger.Debugw("adding-uni", log.Fields{"deviceId": childDevice.Id}) |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 1921 | portCap.Port.RootPort = false |
| 1922 | portCap.Port.Id = port.Label |
| 1923 | portCap.Port.OfpPort.PortNo = port.PortNo |
| 1924 | portCap.Port.DeviceId = childDevice.Id |
| 1925 | portCap.Port.DevicePortNo = port.PortNo |
Kent Hagerman | d9cc2e9 | 2019-11-04 13:28:15 -0500 | [diff] [blame] | 1926 | clonedPorts := clonePorts(ldevice.Ports) |
| 1927 | if clonedPorts == nil { |
| 1928 | clonedPorts = make([]*voltha.LogicalPort, 0) |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 1929 | } |
Kent Hagerman | d9cc2e9 | 2019-11-04 13:28:15 -0500 | [diff] [blame] | 1930 | clonedPorts = append(clonedPorts, portCap.Port) |
| 1931 | if err := agent.updateLogicalDevicePortsWithoutLock(ctx, ldevice, clonedPorts); err != nil { |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 1932 | return false, err |
| 1933 | } |
| 1934 | // Update the device graph with this new logical port |
| 1935 | clonedLP := (proto.Clone(portCap.Port)).(*voltha.LogicalPort) |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 1936 | |
| 1937 | go func() { |
| 1938 | if err := agent.updateRoutes(context.Background(), clonedLP); err != nil { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1939 | 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] | 1940 | } |
| 1941 | }() |
| 1942 | |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 1943 | return true, nil |
khenaidoo | fc1314d | 2019-03-14 09:34:21 -0400 | [diff] [blame] | 1944 | } |
| 1945 | |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 1946 | func (agent *LogicalAgent) packetOut(ctx context.Context, packet *ofp.OfpPacketOut) { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1947 | logger.Debugw("packet-out", log.Fields{ |
Matteo Scandolo | 360605d | 2019-11-05 18:29:17 -0800 | [diff] [blame] | 1948 | "packet": hex.EncodeToString(packet.Data), |
| 1949 | "inPort": packet.GetInPort(), |
| 1950 | }) |
khenaidoo | 68c930b | 2019-05-13 11:46:51 -0400 | [diff] [blame] | 1951 | outPort := fu.GetPacketOutPort(packet) |
khenaidoo | fdbad6e | 2018-11-06 22:26:38 -0500 | [diff] [blame] | 1952 | //frame := packet.GetData() |
| 1953 | //TODO: Use a channel between the logical agent and the device agent |
npujar | 467fe75 | 2020-01-16 20:17:45 +0530 | [diff] [blame] | 1954 | if err := agent.deviceMgr.packetOut(ctx, agent.rootDeviceID, outPort, packet); err != nil { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1955 | logger.Error("packetout-failed", log.Fields{"logicalDeviceID": agent.rootDeviceID}) |
khenaidoo | ca30132 | 2019-01-09 23:06:32 -0500 | [diff] [blame] | 1956 | } |
khenaidoo | fdbad6e | 2018-11-06 22:26:38 -0500 | [diff] [blame] | 1957 | } |
| 1958 | |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 1959 | func (agent *LogicalAgent) packetIn(port uint32, transactionID string, packet []byte) { |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1960 | logger.Debugw("packet-in", log.Fields{ |
Matteo Scandolo | 360605d | 2019-11-05 18:29:17 -0800 | [diff] [blame] | 1961 | "port": port, |
| 1962 | "packet": hex.EncodeToString(packet), |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 1963 | "transactionId": transactionID, |
Matteo Scandolo | 360605d | 2019-11-05 18:29:17 -0800 | [diff] [blame] | 1964 | }) |
khenaidoo | 68c930b | 2019-05-13 11:46:51 -0400 | [diff] [blame] | 1965 | packetIn := fu.MkPacketIn(port, packet) |
Kent Hagerman | 45a13e4 | 2020-04-13 12:23:50 -0400 | [diff] [blame] | 1966 | agent.ldeviceMgr.SendPacketIn(agent.logicalDeviceID, transactionID, packetIn) |
Girish Kumar | f56a468 | 2020-03-20 20:07:46 +0000 | [diff] [blame] | 1967 | logger.Debugw("sending-packet-in", log.Fields{"packet": hex.EncodeToString(packetIn.Data)}) |
khenaidoo | fdbad6e | 2018-11-06 22:26:38 -0500 | [diff] [blame] | 1968 | } |
khenaidoo | 2c6a099 | 2019-04-29 13:46:56 -0400 | [diff] [blame] | 1969 | |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 1970 | func (agent *LogicalAgent) addLogicalPortToMap(portNo uint32, nniPort bool) { |
khenaidoo | 2c6a099 | 2019-04-29 13:46:56 -0400 | [diff] [blame] | 1971 | agent.lockLogicalPortsNo.Lock() |
| 1972 | defer agent.lockLogicalPortsNo.Unlock() |
| 1973 | if exist := agent.logicalPortsNo[portNo]; !exist { |
| 1974 | agent.logicalPortsNo[portNo] = nniPort |
| 1975 | } |
| 1976 | } |
| 1977 | |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 1978 | func (agent *LogicalAgent) deleteLogicalPortsFromMap(portsNo []uint32) { |
khenaidoo | 820197c | 2020-02-13 16:35:33 -0500 | [diff] [blame] | 1979 | agent.lockLogicalPortsNo.Lock() |
| 1980 | defer agent.lockLogicalPortsNo.Unlock() |
| 1981 | for _, pNo := range portsNo { |
| 1982 | delete(agent.logicalPortsNo, pNo) |
| 1983 | } |
| 1984 | } |
| 1985 | |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 1986 | func (agent *LogicalAgent) addLogicalPortsToMap(lps []*voltha.LogicalPort) { |
khenaidoo | 3d3b8c2 | 2019-05-22 18:10:39 -0400 | [diff] [blame] | 1987 | agent.lockLogicalPortsNo.Lock() |
| 1988 | defer agent.lockLogicalPortsNo.Unlock() |
| 1989 | for _, lp := range lps { |
| 1990 | if exist := agent.logicalPortsNo[lp.DevicePortNo]; !exist { |
| 1991 | agent.logicalPortsNo[lp.DevicePortNo] = lp.RootPort |
| 1992 | } |
| 1993 | } |
| 1994 | } |
| 1995 | |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 1996 | func (agent *LogicalAgent) isNNIPort(portNo uint32) bool { |
khenaidoo | 2c6a099 | 2019-04-29 13:46:56 -0400 | [diff] [blame] | 1997 | agent.lockLogicalPortsNo.RLock() |
| 1998 | defer agent.lockLogicalPortsNo.RUnlock() |
| 1999 | if exist := agent.logicalPortsNo[portNo]; exist { |
| 2000 | return agent.logicalPortsNo[portNo] |
| 2001 | } |
| 2002 | return false |
| 2003 | } |
| 2004 | |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 2005 | func (agent *LogicalAgent) getFirstNNIPort() (uint32, error) { |
khenaidoo | 2c6a099 | 2019-04-29 13:46:56 -0400 | [diff] [blame] | 2006 | agent.lockLogicalPortsNo.RLock() |
| 2007 | defer agent.lockLogicalPortsNo.RUnlock() |
| 2008 | for portNo, nni := range agent.logicalPortsNo { |
| 2009 | if nni { |
| 2010 | return portNo, nil |
| 2011 | } |
| 2012 | } |
| 2013 | return 0, status.Error(codes.NotFound, "No NNI port found") |
| 2014 | } |
Esin Karaman | 09959ae | 2019-11-29 13:59:58 +0000 | [diff] [blame] | 2015 | |
| 2016 | //GetNNIPorts returns NNI ports. |
Kent Hagerman | 2b21604 | 2020-04-03 18:28:56 -0400 | [diff] [blame] | 2017 | func (agent *LogicalAgent) GetNNIPorts() []uint32 { |
Esin Karaman | 09959ae | 2019-11-29 13:59:58 +0000 | [diff] [blame] | 2018 | agent.lockLogicalPortsNo.RLock() |
| 2019 | defer agent.lockLogicalPortsNo.RUnlock() |
| 2020 | nniPorts := make([]uint32, 0) |
| 2021 | for portNo, nni := range agent.logicalPortsNo { |
| 2022 | if nni { |
| 2023 | nniPorts = append(nniPorts, portNo) |
| 2024 | } |
| 2025 | } |
| 2026 | return nniPorts |
| 2027 | } |