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! Paolo