lkml.org 
[lkml]   [2019]   [May]   [7]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRe: [net-next v3] net: sched: Introduce act_ctinfo action
Date


> On 7 May 2019, at 20:39, David Miller <davem@davemloft.net> wrote:
>
> From: Kevin 'ldir' Darbyshire-Bryant <ldir@darbyshire-bryant.me.uk>
> Date: Sun, 5 May 2019 13:20:13 +0000
>
>> ctinfo is a new tc filter action module. It is designed to restore
>> information contained in conntrack marks to other places. At present it
>> can restore DSCP values to IPv4/6 diffserv fields and also copy
>> conntrack marks to skb marks. As such the 2nd function effectively
>> replaces the existing act_connmark module

Hi David,

>
> This needs more time for review and therefore I'm deferring this to the
> next merge window.

Thank you.

>
>> +static inline int tcf_ctinfo_dump(struct sk_buff *skb, struct tc_action *a,
>> + int bind, int ref)
>> +{
>> + unsigned char *b = skb_tail_pointer(skb);
>> + struct tcf_ctinfo *ci = to_ctinfo(a);
>> + struct tcf_ctinfo_params *cp;
>> + struct tc_ctinfo opt = {
>> + .index = ci->tcf_index,
>> + .refcnt = refcount_read(&ci->tcf_refcnt) - ref,
>> + .bindcnt = atomic_read(&ci->tcf_bindcnt) - bind,
>> + };
>> + struct tcf_t t;
>> + struct tc_ctinfo_dscp dscpparm;
>> + struct tc_ctinfo_stats_dscp dscpstats;
>

All done, I’ve put struct tcf_ctinfo *ci = to_ctinfo(a); at the top though
as other variables need that initialised, hope that’s ok, else please explain
what I should have done.

> Likewise.
>
> Also, never use the inline keyword in foo.c files, always let the compiler
> decide.

The perils of using existing code, should have spotted that thought, didn’t
inline anything else ‘cos in theory I know better. Oh well.

Shame checkpatch didn’t warn me about christmas trees or inline otherwise it
wouldn’t have got this far.

Thanks for your time and patience.

Cheers,

Kevin D-B

gpg: 012C ACB2 28C6 C53E 9775 9123 B3A2 389B 9DE2 334A

\
 
 \ /
  Last update: 2019-05-07 22:19    [W:0.129 / U:0.532 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site