2011/1/19 Kevin Wolf <kwolf@xxxxxxxxxx>: > Am 19.01.2011 06:44, schrieb Yoshiaki Tamura: >> event-tap function is called only when it is on, and requests sent >> from device emulators. >> >> Signed-off-by: Yoshiaki Tamura <tamura.yoshiaki@xxxxxxxxxxxxx> >> --- >> block.c | 11 +++++++++++ >> 1 files changed, 11 insertions(+), 0 deletions(-) >> >> diff --git a/block.c b/block.c >> index ff2795b..85bd8b8 100644 >> --- a/block.c >> +++ b/block.c >> @@ -28,6 +28,7 @@ >> #include "block_int.h" >> #include "module.h" >> #include "qemu-objects.h" >> +#include "event-tap.h" >> >> #ifdef CONFIG_BSD >> #include <sys/types.h> >> @@ -2111,6 +2112,11 @@ BlockDriverAIOCB *bdrv_aio_writev(BlockDriverState *bs, int64_t sector_num, >> if (bdrv_check_request(bs, sector_num, nb_sectors)) >> return NULL; >> >> + if (bs->device_name && event_tap_is_on()) { > > bs->device_name is a pointer to a char array contained in bs, so it's > never NULL. You probably mean *bs->device_name? Yes, thanks for pointing out. It didn't expose because event_tap_is_on() was false upon flushing after synchronization. Yoshi > > Kevin > -- > To unsubscribe from this list: send the line "unsubscribe kvm" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html > -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html