[PATCH 0/11] Consolidate polling for busy

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

 



Hi Ulf,

This patchset consolidates polling for busy by using mmc_poll_for_busy()
to avoid re-invent the wheel for other code block. The only leftover is
mmc_test which not just poll for busy but also check if the host drivers
abuse MMC_CAP_WAIT_WHILE_BUSY.

Note that the final one replacing ioctl_rpmb_card_status_poll() with
mmc_poll_for_busy() makes it from relax wait to busy wait, which could
probably cause performance regression depending on workload. So the final
patch would be sane to be merged after Martin's work[1], if all looks good
to you.

[1]: https://patchwork.kernel.org/patch/10425191/



Shawn Lin (11):
  mmc: core: Move the sanity check of busy detection into
    mmc_poll_for_busy()
  mmc: core: Allow assigning retry for sending status via
    mmc_poll_for_busy()
  mmc: core: Rename and reorganize mmc_switch_status_error()
  mmc: core: Adjust the checking for device status in
    mmc_send_status_error()
  mmc: core: Adjust the checking for device status in mmc_do_erase()
  mmc: core: Adjust the macro of R1_STATUS(x)
  mmc: core: Consider R1_READY_FOR_DATA as a busy indication in
    mmc_poll_for_busy()
  mmc: core: Export mmc_poll_for_busy()
  mmc: core: Distinguish the caller of mmc_poll_for_busy() by new
    paramter
  mmc: core: Use mmc_poll_for_busy() for polling busy in mmc_do_erase()
  mmc: core: Use mmc_poll_for_busy() for polling busy in
    __mmc_blk_ioctl_cmd()

 drivers/mmc/core/block.c   | 41 ++++-----------------------------
 drivers/mmc/core/core.c    | 40 ++-------------------------------
 drivers/mmc/core/mmc_ops.c | 56 ++++++++++++++++++++++++++++++----------------
 drivers/mmc/core/mmc_ops.h |  3 +++
 include/linux/mmc/mmc.h    |  2 +-
 5 files changed, 47 insertions(+), 95 deletions(-)

-- 
1.9.1


--
To unsubscribe from this list: send the line "unsubscribe linux-mmc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



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

  Powered by Linux