Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752460Ab3GFRrH (ORCPT ); Sat, 6 Jul 2013 13:47:07 -0400 Received: from mail.linuxfoundation.org ([140.211.169.12]:33746 "EHLO mail.linuxfoundation.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751851Ab3GFRrE (ORCPT ); Sat, 6 Jul 2013 13:47:04 -0400 Date: Sat, 6 Jul 2013 10:47:58 -0700 From: Greg Kroah-Hartman To: Guenter Roeck Cc: Jean Delvare , lm-sensors@lm-sensors.org, linux-kernel@vger.kernel.org Subject: Re: [RFC PATCH 1/5] driver core: Introduce device_create_groups Message-ID: <20130706174758.GA8122@kroah.com> References: <1373131495-13465-1-git-send-email-linux@roeck-us.net> <1373131495-13465-2-git-send-email-linux@roeck-us.net> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1373131495-13465-2-git-send-email-linux@roeck-us.net> 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 Content-Length: 1331 Lines: 34 On Sat, Jul 06, 2013 at 10:24:51AM -0700, Guenter Roeck wrote: > device_create_groups lets callers create devices as well as associated > sysfs attributes with a single call. This avoids race conditions seen > if sysfs attributes on new devices are created later. > > Signed-off-by: Guenter Roeck > --- > drivers/base/core.c | 60 +++++++++++++++++++++++++++++++++++++++++++++--- > include/linux/device.h | 4 ++++ > 2 files changed, 61 insertions(+), 3 deletions(-) Ok, in reading the code, it makes more sense to me now. That looks good, but the name is a bit rough. How many device_create() calls do we have in the kernel today? $ git grep -w device_create | wc -l 127 That's really not that many, I wonder how many of those should be using something like this as well? A quick look showed that the sound core should be using this, but maybe not all that many others... My objection to the name is at first glance, it sounds like you are creating "groups", not a device + groups. "device_create_with_groups()"? Ick, naming is hard... greg k-h -- 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/