From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752387AbdLNNce (ORCPT ); Thu, 14 Dec 2017 08:32:34 -0500 Received: from bastet.se.axis.com ([195.60.68.11]:34568 "EHLO bastet.se.axis.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752023AbdLNNcd (ORCPT ); Thu, 14 Dec 2017 08:32:33 -0500 Date: Thu, 14 Dec 2017 14:32:30 +0100 From: Niklas Cassel To: Bjorn Helgaas Cc: linux-pci@vger.kernel.org, kishon@ti.com, linux-kernel@vger.kernel.org Subject: Re: [PATCH v4 0/3] Fix find_first_zero_bit() usage Message-ID: <20171214133230.GA25307@axis.com> References: <20171212141634.5985-1-niklas.cassel@axis.com> <20171213215925.GJ30595@bhelgaas-glaptop.roam.corp.google.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20171213215925.GJ30595@bhelgaas-glaptop.roam.corp.google.com> User-Agent: Mutt/1.9.1+16 (8a41d1c2f267) (2017-09-22) X-TM-AS-GCONF: 00 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Dec 13, 2017 at 03:59:25PM -0600, Bjorn Helgaas wrote: > On Tue, Dec 12, 2017 at 03:16:31PM +0100, Niklas Cassel wrote: > > find_first_zero_bit()'s parameter 'size' is defined in bits, > > not in bytes. > > > > Calling find_first_zero_bit() with the wrong size unit > > will lead to insidious bugs. > > > > Fix all uses of find_first_zero_bit() called with > > sizeof() as size argument in drivers/pci. > > > > Also had to fix broken error handling in pci_epc_epf_link() > > in order to do proper error handling for find_first_zero_bit(). > > > > Niklas Cassel (3): > > PCI: designware-ep: Fix find_first_zero_bit() usage > > PCI: endpoint: Fix error handling in pci_epc_epf_link() > > PCI: endpoint: Fix find_first_zero_bit() usage > > > > drivers/pci/dwc/pcie-designware-ep.c | 34 ++++++++++++++++++++++++++-------- > > drivers/pci/dwc/pcie-designware.h | 8 ++++++-- > > drivers/pci/endpoint/pci-ep-cfs.c | 13 ++++++++----- > > 3 files changed, 40 insertions(+), 15 deletions(-) > > In the interest of making forward progress, I applied these to > for-linus for v4.15. > > The issues apparently have been there since v4.12-rc1, but I guess > this is proposed for for-linus because (a) it fixes insidious bugs > and (b) the endpoint framework is relatively little-used yet so > low-risk. Right? > Hello Bjorn, As far as I know, dra7xx is the only in-tree user of the endpoint framework. Therefore, I see no real need to rush these patches. One benefit of sending them to v4.15 would be if anyone starts developing endpoint support for their driver (with v4.15 as a base), we eliminate the risk that they might get hit by these bugs, and potentially waste time finding bugs that have already been found. Please note that Kishon had some last minute review comments, so I had to submit a V5 of the patch series. Regards, Niklas