Jerone Young
2007-Jul-06 15:24 UTC
[XenPPC] [PATCH][POWERPC] Fix missing ''{'' for mm.c in xen-unstable
A ''{'' was missing from conditional if statement.
Signed-off-by: Jerone Young <jyoung5@us.ibm.com>
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;
_______________________________________________
Xen-ppc-devel mailing list
Xen-ppc-devel@lists.xensource.com
http://lists.xensource.com/xen-ppc-devel
Hollis Blanchard
2007-Jul-06 15:33 UTC
Re: [XenPPC] [PATCH][POWERPC] Fix missing ''{'' for mm.c in xen-unstable
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@us.ibm.com> > > 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@lists.xensource.com http://lists.xensource.com/xen-ppc-devel
Keir Fraser
2007-Jul-06 15:35 UTC
Re: [Xen-devel] Re: [XenPPC] [PATCH][POWERPC] Fix missing ''{'' for mm.c in xen-unstable
On 6/7/07 16:33, "Hollis Blanchard" <hollisb@us.ibm.com> wrote:> 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.Ah, I just threw it into the xen-unstable main tree. -- Keir _______________________________________________ Xen-devel mailing list Xen-devel@lists.xensource.com http://lists.xensource.com/xen-devel