| .. | .. |
|---|
| 1 | +// SPDX-License-Identifier: GPL-2.0-or-later |
|---|
| 1 | 2 | /* |
|---|
| 2 | 3 | * net/sched/sch_cbs.c Credit Based Shaper |
|---|
| 3 | 4 | * |
|---|
| 4 | | - * This program is free software; you can redistribute it and/or |
|---|
| 5 | | - * modify it under the terms of the GNU General Public License |
|---|
| 6 | | - * as published by the Free Software Foundation; either version |
|---|
| 7 | | - * 2 of the License, or (at your option) any later version. |
|---|
| 8 | | - * |
|---|
| 9 | 5 | * Authors: Vinicius Costa Gomes <vinicius.gomes@intel.com> |
|---|
| 10 | | - * |
|---|
| 11 | 6 | */ |
|---|
| 12 | 7 | |
|---|
| 13 | 8 | /* Credit Based Shaper (CBS) |
|---|
| .. | .. |
|---|
| 93 | 88 | struct Qdisc *child, |
|---|
| 94 | 89 | struct sk_buff **to_free) |
|---|
| 95 | 90 | { |
|---|
| 91 | + unsigned int len = qdisc_pkt_len(skb); |
|---|
| 96 | 92 | int err; |
|---|
| 97 | 93 | |
|---|
| 98 | 94 | err = child->ops->enqueue(skb, child, to_free); |
|---|
| 99 | 95 | if (err != NET_XMIT_SUCCESS) |
|---|
| 100 | 96 | return err; |
|---|
| 101 | 97 | |
|---|
| 102 | | - qdisc_qstats_backlog_inc(sch, skb); |
|---|
| 98 | + sch->qstats.backlog += len; |
|---|
| 103 | 99 | sch->q.qlen++; |
|---|
| 104 | 100 | |
|---|
| 105 | 101 | return NET_XMIT_SUCCESS; |
|---|
| .. | .. |
|---|
| 313 | 309 | { |
|---|
| 314 | 310 | struct ethtool_link_ksettings ecmd; |
|---|
| 315 | 311 | int speed = SPEED_10; |
|---|
| 316 | | - int port_rate = -1; |
|---|
| 312 | + int port_rate; |
|---|
| 317 | 313 | int err; |
|---|
| 318 | 314 | |
|---|
| 319 | 315 | err = __ethtool_get_link_ksettings(dev, &ecmd); |
|---|
| .. | .. |
|---|
| 370 | 366 | struct tc_cbs_qopt *qopt; |
|---|
| 371 | 367 | int err; |
|---|
| 372 | 368 | |
|---|
| 373 | | - err = nla_parse_nested(tb, TCA_CBS_MAX, opt, cbs_policy, extack); |
|---|
| 369 | + err = nla_parse_nested_deprecated(tb, TCA_CBS_MAX, opt, cbs_policy, |
|---|
| 370 | + extack); |
|---|
| 374 | 371 | if (err < 0) |
|---|
| 375 | 372 | return err; |
|---|
| 376 | 373 | |
|---|
| .. | .. |
|---|
| 405 | 402 | { |
|---|
| 406 | 403 | struct cbs_sched_data *q = qdisc_priv(sch); |
|---|
| 407 | 404 | struct net_device *dev = qdisc_dev(sch); |
|---|
| 408 | | - int err; |
|---|
| 409 | 405 | |
|---|
| 410 | 406 | if (!opt) { |
|---|
| 411 | 407 | NL_SET_ERR_MSG(extack, "Missing CBS qdisc options which are mandatory"); |
|---|
| .. | .. |
|---|
| 417 | 413 | if (!q->qdisc) |
|---|
| 418 | 414 | return -ENOMEM; |
|---|
| 419 | 415 | |
|---|
| 416 | + spin_lock(&cbs_list_lock); |
|---|
| 417 | + list_add(&q->cbs_list, &cbs_list); |
|---|
| 418 | + spin_unlock(&cbs_list_lock); |
|---|
| 419 | + |
|---|
| 420 | 420 | qdisc_hash_add(q->qdisc, false); |
|---|
| 421 | 421 | |
|---|
| 422 | 422 | q->queue = sch->dev_queue - netdev_get_tx_queue(dev, 0); |
|---|
| .. | .. |
|---|
| 426 | 426 | |
|---|
| 427 | 427 | qdisc_watchdog_init(&q->watchdog, sch); |
|---|
| 428 | 428 | |
|---|
| 429 | | - err = cbs_change(sch, opt, extack); |
|---|
| 430 | | - if (err) |
|---|
| 431 | | - return err; |
|---|
| 432 | | - |
|---|
| 433 | | - if (!q->offload) { |
|---|
| 434 | | - spin_lock(&cbs_list_lock); |
|---|
| 435 | | - list_add(&q->cbs_list, &cbs_list); |
|---|
| 436 | | - spin_unlock(&cbs_list_lock); |
|---|
| 437 | | - } |
|---|
| 438 | | - |
|---|
| 439 | | - return 0; |
|---|
| 429 | + return cbs_change(sch, opt, extack); |
|---|
| 440 | 430 | } |
|---|
| 441 | 431 | |
|---|
| 442 | 432 | static void cbs_destroy(struct Qdisc *sch) |
|---|
| .. | .. |
|---|
| 444 | 434 | struct cbs_sched_data *q = qdisc_priv(sch); |
|---|
| 445 | 435 | struct net_device *dev = qdisc_dev(sch); |
|---|
| 446 | 436 | |
|---|
| 447 | | - spin_lock(&cbs_list_lock); |
|---|
| 448 | | - list_del(&q->cbs_list); |
|---|
| 449 | | - spin_unlock(&cbs_list_lock); |
|---|
| 437 | + /* Nothing to do if we couldn't create the underlying qdisc */ |
|---|
| 438 | + if (!q->qdisc) |
|---|
| 439 | + return; |
|---|
| 450 | 440 | |
|---|
| 451 | 441 | qdisc_watchdog_cancel(&q->watchdog); |
|---|
| 452 | 442 | cbs_disable_offload(dev, q); |
|---|
| 453 | 443 | |
|---|
| 454 | | - if (q->qdisc) |
|---|
| 455 | | - qdisc_put(q->qdisc); |
|---|
| 444 | + spin_lock(&cbs_list_lock); |
|---|
| 445 | + list_del(&q->cbs_list); |
|---|
| 446 | + spin_unlock(&cbs_list_lock); |
|---|
| 447 | + |
|---|
| 448 | + qdisc_put(q->qdisc); |
|---|
| 456 | 449 | } |
|---|
| 457 | 450 | |
|---|
| 458 | 451 | static int cbs_dump(struct Qdisc *sch, struct sk_buff *skb) |
|---|
| .. | .. |
|---|
| 461 | 454 | struct tc_cbs_qopt opt = { }; |
|---|
| 462 | 455 | struct nlattr *nest; |
|---|
| 463 | 456 | |
|---|
| 464 | | - nest = nla_nest_start(skb, TCA_OPTIONS); |
|---|
| 457 | + nest = nla_nest_start_noflag(skb, TCA_OPTIONS); |
|---|
| 465 | 458 | if (!nest) |
|---|
| 466 | 459 | goto nla_put_failure; |
|---|
| 467 | 460 | |
|---|