On 03/17/15 06:37, Ian Campbell wrote: > 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. > >
I do not yet have this as a separate patch. Will do so. -Don Slutz _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel