On 04.07.2016 17:53, Paolo Bonzini wrote:
On 04/07/2016 16:38, Denis V. Lunev wrote:
Changes from v4:
- Moved to write generation scheme instead of dirty flag
- Added retry setup to IDE PIO and FLUSH requests
Changes from v3:
- Fixed a typo in commit message
- Rebased on Kevin'n origin/block
Changes from v2:
- Better comments
- Rebased on latest master
Changes from v1:
- Flush requests that should be skipped will now wait for completion
of any previous requests already in flight
- Fixed IDE and AHCI tests to dirty media for new flush behaviour
- Fixed a problem in IDE CMD_FLUSH_CACHE failure handling
Signed-off-by: Evgeny Yakovlev <eyakov...@virtuozzo.com>
Signed-off-by: Denis V. Lunev <d...@openvz.org>
CC: Kevin Wolf <kw...@redhat.com>
CC: Max Reitz <mre...@redhat.com>
CC: Stefan Hajnoczi <stefa...@redhat.com>
CC: Fam Zheng <f...@redhat.com>
CC: John Snow <js...@redhat.com>
Evgeny Yakovlev (4):
ide: refactor retry_unit set and clear into separate function
ide: set retry_unit for PIO and FLUSH requests
tests: in IDE and AHCI tests perform DMA write before flushing
block: ignore flush requests when storage is clean
block.c | 3 +++
block/io.c | 18 ++++++++++++++++++
hw/ide/core.c | 24 ++++++++++++++++++------
include/block/block_int.h | 5 +++++
tests/ahci-test.c | 34 ++++++++++++++++++++++++++++++++--
tests/ide-test.c | 43 +++++++++++++++++++++++++++++++++++++++++++
6 files changed, 119 insertions(+), 8 deletions(-)
Reviewed-by: Paolo Bonzini <pbonz...@redhat.com>
Thanks!
You're welcome!
Paolo
