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

Reply via email to