Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751626AbdITSJI (ORCPT ); Wed, 20 Sep 2017 14:09:08 -0400 Received: from mx1.redhat.com ([209.132.183.28]:53560 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751283AbdITSJH (ORCPT ); Wed, 20 Sep 2017 14:09:07 -0400 DMARC-Filter: OpenDMARC Filter v1.3.2 mx1.redhat.com 17A01C0587F1 Authentication-Results: ext-mx08.extmail.prod.ext.phx2.redhat.com; dmarc=none (p=none dis=none) header.from=redhat.com Authentication-Results: ext-mx08.extmail.prod.ext.phx2.redhat.com; spf=fail smtp.mailfrom=alex.williamson@redhat.com Date: Wed, 20 Sep 2017 12:09:12 -0600 From: Alex Williamson To: Vadim Lomovtsev Cc: Jan Glauber , Bjorn Helgaas , kvm@vger.kernel.org, linux-pci@vger.kernel.org, linux-kernel@vger.kernel.org, Jon Masters , linux-arm-kernel@lists.infradead.org, david.daney@cavium.com, Robert Richter Subject: Re: [v4,0/3] Workaround for bus/slot reset on Cavium cn8xxx root ports Message-ID: <20170920120912.7544420d@t450s.home> In-Reply-To: <20170912094049.GA14111@localhost.localdomain> References: <20170908081033.3025-1-jglauber@cavium.com> <20170912094049.GA14111@localhost.localdomain> MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.32]); Wed, 20 Sep 2017 18:09:07 +0000 (UTC) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1200 Lines: 40 On Tue, 12 Sep 2017 02:40:49 -0700 Vadim Lomovtsev wrote: > Hi all, > > Are there any updates on this ? > Comments/objections/acks/nacks ? > > WBBR, > Vadim > > On Fri, Sep 08, 2017 at 10:10:30AM +0200, Jan Glauber wrote: > > Using vfio-pci on a combination of cn8xxx and some PCI devices results in > > a kernel panic. This is triggered by issuing a bus or a slot reset > > on the PCI device. > > > > With this series both checks indicate that the reset is not possible > > preventing the kernel panic. > > > > David Daney (2): > > PCI: Allow PCI_DEV_FLAGS_NO_BUS_RESET to be used on bus device > > PCI: Avoid bus reset for Cavium cn8xxx root ports > > > > Jan Glauber (1): > > PCI: Avoid slot reset if bus reset is not possible > > > > drivers/pci/pci.c | 8 ++++++++ > > drivers/pci/quirks.c | 8 ++++++++ > > 2 files changed, 16 insertions(+) Looks ok to me, for series: Reviewed-by: Alex Williamson I am curious why we're happy targeting this quirk at a single device ID while at the same time trying to expand the ACS quirk to a notable fraction of the Cavium PCI device ID address space. Thanks, Alex