From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757709AbYB2Ags (ORCPT ); Thu, 28 Feb 2008 19:36:48 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1754701AbYB2Agi (ORCPT ); Thu, 28 Feb 2008 19:36:38 -0500 Received: from srv5.dvmed.net ([207.36.208.214]:47590 "EHLO mail.dvmed.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753048AbYB2Agh (ORCPT ); Thu, 28 Feb 2008 19:36:37 -0500 Message-ID: <47C75392.3020606@garzik.org> Date: Thu, 28 Feb 2008 19:36:34 -0500 From: Jeff Garzik User-Agent: Thunderbird 2.0.0.9 (X11/20071115) MIME-Version: 1.0 To: Tejun Heo CC: Linux IDE mailing list , LKML Subject: devres and requesting resources Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit X-Spam-Score: -4.4 (----) X-Spam-Report: SpamAssassin version 3.2.3 on srv5.dvmed.net summary: Content analysis details: (-4.4 points, 5.0 required) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Something I just noticed, which I missed during the initial devres review... The vast majority of my ATA drivers originally called pci_request_regions(), to reserve all regions attached to a device. In converting to pcim_iomap_regions(), we no longer reserve /all/ regions, only the ones requested. This is actually a bug: it was intentional to call pci_request_regions(), because that ensures that no other software will use our resources -- even if we are not actively using the resource in question. Or IOW, I wanted to ensure that there would be no device sharing... which this devres conversion accidentally enabled. The simple fix is obviously to replace pci_request_region() call with pci_request_regions() in lib/devres.c, but I wonder if that will break any existing driver? Jeff