On Wed, Aug 30, 2017 at 03:26:59AM -0500, Sergio Andres Gomez Del Real wrote: > +static void hvf_log_start(MemoryListener *listener, > + MemoryRegionSection *section, int old, int new) > +{ > + if (old != 0) > + return;
QEMU coding style uses curly braces even when the if statement body only has 1 line. > + > + hvf_set_dirty_tracking(section, 1); > +} > + > +static void hvf_log_stop(MemoryListener *listener, > + MemoryRegionSection *section, int old, int new) > +{ > + if (new != 0) > + return; Same here.