v6 changelog for mmc ioctl patch

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



The changes in this version are mostly to address Arnd's comments on v4.

Arnd, I added a flag to allow the caller to select between normal CMDs
and ACMDs.  This really makes it a generic passthrough.  However, I only
tested a few commands to see that passthrough was working.  I did not
try any advanced handshaking CMDs.  Hopefully this generic passthrough
is more useful to you than previous incarnations.

I chose not to create higher-level ioctl operations specifically for the
security functions because they don't really buy us anything.  At most,
I could make a function for each security opcode that would specify the
numeric opcode directly (instead of having it come from userspace).  On
the back end, all of these functions would have the same functionality
of packing the opcode and arguments into the appropriate structures,
then issuing the request - essentially the same behaviour as my current
mmc_blk_ioctl_cmd().  The "higher-level" then just becomes an opcode
filter, and that was already deemed to be undesirable by Andrei and
Michał in the v2 thread.  As I explained in the past, I can't implement
any more of the security functionality in the kernel than that.

Anyway, here's the full patch history:

- v6
    - refix 32+64 compat pointer for better portability
    - copy userspace pointer *before* using
    - apply upper limit to data buffer size
    - add flag to allow normal CMD opcodes as well as ACMD opcodes
    - remove unnecessary mutex grab

- v5
    - fix 32-bit compiler warning about the 32+64 compat pointer

- v4
    - replace postsleep udelay() with usleep_range()
    - add cmd_timeout_ms field for R1B commands

- v3
    - copy data from userspace before claiming host
    - break out copy from userspace into its own function
    - verify that caller has CAP_SYS_RAWIO
    - rename ``struct sd_ioc_cmd`` to ``struct mmc_ioc_cmd`` because it
      applies generally, not just to SD
    - make struct mmc_ioc_cmd the same between 32-bit and 64-bit to
      simplify compat_ioctl()
    - export include/linux/mmc/ioctl.h when you ``make headers_install``

- v2
    - make initialization of struct declarations match kernel style
    - only allow ioctl() on whole block device, not partition
    - remove extraneous printks
    - implement compat_ioctl()
    - remove version field from ``struct sd_ioc_cmd``

John

[Index of Archives]     [Linux USB Devel]     [Linux Media]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux