Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752908AbdLANo0 (ORCPT ); Fri, 1 Dec 2017 08:44:26 -0500 Received: from vps0.lunn.ch ([185.16.172.187]:43106 "EHLO vps0.lunn.ch" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752691AbdLANoW (ORCPT ); Fri, 1 Dec 2017 08:44:22 -0500 Date: Fri, 1 Dec 2017 14:44:15 +0100 From: Andrew Lunn To: Salil Mehta Cc: davem@davemloft.net, yisen.zhuang@huawei.com, lipeng321@huawei.com, mehta.salil.lnk@gmail.com, netdev@vger.kernel.org, linux-kernel@vger.kernel.org, linux-rdma@vger.kernel.org, linuxarm@huawei.com Subject: Re: [PATCH net-next] net: hns3: Refactors "reset" handling code in HCLGE layer of HNS3 driver Message-ID: <20171201134415.GD22599@lunn.ch> References: <20171201033744.19104-1-salil.mehta@huawei.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20171201033744.19104-1-salil.mehta@huawei.com> User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1574 Lines: 35 On Fri, Dec 01, 2017 at 03:37:44AM +0000, Salil Mehta wrote: > This patch refactors the code of the reset feature in HCLGE layer > of HNS3 PF driver. Prime motivation to do this change is: > 1. To reduce the time for which common miscellaneous Vector 0 > interrupt is disabled because of the reset. > 2. Simplification of reset request submission and pending reset > logic. > 3. Simplification of the common miscellaneous interrupt handler > routine(for Vector 0) used to handle reset and other sources > of Vector 0 interrupt. > > To achieve above below few things have been done: > 1. Interrupt is disabled while common miscellaneous interrupt > handler is entered and re-enabled before it is exit. This > reduces the interrupt handling latency as compared to older > interrupt handling scheme where interrupt was being disabled > in interrupt handler context and re-enabled in task context > some time later. > 2. Introduces new reset service task for honoring software reset > requests like from network stack related to timeout and serving > the pending reset request(to reset the driver and associated > clients). > 3. Made Miscellaneous interrupt handler more generic to handle > all sources including reset interrupt source. Hi Salil This is a rather large patch. Can you break it up? It seems like you should be able to break it up into at least three parts, maybe more. You are aiming to have small patches which are obviously correct. It is much easier to review than one big patch which is not obvious at all. Andrew