.. | .. |
---|
| 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 | |
---|