On 05/06/2015 01:52 AM, Pavel Fedin wrote: [meta-comment] > set_host_notifier() is introduced into virtio-mmio now. Most of codes came > from virtio-pci. > Missing an 'In-Reply-To: <000501d087d0$39c1bd90$ad4538b0$@samsung.com>' header, which makes your series unthreaded (every message appeared as its own thread). It makes review easier if everything is properly threaded to the cover letter, so you may want to send a series to yourself first as a test case of your setup before sending to the list. Other submission hints at: http://wiki.qemu.org/Contribute/SubmitAPatch > > +static int virtio_mmio_set_host_notifier_internal(VirtIOMMIOProxy *proxy, > + int n, bool assign, bool > set_handler) Long line, you might want to wrap it to fit in 80 columns. -- Eric Blake eblake redhat com +1-919-301-3266 Libvirt virtualization library http://libvirt.org