Re: [GIT] Networking
From: Linus Torvalds
Date: Mon Jul 02 2018 - 14:29:58 EST
On Mon, Jul 2, 2018 at 6:03 AM David Miller <davem@xxxxxxxxxxxxx> wrote:
> are available in the Git repository at:
So the af_smc poll fixes clashed with the revert of the commit that
caused those fixed to be done in the first place.
See the changes to net/smc/af_smc.c in my commit
a11e1d432b51 ("Revert changes to convert to ->poll_mask() and aio
24ac3a08e658 ("net/smc: rebuild nonblocking connect")
I (briefly) considered just dropping Ursula's changes entirely, but
they looked like a nice cleanup on their own, so what I did instead
was to try to fix up my revert instead.
That involved removing the release_sock/lock_sock pair around the
->poll() call, and removing the special "sock_poll_wait()" that got
re-introduced by my revert, but that Ursula's changes seem to obviate.
However, while I can look at the code and say "my merge makes sense to
me", (a) I can't test it, (b) I don't actually know the rules for SMC
sockets in the first place, and (c) I may be just incompetent.
So Ursula - mind checking and testing the end result? I _think_ it's
fine and the merge looked pretty obvious, but maybe af_smc got broken
[ It's still going through by basic build tests, so I haven't pushed
out my merge yet, but it should be in the usual places in a short