From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755091AbbCFSta (ORCPT ); Fri, 6 Mar 2015 13:49:30 -0500 Received: from mail-oi0-f42.google.com ([209.85.218.42]:36636 "EHLO mail-oi0-f42.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751658AbbCFStY (ORCPT ); Fri, 6 Mar 2015 13:49:24 -0500 Date: Fri, 6 Mar 2015 12:49:20 -0600 From: Bjorn Helgaas To: Alex Deucher Cc: Alex Williamson , Linux PCI , Dave Airlie , "Deucher, Alexander" , Maling list - DRI developers , LKML Subject: Re: [PATCH 0/2] PCI: pci_unignore_hotplug() Message-ID: <20150306184920.GI20077@google.com> References: <20150224200603.14443.68971.stgit@gimli.home> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Feb 24, 2015 at 03:23:27PM -0500, Alex Deucher wrote: > On Tue, Feb 24, 2015 at 3:12 PM, Alex Williamson > wrote: > > I'd kinda like to use pci_ignore_hotplug() for devices in use by a > > user via vfio-pci, but the interface seems only partially implemented > > since we can only set ignore_hotplug and not clear it. This adds the > > capability to clear the flag and applies it as seems to be appropriate > > to the existing users. I don't have a system with power control for > > a radeon or nouveau device, so I'd certainly appreciate testing. It > > seems though that once the device is in D0, we've passed the critical > > point where the device could be accidentally unplugged. Thanks, > > Something re-broke this recently. See: > https://bugzilla.kernel.org/show_bug.cgi?id=61891 > https://bugzilla.kernel.org/show_bug.cgi?id=89731 I'm not opposed to adding pci_unignore_hotplug() when we have users for it. I don't really want to throw in the nouveau and radeon changes while there are open issues in the same area, though. I haven't looked at the bugzillas above. Let me know if I should. Bjorn