linux.git/net/iucv, branch v5.1

iucv: Remove SKB list assumptions.

2018-11-11T00:55:11+00:00

Eliminate the assumption that SKBs and SKB list heads can
be cast to eachother in SKB list handling code.

This change also appears to fix a bug since the list->next pointer is
sampled outside of holding the SKB queue lock.

Signed-off-by: David S. Miller

Revert "net: simplify sock_poll_wait"

2018-10-23T17:57:06+00:00

This reverts commit dd979b4df817e9976f18fb6f9d134d6bc4a3c317.

This broke tcp_poll for SMC fallback: An AF_SMC socket establishes an
internal TCP socket for the initial handshake with the remote peer.
Whenever the SMC connection can not be established this TCP socket is
used as a fallback. All socket operations on the SMC socket are then
forwarded to the TCP socket. In case of poll, the file->private_data
pointer references the SMC socket because the TCP socket has no file
assigned. This causes tcp_poll to wait on the wrong socket.

Signed-off-by: Karsten Graul 
Signed-off-by: David S. Miller

net/af_iucv: locate IUCV header via skb_network_header()

2018-09-26T16:56:07+00:00

This patch attempts to untangle the TX and RX code in qeth from
af_iucv's respective HiperTransport path:
On the TX side, pointing skb_network_header() at the IUCV header
means that qeth_l3_fill_af_iucv_hdr() no longer needs a magical offset
to access the header.
On the RX side, qeth pulls the (fake) L2 header off the skb like any
normal ethernet driver would. This makes working with the IUCV header
in af_iucv easier, since we no longer have to assume a fixed skb layout.

While at it, replace the open-coded length checks in af_iucv's RX path
with pskb_may_pull().

Signed-off-by: Julian Wiedmann 
Signed-off-by: David S. Miller

net: iucv: Use FIELD_SIZEOF directly instead of reimplementing its function

2018-09-20T03:58:04+00:00

FIELD_SIZEOF is defined as a macro to calculate the specified value. Therefore,
We prefer to use the macro rather than calculating its value.

Signed-off-by: zhong jiang 
Signed-off-by: David S. Miller

net/iucv: declare iucv_path_table_empty() as static

2018-09-06T05:32:22+00:00

Fixes a compile warning.

Signed-off-by: Julian Wiedmann 
Signed-off-by: David S. Miller

net/af_iucv: fix skb handling on HiperTransport xmit error

2018-09-06T05:32:22+00:00

When sending an skb, afiucv_hs_send() bails out on various error
conditions. But currently the caller has no way of telling whether the
skb was freed or not - resulting in potentially either
a) leaked skbs from iucv_send_ctrl(), or
b) double-free's from iucv_sock_sendmsg().

As dev_queue_xmit() will always consume the skb (even on error), be
consistent and also free the skb from all other error paths. This way
callers no longer need to care about managing the skb.

Signed-off-by: Julian Wiedmann 
Reviewed-by: Ursula Braun 
Signed-off-by: David S. Miller

net/af_iucv: drop inbound packets with invalid flags

2018-09-06T05:32:21+00:00

Inbound packets may have any combination of flag bits set in their iucv
header. If we don't know how to handle a specific combination, drop the
skb instead of leaking it.

To clarify what error is returned in this case, replace the hard-coded
0 with the corresponding macro.

Signed-off-by: Julian Wiedmann 
Signed-off-by: David S. Miller

net:af_iucv: get rid of the unneeded variable 'err' in afiucv_pm_freeze

2018-08-08T16:39:36+00:00

We will not use the variable 'err' after initalization, So remove it and
return 0.

Signed-off-by: zhong jiang 
Signed-off-by: David S. Miller

net: simplify sock_poll_wait

2018-07-30T16:10:25+00:00

The wait_address argument is always directly derived from the filp
argument, so remove it.

Signed-off-by: Christoph Hellwig 
Signed-off-by: David S. Miller

net: remove blank lines at end of file

2018-07-24T21:10:43+00:00

Several files have extra line at end of file.

Signed-off-by: Stephen Hemminger 
Signed-off-by: David S. Miller