From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752305AbdB1DIk (ORCPT ); Mon, 27 Feb 2017 22:08:40 -0500 Received: from a2nlsmtp01-02.prod.iad2.secureserver.net ([198.71.225.36]:56566 "EHLO a2nlsmtp01-02.prod.iad2.secureserver.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750971AbdB1DHa (ORCPT ); Mon, 27 Feb 2017 22:07:30 -0500 x-originating-ip: 107.180.71.197 From: kys@exchange.microsoft.com To: gregkh@linuxfoundation.org, linux-kernel@vger.kernel.org, devel@linuxdriverproject.org, olaf@aepfle.de, apw@canonical.com, vkuznets@redhat.com, jasowang@redhat.com, leann.ogasawara@canonical.com Cc: Stephen Hemminger , Stephen Hemminger , "K. Y. Srinivasan" Subject: [PATCH 05/10] vmbus: remove unnecessary initialization Date: Mon, 27 Feb 2017 20:05:28 -0700 Message-Id: <1488251133-32767-5-git-send-email-kys@exchange.microsoft.com> X-Mailer: git-send-email 1.7.1 In-Reply-To: <1488251083-32718-1-git-send-email-kys@exchange.microsoft.com> References: <1488251083-32718-1-git-send-email-kys@exchange.microsoft.com> Reply-To: kys@microsoft.com X-CMAE-Envelope: MS4wfN3jfGLvrPrpPN+WQ4AS3Ir2Pn/HuLivHAi2cVnr5AXk+pjHZMYVjqL/uZyIlRNu7eRx6S5ioj0Tcv/IXEuhq/I5CE8ghobtBnI19AopYdzMa9BpL76s YPhsxUxWq2urm4OdFVUxyfzLZP8tIhFG9nCq3ihQL+2+rJMrYDsyA4LkXFvmG44LfI9/mxeySYYn66U7RJeEd/HiC0E9SvyHAs9p/buspmtIo92envV1NlBU Qby+KCdfW9t3qk7mVN9RCGv8oKQqqmX+jwLAA7Ds0qp+L1EKuss6OTziz6MPKWeQPAPHReWAyx7yJuQkSmB4YBSYJW1oN11RdU900i2LWdnRFNGhN8AnzBE4 Fr4ZgAGTqCP+V8jW0sWY3w9D68zy8tnOCXKLilm6Ij4oK/yXnjRwczcQCobuYe3lsteAkfBkMWymUaMDJfbcI7FlXGlPI4aQIT4VdMyqRWg5JVIUg6gbdOZ7 b+XYHTFxfK1xw4YBvfCpbOgH0CgROrVm1wqZ9Ga0eWd4JD6Jxti2kwbMWV0= Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org From: Stephen Hemminger Don't initialize variables that are then set a few lines later. Signed-off-by: Stephen Hemminger Signed-off-by: K. Y. Srinivasan --- drivers/hv/ring_buffer.c | 13 +++++-------- 1 files changed, 5 insertions(+), 8 deletions(-) diff --git a/drivers/hv/ring_buffer.c b/drivers/hv/ring_buffer.c index d0ff5b4..52d0556 100644 --- a/drivers/hv/ring_buffer.c +++ b/drivers/hv/ring_buffer.c @@ -265,14 +265,13 @@ void hv_ringbuffer_cleanup(struct hv_ring_buffer_info *ring_info) int hv_ringbuffer_write(struct vmbus_channel *channel, const struct kvec *kv_list, u32 kv_count) { - int i = 0; + int i; u32 bytes_avail_towrite; - u32 totalbytes_towrite = 0; - + u32 totalbytes_towrite = sizeof(u64); u32 next_write_location; u32 old_write; - u64 prev_indices = 0; - unsigned long flags = 0; + u64 prev_indices; + unsigned long flags; struct hv_ring_buffer_info *outring_info = &channel->outbound; if (channel->rescind) @@ -281,8 +280,6 @@ int hv_ringbuffer_write(struct vmbus_channel *channel, for (i = 0; i < kv_count; i++) totalbytes_towrite += kv_list[i].iov_len; - totalbytes_towrite += sizeof(u64); - spin_lock_irqsave(&outring_info->ring_lock, flags); bytes_avail_towrite = hv_get_bytes_to_write(outring_info); @@ -339,7 +336,7 @@ int hv_ringbuffer_read(struct vmbus_channel *channel, u64 *requestid, bool raw) { u32 bytes_avail_toread; - u32 next_read_location = 0; + u32 next_read_location; u64 prev_indices = 0; struct vmpacket_descriptor desc; u32 offset; -- 1.7.1