From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:47555) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Z9nUE-00006K-JD for qemu-devel@nongnu.org; Tue, 30 Jun 2015 00:49:35 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Z9nUD-0004eN-F3 for qemu-devel@nongnu.org; Tue, 30 Jun 2015 00:49:34 -0400 Received: from mail-oi0-x22d.google.com ([2607:f8b0:4003:c06::22d]:32892) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Z9nUD-0004dz-A5 for qemu-devel@nongnu.org; Tue, 30 Jun 2015 00:49:33 -0400 Received: by oiyy130 with SMTP id y130so133121154oiy.0 for ; Mon, 29 Jun 2015 21:49:31 -0700 (PDT) MIME-Version: 1.0 From: Scott Feldman Date: Tue, 30 Jun 2015 00:49:12 -0400 Message-ID: Content-Type: text/plain; charset=UTF-8 Subject: [Qemu-devel] (no subject) List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: QEMU Developers , famz@redhat.com, Stefan Hajnoczi Cc: =?UTF-8?B?SmnFmcOtIFDDrXJrbw==?= Hi Fam, Stefan, I'm running a test with rocker device using UDP sockets connections and I'm seeing the socket s->read_poll stay disabled if the device receives a packet when the device's can_receive returns false. Receive is stuck after that; nothing ever re-enables s->read_poll. I see the first packet queued on queue->packets and that's it. No more receives. If I modify the device to lie and always return can_receive=true, and drop the pkt in driver receive, then things work fine. I think this patch broke can_receive semantics for net/socket.c: commit 6e99c631f116221d169ea53953d91b8aa74d297a Author: Fam Zheng Date: Thu Jun 4 14:45:16 2015 +0800 net/socket: Drop net_socket_can_send Anything jump out? (In the test, rocker device is enabling the netdev port once the guest OS driver signals to enable the port based on STP process running on the guest. The initial STP state is DISABLED, so the port is isolated from the network. As STP algo progresses, the port is opened up and the netdev is enabled for Rx traffic). -scott