Am 04.07.2017 um 10:40 hat Manos Pitsidianakis geschrieben: > Now that passing the call to bs->file is the default for some bdrv_* > callbacks, remove the duplicate implementations in block/raw-format.c > > Reviewed-by: Eric Blake <ebl...@redhat.com> > Signed-off-by: Manos Pitsidianakis <el13...@mail.ntua.gr>
If you change patch 1 as suggested there, most of these functions will have to stay. The exception is .bdrv_has_zero_init, for which I suggested an .is_filter check. However, raw isn't declared a filter currently. If we want to declare it a filter, the definition I proposed in the other mail doesn't work because raw can apply an offset - this why it needs its own function for probing geometry, so with that changed definition checking .is_filter might not be enough any more for all of the functions in patch 1. Maybe it turns out in the end that raw just is special enough that having its own functions is okay. Kevin