hyperv: remove meaningless pr_err() in vmbus_recvpacket_raw()
authorJason Wang <jasowang@redhat.com>
Mon, 30 Jun 2014 06:14:48 +0000 (14:14 +0800)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 9 Jul 2014 21:21:27 +0000 (14:21 -0700)
All its callers depends on the return value of -ENOBUFS to reallocate a
bigger buffer and retry the receiving. So there's no need to call
pr_err() here since it was not a real issue, otherwise syslog will be
flooded by this false warning.

Cc: Haiyang Zhang <haiyangz@microsoft.com>
Signed-off-by: Jason Wang <jasowang@redhat.com>
Signed-off-by: K. Y. Srinivasan <kys@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/hv/channel.c

index 284cf66489f428f1b00434d088da85222f00ea3a..531a593912ec6a5feb00ea308e13f4da93a60268 100644 (file)
@@ -808,12 +808,8 @@ int vmbus_recvpacket_raw(struct vmbus_channel *channel, void *buffer,
 
        *buffer_actual_len = packetlen;
 
-       if (packetlen > bufferlen) {
-               pr_err("Buffer too small - needed %d bytes but "
-                       "got space for only %d bytes\n",
-                       packetlen, bufferlen);
+       if (packetlen > bufferlen)
                return -ENOBUFS;
-       }
 
        *requestid = desc.trans_id;