Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752372Ab2FEU0D (ORCPT ); Tue, 5 Jun 2012 16:26:03 -0400 Received: from mail-pb0-f46.google.com ([209.85.160.46]:56554 "EHLO mail-pb0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750764Ab2FEU0A (ORCPT ); Tue, 5 Jun 2012 16:26:00 -0400 Date: Wed, 6 Jun 2012 05:25:55 +0900 From: Greg KH To: Asai Thambi S P Cc: Jens Axboe , "linux-kernel@vger.kernel.org" , Sam Bradshaw Subject: Re: [PATCH 1/2] mtip32xx: Remove 'registers' and 'flags' from sysfs Message-ID: <20120605202555.GA4727@kroah.com> References: <4FCD0FC7.9010105@micron.com> <20120605093314.GC21560@kroah.com> <4FCE4D67.1030309@micron.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <4FCE4D67.1030309@micron.com> 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: 1529 Lines: 42 On Tue, Jun 05, 2012 at 11:18:15AM -0700, Asai Thambi S P wrote: > On 6/5/2012 2:33 AM, Greg KH wrote: > > > On Mon, Jun 04, 2012 at 12:43:03PM -0700, Asai Thambi S P wrote: > >> > >> This patch removes entries 'registers' and 'flags' from sysfs. Updated ABI file > >> to reflect this change. > >> > >> Reported-by: Greg Kroah-Hartman > >> Signed-off-by: Asai Thambi S P > > > > Much nicer, thanks for doing this: > > > > Acked-by: Greg Kroah-Hartman > > > > But, one question on a different sysfs file: > > > >> What: /sys/block/rssd*/status > >> Date: April 2012 > >> KernelVersion: 3.4 > >> Contact: Asai Thambi S P > >> Description: This is a read-only file. Indicates the status of the device. > > > > What "status" is this showing? Why is this a sysfs file? Who > > needs/wants it? > > > This shows the device status - online, write_protect or thermal_shutdown. This > would be used by management application. Is it used by a management application? Shouldn't such a tool use the "standard" block device status files instead? I thought we exported that information already in the /sys/block/* files. thanks, 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/