Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755215AbdL1ULE (ORCPT ); Thu, 28 Dec 2017 15:11:04 -0500 Received: from mail.kernel.org ([198.145.29.99]:43946 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755068AbdL1UK4 (ORCPT ); Thu, 28 Dec 2017 15:10:56 -0500 DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 0A944218B1 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=kernel.org Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=jaegeuk@kernel.org Date: Thu, 28 Dec 2017 12:10:55 -0800 From: Jaegeuk Kim To: Greg Kroah-Hartman Cc: Avri Altman , "linux-kernel@vger.kernel.org" , "linux-scsi@vger.kernel.org" , Jaegeuk Kim , Alex Lemberg , Stanislav Nijnikov Subject: Re: [PATCH 2/2 v4] scsi: ufs: introduce sysfs entries exposing UFS health info Message-ID: <20171228201055.GA49748@jaegeuk-macbookpro.roam.corp.google.com> References: <20171220191631.50329-1-jaegeuk@kernel.org> <20171220191631.50329-2-jaegeuk@kernel.org> <20171220221325.GA56741@jaegeuk-macbookpro.roam.corp.google.com> <20171221075935.GB9645@kroah.com> <20171227131052.GA13384@kroah.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20171227131052.GA13384@kroah.com> User-Agent: Mutt/1.8.2 (2017-04-18) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2169 Lines: 59 On 12/27, Greg Kroah-Hartman wrote: > On Wed, Dec 27, 2017 at 09:00:10AM +0000, Avri Altman wrote: > > > > > > > -----Original Message----- > > > From: linux-scsi-owner@vger.kernel.org [mailto:linux-scsi- > > > owner@vger.kernel.org] On Behalf Of Greg Kroah-Hartman > > > Sent: Thursday, December 21, 2017 10:00 AM > > > To: Jaegeuk Kim > > > Cc: linux-kernel@vger.kernel.org; linux-scsi@vger.kernel.org; Jaegeuk Kim > > > > > > Subject: Re: [PATCH 2/2 v4] scsi: ufs: introduce sysfs entries exposing UFS > > > health info > > > > > > On Wed, Dec 20, 2017 at 02:13:25PM -0800, Jaegeuk Kim wrote: > > > > This patch adds a new sysfs group, namely health, via: > > > > > > > > /sys/devices/soc/X.ufshc/health/ > > As device health is just one piece of information out of the device management, > > I think that you should address this in a more comprehensive way, > > And set hooks for much more device info: > > Allow access to device descriptors, attributes and flags. > > Add on patches are easy to create for this if people really want and > need it :) > > > The attributes and flags should be placed in separate subfolders > > Why? What is that going to help with? > > > The LUN specific descriptors and attributes should be placed in a luns > > subfolder, and then per descriptor / attribute type > > Again, why? > > > You might also would like to consider differentiating read and write - > > to control those type of accesses as well. > > What do you mean by this exactly? > > As it is, this is a step forward in getting attributes that people are > asking for and already using, into the kernel tree. Please don't object > because not all attributes that are possible are being added here, it > should be trivial to add more as needed, right? > > I'm really tired of seeing all of the various out-of-tree forks of this > driver, it's about time that someone works to get those features merged, > right? Indeed, it seems someone has already done similar work before. Let me drop my patches and dive into that, although I have to clean up all the mess once again. :( Thanks, > > thanks, > > greg k-h