Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S934119AbcCIUxK (ORCPT ); Wed, 9 Mar 2016 15:53:10 -0500 Received: from smtp.codeaurora.org ([198.145.29.96]:57759 "EHLO smtp.codeaurora.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751999AbcCIUxB (ORCPT ); Wed, 9 Mar 2016 15:53:01 -0500 Message-ID: <551d9d1ae5418dd2dfa91aa6db69deb9.squirrel@us.codeaurora.org> In-Reply-To: <20160309201808.GA24568@kroah.com> References: <1457532699-11487-1-git-send-email-ygardi@codeaurora.org> <20160309162943.GC21181@kroah.com> <43117fbc4f3a305acd0f4975a633934a.squirrel@us.codeaurora.org> <20160309201808.GA24568@kroah.com> Date: Wed, 9 Mar 2016 20:52:59 -0000 Subject: Re: [PATCH v7] scsi: ufs: add ioctl interface for query request From: ygardi@codeaurora.org To: "Greg KH" Cc: ygardi@codeaurora.org, james.bottomley@hansenpartnership.com, linux-kernel@vger.kernel.org, linux-scsi@vger.kernel.org, linux-arm-msm@vger.kernel.org, santoshsy@gmail.com, linux-scsi-owner@vger.kernel.org, "Dolev Raviv" , "Gilad Broner" , "Vinayak Holikatti" , "James E.J. Bottomley" , "Martin K. Petersen" , "Michael Neuling" , "Matthew R. Ochs" , "Wen Xiong" , "Subhash Jadavani" , "open list:ABI/API" User-Agent: SquirrelMail/1.4.22-4.el6 MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7BIT X-Priority: 3 (Normal) Importance: Normal Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1874 Lines: 49 > On Wed, Mar 09, 2016 at 07:09:49PM -0000, ygardi@codeaurora.org wrote: >> > On Wed, Mar 09, 2016 at 04:11:33PM +0200, Yaniv Gardi wrote: >> >> This patch exposes the ioctl interface for UFS driver via SCSI device >> >> ioctl interface. As of now UFS driver would provide the ioctl for >> query >> >> interface to connected UFS device. >> >> >> >> Reviewed-by: Subhash Jadavani >> >> Signed-off-by: Dolev Raviv >> >> Signed-off-by: Gilad Broner >> >> Signed-off-by: Yaniv Gardi >> > >> > What tool is going to use this ioctl? Why does userspcae want to do >> > something "special" with UFS devices? Shouldn't they just be treated >> > like any other normal block device? >> > >> >> Any userspace application can be a tool. >> We already implemented and used a user space application, that sent >> queries to the UFS devices in order to get information and descriptors. > > But do you want to do with that information? Why does userspace care? > i don't really understand the subtext of your question - as ANY ioctl cb, we decided to implement the ioctl callback of this scsi device in order to get information like UNIT DESC, DEVICE DESC, FLAGs, ATTRIBUTES. When dealing with UFS devices, one should be able to read the characteristics of the device. why ? well, why not ? during development of this driver, it was useful in many cases to be able to communicate with the device, by simple IOCTL command, rather than implementing ad-hock. regards, Yaniv >> Not only ioctl interface is a useful way to interact with the device, >> we used it, and found it very helpful in varies cases. > > In what case was it helpful? Why does userspace care about ufs > specifics, it should just treat it like any other block device and not > care at all. > > thanks, > > greg k-h >