]> asedeno.scripts.mit.edu Git - linux.git/blobdiff - net/can/gw.c
Merge tag 'clk-for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/clk/linux
[linux.git] / net / can / gw.c
index 72711053ebe66ce8231fbb79d9c37f682071be64..65d60c93af2945e7a28fae8333db344bfcbe120b 100644 (file)
@@ -1,7 +1,7 @@
-/*
- * gw.c - CAN frame Gateway/Router/Bridge with netlink interface
+// SPDX-License-Identifier: ((GPL-2.0 WITH Linux-syscall-note) OR BSD-3-Clause)
+/* gw.c - CAN frame Gateway/Router/Bridge with netlink interface
  *
- * Copyright (c) 2017 Volkswagen Group Electronic Research
+ * Copyright (c) 2019 Volkswagen Group Electronic Research
  * All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
@@ -59,7 +59,7 @@
 #include <net/net_namespace.h>
 #include <net/sock.h>
 
-#define CAN_GW_VERSION "20170425"
+#define CAN_GW_VERSION "20190810"
 #define CAN_GW_NAME "can-gw"
 
 MODULE_DESCRIPTION("PF_CAN netlink gateway");
@@ -85,10 +85,10 @@ static struct kmem_cache *cgw_cache __read_mostly;
 /* structure that contains the (on-the-fly) CAN frame modifications */
 struct cf_mod {
        struct {
-               struct can_frame and;
-               struct can_frame or;
-               struct can_frame xor;
-               struct can_frame set;
+               struct canfd_frame and;
+               struct canfd_frame or;
+               struct canfd_frame xor;
+               struct canfd_frame set;
        } modframe;
        struct {
                u8 and;
@@ -96,7 +96,7 @@ struct cf_mod {
                u8 xor;
                u8 set;
        } modtype;
-       void (*modfunc[MAX_MODFUNCTIONS])(struct can_frame *cf,
+       void (*modfunc[MAX_MODFUNCTIONS])(struct canfd_frame *cf,
                                          struct cf_mod *mod);
 
        /* CAN frame checksum calculation after CAN frame modifications */
@@ -105,15 +105,15 @@ struct cf_mod {
                struct cgw_csum_crc8 crc8;
        } csum;
        struct {
-               void (*xor)(struct can_frame *cf, struct cgw_csum_xor *xor);
-               void (*crc8)(struct can_frame *cf, struct cgw_csum_crc8 *crc8);
+               void (*xor)(struct canfd_frame *cf,
+                           struct cgw_csum_xor *xor);
+               void (*crc8)(struct canfd_frame *cf,
+                            struct cgw_csum_crc8 *crc8);
        } csumfunc;
        u32 uid;
 };
 
-
-/*
- * So far we just support CAN -> CAN routing and frame modifications.
+/* So far we just support CAN -> CAN routing and frame modifications.
  *
  * The internal can_can_gw structure contains data and attributes for
  * a CAN -> CAN gateway job.
@@ -151,39 +151,88 @@ struct cgw_job {
 
 /* modification functions that are invoked in the hot path in can_can_gw_rcv */
 
-#define MODFUNC(func, op) static void func(struct can_frame *cf, \
+#define MODFUNC(func, op) static void func(struct canfd_frame *cf, \
                                           struct cf_mod *mod) { op ; }
 
 MODFUNC(mod_and_id, cf->can_id &= mod->modframe.and.can_id)
-MODFUNC(mod_and_dlc, cf->can_dlc &= mod->modframe.and.can_dlc)
+MODFUNC(mod_and_len, cf->len &= mod->modframe.and.len)
+MODFUNC(mod_and_flags, cf->flags &= mod->modframe.and.flags)
 MODFUNC(mod_and_data, *(u64 *)cf->data &= *(u64 *)mod->modframe.and.data)
 MODFUNC(mod_or_id, cf->can_id |= mod->modframe.or.can_id)
-MODFUNC(mod_or_dlc, cf->can_dlc |= mod->modframe.or.can_dlc)
+MODFUNC(mod_or_len, cf->len |= mod->modframe.or.len)
+MODFUNC(mod_or_flags, cf->flags |= mod->modframe.or.flags)
 MODFUNC(mod_or_data, *(u64 *)cf->data |= *(u64 *)mod->modframe.or.data)
 MODFUNC(mod_xor_id, cf->can_id ^= mod->modframe.xor.can_id)
-MODFUNC(mod_xor_dlc, cf->can_dlc ^= mod->modframe.xor.can_dlc)
+MODFUNC(mod_xor_len, cf->len ^= mod->modframe.xor.len)
+MODFUNC(mod_xor_flags, cf->flags ^= mod->modframe.xor.flags)
 MODFUNC(mod_xor_data, *(u64 *)cf->data ^= *(u64 *)mod->modframe.xor.data)
 MODFUNC(mod_set_id, cf->can_id = mod->modframe.set.can_id)
-MODFUNC(mod_set_dlc, cf->can_dlc = mod->modframe.set.can_dlc)
+MODFUNC(mod_set_len, cf->len = mod->modframe.set.len)
+MODFUNC(mod_set_flags, cf->flags = mod->modframe.set.flags)
 MODFUNC(mod_set_data, *(u64 *)cf->data = *(u64 *)mod->modframe.set.data)
 
-static inline void canframecpy(struct can_frame *dst, struct can_frame *src)
+static void mod_and_fddata(struct canfd_frame *cf, struct cf_mod *mod)
+{
+       int i;
+
+       for (i = 0; i < CANFD_MAX_DLEN; i += 8)
+               *(u64 *)(cf->data + i) &= *(u64 *)(mod->modframe.and.data + i);
+}
+
+static void mod_or_fddata(struct canfd_frame *cf, struct cf_mod *mod)
+{
+       int i;
+
+       for (i = 0; i < CANFD_MAX_DLEN; i += 8)
+               *(u64 *)(cf->data + i) |= *(u64 *)(mod->modframe.or.data + i);
+}
+
+static void mod_xor_fddata(struct canfd_frame *cf, struct cf_mod *mod)
+{
+       int i;
+
+       for (i = 0; i < CANFD_MAX_DLEN; i += 8)
+               *(u64 *)(cf->data + i) ^= *(u64 *)(mod->modframe.xor.data + i);
+}
+
+static void mod_set_fddata(struct canfd_frame *cf, struct cf_mod *mod)
+{
+       memcpy(cf->data, mod->modframe.set.data, CANFD_MAX_DLEN);
+}
+
+static void canframecpy(struct canfd_frame *dst, struct can_frame *src)
 {
-       /*
-        * Copy the struct members separately to ensure that no uninitialized
+       /* Copy the struct members separately to ensure that no uninitialized
         * data are copied in the 3 bytes hole of the struct. This is needed
         * to make easy compares of the data in the struct cf_mod.
         */
 
        dst->can_id = src->can_id;
-       dst->can_dlc = src->can_dlc;
+       dst->len = src->can_dlc;
        *(u64 *)dst->data = *(u64 *)src->data;
 }
 
-static int cgw_chk_csum_parms(s8 fr, s8 to, s8 re)
+static void canfdframecpy(struct canfd_frame *dst, struct canfd_frame *src)
 {
-       /*
-        * absolute dlc values 0 .. 7 => 0 .. 7, e.g. data [0]
+       /* Copy the struct members separately to ensure that no uninitialized
+        * data are copied in the 2 bytes hole of the struct. This is needed
+        * to make easy compares of the data in the struct cf_mod.
+        */
+
+       dst->can_id = src->can_id;
+       dst->flags = src->flags;
+       dst->len = src->len;
+       memcpy(dst->data, src->data, CANFD_MAX_DLEN);
+}
+
+static int cgw_chk_csum_parms(s8 fr, s8 to, s8 re, struct rtcanmsg *r)
+{
+       s8 dlen = CAN_MAX_DLEN;
+
+       if (r->flags & CGW_FLAGS_CAN_FD)
+               dlen = CANFD_MAX_DLEN;
+
+       /* absolute dlc values 0 .. 7 => 0 .. 7, e.g. data [0]
         * relative to received dlc -1 .. -8 :
         * e.g. for received dlc = 8
         * -1 => index = 7 (data[7])
@@ -191,27 +240,27 @@ static int cgw_chk_csum_parms(s8 fr, s8 to, s8 re)
         * -8 => index = 0 (data[0])
         */
 
-       if (fr > -9 && fr < 8 &&
-           to > -9 && to < 8 &&
-           re > -9 && re < 8)
+       if (fr >= -dlen && fr < dlen &&
+           to >= -dlen && to < dlen &&
+           re >= -dlen && re < dlen)
                return 0;
        else
                return -EINVAL;
 }
 
-static inline int calc_idx(int idx, int rx_dlc)
+static inline int calc_idx(int idx, int rx_len)
 {
        if (idx < 0)
-               return rx_dlc + idx;
+               return rx_len + idx;
        else
                return idx;
 }
 
-static void cgw_csum_xor_rel(struct can_frame *cf, struct cgw_csum_xor *xor)
+static void cgw_csum_xor_rel(struct canfd_frame *cf, struct cgw_csum_xor *xor)
 {
-       int from = calc_idx(xor->from_idx, cf->can_dlc);
-       int to = calc_idx(xor->to_idx, cf->can_dlc);
-       int res = calc_idx(xor->result_idx, cf->can_dlc);
+       int from = calc_idx(xor->from_idx, cf->len);
+       int to = calc_idx(xor->to_idx, cf->len);
+       int res = calc_idx(xor->result_idx, cf->len);
        u8 val = xor->init_xor_val;
        int i;
 
@@ -229,7 +278,7 @@ static void cgw_csum_xor_rel(struct can_frame *cf, struct cgw_csum_xor *xor)
        cf->data[res] = val;
 }
 
-static void cgw_csum_xor_pos(struct can_frame *cf, struct cgw_csum_xor *xor)
+static void cgw_csum_xor_pos(struct canfd_frame *cf, struct cgw_csum_xor *xor)
 {
        u8 val = xor->init_xor_val;
        int i;
@@ -240,7 +289,7 @@ static void cgw_csum_xor_pos(struct can_frame *cf, struct cgw_csum_xor *xor)
        cf->data[xor->result_idx] = val;
 }
 
-static void cgw_csum_xor_neg(struct can_frame *cf, struct cgw_csum_xor *xor)
+static void cgw_csum_xor_neg(struct canfd_frame *cf, struct cgw_csum_xor *xor)
 {
        u8 val = xor->init_xor_val;
        int i;
@@ -251,11 +300,12 @@ static void cgw_csum_xor_neg(struct can_frame *cf, struct cgw_csum_xor *xor)
        cf->data[xor->result_idx] = val;
 }
 
-static void cgw_csum_crc8_rel(struct can_frame *cf, struct cgw_csum_crc8 *crc8)
+static void cgw_csum_crc8_rel(struct canfd_frame *cf,
+                             struct cgw_csum_crc8 *crc8)
 {
-       int from = calc_idx(crc8->from_idx, cf->can_dlc);
-       int to = calc_idx(crc8->to_idx, cf->can_dlc);
-       int res = calc_idx(crc8->result_idx, cf->can_dlc);
+       int from = calc_idx(crc8->from_idx, cf->len);
+       int to = calc_idx(crc8->to_idx, cf->len);
+       int res = calc_idx(crc8->result_idx, cf->len);
        u8 crc = crc8->init_crc_val;
        int i;
 
@@ -264,96 +314,102 @@ static void cgw_csum_crc8_rel(struct can_frame *cf, struct cgw_csum_crc8 *crc8)
 
        if (from <= to) {
                for (i = crc8->from_idx; i <= crc8->to_idx; i++)
-                       crc = crc8->crctab[crc^cf->data[i]];
+                       crc = crc8->crctab[crc ^ cf->data[i]];
        } else {
                for (i = crc8->from_idx; i >= crc8->to_idx; i--)
-                       crc = crc8->crctab[crc^cf->data[i]];
+                       crc = crc8->crctab[crc ^ cf->data[i]];
        }
 
        switch (crc8->profile) {
-
        case CGW_CRC8PRF_1U8:
-               crc = crc8->crctab[crc^crc8->profile_data[0]];
+               crc = crc8->crctab[crc ^ crc8->profile_data[0]];
                break;
 
        case  CGW_CRC8PRF_16U8:
-               crc = crc8->crctab[crc^crc8->profile_data[cf->data[1] & 0xF]];
+               crc = crc8->crctab[crc ^ crc8->profile_data[cf->data[1] & 0xF]];
                break;
 
        case CGW_CRC8PRF_SFFID_XOR:
-               crc = crc8->crctab[crc^(cf->can_id & 0xFF)^
+               crc = crc8->crctab[crc ^ (cf->can_id & 0xFF) ^
                                   (cf->can_id >> 8 & 0xFF)];
                break;
-
        }
 
-       cf->data[crc8->result_idx] = crc^crc8->final_xor_val;
+       cf->data[crc8->result_idx] = crc ^ crc8->final_xor_val;
 }
 
-static void cgw_csum_crc8_pos(struct can_frame *cf, struct cgw_csum_crc8 *crc8)
+static void cgw_csum_crc8_pos(struct canfd_frame *cf,
+                             struct cgw_csum_crc8 *crc8)
 {
        u8 crc = crc8->init_crc_val;
        int i;
 
        for (i = crc8->from_idx; i <= crc8->to_idx; i++)
-               crc = crc8->crctab[crc^cf->data[i]];
+               crc = crc8->crctab[crc ^ cf->data[i]];
 
        switch (crc8->profile) {
-
        case CGW_CRC8PRF_1U8:
-               crc = crc8->crctab[crc^crc8->profile_data[0]];
+               crc = crc8->crctab[crc ^ crc8->profile_data[0]];
                break;
 
        case  CGW_CRC8PRF_16U8:
-               crc = crc8->crctab[crc^crc8->profile_data[cf->data[1] & 0xF]];
+               crc = crc8->crctab[crc ^ crc8->profile_data[cf->data[1] & 0xF]];
                break;
 
        case CGW_CRC8PRF_SFFID_XOR:
-               crc = crc8->crctab[crc^(cf->can_id & 0xFF)^
+               crc = crc8->crctab[crc ^ (cf->can_id & 0xFF) ^
                                   (cf->can_id >> 8 & 0xFF)];
                break;
        }
 
-       cf->data[crc8->result_idx] = crc^crc8->final_xor_val;
+       cf->data[crc8->result_idx] = crc ^ crc8->final_xor_val;
 }
 
-static void cgw_csum_crc8_neg(struct can_frame *cf, struct cgw_csum_crc8 *crc8)
+static void cgw_csum_crc8_neg(struct canfd_frame *cf,
+                             struct cgw_csum_crc8 *crc8)
 {
        u8 crc = crc8->init_crc_val;
        int i;
 
        for (i = crc8->from_idx; i >= crc8->to_idx; i--)
-               crc = crc8->crctab[crc^cf->data[i]];
+               crc = crc8->crctab[crc ^ cf->data[i]];
 
        switch (crc8->profile) {
-
        case CGW_CRC8PRF_1U8:
-               crc = crc8->crctab[crc^crc8->profile_data[0]];
+               crc = crc8->crctab[crc ^ crc8->profile_data[0]];
                break;
 
        case  CGW_CRC8PRF_16U8:
-               crc = crc8->crctab[crc^crc8->profile_data[cf->data[1] & 0xF]];
+               crc = crc8->crctab[crc ^ crc8->profile_data[cf->data[1] & 0xF]];
                break;
 
        case CGW_CRC8PRF_SFFID_XOR:
-               crc = crc8->crctab[crc^(cf->can_id & 0xFF)^
+               crc = crc8->crctab[crc ^ (cf->can_id & 0xFF) ^
                                   (cf->can_id >> 8 & 0xFF)];
                break;
        }
 
-       cf->data[crc8->result_idx] = crc^crc8->final_xor_val;
+       cf->data[crc8->result_idx] = crc ^ crc8->final_xor_val;
 }
 
 /* the receive & process & send function */
 static void can_can_gw_rcv(struct sk_buff *skb, void *data)
 {
        struct cgw_job *gwj = (struct cgw_job *)data;
-       struct can_frame *cf;
+       struct canfd_frame *cf;
        struct sk_buff *nskb;
        int modidx = 0;
 
-       /*
-        * Do not handle CAN frames routed more than 'max_hops' times.
+       /* process strictly Classic CAN or CAN FD frames */
+       if (gwj->flags & CGW_FLAGS_CAN_FD) {
+               if (skb->len != CANFD_MTU)
+                       return;
+       } else {
+               if (skb->len != CAN_MTU)
+                       return;
+       }
+
+       /* Do not handle CAN frames routed more than 'max_hops' times.
         * In general we should never catch this delimiter which is intended
         * to cover a misconfiguration protection (e.g. circular CAN routes).
         *
@@ -384,8 +440,7 @@ static void can_can_gw_rcv(struct sk_buff *skb, void *data)
            can_skb_prv(skb)->ifindex == gwj->dst.dev->ifindex)
                return;
 
-       /*
-        * clone the given skb, which has not been done in can_rcv()
+       /* clone the given skb, which has not been done in can_rcv()
         *
         * When there is at least one modification function activated,
         * we need to copy the skb as we want to modify skb->data.
@@ -410,7 +465,7 @@ static void can_can_gw_rcv(struct sk_buff *skb, void *data)
        nskb->dev = gwj->dst.dev;
 
        /* pointer to modifiable CAN frame */
-       cf = (struct can_frame *)nskb->data;
+       cf = (struct canfd_frame *)nskb->data;
 
        /* perform preprocessed modification functions if there are any */
        while (modidx < MAX_MODFUNCTIONS && gwj->mod.modfunc[modidx])
@@ -419,26 +474,22 @@ static void can_can_gw_rcv(struct sk_buff *skb, void *data)
        /* Has the CAN frame been modified? */
        if (modidx) {
                /* get available space for the processed CAN frame type */
-               int max_len = nskb->len - offsetof(struct can_frame, data);
+               int max_len = nskb->len - offsetof(struct canfd_frame, data);
 
                /* dlc may have changed, make sure it fits to the CAN frame */
-               if (cf->can_dlc > max_len)
-                       goto out_delete;
-
-               /* check for checksum updates in classic CAN length only */
-               if (gwj->mod.csumfunc.crc8) {
-                       if (cf->can_dlc > 8)
-                               goto out_delete;
-
-                       (*gwj->mod.csumfunc.crc8)(cf, &gwj->mod.csum.crc8);
+               if (cf->len > max_len) {
+                       /* delete frame due to misconfiguration */
+                       gwj->deleted_frames++;
+                       kfree_skb(nskb);
+                       return;
                }
 
-               if (gwj->mod.csumfunc.xor) {
-                       if (cf->can_dlc > 8)
-                               goto out_delete;
+               /* check for checksum updates */
+               if (gwj->mod.csumfunc.crc8)
+                       (*gwj->mod.csumfunc.crc8)(cf, &gwj->mod.csum.crc8);
 
+               if (gwj->mod.csumfunc.xor)
                        (*gwj->mod.csumfunc.xor)(cf, &gwj->mod.csum.xor);
-               }
        }
 
        /* clear the skb timestamp if not configured the other way */
@@ -450,14 +501,6 @@ static void can_can_gw_rcv(struct sk_buff *skb, void *data)
                gwj->dropped_frames++;
        else
                gwj->handled_frames++;
-
-       return;
-
- out_delete:
-       /* delete frame due to misconfiguration */
-       gwj->deleted_frames++;
-       kfree_skb(nskb);
-       return;
 }
 
 static inline int cgw_register_filter(struct net *net, struct cgw_job *gwj)
@@ -483,14 +526,12 @@ static int cgw_notifier(struct notifier_block *nb,
                return NOTIFY_DONE;
 
        if (msg == NETDEV_UNREGISTER) {
-
                struct cgw_job *gwj = NULL;
                struct hlist_node *nx;
 
                ASSERT_RTNL();
 
                hlist_for_each_entry_safe(gwj, nx, &net->can.cgw_list, list) {
-
                        if (gwj->src.dev == dev || gwj->dst.dev == dev) {
                                hlist_del(&gwj->list);
                                cgw_unregister_filter(net, gwj);
@@ -505,7 +546,6 @@ static int cgw_notifier(struct notifier_block *nb,
 static int cgw_put_job(struct sk_buff *skb, struct cgw_job *gwj, int type,
                       u32 pid, u32 seq, int flags)
 {
-       struct cgw_frame_mod mb;
        struct rtcanmsg *rtcan;
        struct nlmsghdr *nlh;
 
@@ -542,32 +582,66 @@ static int cgw_put_job(struct sk_buff *skb, struct cgw_job *gwj, int type,
                        goto cancel;
        }
 
-       if (gwj->mod.modtype.and) {
-               memcpy(&mb.cf, &gwj->mod.modframe.and, sizeof(mb.cf));
-               mb.modtype = gwj->mod.modtype.and;
-               if (nla_put(skb, CGW_MOD_AND, sizeof(mb), &mb) < 0)
-                       goto cancel;
-       }
+       if (gwj->flags & CGW_FLAGS_CAN_FD) {
+               struct cgw_fdframe_mod mb;
 
-       if (gwj->mod.modtype.or) {
-               memcpy(&mb.cf, &gwj->mod.modframe.or, sizeof(mb.cf));
-               mb.modtype = gwj->mod.modtype.or;
-               if (nla_put(skb, CGW_MOD_OR, sizeof(mb), &mb) < 0)
-                       goto cancel;
-       }
+               if (gwj->mod.modtype.and) {
+                       memcpy(&mb.cf, &gwj->mod.modframe.and, sizeof(mb.cf));
+                       mb.modtype = gwj->mod.modtype.and;
+                       if (nla_put(skb, CGW_FDMOD_AND, sizeof(mb), &mb) < 0)
+                               goto cancel;
+               }
 
-       if (gwj->mod.modtype.xor) {
-               memcpy(&mb.cf, &gwj->mod.modframe.xor, sizeof(mb.cf));
-               mb.modtype = gwj->mod.modtype.xor;
-               if (nla_put(skb, CGW_MOD_XOR, sizeof(mb), &mb) < 0)
-                       goto cancel;
-       }
+               if (gwj->mod.modtype.or) {
+                       memcpy(&mb.cf, &gwj->mod.modframe.or, sizeof(mb.cf));
+                       mb.modtype = gwj->mod.modtype.or;
+                       if (nla_put(skb, CGW_FDMOD_OR, sizeof(mb), &mb) < 0)
+                               goto cancel;
+               }
 
-       if (gwj->mod.modtype.set) {
-               memcpy(&mb.cf, &gwj->mod.modframe.set, sizeof(mb.cf));
-               mb.modtype = gwj->mod.modtype.set;
-               if (nla_put(skb, CGW_MOD_SET, sizeof(mb), &mb) < 0)
-                       goto cancel;
+               if (gwj->mod.modtype.xor) {
+                       memcpy(&mb.cf, &gwj->mod.modframe.xor, sizeof(mb.cf));
+                       mb.modtype = gwj->mod.modtype.xor;
+                       if (nla_put(skb, CGW_FDMOD_XOR, sizeof(mb), &mb) < 0)
+                               goto cancel;
+               }
+
+               if (gwj->mod.modtype.set) {
+                       memcpy(&mb.cf, &gwj->mod.modframe.set, sizeof(mb.cf));
+                       mb.modtype = gwj->mod.modtype.set;
+                       if (nla_put(skb, CGW_FDMOD_SET, sizeof(mb), &mb) < 0)
+                               goto cancel;
+               }
+       } else {
+               struct cgw_frame_mod mb;
+
+               if (gwj->mod.modtype.and) {
+                       memcpy(&mb.cf, &gwj->mod.modframe.and, sizeof(mb.cf));
+                       mb.modtype = gwj->mod.modtype.and;
+                       if (nla_put(skb, CGW_MOD_AND, sizeof(mb), &mb) < 0)
+                               goto cancel;
+               }
+
+               if (gwj->mod.modtype.or) {
+                       memcpy(&mb.cf, &gwj->mod.modframe.or, sizeof(mb.cf));
+                       mb.modtype = gwj->mod.modtype.or;
+                       if (nla_put(skb, CGW_MOD_OR, sizeof(mb), &mb) < 0)
+                               goto cancel;
+               }
+
+               if (gwj->mod.modtype.xor) {
+                       memcpy(&mb.cf, &gwj->mod.modframe.xor, sizeof(mb.cf));
+                       mb.modtype = gwj->mod.modtype.xor;
+                       if (nla_put(skb, CGW_MOD_XOR, sizeof(mb), &mb) < 0)
+                               goto cancel;
+               }
+
+               if (gwj->mod.modtype.set) {
+                       memcpy(&mb.cf, &gwj->mod.modframe.set, sizeof(mb.cf));
+                       mb.modtype = gwj->mod.modtype.set;
+                       if (nla_put(skb, CGW_MOD_SET, sizeof(mb), &mb) < 0)
+                               goto cancel;
+               }
        }
 
        if (gwj->mod.uid) {
@@ -588,7 +662,6 @@ static int cgw_put_job(struct sk_buff *skb, struct cgw_job *gwj, int type,
        }
 
        if (gwj->gwtype == CGW_TYPE_CAN_CAN) {
-
                if (gwj->ccgw.filter.can_id || gwj->ccgw.filter.can_mask) {
                        if (nla_put(skb, CGW_FILTER, sizeof(struct can_filter),
                                    &gwj->ccgw.filter) < 0)
@@ -623,8 +696,9 @@ static int cgw_dump_jobs(struct sk_buff *skb, struct netlink_callback *cb)
                if (idx < s_idx)
                        goto cont;
 
-               if (cgw_put_job(skb, gwj, RTM_NEWROUTE, NETLINK_CB(cb->skb).portid,
-                   cb->nlh->nlmsg_seq, NLM_F_MULTI) < 0)
+               if (cgw_put_job(skb, gwj, RTM_NEWROUTE,
+                               NETLINK_CB(cb->skb).portid,
+                               cb->nlh->nlmsg_seq, NLM_F_MULTI) < 0)
                        break;
 cont:
                idx++;
@@ -636,7 +710,7 @@ static int cgw_dump_jobs(struct sk_buff *skb, struct netlink_callback *cb)
        return skb->len;
 }
 
-static const struct nla_policy cgw_policy[CGW_MAX+1] = {
+static const struct nla_policy cgw_policy[CGW_MAX + 1] = {
        [CGW_MOD_AND]   = { .len = sizeof(struct cgw_frame_mod) },
        [CGW_MOD_OR]    = { .len = sizeof(struct cgw_frame_mod) },
        [CGW_MOD_XOR]   = { .len = sizeof(struct cgw_frame_mod) },
@@ -648,14 +722,18 @@ static const struct nla_policy cgw_policy[CGW_MAX+1] = {
        [CGW_FILTER]    = { .len = sizeof(struct can_filter) },
        [CGW_LIM_HOPS]  = { .type = NLA_U8 },
        [CGW_MOD_UID]   = { .type = NLA_U32 },
+       [CGW_FDMOD_AND] = { .len = sizeof(struct cgw_fdframe_mod) },
+       [CGW_FDMOD_OR]  = { .len = sizeof(struct cgw_fdframe_mod) },
+       [CGW_FDMOD_XOR] = { .len = sizeof(struct cgw_fdframe_mod) },
+       [CGW_FDMOD_SET] = { .len = sizeof(struct cgw_fdframe_mod) },
 };
 
 /* check for common and gwtype specific attributes */
 static int cgw_parse_attr(struct nlmsghdr *nlh, struct cf_mod *mod,
                          u8 gwtype, void *gwtypeattr, u8 *limhops)
 {
-       struct nlattr *tb[CGW_MAX+1];
-       struct cgw_frame_mod mb;
+       struct nlattr *tb[CGW_MAX + 1];
+       struct rtcanmsg *r = nlmsg_data(nlh);
        int modidx = 0;
        int err = 0;
 
@@ -675,87 +753,166 @@ static int cgw_parse_attr(struct nlmsghdr *nlh, struct cf_mod *mod,
        }
 
        /* check for AND/OR/XOR/SET modifications */
+       if (r->flags & CGW_FLAGS_CAN_FD) {
+               struct cgw_fdframe_mod mb;
 
-       if (tb[CGW_MOD_AND]) {
-               nla_memcpy(&mb, tb[CGW_MOD_AND], CGW_MODATTR_LEN);
+               if (tb[CGW_FDMOD_AND]) {
+                       nla_memcpy(&mb, tb[CGW_FDMOD_AND], CGW_FDMODATTR_LEN);
 
-               canframecpy(&mod->modframe.and, &mb.cf);
-               mod->modtype.and = mb.modtype;
+                       canfdframecpy(&mod->modframe.and, &mb.cf);
+                       mod->modtype.and = mb.modtype;
 
-               if (mb.modtype & CGW_MOD_ID)
-                       mod->modfunc[modidx++] = mod_and_id;
+                       if (mb.modtype & CGW_MOD_ID)
+                               mod->modfunc[modidx++] = mod_and_id;
 
-               if (mb.modtype & CGW_MOD_DLC)
-                       mod->modfunc[modidx++] = mod_and_dlc;
+                       if (mb.modtype & CGW_MOD_LEN)
+                               mod->modfunc[modidx++] = mod_and_len;
 
-               if (mb.modtype & CGW_MOD_DATA)
-                       mod->modfunc[modidx++] = mod_and_data;
-       }
+                       if (mb.modtype & CGW_MOD_FLAGS)
+                               mod->modfunc[modidx++] = mod_and_flags;
+
+                       if (mb.modtype & CGW_MOD_DATA)
+                               mod->modfunc[modidx++] = mod_and_fddata;
+               }
 
-       if (tb[CGW_MOD_OR]) {
-               nla_memcpy(&mb, tb[CGW_MOD_OR], CGW_MODATTR_LEN);
+               if (tb[CGW_FDMOD_OR]) {
+                       nla_memcpy(&mb, tb[CGW_FDMOD_OR], CGW_FDMODATTR_LEN);
 
-               canframecpy(&mod->modframe.or, &mb.cf);
-               mod->modtype.or = mb.modtype;
+                       canfdframecpy(&mod->modframe.or, &mb.cf);
+                       mod->modtype.or = mb.modtype;
 
-               if (mb.modtype & CGW_MOD_ID)
-                       mod->modfunc[modidx++] = mod_or_id;
+                       if (mb.modtype & CGW_MOD_ID)
+                               mod->modfunc[modidx++] = mod_or_id;
 
-               if (mb.modtype & CGW_MOD_DLC)
-                       mod->modfunc[modidx++] = mod_or_dlc;
+                       if (mb.modtype & CGW_MOD_LEN)
+                               mod->modfunc[modidx++] = mod_or_len;
 
-               if (mb.modtype & CGW_MOD_DATA)
-                       mod->modfunc[modidx++] = mod_or_data;
-       }
+                       if (mb.modtype & CGW_MOD_FLAGS)
+                               mod->modfunc[modidx++] = mod_or_flags;
 
-       if (tb[CGW_MOD_XOR]) {
-               nla_memcpy(&mb, tb[CGW_MOD_XOR], CGW_MODATTR_LEN);
+                       if (mb.modtype & CGW_MOD_DATA)
+                               mod->modfunc[modidx++] = mod_or_fddata;
+               }
 
-               canframecpy(&mod->modframe.xor, &mb.cf);
-               mod->modtype.xor = mb.modtype;
+               if (tb[CGW_FDMOD_XOR]) {
+                       nla_memcpy(&mb, tb[CGW_FDMOD_XOR], CGW_FDMODATTR_LEN);
 
-               if (mb.modtype & CGW_MOD_ID)
-                       mod->modfunc[modidx++] = mod_xor_id;
+                       canfdframecpy(&mod->modframe.xor, &mb.cf);
+                       mod->modtype.xor = mb.modtype;
 
-               if (mb.modtype & CGW_MOD_DLC)
-                       mod->modfunc[modidx++] = mod_xor_dlc;
+                       if (mb.modtype & CGW_MOD_ID)
+                               mod->modfunc[modidx++] = mod_xor_id;
 
-               if (mb.modtype & CGW_MOD_DATA)
-                       mod->modfunc[modidx++] = mod_xor_data;
-       }
+                       if (mb.modtype & CGW_MOD_LEN)
+                               mod->modfunc[modidx++] = mod_xor_len;
 
-       if (tb[CGW_MOD_SET]) {
-               nla_memcpy(&mb, tb[CGW_MOD_SET], CGW_MODATTR_LEN);
+                       if (mb.modtype & CGW_MOD_FLAGS)
+                               mod->modfunc[modidx++] = mod_xor_flags;
+
+                       if (mb.modtype & CGW_MOD_DATA)
+                               mod->modfunc[modidx++] = mod_xor_fddata;
+               }
 
-               canframecpy(&mod->modframe.set, &mb.cf);
-               mod->modtype.set = mb.modtype;
+               if (tb[CGW_FDMOD_SET]) {
+                       nla_memcpy(&mb, tb[CGW_FDMOD_SET], CGW_FDMODATTR_LEN);
+
+                       canfdframecpy(&mod->modframe.set, &mb.cf);
+                       mod->modtype.set = mb.modtype;
+
+                       if (mb.modtype & CGW_MOD_ID)
+                               mod->modfunc[modidx++] = mod_set_id;
+
+                       if (mb.modtype & CGW_MOD_LEN)
+                               mod->modfunc[modidx++] = mod_set_len;
+
+                       if (mb.modtype & CGW_MOD_FLAGS)
+                               mod->modfunc[modidx++] = mod_set_flags;
+
+                       if (mb.modtype & CGW_MOD_DATA)
+                               mod->modfunc[modidx++] = mod_set_fddata;
+               }
+       } else {
+               struct cgw_frame_mod mb;
 
-               if (mb.modtype & CGW_MOD_ID)
-                       mod->modfunc[modidx++] = mod_set_id;
+               if (tb[CGW_MOD_AND]) {
+                       nla_memcpy(&mb, tb[CGW_MOD_AND], CGW_MODATTR_LEN);
 
-               if (mb.modtype & CGW_MOD_DLC)
-                       mod->modfunc[modidx++] = mod_set_dlc;
+                       canframecpy(&mod->modframe.and, &mb.cf);
+                       mod->modtype.and = mb.modtype;
 
-               if (mb.modtype & CGW_MOD_DATA)
-                       mod->modfunc[modidx++] = mod_set_data;
+                       if (mb.modtype & CGW_MOD_ID)
+                               mod->modfunc[modidx++] = mod_and_id;
+
+                       if (mb.modtype & CGW_MOD_LEN)
+                               mod->modfunc[modidx++] = mod_and_len;
+
+                       if (mb.modtype & CGW_MOD_DATA)
+                               mod->modfunc[modidx++] = mod_and_data;
+               }
+
+               if (tb[CGW_MOD_OR]) {
+                       nla_memcpy(&mb, tb[CGW_MOD_OR], CGW_MODATTR_LEN);
+
+                       canframecpy(&mod->modframe.or, &mb.cf);
+                       mod->modtype.or = mb.modtype;
+
+                       if (mb.modtype & CGW_MOD_ID)
+                               mod->modfunc[modidx++] = mod_or_id;
+
+                       if (mb.modtype & CGW_MOD_LEN)
+                               mod->modfunc[modidx++] = mod_or_len;
+
+                       if (mb.modtype & CGW_MOD_DATA)
+                               mod->modfunc[modidx++] = mod_or_data;
+               }
+
+               if (tb[CGW_MOD_XOR]) {
+                       nla_memcpy(&mb, tb[CGW_MOD_XOR], CGW_MODATTR_LEN);
+
+                       canframecpy(&mod->modframe.xor, &mb.cf);
+                       mod->modtype.xor = mb.modtype;
+
+                       if (mb.modtype & CGW_MOD_ID)
+                               mod->modfunc[modidx++] = mod_xor_id;
+
+                       if (mb.modtype & CGW_MOD_LEN)
+                               mod->modfunc[modidx++] = mod_xor_len;
+
+                       if (mb.modtype & CGW_MOD_DATA)
+                               mod->modfunc[modidx++] = mod_xor_data;
+               }
+
+               if (tb[CGW_MOD_SET]) {
+                       nla_memcpy(&mb, tb[CGW_MOD_SET], CGW_MODATTR_LEN);
+
+                       canframecpy(&mod->modframe.set, &mb.cf);
+                       mod->modtype.set = mb.modtype;
+
+                       if (mb.modtype & CGW_MOD_ID)
+                               mod->modfunc[modidx++] = mod_set_id;
+
+                       if (mb.modtype & CGW_MOD_LEN)
+                               mod->modfunc[modidx++] = mod_set_len;
+
+                       if (mb.modtype & CGW_MOD_DATA)
+                               mod->modfunc[modidx++] = mod_set_data;
+               }
        }
 
        /* check for checksum operations after CAN frame modifications */
        if (modidx) {
-
                if (tb[CGW_CS_CRC8]) {
                        struct cgw_csum_crc8 *c = nla_data(tb[CGW_CS_CRC8]);
 
                        err = cgw_chk_csum_parms(c->from_idx, c->to_idx,
-                                                c->result_idx);
+                                                c->result_idx, r);
                        if (err)
                                return err;
 
                        nla_memcpy(&mod->csum.crc8, tb[CGW_CS_CRC8],
                                   CGW_CS_CRC8_LEN);
 
-                       /*
-                        * select dedicated processing function to reduce
+                       /* select dedicated processing function to reduce
                         * runtime operations in receive hot path.
                         */
                        if (c->from_idx < 0 || c->to_idx < 0 ||
@@ -771,15 +928,14 @@ static int cgw_parse_attr(struct nlmsghdr *nlh, struct cf_mod *mod,
                        struct cgw_csum_xor *c = nla_data(tb[CGW_CS_XOR]);
 
                        err = cgw_chk_csum_parms(c->from_idx, c->to_idx,
-                                                c->result_idx);
+                                                c->result_idx, r);
                        if (err)
                                return err;
 
                        nla_memcpy(&mod->csum.xor, tb[CGW_CS_XOR],
                                   CGW_CS_XOR_LEN);
 
-                       /*
-                        * select dedicated processing function to reduce
+                       /* select dedicated processing function to reduce
                         * runtime operations in receive hot path.
                         */
                        if (c->from_idx < 0 || c->to_idx < 0 ||
@@ -791,16 +947,14 @@ static int cgw_parse_attr(struct nlmsghdr *nlh, struct cf_mod *mod,
                                mod->csumfunc.xor = cgw_csum_xor_neg;
                }
 
-               if (tb[CGW_MOD_UID]) {
+               if (tb[CGW_MOD_UID])
                        nla_memcpy(&mod->uid, tb[CGW_MOD_UID], sizeof(u32));
-               }
        }
 
        if (gwtype == CGW_TYPE_CAN_CAN) {
-
                /* check CGW_TYPE_CAN_CAN specific attributes */
-
                struct can_can_gw *ccgw = (struct can_can_gw *)gwtypeattr;
+
                memset(ccgw, 0, sizeof(*ccgw));
 
                /* check for can_filter in attributes */
@@ -861,12 +1015,10 @@ static int cgw_create_job(struct sk_buff *skb,  struct nlmsghdr *nlh,
                return err;
 
        if (mod.uid) {
-
                ASSERT_RTNL();
 
                /* check for updating an existing job with identical uid */
                hlist_for_each_entry(gwj, &net->can.cgw_list, list) {
-
                        if (gwj->mod.uid != mod.uid)
                                continue;
 
@@ -987,7 +1139,6 @@ static int cgw_remove_job(struct sk_buff *skb, struct nlmsghdr *nlh,
 
        /* remove only the first matching entry */
        hlist_for_each_entry_safe(gwj, nx, &net->can.cgw_list, list) {
-
                if (gwj->flags != r->flags)
                        continue;