On Tue, 2015-03-17 at 00:10 +0000, Slutz, Donald Christopher wrote: > @@ -715,7 +715,7 @@ static int hvm_ioreq_server_map_pages(struct > hvm_ioreq_server *s, > bool_t is_default, bool_t > handle_bufioreq) > { > struct domain *d = s->domain; > - unsigned long ioreq_pfn, bufioreq_pfn; > + unsigned long ioreq_pfn, bufioreq_pfn = 0; > int rc; > > if ( is_default ) > > > Which "fixed" it for me. It would be good for you to try this.
Are you going to send this as a proper patch? I think usually we would add a brief comment to explain the unnecessary seeming initialiser. e..g in radix-tree.c "/* uninitialised var warning */" or "/* gcc ... */" in various other places. _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel