From mboxrd@z Thu Jan  1 00:00:00 1970
From: Ian Campbell <ian.campbell@citrix.com>
Subject: Re:
 =?utf-8?q?Compliling_Xen_4=2E5=2E0_Fails_with_error?=
 =?utf-8?q?=3A_=E2=80=98bufioreq=5Fpfn=E2=80=99_may_be_used_uninitialised_?=
 =?utf-8?q?in_this_function__=5B-Werror=3Duninitialized=5D?=
Date: Tue, 17 Mar 2015 10:37:44 +0000
Message-ID: <1426588664.18247.172.camel@citrix.com>
References: <9AAE0902D5BC7E449B7C8E4E778ABCD0258434BF@AMSPEX01CL01.citrite.net>
	<887729013.1172497.1426540794859.JavaMail.yahoo@mail.yahoo.com>
	<A3CD31A5D207064088A18AC2AF7B5DC6C27A1E9F@MIA20725MBX891B.apps.tmrk.corp>
Mime-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Return-path: <xen-devel-bounces@lists.xen.org>
In-Reply-To: <A3CD31A5D207064088A18AC2AF7B5DC6C27A1E9F@MIA20725MBX891B.apps.tmrk.corp>
List-Unsubscribe: <http://lists.xen.org/cgi-bin/mailman/options/xen-devel>,
	<mailto:xen-devel-request@lists.xen.org?subject=unsubscribe>
List-Post: <mailto:xen-devel@lists.xen.org>
List-Help: <mailto:xen-devel-request@lists.xen.org?subject=help>
List-Subscribe: <http://lists.xen.org/cgi-bin/mailman/listinfo/xen-devel>,
	<mailto:xen-devel-request@lists.xen.org?subject=subscribe>
Sender: xen-devel-bounces@lists.xen.org
Errors-To: xen-devel-bounces@lists.xen.org
To: "Slutz, Donald Christopher" <dslutz@verizon.com>
Cc: Ian Murray <murrayie@yahoo.co.uk>, Paul Durrant <Paul.Durrant@citrix.com>, "xen-devel@lists.xensource.com" <xen-devel@lists.xensource.com>, Jan Beulich <JBeulich@suse.com>
List-Id: xen-devel@lists.xenproject.org

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.