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