| .. | .. |
|---|
| 1 | +// SPDX-License-Identifier: GPL-2.0-only |
|---|
| 1 | 2 | /* |
|---|
| 2 | 3 | * Copyright (c) 2017 Nicira, Inc. |
|---|
| 3 | | - * |
|---|
| 4 | | - * This program is free software; you can redistribute it and/or |
|---|
| 5 | | - * modify it under the terms of version 2 of the GNU General Public |
|---|
| 6 | | - * License as published by the Free Software Foundation. |
|---|
| 7 | 4 | */ |
|---|
| 8 | 5 | |
|---|
| 9 | 6 | #define pr_fmt(fmt) KBUILD_MODNAME ": " fmt |
|---|
| .. | .. |
|---|
| 15 | 12 | #include <linux/openvswitch.h> |
|---|
| 16 | 13 | #include <linux/netlink.h> |
|---|
| 17 | 14 | #include <linux/rculist.h> |
|---|
| 15 | +#include <linux/swap.h> |
|---|
| 18 | 16 | |
|---|
| 19 | 17 | #include <net/netlink.h> |
|---|
| 20 | 18 | #include <net/genetlink.h> |
|---|
| 21 | 19 | |
|---|
| 22 | 20 | #include "datapath.h" |
|---|
| 23 | 21 | #include "meter.h" |
|---|
| 24 | | - |
|---|
| 25 | | -#define METER_HASH_BUCKETS 1024 |
|---|
| 26 | 22 | |
|---|
| 27 | 23 | static const struct nla_policy meter_policy[OVS_METER_ATTR_MAX + 1] = { |
|---|
| 28 | 24 | [OVS_METER_ATTR_ID] = { .type = NLA_U32, }, |
|---|
| .. | .. |
|---|
| 42 | 38 | [OVS_BAND_ATTR_STATS] = { .len = sizeof(struct ovs_flow_stats) }, |
|---|
| 43 | 39 | }; |
|---|
| 44 | 40 | |
|---|
| 41 | +static u32 meter_hash(struct dp_meter_instance *ti, u32 id) |
|---|
| 42 | +{ |
|---|
| 43 | + return id % ti->n_meters; |
|---|
| 44 | +} |
|---|
| 45 | + |
|---|
| 45 | 46 | static void ovs_meter_free(struct dp_meter *meter) |
|---|
| 46 | 47 | { |
|---|
| 47 | 48 | if (!meter) |
|---|
| .. | .. |
|---|
| 50 | 51 | kfree_rcu(meter, rcu); |
|---|
| 51 | 52 | } |
|---|
| 52 | 53 | |
|---|
| 53 | | -static struct hlist_head *meter_hash_bucket(const struct datapath *dp, |
|---|
| 54 | | - u32 meter_id) |
|---|
| 55 | | -{ |
|---|
| 56 | | - return &dp->meters[meter_id & (METER_HASH_BUCKETS - 1)]; |
|---|
| 57 | | -} |
|---|
| 58 | | - |
|---|
| 59 | 54 | /* Call with ovs_mutex or RCU read lock. */ |
|---|
| 60 | | -static struct dp_meter *lookup_meter(const struct datapath *dp, |
|---|
| 55 | +static struct dp_meter *lookup_meter(const struct dp_meter_table *tbl, |
|---|
| 61 | 56 | u32 meter_id) |
|---|
| 62 | 57 | { |
|---|
| 58 | + struct dp_meter_instance *ti = rcu_dereference_ovsl(tbl->ti); |
|---|
| 59 | + u32 hash = meter_hash(ti, meter_id); |
|---|
| 63 | 60 | struct dp_meter *meter; |
|---|
| 64 | | - struct hlist_head *head; |
|---|
| 65 | 61 | |
|---|
| 66 | | - head = meter_hash_bucket(dp, meter_id); |
|---|
| 67 | | - hlist_for_each_entry_rcu(meter, head, dp_hash_node) { |
|---|
| 68 | | - if (meter->id == meter_id) |
|---|
| 69 | | - return meter; |
|---|
| 70 | | - } |
|---|
| 62 | + meter = rcu_dereference_ovsl(ti->dp_meters[hash]); |
|---|
| 63 | + if (meter && likely(meter->id == meter_id)) |
|---|
| 64 | + return meter; |
|---|
| 65 | + |
|---|
| 71 | 66 | return NULL; |
|---|
| 72 | 67 | } |
|---|
| 73 | 68 | |
|---|
| 74 | | -static void attach_meter(struct datapath *dp, struct dp_meter *meter) |
|---|
| 69 | +static struct dp_meter_instance *dp_meter_instance_alloc(const u32 size) |
|---|
| 75 | 70 | { |
|---|
| 76 | | - struct hlist_head *head = meter_hash_bucket(dp, meter->id); |
|---|
| 71 | + struct dp_meter_instance *ti; |
|---|
| 77 | 72 | |
|---|
| 78 | | - hlist_add_head_rcu(&meter->dp_hash_node, head); |
|---|
| 73 | + ti = kvzalloc(sizeof(*ti) + |
|---|
| 74 | + sizeof(struct dp_meter *) * size, |
|---|
| 75 | + GFP_KERNEL); |
|---|
| 76 | + if (!ti) |
|---|
| 77 | + return NULL; |
|---|
| 78 | + |
|---|
| 79 | + ti->n_meters = size; |
|---|
| 80 | + |
|---|
| 81 | + return ti; |
|---|
| 79 | 82 | } |
|---|
| 80 | 83 | |
|---|
| 81 | | -static void detach_meter(struct dp_meter *meter) |
|---|
| 84 | +static void dp_meter_instance_free(struct dp_meter_instance *ti) |
|---|
| 82 | 85 | { |
|---|
| 86 | + kvfree(ti); |
|---|
| 87 | +} |
|---|
| 88 | + |
|---|
| 89 | +static void dp_meter_instance_free_rcu(struct rcu_head *rcu) |
|---|
| 90 | +{ |
|---|
| 91 | + struct dp_meter_instance *ti; |
|---|
| 92 | + |
|---|
| 93 | + ti = container_of(rcu, struct dp_meter_instance, rcu); |
|---|
| 94 | + kvfree(ti); |
|---|
| 95 | +} |
|---|
| 96 | + |
|---|
| 97 | +static int |
|---|
| 98 | +dp_meter_instance_realloc(struct dp_meter_table *tbl, u32 size) |
|---|
| 99 | +{ |
|---|
| 100 | + struct dp_meter_instance *ti = rcu_dereference_ovsl(tbl->ti); |
|---|
| 101 | + int n_meters = min(size, ti->n_meters); |
|---|
| 102 | + struct dp_meter_instance *new_ti; |
|---|
| 103 | + int i; |
|---|
| 104 | + |
|---|
| 105 | + new_ti = dp_meter_instance_alloc(size); |
|---|
| 106 | + if (!new_ti) |
|---|
| 107 | + return -ENOMEM; |
|---|
| 108 | + |
|---|
| 109 | + for (i = 0; i < n_meters; i++) |
|---|
| 110 | + if (rcu_dereference_ovsl(ti->dp_meters[i])) |
|---|
| 111 | + new_ti->dp_meters[i] = ti->dp_meters[i]; |
|---|
| 112 | + |
|---|
| 113 | + rcu_assign_pointer(tbl->ti, new_ti); |
|---|
| 114 | + call_rcu(&ti->rcu, dp_meter_instance_free_rcu); |
|---|
| 115 | + |
|---|
| 116 | + return 0; |
|---|
| 117 | +} |
|---|
| 118 | + |
|---|
| 119 | +static void dp_meter_instance_insert(struct dp_meter_instance *ti, |
|---|
| 120 | + struct dp_meter *meter) |
|---|
| 121 | +{ |
|---|
| 122 | + u32 hash; |
|---|
| 123 | + |
|---|
| 124 | + hash = meter_hash(ti, meter->id); |
|---|
| 125 | + rcu_assign_pointer(ti->dp_meters[hash], meter); |
|---|
| 126 | +} |
|---|
| 127 | + |
|---|
| 128 | +static void dp_meter_instance_remove(struct dp_meter_instance *ti, |
|---|
| 129 | + struct dp_meter *meter) |
|---|
| 130 | +{ |
|---|
| 131 | + u32 hash; |
|---|
| 132 | + |
|---|
| 133 | + hash = meter_hash(ti, meter->id); |
|---|
| 134 | + RCU_INIT_POINTER(ti->dp_meters[hash], NULL); |
|---|
| 135 | +} |
|---|
| 136 | + |
|---|
| 137 | +static int attach_meter(struct dp_meter_table *tbl, struct dp_meter *meter) |
|---|
| 138 | +{ |
|---|
| 139 | + struct dp_meter_instance *ti = rcu_dereference_ovsl(tbl->ti); |
|---|
| 140 | + u32 hash = meter_hash(ti, meter->id); |
|---|
| 141 | + int err; |
|---|
| 142 | + |
|---|
| 143 | + /* In generally, slots selected should be empty, because |
|---|
| 144 | + * OvS uses id-pool to fetch a available id. |
|---|
| 145 | + */ |
|---|
| 146 | + if (unlikely(rcu_dereference_ovsl(ti->dp_meters[hash]))) |
|---|
| 147 | + return -EBUSY; |
|---|
| 148 | + |
|---|
| 149 | + dp_meter_instance_insert(ti, meter); |
|---|
| 150 | + |
|---|
| 151 | + /* That function is thread-safe. */ |
|---|
| 152 | + tbl->count++; |
|---|
| 153 | + if (tbl->count >= tbl->max_meters_allowed) { |
|---|
| 154 | + err = -EFBIG; |
|---|
| 155 | + goto attach_err; |
|---|
| 156 | + } |
|---|
| 157 | + |
|---|
| 158 | + if (tbl->count >= ti->n_meters && |
|---|
| 159 | + dp_meter_instance_realloc(tbl, ti->n_meters * 2)) { |
|---|
| 160 | + err = -ENOMEM; |
|---|
| 161 | + goto attach_err; |
|---|
| 162 | + } |
|---|
| 163 | + |
|---|
| 164 | + return 0; |
|---|
| 165 | + |
|---|
| 166 | +attach_err: |
|---|
| 167 | + dp_meter_instance_remove(ti, meter); |
|---|
| 168 | + tbl->count--; |
|---|
| 169 | + return err; |
|---|
| 170 | +} |
|---|
| 171 | + |
|---|
| 172 | +static int detach_meter(struct dp_meter_table *tbl, struct dp_meter *meter) |
|---|
| 173 | +{ |
|---|
| 174 | + struct dp_meter_instance *ti; |
|---|
| 175 | + |
|---|
| 83 | 176 | ASSERT_OVSL(); |
|---|
| 84 | | - if (meter) |
|---|
| 85 | | - hlist_del_rcu(&meter->dp_hash_node); |
|---|
| 177 | + if (!meter) |
|---|
| 178 | + return 0; |
|---|
| 179 | + |
|---|
| 180 | + ti = rcu_dereference_ovsl(tbl->ti); |
|---|
| 181 | + dp_meter_instance_remove(ti, meter); |
|---|
| 182 | + |
|---|
| 183 | + tbl->count--; |
|---|
| 184 | + |
|---|
| 185 | + /* Shrink the meter array if necessary. */ |
|---|
| 186 | + if (ti->n_meters > DP_METER_ARRAY_SIZE_MIN && |
|---|
| 187 | + tbl->count <= (ti->n_meters / 4)) { |
|---|
| 188 | + int half_size = ti->n_meters / 2; |
|---|
| 189 | + int i; |
|---|
| 190 | + |
|---|
| 191 | + /* Avoid hash collision, don't move slots to other place. |
|---|
| 192 | + * Make sure there are no references of meters in array |
|---|
| 193 | + * which will be released. |
|---|
| 194 | + */ |
|---|
| 195 | + for (i = half_size; i < ti->n_meters; i++) |
|---|
| 196 | + if (rcu_dereference_ovsl(ti->dp_meters[i])) |
|---|
| 197 | + goto out; |
|---|
| 198 | + |
|---|
| 199 | + if (dp_meter_instance_realloc(tbl, half_size)) |
|---|
| 200 | + goto shrink_err; |
|---|
| 201 | + } |
|---|
| 202 | + |
|---|
| 203 | +out: |
|---|
| 204 | + return 0; |
|---|
| 205 | + |
|---|
| 206 | +shrink_err: |
|---|
| 207 | + dp_meter_instance_insert(ti, meter); |
|---|
| 208 | + tbl->count++; |
|---|
| 209 | + return -ENOMEM; |
|---|
| 86 | 210 | } |
|---|
| 87 | 211 | |
|---|
| 88 | 212 | static struct sk_buff * |
|---|
| .. | .. |
|---|
| 118 | 242 | if (nla_put_u32(reply, OVS_METER_ATTR_ID, meter_id)) |
|---|
| 119 | 243 | goto error; |
|---|
| 120 | 244 | |
|---|
| 121 | | - if (!meter) |
|---|
| 122 | | - return 0; |
|---|
| 123 | | - |
|---|
| 124 | 245 | if (nla_put(reply, OVS_METER_ATTR_STATS, |
|---|
| 125 | | - sizeof(struct ovs_flow_stats), &meter->stats) || |
|---|
| 126 | | - nla_put_u64_64bit(reply, OVS_METER_ATTR_USED, meter->used, |
|---|
| 246 | + sizeof(struct ovs_flow_stats), &meter->stats)) |
|---|
| 247 | + goto error; |
|---|
| 248 | + |
|---|
| 249 | + if (nla_put_u64_64bit(reply, OVS_METER_ATTR_USED, meter->used, |
|---|
| 127 | 250 | OVS_METER_ATTR_PAD)) |
|---|
| 128 | 251 | goto error; |
|---|
| 129 | 252 | |
|---|
| 130 | | - nla = nla_nest_start(reply, OVS_METER_ATTR_BANDS); |
|---|
| 253 | + nla = nla_nest_start_noflag(reply, OVS_METER_ATTR_BANDS); |
|---|
| 131 | 254 | if (!nla) |
|---|
| 132 | 255 | goto error; |
|---|
| 133 | 256 | |
|---|
| .. | .. |
|---|
| 136 | 259 | for (i = 0; i < meter->n_bands; ++i, ++band) { |
|---|
| 137 | 260 | struct nlattr *band_nla; |
|---|
| 138 | 261 | |
|---|
| 139 | | - band_nla = nla_nest_start(reply, OVS_BAND_ATTR_UNSPEC); |
|---|
| 262 | + band_nla = nla_nest_start_noflag(reply, OVS_BAND_ATTR_UNSPEC); |
|---|
| 140 | 263 | if (!band_nla || nla_put(reply, OVS_BAND_ATTR_STATS, |
|---|
| 141 | 264 | sizeof(struct ovs_flow_stats), |
|---|
| 142 | 265 | &band->stats)) |
|---|
| .. | .. |
|---|
| 152 | 275 | |
|---|
| 153 | 276 | static int ovs_meter_cmd_features(struct sk_buff *skb, struct genl_info *info) |
|---|
| 154 | 277 | { |
|---|
| 155 | | - struct sk_buff *reply; |
|---|
| 278 | + struct ovs_header *ovs_header = info->userhdr; |
|---|
| 156 | 279 | struct ovs_header *ovs_reply_header; |
|---|
| 157 | 280 | struct nlattr *nla, *band_nla; |
|---|
| 158 | | - int err; |
|---|
| 281 | + struct sk_buff *reply; |
|---|
| 282 | + struct datapath *dp; |
|---|
| 283 | + int err = -EMSGSIZE; |
|---|
| 159 | 284 | |
|---|
| 160 | 285 | reply = ovs_meter_cmd_reply_start(info, OVS_METER_CMD_FEATURES, |
|---|
| 161 | 286 | &ovs_reply_header); |
|---|
| 162 | 287 | if (IS_ERR(reply)) |
|---|
| 163 | 288 | return PTR_ERR(reply); |
|---|
| 164 | 289 | |
|---|
| 165 | | - if (nla_put_u32(reply, OVS_METER_ATTR_MAX_METERS, U32_MAX) || |
|---|
| 166 | | - nla_put_u32(reply, OVS_METER_ATTR_MAX_BANDS, DP_MAX_BANDS)) |
|---|
| 290 | + ovs_lock(); |
|---|
| 291 | + dp = get_dp(sock_net(skb->sk), ovs_header->dp_ifindex); |
|---|
| 292 | + if (!dp) { |
|---|
| 293 | + err = -ENODEV; |
|---|
| 294 | + goto exit_unlock; |
|---|
| 295 | + } |
|---|
| 296 | + |
|---|
| 297 | + if (nla_put_u32(reply, OVS_METER_ATTR_MAX_METERS, |
|---|
| 298 | + dp->meter_tbl.max_meters_allowed)) |
|---|
| 299 | + goto exit_unlock; |
|---|
| 300 | + |
|---|
| 301 | + ovs_unlock(); |
|---|
| 302 | + |
|---|
| 303 | + if (nla_put_u32(reply, OVS_METER_ATTR_MAX_BANDS, DP_MAX_BANDS)) |
|---|
| 167 | 304 | goto nla_put_failure; |
|---|
| 168 | 305 | |
|---|
| 169 | | - nla = nla_nest_start(reply, OVS_METER_ATTR_BANDS); |
|---|
| 306 | + nla = nla_nest_start_noflag(reply, OVS_METER_ATTR_BANDS); |
|---|
| 170 | 307 | if (!nla) |
|---|
| 171 | 308 | goto nla_put_failure; |
|---|
| 172 | 309 | |
|---|
| 173 | | - band_nla = nla_nest_start(reply, OVS_BAND_ATTR_UNSPEC); |
|---|
| 310 | + band_nla = nla_nest_start_noflag(reply, OVS_BAND_ATTR_UNSPEC); |
|---|
| 174 | 311 | if (!band_nla) |
|---|
| 175 | 312 | goto nla_put_failure; |
|---|
| 176 | 313 | /* Currently only DROP band type is supported. */ |
|---|
| .. | .. |
|---|
| 182 | 319 | genlmsg_end(reply, ovs_reply_header); |
|---|
| 183 | 320 | return genlmsg_reply(reply, info); |
|---|
| 184 | 321 | |
|---|
| 322 | +exit_unlock: |
|---|
| 323 | + ovs_unlock(); |
|---|
| 185 | 324 | nla_put_failure: |
|---|
| 186 | 325 | nlmsg_free(reply); |
|---|
| 187 | | - err = -EMSGSIZE; |
|---|
| 188 | 326 | return err; |
|---|
| 189 | 327 | } |
|---|
| 190 | 328 | |
|---|
| .. | .. |
|---|
| 206 | 344 | return ERR_PTR(-EINVAL); |
|---|
| 207 | 345 | |
|---|
| 208 | 346 | /* Allocate and set up the meter before locking anything. */ |
|---|
| 209 | | - meter = kzalloc(n_bands * sizeof(struct dp_meter_band) + |
|---|
| 210 | | - sizeof(*meter), GFP_KERNEL); |
|---|
| 347 | + meter = kzalloc(struct_size(meter, bands, n_bands), GFP_KERNEL); |
|---|
| 211 | 348 | if (!meter) |
|---|
| 212 | 349 | return ERR_PTR(-ENOMEM); |
|---|
| 213 | 350 | |
|---|
| .. | .. |
|---|
| 228 | 365 | struct nlattr *attr[OVS_BAND_ATTR_MAX + 1]; |
|---|
| 229 | 366 | u32 band_max_delta_t; |
|---|
| 230 | 367 | |
|---|
| 231 | | - err = nla_parse((struct nlattr **)&attr, OVS_BAND_ATTR_MAX, |
|---|
| 232 | | - nla_data(nla), nla_len(nla), band_policy, |
|---|
| 233 | | - NULL); |
|---|
| 368 | + err = nla_parse_deprecated((struct nlattr **)&attr, |
|---|
| 369 | + OVS_BAND_ATTR_MAX, nla_data(nla), |
|---|
| 370 | + nla_len(nla), band_policy, NULL); |
|---|
| 234 | 371 | if (err) |
|---|
| 235 | 372 | goto exit_free_meter; |
|---|
| 236 | 373 | |
|---|
| .. | .. |
|---|
| 276 | 413 | struct sk_buff *reply; |
|---|
| 277 | 414 | struct ovs_header *ovs_reply_header; |
|---|
| 278 | 415 | struct ovs_header *ovs_header = info->userhdr; |
|---|
| 416 | + struct dp_meter_table *meter_tbl; |
|---|
| 279 | 417 | struct datapath *dp; |
|---|
| 280 | 418 | int err; |
|---|
| 281 | 419 | u32 meter_id; |
|---|
| 282 | 420 | bool failed; |
|---|
| 283 | 421 | |
|---|
| 284 | | - if (!a[OVS_METER_ATTR_ID]) { |
|---|
| 285 | | - return -ENODEV; |
|---|
| 286 | | - } |
|---|
| 422 | + if (!a[OVS_METER_ATTR_ID]) |
|---|
| 423 | + return -EINVAL; |
|---|
| 287 | 424 | |
|---|
| 288 | 425 | meter = dp_meter_create(a); |
|---|
| 289 | 426 | if (IS_ERR_OR_NULL(meter)) |
|---|
| .. | .. |
|---|
| 303 | 440 | goto exit_unlock; |
|---|
| 304 | 441 | } |
|---|
| 305 | 442 | |
|---|
| 443 | + meter_tbl = &dp->meter_tbl; |
|---|
| 306 | 444 | meter_id = nla_get_u32(a[OVS_METER_ATTR_ID]); |
|---|
| 307 | 445 | |
|---|
| 308 | | - /* Cannot fail after this. */ |
|---|
| 309 | | - old_meter = lookup_meter(dp, meter_id); |
|---|
| 310 | | - detach_meter(old_meter); |
|---|
| 311 | | - attach_meter(dp, meter); |
|---|
| 446 | + old_meter = lookup_meter(meter_tbl, meter_id); |
|---|
| 447 | + err = detach_meter(meter_tbl, old_meter); |
|---|
| 448 | + if (err) |
|---|
| 449 | + goto exit_unlock; |
|---|
| 450 | + |
|---|
| 451 | + err = attach_meter(meter_tbl, meter); |
|---|
| 452 | + if (err) |
|---|
| 453 | + goto exit_free_old_meter; |
|---|
| 454 | + |
|---|
| 312 | 455 | ovs_unlock(); |
|---|
| 313 | 456 | |
|---|
| 314 | 457 | /* Build response with the meter_id and stats from |
|---|
| .. | .. |
|---|
| 330 | 473 | genlmsg_end(reply, ovs_reply_header); |
|---|
| 331 | 474 | return genlmsg_reply(reply, info); |
|---|
| 332 | 475 | |
|---|
| 476 | +exit_free_old_meter: |
|---|
| 477 | + ovs_meter_free(old_meter); |
|---|
| 333 | 478 | exit_unlock: |
|---|
| 334 | 479 | ovs_unlock(); |
|---|
| 335 | 480 | nlmsg_free(reply); |
|---|
| .. | .. |
|---|
| 340 | 485 | |
|---|
| 341 | 486 | static int ovs_meter_cmd_get(struct sk_buff *skb, struct genl_info *info) |
|---|
| 342 | 487 | { |
|---|
| 343 | | - struct nlattr **a = info->attrs; |
|---|
| 344 | | - u32 meter_id; |
|---|
| 345 | 488 | struct ovs_header *ovs_header = info->userhdr; |
|---|
| 346 | 489 | struct ovs_header *ovs_reply_header; |
|---|
| 347 | | - struct datapath *dp; |
|---|
| 348 | | - int err; |
|---|
| 349 | | - struct sk_buff *reply; |
|---|
| 490 | + struct nlattr **a = info->attrs; |
|---|
| 350 | 491 | struct dp_meter *meter; |
|---|
| 492 | + struct sk_buff *reply; |
|---|
| 493 | + struct datapath *dp; |
|---|
| 494 | + u32 meter_id; |
|---|
| 495 | + int err; |
|---|
| 351 | 496 | |
|---|
| 352 | 497 | if (!a[OVS_METER_ATTR_ID]) |
|---|
| 353 | 498 | return -EINVAL; |
|---|
| .. | .. |
|---|
| 368 | 513 | } |
|---|
| 369 | 514 | |
|---|
| 370 | 515 | /* Locate meter, copy stats. */ |
|---|
| 371 | | - meter = lookup_meter(dp, meter_id); |
|---|
| 516 | + meter = lookup_meter(&dp->meter_tbl, meter_id); |
|---|
| 372 | 517 | if (!meter) { |
|---|
| 373 | 518 | err = -ENOENT; |
|---|
| 374 | 519 | goto exit_unlock; |
|---|
| .. | .. |
|---|
| 393 | 538 | |
|---|
| 394 | 539 | static int ovs_meter_cmd_del(struct sk_buff *skb, struct genl_info *info) |
|---|
| 395 | 540 | { |
|---|
| 396 | | - struct nlattr **a = info->attrs; |
|---|
| 397 | | - u32 meter_id; |
|---|
| 398 | 541 | struct ovs_header *ovs_header = info->userhdr; |
|---|
| 399 | 542 | struct ovs_header *ovs_reply_header; |
|---|
| 400 | | - struct datapath *dp; |
|---|
| 401 | | - int err; |
|---|
| 402 | | - struct sk_buff *reply; |
|---|
| 543 | + struct nlattr **a = info->attrs; |
|---|
| 403 | 544 | struct dp_meter *old_meter; |
|---|
| 545 | + struct sk_buff *reply; |
|---|
| 546 | + struct datapath *dp; |
|---|
| 547 | + u32 meter_id; |
|---|
| 548 | + int err; |
|---|
| 404 | 549 | |
|---|
| 405 | 550 | if (!a[OVS_METER_ATTR_ID]) |
|---|
| 406 | 551 | return -EINVAL; |
|---|
| 407 | | - meter_id = nla_get_u32(a[OVS_METER_ATTR_ID]); |
|---|
| 408 | 552 | |
|---|
| 409 | 553 | reply = ovs_meter_cmd_reply_start(info, OVS_METER_CMD_DEL, |
|---|
| 410 | 554 | &ovs_reply_header); |
|---|
| .. | .. |
|---|
| 419 | 563 | goto exit_unlock; |
|---|
| 420 | 564 | } |
|---|
| 421 | 565 | |
|---|
| 422 | | - old_meter = lookup_meter(dp, meter_id); |
|---|
| 566 | + meter_id = nla_get_u32(a[OVS_METER_ATTR_ID]); |
|---|
| 567 | + old_meter = lookup_meter(&dp->meter_tbl, meter_id); |
|---|
| 423 | 568 | if (old_meter) { |
|---|
| 424 | 569 | spin_lock_bh(&old_meter->lock); |
|---|
| 425 | 570 | err = ovs_meter_cmd_reply_stats(reply, meter_id, old_meter); |
|---|
| 426 | 571 | WARN_ON(err); |
|---|
| 427 | 572 | spin_unlock_bh(&old_meter->lock); |
|---|
| 428 | | - detach_meter(old_meter); |
|---|
| 573 | + |
|---|
| 574 | + err = detach_meter(&dp->meter_tbl, old_meter); |
|---|
| 575 | + if (err) |
|---|
| 576 | + goto exit_unlock; |
|---|
| 429 | 577 | } |
|---|
| 578 | + |
|---|
| 430 | 579 | ovs_unlock(); |
|---|
| 431 | 580 | ovs_meter_free(old_meter); |
|---|
| 432 | 581 | genlmsg_end(reply, ovs_reply_header); |
|---|
| .. | .. |
|---|
| 446 | 595 | bool ovs_meter_execute(struct datapath *dp, struct sk_buff *skb, |
|---|
| 447 | 596 | struct sw_flow_key *key, u32 meter_id) |
|---|
| 448 | 597 | { |
|---|
| 449 | | - struct dp_meter *meter; |
|---|
| 450 | | - struct dp_meter_band *band; |
|---|
| 451 | 598 | long long int now_ms = div_u64(ktime_get_ns(), 1000 * 1000); |
|---|
| 452 | 599 | long long int long_delta_ms; |
|---|
| 453 | | - u32 delta_ms; |
|---|
| 454 | | - u32 cost; |
|---|
| 600 | + struct dp_meter_band *band; |
|---|
| 601 | + struct dp_meter *meter; |
|---|
| 455 | 602 | int i, band_exceeded_max = -1; |
|---|
| 456 | 603 | u32 band_exceeded_rate = 0; |
|---|
| 604 | + u32 delta_ms; |
|---|
| 605 | + u32 cost; |
|---|
| 457 | 606 | |
|---|
| 458 | | - meter = lookup_meter(dp, meter_id); |
|---|
| 607 | + meter = lookup_meter(&dp->meter_tbl, meter_id); |
|---|
| 459 | 608 | /* Do not drop the packet when there is no meter. */ |
|---|
| 460 | 609 | if (!meter) |
|---|
| 461 | 610 | return false; |
|---|
| .. | .. |
|---|
| 533 | 682 | return false; |
|---|
| 534 | 683 | } |
|---|
| 535 | 684 | |
|---|
| 536 | | -static struct genl_ops dp_meter_genl_ops[] = { |
|---|
| 685 | +static const struct genl_small_ops dp_meter_genl_ops[] = { |
|---|
| 537 | 686 | { .cmd = OVS_METER_CMD_FEATURES, |
|---|
| 687 | + .validate = GENL_DONT_VALIDATE_STRICT | GENL_DONT_VALIDATE_DUMP, |
|---|
| 538 | 688 | .flags = 0, /* OK for unprivileged users. */ |
|---|
| 539 | | - .policy = meter_policy, |
|---|
| 540 | 689 | .doit = ovs_meter_cmd_features |
|---|
| 541 | 690 | }, |
|---|
| 542 | 691 | { .cmd = OVS_METER_CMD_SET, |
|---|
| 692 | + .validate = GENL_DONT_VALIDATE_STRICT | GENL_DONT_VALIDATE_DUMP, |
|---|
| 543 | 693 | .flags = GENL_ADMIN_PERM, /* Requires CAP_NET_ADMIN |
|---|
| 544 | 694 | * privilege. |
|---|
| 545 | 695 | */ |
|---|
| 546 | | - .policy = meter_policy, |
|---|
| 547 | 696 | .doit = ovs_meter_cmd_set, |
|---|
| 548 | 697 | }, |
|---|
| 549 | 698 | { .cmd = OVS_METER_CMD_GET, |
|---|
| 699 | + .validate = GENL_DONT_VALIDATE_STRICT | GENL_DONT_VALIDATE_DUMP, |
|---|
| 550 | 700 | .flags = 0, /* OK for unprivileged users. */ |
|---|
| 551 | | - .policy = meter_policy, |
|---|
| 552 | 701 | .doit = ovs_meter_cmd_get, |
|---|
| 553 | 702 | }, |
|---|
| 554 | 703 | { .cmd = OVS_METER_CMD_DEL, |
|---|
| 704 | + .validate = GENL_DONT_VALIDATE_STRICT | GENL_DONT_VALIDATE_DUMP, |
|---|
| 555 | 705 | .flags = GENL_ADMIN_PERM, /* Requires CAP_NET_ADMIN |
|---|
| 556 | 706 | * privilege. |
|---|
| 557 | 707 | */ |
|---|
| 558 | | - .policy = meter_policy, |
|---|
| 559 | 708 | .doit = ovs_meter_cmd_del |
|---|
| 560 | 709 | }, |
|---|
| 561 | 710 | }; |
|---|
| .. | .. |
|---|
| 569 | 718 | .name = OVS_METER_FAMILY, |
|---|
| 570 | 719 | .version = OVS_METER_VERSION, |
|---|
| 571 | 720 | .maxattr = OVS_METER_ATTR_MAX, |
|---|
| 721 | + .policy = meter_policy, |
|---|
| 572 | 722 | .netnsok = true, |
|---|
| 573 | 723 | .parallel_ops = true, |
|---|
| 574 | | - .ops = dp_meter_genl_ops, |
|---|
| 575 | | - .n_ops = ARRAY_SIZE(dp_meter_genl_ops), |
|---|
| 724 | + .small_ops = dp_meter_genl_ops, |
|---|
| 725 | + .n_small_ops = ARRAY_SIZE(dp_meter_genl_ops), |
|---|
| 576 | 726 | .mcgrps = &ovs_meter_multicast_group, |
|---|
| 577 | 727 | .n_mcgrps = 1, |
|---|
| 578 | 728 | .module = THIS_MODULE, |
|---|
| .. | .. |
|---|
| 580 | 730 | |
|---|
| 581 | 731 | int ovs_meters_init(struct datapath *dp) |
|---|
| 582 | 732 | { |
|---|
| 583 | | - int i; |
|---|
| 733 | + struct dp_meter_table *tbl = &dp->meter_tbl; |
|---|
| 734 | + struct dp_meter_instance *ti; |
|---|
| 735 | + unsigned long free_mem_bytes; |
|---|
| 584 | 736 | |
|---|
| 585 | | - dp->meters = kmalloc_array(METER_HASH_BUCKETS, |
|---|
| 586 | | - sizeof(struct hlist_head), GFP_KERNEL); |
|---|
| 587 | | - |
|---|
| 588 | | - if (!dp->meters) |
|---|
| 737 | + ti = dp_meter_instance_alloc(DP_METER_ARRAY_SIZE_MIN); |
|---|
| 738 | + if (!ti) |
|---|
| 589 | 739 | return -ENOMEM; |
|---|
| 590 | 740 | |
|---|
| 591 | | - for (i = 0; i < METER_HASH_BUCKETS; i++) |
|---|
| 592 | | - INIT_HLIST_HEAD(&dp->meters[i]); |
|---|
| 741 | + /* Allow meters in a datapath to use ~3.12% of physical memory. */ |
|---|
| 742 | + free_mem_bytes = nr_free_buffer_pages() * (PAGE_SIZE >> 5); |
|---|
| 743 | + tbl->max_meters_allowed = min(free_mem_bytes / sizeof(struct dp_meter), |
|---|
| 744 | + DP_METER_NUM_MAX); |
|---|
| 745 | + if (!tbl->max_meters_allowed) |
|---|
| 746 | + goto out_err; |
|---|
| 747 | + |
|---|
| 748 | + rcu_assign_pointer(tbl->ti, ti); |
|---|
| 749 | + tbl->count = 0; |
|---|
| 593 | 750 | |
|---|
| 594 | 751 | return 0; |
|---|
| 752 | + |
|---|
| 753 | +out_err: |
|---|
| 754 | + dp_meter_instance_free(ti); |
|---|
| 755 | + return -ENOMEM; |
|---|
| 595 | 756 | } |
|---|
| 596 | 757 | |
|---|
| 597 | 758 | void ovs_meters_exit(struct datapath *dp) |
|---|
| 598 | 759 | { |
|---|
| 760 | + struct dp_meter_table *tbl = &dp->meter_tbl; |
|---|
| 761 | + struct dp_meter_instance *ti = rcu_dereference_raw(tbl->ti); |
|---|
| 599 | 762 | int i; |
|---|
| 600 | 763 | |
|---|
| 601 | | - for (i = 0; i < METER_HASH_BUCKETS; i++) { |
|---|
| 602 | | - struct hlist_head *head = &dp->meters[i]; |
|---|
| 603 | | - struct dp_meter *meter; |
|---|
| 604 | | - struct hlist_node *n; |
|---|
| 764 | + for (i = 0; i < ti->n_meters; i++) |
|---|
| 765 | + ovs_meter_free(rcu_dereference_raw(ti->dp_meters[i])); |
|---|
| 605 | 766 | |
|---|
| 606 | | - hlist_for_each_entry_safe(meter, n, head, dp_hash_node) |
|---|
| 607 | | - kfree(meter); |
|---|
| 608 | | - } |
|---|
| 609 | | - |
|---|
| 610 | | - kfree(dp->meters); |
|---|
| 767 | + dp_meter_instance_free(ti); |
|---|
| 611 | 768 | } |
|---|