diff options
author | Ying Xue <ying.xue@windriver.com> | 2013-06-17 10:54:43 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2013-06-17 18:53:00 -0400 |
commit | f1733d7580ff94deb8ea071a293c23939ae0d450 (patch) | |
tree | 41e10295a1546d044cf529267f703e09b87b03d3 /net/tipc/link.c | |
parent | 198d73b82bf78739f8f11cf7ff567a2e0da1dbef (diff) |
tipc: remove user_port instance from tipc_port structure
After the native API has been completely removed, the 'user_port'
field in struct tipc_port becomes unused, and can be removed.
As a consequence, the "usrmem" argument in tipc_msg_build() is no
longer needed, and so we remove that one too.
Signed-off-by: Ying Xue <ying.xue@windriver.com>
Signed-off-by: Jon Maloy <jon.maloy@ericsson.com>
Signed-off-by: Paul Gortmaker <paul.gortmaker@windriver.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/tipc/link.c')
-rw-r--r-- | net/tipc/link.c | 18 |
1 files changed, 7 insertions, 11 deletions
diff --git a/net/tipc/link.c b/net/tipc/link.c index 0a4c3a1bb9cc..d34429d03c16 100644 --- a/net/tipc/link.c +++ b/net/tipc/link.c | |||
@@ -1081,7 +1081,7 @@ again: | |||
1081 | * (Must not hold any locks while building message.) | 1081 | * (Must not hold any locks while building message.) |
1082 | */ | 1082 | */ |
1083 | res = tipc_msg_build(hdr, msg_sect, num_sect, total_len, | 1083 | res = tipc_msg_build(hdr, msg_sect, num_sect, total_len, |
1084 | sender->max_pkt, !sender->user_port, &buf); | 1084 | sender->max_pkt, &buf); |
1085 | 1085 | ||
1086 | read_lock_bh(&tipc_net_lock); | 1086 | read_lock_bh(&tipc_net_lock); |
1087 | node = tipc_node_find(destaddr); | 1087 | node = tipc_node_find(destaddr); |
@@ -1216,18 +1216,14 @@ again: | |||
1216 | else | 1216 | else |
1217 | sz = fragm_rest; | 1217 | sz = fragm_rest; |
1218 | 1218 | ||
1219 | if (likely(!sender->user_port)) { | 1219 | if (copy_from_user(buf->data + fragm_crs, sect_crs, sz)) { |
1220 | if (copy_from_user(buf->data + fragm_crs, sect_crs, sz)) { | ||
1221 | error: | 1220 | error: |
1222 | for (; buf_chain; buf_chain = buf) { | 1221 | for (; buf_chain; buf_chain = buf) { |
1223 | buf = buf_chain->next; | 1222 | buf = buf_chain->next; |
1224 | kfree_skb(buf_chain); | 1223 | kfree_skb(buf_chain); |
1225 | } | ||
1226 | return -EFAULT; | ||
1227 | } | 1224 | } |
1228 | } else | 1225 | return -EFAULT; |
1229 | skb_copy_to_linear_data_offset(buf, fragm_crs, | 1226 | } |
1230 | sect_crs, sz); | ||
1231 | sect_crs += sz; | 1227 | sect_crs += sz; |
1232 | sect_rest -= sz; | 1228 | sect_rest -= sz; |
1233 | fragm_crs += sz; | 1229 | fragm_crs += sz; |