From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752167AbdHRPoa (ORCPT ); Fri, 18 Aug 2017 11:44:30 -0400 Received: from mx1.redhat.com ([209.132.183.28]:53006 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751814AbdHRPo2 (ORCPT ); Fri, 18 Aug 2017 11:44:28 -0400 DMARC-Filter: OpenDMARC Filter v1.3.2 mx1.redhat.com F28AD13A88 Authentication-Results: ext-mx05.extmail.prod.ext.phx2.redhat.com; dmarc=none (p=none dis=none) header.from=redhat.com Authentication-Results: ext-mx05.extmail.prod.ext.phx2.redhat.com; spf=fail smtp.mailfrom=dledford@redhat.com Message-ID: <1503071066.2598.15.camel@redhat.com> Subject: Re: PCI/IB: add support for pci driver attribute groups From: Doug Ledford To: Bjorn Helgaas , Greg KH Cc: Faisal Latif , Sean Hefty , Hal Rosenstock , Bjorn Helgaas , linux-rdma@vger.kernel.org, linux-pci@vger.kernel.org, linux-kernel@vger.kernel.org Date: Fri, 18 Aug 2017 11:44:26 -0400 In-Reply-To: <20170801214046.GB26498@bhelgaas-glaptop.roam.corp.google.com> References: <20170719130106.GE17776@kroah.com> <20170801214046.GB26498@bhelgaas-glaptop.roam.corp.google.com> Organization: Red Hat, Inc. Content-Type: text/plain; charset="UTF-8" Mime-Version: 1.0 Content-Transfer-Encoding: 7bit X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.29]); Fri, 18 Aug 2017 15:44:28 +0000 (UTC) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, 2017-08-01 at 16:40 -0500, Bjorn Helgaas wrote: > On Wed, Jul 19, 2017 at 03:01:06PM +0200, Greg KH wrote: > > From: Greg Kroah-Hartman > > > > Some drivers (specifically the nes IB driver), want to create a lot > > of > > sysfs driver attributes. Instead of open-coding the creation and > > removal of these files (and getting it wrong btw), it's a better > > idea to > > let the driver core handle all of this logic for us. > > > > So add a new field to the pci driver structure, **groups, that > > allows > > pci drivers to specify an attribute group list it wishes to have > > created > > when it is registered with the driver core. > > > > Big bonus is now the driver doesn't race with userspace when the > > sysfs > > files are created vs. when the kobject is announced, so any > > script/tool > > that actually wanted to use these files will not have to poll > > waiting > > for them to show up. > > > > Cc: Faisal Latif > > Cc: Doug Ledford > > Cc: Sean Hefty > > Cc: Hal Rosenstock > > Cc: Bjorn Helgaas > > Signed-off-by: Greg Kroah-Hartman > > Acked-by: Bjorn Helgaas > > If you want me to take this let me know; otherwise I'll assume Doug > will > merge it. I've picked it up for for-next. Thanks. -- Doug Ledford GPG KeyID: B826A3330E572FDD Key fingerprint = AE6B 1BDA 122B 23B4 265B 1274 B826 A333 0E57 2FDD