Re: [PATCH v7] scsi: ufs: add ioctl interface for query request

From	Greg KH <gregkh@linuxfoundation.org>
Newsgroups	linux.kernel
Subject	Re: [PATCH v7] scsi: ufs: add ioctl interface for query request
Date	2016-03-10 17:30 +0100
Message-ID	<rbbDk-1c2-15@gated-at.bofh.it> (permalink)
References	(2 earlier) <raROh-3VS-1@gated-at.bofh.it> <raSKm-4zp-9@gated-at.bofh.it> <raTn5-4Os-31@gated-at.bofh.it> <raV5w-6jx-5@gated-at.bofh.it> <rbbai-G2-19@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

On Thu, Mar 10, 2016 at 03:52:54PM -0000, ygardi@codeaurora.org wrote:
> > On Wed, Mar 09, 2016 at 08:52:59PM -0000, ygardi@codeaurora.org wrote:
> >> > 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 <subhashj@codeaurora.org>
> >> >> >> Signed-off-by: Dolev Raviv <draviv@codeaurora.org>
> >> >> >> Signed-off-by: Gilad Broner <gbroner@codeaurora.org>
> >> >> >> Signed-off-by: Yaniv Gardi <ygardi@codeaurora.org>
> >> >> >
> >> >> > 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 ?
> >
> > Why aren't those characteristics just exported as sysfs attributes under
> > control by the UFS controller driver?  Why do you need/want an ioctl for
> > this?
> >
> 
> Hi greg k-h,
> 
> in our code, we used the IOCTL during runtime, in order to determine some
> information about the RPMB well known lun.
> with the rpmb lun ID we could then go to /dev/sgX and issue
> UFS_IOCTL_QUERY to this lun and get the data -
> reading the QUERY_DESC_IDN_GEOMETRY descriptor and reading the
> QUERY_DESC_IDN_UNIT descriptor.
> 
> this was crucial to the work we do in RPMB.

What is RPMB?

And again, why not just use sysfs attributes on your host controller
device?  Why does this have to be a custom ioctl?

greg k-h

Thread

[PATCH v7] scsi: ufs: add ioctl interface for query request Yaniv Gardi <ygardi@codeaurora.org> - 2016-03-09 15:20 +0100
  Re: [PATCH v7] scsi: ufs: add ioctl interface for query request Greg KH <gregkh@linuxfoundation.org> - 2016-03-09 17:30 +0100
    Re: [PATCH v7] scsi: ufs: add ioctl interface for query request ygardi@codeaurora.org - 2016-03-09 20:20 +0100
      Re: [PATCH v7] scsi: ufs: add ioctl interface for query request Greg KH <gregkh@linuxfoundation.org> - 2016-03-09 21:20 +0100
        Re: [PATCH v7] scsi: ufs: add ioctl interface for query request ygardi@codeaurora.org - 2016-03-09 22:00 +0100
          Re: [PATCH v7] scsi: ufs: add ioctl interface for query request Greg KH <gregkh@linuxfoundation.org> - 2016-03-09 23:50 +0100
            Re: [PATCH v7] scsi: ufs: add ioctl interface for query request ygardi@codeaurora.org - 2016-03-10 17:00 +0100
              Re: [PATCH v7] scsi: ufs: add ioctl interface for query request Greg KH <gregkh@linuxfoundation.org> - 2016-03-10 17:30 +0100
                Re: [PATCH v7] scsi: ufs: add ioctl interface for query request Greg KH <gregkh@linuxfoundation.org> - 2016-03-10 17:40 +0100
                Re: [PATCH v7] scsi: ufs: add ioctl interface for query request ygardi@codeaurora.org - 2016-03-10 19:50 +0100
                Re: [PATCH v7] scsi: ufs: add ioctl interface for query request Greg KH <gregkh@linuxfoundation.org> - 2016-03-10 20:00 +0100
                Re: [PATCH v7] scsi: ufs: add ioctl interface for query request ygardi@codeaurora.org - 2016-03-10 17:40 +0100
    Re: [PATCH v7] scsi: ufs: add ioctl interface for query request ygardi@codeaurora.org - 2016-03-09 20:20 +0100
      Re: [PATCH v7] scsi: ufs: add ioctl interface for query request Arnd Bergmann <arnd@arndb.de> - 2016-03-10 18:20 +0100
        Re: [PATCH v7] scsi: ufs: add ioctl interface for query request Arnd Bergmann <arnd@arndb.de> - 2016-03-10 20:30 +0100
          Re: [PATCH v7] scsi: ufs: add ioctl interface for query request "Martin K. Petersen" <martin.petersen@oracle.com> - 2016-03-11 02:50 +0100
            Re: [PATCH v7] scsi: ufs: add ioctl interface for query request Hannes Reinecke <hare@suse.de> - 2016-03-11 09:50 +0100
              RE: [PATCH v7] scsi: ufs: add ioctl interface for query request "Winkler, Tomas" <tomas.winkler@intel.com> - 2016-03-13 13:50 +0100

csiph-web