Messages in this thread | | | From | Dexuan Cui <> | Subject | RE: [PATCH v12 net-next 1/1] hv_sock: introduce Hyper-V Sockets | Date | Tue, 28 Jun 2016 15:29:47 +0000 |
| |
> From: David Miller [mailto:davem@davemloft.net] > Sent: Tuesday, June 28, 2016 21:45 > To: Dexuan Cui <decui@microsoft.com> > Cc: gregkh@linuxfoundation.org; netdev@vger.kernel.org; linux- > kernel@vger.kernel.org; devel@linuxdriverproject.org; olaf@aepfle.de; > apw@canonical.com; jasowang@redhat.com; vkuznets@redhat.com; > cavery@redhat.com; KY Srinivasan <kys@microsoft.com>; Haiyang Zhang > <haiyangz@microsoft.com>; joe@perches.com > Subject: Re: [PATCH v12 net-next 1/1] hv_sock: introduce Hyper-V Sockets > > From: Dexuan Cui <decui@microsoft.com> > Date: Tue, 28 Jun 2016 09:59:21 +0000 > > > The idea here is: IMO the syscalls sys_read()/write() shoudn't return > > -ENOMEM, so I have to make sure the buffer allocation succeeds? > > You have to fail if resources cannot be allocated.
OK, I'll try to fix this, probably by returning -EAGAIN or -ERESTARTSYS.
I'll report back ASAP.
Thanks, -- Dexuan
| |