netdev - Re: [PATCH 6/7] Make qdisc changeable.

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Mon, 30 Jul 2007 16:11:22 +0200
From:	Patrick McHardy <kaber@...sh.net>
To:	Corey Hickey <bugfood-ml@...ooh.org>
CC:	netdev@...r.kernel.org
Subject: Re: [PATCH 6/7] Make qdisc changeable.

Corey Hickey wrote:
> Re-implement sfq_change() and enable Qdisc_opts.change so "tc qdisc
> change" will work.
> 
> Signed-off-by: Corey Hickey <bugfood-ml@...ooh.org>
> ---
>  net/sched/sch_sfq.c |   51 ++++++++++++++++++++++++++++++++++++++++++++++++++-
>  1 files changed, 50 insertions(+), 1 deletions(-)
> 
> diff --git a/net/sched/sch_sfq.c b/net/sched/sch_sfq.c
> index e6a6a21..e042cd0 100644
> --- a/net/sched/sch_sfq.c
> +++ b/net/sched/sch_sfq.c
> @@ -485,6 +485,55 @@ static int sfq_init(struct Qdisc *sch, struct rtattr *opt)
>  	return 0;
>  }
>  
> +static int sfq_change(struct Qdisc *sch, struct rtattr *opt)
> +{
> +	struct sfq_sched_data *q = qdisc_priv(sch);
> +	struct sfq_sched_data tmp;
> +	struct sk_buff *skb;
> +	int err;
> +	
> +	/* set up tmp queue */
> +	memset(&tmp, 0, sizeof(struct sfq_sched_data));
> +	tmp.quantum = psched_mtu(sch->dev); /* default */


If no value is given it should use the old value instead of
reinitializing to the default.

> +	if ((err = sfq_q_init(&tmp, opt)))
> +		return err;


This will also use defaults for all unspecified values. It would
be more consistent with other qdiscs to only change those values
that are actually specified, so something like "tc qdisc change ...
perturb 10" will *only* change the perturbation parameter.
I'm not sure reusing the initialization function and copying the
parameters is the best way to do this.

> +
> +	/* copy packets from the old queue to the tmp queue */
> +	sch_tree_lock(sch);
> +	while (sch->q.qlen >= tmp.limit - 1)
> +		sfq_drop(sch);
> +	while ((skb = sfq_q_dequeue(q)) != NULL)
> +		sfq_q_enqueue(skb, &tmp, SFQ_TAIL);
> +	
> +	/* clean up the old queue */
> +	sfq_q_destroy(q);
> +
> +	/* copy elements of the tmp queue into the old queue */
> +	q->perturb_period = tmp.perturb_period;
> +	q->quantum        = tmp.quantum;
> +	q->limit          = tmp.limit;
> +	q->depth          = tmp.depth;
> +	q->hash_divisor   = tmp.hash_divisor;
> +	q->tail           = tmp.tail;
> +	q->max_depth      = tmp.max_depth;
> +	q->ht    = tmp.ht;
> +	q->dep   = tmp.dep;
> +	q->next  = tmp.next;
> +	q->allot = tmp.allot;
> +	q->hash  = tmp.hash;
> +	q->qs    = tmp.qs;
> +
> +	/* finish up */
> +	if (q->perturb_period) {
> +		q->perturb_timer.expires = jiffies + q->perturb_period;
> +		add_timer(&q->perturb_timer);
> +	} else {
> +		q->perturbation = 0;
> +	}
> +	sch_tree_unlock(sch);
> +	return 0;
> +}
> +
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html