diff options
author | Bartlomiej Zolnierkiewicz <bzolnier@gmail.com> | 2007-03-26 23:03:19 +0200 |
---|---|---|
committer | Bartlomiej Zolnierkiewicz <bzolnier@gmail.com> | 2007-03-26 23:03:19 +0200 |
commit | 8799620400b0b1a4729d8be828b5bfb3d2a8db1a (patch) | |
tree | 4c5f3bb3360dddcc9638c4e92ddb7f1f2e0b1578 /drivers/ide/Kconfig | |
parent | f68d9320cd06fdec19735143b42e5197b63165b4 (diff) | |
download | lwn-8799620400b0b1a4729d8be828b5bfb3d2a8db1a.tar.gz lwn-8799620400b0b1a4729d8be828b5bfb3d2a8db1a.zip |
ide: fix locking for manual DMA enable/disable ("hdparm -d")
Since hwif->ide_dma_check and hwif->ide_dma_on never queue any commands
(ide_config_drive_speed() sets transfer mode using polling and has no error
recovery) we are safe with setting hwgroup->busy for the time while DMA
setting for a drive is changed (so it won't race against I/O commands in fly).
I audited briefly all ->ide_dma_check/->ide_dma_on/->tuneproc/->speedproc
implementations and they all look OK wrt to this change.
This patch finally allowed me to close kernel bugzilla bug #8169
(once again thanks to Patrick Horn for reporting the issue & testing patches).
Cc: Sergei Shtylyov <sshtylyov@ru.mvista.com>
Cc: Alan Cox <alan@lxorguk.ukuu.org.uk>
Signed-off-by: Bartlomiej Zolnierkiewicz <bzolnier@gmail.com>
Diffstat (limited to 'drivers/ide/Kconfig')
0 files changed, 0 insertions, 0 deletions