Commit 7a446d63 authored Jun 30, 2014 by Jason Wang Committed by Greg Kroah-Hartman Jul 09, 2014

hyperv: remove meaningless pr_err() in vmbus_recvpacket_raw()



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>

parent e013ac31

drivers/hv/channel.c

+1 −5

Original line number	Diff line number	Diff line
		@@ -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;