isotp[server|tun]: update padding and CAN FD features

Update padding and CAN FD features according to the latest isotpsend and
isotprecv changes.

Signed-off-by: Oliver Hartkopp <socketcan@hartkopp.net>

diff --git a/isotpserver.c b/isotpserver.c
index ec4a7908d388946a0a764817a1b8729481c42234..b900f279b01dd04427974213b21710161204f0a6 100644 (file)
--- a/isotpserver.c
+++ b/isotpserver.c
@@ -77,6 +77,9 @@
 
 #define NO_CAN_ID 0xFFFFFFFFU
 
+/* allow PDUs greater 4095 bytes according ISO 15765-2:2015 */
+#define MAX_PDU_LENGTH 6000
+
 int b64hex(char *asc, unsigned char *bin, int len)
 {
        int i;
@@ -104,12 +107,12 @@ void print_usage(char *prg)
        fprintf(stderr, "isotp adressing:\n");
        fprintf(stderr, " *       -s <can_id>  (source can_id. Use 8 digits for extended IDs)\n");
        fprintf(stderr, " *       -d <can_id>  (destination can_id. Use 8 digits for extended IDs)\n");
-       fprintf(stderr, "         -x <addr>    (extended addressing mode)\n");
+       fprintf(stderr, "         -x <addr>[:<rxaddr>] (extended addressing / opt. separate rxaddr)\n");
+       fprintf(stderr, "         -L <mtu>:<tx_dl>:<tx_flags> (link layer options for CAN FD)\n");
        fprintf(stderr, "\n");
        fprintf(stderr, "padding:\n");
-       fprintf(stderr, "         -p <byte>    (set and enable tx padding byte)\n");
-       fprintf(stderr, "         -r <byte>    (set and enable rx padding byte)\n");
-       fprintf(stderr, "         -P <mode>    (check padding in SF/CF. (l)ength (c)ontent (a)ll)\n");
+       fprintf(stderr, "         -p [tx]:[rx] (set and enable tx/rx padding bytes)\n");
+       fprintf(stderr, "         -P <mode>    (check rx padding for (l)ength (c)ontent (a)ll)\n");
        fprintf(stderr, "\n");
        fprintf(stderr, "rx path: (config, which is sent to the sender / data source)\n");
        fprintf(stderr, "         -b <bs>      (blocksize. 0 = off)\n");
@@ -133,6 +136,7 @@ int main(int argc, char **argv)
        struct sockaddr_can caddr;
        static struct can_isotp_options opts;
        static struct can_isotp_fc_options fcopts;
+       static struct can_isotp_ll_options llopts;
        struct ifreq ifr;
        socklen_t sin_size = sizeof(clientaddr);
        socklen_t caddrlen = sizeof(caddr);
@@ -150,14 +154,14 @@ int main(int argc, char **argv)
 
        int idx = 0; /* index in txmsg[] */
 
-       unsigned char msg[4096];   /* isotp socket message buffer (4095 + test_for_too_long_byte)*/
-       char rxmsg[8194]; /* isotp->tcp ASCII message buffer (4095*2 + < > \n null) */
-       char txmsg[8193]; /* tcp->isotp ASCII message buffer (4095*2 + < > null) */
+       unsigned char msg[MAX_PDU_LENGTH + 1];   /* isotp socket message buffer (4095 + test_for_too_long_byte)*/
+       char rxmsg[MAX_PDU_LENGTH * 2 + 4]; /* isotp->tcp ASCII message buffer (4095*2 + < > \n null) */
+       char txmsg[MAX_PDU_LENGTH * 2 + 3]; /* tcp->isotp ASCII message buffer (4095*2 + < > null) */
 
        /* mark missing mandatory commandline options as missing */
        caddr.can_addr.tp.tx_id = caddr.can_addr.tp.rx_id = NO_CAN_ID;
 
-       while ((opt = getopt(argc, argv, "l:s:d:x:p:r:P:b:m:w:t:v?")) != -1) {
+       while ((opt = getopt(argc, argv, "l:s:d:x:p:P:b:m:w:t:L:v?")) != -1) {
                switch (opt) {
                case 'l':
                        local_port = strtoul(optarg, (char **)NULL, 10);
@@ -176,19 +180,42 @@ int main(int argc, char **argv)
                        break;
 
                case 'x':
-                       opts.flags |= CAN_ISOTP_EXTEND_ADDR;
-                       opts.ext_address = strtoul(optarg, (char **)NULL, 16) & 0xFF;
+               {
+                       int elements = sscanf(optarg, "%hhx:%hhx",
+                                             &opts.ext_address,
+                                             &opts.rx_ext_address);
+
+                       if (elements == 1)
+                               opts.flags |= CAN_ISOTP_EXTEND_ADDR;
+                       else if (elements == 2)
+                               opts.flags |= (CAN_ISOTP_EXTEND_ADDR | CAN_ISOTP_RX_EXT_ADDR);
+                       else {
+                               printf("incorrect extended addr values '%s'.\n", optarg);
+                               print_usage(basename(argv[0]));
+                               exit(0);
+                       }
                        break;
+               }
 
                case 'p':
-                       opts.flags |= CAN_ISOTP_TX_PADDING;
-                       opts.txpad_content = strtoul(optarg, (char **)NULL, 16) & 0xFF;
-                       break;
-
-               case 'r':
-                       opts.flags |= CAN_ISOTP_RX_PADDING;
-                       opts.rxpad_content = strtoul(optarg, (char **)NULL, 16) & 0xFF;
+               {
+                       int elements = sscanf(optarg, "%hhx:%hhx",
+                                             &opts.txpad_content,
+                                             &opts.rxpad_content);
+
+                       if (elements == 1)
+                               opts.flags |= CAN_ISOTP_TX_PADDING;
+                       else if (elements == 2)
+                               opts.flags |= (CAN_ISOTP_TX_PADDING | CAN_ISOTP_RX_PADDING);
+                       else if (sscanf(optarg, ":%hhx", &opts.rxpad_content) == 1)
+                               opts.flags |= CAN_ISOTP_RX_PADDING;
+                       else {
+                               printf("incorrect padding values '%s'.\n", optarg);
+                               print_usage(basename(argv[0]));
+                               exit(0);
+                       }
                        break;
+               }
 
                case 'P':
                        if (optarg[0] == 'l')
@@ -220,6 +247,17 @@ int main(int argc, char **argv)
                        opts.frame_txtime = strtoul(optarg, (char **)NULL, 10);
                        break;
 
+               case 'L':
+                       if (sscanf(optarg, "%hhu:%hhu:%hhu",
+                                  &llopts.mtu,
+                                  &llopts.tx_dl,
+                                  &llopts.tx_flags) != 3) {
+                               printf("unknown link layer options '%s'.\n", optarg);
+                               print_usage(basename(argv[0]));
+                               exit(0);
+                       }
+                       break;
+
                case 'v':
                        verbose = 1;
                        break;
@@ -299,6 +337,13 @@ int main(int argc, char **argv)
        setsockopt(sc, SOL_CAN_ISOTP, CAN_ISOTP_OPTS, &opts, sizeof(opts));
        setsockopt(sc, SOL_CAN_ISOTP, CAN_ISOTP_RECV_FC, &fcopts, sizeof(fcopts));
 
+       if (llopts.tx_dl) {
+               if (setsockopt(sc, SOL_CAN_ISOTP, CAN_ISOTP_LL_OPTS, &llopts, sizeof(llopts)) < 0) {
+                       perror("link layer sockopt");
+                       exit(1);
+               }
+       }
+
        caddr.can_family = AF_CAN;
        strcpy(ifr.ifr_name, argv[optind]);
        if (ioctl(sc, SIOCGIFINDEX, &ifr) < 0) {
@@ -323,9 +368,9 @@ int main(int argc, char **argv)
                if (FD_ISSET(sc, &readfds)) {
 
 
-                       nbytes = read(sc, &msg, 4096);
+                       nbytes = read(sc, &msg, MAX_PDU_LENGTH + 1);
 
-                       if (nbytes < 1 || nbytes > 4095) {
+                       if (nbytes < 1 || nbytes > MAX_PDU_LENGTH) {
                                perror("read from isotp socket");
                                exit(1);
                        }
@@ -360,7 +405,7 @@ int main(int argc, char **argv)
                        }
 
                        /* max len is 4095*2 + '<' + '>' = 8192. The buffer index starts with 0 */
-                       if (idx > 8191) {
+                       if (idx > MAX_PDU_LENGTH * 2 + 1) {
                                idx = 0;
                                continue;
                        }

diff --git a/isotptun.c b/isotptun.c
index 1c2e66aa682f9ff0d76760a140e08c14a8101437..c793f42015fb080c84fc88ed35fb1dbf1d04cfe7 100644 (file)
--- a/isotptun.c
+++ b/isotptun.c
@@ -68,6 +68,10 @@
 #define NO_CAN_ID 0xFFFFFFFFU
 #define DEFAULT_NAME "ctun%d"
 
+/* stay on 4095 bytes for the max. PDU length which is still much more than the standard ethernet MTU */
+#define MAX_PDU_LENGTH 4095
+#define BUF_LEN (MAX_PDU_LENGTH + 1)
+
 static volatile int running = 1;
 
 void print_usage(char *prg)
@@ -78,10 +82,10 @@ void print_usage(char *prg)
        fprintf(stderr, "Options: -s <can_id>  (source can_id. Use 8 digits for extended IDs)\n");
        fprintf(stderr, "         -d <can_id>  (destination can_id. Use 8 digits for extended IDs)\n");
        fprintf(stderr, "         -n <name>    (name of created IP netdevice. Default: '%s')\n", DEFAULT_NAME);
-       fprintf(stderr, "         -x <addr>    (extended addressing mode.)\n");
-       fprintf(stderr, "         -p <byte>    (padding byte rx path)\n");
-       fprintf(stderr, "         -q <byte>    (padding byte tx path)\n");
-       fprintf(stderr, "         -P <mode>    (check padding. (l)ength (c)ontent (a)ll)\n");
+       fprintf(stderr, "         -x <addr>[:<rxaddr>] (extended addressing / opt. separate rxaddr)\n");
+       fprintf(stderr, "         -L <mtu>:<tx_dl>:<tx_flags> (link layer options for CAN FD)\n");
+       fprintf(stderr, "         -p [tx]:[rx] (set and enable tx/rx padding bytes)\n");
+       fprintf(stderr, "         -P <mode>    (check rx padding for (l)ength (c)ontent (a)ll)\n");
        fprintf(stderr, "         -t <time ns> (transmit time in nanosecs)\n");
        fprintf(stderr, "         -b <bs>      (blocksize. 0 = off)\n");
        fprintf(stderr, "         -m <val>     (STmin in ms/ns. See spec.)\n");
@@ -107,10 +111,11 @@ int main(int argc, char **argv)
        struct ifreq ifr;
        static struct can_isotp_options opts;
        static struct can_isotp_fc_options fcopts;
+       static struct can_isotp_ll_options llopts;
        int opt, ret;
        extern int optind, opterr, optopt;
        static int verbose;
-       unsigned char buffer[4096];
+       unsigned char buffer[BUF_LEN];
        static char name[IFNAMSIZ] = DEFAULT_NAME;
        int nbytes;
 
@@ -120,7 +125,7 @@ int main(int argc, char **argv)
 
        addr.can_addr.tp.tx_id = addr.can_addr.tp.rx_id = NO_CAN_ID;
 
-       while ((opt = getopt(argc, argv, "s:d:n:x:p:q:P:t:b:m:whv?")) != -1) {
+       while ((opt = getopt(argc, argv, "s:d:n:x:p:P:t:b:m:whL:v?")) != -1) {
                switch (opt) {
                case 's':
                        addr.can_addr.tp.tx_id = strtoul(optarg, (char **)NULL, 16);
@@ -139,19 +144,42 @@ int main(int argc, char **argv)
                        break;
 
                case 'x':
-                       opts.flags |= CAN_ISOTP_EXTEND_ADDR;
-                       opts.ext_address = strtoul(optarg, (char **)NULL, 16) & 0xFF;
+               {
+                       int elements = sscanf(optarg, "%hhx:%hhx",
+                                             &opts.ext_address,
+                                             &opts.rx_ext_address);
+
+                       if (elements == 1)
+                               opts.flags |= CAN_ISOTP_EXTEND_ADDR;
+                       else if (elements == 2)
+                               opts.flags |= (CAN_ISOTP_EXTEND_ADDR | CAN_ISOTP_RX_EXT_ADDR);
+                       else {
+                               printf("incorrect extended addr values '%s'.\n", optarg);
+                               print_usage(basename(argv[0]));
+                               exit(0);
+                       }
                        break;
+               }
 
                case 'p':
-                       opts.flags |= CAN_ISOTP_RX_PADDING;
-                       opts.rxpad_content = strtoul(optarg, (char **)NULL, 16) & 0xFF;
-                       break;
-
-               case 'q':
-                       opts.flags |= CAN_ISOTP_TX_PADDING;
-                       opts.txpad_content = strtoul(optarg, (char **)NULL, 16) & 0xFF;
+               {
+                       int elements = sscanf(optarg, "%hhx:%hhx",
+                                             &opts.txpad_content,
+                                             &opts.rxpad_content);
+
+                       if (elements == 1)
+                               opts.flags |= CAN_ISOTP_TX_PADDING;
+                       else if (elements == 2)
+                               opts.flags |= (CAN_ISOTP_TX_PADDING | CAN_ISOTP_RX_PADDING);
+                       else if (sscanf(optarg, ":%hhx", &opts.rxpad_content) == 1)
+                               opts.flags |= CAN_ISOTP_RX_PADDING;
+                       else {
+                               printf("incorrect padding values '%s'.\n", optarg);
+                               print_usage(basename(argv[0]));
+                               exit(0);
+                       }
                        break;
+               }
 
                case 'P':
                        if (optarg[0] == 'l')
@@ -187,6 +215,17 @@ int main(int argc, char **argv)
                        opts.flags |= CAN_ISOTP_HALF_DUPLEX;
                        break;
 
+               case 'L':
+                       if (sscanf(optarg, "%hhu:%hhu:%hhu",
+                                  &llopts.mtu,
+                                  &llopts.tx_dl,
+                                  &llopts.tx_flags) != 3) {
+                               printf("unknown link layer options '%s'.\n", optarg);
+                               print_usage(basename(argv[0]));
+                               exit(0);
+                       }
+                       break;
+
                case 'v':
                        verbose = 1;
                        break;
@@ -219,6 +258,13 @@ int main(int argc, char **argv)
        setsockopt(s, SOL_CAN_ISOTP, CAN_ISOTP_OPTS, &opts, sizeof(opts));
        setsockopt(s, SOL_CAN_ISOTP, CAN_ISOTP_RECV_FC, &fcopts, sizeof(fcopts));
 
+       if (llopts.tx_dl) {
+               if (setsockopt(s, SOL_CAN_ISOTP, CAN_ISOTP_LL_OPTS, &llopts, sizeof(llopts)) < 0) {
+                       perror("link layer sockopt");
+                       exit(1);
+               }
+       }
+
        addr.can_family = AF_CAN;
        strcpy(ifr.ifr_name, argv[optind]);
        ioctl(s, SIOCGIFINDEX, &ifr);
@@ -260,12 +306,12 @@ int main(int argc, char **argv)
                }
 
                if (FD_ISSET(s, &rdfs)) {
-                       nbytes = read(s, buffer, 4096);
+                       nbytes = read(s, buffer, BUF_LEN);
                        if (nbytes < 0) {
                                perror("read isotp socket");
                                return -1;
                        }
-                       if (nbytes > 4095)
+                       if (nbytes > MAX_PDU_LENGTH)
                                return -1;
                        ret = write(t, buffer, nbytes);
                        if (verbose) {
@@ -278,12 +324,12 @@ int main(int argc, char **argv)
                }
 
                if (FD_ISSET(t, &rdfs)) {
-                       nbytes = read(t, buffer, 4096);
+                       nbytes = read(t, buffer, BUF_LEN);
                        if (nbytes < 0) {
                                perror("read tunfd");
                                return -1;
                        }
-                       if (nbytes > 4095)
+                       if (nbytes > MAX_PDU_LENGTH)
                                return -1;
                        ret = write(s, buffer, nbytes);
                        if (verbose) {