From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-pd0-f169.google.com ([209.85.192.169]:33940 "EHLO mail-pd0-f169.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756814AbbFQRX4 (ORCPT ); Wed, 17 Jun 2015 13:23:56 -0400 Message-ID: <5581AD29.2040406@gmail.com> Date: Wed, 17 Jun 2015 10:23:53 -0700 From: Alexander Duyck MIME-Version: 1.0 To: "Rustad, Mark D" CC: "bhelgaas@google.com" , "linux-pci@vger.kernel.org" , "intel-wired-lan@lists.osuosl.org" , "netdev@vger.kernel.org" Subject: Re: [PATCH V3 0/2] pci: Provide a flag to access VPD through function 0 References: <20150603184445.109080.36387.stgit@mdrustad-wks.jf.intel.com> <051B68B4-3E77-4EB0-B9FE-8523631884A2@intel.com> <45099CC7-DDAB-41D9-AB74-5A81E2AAF64C@intel.com> In-Reply-To: <45099CC7-DDAB-41D9-AB74-5A81E2AAF64C@intel.com> Content-Type: text/plain; charset=windows-1252; format=flowed Sender: linux-pci-owner@vger.kernel.org List-ID: On 06/17/2015 09:29 AM, Rustad, Mark D wrote: > + Alex > >> On Jun 5, 2015, at 2:59 PM, Rustad, Mark D wrote: >> >>> On Jun 3, 2015, at 11:46 AM, Mark D Rustad wrote: >>> >>> Many multi-function devices provide shared registers in extended >>> config space for accessing VPD. The behavior of these registers >>> means that the state must be tracked and access locked correctly >>> for accesses not to hang or worse. One way to meet these needs is >>> to always perform the accesses through function 0, thereby using >>> the state tracking and mutex that already exists. >>> >>> To provide this behavior, add a dev_flags bit to indicate that this >>> should be done. This bit can then be set for any non-zero function >>> that needs to redirect such VPD access to function 0. Do not set >>> this bit on the zero function or there will be an infinite recursion. >>> >>> The second patch uses this new flag to invoke this behavior on all >>> multi-function Intel Ethernet devices. >>> >>> Signed-off-by: Mark Rustad >>> >>> --- >>> Changes in V2: >>> - Corrected a spelling error in a log message >>> - Added checks to see that the referenced function 0 is reasonable >>> Changes in V3: >>> - Don't leak a device reference >>> - Check that function 0 has VPD >>> - Make a helper for the function 0 checks >>> - Moved a multifunction check to the quirk patch >> So does this series look acceptable now? I think I addressed the issues that Alex raised. Can these also be considered for -stable? > More than a week has passed without any comment. Is this going to be accepted or is there still an issue? Yeah, this looks like it has addressed most of the corner cases so I am good with it. Acked-by: Alexander Duyck