can: raw: add missing error queue support
authorVincent Mailhol <mailhol.vincent@wanadoo.fr>
Sat, 26 Sep 2020 16:24:31 +0000 (01:24 +0900)
committerMarc Kleine-Budde <mkl@pengutronix.de>
Tue, 6 Oct 2020 20:44:27 +0000 (22:44 +0200)
Error queue are not yet implemented in CAN-raw sockets.

The problem: a userland call to recvmsg(soc, msg, MSG_ERRQUEUE) on a
CAN-raw socket would unqueue messages from the normal queue without
any kind of error or warning. As such, it prevented CAN drivers from
using the functionalities that relies on the error queue such as
skb_tx_timestamp().

SCM_CAN_RAW_ERRQUEUE is defined as the type for the CAN raw error
queue. SCM stands for "Socket control messages". The name is inspired
from SCM_J1939_ERRQUEUE of include/uapi/linux/can/j1939.h.

Signed-off-by: Vincent Mailhol <mailhol.vincent@wanadoo.fr>
Link: https://lore.kernel.org/r/20200926162527.270030-1-mailhol.vincent@wanadoo.fr
Acked-by: Oliver Hartkopp <socketcan@hartkopp.net>
Signed-off-by: Marc Kleine-Budde <mkl@pengutronix.de>
include/uapi/linux/can/raw.h
net/can/raw.c

index 6a11d308eb5ca70842713ccbed660b1fdcf34a68..3386aa81fdf2f1f76dbc2789bd9ed2bc43c32fb1 100644 (file)
@@ -49,6 +49,9 @@
 #include <linux/can.h>
 
 #define SOL_CAN_RAW (SOL_CAN_BASE + CAN_RAW)
+enum {
+       SCM_CAN_RAW_ERRQUEUE = 1,
+};
 
 /* for socket options affecting the socket (not the global system) */
 
index 24db4b4afdc7fe43e117531ec6c425e67ea83ff0..ea70850f91529263b102b71c333c70fc1256fab4 100644 (file)
@@ -804,6 +804,10 @@ static int raw_recvmsg(struct socket *sock, struct msghdr *msg, size_t size,
        noblock = flags & MSG_DONTWAIT;
        flags &= ~MSG_DONTWAIT;
 
+       if (flags & MSG_ERRQUEUE)
+               return sock_recv_errqueue(sk, msg, size,
+                                         SOL_CAN_RAW, SCM_CAN_RAW_ERRQUEUE);
+
        skb = skb_recv_datagram(sk, flags, noblock, &err);
        if (!skb)
                return err;