Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754377AbYK1S5u (ORCPT ); Fri, 28 Nov 2008 13:57:50 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1752636AbYK1S5k (ORCPT ); Fri, 28 Nov 2008 13:57:40 -0500 Received: from palinux.external.hp.com ([192.25.206.14]:38516 "EHLO mail.parisc-linux.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752570AbYK1S5j (ORCPT ); Fri, 28 Nov 2008 13:57:39 -0500 Date: Fri, 28 Nov 2008 11:57:23 -0700 From: Matthew Wilcox To: Trent Piepho Cc: Alex Chiang , "Darrick J. Wong" , linux-kernel , Jesse Barnes , linux-pci Subject: Re: [PATCH] fakephp: Allocate PCI resources before adding the device Message-ID: <20081128185722.GU25548@parisc-linux.org> References: <20081125212422.22631.69619.stgit@elm3a70.beaverton.ibm.com> <20081126074808.GE6539@plum> <20081126181841.GF6539@plum> <20081126225535.GA27936@ldl.fc.hp.com> <20081127024204.GJ25548@parisc-linux.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.13 (2006-08-11) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2301 Lines: 46 On Fri, Nov 28, 2008 at 02:11:49AM -0800, Trent Piepho wrote: > On Wed, 26 Nov 2008, Matthew Wilcox wrote: > > Because fakephp was the odd one out. Every other hotplug driver > > registers a slot number and puts the address in the directory. We're > > making hotplug drivers simpler (by sharing more of the logic in the > > core) and so fakephp had to change to match the other drivers. I'm > > sorry for your inconvenience, but it's necessary. > > Seems like changing a long established interface like this should be put on > the feature removable schedule. sysfs is more fluid than other interfaces. Large parts of it change without warning. I suppose it could have gone through feature-removal-schedule, and perhaps we should try to do that in future. > > We can discuss other ways to make your life better, but it can't be > > changed back to one 'slot' per PCI function. > > Why not have a slot per pci function with each hotplug driver adding > attributes to the slot? The slot would be registered by the hotplug core. What we have now is the 'address' file managed by the core. This doesn't include the function number. > Is there any reason the fakephp driver should not be allowed to enable and > disable individual functions? There a lot of mutli-function devices that > allow individual functions to be enabled and disabled with registers on the > primary function. For instance, it would be possible to work around a BIOS > that doesn't have an option to disabled a southbridge's USB controller by > doing so in Linux and then using fakephp to remove the device, and thus > free address space that other devices might need. There are lots of circumstances where enabling and disabling individual functions is a good idea, but I don't think that fakephp is the right way to do it. We need a better interface. -- Matthew Wilcox Intel Open Source Technology Centre "Bill, look, we understand that you're interested in selling us this operating system, but compare it to ours. We can't possibly take such a retrograde step." -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/