[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date: Tue, 05 May 2015 13:06:46 +0200
From: Daniel Borkmann <daniel@...earbox.net>
To: Florian Westphal <fw@...len.de>, netdev@...r.kernel.org
CC: jhs@...atatu.com, alexei.starovoitov@...il.com
Subject: Re: [PATCH -next 4/5] net: sched: remove AT INGRESS/EGRESS
On 05/04/2015 08:48 PM, Florian Westphal wrote:
> use single marker to propagate location.
> tc_at_ingress 1: ingress, 0 is egress.
>
> The new flag is set/unset in sch_ingress instead of the core.
> We will also no longer set skb->tc_verd to AT_EGRESS in the xmit
> handler.
>
> Signed-off-by: Florian Westphal <fw@...len.de>
...
> diff --git a/include/linux/skbuff.h b/include/linux/skbuff.h
> index 911d84e..d794077 100644
> --- a/include/linux/skbuff.h
> +++ b/include/linux/skbuff.h
> @@ -488,6 +488,7 @@ static inline u32 skb_mstamp_us_delta(const struct skb_mstamp *t1,
> * @queue_mapping: Queue mapping for multiqueue devices
> * @xmit_more: More SKBs are pending for this queue
> * @tc_from_ingress: skb is processed during rx, not transmit
> + * @tc_at_ingress: skb is processed during rx, not transmit
Minor nit:
I think both comments needs to be a bit more clear. I.e. tc_at_ingress
tells the *current* location whether we are invoked from ingress or
egress. And, tc_from_ingress explains the *previous* location, whether
the skbs has been mirred from ingress path or egress path.
Otherwise, the rest looks good to me.
Acked-by: Daniel Borkmann <daniel@...earbox.net>
--
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
Powered by blists - more mailing lists