[NET] NETNS: Omit net_device->nd_net without CONFIG_NET_NS.
[safe/jmp/linux-2.6] / net / tipc / socket.c
index a1f2210..3220d5c 100644 (file)
@@ -1,8 +1,8 @@
 /*
  * net/tipc/socket.c: TIPC socket API
- * 
- * Copyright (c) 2001-2006, Ericsson AB
- * Copyright (c) 2004-2005, Wind River Systems
+ *
+ * Copyright (c) 2001-2007, Ericsson AB
+ * Copyright (c) 2004-2007, Wind River Systems
  * All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
@@ -43,7 +43,7 @@
 #include <linux/slab.h>
 #include <linux/poll.h>
 #include <linux/fcntl.h>
-#include <asm/semaphore.h>
+#include <linux/mutex.h>
 #include <asm/string.h>
 #include <asm/atomic.h>
 #include <net/sock.h>
@@ -63,7 +63,7 @@
 struct tipc_sock {
        struct sock sk;
        struct tipc_port *p;
-       struct semaphore sem;
+       struct mutex lock;
 };
 
 #define tipc_sk(sk) ((struct tipc_sock*)sk)
@@ -71,9 +71,9 @@ struct tipc_sock {
 static u32 dispatch(struct tipc_port *tport, struct sk_buff *buf);
 static void wakeupdispatch(struct tipc_port *tport);
 
-static struct proto_ops packet_ops;
-static struct proto_ops stream_ops;
-static struct proto_ops msg_ops;
+static const struct proto_ops packet_ops;
+static const struct proto_ops stream_ops;
+static const struct proto_ops msg_ops;
 
 static struct proto tipc_proto;
 
@@ -82,29 +82,29 @@ static int sockets_enabled = 0;
 static atomic_t tipc_queue_size = ATOMIC_INIT(0);
 
 
-/* 
- * sock_lock(): Lock a port/socket pair. lock_sock() can 
- * not be used here, since the same lock must protect ports 
+/*
+ * sock_lock(): Lock a port/socket pair. lock_sock() can
+ * not be used here, since the same lock must protect ports
  * with non-socket interfaces.
  * See net.c for description of locking policy.
  */
 static void sock_lock(struct tipc_sock* tsock)
 {
-        spin_lock_bh(tsock->p->lock);       
+       spin_lock_bh(tsock->p->lock);
 }
 
-/* 
+/*
  * sock_unlock(): Unlock a port/socket pair
  */
 static void sock_unlock(struct tipc_sock* tsock)
 {
-        spin_unlock_bh(tsock->p->lock);
+       spin_unlock_bh(tsock->p->lock);
 }
 
 /**
  * pollmask - determine the current set of poll() events for a socket
  * @sock: socket structure
- * 
+ *
  * TIPC sets the returned events as follows:
  * a) POLLRDNORM and POLLIN are set if the socket's receive queue is non-empty
  *    or if a connection-oriented socket is does not have an active connection
@@ -115,7 +115,7 @@ static void sock_unlock(struct tipc_sock* tsock)
  *
  * IMPORTANT: The fact that a read or write operation will not block does NOT
  * imply that the operation will succeed!
- * 
+ *
  * Returns pollmask value
  */
 
@@ -130,7 +130,7 @@ static u32 pollmask(struct socket *sock)
        else
                mask = 0;
 
-       if (sock->state == SS_DISCONNECTING) 
+       if (sock->state == SS_DISCONNECTING)
                mask |= POLLHUP;
        else
                mask |= POLLOUT;
@@ -146,9 +146,9 @@ static u32 pollmask(struct socket *sock)
 
 static void advance_queue(struct tipc_sock *tsock)
 {
-        sock_lock(tsock);
+       sock_lock(tsock);
        buf_discard(skb_dequeue(&tsock->sk.sk_receive_queue));
-        sock_unlock(tsock);
+       sock_unlock(tsock);
        atomic_dec(&tipc_queue_size);
 }
 
@@ -156,18 +156,21 @@ static void advance_queue(struct tipc_sock *tsock)
  * tipc_create - create a TIPC socket
  * @sock: pre-allocated socket structure
  * @protocol: protocol indicator (must be 0)
- * 
+ *
  * This routine creates and attaches a 'struct sock' to the 'struct socket',
  * then create and attaches a TIPC port to the 'struct sock' part.
  *
  * Returns 0 on success, errno otherwise
  */
-static int tipc_create(struct socket *sock, int protocol)
+static int tipc_create(struct net *net, struct socket *sock, int protocol)
 {
        struct tipc_sock *tsock;
        struct tipc_port *port;
        struct sock *sk;
-        u32 ref;
+       u32 ref;
+
+       if (net != &init_net)
+               return -EAFNOSUPPORT;
 
        if (unlikely(protocol != 0))
                return -EPROTONOSUPPORT;
@@ -198,7 +201,7 @@ static int tipc_create(struct socket *sock, int protocol)
                return -EPROTOTYPE;
        }
 
-       sk = sk_alloc(AF_TIPC, GFP_KERNEL, &tipc_proto, 1);
+       sk = sk_alloc(net, AF_TIPC, GFP_KERNEL, &tipc_proto);
        if (!sk) {
                tipc_deleteport(ref);
                return -ENOMEM;
@@ -214,7 +217,7 @@ static int tipc_create(struct socket *sock, int protocol)
        tsock->p = port;
        port->usr_handle = tsock;
 
-       init_MUTEX(&tsock->sem);
+       mutex_init(&tsock->lock);
 
        dbg("sock_create: %x\n",tsock);
 
@@ -232,7 +235,7 @@ static int tipc_create(struct socket *sock, int protocol)
  * For SEQPACKET and STREAM socket types, the first message is rejected
  * and any others are discarded.  (If the first message on a STREAM socket
  * is partially-read, it is discarded and the next one is rejected instead.)
- * 
+ *
  * NOTE: Rejected messages are not necessarily returned to the sender!  They
  * are returned or discarded according to the "destination droppable" setting
  * specified for the message by the sender.
@@ -247,15 +250,15 @@ static int release(struct socket *sock)
        int res = TIPC_OK;
        struct sk_buff *buf;
 
-        dbg("sock_delete: %x\n",tsock);
+       dbg("sock_delete: %x\n",tsock);
        if (!tsock)
                return 0;
-       down_interruptible(&tsock->sem);
+       mutex_lock(&tsock->lock);
        if (!sock->sk) {
-               up(&tsock->sem);
+               mutex_unlock(&tsock->lock);
                return 0;
        }
-       
+
        /* Reject unreceived messages, unless no longer connected */
 
        while (sock->state != SS_DISCONNECTING) {
@@ -285,11 +288,11 @@ static int release(struct socket *sock)
                atomic_dec(&tipc_queue_size);
        }
 
-       up(&tsock->sem);
+       mutex_unlock(&tsock->lock);
 
        sock_put(sk);
 
-        atomic_dec(&tipc_user_count);
+       atomic_dec(&tipc_user_count);
        return res;
 }
 
@@ -298,11 +301,11 @@ static int release(struct socket *sock)
  * @sock: socket structure
  * @uaddr: socket address describing name(s) and desired operation
  * @uaddr_len: size of socket address data structure
- * 
+ *
  * Name and name sequence binding is indicated using a positive scope value;
  * a negative scope value unbinds the specified name.  Specifying no name
  * (i.e. a socket address length of 0) unbinds all names from the socket.
- * 
+ *
  * Returns 0 on success, errno otherwise
  */
 
@@ -312,9 +315,9 @@ static int bind(struct socket *sock, struct sockaddr *uaddr, int uaddr_len)
        struct sockaddr_tipc *addr = (struct sockaddr_tipc *)uaddr;
        int res;
 
-       if (down_interruptible(&tsock->sem))
+       if (mutex_lock_interruptible(&tsock->lock))
                return -ERESTARTSYS;
-       
+
        if (unlikely(!uaddr_len)) {
                res = tipc_withdraw(tsock->p->ref, 0, NULL);
                goto exit;
@@ -335,36 +338,36 @@ static int bind(struct socket *sock, struct sockaddr *uaddr, int uaddr_len)
                res = -EAFNOSUPPORT;
                goto exit;
        }
-        
-               if (addr->scope > 0)
+
+       if (addr->scope > 0)
                res = tipc_publish(tsock->p->ref, addr->scope,
                                   &addr->addr.nameseq);
        else
                res = tipc_withdraw(tsock->p->ref, -addr->scope,
                                    &addr->addr.nameseq);
 exit:
-       up(&tsock->sem);
+       mutex_unlock(&tsock->lock);
        return res;
 }
 
-/** 
+/**
  * get_name - get port ID of socket or peer socket
  * @sock: socket structure
  * @uaddr: area for returned socket address
  * @uaddr_len: area for returned length of socket address
  * @peer: 0 to obtain socket name, 1 to obtain peer socket name
- * 
+ *
  * Returns 0 on success, errno otherwise
  */
 
-static int get_name(struct socket *sock, struct sockaddr *uaddr, 
+static int get_name(struct socket *sock, struct sockaddr *uaddr,
                    int *uaddr_len, int peer)
 {
        struct tipc_sock *tsock = tipc_sk(sock->sk);
        struct sockaddr_tipc *addr = (struct sockaddr_tipc *)uaddr;
        u32 res;
 
-       if (down_interruptible(&tsock->sem))
+       if (mutex_lock_interruptible(&tsock->lock))
                return -ERESTARTSYS;
 
        *uaddr_len = sizeof(*addr);
@@ -377,7 +380,7 @@ static int get_name(struct socket *sock, struct sockaddr *uaddr,
                res = tipc_ownidentity(tsock->p->ref, &addr->addr.id);
        addr->addr.name.domain = 0;
 
-       up(&tsock->sem);
+       mutex_unlock(&tsock->lock);
        return res;
 }
 
@@ -390,7 +393,7 @@ static int get_name(struct socket *sock, struct sockaddr *uaddr,
  * Returns the pollmask
  */
 
-static unsigned int poll(struct file *file, struct socket *sock, 
+static unsigned int poll(struct file *file, struct socket *sock,
                         poll_table *wait)
 {
        poll_wait(file, sock->sk->sk_sleep, wait);
@@ -398,14 +401,14 @@ static unsigned int poll(struct file *file, struct socket *sock,
        return pollmask(sock);
 }
 
-/** 
+/**
  * dest_name_check - verify user is permitted to send to specified port name
  * @dest: destination address
  * @m: descriptor for message to be sent
- * 
+ *
  * Prevents restricted configuration commands from being issued by
  * unauthorized users.
- * 
+ *
  * Returns 0 if permission is granted, otherwise errno
  */
 
@@ -413,19 +416,19 @@ static int dest_name_check(struct sockaddr_tipc *dest, struct msghdr *m)
 {
        struct tipc_cfg_msg_hdr hdr;
 
-        if (likely(dest->addr.name.name.type >= TIPC_RESERVED_TYPES))
-                return 0;
-        if (likely(dest->addr.name.name.type == TIPC_TOP_SRV))
-                return 0;
+       if (likely(dest->addr.name.name.type >= TIPC_RESERVED_TYPES))
+               return 0;
+       if (likely(dest->addr.name.name.type == TIPC_TOP_SRV))
+               return 0;
 
-        if (likely(dest->addr.name.name.type != TIPC_CFG_SRV))
-                return -EACCES;
+       if (likely(dest->addr.name.name.type != TIPC_CFG_SRV))
+               return -EACCES;
 
-        if (copy_from_user(&hdr, m->msg_iov[0].iov_base, sizeof(hdr)))
+       if (copy_from_user(&hdr, m->msg_iov[0].iov_base, sizeof(hdr)))
                return -EFAULT;
        if ((ntohs(hdr.tcm_type) & 0xC000) && (!capable(CAP_NET_ADMIN)))
                return -EACCES;
-        
+
        return 0;
 }
 
@@ -435,12 +438,12 @@ static int dest_name_check(struct sockaddr_tipc *dest, struct msghdr *m)
  * @sock: socket structure
  * @m: message to send
  * @total_len: length of message
- * 
+ *
  * Message must have an destination specified explicitly.
- * Used for SOCK_RDM and SOCK_DGRAM messages, 
+ * Used for SOCK_RDM and SOCK_DGRAM messages,
  * and for 'SYN' messages on SOCK_SEQPACKET and SOCK_STREAM connections.
  * (Note: 'SYN+' is prohibited on SOCK_STREAM.)
- * 
+ *
  * Returns the number of bytes sent on success, or errno otherwise
  */
 
@@ -448,14 +451,15 @@ static int send_msg(struct kiocb *iocb, struct socket *sock,
                    struct msghdr *m, size_t total_len)
 {
        struct tipc_sock *tsock = tipc_sk(sock->sk);
-        struct sockaddr_tipc *dest = (struct sockaddr_tipc *)m->msg_name;
+       struct sockaddr_tipc *dest = (struct sockaddr_tipc *)m->msg_name;
        struct sk_buff *buf;
        int needs_conn;
        int res = -EINVAL;
 
        if (unlikely(!dest))
                return -EDESTADDRREQ;
-       if (unlikely(dest->family != AF_TIPC))
+       if (unlikely((m->msg_namelen < sizeof(*dest)) ||
+                    (dest->family != AF_TIPC)))
                return -EINVAL;
 
        needs_conn = (sock->state != SS_READY);
@@ -473,7 +477,7 @@ static int send_msg(struct kiocb *iocb, struct socket *sock,
                }
        }
 
-       if (down_interruptible(&tsock->sem))
+       if (mutex_lock_interruptible(&tsock->lock))
                return -ERESTARTSYS;
 
        if (needs_conn) {
@@ -488,61 +492,61 @@ static int send_msg(struct kiocb *iocb, struct socket *sock,
                sock->state = SS_CONNECTING;
        }
 
-        do {
-                if (dest->addrtype == TIPC_ADDR_NAME) {
-                        if ((res = dest_name_check(dest, m)))
-                                goto exit;
-                        res = tipc_send2name(tsock->p->ref,
-                                             &dest->addr.name.name,
-                                             dest->addr.name.domain, 
-                                             m->msg_iovlen,
-                                             m->msg_iov);
-                }
-                else if (dest->addrtype == TIPC_ADDR_ID) {
-                        res = tipc_send2port(tsock->p->ref,
-                                             &dest->addr.id,
-                                             m->msg_iovlen,
-                                             m->msg_iov);
-                }
-                else if (dest->addrtype == TIPC_ADDR_MCAST) {
+       do {
+               if (dest->addrtype == TIPC_ADDR_NAME) {
+                       if ((res = dest_name_check(dest, m)))
+                               goto exit;
+                       res = tipc_send2name(tsock->p->ref,
+                                            &dest->addr.name.name,
+                                            dest->addr.name.domain,
+                                            m->msg_iovlen,
+                                            m->msg_iov);
+               }
+               else if (dest->addrtype == TIPC_ADDR_ID) {
+                       res = tipc_send2port(tsock->p->ref,
+                                            &dest->addr.id,
+                                            m->msg_iovlen,
+                                            m->msg_iov);
+               }
+               else if (dest->addrtype == TIPC_ADDR_MCAST) {
                        if (needs_conn) {
                                res = -EOPNOTSUPP;
                                goto exit;
                        }
-                        if ((res = dest_name_check(dest, m)))
-                                goto exit;
-                        res = tipc_multicast(tsock->p->ref,
-                                             &dest->addr.nameseq,
-                                             0,
-                                             m->msg_iovlen,
-                                             m->msg_iov);
-                }
-                if (likely(res != -ELINKCONG)) {
-exit:                                
-                        up(&tsock->sem);
-                        return res;
-                }
+                       if ((res = dest_name_check(dest, m)))
+                               goto exit;
+                       res = tipc_multicast(tsock->p->ref,
+                                            &dest->addr.nameseq,
+                                            0,
+                                            m->msg_iovlen,
+                                            m->msg_iov);
+               }
+               if (likely(res != -ELINKCONG)) {
+exit:
+                       mutex_unlock(&tsock->lock);
+                       return res;
+               }
                if (m->msg_flags & MSG_DONTWAIT) {
                        res = -EWOULDBLOCK;
                        goto exit;
                }
-                if (wait_event_interruptible(*sock->sk->sk_sleep,
-                                             !tsock->p->congested)) {
-                    res = -ERESTARTSYS;
-                    goto exit;
-                }
-        } while (1);
+               if (wait_event_interruptible(*sock->sk->sk_sleep,
+                                            !tsock->p->congested)) {
+                   res = -ERESTARTSYS;
+                   goto exit;
+               }
+       } while (1);
 }
 
-/** 
+/**
  * send_packet - send a connection-oriented message
  * @iocb: (unused)
  * @sock: socket structure
  * @m: message to send
  * @total_len: length of message
- * 
+ *
  * Used for SOCK_SEQPACKET messages and SOCK_STREAM data.
- * 
+ *
  * Returns the number of bytes sent on success, or errno otherwise
  */
 
@@ -550,7 +554,7 @@ static int send_packet(struct kiocb *iocb, struct socket *sock,
                       struct msghdr *m, size_t total_len)
 {
        struct tipc_sock *tsock = tipc_sk(sock->sk);
-        struct sockaddr_tipc *dest = (struct sockaddr_tipc *)m->msg_name;
+       struct sockaddr_tipc *dest = (struct sockaddr_tipc *)m->msg_name;
        int res;
 
        /* Handle implied connection establishment */
@@ -558,47 +562,47 @@ static int send_packet(struct kiocb *iocb, struct socket *sock,
        if (unlikely(dest))
                return send_msg(iocb, sock, m, total_len);
 
-       if (down_interruptible(&tsock->sem)) {
+       if (mutex_lock_interruptible(&tsock->lock)) {
                return -ERESTARTSYS;
-        }
+       }
 
-        do {
+       do {
                if (unlikely(sock->state != SS_CONNECTED)) {
                        if (sock->state == SS_DISCONNECTING)
-                               res = -EPIPE;   
+                               res = -EPIPE;
                        else
                                res = -ENOTCONN;
                        goto exit;
                }
 
-                res = tipc_send(tsock->p->ref, m->msg_iovlen, m->msg_iov);
-                if (likely(res != -ELINKCONG)) {
+               res = tipc_send(tsock->p->ref, m->msg_iovlen, m->msg_iov);
+               if (likely(res != -ELINKCONG)) {
 exit:
-                        up(&tsock->sem);
-                        return res;
-                }
+                       mutex_unlock(&tsock->lock);
+                       return res;
+               }
                if (m->msg_flags & MSG_DONTWAIT) {
                        res = -EWOULDBLOCK;
                        goto exit;
                }
-                if (wait_event_interruptible(*sock->sk->sk_sleep,
-                                             !tsock->p->congested)) {
-                    res = -ERESTARTSYS;
-                    goto exit;
-                }
-        } while (1);
+               if (wait_event_interruptible(*sock->sk->sk_sleep,
+                                            !tsock->p->congested)) {
+                   res = -ERESTARTSYS;
+                   goto exit;
+               }
+       } while (1);
 }
 
-/** 
+/**
  * send_stream - send stream-oriented data
  * @iocb: (unused)
  * @sock: socket structure
  * @m: data to send
  * @total_len: total length of data to be sent
- * 
+ *
  * Used for SOCK_STREAM data.
- * 
- * Returns the number of bytes sent on success (or partial success), 
+ *
+ * Returns the number of bytes sent on success (or partial success),
  * or errno if no data sent
  */
 
@@ -606,32 +610,36 @@ exit:
 static int send_stream(struct kiocb *iocb, struct socket *sock,
                       struct msghdr *m, size_t total_len)
 {
+       struct tipc_port *tport;
        struct msghdr my_msg;
        struct iovec my_iov;
        struct iovec *curr_iov;
        int curr_iovlen;
        char __user *curr_start;
+       u32 hdr_size;
        int curr_left;
        int bytes_to_send;
        int bytes_sent;
        int res;
-       
-       if (likely(total_len <= TIPC_MAX_USER_MSG_SIZE))
-               return send_packet(iocb, sock, m, total_len);
 
-       /* Can only send large data streams if already connected */
+       /* Handle special cases where there is no connection */
+
+       if (unlikely(sock->state != SS_CONNECTED)) {
+               if (sock->state == SS_UNCONNECTED)
+                       return send_packet(iocb, sock, m, total_len);
+               else if (sock->state == SS_DISCONNECTING)
+                       return -EPIPE;
+               else
+                       return -ENOTCONN;
+       }
 
-        if (unlikely(sock->state != SS_CONNECTED)) {
-                if (sock->state == SS_DISCONNECTING)
-                        return -EPIPE;   
-                else
-                        return -ENOTCONN;
-        }
+       if (unlikely(m->msg_name))
+               return -EISCONN;
 
-       /* 
+       /*
         * Send each iovec entry using one or more messages
         *
-        * Note: This algorithm is good for the most likely case 
+        * Note: This algorithm is good for the most likely case
         * (i.e. one large iovec entry), but could be improved to pass sets
         * of small iovec entries into send_packet().
         */
@@ -640,19 +648,29 @@ static int send_stream(struct kiocb *iocb, struct socket *sock,
        curr_iovlen = m->msg_iovlen;
        my_msg.msg_iov = &my_iov;
        my_msg.msg_iovlen = 1;
+       my_msg.msg_flags = m->msg_flags;
+       my_msg.msg_name = NULL;
        bytes_sent = 0;
 
+       tport = tipc_sk(sock->sk)->p;
+       hdr_size = msg_hdr_sz(&tport->phdr);
+
        while (curr_iovlen--) {
                curr_start = curr_iov->iov_base;
                curr_left = curr_iov->iov_len;
 
                while (curr_left) {
-                       bytes_to_send = (curr_left < TIPC_MAX_USER_MSG_SIZE)
-                               ? curr_left : TIPC_MAX_USER_MSG_SIZE;
+                       bytes_to_send = tport->max_pkt - hdr_size;
+                       if (bytes_to_send > TIPC_MAX_USER_MSG_SIZE)
+                               bytes_to_send = TIPC_MAX_USER_MSG_SIZE;
+                       if (curr_left < bytes_to_send)
+                               bytes_to_send = curr_left;
                        my_iov.iov_base = curr_start;
                        my_iov.iov_len = bytes_to_send;
-                        if ((res = send_packet(iocb, sock, &my_msg, 0)) < 0) {
-                               return bytes_sent ? bytes_sent : res;
+                       if ((res = send_packet(iocb, sock, &my_msg, 0)) < 0) {
+                               if (bytes_sent != 0)
+                                       res = bytes_sent;
+                               return res;
                        }
                        curr_left -= bytes_to_send;
                        curr_start += bytes_to_send;
@@ -670,11 +688,11 @@ static int send_stream(struct kiocb *iocb, struct socket *sock,
  * @sock: socket structure
  * @tsock: TIPC-specific socket structure
  * @msg: peer's response message
- * 
+ *
  * Returns 0 on success, errno otherwise
  */
 
-static int auto_connect(struct socket *sock, struct tipc_sock *tsock, 
+static int auto_connect(struct socket *sock, struct tipc_sock *tsock,
                        struct tipc_msg *msg)
 {
        struct tipc_portid peer;
@@ -696,15 +714,15 @@ static int auto_connect(struct socket *sock, struct tipc_sock *tsock,
  * set_orig_addr - capture sender's address for received message
  * @m: descriptor for message info
  * @msg: received message header
- * 
+ *
  * Note: Address is not captured if not requested by receiver.
  */
 
 static void set_orig_addr(struct msghdr *m, struct tipc_msg *msg)
 {
-        struct sockaddr_tipc *addr = (struct sockaddr_tipc *)m->msg_name;
+       struct sockaddr_tipc *addr = (struct sockaddr_tipc *)m->msg_name;
 
-        if (addr) {
+       if (addr) {
                addr->family = AF_TIPC;
                addr->addrtype = TIPC_ADDR_ID;
                addr->addr.id.ref = msg_origport(msg);
@@ -716,13 +734,13 @@ static void set_orig_addr(struct msghdr *m, struct tipc_msg *msg)
 }
 
 /**
- * anc_data_recv - optionally capture ancillary data for received message 
+ * anc_data_recv - optionally capture ancillary data for received message
  * @m: descriptor for message info
  * @msg: received message header
  * @tport: TIPC port associated with message
- * 
+ *
  * Note: Ancillary data is not captured if not requested by receiver.
- * 
+ *
  * Returns 0 if successful, otherwise errno
  */
 
@@ -744,10 +762,10 @@ static int anc_data_recv(struct msghdr *m, struct tipc_msg *msg,
        if (unlikely(err)) {
                anc_data[0] = err;
                anc_data[1] = msg_data_sz(msg);
-               if ((res = put_cmsg(m, SOL_SOCKET, TIPC_ERRINFO, 8, anc_data)))
+               if ((res = put_cmsg(m, SOL_TIPC, TIPC_ERRINFO, 8, anc_data)))
                        return res;
                if (anc_data[1] &&
-                   (res = put_cmsg(m, SOL_SOCKET, TIPC_RETDATA, anc_data[1], 
+                   (res = put_cmsg(m, SOL_TIPC, TIPC_RETDATA, anc_data[1],
                                    msg_data(msg))))
                        return res;
        }
@@ -778,19 +796,19 @@ static int anc_data_recv(struct msghdr *m, struct tipc_msg *msg,
                has_name = 0;
        }
        if (has_name &&
-           (res = put_cmsg(m, SOL_SOCKET, TIPC_DESTNAME, 12, anc_data)))
+           (res = put_cmsg(m, SOL_TIPC, TIPC_DESTNAME, 12, anc_data)))
                return res;
 
        return 0;
 }
 
-/** 
+/**
  * recv_msg - receive packet-oriented message
  * @iocb: (unused)
  * @m: descriptor for message info
  * @buf_len: total size of user buffer area
  * @flags: receive flags
- * 
+ *
  * Used for SOCK_DGRAM, SOCK_RDM, and SOCK_SEQPACKET messages.
  * If the complete message doesn't fit in user area, truncate it.
  *
@@ -821,14 +839,14 @@ static int recv_msg(struct kiocb *iocb, struct socket *sock,
        if (sock->type == SOCK_SEQPACKET) {
                if (unlikely(sock->state == SS_UNCONNECTED))
                        return -ENOTCONN;
-               if (unlikely((sock->state == SS_DISCONNECTING) && 
+               if (unlikely((sock->state == SS_DISCONNECTING) &&
                             (skb_queue_len(&sock->sk->sk_receive_queue) == 0)))
-                       return -ENOTCONN;
+                       return -ENOTCONN;
        }
 
        /* Look for a message in receive queue; wait if necessary */
 
-       if (unlikely(down_interruptible(&tsock->sem)))
+       if (unlikely(mutex_lock_interruptible(&tsock->lock)))
                return -ERESTARTSYS;
 
 restart:
@@ -839,7 +857,7 @@ restart:
        }
 
        if ((res = wait_event_interruptible(
-               *sock->sk->sk_sleep, 
+               *sock->sk->sk_sleep,
                ((q_len = skb_queue_len(&sock->sk->sk_receive_queue)) ||
                 (sock->state == SS_DISCONNECTING))) )) {
                goto exit;
@@ -884,7 +902,7 @@ restart:
                goto exit;
 
        /* Capture message data (if valid) & compute return value (always) */
-       
+
        if (!err) {
                if (unlikely(buf_len < sz)) {
                        sz = buf_len;
@@ -907,23 +925,23 @@ restart:
        /* Consume received message (optional) */
 
        if (likely(!(flags & MSG_PEEK))) {
-                if (unlikely(++tsock->p->conn_unacked >= TIPC_FLOW_CONTROL_WIN))
-                        tipc_acknowledge(tsock->p->ref, tsock->p->conn_unacked);
+               if (unlikely(++tsock->p->conn_unacked >= TIPC_FLOW_CONTROL_WIN))
+                       tipc_acknowledge(tsock->p->ref, tsock->p->conn_unacked);
                advance_queue(tsock);
-        }
+       }
 exit:
-       up(&tsock->sem);
+       mutex_unlock(&tsock->lock);
        return res;
 }
 
-/** 
+/**
  * recv_stream - receive stream-oriented data
  * @iocb: (unused)
  * @m: descriptor for message info
  * @buf_len: total size of user buffer area
  * @flags: receive flags
- * 
- * Used for SOCK_STREAM messages only.  If not enough data is available 
+ *
+ * Used for SOCK_STREAM messages only.  If not enough data is available
  * will optionally wait for more; never truncates data.
  *
  * Returns size of returned message data, errno otherwise
@@ -940,7 +958,7 @@ static int recv_stream(struct kiocb *iocb, struct socket *sock,
        int sz_to_copy;
        int sz_copied = 0;
        int needed;
-       char *crs = m->msg_iov->iov_base;
+       char __user *crs = m->msg_iov->iov_base;
        unsigned char *buf_crs;
        u32 err;
        int res;
@@ -963,18 +981,18 @@ static int recv_stream(struct kiocb *iocb, struct socket *sock,
 
        /* Look for a message in receive queue; wait if necessary */
 
-       if (unlikely(down_interruptible(&tsock->sem)))
+       if (unlikely(mutex_lock_interruptible(&tsock->lock)))
                return -ERESTARTSYS;
 
 restart:
        if (unlikely((skb_queue_len(&sock->sk->sk_receive_queue) == 0) &&
                     (flags & MSG_DONTWAIT))) {
-               res = (sz_copied == 0) ? -EWOULDBLOCK : 0;
+               res = -EWOULDBLOCK;
                goto exit;
        }
 
        if ((res = wait_event_interruptible(
-               *sock->sk->sk_sleep, 
+               *sock->sk->sk_sleep,
                ((q_len = skb_queue_len(&sock->sk->sk_receive_queue)) ||
                 (sock->state == SS_DISCONNECTING))) )) {
                goto exit;
@@ -1011,10 +1029,10 @@ restart:
        }
 
        /* Capture message data (if valid) & compute return value (always) */
-       
+
        if (!err) {
                buf_crs = (unsigned char *)(TIPC_SKB_CB(buf)->handle);
-               sz = buf->tail - buf_crs;
+               sz = skb_tail_pointer(buf) - buf_crs;
 
                needed = (buf_len - sz_copied);
                sz_to_copy = (sz <= needed) ? sz : needed;
@@ -1044,14 +1062,14 @@ restart:
        /* Consume received message (optional) */
 
        if (likely(!(flags & MSG_PEEK))) {
-                if (unlikely(++tsock->p->conn_unacked >= TIPC_FLOW_CONTROL_WIN))
-                        tipc_acknowledge(tsock->p->ref, tsock->p->conn_unacked);
+               if (unlikely(++tsock->p->conn_unacked >= TIPC_FLOW_CONTROL_WIN))
+                       tipc_acknowledge(tsock->p->ref, tsock->p->conn_unacked);
                advance_queue(tsock);
-        }
+       }
 
        /* Loop around if more data is required */
 
-       if ((sz_copied < buf_len)    /* didn't get all requested data */ 
+       if ((sz_copied < buf_len)    /* didn't get all requested data */
            && (flags & MSG_WAITALL) /* ... and need to wait for more */
            && (!(flags & MSG_PEEK)) /* ... and aren't just peeking at data */
            && (!err)                /* ... and haven't reached a FIN */
@@ -1059,8 +1077,8 @@ restart:
                goto restart;
 
 exit:
-       up(&tsock->sem);
-       return res ? res : sz_copied;
+       mutex_unlock(&tsock->lock);
+       return sz_copied ? sz_copied : res;
 }
 
 /**
@@ -1068,7 +1086,7 @@ exit:
  * @queue_size: current size of queue
  * @base: nominal maximum size of queue
  * @msg: message to be added to queue
- * 
+ *
  * Returns 1 if queue is currently overloaded, 0 otherwise
  */
 
@@ -1092,7 +1110,7 @@ static int queue_overloaded(u32 queue_size, u32 base, struct tipc_msg *msg)
        return (queue_size > threshold);
 }
 
-/** 
+/**
  * async_disconnect - wrapper function used to disconnect port
  * @portref: TIPC port reference (passed as pointer-sized value)
  */
@@ -1102,13 +1120,13 @@ static void async_disconnect(unsigned long portref)
        tipc_disconnect((u32)portref);
 }
 
-/** 
+/**
  * dispatch - handle arriving message
  * @tport: TIPC port that received message
  * @buf: message
- * 
+ *
  * Called with port locked.  Must not take socket lock to avoid deadlock risk.
- * 
+ *
  * Returns TIPC error status code (TIPC_OK if message is not to be rejected)
  */
 
@@ -1153,13 +1171,13 @@ static u32 dispatch(struct tipc_port *tport, struct sk_buff *buf)
                                msg_dbg(msg, "dispatch filter 4\n");
                                return TIPC_ERR_NO_PORT;
                        }
-               } 
+               }
                else if (sock->state == SS_LISTENING) {
                        if (msg_connected(msg) || msg_errcode(msg)) {
                                msg_dbg(msg, "dispatch filter 5\n");
                                return TIPC_ERR_NO_PORT;
                        }
-               } 
+               }
                else if (sock->state == SS_DISCONNECTING) {
                        msg_dbg(msg, "dispatch filter 6\n");
                        return TIPC_ERR_NO_PORT;
@@ -1174,18 +1192,18 @@ static u32 dispatch(struct tipc_port *tport, struct sk_buff *buf)
 
        /* Reject message if there isn't room to queue it */
 
-       if (unlikely((u32)atomic_read(&tipc_queue_size) > 
+       if (unlikely((u32)atomic_read(&tipc_queue_size) >
                     OVERLOAD_LIMIT_BASE)) {
-               if (queue_overloaded(atomic_read(&tipc_queue_size), 
+               if (queue_overloaded(atomic_read(&tipc_queue_size),
                                     OVERLOAD_LIMIT_BASE, msg))
                        return TIPC_ERR_OVERLOAD;
-        }
+       }
        recv_q_len = skb_queue_len(&tsock->sk.sk_receive_queue);
        if (unlikely(recv_q_len > (OVERLOAD_LIMIT_BASE / 2))) {
-               if (queue_overloaded(recv_q_len, 
-                                    OVERLOAD_LIMIT_BASE / 2, msg)) 
+               if (queue_overloaded(recv_q_len,
+                                    OVERLOAD_LIMIT_BASE / 2, msg))
                        return TIPC_ERR_OVERLOAD;
-        }
+       }
 
        /* Initiate connection termination for an incoming 'FIN' */
 
@@ -1202,14 +1220,15 @@ static u32 dispatch(struct tipc_port *tport, struct sk_buff *buf)
        atomic_inc(&tipc_queue_size);
        skb_queue_tail(&sock->sk->sk_receive_queue, buf);
 
-        wake_up_interruptible(sock->sk->sk_sleep);
+       if (waitqueue_active(sock->sk->sk_sleep))
+               wake_up_interruptible(sock->sk->sk_sleep);
        return TIPC_OK;
 }
 
-/** 
+/**
  * wakeupdispatch - wake up port after congestion
  * @tport: port to wakeup
- * 
+ *
  * Called with port lock on.
  */
 
@@ -1217,7 +1236,8 @@ static void wakeupdispatch(struct tipc_port *tport)
 {
        struct tipc_sock *tsock = (struct tipc_sock *)tport->usr_handle;
 
-        wake_up_interruptible(tsock->sk.sk_sleep);
+       if (waitqueue_active(tsock->sk.sk_sleep))
+               wake_up_interruptible(tsock->sk.sk_sleep);
 }
 
 /**
@@ -1230,7 +1250,7 @@ static void wakeupdispatch(struct tipc_port *tport)
  * Returns 0 on success, errno otherwise
  */
 
-static int connect(struct socket *sock, struct sockaddr *dest, int destlen, 
+static int connect(struct socket *sock, struct sockaddr *dest, int destlen,
                   int flags)
 {
    struct tipc_sock *tsock = tipc_sk(sock->sk);
@@ -1245,39 +1265,47 @@ static int connect(struct socket *sock, struct sockaddr *dest, int destlen,
    if (sock->state == SS_READY)
           return -EOPNOTSUPP;
 
-   /* MOVE THE REST OF THIS ERROR CHECKING TO send_msg()? */
+   /* Issue Posix-compliant error code if socket is in the wrong state */
+
    if (sock->state == SS_LISTENING)
           return -EOPNOTSUPP;
    if (sock->state == SS_CONNECTING)
           return -EALREADY;
    if (sock->state != SS_UNCONNECTED)
-           return -EISCONN;
+          return -EISCONN;
 
-   if ((destlen < sizeof(*dst)) || (dst->family != AF_TIPC) ||
-       ((dst->addrtype != TIPC_ADDR_NAME) && (dst->addrtype != TIPC_ADDR_ID)))
-           return -EINVAL;
+   /*
+    * Reject connection attempt using multicast address
+    *
+    * Note: send_msg() validates the rest of the address fields,
+    *       so there's no need to do it here
+    */
+
+   if (dst->addrtype == TIPC_ADDR_MCAST)
+          return -EINVAL;
 
    /* Send a 'SYN-' to destination */
 
    m.msg_name = dest;
+   m.msg_namelen = destlen;
    if ((res = send_msg(NULL, sock, &m, 0)) < 0) {
           sock->state = SS_DISCONNECTING;
           return res;
    }
 
-   if (down_interruptible(&tsock->sem)) 
-           return -ERESTARTSYS;
-       
+   if (mutex_lock_interruptible(&tsock->lock))
+          return -ERESTARTSYS;
+
    /* Wait for destination's 'ACK' response */
 
    res = wait_event_interruptible_timeout(*sock->sk->sk_sleep,
-                                          skb_queue_len(&sock->sk->sk_receive_queue),
+                                         skb_queue_len(&sock->sk->sk_receive_queue),
                                          sock->sk->sk_rcvtimeo);
    buf = skb_peek(&sock->sk->sk_receive_queue);
    if (res > 0) {
           msg = buf_msg(buf);
-           res = auto_connect(sock, tsock, msg);
-           if (!res) {
+          res = auto_connect(sock, tsock, msg);
+          if (!res) {
                   if (!msg_data_sz(msg))
                           advance_queue(tsock);
           }
@@ -1285,19 +1313,19 @@ static int connect(struct socket *sock, struct sockaddr *dest, int destlen,
           if (res == 0) {
                   res = -ETIMEDOUT;
           } else
-                  { /* leave "res" unchanged */ }
+                  { /* leave "res" unchanged */ }
           sock->state = SS_DISCONNECTING;
    }
 
-   up(&tsock->sem);
+   mutex_unlock(&tsock->lock);
    return res;
 }
 
-/** 
+/**
  * listen - allow socket to listen for incoming connections
  * @sock: socket structure
  * @len: (unused)
- * 
+ *
  * Returns 0 on success, errno otherwise
  */
 
@@ -1310,15 +1338,15 @@ static int listen(struct socket *sock, int len)
        if (sock->state != SS_UNCONNECTED)
                return -EINVAL;
        sock->state = SS_LISTENING;
-        return 0;
+       return 0;
 }
 
-/** 
+/**
  * accept - wait for connection request
  * @sock: listening socket
  * @newsock: new socket that is to be connected
  * @flags: file-related flags associated with socket
- * 
+ *
  * Returns 0 on success, errno otherwise
  */
 
@@ -1332,22 +1360,22 @@ static int accept(struct socket *sock, struct socket *newsock, int flags)
                return -EOPNOTSUPP;
        if (sock->state != SS_LISTENING)
                return -EINVAL;
-       
-       if (unlikely((skb_queue_len(&sock->sk->sk_receive_queue) == 0) && 
+
+       if (unlikely((skb_queue_len(&sock->sk->sk_receive_queue) == 0) &&
                     (flags & O_NONBLOCK)))
                return -EWOULDBLOCK;
 
-       if (down_interruptible(&tsock->sem))
+       if (mutex_lock_interruptible(&tsock->lock))
                return -ERESTARTSYS;
 
-       if (wait_event_interruptible(*sock->sk->sk_sleep, 
+       if (wait_event_interruptible(*sock->sk->sk_sleep,
                                     skb_queue_len(&sock->sk->sk_receive_queue))) {
                res = -ERESTARTSYS;
                goto exit;
        }
        buf = skb_peek(&sock->sk->sk_receive_queue);
 
-       res = tipc_create(newsock, 0);
+       res = tipc_create(sock->sk->sk_net, newsock, 0);
        if (!res) {
                struct tipc_sock *new_tsock = tipc_sk(newsock->sk);
                struct tipc_portid id;
@@ -1365,18 +1393,18 @@ static int accept(struct socket *sock, struct socket *newsock, int flags)
                        new_tsock->p->conn_instance = msg_nameinst(msg);
                }
 
-               /* 
+              /*
                 * Respond to 'SYN-' by discarding it & returning 'ACK'-.
                 * Respond to 'SYN+' by queuing it on new socket.
                 */
 
                msg_dbg(msg,"<ACC<: ");
-                if (!msg_data_sz(msg)) {
-                        struct msghdr m = {NULL,};
+               if (!msg_data_sz(msg)) {
+                       struct msghdr m = {NULL,};
 
-                        send_packet(NULL, newsock, &m, 0);
-                        advance_queue(tsock);
-                } else {
+                       send_packet(NULL, newsock, &m, 0);
+                       advance_queue(tsock);
+               } else {
                        sock_lock(tsock);
                        skb_dequeue(&sock->sk->sk_receive_queue);
                        sock_unlock(tsock);
@@ -1384,17 +1412,17 @@ static int accept(struct socket *sock, struct socket *newsock, int flags)
                }
        }
 exit:
-       up(&tsock->sem);
+       mutex_unlock(&tsock->lock);
        return res;
 }
 
 /**
  * shutdown - shutdown socket connection
  * @sock: socket structure
- * @how: direction to close (unused; always treated as read + write)
+ * @how: direction to close (must be SHUT_RDWR)
  *
  * Terminates connection (if necessary), then purges socket's receive queue.
- * 
+ *
  * Returns 0 on success, errno otherwise
  */
 
@@ -1404,9 +1432,10 @@ static int shutdown(struct socket *sock, int how)
        struct sk_buff *buf;
        int res;
 
-       /* Could return -EINVAL for an invalid "how", but why bother? */
+       if (how != SHUT_RDWR)
+               return -EINVAL;
 
-       if (down_interruptible(&tsock->sem))
+       if (mutex_lock_interruptible(&tsock->lock))
                return -ERESTARTSYS;
 
        sock_lock(tsock);
@@ -1456,7 +1485,7 @@ restart:
 
        sock_unlock(tsock);
 
-       up(&tsock->sem);
+       mutex_unlock(&tsock->lock);
        return res;
 }
 
@@ -1467,32 +1496,32 @@ restart:
  * @opt: option identifier
  * @ov: pointer to new option value
  * @ol: length of option value
- * 
- * For stream sockets only, accepts and ignores all IPPROTO_TCP options 
+ *
+ * For stream sockets only, accepts and ignores all IPPROTO_TCP options
  * (to ease compatibility).
- * 
+ *
  * Returns 0 on success, errno otherwise
  */
 
-static int setsockopt(struct socket *sock, 
+static int setsockopt(struct socket *sock,
                      int lvl, int opt, char __user *ov, int ol)
 {
        struct tipc_sock *tsock = tipc_sk(sock->sk);
        u32 value;
        int res;
 
-        if ((lvl == IPPROTO_TCP) && (sock->type == SOCK_STREAM))
-                return 0;
+       if ((lvl == IPPROTO_TCP) && (sock->type == SOCK_STREAM))
+               return 0;
        if (lvl != SOL_TIPC)
                return -ENOPROTOOPT;
        if (ol < sizeof(value))
                return -EINVAL;
-        if ((res = get_user(value, (u32 *)ov)))
+       if ((res = get_user(value, (u32 __user *)ov)))
                return res;
 
-       if (down_interruptible(&tsock->sem)) 
+       if (mutex_lock_interruptible(&tsock->lock))
                return -ERESTARTSYS;
-       
+
        switch (opt) {
        case TIPC_IMPORTANCE:
                res = tipc_set_portimportance(tsock->p->ref, value);
@@ -1500,7 +1529,7 @@ static int setsockopt(struct socket *sock,
        case TIPC_SRC_DROPPABLE:
                if (sock->type != SOCK_STREAM)
                        res = tipc_set_portunreliable(tsock->p->ref, value);
-               else 
+               else
                        res = -ENOPROTOOPT;
                break;
        case TIPC_DEST_DROPPABLE:
@@ -1513,7 +1542,7 @@ static int setsockopt(struct socket *sock,
                res = -EINVAL;
        }
 
-       up(&tsock->sem);
+       mutex_unlock(&tsock->lock);
        return res;
 }
 
@@ -1524,29 +1553,29 @@ static int setsockopt(struct socket *sock,
  * @opt: option identifier
  * @ov: receptacle for option value
  * @ol: receptacle for length of option value
- * 
- * For stream sockets only, returns 0 length result for all IPPROTO_TCP options 
+ *
+ * For stream sockets only, returns 0 length result for all IPPROTO_TCP options
  * (to ease compatibility).
- * 
+ *
  * Returns 0 on success, errno otherwise
  */
 
-static int getsockopt(struct socket *sock, 
-                     int lvl, int opt, char __user *ov, int *ol)
+static int getsockopt(struct socket *sock,
+                     int lvl, int opt, char __user *ov, int __user *ol)
 {
        struct tipc_sock *tsock = tipc_sk(sock->sk);
-        int len;
+       int len;
        u32 value;
-        int res;
+       int res;
 
-        if ((lvl == IPPROTO_TCP) && (sock->type == SOCK_STREAM))
-                return put_user(0, ol);
+       if ((lvl == IPPROTO_TCP) && (sock->type == SOCK_STREAM))
+               return put_user(0, ol);
        if (lvl != SOL_TIPC)
                return -ENOPROTOOPT;
-        if ((res = get_user(len, ol)))
-                return res;
+       if ((res = get_user(len, ol)))
+               return res;
 
-       if (down_interruptible(&tsock->sem)) 
+       if (mutex_lock_interruptible(&tsock->lock))
                return -ERESTARTSYS;
 
        switch (opt) {
@@ -1579,105 +1608,78 @@ static int getsockopt(struct socket *sock,
                res = put_user(sizeof(value), ol);
        }
 
-        up(&tsock->sem);
+       mutex_unlock(&tsock->lock);
        return res;
 }
 
 /**
- * Placeholders for non-implemented functionality
- * 
- * Returns error code (POSIX-compliant where defined)
- */
-
-static int ioctl(struct socket *s, u32 cmd, unsigned long arg)
-{
-        return -EINVAL;
-}
-
-static int no_mmap(struct file *file, struct socket *sock,
-                   struct vm_area_struct *vma)
-{
-        return -EINVAL;
-}
-static ssize_t no_sendpage(struct socket *sock, struct page *page,
-                           int offset, size_t size, int flags)
-{
-        return -EINVAL;
-}
-
-static int no_skpair(struct socket *s1, struct socket *s2)
-{
-       return -EOPNOTSUPP;
-}
-
-/**
  * Protocol switches for the various types of TIPC sockets
  */
 
-static struct proto_ops msg_ops = {
+static const struct proto_ops msg_ops = {
        .owner          = THIS_MODULE,
        .family         = AF_TIPC,
        .release        = release,
        .bind           = bind,
        .connect        = connect,
-       .socketpair     = no_skpair,
+       .socketpair     = sock_no_socketpair,
        .accept         = accept,
        .getname        = get_name,
        .poll           = poll,
-       .ioctl          = ioctl,
+       .ioctl          = sock_no_ioctl,
        .listen         = listen,
        .shutdown       = shutdown,
        .setsockopt     = setsockopt,
        .getsockopt     = getsockopt,
        .sendmsg        = send_msg,
        .recvmsg        = recv_msg,
-        .mmap          = no_mmap,
-        .sendpage      = no_sendpage
+       .mmap           = sock_no_mmap,
+       .sendpage       = sock_no_sendpage
 };
 
-static struct proto_ops packet_ops = {
+static const struct proto_ops packet_ops = {
        .owner          = THIS_MODULE,
        .family         = AF_TIPC,
        .release        = release,
        .bind           = bind,
        .connect        = connect,
-       .socketpair     = no_skpair,
+       .socketpair     = sock_no_socketpair,
        .accept         = accept,
        .getname        = get_name,
        .poll           = poll,
-       .ioctl          = ioctl,
+       .ioctl          = sock_no_ioctl,
        .listen         = listen,
        .shutdown       = shutdown,
        .setsockopt     = setsockopt,
        .getsockopt     = getsockopt,
        .sendmsg        = send_packet,
        .recvmsg        = recv_msg,
-        .mmap          = no_mmap,
-        .sendpage      = no_sendpage
+       .mmap           = sock_no_mmap,
+       .sendpage       = sock_no_sendpage
 };
 
-static struct proto_ops stream_ops = {
+static const struct proto_ops stream_ops = {
        .owner          = THIS_MODULE,
        .family         = AF_TIPC,
        .release        = release,
        .bind           = bind,
        .connect        = connect,
-       .socketpair     = no_skpair,
+       .socketpair     = sock_no_socketpair,
        .accept         = accept,
        .getname        = get_name,
        .poll           = poll,
-       .ioctl          = ioctl,
+       .ioctl          = sock_no_ioctl,
        .listen         = listen,
        .shutdown       = shutdown,
        .setsockopt     = setsockopt,
        .getsockopt     = getsockopt,
        .sendmsg        = send_stream,
        .recvmsg        = recv_stream,
-        .mmap          = no_mmap,
-        .sendpage      = no_sendpage
+       .mmap           = sock_no_mmap,
+       .sendpage       = sock_no_sendpage
 };
 
-static struct net_proto_family tipc_family_ops = {
+static const struct net_proto_family tipc_family_ops = {
        .owner          = THIS_MODULE,
        .family         = AF_TIPC,
        .create         = tipc_create
@@ -1691,14 +1693,14 @@ static struct proto tipc_proto = {
 
 /**
  * tipc_socket_init - initialize TIPC socket interface
- * 
+ *
  * Returns 0 on success, errno otherwise
  */
 int tipc_socket_init(void)
 {
        int res;
 
-        res = proto_register(&tipc_proto, 1);
+       res = proto_register(&tipc_proto, 1);
        if (res) {
                err("Failed to register TIPC protocol type\n");
                goto out;