Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1355237

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

Show all headers | View raw

> 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.

thanks,
Yaniv

>> 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.
>
> Do other storage busses have these types of "custom" ioctls for their
> bus-type alone?  For simple attributes like this, shouldn't you be using
> sysfs instead so that it is much easier for userspace tools to get
> access to them?
>
> thanks,
>
> greg k-h
>

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

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