WARNING - OLD ARCHIVES

This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-devel

Re: [XenPPC] [PATCH][POWERPC] Fix missing '{' for mm.c in xen-unstable

To: jyoung5@xxxxxxxxxx
Subject: Re: [XenPPC] [PATCH][POWERPC] Fix missing '{' for mm.c in xen-unstable
From: Hollis Blanchard <hollisb@xxxxxxxxxx>
Date: Fri, 06 Jul 2007 10:33:34 -0500
Cc: xen-devel <xen-devel@xxxxxxxxxxxxxxxxxxx>, xen-ppc-devel <xen-ppc-devel@xxxxxxxxxxxxxxxxxxx>
Delivery-date: Fri, 06 Jul 2007 08:31:29 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxx
In-reply-to: <1183735483.6042.4.camel@laptop>
List-help: <mailto:xen-ppc-devel-request@lists.xensource.com?subject=help>
List-id: Xen PPC development <xen-ppc-devel.lists.xensource.com>
List-post: <mailto:xen-ppc-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-ppc-devel>, <mailto:xen-ppc-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-ppc-devel>, <mailto:xen-ppc-devel-request@lists.xensource.com?subject=unsubscribe>
Organization: IBM Linux Technology Center
References: <1183735483.6042.4.camel@laptop>
Reply-to: Hollis Blanchard <hollisb@xxxxxxxxxx>
Sender: xen-ppc-devel-bounces@xxxxxxxxxxxxxxxxxxx
On Fri, 2007-07-06 at 10:24 -0500, Jerone Young wrote:
> A '{' was missing from conditional if statement.
> 
> Signed-off-by: Jerone Young <jyoung5@xxxxxxxxxx>
> 
> diff -r d54d47fc8c6c xen/arch/powerpc/mm.c
> --- a/xen/arch/powerpc/mm.c     Thu Jul 05 10:19:25 2007 +0100
> +++ b/xen/arch/powerpc/mm.c     Fri Jul 06 08:48:23 2007 -0500
> @@ -187,7 +187,7 @@ int replace_grant_host_mapping(
>      unsigned long addr, unsigned long frame, unsigned long new_addr,
>      unsigned int flags)
>  {
> -    if (new_addr)
> +    if (new_addr) {
>          printk("%s: new_addr not supported\n", __func__);
>          BUG();
>          return GNTST_general_error;

Doh, thanks. I have had this locally, but apparently forgot to push
it...

I will apply to the PPC tree, along with a few other changes, and push
upstream later.

-- 
Hollis Blanchard
IBM Linux Technology Center


_______________________________________________
Xen-ppc-devel mailing list
Xen-ppc-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-ppc-devel

<Prev in Thread] Current Thread [Next in Thread>