Re: [Patch net-next] net_sched: remove the leftover cleanup_a()

to -net not -net-next and should specify something along the lines of "broken-by .." cheers, jamal

Re: [iproute PATCH 3/4] tc: use action_a2n() everywhere

On 16-07-23 07:28 AM, Phil Sutter wrote: Signed-off-by: Phil Sutter <p...@nwl.cc> Thanks for the nice work Phil. Acked-by: Jamal Hadi Salim <j...@mojatatu.com> cheers, jamal

Re: [iproute PATCH 2/4] tc: util: bore up action_a2n()

-by: Phil Sutter <p...@nwl.cc> Acked-by: Jamal Hadi Salim <j...@mojatatu.com> cheers, jamal

Re: [Patch net-next v2 1/2] net_sched: move tc_action into tcf_common

inherit from it. Cc: Jamal Hadi Salim <j...@mojatatu.com> Signed-off-by: Cong Wang <xiyou.wangc...@gmail.com> Cong - i need to stare at this longer than cursory to give feedback. I apologize, I am in travel mode but will get back to you. cheers, jamal

Re: [patch net-next 1/9] mlxsw: pci: Add resources query implementation.

as style cop ;-> Ive taken enough beating ive learnt my lesson ;-> There are a few of those in the different patches. cheers, jamal

Re: [patch net-next 1/9] mlxsw: pci: Add resources query implementation.

ic int mlxsw_pci_resources_query(struct mlxsw_pci *mlxsw_pci, char *mbox, struct mlxsw_resources *resources, u8 query_enabled) { Email is splitting that first line - but it is 80 chars or less cheers, jamal

Re: [patch net-next 0/9] mlxsw: implement port mirroring offload

cheers, jamal

Re: [patch net-next 0/9] mlxsw: implement port mirroring offload

mirred ... BTW: I am not a big styling lawyer on netdev (I am normally the victim) but would be useful to look at some of these patches with that coding style in in mind (I think some of the chip specific patches had some style issue in function definition). again - kudos cheers, jamal

Re: [Patch net-next 1/2] net_sched: move tc_action into tcf_common

(a) --- Essentially this reduces to struct tcf_mirred *m = (struct tcf_mirred *)a How would that work? Do we still need the void *priv cheers, jamal

WAS ( Re: [PATCH net-next 1/1] net_sched: Introduce skbmod action

On 16-07-19 08:23 PM, Daniel Borkmann wrote: On 07/19/2016 08:04 PM, Cong Wang wrote: On Tue, Jul 19, 2016 at 8:03 AM, Daniel Borkmann <dan...@iogearbox.net> wrote: On 07/19/2016 03:56 PM, Jamal Hadi Salim wrote: [...] [..] I don't think so. 1) checksum is supposed to be done by csum

Re: [PATCH net-next 1/1] net_sched: Introduce skbmod action

. Any suggestions? On tcf_skbmod_run, mostly ignorance: while doing only ethernet updates; is it still needed to do the checksum complete? cheers, jamal

Re: [PATCH net-next 1/1] net_sched: Introduce skbmod action

On 16-07-18 01:38 PM, Cong Wang wrote: On Mon, Jul 18, 2016 at 3:26 AM, Jamal Hadi Salim <j...@mojatatu.com> wrote: On 16-07-18 06:07 AM, Thomas Graf wrote: Right. I was at the same point as Jamal and it is nasty to try and reverse engineer the dumps without any further hints. I

Re: [PATCH net-next 1/1] net_sched: Introduce skbmod action

On 16-07-18 06:07 AM, Thomas Graf wrote: Right. I was at the same point as Jamal and it is nasty to try and reverse engineer the dumps without any further hints. I assume that's what he is referring to with difficulties. That +: if you get me a field which says "dstmac" i dont

Re: [PATCH net-next 1/1] net_sched: Introduce skbmod action

On 16-07-18 05:44 AM, Daniel Borkmann wrote: On 07/18/2016 08:51 AM, Jamal Hadi Salim wrote: On 16-07-18 12:19 AM, Alexei Starovoitov wrote: Looking at that just out of curiosity on how complex it could look for src/dst mac, is it actually functional in iproute2 upstream tree

Re: [PATCH net-next 1/1] net_sched: Introduce skbmod action

On 16-07-18 12:19 AM, Alexei Starovoitov wrote: On Sun, Jul 17, 2016 at 04:41:24AM -0400, Jamal Hadi Salim wrote: From: Jamal Hadi Salim <j...@mojatatu.com> This action is intended to be an upgrade from a usability perspective from pedit (as well as operational debugability). C

[PATCH net-next 1/1] net_sched: Introduce skbmod action

From: Jamal Hadi Salim <j...@mojatatu.com> This action is intended to be an upgrade from a usability perspective from pedit (as well as operational debugability). Compare this: sudo tc filter add dev $ETH parent 1: protocol ip prio 10 \ u32 match ip protocol 1 0xff flowid 1:2 \ action

Re: [PATCH v6 05/12] Add sample for adding simple drop program to link

On 16-07-11 09:37 AM, Jesper Dangaard Brouer wrote: On Mon, 11 Jul 2016 07:09:26 -0400 Jamal Hadi Salim <j...@mojatatu.com> wrote: $ perf record -a samples/bpf/xdp1 $( [..] So - devil's advocate speaking: I can filter and drop with this very specific NIC at 10x as fast in ha

Re: [RFC PATCH net-next 1/1] Introduce skbmod action

On 16-07-12 01:10 PM, Cong Wang wrote: On Mon, Jul 11, 2016 at 5:12 AM, Jamal Hadi Salim <j...@mojatatu.com> wrote: From: Jamal Hadi Salim <j...@mojatatu.com> This action is intended to be an upgrade from a usability perspective from pedit. Compare this: Definitely agree we need

[RFC PATCH net-next 1/1] Introduce skbmod action

From: Jamal Hadi Salim <j...@mojatatu.com> This action is intended to be an upgrade from a usability perspective from pedit. Compare this: sudo tc filter add dev $ETH parent 1: protocol ip prio 10 \ u32 match ip protocol 1 0xff flowid 1:2 \ action pedit munge offset -14 u8 se

Re: [PATCH v6 05/12] Add sample for adding simple drop program to link

driver overhead. cheers, jamal

Re: [PATCH v2 net-next 1/1] net sched actions: mirred add support for setting Dst MAC address

On 16-07-05 02:19 PM, Cong Wang wrote: On Tue, Jul 5, 2016 at 4:04 AM, Jamal Hadi Salim <j...@mojatatu.com> wrote: Second arguement usability, from: to: $TC filter add dev $ETH parent 1: protocol ip prio 10 \ u32 match ip protocol 1 0xff flowid 1:2 \ action mirred egress re

Re: [PATCH v2 net-next 1/1] net sched actions: mirred add support for setting Dst MAC address

. An opaque blob of a single action could do whatever it wants. cheers, jamal

Re: [PATCH v2 net-next 1/1] net sched actions: mirred add support for setting Dst MAC address

On 16-07-02 12:58 PM, Cong Wang wrote: On Sat, Jul 2, 2016 at 7:34 AM, Jamal Hadi Salim <j...@mojatatu.com> wrote: From: Jamal Hadi Salim <j...@mojatatu.com> Often redirecting or mirroring requires that we set the dstMAC address of the target device. While it is possible to pip

[PATCH v2 net-next 1/1] net sched actions: mirred add support for setting Dst MAC address

From: Jamal Hadi Salim <j...@mojatatu.com> Often redirecting or mirroring requires that we set the dstMAC address of the target device. While it is possible to pipe to a pedit action, this patch obsoletes the need for that. This is a justified feature because the dst MAC addresses r

Re: [PATCH net-next 1/1] net sched actions: mirred add support for setting Dst MAC address

On 16-07-02 10:16 AM, Jamal Hadi Salim wrote: On 16-07-02 10:07 AM, Nikolay Aleksandrov wrote: On 02/07/16 16:02, Jamal Hadi Salim wrote: On 16-07-02 09:49 AM, Nikolay Aleksandrov wrote: Also a minor suggestion - maybe err out on a zero mac address, otherwise the user might think

Re: [PATCH net-next 1/1] net sched actions: mirred add support for setting Dst MAC address

On 16-07-02 10:07 AM, Nikolay Aleksandrov wrote: On 02/07/16 16:02, Jamal Hadi Salim wrote: On 16-07-02 09:49 AM, Nikolay Aleksandrov wrote: Also a minor suggestion - maybe err out on a zero mac address, otherwise the user might think the operation was successful. Is a zero mac address

Re: [PATCH net-next 1/1] net sched actions: mirred add support for setting Dst MAC address

On 16-07-02 09:49 AM, Nikolay Aleksandrov wrote: On 02/07/16 15:26, Jamal Hadi Salim wrote: From: Jamal Hadi Salim <j...@mojatatu.com> Often redirecting or mirroring requires that we set the MAC address of the target device. While it is possible to pipe to a pedit action this obs

[PATCH net-next 1/1] net sched actions: mirred add support for setting Dst MAC address

From: Jamal Hadi Salim <j...@mojatatu.com> Often redirecting or mirroring requires that we set the MAC address of the target device. While it is possible to pipe to a pedit action this obsoletes the need for that. This is justified feature because the dst MAC addresses rewrite is such a

[iproute2 1/1] actions: skbedit add support for mod-ing skb pkt_type

From: Jamal Hadi Salim <j...@mojatatu.com> I'll make a formal submission sans the header when the kernel patches makes it in. This version is for someone who wants to play around with the net-next kernel patches i sent Signed-off-by: Jamal Hadi Salim <j...@mojatatu.com> --- include/

[PATCH v2 net-next 3/3] net sched actions: skbedit convert to use more modern nla_put_xxx

From: Jamal Hadi Salim <j...@mojatatu.com> Signed-off-by: Jamal Hadi Salim <j...@mojatatu.com> --- net/sched/act_skbedit.c | 12 1 file changed, 4 insertions(+), 8 deletions(-) diff --git a/net/sched/act_skbedit.c b/net/sched/act_skbedit.c index 1c4c924..8e573c0 1006

[PATCH v2 net-next 2/3] net sched actions: skbedit add support for mod-ing skb pkt_type

From: Jamal Hadi Salim <j...@mojatatu.com> Extremely useful for setting packet type to host so i dont have to modify the dst mac address using pedit (which requires that i know the mac address) Example usage: tc filter add dev eth0 parent : protocol ip pref 9 u32 \ match ip src 5.5.

[PATCH v2 net-next 1/3] net: simplify and make pkt_type_ok() available for other users

From: Jamal Hadi Salim <j...@mojatatu.com> Suggested-by: Daniel Borkmann <dan...@iogearbox.net> Signed-off-by: Jamal Hadi Salim <j...@mojatatu.com> --- include/linux/skbuff.h | 10 ++ net/netfilter/nft_meta.c | 9 + 2 files changed, 11 insertions(+), 8 deleti

Re: [net-next PATCH 1/2] net: pktgen: support injecting packets for qdisc testing

On 16-06-30 12:53 PM, John Fastabend wrote: On 16-06-30 03:21 AM, Jamal Hadi Salim wrote: On 16-06-29 03:47 PM, John Fastabend wrote: Taking a look at the link couple differences exist. First the patch linked does a 'netif_xmit_frozen_or_drv_stopped(txq)' check but this really shouldn't

Re: [Patch net] net_sched: fix mirrored packets checksum

On 16-06-30 01:15 PM, Cong Wang wrote: Similar to commit 9b368814b336 ("net: fix bridge multicast packet checksum validation") we need to fixup the checksum for CHECKSUM_COMPLETE when pushing skb on RX path. Otherwise we get similar splats. Cc: Jamal Hadi Salim <j...@mojatatu

Re: [net-next PATCH 1/2] net: pktgen: support injecting packets for qdisc testing

pkt_dev->errors++; + net_info_ratelimited("%s xmit error: %d\n", + pkt_dev->odevname, ret); + break; + } + goto out; } txq = skb_get_tx_queue(odev, pkt_dev-&g

Re: [Patch net 2/2] act_ife: acquire ife_mod_lock before reading ifeoplist

On 16-06-20 04:37 PM, Cong Wang wrote: Cc: Jamal Hadi Salim <j...@mojatatu.com> Signed-off-by: Cong Wang <xiyou.wangc...@gmail.com> Acked-by: Jamal Hadi Salim <j...@mojatatu.com> cheers, jamal

Re: [Patch net 1/2] act_ife: only acquire tcf_lock for existing actions

to make sure the whole update is atomic. For net-next, we can get rid of this spinlock because we already hold the RTNL lock on slow path, and on fast path we can use RCU to protect the metalist. Joint work with Jamal. Reported-by: Alexey Khoroshilov <khoroshi...@ispras.ru> Cc: Jamal Hadi Sa

Re: [patch net-next v4 0/4] return offloaded stats as default and expose original sw stats

e read separately today. Maybe it makes sense to have a brand new TLV for these aggregated stats as Jiri was suggesting.That means two new TLVs not one. 1) TLV for aggregated stats - which cant be current one 2) TLV for h/w stats The existing stat implies s/ware only. cheers, jamal

Re: act_mirred: remove spinlock in fast path

> and on the rare occassion that _init() is doing a write to <A,B,C> then if it should spin lock it would not corrupt what fast path sees as <A, B, C> during the transition. Am i misunderstanding? cheers, jamal

[PATCH net-next 1/2] net: simplify and make pkt_type_ok() available for other users

From: Jamal Hadi Salim <j...@mojatatu.com> Suggested-by: Daniel Borkmann <dan...@iogearbox.net> Signed-off-by: Jamal Hadi Salim <j...@mojatatu.com> --- include/linux/skbuff.h | 11 +++ net/netfilter/nft_meta.c | 9 + 2 files changed, 12 insertions(+), 8

[PATCH net-next 2/2] net sched actions: skbedit add support for mod-ing skb pkt_type

From: Jamal Hadi Salim <j...@mojatatu.com> Extremely useful for setting packet type to host so i dont have to modify the dst mac address using pedit (which requires that i know the mac address) Example usage: tc filter add dev eth0 parent : protocol ip pref 9 u32 \ match ip src 5.5.

Re: [BUG] act_ife: sleeping functions called in atomic context

net is better, although it is slightly large as a bug fix. I am conflicted. There are a lot of changes in net-next at the moment; adding this to -net seems like will definetely cause merge issues for Dave. Ok, Cong - patch attached and tested. Let me know what you think. cheers, jamal diff --

Re: [patch net-next v4 0/4] return offloaded stats as default and expose original sw stats

stats for slow path (non-forwarded by ASIC). Also, it's just another rtnl attr. Easy. So Jiri, I see: IFLA_SW_STATS64 should that be: IFLA_HW_STATS_LINK_64? I think IFLA_STATS_LINK_64 should continue to send s/ware stats. cheers, jamal

Re: [PATCH v2 net-next 0/3] net sched action timestamp improvements

On 16-06-18 12:31 AM, David Miller wrote: From: Jamal Hadi Salim <j...@mojatatu.com> Date: Fri, 17 Jun 2016 07:15:27 -0400 Ok, mystery solved. Dave, this patch series is missing from current net-next. Then what is: commit 53eb440f4ada034ea43b295891feec3df0fa7a29 /blink, blink sorry

Re: act_mirred: remove spinlock in fast path

point - but: as established update/replace is _a rare occassion_ ;-> cheers, jamal

Re: [patch net-next v4 0/4] return offloaded stats as default and expose original sw stats

netlink stats instead to indicate they are hardware or software? We already have a use case with the tc where someone could get/set hardware and/or software. cheers, jamal

Fwd: Re: [PATCH v2 net-next 0/3] net sched action timestamp improvements

Ok, mystery solved. Dave, this patch series is missing from current net-next. cheers, jamal Forwarded Message Subject: Re: [PATCH v2 net-next 0/3] net sched action timestamp improvements Date: Tue, 07 Jun 2016 15:54:05 -0700 (PDT) From: David Miller <da...@davemloft.net>

Re: [BUG] act_ife: sleeping functions called in atomic context

(you said it in your earlier email and I thought about that afterwards). cheers, jamal

Re: [BUG] act_ife: sleeping functions called in atomic context

-> module_put() The same problem is actual for tcf_ife_cleanup() as well. Huh? Both module_put() and kfree() should not sleep, right? I dont think there is any sleeping there ;-> cheers, jamal diff --git a/net/sched/act_ife.c b/net/sched/act_ife.c index 6bbc518..e341bef 100

Re: [PATCH net-next V2] tun: introduce tx skb ring

On 16-06-15 07:52 AM, Jamal Hadi Salim wrote: > On 16-06-15 04:38 AM, Jason Wang wrote: >> We used to queue tx packets in sk_receive_queue, this is less >> efficient since it requires spinlocks to synchronize between producer > > So this is more exercising the skb array

Re: [PATCH net-next V2] tun: introduce tx skb ring

l to see general performance numbers on user/kernel crossing (i.e tun read/write). If you have the cycles can you run such tests? cheers, jamal

Re: [PATCH net-next 00/10] net_sched: defer skb freeing while changing qdiscs

on quite fast hardware... This patch series adds a simple queue protected by RTNL where skbs can be placed until RTNL is released. Note that this might also serve in the future for optional reinjection of packets when a qdisc is replaced. Nice optimization Eric. cheers, jamal

[PATCH net 1/1] net sched actions: bug fix dumping actions directly didnt produce NLMSG_DONE

From: Jamal Hadi Salim <j...@mojatatu.com> This refers to commands to direct action access as follows: sudo tc actions add action drop index 12 sudo tc actions add action pipe index 10 And then dumping them like so: sudo tc actions ls action gact iproute2 worked because it depended on a

Re: [Patch net-next] net_sched: make tcf_hash_check() boolean

On 16-06-13 04:46 PM, Cong Wang wrote: Cc: Jamal Hadi Salim <j...@mojatatu.com> Signed-off-by: Cong Wang <xiyou.wangc...@gmail.com> Acked-by: Jamal Hadi Salim <j...@mojatatu.com> cheers, jamal

Re: [Patch net] act_ipt: fix a bind refcnt leak

On 16-06-13 04:44 PM, Cong Wang wrote: And avoid calling tcf_hash_check() twice. Fixes: a57f19d30b2d ("net sched: ipt action fix late binding") Cc: Jamal Hadi Salim <j...@mojatatu.com> Signed-off-by: Cong Wang <xiyou.wangc...@gmail.com> Good catch. Acked-b

Re: [Patch net-next] act_police: rename tcf_act_police_locate() to tcf_act_police_init()

On 16-06-13 01:47 PM, Cong Wang wrote: This function is just ->init(), rename it to make it obvious. Cc: Jamal Hadi Salim <j...@mojatatu.com> Signed-off-by: Cong Wang <xiyou.wangc...@gmail.com> Acked-by: Jamal Hadi Salim <j...@mojatatu.com> cheers, jamal

Re: [Patch net-next] net_sched: remove internal use of TC_POLICE_*

On 16-06-13 01:47 PM, Cong Wang wrote: These should be gone when we removed CONFIG_NET_CLS_POLICE. We can not totally remove them since they are exposed to userspace. Cc: Jamal Hadi Salim <j...@mojatatu.com> Signed-off-by: Cong Wang <xiyou.wangc...@gmail.com> Acked-by: Jamal H

Re: [net-next PATCH v2 1/1] net sched actions: skbedit add support for mod-ing skb pkt_type

a generic helper as in skb_pkt_type_mangle() and place it into skbuff.h? Ok, will do with next update. cheers, jamal

Re: [net-next PATCH v2 1/1] net sched actions: skbedit add support for mod-ing skb pkt_type

On 16-06-13 04:00 AM, Daniel Borkmann wrote: Hi Jamal, On 06/12/2016 11:24 PM, Jamal Hadi Salim wrote: From: Jamal Hadi Salim <h...@mojatatu.com> Extremely useful for setting packet type to host so i dont have to modify the dst mac address using pedit (which requires that i know t

[iproute2 PATCH 1/1] action pedit: stylistic changes

From: Jamal Hadi Salim <j...@mojatatu.com> More modern layout. Signed-off-by: Jamal Hadi Salim <j...@mojatatu.com> --- tc/m_pedit.c | 118 +++ 1 file changed, 62 insertions(+), 56 deletions(-) diff --git a/tc/m_pedit.c b/

[net-next PATCH v2 1/1] net sched actions: skbedit add support for mod-ing skb pkt_type

From: Jamal Hadi Salim <h...@mojatatu.com> Extremely useful for setting packet type to host so i dont have to modify the dst mac address using pedit (which requires that i know the mac address) Signed-off-by: Jamal Hadi Salim <j...@mojatatu.com> --- include/net/tc_act/tc_skbedit.h

Re: [net-next PATCH 1/1] net sched actions: skbedit add support for mod-ing skb pkt_type

On 16-06-12 05:12 PM, Jamal Hadi Salim wrote: From: Jamal Hadi Salim <j...@mojatatu.com> Extremely useful for setting packet type to host so i dont have to modify the dst mac address using pedit (which requires that i know the mac address) Signed-off-by: Jamal Hadi Salim <j...@moj

[net-next PATCH 1/1] net sched actions: skbedit add support for mod-ing skb pkt_type

From: Jamal Hadi Salim <j...@mojatatu.com> Extremely useful for setting packet type to host so i dont have to modify the dst mac address using pedit (which requires that i know the mac address) Signed-off-by: Jamal Hadi Salim <j...@mojatatu.com> --- include/net/tc_act/tc_skbedit.h

Re: [PATCH v2 net] fq_codel: fix NET_XMIT_CN behavior

On 16-06-06 12:18 PM, Florian Westphal wrote: Eric Dumazet <eric.duma...@gmail.com> wrote: On Mon, 2016-06-06 at 13:42 +0200, Florian Westphal wrote: Jamal Hadi Salim <j...@mojatatu.com> wrote: BTW, returning NET_XMIT_CN could be confusing to tcp; it does not mean that the pa

Re: [PATCH net-next] net, cls: allow for deleting all filters for given parent

ce won't get the updates (except for queries with RTM_GETTFILTER). Don't worry about any failure in tfilter_notify(), we call it in non-wildcard removal too. Your current patch looks good to me. Thanks! Daniel, I acked this already (sorry I missed the event notification bits). Acked-by: Jamal H

Re: [PATCH v2 net] fq_codel: fix NET_XMIT_CN behavior

variants of early drops); maybe codel needs one. The parent also must account for childs drops. BTW, returning NET_XMIT_CN could be confusing to tcp; it does not mean that the packet that we are getting return code for was dropped; it could mean _another_ packet in the queue was dropped. cheers, jamal

[PATCH v2 net-next 2/3] net sched actions: introduce timestamp for firsttime use

From: Jamal Hadi Salim <j...@mojatatu.com> Useful to know when the action was first used for accounting (and debugging) Signed-off-by: Jamal Hadi Salim <j...@mojatatu.com> --- include/net/act_api.h| 2 ++ include/uapi/linux/pkt_cls.h | 1 + net/sched/act_api.c | 1

[PATCH v2 net-next 1/3] net sched: actions use tcf_lastuse_update for consistency

From: Jamal Hadi Salim <j...@mojatatu.com> Signed-off-by: Jamal Hadi Salim <j...@mojatatu.com> --- net/sched/act_connmark.c | 2 +- net/sched/act_csum.c | 2 +- net/sched/act_ife.c | 6 +++--- net/sched/act_ipt.c | 2 +- net/sched/act_mirred.c | 1 - net/sched/act_na

[PATCH v2 net-next 3/3] net sched actions: aggregate dumping of actions timeinfo

From: Jamal Hadi Salim <j...@mojatatu.com> Signed-off-by: Jamal Hadi Salim <j...@mojatatu.com> --- include/net/act_api.h| 8 net/sched/act_bpf.c | 6 +- net/sched/act_connmark.c | 5 + net/sched/act_csum.c | 6 ++ net/sched/act_gact.c | 5 +---

[PATCH v2 net-next 0/3] net sched action timestamp improvements

From: Jamal Hadi Salim <j...@mojatatu.com> Various aggregations of duplicated code, fixes and introduction of firstused timestamp v2: add const for source time info per suggestion from Cong Jamal Hadi Salim (3): net sched: actions use tcf_lastuse_update for consistency net sched a

Re: [net-next PATCH 3/3] net sched actions: aggregate dumping of actions timeinfo

On 16-06-06 01:39 AM, Cong Wang wrote: On Sun, Jun 5, 2016 at 7:26 AM, Jamal Hadi Salim <j...@mojatatu.com> wrote: diff --git a/include/net/act_api.h b/include/net/act_api.h index 8389c00..5453f52 100644 --- a/include/net/act_api.h +++ b/include/net/act_api.h @@ -80,6 +80,14 @@ static

[net-next PATCH 1/1] net sched: indentation and other OCD stylistic fixes

From: Jamal Hadi Salim <j...@mojatatu.com> Signed-off-by: Jamal Hadi Salim <j...@mojatatu.com> --- include/net/act_api.h | 14 -- include/net/tc_act/tc_defact.h | 4 ++-- include/uapi/linux/pkt_cls.h | 6 +++--- net/sched/act_api.c

[net-next PATCH 0/3] net sched action timestamp improvements

From: Jamal Hadi Salim <j...@mojatatu.com> Various aggregations of duplicated code, fixes and introduction of firstused timestamp Jamal Hadi Salim (3): net sched: actions use tcf_lastuse_update for consistency net sched actions: introduce timestamp for firsttime use net sched a

[net-next PATCH 1/3] net sched: actions use tcf_lastuse_update for consistency

From: Jamal Hadi Salim <j...@mojatatu.com> Signed-off-by: Jamal Hadi Salim <j...@mojatatu.com> --- net/sched/act_connmark.c | 2 +- net/sched/act_csum.c | 2 +- net/sched/act_ife.c | 6 +++--- net/sched/act_ipt.c | 2 +- net/sched/act_mirred.c | 1 - net/sched/act_na

[net-next PATCH 3/3] net sched actions: aggregate dumping of actions timeinfo

From: Jamal Hadi Salim <j...@mojatatu.com> Signed-off-by: Jamal Hadi Salim <j...@mojatatu.com> --- include/net/act_api.h| 8 net/sched/act_bpf.c | 6 +- net/sched/act_connmark.c | 5 + net/sched/act_csum.c | 6 ++ net/sched/act_gact.c | 5 +---

[net-next PATCH 2/3] net sched actions: introduce timestamp for firsttime use

From: Jamal Hadi Salim <j...@mojatatu.com> Useful to know when the action was first used for accounting (and debugging) Signed-off-by: Jamal Hadi Salim <j...@mojatatu.com> --- include/net/act_api.h| 2 ++ include/uapi/linux/pkt_cls.h | 1 + net/sched/act_api.c | 1

[iproute2 resend PATCH 1/1] man: tc-ife.8: man page for ife action

From: Lucas Bates <luc...@mojatatu.com> Signed-off-by: Lucas Bates <luc...@mojatatu.com> Signed-off-by: Jamal Hadi Salim <j...@mojatatu.com> Acked-by: Phil Sutter <p...@nwl.cc> --- man/man8/tc-ife.8 | 117 ++ 1 file

Re: [PATCH v2 net] fq_codel: fix NET_XMIT_CN behavior

o") Reported-by: Stas Nichiporovich <stas...@gmail.com> Signed-off-by: Eric Dumazet <eduma...@google.com> Cc: WANG Cong <xiyou.wangc...@gmail.com> Cc: Jamal Hadi Salim <j...@mojatatu.com> Acked-by: Jamal Hadi Salim <j...@mojatatu.com> cheers, jamal

Re: [PATCH net-next] net, cls: allow for deleting all filters for given parent

and NLM_F_CREATE flag were allowed where the kernel would auto-generate a pref/prio. We can piggyback on that and use prio of 0 as a wildcard for requests of RTM_DELTFILTER. Signed-off-by: Daniel Borkmann <dan...@iogearbox.net> Looks good to me. Acked-by: Jamal Hadi Salim <j...@mojatatu.com

[iproute2 PATCH 1/1] tc filter u32: Coding style fixes

From: Jamal Hadi Salim <j...@mojatatu.com> "handle" was being used several times for different things. Fix the 80 character limit abuse and other little issues while at it. Signed-off-by: Jamal Hadi Salim <j...@mojatatu.com>

Re: [iproute2 PATH 2/2] tc action policer: enable timestamp display

Stephen, This requires you pull Dave's -net headers. cheers, jamal On 16-05-25 06:05 AM, Jamal Hadi Salim wrote: From: Jamal Hadi Salim <j...@mojatatu.com> Signed-off-by: Jamal Hadi Salim <j...@mojatatu.com> --- tc/m_police.c | 11 ++- 1 file changed, 10 insertions(+)

[iproute2 PATH 1/2] tc action policer: Avoid nonsensical input

From: Jamal Hadi Salim <j...@mojatatu.com> The user must at least specify a choice of the token bucket or ewma policing or late binding index. TB policing requires at minimal a rate and burst. In addition fix formatting issues (80 chars etc). Signed-off-by: Jamal Hadi Salim <j...@moj

[iproute2 PATH 2/2] tc action policer: enable timestamp display

From: Jamal Hadi Salim <j...@mojatatu.com> Signed-off-by: Jamal Hadi Salim <j...@mojatatu.com> --- tc/m_police.c | 11 ++- 1 file changed, 10 insertions(+), 1 deletion(-) diff --git a/tc/m_police.c b/tc/m_police.c index 8752d4f..cb17c9e 100644 --- a/tc/m_police.c +++ b/t

Re: [iproute2 PATCH v2 1/1] vv4.6.0

More git foo. I have no idea how this went out ;-> Please ignore this but not the other. cheers, jamal On 16-05-24 07:52 AM, Jamal Hadi Salim wrote: From: Stephen Hemminger <shemm...@brocade.com> --- include/SNAPSHOT.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) d

[iproute2 PATCH v2 1/1] tc simple action: bug fix

From: Jamal Hadi Salim <j...@mojatatu.com> Failed compile m_simple.c: In function ‘parse_simple’: m_simple.c:154:6: warning: too many arguments for format [-Wformat-extra-args] *argv); ^ m_simple.c:103:14: warning: unused variable ‘maybe_bind’ [-Wunused-variable] Reported-by:

[iproute2 PATCH v2 1/1] vv4.6.0

From: Stephen Hemminger --- include/SNAPSHOT.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/include/SNAPSHOT.h b/include/SNAPSHOT.h index 9220f77..cc67058 100644 --- a/include/SNAPSHOT.h +++ b/include/SNAPSHOT.h @@ -1 +1 @@ -static const char

Re: [iproute2 1/1] tc simple action: bug fix

On 16-05-24 06:30 AM, Daniel Borkmann wrote: On 05/24/2016 03:04 AM, Jamal Hadi Salim wrote: From: Jamal Hadi Salim <j...@mojatatu.com> Failed compile m_simple.c: In function ‘parse_simple’: m_simple.c:154:6: warning: too many arguments for format [-Wformat-extra-args]

[net PATCH v2 1/1] net sched actions: policer missing timestamp processing

From: Jamal Hadi Salim <j...@mojatatu.com> Policer was not dumping or updating timestamps Signed-off-by: Jamal Hadi Salim <j...@mojatatu.com> --- include/uapi/linux/pkt_cls.h | 4 +++- net/sched/act_police.c | 11 +++ 2 files changed, 14 insertions(+), 1 deletion(-)

[iproute2 1/1] tc simple action: bug fix

From: Jamal Hadi Salim <j...@mojatatu.com> Failed compile m_simple.c: In function ‘parse_simple’: m_simple.c:154:6: warning: too many arguments for format [-Wformat-extra-args] *argv); ^ m_simple.c:103:14: warning: unused variable ‘maybe_bind’ [-Wunused-variable] Reported-by:

Re: [iproute2 PATCH v2 1/1] tc simple action update and breakage

; ^ Will send a fix. Thanks Daniel. cheers, jamal

Re: [net PATCH 1/1] net sched actions: policer missing timestamp processing

On 16-05-23 04:52 PM, Cong Wang wrote: + tcf_tm_dump(, >tcf_tm); This depends on a previous patch you sent, right? Sigh. There goes my git foo. Will resubmit an updated version. cheers, jamal

[iproute2 PATCH 1/1] tc fix ife late binding

From: Jamal Hadi Salim <j...@mojatatu.com> following late binding didn't work sudo tc actions add action ife encode \ type 0xDEAD allow mark dst 02:15:15:15:15:15 index 1 Signed-off-by: Jamal Hadi Salim <j...@mojatatu.com> --- tc/m_ife.c | 1 + 1 file changed, 1 insertion(+) dif

Re: [iproute2 1/1] man: tc-ife.8: man page for ife action

On 16-05-02 07:40 PM, Jamal Hadi Salim wrote: On 16-05-02 06:10 PM, Stephen Hemminger wrote: On Sat, 30 Apr 2016 06:58:04 -0400 Jamal Hadi Salim <j...@mojatatu.com> wrote: From: Lucas Bates <luc...@mojatatu.com> Signed-off-by: Lucas Bates <luc...@mojatatu.com> Signed-off-by

Re: [RFC net-next] net: sched: do not acquire qdisc spinlock in qdisc/class stats dump

- hence the rcu suggestion. I would have problems with action stats being off (where they are used for billing). cheers, jamal

[net PATCH 1/1] net sched actions: policer missing timestamp processing

From: Jamal Hadi Salim <j...@mojatatu.com> Policer was not dumping or updating timestamps Signed-off-by: Jamal Hadi Salim <j...@mojatatu.com> --- include/uapi/linux/pkt_cls.h | 4 +++- net/sched/act_police.c | 10 ++ 2 files changed, 13 insertions(+), 1 deletion(-)

Re: [net-next PATCH 0/2] net sched small OCD cleanups

On 16-05-19 02:28 PM, David Miller wrote: From: Jamal Hadi Salim <j...@mojatatu.com> Date: Tue, 17 May 2016 17:19:13 -0400 I have been staring at these small OCD cleanups for a while. Got cycles to do them as prep for another patch i am working on. Indentation/stylistic and consistency u

Re: [net-next RFC 1/1] net sched actions: introduce timestamp for first time used

On 16-05-20 01:16 AM, Cong Wang wrote: One nit: It is better if you can swap 'stm' with 'dtm', it looks odd to put destination on the right side. Will make the change for when net-next opens up. cheers, jamal

Re: [RFC net-next] net: sched: do not acquire qdisc spinlock in qdisc/class stats dump

ch all counters like packets and bytes sent at the expense of interfering in fast path (queue and dequeue packets) is simply not worth the pain, as the values are generally stale after 1 usec. These lock acquisitions slow down the fast path by 10 to 20 % Acked-by: Jamal Hadi Salim <j...@moj

Re: [net-next RFC 1/1] net sched actions: introduce timestamp for first time used

On 16-05-19 07:54 AM, Jamal Hadi Salim wrote: On 16-05-18 01:17 PM, Cong Wang wrote: I think it is the time to add a wrapper for these tm.XX = jiffies_to_clock_t(XXX). Agreed. Will send a patch. Actually I will resend the firstuse as well without RFC tag and the wrapper one as separate

Re: [net-next RFC 1/1] net sched actions: introduce timestamp for first time used

On 16-05-18 01:17 PM, Cong Wang wrote: On Tue, May 17, 2016 at 4:38 PM, Jamal Hadi Salim <j...@mojatatu.com> wrote: diff --git a/net/sched/act_bpf.c b/net/sched/act_bpf.c index 014f9a6..f581e01 100644 --- a/net/sched/act_bpf.c +++ b/net/sched/act_bpf.c @@ -156,6 +156,7 @@ stat

<    3   4   5   6   7   8   9   10   11   12   >